feat: add json output mode for check command

As #47 said, I would like to to have json output when runing npx nolyfill check, which is useful for programmatically make a pull request comment or any other things. By runing cli instead of using nolyfill package as a dependency, We can always execute the latest version cli with npx.

Could you make a release on npm? @SukkaW

Could you make a release on npm? @SukkaW

I have a few packages to be added to the nolyfill. I will release a newer version after those packages are added.


Fix detection of "navigable" URLs involving hashes.

Fixes detection of a "navigable" URL when transitioning between a hash
URL and a standard URL (e.g. /page#target to /page).  Before, a request
would be triggered for this case, and after this fix, it will not.
Expand tests to catch future regressions.

Closes #314.

In honor of Pi Day, a fix for #314 on 3/14!

lgtm


Documentation for display()

This is arguably the most important function mentioned in this article. Yet there seems to be no documentation within around it in Azure Synapse Analytics reference documents.
Is that the case - is this a documentation gap? If so, how can we know what syntax this function requires (ex. parameters to pass, etc.)?

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

ID: 61bb471d-65c3-e53a-fe2c-2840f57b5f08
Version Independent ID: 05dd1440-dc85-ba91-aef9-c684dc47b149
Content: Python Visualizations - Azure Synapse Analytics
Content Source: articles/synapse-analytics/spark/apache-spark-data-visualization.md
Service: synapse-analytics
Sub-service: spark
GitHub Login: @midesa
Microsoft Alias: midesa

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

@VitalyMCT
Thank you for bringing this to our attention.
I've delegated this to content author @midesa, who will review it and offer their insightful opinions.

@jingyanjingyan
Can you please check and add your comments on this doc update request as applicable.

@ruixinxu
Please review it.


Add Offline Support to the Website

Is your feature request related to a problem? Please describe.
Users currently face issues when accessing the website without an active internet connection. This impacts usability, especially in areas with poor connectivity, as the website becomes unusable without internet access.
PWA (Progressive web app approach)
Describe the solution you'd like
Implement offline support for the React-based website using a Service Worker. Key features of the solution include:

Caching Strategies:

Cache static assets (HTML, CSS, JavaScript) for offline availability.
Cache frequently accessed dynamic content to ensure the core functionality works offline.

Service Worker Integration:

Use Workbox or a similar library to manage Service Worker registration and caching.
Precache essential resources during the initial page load.

Please suggest any additional ideas or improvements to ensure a smooth offline experience.

FYI ( https://github.com/everyone ): As per discussion on slack , this is the not the priority issue right now ...but we will surely look into it in upcoming iterations.


bug met herobrine

Herobrine is not attacking me even though he should

Hey SmartStreamLabstesterCamusfade, This bug have been fixed in the new bug fix update. You can download it here: https://modrinth.com/mod/mysticrift-mobspawner-spawn-eggs/versions#all-versions


Exception when trying to deserialize from file: System.InvalidOperationException: There is an error in XML document

Hm... something is not happy:
Exception when trying to deserialize from file: System.InvalidOperationException: There is an error in XML document. ---> System.InvalidOperationException: <ShapingPoint xmlns='http://www.garmin.com/xmlschemas/TripExtensions/v1'> was not expected
  at System.Xml.Serialization.XmlSerializationReaderInterpreter.ReadRoot (System.Xml.Serialization.XmlTypeMapping rootMap) [0x0004f] in /Users/builder/jenkins/workspace/archive-mono/2020-02/android/release/mcs/class/System.XML/System.Xml.Serialization/XmlSerializationReaderInterpreter.cs:182
  at System.Xml.Serialization.XmlSerializationReaderInterpreter.ReadRoot () [0x00022] in /Users/builder/jenkins/workspace/archive-mono/2020-02/android/release/mcs/class/System.XML/System.Xml.Serialization/XmlSerializationReaderInterpreter.cs:87
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.Serialization.XmlSerializationReader reader) [0x00017] in /Users/builder/jenkins/workspace/archive-mono/2020-02/android/release/mcs/class/System.XML/System.Xml.Serialization/XmlSerializer.cs:369
   --- End of inner exception stack trace ---
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.Serialization.XmlSerializationReader reader) [0x00052] in /Users/builder/jenkins/workspace/archive-mono/2020-02/android/release/mcs/class/System.XML/System.Xml.Serialization/XmlSerializer.cs:379
  at System.Xml.Serialization.XmlSerializer.Deserialize (System.Xml.XmlReader xmlReader) [0x00026] in /Users/builder/jenkins/workspace/archive-mono/2020-02/android/release/mcs/class/System.XML/System.Xml.Serialization/XmlSerializer.cs:358
  at Utility.Serializer.Deserialize[T] (System.IO.TextReader reader, System.Type[] types, System.Xml.XmlReaderSettings settings) [0x0002b] in <f7fbc81366034ae9805dafa1da1c0dad>:0

GPX file causing the issue. Export from Garmin Basecamp on Windows: You Yangs.txt
Assuming it's a problem with SharpGPX, but it could be me...

Hello!
I just loaded the file using one of the tools under the Examples folder and it seems to work fine.
Looking at the stack trace I see that it seems to be using a Mono implementation of System.Xml, have you cloned the code into another project and linked it with Mono?

Hi and I am using the SharpGPX NuGet package with a Xamarin.Android project, which uses Mono under the hood, I believe.
So the thinking is that it's not an issue with SharpGPX, but something with the System.XML implementation within Xamarin.Android that is unhappy with some part of the XML file?

I think it's something in Xamarin.Android, is there any way you can give me access to the project so I can see the code? It would be nice for SharpGPX to work with Xamarin

Hi and yes, its all on https://github.com/johnjore/hajk, and the GPX file I imported should be in my first post. Note: I'm not a developer, so its a bit messy, but any issues, please reach out?

From: Michael Coyle @.***>
Sent: 02 May 2022 07:29
To: MichaelFCoyle/SharpGPX
Cc: John Jore; Author
Subject: Re: [MichaelFCoyle/SharpGPX] Exception when trying to deserialize from file: System.InvalidOperationException: There is an error in XML document (Issue #9)
I think it's something in Xamarin.Android, is there any way you can give me access to the project so I can see the code? It would be nice for SharpGPX to work with Xamarin
—
Reply to this email directly, view it on GitHubhttps://github.com/MichaelFCoyle/SharpGPX/issues/9#issuecomment-1114341341, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADC2CITCZB2DBVYJNYJAPM3VH3ZU5ANCNFSM5UYHVYUA.
You are receiving this because you authored the thread.Message ID: @.***>
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/MichaelFCoyle/SharpGPX/issues/9#issuecomment-1114341341", "url": "https://github.com/MichaelFCoyle/SharpGPX/issues/9#issuecomment-1114341341", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]


Bugfix

Fixed issue with class instance properties coming up as undefined, introduced use of typdi for dependency injection


viostor: It seems that RhelScsiVerify has an improperly handled boundary condition.

Describe the bug
In the RhelScsiVerify function, I understand that the case where lba + blocks == adaptExt->lastLBA should be a normal SRB. As shown in the figure below.  If I am mistaken, please point it out to me.

Right, we probably missed this case when applied https://github.com/virtio-win/kvm-guest-drivers-windows/commit/67f64d01967769d4b9673e4c1bd08fab184469d7
Thank you for spotting it out.
The fix is welcome.
Vadim.

Other location I found it seems to be correct: https://github.com/virtio-win/kvm-guest-drivers-windows/blob/657ad7efb539dd186cec1fd33cf31ba710f5dfb1/viostor/virtio_stor.c#L1333

Right, we probably missed this case when applied 67f64d0
Thank you for spotting it out. The fix is welcome. Vadim.
@vrozenfe
ok, I submit a patch to fix it
JianMing

yes, it can be closed


Problem with reversalreceipt

On DATECS WP-50X
Response contains
"ok":true
but nothing is printed on ECR
No such problem with TREMOL S21
debug.log
data.txt
Responce.txt

There is extra space after the date in this field: "receiptDateTime":"2019-10-10T11:01:30 "
It should be: "receiptDateTime":"2019-10-10T11:01:30"


iPad M1 iOS15 support.

Hello guys!
First of all I want to say THANK YOU for your great work and UTM project. It's really can be a future of iOS mobile devices.
I understand that all this work is really hard - but I want to ask you - do you have a plans to make UTM support for iPads with M1 chip on new iOS15?

Hi @AnatolyPristensky and thanks for your kind message. You will be glad to know UTM works on M1 iPad and iOS 15 will be officially supported in the fall, though UTM already has been reported to work on the more recent betas.
If you get a popup saying "This version of iOS does not support running VMs while unmodified. …" this does not mean that UTM doesn't work with this version of iOS. As the message suggests, it's the other way around: Apple's iPadOS doesn't provide the necessary API to run UTM like, e. g. on M1 Mac. So we need workarounds like debugger connection ("tethering") to get acceptable performance, or slow emulation as a fallback which will likely keep working regardless of iOS updates.
In some previous versions of iOS it was possible to run UTM without the above workarounds. TLDR; Apple decided to remove this capability in iOS 14.4 #2271


Updates to overview page

https://issues.redhat.com/browse/AA-1163
before

after

@trahman73 please review
/cc Tina Yip

Thanks @h-kataria looking good! Just two points below:

Can we use this text for the Automation Analytics card? Right now it's just repeating the same as the Hub card.

Gain insights into your deployments through visual dashboards and organization statistics, calculate your return on investment, and explore automation processes details.

Can we horizontally center the "Learn more" link on the Hub card as in the mockups?
Mockups: https://marvelapp.com/prototype/aic093b/screen/86290519

@h-kataria Looks great, thank you for updating this! +1 to what @trahman73 mentioned. Would it also be possible to get a link if these changes are staged just to confirm some of the other items from the Jira issue such as the 3. "Critical" & "Warnings" label linking and 4. Navigation? Thank you!

Thanks @h-kataria looking good! Just two points below:
* Can we use this text for the Automation Analytics card? Right now it's just repeating the same as the Hub card.

Gain insights into your deployments through visual dashboards and organization statistics, calculate your return on investment, and explore automation processes details.
* Can we horizontally center the "Learn more" link on the Hub card as in the mockups?
  Mockups: https://marvelapp.com/prototype/aic093b/screen/86290519

made requested changes

@h-kataria - code changes look good - just the lint errors need to be addressed

Codecov Report

Merging #94 (d563a4e) into main (ffe3f9a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #94   +/-   ##
=======================================
  Coverage   69.43%   69.43%
=======================================
  Files          58       58
  Lines         553      553
  Branches      107      107
=======================================
  Hits          384      384
  Misses        157      157
  Partials       12       12

Impacted Files
Coverage Δ

src/components/automation-hub/hub-card.js
91.22% <ø> (ø)

src/messages/messages.js
100.00% <ø> (ø)

src/components/analytics/analytics-card.js
88.88% <100.00%> (ø)

Continue to review full report at Codecov.

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


(BOLT-1433) Load resource types for bolt plan execution

This commit enables loading resource types for bolt plan execution to support bolt plans being more resource aware. Loading resource types was originally disabled for two reasons:

Prevent name conflicts between type names and task names
prevent errors from manifest code being loaded that had types that depend on the concept of a 'Catalog' which is missing in 'script/bolt' mode

The naming conflics (1) appear to be solved because tasks are not types (and therefore will not have naming conflicts)
The Catalog dependent type issue is a bit trickier, an experimental workaround is included in https://github.com/puppetlabs/bolt/pull/1102 whereby a squelch_parse_errors override is included in bolt's puppet environment.
This reverts commit 4f45edfae91fd27a2dfe4eb17fce132cb61912aa

I'm trying to reproduce the integration test that is failing on my local ubuntu 1604 laptop. For both this branch and master I get the same failure running bundle exec rspec ./spec/integration/provider/service/systemd_spec.rb:10. Are there additional test setup steps I need to run that integration test?

CLA signed by all contributors.

@hlindberg and @thallgren Is this something we can safely do for bolt? We looked a bit in to doing this in bolt but it does not seem straight forward.

What are the requirements wrt to being able to use Resource types? What exists and what parameters they take only? Or do they need to be operational (read/apply state).
I am assuming user define resources are not included? Would it be possible to limit the set of resources available to those that are loaded as result of "puppet generate"? I.e. no actual implementation code is loaded - just the description of the resource types.
is it Puppet::Parser::Resource, or Puppet::Resource that are wanted?
Asking all these questions since the entanglement of everything in the 3x runtime for resource/catalog/ etc. is very high.

@hlindberg I tried using the puppet generate face you recommended looking in to in bolt. I was unable to provide the generated types to the plan scope. Also I was unable to generate types on the modulepath that were note core types. With the work in this PR it seems that built in types, core types, and types that are defined in modules on the modulepath are all able to be referenced
with a call to get_targets in a plan without an Evaluation Error: Resource type not found.
At this point our only well defined use case for resources in plans is to be able to reference them by their type names as an argument to a plan function. We still need to think about other things we would like to do with Resources in plans.

For now we will be using the generate types face in bolt to only consider the metatdata for the types instead of using the full 3x runtime. This approach should not require any changes to puppet.


Option to prevent automatic recoding of papers

Addresses #71
Display an animal from https://www.asciiart.eu/

In case of issue the full error will be logged in the popup's console (right click + "Inspect") and an error message will be displayed:

npm run test passes ✅


Allows partial regex for named parameters

I must to re-create a merge request for #704 because my github repository was deleted.... my apologies

Welll, I don't understand why they have 22 modified files.... I cancel this merge


Modified cover page results in IndexError

I copied style.docx to my source directory and I only changed the document title, subject and author in LibreOffice Writer 6.3.2. I saved the file and ran a make docx with a docx_style = 'style.docx' in my conf.py. This results in an exception:
# Sphinx version: 1.8.4
# Python version: 3.6.8 (CPython)
# Docutils version: 0.14
# Jinja2 version: 2.10
# Last messages:
#   processing test.docx...
#
#
#
#   resolving references...
#
#
#
#   writing...
#
# Loaded extensions:
#   sphinx.ext.mathjax (1.8.4) from /usr/local/lib/python3.6/site-packages/sphinx/ext/mathjax.py
#   alabaster (0.7.12) from /usr/local/lib/python3.6/site-packages/alabaster/__init__.py
#   sphinx.ext.todo (1.8.4) from /usr/local/lib/python3.6/site-packages/sphinx/ext/todo.py
#   docxbuilder (unknown version) from /usr/local/lib/python3.6/site-packages/docxbuilder/__init__.py
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/sphinx/cmd/build.py", line 304, in build_main
    app.build(args.force_all, filenames)
  File "/usr/local/lib/python3.6/site-packages/sphinx/application.py", line 341, in build
    self.builder.build_update()
  File "/usr/local/lib/python3.6/site-packages/sphinx/builders/__init__.py", line 342, in build_update
    self.build(['__all__'], to_build)
  File "/usr/local/lib/python3.6/site-packages/sphinx/builders/__init__.py", line 412, in build
    self.write(docnames, list(updated_docnames), method)
  File "/usr/local/lib/python3.6/site-packages/docxbuilder/builder.py", line 115, in write
    self.write_doc(docname, doctree)
  File "/usr/local/lib/python3.6/site-packages/docxbuilder/builder.py", line 122, in write_doc
    self.writer.write(doctree, destination)
  File "/usr/local/lib/python3.6/site-packages/docutils/writers/__init__.py", line 80, in write
    self.translate()
  File "/usr/local/lib/python3.6/site-packages/docxbuilder/writer.py", line 163, in translate
    visitor = self.builder.create_translator(self.document, self.builder)
  File "/usr/local/lib/python3.6/site-packages/sphinx/builders/__init__.py", line 139, in create_translator
    return self.app.registry.create_translator(self, *args)
  File "/usr/local/lib/python3.6/site-packages/sphinx/registry.py", line 386, in create_translator
    translator = translator_class(*args)
  File "/usr/local/lib/python3.6/site-packages/docxbuilder/writer.py", line 832, in __init__
    self._bullet_list_id = self._docx.get_bullet_list_num_id('List Bullet')
  File "/usr/local/lib/python3.6/site-packages/docxbuilder/docx/docx.py", line 1458, in get_bullet_list_num_id
    return self.style_docx.get_numbering_style_id(style_name)
  File "/usr/local/lib/python3.6/site-packages/docxbuilder/docx/docx.py", line 1236, in get_numbering_style_id
    style_elem, 'w:pPr/w:numPr/w:numId')[0]
IndexError: list index out of range

Any idea what is going on? The document title, subject and author are really the only things I have changed in the style file. When I replace the file by the original one, it works, but I'd like to modify the text fields on the cover page.

Hi, @koenvervloesem . Thank you for your issue report.
This issue is because LibreOffice removes relationships between styles and bullet/enumeration list definitions (Opening style.docx by LibreOffice removes numbering style from List Bullet and List Number styles).
Therefore, restoring the relationships manually avoid IndexError (Open style edition dialog from Styles pain, select Numbering style. WWNum2 is for List Bullet. WWNum1 is for List Number).
Unfortunately, LibreOffice removes also invisible cover page information from default style.docx file. So, generated document based on the style.docx modified by LibreOffice will not include cover page.
For this issue, I will add an option to use first page or section as cover page.
Tentatively, please use this style.docx instead (This includes dummy whitespace in cover page to retain cover page information).

Awesome, @amedama41, that works! The only issue I have now with your new style.docx is that the sample TOC is generated in my document after the cover page, before my real TOC.

@koenvervloesem LibreOffice had broken the original cover page range.
I updated style.docx (Inserting empty table between cover page and sample TOC).
Please try this style file.

Great, this works perfectly now! Thank you!
This is really an awesome project. I have started writing a book with Sphinx in reStructuredText and the docx output of docxbuilder is amazing.


[$250] Android - Chat - When context menu is closed, keyboard doesn´t open again

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.60-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5204110&group_by=cases:section_id&group_order=asc&group_id=292106
Issue reported by: Applause Internal Team
Action Performed:

Open the Expensify app.
Open any chat.
Tap on compose box to open the keyboard.
Long tap on any message to open context menu.
Verify that keyboard is closed when context menu is opened.
Tap outside of the context menu to close it.
Verify that keyboard is opened again.

Expected Result:
When the context menu is closed, keyboard should be displayed again.
Actual Result:
After the context menu is closed, keyboard is not opened again, despite compose box being focused.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/6e8fea20-14ca-44b8-beb5-c12cdab1d4dd

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021856600731292480706
Upwork Job ID:<PHONE_NUMBER>292480706
Last Price Increase: 2024-11-27

Issue OwnerCurrent Issue Owner: @kirillzyusko

Proposal
Please re-state the problem that we are trying to solve in this issue.
Android - Chat - When context menu is closed, keyboard doesn´t open again
What is the root cause of that problem?
The keyboard is dismissed because the popup menu's appearance does not cause the input to lose focus; when the popup menu disappears, the input is still focused.
What changes do you think we should make in order to solve the problem?

We need to add these lines to track whether the input is focused and popup menu is active

const [isMenuActive, setIsMenuActive] = useState(false);
const isPrevMenuActive = usePrevious(isMenuActive);
const [isReportInputFocused, setIsReportInputFocused] = useState(false);
const keyboardState = useKeyboardState();

To track that input has been focused once, add this useEffect

useEffect(() => {
        if (!keyboardState.isKeyboardShown) {
            return;
        }
        setIsReportInputFocused(true);
}, [keyboardState.isKeyboardShown]);

Add this useEffect to track whether pop up menu is shown

useEffect(() => {
        setIsMenuActive(isContextMenuActive || !!(isEmojiPickerActive ?? isPaymentMethodPopoverActive));
}, [isContextMenuActive, isEmojiPickerActive, isPaymentMethodPopoverActive]);

After all, add this useEffect to execute the logic

useEffect(() => {
        if (!isPrevMenuActive && isMenuActive) {
            ReportActionComposeFocusManager.composerRef.current?.blur();
        }

        if (isPrevMenuActive && !isMenuActive && isReportInputFocused) {
            ReportActionComposeFocusManager.composerRef.current?.focus();
            setIsReportInputFocused(ReportActionComposeFocusManager.composerRef.current?.isFocused() ?? true);
        }
}, [isMenuActive, isPrevMenuActive, isReportInputFocused]);

What alternative solutions did you explore? (Optional)
Result
https://github.com/user-attachments/assets/056bf0f3-89d8-44db-b4b8-7f1460ec9625
Reminder: Please use plain English, be brief and avoid jargon. Feel free to use images, charts or pseudo-code if necessary. Do not post large multi-line diffs or write walls of text. Do not create PRs unless you have been hired for this job.

hi @jjcoffee can you check on the above proposal when you have a moment? TY!

@Pholenk Thanks for your proposal! Your RCA is plausible, but I'm missing a bit more detail on why this only occurs on Android native.
For your solution, it would also be helpful for you to add where you propose to make your changes, ideally by linking to a relevant code snippet.

Proposal
Updated

@Pholenk Thanks for your proposal! Your RCA is plausible, but I'm missing a bit more detail on why this only occurs on Android native.
For your solution, it would also be helpful for you to add where you propose to make your changes, ideally by linking to a relevant code snippet.

I have checked that this issue behaves the same way as when the user presses the physical back button while the keyboard is open/shown and, based on react native official documentation, it is a known issue for Android.

I actually can't reproduce on latest main, are you still able to reproduce @Pholenk?
https://github.com/user-attachments/assets/c0318f94-ff65-4750-a425-71977030c854
@Christinadobrzyn Can we ask for a retest?

I tested again on Google Pixel 6 Pro 13.0 NewDot version 9.0.60-0. it looks like the keyboard is not opening on Step 7 (of the OP)
https://github.com/user-attachments/assets/7dddc592-9952-4ef3-a38c-f0bba8dc3eb6

Hmm strange, I can't reproduce it at all (using Medium Phone API 34)! Are there any steps I might be missing? Have you tried with a small(er) chat, e.g. with a brand new user?
@Pholenk I don't think I can accept the RCA in your proposal without it dealing with why the bug is not happening consistently.

If that's the case, have you tried with the lower API such as 33 or lower @jjcoffee?
I'm afraid it's not reproducible only for API 34 because I and Christina used the same API (API 33 for Android 13)
I tried with a newly created account, latest version of main branch, and this issue is still there.
https://github.com/user-attachments/assets/0a321075-d68b-47fd-a250-6cbf149f932c

@Pholenk Ah yes, I can reproduce it now on API 33, thanks for the tip! Can you explain why that is?

You're most welcome @jjcoffee.

I have checked that this issue behaves the same way as when the user presses the physical back button while the keyboard is open/shown and, based on react native official documentation, it is a known issue for Android.

As far as I know, this happened because of this known issue of the text input focus.
That's why I propose to make the text input lose its focus programmatically before the context menu shows up and re-focus it again after the context menu disappears.

@Pholenk I guess my problem here is that the RCA in your proposal is still quite vague (maybe it's related to a known issue), and also that if it is a known issue then we would have seen this before. I'd be interested to see why this started happening.

Pholenk I guess my problem here is that the RCA in your proposal is still quite vague (maybe it's related to a known issue), and also that if it is a known issue then we would have seen this before. I'd be interested to see why this started happening recently (this was caught during regression testing, which means it must have worked previously).

I'm not sure what makes you vague about the RCA in my proposal. The only thing I know is that this issue is there and was found during the regression test, but I don't know what issues came in before the regression test that then made this a regression. If you can find what issues came in before the regression test, you can start tracking down those issues and finding what the root cause is.

@Pholenk The RCA you have implies that this should always have been a problem, but that can't be the case if a regression test was created for it. Your solution may be correct, but it may also be just a workaround, so I'd like to be sure that we're tackling the underlying cause.
I've bumped on Slack to get more eyes on this.

Hello, I'm Kiryl from Margelo expert agency and I'd like to work on that issue (if @Pholenk is not against, because I can see he was actively involved into solving that problem)

Thanks @kirillzyusko! I think if we end up using @Pholenk's solution in the end, we can offer partial compensation.

@kirillzyusko Any update on this?

@jjcoffee no, not yet, sorry. I was busy (and I'm still busy) with resolving some urgent deploy blockers 😔 Hopefully we can squash all of them soon and I'll get back to all issues that I was assigned to!

Just checking in on this @kirillzyusko - if you have an update on the status of working on this that'd be great! TY!

Not overdue - just waiting for @kirillzyusko to get a chance to look into this :pray:

Looking into it right now 👀

Okay, the problem is present in react-native itself. And iOS and Android have a different behavior: iOS will show a keyboard again, but Android may not show keyboard again (depends on Android version, Android 14-15 shows the keyboard again when Modal gets hidden, Android 13 doesn't do that by default).
The solution would be to force show a keyboard on Android to match the iOS behavior, however:

we can't simply call ref.focus() because as @Pholenk pointed out it's a known bug in RN;
the solution proposed by @Pholenk includes blurring input when context menu gets shown and then requesting a focus when menu gets hidden, while it works I have a feeling that we add more complexity because for force showing a keyboard we have to remember about blurring the keyboard and restoring a focus;
I think a slightly better solution would be to have something like this:

const shouldRestoreFocus = useRef(false);

useEffect(() => {
    if (isVisible) {
        shouldRestoreFocus.current = Keyboard.isVisible();
    }

    if (!isVisible && shouldRestoreFocus.current) {
        InteractionManager.runAfterInteractions(() => {
            KeyboardController.setFocusTo("current");
        });
    }
}, [isVisible]);

I put this code into BaseReportActionContextMenu.tsx, but I think it can be moved into a separate re-usable hook useFocusRestoreEffect.
The key difference in the code that I propose is that:

we don't need to use transitive refs (such as ReportActionComposeFocusManager.composerRef.current) - setFocusTo("current") will automatically restore a focus to last focused field;
setFocusTo("current") doesn't have a limitation which is present in react-native (you still can set focus and keyboard will be shown).

@jjcoffee curious to know your thoughts 🙌

@kirillzyusko That sounds like a good, clean solution to me - I also like the idea of moving it into a reusable hook. Let's do it!
:ribbon::eyes::ribbon: C+ reviewed

PR merged

Preparing for payment - @jjcoffee do we need a regression test for this?

BugZero Checklist:

[ ] [Contributor] Classify the bug:

Bug classification

Source of bug:

[ ] 1a. Result of the original design (eg. a case wasn't considered)
[ ] 1b. Mistake during implementation
[ ] 1c. Backend bug
[ ] 1z. Other:

Where bug was reported:

[ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
[ ] 2b. Reported on staging (eg. found during regression or PR testing)
[ ] 2d. Reported on a PR
[ ] 2z. Other:

Who reported the bug:

[ ] 3a. Expensify user
[ ] 3b. Expensify employee
[ ] 3c. Contributor
[ ] 3d. QA
[ ] 3z. Other:

[ ] [Contributor] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
Link to comment:

[ ] [Contributor] If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in #expensify-open-source has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.
Link to discussion:

[ ] [Contributor] If it was decided to create a regression test for the bug, please propose the regression test steps using the template below to ensure the same bug will not reach production again.

Regression Test Proposal Template

[ ] [BugZero Assignee] Create a GH issue for creating/updating the regression test once above steps have been agreed upon.
Link to issue:

Regression Test Proposal
Precondition:

Test:

Do we agree 👍 or 👎

Payment date:
Contributor+: @jjcoffee owed $250 via NewDot
Regression test https://github.com/Expensify/Expensify/issues/456375


Upgrade golang to 1.17

Signed-off-by: clyang82<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[x] Change is not relevant to the end user.

Changes
Golang 1.15 is now at end of life - https://endoflife.date/go

Verification

I think lint fails due to: https://github.com/fatih/faillint/issues/24.

@clyang82 you can now pull the latest version of faillint as the fix has been merged with: bingo get<EMAIL_ADDRESS>🙂

@clyang82 you can now pull the latest version of faillint as the fix has been merged with: bingo get<EMAIL_ADDRESS>🙂

Updated. Thanks @saswatamcode for your help.


vue-chartjs dependency fails to load

We have:
// nuxt.config.js

{
  src: '~plugins/vue-chartjs.js',
  ssr: false
},

// plugins/vue-chartjs.js

import Vue from 'vue'
import { Doughnut, Bar, mixins } from 'vue-chartjs'

Vue.component('portolio-chart', Doughnut.extend({
  props: ['data', 'options'],
  mounted () {
    this.renderChart(this.data, this.options)
  }
}))

This works fine on my colleague's Ubuntu machine, but when I try to run it on my Macbook Pro, or even in a Ubuntu VM, the dependency fails to load:
This dependency was not found:

* vue-chartjs in ./plugins/vue-chartjs.js

To install it, you can run: npm install --save vue-chartjs

Of course, the dependency is already installed, just as in https://github.com/nuxt/nuxt.js/issues/917 and https://github.com/nuxt/nuxt.js/issues/673.
I notice if I try to load the vue-chartjs library in a node.js console, I get:
ReferenceError: window is not defined

Which makes sense - it was designed for a browser environment.  But I would expect the {ssr: false} to prevent the library from attempting to load on the server.  Is {ssr: false} supported on our version of nuxt.js?  We're using<EMAIL_ADDRESS><EMAIL_ADDRESS>Current approach seems consistent with proposed wrapper for vue-chartjs: https://github.com/apertureless/vue-chartjs/issues/168
What else could I be missing?

Have you checked the nuxt-chartjs example ?

I found the issue.  I had my node_modules directory symlinked which caused some confusion and I ended up with my node_modules in the project directory not being the one that I was npm installing to.  Our configuration works fine though.
Closing.


Check for Deferral on Non-additional inputs

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify.
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Because Side Inputs are represented within the expanded inputs, the
check that the transform is a Combine with Side Inputs would never be
hit. This ensures that we do not consider additional inputs during the
check to defer evaluation of the node.

R: @aviemzur
CC: @amitsela @kennknowles

run spark validatesrunner

Coverage decreased (-0.003%) to 70.665% when pulling 1ac18b2eb1371422e60d50a8c3f37b3b24d59611 on tgroh:spark_non_deferral_fix into ce3dd4583294fcbb65121f5fecfc49a8b85634ad on apache:master.

I had thought to leave this for Spark runner experts to take a look, but since it fixes a broken postcommit it is probably nicer to merge it so everyone wakes up to a green build.
But someone else should definitely take a look at this and see if there's any further implications, or any questions about why it needed to be done.


garden: add a garden file for jsonpickle.github.io update tasks

Add a garden file for use with garden, a simple yet powerful command runner and git powertool.
Running garden doc will clone the github pages project and build the documentation into the pages repository.
Related-to: #484

I'm going to merge for now, and we can update the black target versions later.


Silent renew error

Hello,
I just have a problem with the automaticSilentRenew.
As I understand I don't have to do anything for it to work but after the silent renew has been made if I try to call an endpoint again I have a 401 error because the token used is still the expired one.
In auth I can see this error : Invalid token specified: Cannot read properties of undefined (reading 'replace').
I'm pretty sur the silent renew is made but the token is not updated.
Can you help me understand where is the problem ?
Thanks in advance

It's
validateRefreshResponse
  - _processClaims

Does MR #445 fix your issue?

Yes thank you, it's working now !


SIG: Allow eth to send to non-eth with regular signature

This change changes the signature checks to allow the case where an ethereum account wants to send a message to a non-ethereum account, e.g. to fund an f1 account from an f410 account, where the f410 address was funded on the subnet by the ipc-cli.
So far this has not been allowed because if message.from was an ethereum address then the method number had to be evm::Method::InvokeContract, which didn't work with custom actors (although an Account would have been able to receive funds this way), but due to the vulnearability fixed in https://github.com/consensus-shipyard/ipc/pull/772 a message sent from the EthAPI can only go to an Ethereum account, so f1 wasn't allowed (or, should not have been allowed).
The solution on Filecoin is https://docs.filecoin.io/smart-contracts/filecoin-evm-runtime/filforwarder which is a Solidity contract that does the forwarding and need to be deployed on an IPC subnet.
With the change in this PR, we should be able to use use the raw Fendermint RPC to produce a message signed by the Secp256k1 public key of the f410 account, but set the message.to to an f1 address, and sign it as a regular message with SignedMessage::new_secp256k1 (as in the unit test), and it should not hit the restrictions on eth-to-eth transactions.
Testing
I added a new integration test to check this works; it's part of the smoke-tests now:
cd testing/smoke-test/
cargo make setup
cargo make transfer-example
docker logs smoke-fendermint
cargo make teardown

The container logs show that the transaction was indeed delivered:
$ docker logs smoke-fendermint
...
2024-03-05T14:13:22.848200Z  INFO query{request=Query { data: b"\xa1jActorStateV\x04\n%X\x1f\n\x8a\x0eA\xca`\xedw\x04\xef7\x91~*\x06Z\xa5", path: "", height: block::Height(0), prove: false }}: fendermint/vm/interpreter/src/fvm/query.rs:66: query actor state height=43 pending=false addr="f410fevmb6cukbza4uyhno4co6n4rpyvamwvfd6grqfa" found=true
2024-03-05T14:13:22.857919Z  INFO fendermint/vm/interpreter/src/fvm/check.rs:51: check transaction exit_code=0 from="f410fevmb6cukbza4uyhno4co6n4rpyvamwvfd6grqfa" to="f1cwd2uvx62r44smul6bwt7byxobphhnc5uvusayy" method_num=0 gas_limit=10000000000 gas_used=5667215 info=""
2024-03-05T14:13:23.043714Z  INFO fendermint/app/src/app.rs:668: event=ProposalProcessed is_accepted=true height=44 size=1 hash="5657862577158E5ECAE51062F00935C416BCD7FEC9948C66923E8E0D99A91AE6" proposer="3FD7936E572A42FB9B8693B543AA93B46149E558"
2024-03-05T14:13:23.075544Z  INFO fendermint/vm/interpreter/src/fvm/exec.rs:143: tx delivered height=44 from="f410fevmb6cukbza4uyhno4co6n4rpyvamwvfd6grqfa" to="f1cwd2uvx62r44smul6bwt7byxobphhnc5uvusayy" method_num=0 exit_code=0 gas_used=5667215
...

You can see from="f410fevmb6cukbza4uyhno4co6n4rpyvamwvfd6grqfa" to="f1cwd2uvx62r44smul6bwt7byxobphhnc5uvusayy" method_num=0 indicates a simple transfer from f410 to f1.

@aakoshh could you confirm that this can be deployed in a running subnet as a soft fork? My understanding is yet since it strictly introduces a new capability.

@raulk that's right, unless some nodes on the subnet accept these transactions while others don't, and such transaction is actually sent (which an attacker who knows this could do to cause a fork!), there should be no problem.


Tslint stops working in VSCode, not showing unhandled exceptions

I was requested due to my review to create a new issue describing my problem and solution.
We were using tslint without any issues and then one day it stopped working. I was able to install the older deprecated version of tslint in visual studio code and that was able to actually give me some errors. The error was an un-handled exception due to having a list of mismatched projects in our angular.json and nx.json. By removing the un-necessary projects the old extension began to work correctly again, I then uninstalled the old version and re-installed the newer extension and all was working again. I noticed on the reviews a lot of people are reporting that the extension just stops working, I believe this is due to some un-handled exception being thrown like in my case but the new extension is not showing these errors for us like the old one was so there is no way of knowing how to fix it.
To reproduce this you just need to have an angular project that is using narwhal (nx), then under the nx.json add an extra project under the listed projects or remove a project from the angular.json this should throw an error with the old extension but will simply break the new extension.
The issue is not the extension breaking, but rather the extension not showing errors when it breaks like the old one was. We have fixed our own project and the extension is working great again, this is more for the information of other developers who may come across this issue, it would be great if we could see these errors instead of having to re-install the old deprecated one.

To reproduce this you just need to have an angular project that is using narwhal (nx), then under the nx.json add an extra project under the listed projects or remove a project from the angular.json this should throw an error with the old extension but will simply break the new extension.
Could you specify what you mean by this?

I'm using nrwl/nx. I just realized that the tslint extension was not working as it should. Doing

Uninstall tslint extension
Install deprecated version
Uninstall deprecated version
Install tslint extension
made it work again but I did not modify the angular.json file. I am wondering what the unhandled exception might have been.

I think in your case you may have had a different error than me but sounds like the problem is the same, the errors are not being shown. Maybe the deprecated version had some sort of clean up that fixed whatever wasn't working correctly.

Only version 1.2.3 works for me. Newer versions don't work in VSCode.

It stops working for me as well, seemingly randomly. I have 1.3.3. I tried disable, restart, enable. Then the tslint errors show up, but the autofix feature still doesn't work. I had to exit VS Code and run it again, and now it's working.

Closing as TSLint has been deprecated in favor of ESLint and this extension is no longer being maintained
Please look into migrating to use ESLint and the ESLint VS Code extension


Add kuberenetes resource limits & security context configs

This PR adds more support for secure computing environments for metaflow in kubernetes
• Added support for kubernetes security_context field in the metaflow config file or decorators.
Use the 'METAFLOW_KUBERNETES_SECURITY_CONTEXT' field in the config file or 'security_contxt'
The input is a JSONified version of the YML in the kuberentes specification for security context.
Example from the config file
...
       "METAFLOW_KUBERNETES_SECURITY_CONTEXT": {
        "allowPrivilegeEscalation": false,
    "runAsNonRoot": true,
    "runAsUser": 1000,
    "seccompProfile": {
      "type": "RuntimeDefault"
    }
  }
...

Note that modifications in your docker images may be needed to to support this (such as creating a non-root user etc)
• Added support for kubernetes resource memory and cpu limits field in the metaflow config file or decorators.
This follows the format for memory and cpu requirements for the resource requests field. Examples usage:
...     "METAFLOW_KUBERNETES_RESOURCE_LIMITS_MEMORY": 4096,     "METAFLOW_KUBERNETES_RESOURCE_LIMITS_CPU": 1, ...
@kubernetes(
    memory=2048,
    cpu=1
    resource_limits_memory=4096,
    resource_limits_cpu=1
)

Just an FYI, but there is a (semi-)related PR for adding support for limits. There is a related issue on it too, which is linked in the PR.
Not sure if the approach in the issue is still the desired approach from the Metaflow team though. Currently waiting for a review on the PR.

Just an FYI, but there is a (semi-)related PR for adding support for limits. There is a related issue on it too, which is linked in the PR.
Not sure if the approach in the issue is still the desired approach from the Metaflow team though. Currently waiting for a review on the PR.

Thanks. Maybe 'limit_memory' and 'limit_cpu' in the decorator is the cleanest way to specify it

Just an FYI, but there is a (semi-)related PR for adding support for limits. There is a related issue on it too, which is linked in the PR.
Not sure if the approach in the issue is still the desired approach from the Metaflow team though. Currently waiting for a review on the PR.

Thanks. Maybe 'limit_memory' and 'limit_cpu' in the decorator is the cleanest way to specify it

Perhaps, the implementation in the other PR was based on a discussion with the Metaflow team in Slack. (Think there is a link to the Slack thread in the original issue.)
You are not the first person to raise the alternative suggestion. Someone else raised it in the issue too. I've referenced it in the other PR and was waiting to hear back from the Metaflow folk.

Just an FYI, but there is a (semi-)related PR for adding support for limits. There is a related issue on it too, which is linked in the PR.
Not sure if the approach in the issue is still the desired approach from the Metaflow team though. Currently waiting for a review on the PR.

Thanks. Maybe 'limit_memory' and 'limit_cpu' in the decorator is the cleanest way to specify it

Perhaps, the implementation in the other PR was based on a discussion with the Metaflow team in Slack. (Think there is a link to the Slack thread in the original issue.)
You are not the first person to raise the alternative suggestion. Someone else raised it in the issue too. I've referenced it in the other PR and was waiting to hear back from the Metaflow folk.

If it is decided to go with the approach in this PR, then I guess Argo Workflow functionality will need to be updated too. Also, I guess ephemeral-storage?

@roofurmston I've just refactored the code to support argo as well. There was a 'TODO' in the original code to use shared code for the Argo and Kubernetes container creation. I did that by creating a utility function used by both.

@roofurmston checking in on this. Since there is a refactor in here it will be hard to keep this PR open & up-to date for an extended period.

@roofurmston checking in on this. Since there is a refactor in here it will be hard to keep this PR open & up-to date for an extended period.

I'm not a maintainer of Metaflow, I'm afraid. :)
I left the initial comment as I had a related PR. Mine is still waiting to be reviewed.
Maybe it has slipped the radar of the Metaflow folk and it is worth raising with them?

For all K8s options, there is a possibility to specify them also on (a) the command line and (b) as args to the @kubernetes decorator.
Will the security context be able to be specified the same way? The user-experience won't be great since it's a json object.
The benefit of having the @kubernetes decorator is that there can be different values for different steps. Especially if there are different docker images used for different steps, a global security context may not work. E.g. a Docker image the user authored could have a non-root user, but a Docker image taken from an OSS project may only have a root user.
Not sure if this is a good idea but one option could be to specify the security_context in the @kubernetes decorator to point to a file. This file could have the security context json.
Similarly, will it be able to specify resource_limits in the decorator?

Yes @shrinandj all 3 fields that have been added to the config also support the kubernetes decorator.
For example we support
  @kubernetes(security_context=secure_environment_config,
               memory=100,
               resource_limits_memory=200,
               resource_limits_cpu=4)
   @step
   def hello(self):
         pass

Where secure_environment_config is some complex setting like the below (which is what we use):
{
        "allowPrivilegeEscalation": False,
    "capabilities": {
      "drop": [
        "ALL"
      ]
    },
    "runAsNonRoot": True,
    "runAsUser": 1000,
    "seccompProfile": {
      "type": "RuntimeDefault"
    }
}

There are so many recursive params in the security settings it seemed to me to make sense to use json

Is this PR going to happen? We wanted to apply some Kubernetes quota on one of our namespaces for limiting excessive overuse but realized that Kubernetes quotas require limits, which is currently unsupported. Would be interested in that feature.

The feedback I have had from the Metaflow team is that you need to set limits through a policy agent. This is the approach we will be investigating in due course.

Hey @roofurmston thanks for your response.
Well I am certainly not an k8s expert, but afaik those policies check for example that the pods do have limits and quotes in valid ranges. I was not / I am not aware that you can apply limits that way as well.... have to look around and familiarize myself with that concept.
Cheers

I was pointed in the direction of https://kyverno.io/. Other policy agents also allow mutations, such as open policy agent gatekeeper, but I couldn't find a way to set the resources through that policy agent.

Many thanks for the pointer. Will definitely try that.


Article references the SkypeOnlineConnector module - Should be updated to reflect the Teams module

[Enter feedback here]
From the article:
"Note
Before you can run these cmdlets you must be connected to Skype for Business Online PowerShell. For more information, see Manage Skype for Business Online with Microsoft 365 or Office 365 PowerShell."
I don't see any commands or arguments not supported by the Teams PowerShell module so why not reference that module, given that Skype Online is soon-to-be dead in the water? Also, "The Teams PowerShell public release version 2.0 or higher includes all Skype for Business Online Connector cmdlets, providing a single module for Teams PowerShell management."
Reference: https://techcommunity.microsoft.com/t5/microsoft-teams/skype-for-business-online-connector-retires-in-february-2021/m-p/2005963
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 2b2fe730-a95d-d12d-adb8-49d1397c77c7
Version Independent ID: 1f6077c3-6114-cb5e-8b1a-7587188f2cdf
Content: Set up for webinars in Microsoft Teams - Microsoft Teams
Content Source: Teams/set-up-webinars.md
Service: msteams
GitHub Login: @KarliStites
Microsoft Alias: kastites

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

I'm merely pointing out that the doc in question is using deprecated technology (the Skype PS module) and should be updated to reflect what's current, the Teams Online 'MicrosoftTeams' PS module. No need for support ;-)


Colab failing/zipping before it begins training [BUG]

Describe the bug
The Colab encounters an error then proceeds to zip the results. Training phase never starts. I had this working 2 days ago (with a much shorter audio file) but repeated attempts now fail (12 minute mp3 file/31Mb).
System Info

OS X Catalina
Logic 10.15.1
v1.0.2 DDSP VST

To Reproduce
Start Colab, click run, OK G Drive, select folder with audio, chugs awhile, then fails...
Screenshots
See copied output below (don't know if that helps).
Additional context
Output:
Installing DDSP...
This should take about 2 minutes...
Copying audio to colab for training...
Copying /content/gdrive/MyDrive/Train_VST/Monochord short.mp3 to audio/Monochord_short.mp3
Preparing new dataset from audio/
Creating dataset...
This usually takes around 2-3 minutes for each minute of audio
(10 minutes of training audio -> 20-30 minutes)
Training...
I0929 00:21:29.349751<PHONE_NUMBER>78784 ddsp_run.py:179] Restore Dir: /content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020
I0929 00:21:29.350117<PHONE_NUMBER>78784 ddsp_run.py:180] Save Dir: /content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020
I0929 00:21:29.350983<PHONE_NUMBER>78784 resource_reader.py:50] system_path_file_exists:optimization/base.gin
E0929 00:21:29.351246<PHONE_NUMBER>78784 resource_reader.py:55] Path not found: optimization/base.gin
I0929 00:21:29.352814<PHONE_NUMBER>78784 resource_reader.py:50] system_path_file_exists:eval/basic.gin
E0929 00:21:29.353032<PHONE_NUMBER>78784 resource_reader.py:55] Path not found: eval/basic.gin
I0929 00:21:29.355939<PHONE_NUMBER>78784 ddsp_run.py:152] Operative config not found in /content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020
I0929 00:21:29.356218<PHONE_NUMBER>78784 resource_reader.py:50] system_path_file_exists:models/vst/vst.gin
E0929 00:21:29.356484<PHONE_NUMBER>78784 resource_reader.py:55] Path not found: models/vst/vst.gin
I0929 00:21:29.362102<PHONE_NUMBER>78784 resource_reader.py:50] system_path_file_exists:datasets/tfrecord.gin
E0929 00:21:29.362300<PHONE_NUMBER>78784 resource_reader.py:55] Path not found: datasets/tfrecord.gin
I0929 00:21:29.362647<PHONE_NUMBER>78784 resource_reader.py:50] system_path_file_exists:datasets/base.gin
E0929 00:21:29.362849<PHONE_NUMBER>78784 resource_reader.py:55] Path not found: datasets/base.gin
I0929 00:21:29.370896<PHONE_NUMBER>78784 ddsp_run.py:184] Operative Gin Config:
import ddsp
import ddsp.training as ddsp2
Macros:
==============================================================================
batch_size = 16
evaluators = [@BasicEvaluator]
frame_rate = 50
frame_size = 1024
learning_rate = 0.0003
n_samples = 64320
sample_rate = 16000
Parameters for processors.Add:
==============================================================================
processors.Add.name = 'add'
Parameters for Autoencoder:
==============================================================================
Autoencoder.decoder = @decoders.RnnFcDecoder()
Autoencoder.encoder = None
Autoencoder.losses = [@losses.SpectralLoss()]
Autoencoder.preprocessor = @preprocessing.OnlineF0PowerPreprocessor()
Autoencoder.processor_group = @processors.ProcessorGroup()
Parameters for Crop:
==============================================================================
Crop.crop_location = 'back'
Crop.frame_size = 320
Parameters for evaluate:
==============================================================================
evaluate.batch_size = 32
evaluate.data_provider = @data.TFRecordProvider()
evaluate.evaluator_classes = %evaluators
evaluate.num_batches = 5
Parameters for FilteredNoise:
==============================================================================
FilteredNoise.n_samples = %n_samples
FilteredNoise.name = 'filtered_noise'
FilteredNoise.scale_fn = @core.exp_sigmoid
FilteredNoise.window_size = 0
Parameters for FilteredNoiseReverb:
==============================================================================
FilteredNoiseReverb.n_filter_banks = 32
FilteredNoiseReverb.n_frames = 500
FilteredNoiseReverb.name = 'reverb'
FilteredNoiseReverb.reverb_length = 24000
FilteredNoiseReverb.trainable = True
Parameters for get_model:
==============================================================================
get_model.model = @models.Autoencoder()
Parameters for Harmonic:
==============================================================================
Harmonic.amp_resample_method = 'linear'
Harmonic.n_samples = %n_samples
Harmonic.name = 'harmonic'
Harmonic.normalize_below_nyquist = True
Harmonic.sample_rate = %sample_rate
Harmonic.scale_fn = @core.exp_sigmoid
Parameters for OnlineF0PowerPreprocessor:
==============================================================================
OnlineF0PowerPreprocessor.compute_f0 = False
OnlineF0PowerPreprocessor.compute_power = True
OnlineF0PowerPreprocessor.crepe_saved_model_path = None
OnlineF0PowerPreprocessor.frame_rate = %frame_rate
OnlineF0PowerPreprocessor.frame_size = %frame_size
OnlineF0PowerPreprocessor.padding = 'center'
Parameters for ProcessorGroup:
==============================================================================
ProcessorGroup.dag =
[(@synths.Harmonic(), ['amps', 'harmonic_distribution', 'f0_hz']),
(@synths.FilteredNoise(), ['noise_magnitudes']),
(@processors.Add(), ['filtered_noise/signal', 'harmonic/signal']),
(@effects.FilteredNoiseReverb(), ['add/signal']),
(@processors.Crop(), ['reverb/signal'])]
Parameters for RnnFcDecoder:
==============================================================================
RnnFcDecoder.ch = 256
RnnFcDecoder.input_keys = ('pw_scaled', 'f0_scaled')
RnnFcDecoder.layers_per_stack = 1
RnnFcDecoder.output_splits =
(('amps', 1), ('harmonic_distribution', 60), ('noise_magnitudes', 65))
RnnFcDecoder.rnn_channels = 512
RnnFcDecoder.rnn_type = 'gru'
Parameters for sample:
==============================================================================
sample.batch_size = 16
sample.ckpt_delay_secs = 300
sample.data_provider = @data.TFRecordProvider()
sample.evaluator_classes = %evaluators
sample.num_batches = 1
Parameters for SpectralLoss:
==============================================================================
SpectralLoss.logmag_weight = 1.0
SpectralLoss.loss_type = 'L1'
SpectralLoss.mag_weight = 1.0
Parameters for TFRecordProvider:
==============================================================================
TFRecordProvider.centered = True
TFRecordProvider.file_pattern = 'data/train.tfrecord*'
TFRecordProvider.frame_rate = 50
Parameters for train:
==============================================================================
train.batch_size = %batch_size
train.data_provider = @data.TFRecordProvider()
train.num_steps = 30000
train.steps_per_save = 300
train.steps_per_summary = 300
Parameters for Trainer:
==============================================================================
Trainer.checkpoints_to_keep = 3
Trainer.grad_clip_norm = 3.0
Trainer.learning_rate = %learning_rate
Trainer.lr_decay_rate = 0.98
Trainer.lr_decay_steps = 10000
I0929 00:21:29.371028<PHONE_NUMBER>78784 train_util.py:76] Defaulting to MirroredStrategy
2022-09-29 00:21:30.042095: W tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:39] Overriding allow_growth setting because the TF_FORCE_GPU_ALLOW_GROWTH environment variable is set. Original config value was 0.
INFO:tensorflow:Using MirroredStrategy with devices ('/job:localhost/replica:0/task:0/device:GPU:0',)
I0929 00:21:30.047340<PHONE_NUMBER>78784 mirrored_strategy.py:374] Using MirroredStrategy with devices ('/job:localhost/replica:0/task:0/device:GPU:0',)
Traceback (most recent call last):
File "/usr/local/bin/ddsp_run", line 8, in
sys.exit(console_entry_point())
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/ddsp_run.py", line 227, in console_entry_point
app.run(main)
File "/usr/local/lib/python3.7/dist-packages/absl/app.py", line 308, in run
_run_main(main, args)
File "/usr/local/lib/python3.7/dist-packages/absl/app.py", line 254, in _run_main
sys.exit(main(argv))
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/ddsp_run.py", line 202, in main
report_loss_to_hypertune=FLAGS.hypertune)
File "/usr/local/lib/python3.7/dist-packages/gin/config.py", line 1605, in gin_wrapper
utils.augment_exception_message_and_reraise(e, err_str)
File "/usr/local/lib/python3.7/dist-packages/gin/utils.py", line 41, in augment_exception_message_and_reraise
raise proxy.with_traceback(exception.traceback) from None
File "/usr/local/lib/python3.7/dist-packages/gin/config.py", line 1582, in gin_wrapper
return fn(*new_args, *new_kwargs)
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/train_util.py", line 242, in train
dataset = data_provider.get_batch(batch_size, shuffle=True, repeats=-1)
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/data.py", line 74, in get_batch
dataset = self.get_dataset(shuffle)
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/data.py", line 248, in get_dataset
filenames = tf.data.Dataset.list_files(self._file_pattern, shuffle=shuffle)
File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/data/ops/dataset_ops.py", line 1349, in list_files
condition, [message], summarize=1, name="assert_not_empty")
File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/util/traceback_utils.py", line 153, in error_handler
raise e.with_traceback(filtered_tb) from None
File "/usr/local/lib/python3.7/dist-packages/tensorflow/python/ops/control_flow_ops.py", line 161, in Assert
(condition, "\n".join(data_str)))
tensorflow.python.framework.errors_impl.InvalidArgumentError: Expected 'tf.Tensor(False, shape=(), dtype=bool)' to be true. Summarized data: b'No files matched pattern: data/train.tfrecord'
In call to configurable 'train' (<function train at 0x7fe885577b00>)
Exporting model...
Traceback (most recent call last):
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/train_util.py", line 165, in get_latest_operative_config
restore_dir, prefix='operative_config-', suffix='.gin')
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/train_util.py", line 106, in get_latest_file
f'No files found matching the pattern '{search_pattern}'.')
FileNotFoundError: No files found matching the pattern '/content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020/operative_config-*.gin'.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/bin/ddsp_export", line 8, in
sys.exit(console_entry_point())
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/ddsp_export.py", line 364, in console_entry_point
app.run(main)
File "/usr/local/lib/python3.7/dist-packages/absl/app.py", line 308, in run
_run_main(main, args)
File "/usr/local/lib/python3.7/dist-packages/absl/app.py", line 254, in _run_main
sys.exit(main(argv))
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/ddsp_export.py", line 333, in main
export_impulse_response(model_path, save_dir, FLAGS.reverb_sample_rate)
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/ddsp_export.py", line 272, in export_impulse_response
ddsp.training.inference.parse_operative_config(model_path)
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/inference.py", line 41, in parse_operative_config
operative_config = train_util.get_latest_operative_config(ckpt_dir)
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/train_util.py", line 168, in get_latest_operative_config
os.path.dirname(restore_dir), prefix='operative_config-', suffix='.gin')
File "/usr/local/lib/python3.7/dist-packages/ddsp/training/train_util.py", line 106, in get_latest_file
f'No files found matching the pattern '{search_pattern}'.')
FileNotFoundError: No files found matching the pattern '/content/gdrive/MyDrive/Train_VST/operative_config-*.gin'.
Export complete! Zipping /content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020/My_Instrument to /content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020/My_Instrument.zip
adding: My_Instrument/ (stored 0%)
Zipping Complete! Downloading... My_Instrument.zip
You can also find your model at /content/gdrive/MyDrive/Train_VST/ddsp-training-2022-09-29-0020/My_Instrument

I'm receiving the same bug - I've been able to determine that it occurs because the dataset creation step fails, thus there are no tfrecords to train the model (hence the training seemingly being skipped).
The error happening during the dataset creation is: "ValueError: Size of each quantile should be size of p: received 1, but expected 360."
Logs for the failing dataset creation:
Creating dataset...
This usually takes around 2-3 minutes for each minute of audio
(10 minutes of training audio -> 20-30 minutes)
I0929 05:04:12.706241<PHONE_NUMBER>48608 environments.py:376] Default Python SDK image for environment is apache/beam_python3.7_sdk:2.41.0
I0929 05:04:12.804227<PHONE_NUMBER>48608 translations.py:714] ==================== <function annotate_downstream_side_inputs at 0x7f1351b54e60> ====================
I0929 05:04:12.804770<PHONE_NUMBER>48608 translations.py:714] ==================== <function fix_side_input_pcoll_coders at 0x7f1351b54f80> ====================
I0929 05:04:12.805119<PHONE_NUMBER>48608 translations.py:714] ==================== <function pack_combiners at 0x7f1351b584d0> ====================
I0929 05:04:12.805743<PHONE_NUMBER>48608 translations.py:714] ==================== <function lift_combiners at 0x7f1351b58560> ====================
I0929 05:04:12.805921<PHONE_NUMBER>48608 translations.py:714] ==================== <function expand_sdf at 0x7f1351b58710> ====================
I0929 05:04:12.806161<PHONE_NUMBER>48608 translations.py:714] ==================== <function expand_gbk at 0x7f1351b587a0> ====================
I0929 05:04:12.806630<PHONE_NUMBER>48608 translations.py:714] ==================== <function sink_flattens at 0x7f1351b588c0> ====================
I0929 05:04:12.806832<PHONE_NUMBER>48608 translations.py:714] ==================== <function greedily_fuse at 0x7f1351b58950> ====================
I0929 05:04:12.808167<PHONE_NUMBER>48608 translations.py:714] ==================== <function read_to_impulse at 0x7f1351b589e0> ====================
I0929 05:04:12.808335<PHONE_NUMBER>48608 translations.py:714] ==================== <function impulse_to_input at 0x7f1351b58a70> ====================
I0929 05:04:12.808505<PHONE_NUMBER>48608 translations.py:714] ==================== <function sort_stages at 0x7f1351b58cb0> ====================
I0929 05:04:12.808886<PHONE_NUMBER>48608 translations.py:714] ==================== <function add_impulse_to_dangling_transforms at 0x7f1351b58dd0> ====================
I0929 05:04:12.809025<PHONE_NUMBER>48608 translations.py:714] ==================== <function setup_timer_mapping at 0x7f1351b58c20> ====================
I0929 05:04:12.809275<PHONE_NUMBER>48608 translations.py:714] ==================== <function populate_data_channel_coders at 0x7f1351b58d40> ====================
I0929 05:04:12.812187<PHONE_NUMBER>48608 statecache.py:172] Creating state cache with size 100
I0929 05:04:12.812885<PHONE_NUMBER>48608 worker_handlers.py:908] Created Worker handler <apache_beam.runners.portability.fn_api_runner.worker_handlers.EmbeddedWorkerHandler object at 0x7f1351a89750> for environment ref_Environment_default_environment_1 (beam:env:embedded_python:v1, b'')
I0929 05:04:12.851229<PHONE_NUMBER>48608 prepare_tfrecord_lib.py:58] Loading 'audio/Piano.wav'.
2022-09-29 05:04:22.884120: W tensorflow/core/common_runtime/gpu/gpu_bfc_allocator.cc:39] Overriding allow_growth setting because the TF_FORCE_GPU_ALLOW_GROWTH environment variable is set. Original config value was 0.
W0929 05:04:44.607767<PHONE_NUMBER>48608 hmm.py:402] MultinomialHMM has undergone major changes. The previous version was implementing a CategoricalHMM (a special case of MultinomialHMM). This new implementation follows the standard definition for a Multinomial distribution (e.g. as in https://en.wikipedia.org/wiki/Multinomial_distribution). See these issues for details:
https://github.com/hmmlearn/hmmlearn/issues/335
https://github.com/hmmlearn/hmmlearn/issues/340
Traceback (most recent call last):
  File "apache_beam/runners/common.py", line 1417, in apache_beam.runners.common.DoFnRunner.process
  File "apache_beam/runners/common.py", line 837, in apache_beam.runners.common.PerWindowInvoker.invoke_process
  File "apache_beam/runners/common.py", line 983, in apache_beam.runners.common.PerWindowInvoker._invoke_process_per_window
  File "/usr/local/lib/python3.7/dist-packages/apache_beam/transforms/core.py", line 1877, in <lambda>
    wrapper = lambda x, *args, **kwargs: [fn(x, *args, **kwargs)]
  File "/usr/local/lib/python3.7/dist-packages/ddsp/training/data_preparation/prepare_tfrecord_lib.py", line 89, in _add_f0_estimate
    audio, frame_rate, viterbi=viterbi, padding=padding)
  File "/usr/local/lib/python3.7/dist-packages/ddsp/spectral_ops.py", line 357, in compute_f0
    verbose=0)
  File "/usr/local/lib/python3.7/dist-packages/crepe/core.py", line 261, in predict
    cents = to_viterbi_cents(activation)
  File "/usr/local/lib/python3.7/dist-packages/crepe/core.py", line 150, in to_viterbi_cents
    path = model.predict(observations.reshape(-1, 1), [len(observations)])
  File "/usr/local/lib/python3.7/dist-packages/hmmlearn/base.py", line 396, in predict
    _, state_sequence = self.decode(X, lengths)
  File "/usr/local/lib/python3.7/dist-packages/hmmlearn/base.py", line 373, in decode
    sub_log_prob, sub_state_sequence = decoder(sub_X)
  File "/usr/local/lib/python3.7/dist-packages/hmmlearn/base.py", line 318, in _decode_viterbi
    log_frameprob = self._compute_log_likelihood(X)
  File "/usr/local/lib/python3.7/dist-packages/hmmlearn/hmm.py", line 471, in _compute_log_likelihood
    X, n=self.n_trials, p=self.emissionprob_[component, :])
  File "/usr/local/lib/python3.7/dist-packages/scipy/stats/_multivariate.py", line 3074, in logpmf
    x, xcond = self._process_quantiles(x, n, p)
  File "/usr/local/lib/python3.7/dist-packages/scipy/stats/_multivariate.py", line 3032, in _process_quantiles
    (xx.shape[-1], p.shape[-1]))
ValueError: Size of each quantile should be size of p: received 1, but expected 360.

Confirmed that rolling back the hmmlearn package to version 0.2.7 by adding the line:
!pip install --upgrade hmmlearn==0.2.7
after installing ddsp into the notebook fixes the issue for now.

Confirmed that rolling back the hmmlearn package to version 0.2.7 by adding the line:
!pip install --upgrade hmmlearn==0.2.7
after installing ddsp into the notebook fixes the issue for now.

this works for me too, thanks

Confirmed that rolling back the hmmlearn package to version 0.2.7 by adding the line:
!pip install --upgrade hmmlearn==0.2.7
after installing ddsp into the notebook fixes the issue for now.

Yes, this worked, thanks.

Confirmed that rolling back the hmmlearn package to version 0.2.7 by adding the line:
!pip install --upgrade hmmlearn==0.2.7
after installing ddsp into the notebook fixes the issue for now.

Yes, this worked, thanks.

Is this fix going to be rolled into the web app? I don't have the chops to run locally...

Confirmed that rolling back the hmmlearn package to version 0.2.7 by adding the line:
!pip install --upgrade hmmlearn==0.2.7
after installing ddsp into the notebook fixes the issue for now.

Yes, this worked, thanks.

Is this fix going to be rolled into the web app? I don't have the chops to run locally...

A fix would need to be made to the main ddsp repository to either work with the new version of the hmmlearn package or to set a version dependency so that it always pulls down a version that doesn't break the project - and then that new version would need to be built/published to PyPi to be able to pull down with pip and the version of the ddsp package that the notebook is pulling would need to be updated. When I get a chance, I can make a PR to that repository to hopefully fix things up, however for now the easiest will be just to click on "Show Code" at the bottom of the notebook cell and paste in the line mentioned above after the section that reads:
print('Installing DDSP...')   print('This should take about 2 minutes...')   !sudo apt-get install libportaudio2 &> /dev/null   !pip install -U ddsp[data_preparation]==3.4.3 &> /dev/null

@ColtonOsterlund You are an angel - thanks!


Link to documentation pdf in README.md broken

The README.md contains a link to the documentation:
http://hci.iwr.uni-heidelberg.de//opengm2/download/opengm-2.0.2-beta-manual.pdf
Which gives an error 404.

Thanks for the feedback.
I forgot to replace these links after we moved the webserver.
I have fixed this.


Feature Request : Require external md(x) & Override

Hi, it's me again :P
To be truly awesome, the last feature for us will be the ability to required markdown/docz project from external package and override header metadata.
For example (quick code... just a sample that can be false xD)) :
module.exports = {
  external: {
    '~/awesome-doc': {
      // load from node_modules/awesome-doc
      '/architecturePattern/event-sourcing.mdx': {
        route: '/architecture/event-sourcing',
        menu: 'Principle',
      },
    },
    '~/my-backend': {
      // load from node_modules/my-backend
      menu: '/project/backend', // inject full docz from my-backend
    },
  },
};

I think the principle above could be implemented differently using a docz.js config file (markdown import for example).
For my company, the idea behind is to capitalize documentation from multiple sources (ui guidelines, internal and external components, architecture pattern, api doc, etc.).
What do you think about that?
(fyi, I'll be on an internal subject "living documentation" today and docz will be a part in my approach, thanks a lot for your great work.)

👍 for that feature.
In my company we need exactly the same, because we have several ui modules and each is a separate node package


Lines with newlines are not being normalized

Take a look at this big "To have a great conversation" string... that uses newlines in it and in the bundle, they are \r\n but on non-Windows we'll be asking for \n... so we need to normalize these newlines.

talking about this one specifically:
const helpPostfix = vscode.l10n.t({
    message: `To have a great conversation, ask me questions as if I was a real programmer:

* **Show me the code** you want to talk about by having the files open and selecting the most important lines.
* **Make refinements** by asking me follow-up questions, adding clarifications, providing errors, etc.
* **Review my suggested code** and tell me about issues or improvements, so I can iterate on it.

You can also ask me questions about your editor selection by [starting an inline chat session](command:inlineChat.start).

Learn more about [GitHub Copilot](https://docs.github.com/copilot/getting-started-with-github-copilot?tool=vscode) in [Visual Studio Code](https://code.visualstudio.com/docs/editor/artificial-intelligence).`,
    comment: "{Locked='](command:inlineChat.start)'}"
});

we use to only support 'and " …neither of these had the ability to do multi-line strings unless you specified the CRLF or LF yourself. This ensured consistency.
template strings, however, are ambiguous due to platform differences… so we have to normalize the strings now to align to a single newline strategy that can be used in either Windows or unix-like.

/duplicate https://github.com/microsoft/vscode-l10n/issues/143

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


fix: automatically install using make if flock isn't installed. and use bashsource to overcome shebang

@bukzor

new PR


Give higher .well-known priority vs nextcloud

Problem
Nextcloud uses a location ^~ /.well-known { block, which will supersede simple location /.well-known/... blocks from this app if they happen to be installed on the same domain.
Solution
Replace this app's block by location = /.well-known, which has higher priority.
cf. https://docs.nginx.com/nginx/admin-guide/web-server/web-server/#nginx-location-priority
PR Status

[x] Code finished and ready to be reviewed/tested
[ ] The fix/enhancement were manually tested (if applicable)

Automatic tests
Automatic tests can be triggered on https://ci-apps-dev.yunohost.org/ after creating the PR, by commenting "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!". (N.B. : for this to work you need to be a member of the Yunohost-Apps organization)

!testme

I have removed my name as maintainer for this app (along with other apps) feel free to take over!

Taking over!


MIR optimization level 3 works again :)

They got fixed by https://github.com/rust-lang/rust/pull/55244.

AppVeyor missed this PR. Let's see if closing and opening again helps...


feat: automatically detect test and script folders for remappings

Component
Forge
Describe the feature you would like
I have test files that import other test files, and prefer using absolute path imports since they are more readable and don't break as often. To do that, I have to add remappings = ["test/=test/"]
With scripting on the way, a root scripts folder with a deploy file will be common, and many users will likely use this to deploy their test suite and therefore will want to import from the script folder. Therefore when testing out #1208 my remappings are now remappings = ["test/=test/", "scripts/=scripts/"]
It feels a bit silly to have to declare those remappings for common, root-level directories, so the feature request here is to include those in the automatic detection to remove the need for that. More generally, it my be worth considering automatically detecting all root level folders.
Additional context
No response

we do something similar with the src folder already where we be default allow a src/=src/ remapping so users can use absolute paths from the src/ dir as imports.
seems like we can solve this by simply extending this with test and scripts?

seems like we can solve this by simply extending this with test and scripts?

Yep, that is exactly the solution I had in mind!


NodeInvocationException: Extra line break is added in exception message

Describe the bug
I am trying to play with Microsoft.AspNetCore.NodeServices.
When JS script ends executing with an error by passing error message as a first argument to callback function then NodeInvocationException  is raised on NET side with a message that have extra line break added to the end of the message.
To Reproduce
Steps to reproduce the behavior:

Using this version of ASP.NET Core '2.2'
Run this code

function calc(callback, a,b) {
	var res = a + b;
	callback('123', res);
	return res;
};
module.exports = {
	calc: calc
};

nodeInstance.InvokeExportAsync<int>("index.js", "calc", 1, 2).GetAwaiter().GetResult();

See error
NodeInvocationException with Message = "123\r\n"

Expected behavior
Exception Message is "123" with no line breaks.
Screenshots

Additional context
The line break is added by NodeInvocationException constructor.
I think the issue can be fixed by checking the details argument for null.
class NodeInvocationException : Exception
{
	public NodeInvocationException(string message, string details)
		: base(details != null
				? message + Environment.NewLine + details
				: message)
	{
	}
}

Thanks for contacting us, @RoboBurned.
This is not an area of active investment at the moment. We would, however, happily consider a PR, if you're up to it.


Are we not allowed to contribute to this Project?

Earlier today i contribute to this project but instead by pull request was closed and by ideas where used,so am just asking if we are not allowed to contribute to this project,why is it here in first place? Thank you

Of course, everyone is welcome to contribute to KickTheOut. It's just that there were some changes we didn't want to make, in your last contribution. @SentinelWarren


为什么在回归任务中，基线模型的效果跟基线模型原文不符合？

We repeat all experiments (including all baselines and our models) five times and report the mean and standard deviation of experimental results.
Detailed experimental settings for baselines can be found in Appendix B.

请问是什么原因让graphsim的基线和原文差距这么大？

We just run the original source code of all baselines and report the experimental results.
For the issues related to graphsim, perhaps you need to run it yourself or ask for help from its authors.

谢谢回答！


New version: Redisant.RedisAssistant version <IP_ADDRESS>

Created by 🥟 Dumplings in workflow run #6006.

Log
Updated: <IP_ADDRESS> → <IP_ADDRESS>
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 886.)


Unexpected behavior when using greaterThanOrEqualTo and inset

New Issue Checklist

[X] I have looked at the Documentation
[X] I have read the F.A.Q.
[X] I have filled out this issue template.

Issue Info

Info
Value

Platform
iOS

Platform Version
12.1

SnapKit Version
4.0.0

Integration Method
CocoaPods

Issue Description
Probably just me being caught off guard on that one, but I was expecting greaterThanOrEqualToSuperview(), and by extension greaterThanOrEqualTo() and related methods, to behave the same as inset() when used together.
Take this:
label.snp.makeConstraints { (make) in
    make.edges.greaterThanOrEqualToSuperview().inset(UIEdgeInsets(top: 24, left: 8, bottom: 24, right: 8))
    make.center.equalToSuperview()
}

The effect I was after was along the line of: I want my right edge to be inset by 8 pt or more. But what ended up happening was: the right edge was constraints to -8 pt or more from its parent right edge. This lead the inner view to use all the space within the insets of its superview instead of shrinking to its content size while being centered.
This behavior makes sense after thinking about it, but it's not straightforward in regard to all inset values being positive.
What I should have used instead (which defeats the purpose of using SnapKit IMO):
label.snp.makeConstraints { (make) in
    make.top.greaterThanOrEqualToSuperview().offset(24)
    make.left.greaterThanOrEqualToSuperview().offset(8)
    make.bottom.lessThanOrEqualToSuperview().offset(-24)
    make.right.lessThanOrEqualToSuperview().offset(-8)
    make.center.equalToSuperview()
}

Am I missing something?
I understand that the behavior of greaterThanOrEqualToSuperview() & co cannot be changed now that it's probably being used in hundreds of apps. But would it make sense to build a new API with the above behavior?

+1


Update mathjax config

This updates the mathjax configuration for mathjax 2.6, and enables the Safe mode.

Trying it locally to confirm.

LGTM, thanks!

Thanks!


Error in types.d.ts for Dictionary

I'm getting a build error on Typescript 3.4.5
export declare type Dictionary<T, K extends string | number = string> = {
    [key in K]: T;
};

Error:
Type 'K' is not assignable to type 'string'.
  Type 'string | number' is not assignable to type 'string'.
    Type 'number' is not assignable to type 'string'

Seems related to this: https://github.com/microsoft/TypeScript/issues/13398

Thanks for the report. I will try to have a look today.

@afreix I can't reproduce it on<EMAIL_ADDRESS>

@krzkaczor We're still seeing this on TS 3.5.3.

@glen-84 can you provide a minimal failing reproduction?

@quezak,
It looks like it's caused by "keyofStringsOnly": true in tsconfig.json.
We don't need this option enabled, so I will remove it.
Thanks.


DC sign in tests

Jira ticket: MWPW-123530
Validates the redirect URLs of Frictionless pages if the visitor is already signed-in.
Test URLs:
https://www.adobe.com/acrobat/online/sign-pdf.html
https://www.adobe.com/acrobat/online/request-signature.html
https://www.adobe.com/acrobat/online/crop-pdf.html
https://www.adobe.com/acrobat/online/delete-pdf-pages.html
https://www.adobe.com/acrobat/online/rotate-pdf.html
https://www.adobe.com/acrobat/online/rearrange-pdf.html
https://www.adobe.com/acrobat/online/split-pdf.html
https://www.adobe.com/acrobat/online/add-pages-to-pdf.html
https://www.adobe.com/acrobat/online/extract-pdf-pages.html
https://www.adobe.com/acrobat/online/pdf-editor.html
https://www.adobe.com/acrobat/online/merge-pdf.html
https://www.adobe.com/acrobat/online/pdf-to-ppt.html
https://www.adobe.com/acrobat/online/pdf-to-jpg.html
https://www.adobe.com/acrobat/online/pdf-to-word.html
https://www.adobe.com/acrobat/online/pdf-to-excel.html
https://www.adobe.com/acrobat/online/convert-pdf.html
https://www.adobe.com/acrobat/online/ppt-to-pdf.html
https://www.adobe.com/acrobat/online/jpg-to-pdf.html
https://www.adobe.com/acrobat/online/word-to-pdf.html
https://www.adobe.com/acrobat/online/excel-to-pdf.html

Hey I love the moving the IMS Login out to a utils method.  In this PR #81 I had to make some temporary bypasses and found a way to address testing on bacom's hlx.live URL for stage testing.  We will just want to make sure that those changes make it into your PR for the imslogin.js utils file.

Where this is just a merge into another feature branch I'm okay with it going forward without a full review.  The subsequent PR with the dc-tests branch going into 'main' will need to be fully reviewed.


Add Nong Song

Song Name
Virtual Idols
Artist Name
remix
Source
Direct file download
Youtube Link
No response
Direct File Link
https://www.mediafire.com/file/uzg1dwybhe12c1w/794165.mp3?dkey=nmg8mhn20nr&r=780
Song ID
794165
Extra
No response

reject
non-YouTube sources are not supported


Add a Sources list to *bazel.ActionInfo. [phab:D1815]

Created by fromberger at 2017-07-28 23:44:50:
Although Spawn actions do not provide an explicit sources list, some of the
other Bazel rule classes do list source files separately. To make it easier to
plumb these through the extraction process, add a field for explicit source
files to the ActionInfo record.
When extracting, any such files are added to the sources list if they are also
selected as inputs by the caller-supplied rules.

zarko accepted this revision at 2017-07-28 23:45:59

fromberger closed via commit 84f5834ab4890bb089acb836731a2ed46be4dffa at 2017-07-29 01:27:23


How to property configure DNS?

I'm not an expert with DNS and I'm having trouble getting the setup working. I own a domain I want to use exclusively for that Bind9 server, let's call it foo.bar. I've run the script in a clean Ubuntu 18.04 Droplet on DigitalOcean. The server is running on port 53:
root@oob:~# netstat -tulpn
Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*               LISTEN      1996/named
tcp        0      0 <IP_ADDRESS>.103:53      <IP_ADDRESS>:*               LISTEN      1996/named
tcp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*               LISTEN      1996/named
tcp        0      0 <IP_ADDRESS>:53           <IP_ADDRESS>:*               LISTEN      582/systemd-resolve
tcp        0      0 <IP_ADDRESS>:22              <IP_ADDRESS>:*               LISTEN      949/sshd
tcp        0      0 <IP_ADDRESS>:953           <IP_ADDRESS>:*               LISTEN      1996/named
tcp6       0      0 :::53                   :::*                    LISTEN      1996/named
tcp6       0      0 :::22                   :::*                    LISTEN      949/sshd
tcp6       0      0 ::1:953                 :::*                    LISTEN      1996/named
udp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*                           1996/named
udp        0      0 <IP_ADDRESS>.103:53      <IP_ADDRESS>:*                           1996/named
udp        0      0 <IP_ADDRESS>:53            <IP_ADDRESS>:*                           1996/named
udp        0      0 <IP_ADDRESS>:53           <IP_ADDRESS>:*                           582/systemd-resolve
udp6       0      0 :::53                   :::*                                1996/named

The part I'm struggling with is actually setting up my domain. The docs say

Note—for this to work without specifying nameservers (i.e. dig A +short foo.bar @ns1.foo.bar)

Let's focus on that first part first. If I run dig @<IP_ADDRESS> random.foo.bar. (the IP is redacted) I get
;; WARNING: recursion requested but not available

and also

;gh.foo.bar.      IN  A

Shouldn't that resolve to the same IP?
The second part is where my lack of DNS knowledge comes in.

you would need your domain provider to have the domain point to your custom domain for example:

Do I do that with a glue record? Because that's what I've done (having ns1.foo.bar and ns2.foo.bar point to the droplet). I can confirm that the glue record works by asking the nameserver on the bar. level. Do I need to have a SOA as well? But I assume that's something the Bind9 server would provide anyway, right?
So I assume my DNS setup is correct (the root level nameserver knows about my ns1 and ns2) but the server is not installed/configured properly. If I run a plain dig random-stuff.foo.bar I get the same empty ";ffewf.oob.party.    IN  A
" line as above when I queried it directly from the server.
Edit: I just realized both dig and host also say "SERVFAIL"
Edit2: Forgot to mention the logs. A named.log does not exist, but syslog has
Apr  1 16:21:08 oob named[1996]: client @0x7fb1281c7240 <IP_ADDRESS>#10130 (ns2.oob.party): query (cache) 'stuff.foo.bar/A/IN' denied

so their might be a permission issue? Or maybe the Bind9 config is missing something?

I did some more digging. I directly asked my server for www.google.com and it worked. Asking for my own domain however the logs now say
query failed (SERVFAIL) for baz.foo.bar/IN/A at ../../../bin/named/query.c:8402

I also have a named.log now, I believe it is because I restarted Bind9 (the setup script likely needs an service bind9 restart or whatever).
Maybe this is an Ubuntu thing and the Bind9 in the repo is borked?

Hey @Prinzhorn -- Unfortunately it's been a while since I ran this, I'll try to find some time this weekend to reproduce it again. But let me try to help.

Do I do that with a glue record? Because that's what I've done (having ns1.foo.bar and ns2.foo.bar point to the droplet). I can confirm that the glue record works by asking the nameserver on the bar

Generally you would need to go to your registrar (e.g. GoDaddy) and tell them that for your domain, the nameservers should be ns1.foo.bar and ns2.foo.bar instead of whatever resolver they opt to use. From that point, you go to DigitalOcean and have those records point to the IP of your droplet.
The fact that querying dig +short google.com @ns1.foo.bar works just fine but not something like dig +short random-payload.foo.bar is.. weird.
It could be that the config isn't correct or something is missing. Unfortunately right now I can't take too much of a look into it but I'll try to get to it. In the meantime keep me updated and I'll gladly reply to further questions.

Could you show me what you configured for the following https://github.com/JuxhinDB/OOB-Server/blob/692ff6974c9348e4fdca741da324992cfb81622a/_config/db.local#L1-L14

Here's what the script created (I just realized that there's the SOA as well, which I was looking for)
$TTL    120
@       IN      SOA     foo.bar. root.foo.bar. (
                              2         ; Serial
                         604800         ; Refresh
                          86400         ; Retry
                        2419200         ; Expire
                         604800 )       ; Negative Cache TTL
;
ns1              IN      A       <IP_ADDRESS>
ns2              IN      A       <IP_ADDRESS>
@                IN      A       <IP_ADDRESS>
*                IN      CNAME   foo.bar
@                IN      NS      ns1.foo.bar.
@                IN      NS      ns2.foo.bar.

At face-value that looks fine. I'll try to take a look at this; meanwhile let me know if you figure it out. For instance, the zone block above should be something that the script handles automatically ideally.

I found a setup that works for me. Two things I had to change

Create a zone for the domain instead of using localhost

zone "foo.bar" {
        type master;
        file "/etc/bind/db.foo.bar";
};

Remove the CNAME and other duplicate stuff

$TTL    120
@       IN      SOA     foo.bar. root.foo.bar. (
                              2         ; Serial
                         604800         ; Refresh
                          86400         ; Retry
                        2419200         ; Expire
                         604800 )       ; Negative Cache TTL
;
*                IN      A       <IP_ADDRESS>
@                IN      NS      ns1.foo.bar.
@                IN      NS      ns2.foo.bar.

I don't know up to which level this is compatible with your script. I also removed all other zones and forwarders. I don't want my nameserver to resolve anything but foo.bar.

Generally you would need to go to your registrar (e.g. GoDaddy) and tell them that for your domain, the nameservers should be ns1.foo.bar and ns2.foo.bar instead of whatever resolver they opt to use. From that point, you go to DigitalOcean and have those records point to the IP of your droplet.

Also for the record if anyone comes across this. As you've described it you end up with a chicken-egg issue. You say ns1.foo.bar is your new nameserver but nobody is able to resolve ns1.foo.bar because you can't connect to it before resolving it first. So I think that's what glue records are for and like I said I created ns1 and ns2 glue records and configured these two as nameservers for my domain.

That's fine, thanks for letting me know. I'll try this out and see what I can do. I could add another flag in the setup to decide whether or not to be an open resolver.


List available languages for project

Proposed API update here https://github.com/UNIwise/parrot/tree/project-meta-endpoint

Great idea!
A couple of requests, though:

updatedAt value for every language instead of a general updatedAt value for the entire project
translation percentage to show whether a language is actually complete or usable

Both should be possible with https://poeditor.com/docs/api#languages_list


ZIP Code data -- from geonames.org

Type of Feature:

[X] Collaboration or partnership
[X] Improvement or clarification
[X] New Processing

Description of Feature
Pull in Zip code entries from geonames.org as taggable text -- supported on-demand, not by default.
given the style of text it would make more sense to create "postal" index, apart from gazetteer index.

postal index created in ./solr;  PostalGeocoder created along side PlaceGeocoder


Conflict with indentLine

IndentLine and vim-lambdify are using the same hightlight:

It's possible to use different colors?

Interesting. I haven't looked very deeply into coloration yet. I suspect that the syn match that matches the lambda could be keyed off of for color. In most languages, the color seems appropriate enough that I just haven't tried changing it. I can see how in the above its a bit odd.
Let me know what you find. I suspect this to be more of a vimrc one liner or a change to vim-polyglot java syntax file.

I will keep you updated. I did some tests, but no solution until now.

After many attempts I haven't found any solution for this plugin to work together with indentLine, since they share the same highlight for conceal.
I'm using vim-indent-guides that works fine with this plugin.
I will close this issue for now.


Blurred display of images

Image/Screenshot is blurred in the accordion plugin. Text cannot be read.

Please also consider our blog page. We have some new graphics which would need (almost) full width and a bigger height - similar to the  like the title image. So the picture display options in the main text should be adaptable to different sizes.
Example of big/small images: https://unikube.io/blog/new-features-django-hurricane/

This image should be visible (text readable) for example

@tobifroe please also check this page for image resolution:
https://unikube.io/concept-draft/


Why theading not work

When I use Theading as follow. My expected printed message is 『Do something first!』『After do something!』，but the result runing is opposite. Is there anything wrong about Threading ? And How could I perform in main thread?
func handler(request: HTTPRequest, response: HTTPResponse) {
let event = Threading.Event()
Threading.dispatch {
    Threading.sleep(seconds: 3)
    print("Do something first!")
    event.signal()
}

_ = event.wait()
print("After do something!")

// Respond with a simple message.
response.setHeader(.contentType, value: "text/html")
response.appendBody(string: "<title>Hello, world!</title>Hello, world!")
// Ensure that response.completed() is called when your processing is done.
response.completed()

}

You have to call .lock() before calling signal or wait, followed by .unlock().
It's described more in this document:
https://www.perfect.org/docs/thread.html

@kjessup I have try adding lock to it , but it is also not working.


Doc Updates - November Release

Features and its doc updates. @gkulin please work with respective folks.

 Will be good for us to update the doc to include them @weikanglim can you help @gkulin on that?

Submitted https://github.com/MicrosoftDocs/azure-dev-docs/pull/1283 as a suggested improvement we could make


Fix alternate mnemonic

I believe what's there is wrong, given the GameBoy CPU manual and a few other references, including the above opcode table.

I still think this is an issue but I want stale PRs off my PR list.  Thanks!


fix: expose backup-crn variable in fscloud

Description
Issue: https://github.ibm.com/GoldenEye/issues/issues/7845

Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

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