2024-02-29 Issues with Daily-Note navigation links after plugin update

Hi @mathisgauthey!
And thanks for the template. I've been testing it out for few days and it feels great for my use case. I was just about to migrate fully, but after I updated the plugins today I'm experiencing issues with the Daily-Note navigation links e.g. these:

When clicking any of the links (2024/Q1/February/Week 09) It creates a page and the template quickly flashes/shows but after that it's an empty page:

"Open today" works for the left sidebar
Command palette "Periodic Notes" commands seem to work
This is an empty vault (no modifications), version 1.0.5
Plugins bulk updated to the latest versions using the settings menu

Any ideas how to fix and what could cause this? I'd like to start using the template and this functionality is handy on the daily note.
Thanks!

Hey there, I just cloned the repository on last version using these versions :
SYSTEM INFO:
	Obsidian version: v1.5.8
	Installer version: v1.4.16
	Operating system: Windows 10 Pro 10.0.22631
	Login status: logged in
	Catalyst license: none
	Insider build toggle: off
	Live preview: on
	Base theme: adapt to system
	Community theme: AnuPpuccin v1.4.4
	Snippets enabled: 4
	Restricted mode: off
	Plugins installed: 39
	Plugins enabled: 39
		1: Style Settings v1.0.7
		2: Periodic Notes v0.0.17
		3: Omnivore v1.8.1
		4: Kanban v1.5.3
		5: Calendar v1.5.10
		6: Templater v1.16.0
		7: Commander v0.5.1
		8: Scroll to Top Plugin v2.1.3
		9: Iconize v2.6.0
		10: File Explorer Note Count v1.2.1
		11: Filename Heading Sync v1.9.0
		12: Colorful Tag v1.3.2
		13: Code Editor Shortcuts v1.14.0
		14: Find orphaned files and broken links v1.9.1
		15: Natural Language Dates v0.6.1
		16: Paste URL into selection v1.7.0
		17: Shortcuts extender v2.2.0
		18: Archiver v0.26.1
		19: Auto Link Title v1.4.1
		20: Better footnote v1.0.1
		21: Better Word Count v0.10.0
		22: Callout Manager v1.0.1
		23: Clear Unused Images v1.1.0
		24: Completr v3.2.0
		25: Dataview v0.5.64
		26: Editing Toolbar v2.4.0
		27: Footnote Shortcut v0.1.3
		28: Github Publisher v6.3.2
		29: Heatmap Calendar v0.6.0
		30: Image Toolkit v1.4.0
		31: Linter v1.20.0
		32: Note Refactor v1.8.2
		33: Paste image rename v1.6.1
		34: Show Whitespace v0.3.1
		35: Symbols Prettifier v1.1.1
		36: Tag Wrangler v0.6.1
		37: Tasks v5.5.0
		38: Todoist Plugin v1.11.1
		39: QuickAdd v1.4.0

RECOMMENDATIONS:
	Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
	Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

I can't reproduce the issue on a freshly cloned vault with the last Obsidian version, I probably won't be able to help more now.
But I'd suggest to :

Check if Templater has the Trigger Templater on file creation and is setup on Filename-Template file.
Filename heading sync plugin has Use file Open Hook disabled

Maybe try to disable every plugin except Templater and Periodic Note ? If it works, then a plugin is causing the issue. So try by activating half, and then another half using a divide and conquer strategy to find the potential plugin that causes the issue ?

Hey! And thanks for the reply.
I just extracted the latest vault version and here's my starting point after updating the plugins:
SYSTEM INFO:
	Obsidian version: v1.5.8
	Installer version: v1.5.3
	Operating system: Windows 10 Enterprise 10.0.22631
	Login status: not logged in
	Insider build toggle: off
	Live preview: on
	Base theme: adapt to system
	Community theme: AnuPpuccin v1.4.5
	Snippets enabled: 4
	Restricted mode: off
	Plugins installed: 39
	Plugins enabled: 39
		1: Periodic Notes v0.0.17
		2: Templater v2.2.1
		3: Archiver v0.26.1
		4: Auto Link Title v1.5.3
		5: Better footnote v1.0.1
		6: Better Word Count v0.10.1
		7: Calendar v1.5.10
		8: Callout Manager v1.0.1
		9: Clear Unused Images v1.1.0
		10: Code Editor Shortcuts v1.14.0
		11: Colorful Tag v1.3.2
		12: Commander v0.5.1
		13: Completr v3.2.0
		14: Dataview v0.5.64
		15: Editing Toolbar v2.4.0
		16: File Explorer Note Count v1.2.1
		17: Find orphaned files and broken links v1.9.1
		18: Footnote Shortcut v0.1.3
		19: Github Publisher v7.0.2
		20: Heatmap Calendar v0.6.0
		21: Iconize v2.10.1
		22: Image Toolkit v1.4.1
		23: Kanban v1.5.3
		24: Linter v1.23.1
		25: Natural Language Dates v0.6.2
		26: Note Refactor v1.8.2
		27: Omnivore v1.9.4
		28: Paste image rename v1.6.1
		29: Paste URL into selection v1.7.0
		30: QuickAdd v1.7.0
		31: Scroll to Top v2.1.4
		32: Shortcuts extender v2.2.0
		33: Show Whitespace v0.3.1
		34: Style Settings v1.0.7
		35: Symbols Prettifier v1.1.1
		36: Tag Wrangler v0.6.1
		37: Tasks v6.0.0
		38: Todoist Sync v1.12.0
		39: Filename Heading Sync v1.9.0

RECOMMENDATIONS:
	Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
	Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

I started to disable plugins and with only Templater and Periodic Notes everything seem to work when clicking the link e.g. "Week 09":
SYSTEM INFO:
	Obsidian version: v1.5.8
	Installer version: v1.5.3
	Operating system: Windows 10 Enterprise 10.0.22631
	Login status: not logged in
	Insider build toggle: off
	Live preview: on
	Base theme: adapt to system
	Community theme: AnuPpuccin v1.4.5
	Snippets enabled: 4
	Restricted mode: off
	Plugins installed: 39
	Plugins enabled: 2
		1: Templater v2.2.1
		2: Periodic Notes v0.0.17

The page gets created, has content and looks normal:

Then I tried to enable Filename Heading Sync and the issue is back:
SYSTEM INFO:
	Obsidian version: v1.5.8
	Installer version: v1.5.3
	Operating system: Windows 10 Enterprise 10.0.22631
	Login status: not logged in
	Insider build toggle: off
	Live preview: on
	Base theme: adapt to system
	Community theme: AnuPpuccin v1.4.5
	Snippets enabled: 4
	Restricted mode: off
	Plugins installed: 39
	Plugins enabled: 3
		1: Templater v2.2.1
		2: Periodic Notes v0.0.17
		3: Filename Heading Sync v1.9.0

The page gets created but it's empty.

Here's the settings for Filename Heading Sync - I didn't touch anything:

And here's the default Filename-Template.md
<%*
let templater = app.plugins.plugins["templater-obsidian"];
let templateFolder = templater.settings.templates_folder;

/**
 * format:   momentjs format strings for filename
 * template: name of template
 *
 * If the file name matches the format, respective
 * template will be inserted.
 */
let noteTypes = [
  { format: "YYYY-MM-DD", template: "Daily-Notes.md" },
  { format: "GGGG-[W]WW", template: "Weekly-Notes.md" },
  { format: "YYYY-MM", template: "Monthly-Notes.md" },
  { format: "YYYY-[Q]Q", template: "Quarterly-Notes.md" },
  { format: "YYYY", template: "Yearly-Notes.md" },
  { format: "2-Areas", template: "Areas.md" },
  { format: "0-Inbox", template: "Untitled.md" },
];

for (const noteType of noteTypes) {
  // If found a matching note title
  if (moment(tp.file.title, noteType.format, true).isValid()) {
    // Get TFile of template
    let templateTFile;
    try {
      // Get TFile
      templateTFile = app.vault.getAbstractFileByPath(
        `${templateFolder}/${noteType.template}`
      );

      // Error handling
      if (!templateTFile) {
        let msg =
          "Template not found\n" +
          `${templateFolder}/${noteType.template} does not exist.`;
        new Notice(msg);
        throw Error(msg);
      }
    } catch (e) {
      console.error(e);
      return;
    }

    // Insert template
    templater.templater.append_template_to_active_file(templateTFile);
    break; // Task completed, leaving the loop
  }

  // If found a matching note title
  else if (tp.file.folder(true) == noteType.format) {
    // Get TFile of template
    let templateTFile;
    try {
      // Get TFile
      templateTFile = app.vault.getAbstractFileByPath(
        `${templateFolder}/${noteType.template}`
      );

      // Error handling
      if (!templateTFile) {
        let msg =
          "Template not found\n" +
          `${templateFolder}/${noteType.template} does not exist.`;
        new Notice(msg);
        throw Error(msg);
      }
    } catch (e) {
      console.error(e);
      return;
    }

    // Insert template
    templater.templater.append_template_to_active_file(templateTFile);
    break; // Task completed, leaving the loop
  }
}
%>

I understand it's hard to debug if you cant reproduce the issue. Let me know if you spot anything weird or have any ideas what I could try next, thanks!

Alright, thanks for the detailed message.
I was able to reproduce by updating all plugins. I'm looking for a fix right now, I'll keep you updated here.

Hello! I've found a solution for the template application error occurring in the latest version of Templater. The issue seems to arise from the concurrent execution of 1. template application and 2. file movement. So I tried to clearly separate these two steps. Here's the example code:
[Filename-Template.md]
<%*
let templater = app.plugins.plugins["templater-obsidian"];
let templateFolder = templater.settings.templates_folder;

// Template configurations
const noteTypes = [
  { format: "YYYY-MM-DD", template: "Daily-Notes.md" },
  { format: "GGGG-[W]WW", template: "Weekly-Notes.md" },
  { format: "YYYY-MM", template: "Monthly-Notes.md" },
  { format: "YYYY-[Q]Q", template: "Quarterly-Notes.md" },
  { format: "YYYY", template: "Yearly-Notes.md" },
  { format: "2-Areas", template: "Areas.md" },
  { format: "0-Inbox", template: "Untitled.md" },
];

// Helper function to get template file
async function getTemplateFile(templatePath) {
  const templateFile = app.vault.getAbstractFileByPath(templatePath);

  if (!templateFile) {
    const msg = `Template not found: ${templatePath} does not exist.`;
    new Notice(msg);
    throw new Error(msg);
  }

  return templateFile;
}

// Helper function to check if note matches format
function isMatchingNote(noteTitle, format) {
  return moment(noteTitle, format, true).isValid() || tp.file.folder(true) === format;
}

// Main function to process template
async function processTemplate() {
  try {
    for (const noteType of noteTypes) {
      if (isMatchingNote(tp.file.title, noteType.format)) {
        const templatePath = `${templateFolder}/${noteType.template}`;
        const templateFile = await getTemplateFile(templatePath);
        await templater.templater.append_template_to_active_file(templateFile);
        return; // Exit after successful template insertion
      }
    }
  } catch (error) {
    console.error('Template processing failed:', error);
    new Notice(`Template processing failed: ${error.message}`);
  }
}

// Execute
await processTemplate();
%>`

[Daily Notes.md]
<%*
async function applyTemplate() {
    try {
        // 1. Validate the date format
        const currentMoment = moment(tp.file.title, "YYYY-MM-DD", true);
        if (!currentMoment.isValid()) {
            new Notice("Invalid date format. Expected YYYY-MM-DD");
            return "";
        }

        // 2. Add initial delay before file movement
        await new Promise(resolve => setTimeout(resolve, 300));

        // 3. Move the file to target location
        const targetPath = "/-Daily-Notes/" + tp.file.title;
        await tp.file.move(targetPath);

        // 4. Wait for file movement to complete
        await new Promise(resolve => setTimeout(resolve, 300));

        // 5. Generate template content
        let content = "";

        // Main date header
        content += "# " + currentMoment.format("YYYY-MM-DD") + "\n";

        // Hierarchical navigation
        content += "❮ ";
        content += "[[" + currentMoment.format("YYYY") + "]]" + " / ";
        content += "[[" + currentMoment.format("YYYY-[Q]Q|[Q]Q") + "]]" + " / ";
        content += "[[" + currentMoment.format("YYYY-MM|MMMM") + "]]" + " / ";
        content += "[[" + currentMoment.format("GGGG-[W]WW|[Week] WW") + "]]";
        content += " ❯\n\n";

        // Day navigation
        content += "❮ ";
        content += "[[" + moment(currentMoment).subtract(1, "days").format("YYYY-MM-DD|dddd Do") + "]]" + " | ";
        content += currentMoment.format("dddd Do") + " | ";
        content += "[[" + moment(currentMoment).add(1, "days").format("YYYY-MM-DD|dddd Do") + "]]";
        content += " ❯\n\n";

        // Tasks sections
        content += `> [!CHECK]+ Super Productivity ☑
>
> \`\`\`tasks
> not done
> (happens on or before 2024-07-30) OR (status.type is IN_PROGRESS)
> (heading does not include Focus) AND (heading does not include Goals)
> group by function (task.isRecurring) ? '%%3%% Recurring Tasks' : result = task.description.includes("[[") ? '%%1%% Projects' : '%%2%% Tasks'
> \`\`\`

> [!IMPORTANT]+ Short Next Actions 🏃
>
> \`\`\`tasks
> not done
> is not recurring
> description does not include ]]
> tags include #next
> group by function task.tags.filter( (tag) => tag.includes("#next") )
> short mode
> \`\`\`\n`;

        // Notes section
        content += "## Notes 📝\n";

        // 6. Add final delay before content application
        await new Promise(resolve => setTimeout(resolve, 300));

        return content;

    } catch (error) {
        console.error("Template Error:", error);
        new Notice(`Template Error: ${error.message}`);
        return "";
    }
}

// Execute the template
tR = await applyTemplate();
%>

(The example templates might slightly differ from your original templates.) While this approach makes the template application slightly slower, it seems to work stably even in the latest version of Templater (version 2.9.0 in my case).
(P.S. I've been using your Obsidian workflow with great satisfaction. Thank you for sharing it with the community.)

Hey there, thanks for the correction, feel free to submit a PR to review and I'll add it to the next version of the workflow !
I'm glad my work helped you somehow, thanks for your kind words !


prehashing before ed25519

This can be replaced by the changes in https://github.com/tendermint/go-crypto/pull/85 (should work already); ideally we import go-crypto into the demo client.


[Event Request] Codeunit 80 "Sales-Post" PostInvoicePostBuffer

"OnAfter" Event Publisher required inside PostInvoicePostBuffer procedure after JobPostLine.PostSalesGLAccounts is called:
local procedure PostInvoicePostBuffer(SalesHeader: Record "Sales Header"; var TempInvoicePostBuffer: Record "Invoice Post. Buffer" temporary)
var
    LineCount: Integer;
    GLEntryNo: Integer;
begin
    OnBeforePostInvoicePostBuffer(SalesHeader, TempInvoicePostBuffer, TotalSalesLine, TotalSalesLineLCY);

    LineCount := 0;
    if TempInvoicePostBuffer.Find('+') then
        repeat
            LineCount := LineCount + 1;
            if GuiAllowed and not HideProgressWindow then
                Window.Update(3, LineCount);

            GLEntryNo := PostInvoicePostBufferLine(SalesHeader, TempInvoicePostBuffer);

            if (TempInvoicePostBuffer."Job No." <> '') and
                (TempInvoicePostBuffer.Type = TempInvoicePostBuffer.Type::"G/L Account")
            then
                JobPostLine.PostSalesGLAccounts(TempInvoicePostBuffer, GLEntryNo);

            //New publisher:
            //OnPostInvoicePostBufferOnAfterPostSalesGLAccounts(SalesHeader, TempInvoicePostBuffer, TotalSalesLine, TotalSalesLineLCY, GenJnlLineDocNo, GLEntry No, SrcCode);

        until TempInvoicePostBuffer.Next(-1) = 0;

    OnPostInvoicePostBufferOnBeforeTempInvoicePostBufferDeleteAll(
        SalesHeader, GenJnlPostLine, TotalSalesLine, TotalSalesLineLCY, GenJnlLineDocType, GenJnlLineDocNo, GenJnlLineExtDocNo, SrcCode);
    TempInvoicePostBuffer.DeleteAll();
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Reader get stunk on mem.getLongVolatile when calling next()

I can run sample using Object fine, bu I can't seem to be able to implement the same on my own project.
My reader next() function always return false on
if (mem.getLongVolatile(Structure.Limit) <= limit) { return false; }
I am not sure what is the purpose of this function and also I can't seem to figure out why my project is different than the sample.
My project create the Reader and Writer in the same process.

That's the check to see if anything has been written to the file.  Have you called the write method on writer to the same file?
If you send a minimal test code showing your issue I'll check.

Finally my problem was how I implemented the writer, I was not updating position (pos +4, pos + 8, etc.).


Use the value of the content-type header for the message's content-type in SimpleMessageConverter

AMQP-819: Use the value of the content-type header for the message's content-type in SimpleMessageConverter.

JIRA: https://jira.spring.io/browse/AMQP-819

Additional test case.

I am not convinced this is needed - you can already do this with a MessagePostProcessor
template.convertAndSend(exchange, key, data, m -> {
    m.getMessageProperties.setContentType(...);
    return m;
}

If we do go with your approach, it would have to be a well-defined header name (in AmqpHeaders rather than a literal "content-type").

@garyrussell Could you please explain more, how can i configure this MessagePostProcessor? I've tried Create a Bean, without success :(
import org.springframework.amqp.core.MessagePostProcessor;
@Component
public class MyMessagePostProcessor implements MessagePostProcessor {

    @Override
    public Message postProcessMessage(Message message) throws AmqpException {
        message.getMessageProperties().setContentType("application/json");
        return message;
    }
}

You can either invoke it inline as in my example above, or you can configure it into the RabbitTemplate...
/**
 * Set {@link MessagePostProcessor}s that will be invoked immediately before invoking
 * {@code Channel#basicPublish()}, after all other processing, except creating the
 * {@link BasicProperties} from {@link MessageProperties}. May be used for operations
 * such as compression. Processors are invoked in order, depending on {@code PriorityOrder},
 * {@code Order} and finally unordered.
 * @param beforePublishPostProcessors the post processor.
 * @since 1.4.2
 */
public void setBeforePublishPostProcessors(MessagePostProcessor... beforePublishPostProcessors) {

@garyrussell You can't do it if you're using Spring Cloud Streams because RabbitMessageChannelBinder creates RabbitTemplate instances on the fly. Therefore, I think it's a good idea to allow that header. I agree with you that the header should be well defined in AmqpHeaders, though.

Thanks @garyrussell! That solved my issue

@rydnr , @arthurpessoa ,
doesn't that sound like we can close this with a "Works as Designed" resolution?
Thanks

Sure! Thanks everyone!


Dropbox 401

Hey,
Thanks for the great work.
I use draw.io for multi level flowcharts, and connect an item in the parent flowchart to the details by a link to detail file in dropbox, when I do that, it just works fine, but when after some time I come back, open the master file in draw.io from my dropbox account, and in the master file click on a link to open the detail file  I get this error:

Error (401)
It seems you don't belong here! Check out our Help Center and forums for help, or head back to home.

seems to have something to do with dropbox implementation in draw.io, since it works well at the moment when I create the link, also the file still exist there, and when I edit the link, select the file from dropbox again, everything works fine, until some time later it all starts happening again.

This error means you're not authorized to view the file. What link are you using to reference the file, ie. is it a link to draw.io or dropbox.com?

Please note we'd prefer general usage question on the Google Group and to define the scope of Github issues as being development related.

to create the link,
I right-click on the item I want to link it, select Edit-Link, then Dropbox, and then from the dropbox window, I select my other draw.io file (which I saved earlier) ...
the result would be a dropbox link.
am I doing something wrong? or is it a problem with your implementation of dropbox api?!


C attributes

I just enabled documentation warnings in Xcode, and it started complaining a lot about the Doxygen documentation in glfw3.h. One of these complaints, is that types and functions marked as deprecated for Doxygen, are not attributed as deprecated in C. Xcode suggests a fix which prepends the declaration with __attribute__((deprecated)). This is supported by Clang and GCC, but not MSVC, however, that's solved with macros. I think adding these attributes is a reasonable enhancement to GLFW.
C23 adds built-in attributes. This is a better option, but is obviously not supported in MSVC either. Should be solvable the same way.

Perhaps it's better to remove the Doxygen tag until there's a (3.x or 4.0) replacement?

No, no, absolutely not. This is just a warning that’s generated. It’s safe to ignore. Removing a tag stating that something deprecated is deprecated, serves no purpose. That would just worsen the situation. Adding these attributes is about giving more direct hints to the compiler and analyzers, thereby helping the developers more. Removing these tags would only take away useful information. It would also introduce very confusing situations, with an old API almost identical to a new API, with no obvious reason why two almost identical APIs exist.


pregnancy-episode-of-care

Draft pull request to look for a patient's pregnancy status via the EpisodeOfCare FHIR resource. This PR add's EpisodeOfCare to the Patient History, and also specifically adds logic to the Pregnant expression to consider a patient pregnant if they have an active EpisodeOfCare.

CQL to ELM Translation warning:

Warning:[n/a] List-valued expression was demoted to a singleton.

Added tests two tests in https://github.com/ccsm-cds-tools/ccsm-cds-with-tests/pull/119/commits/2462256d42d3a13ffd53bb365a55d8543094388b, one for an active pregnancy, and one for a finished pregnancy.

The CQL to ELM translation warning was brought up on Zulip. Other than resolving that warning, this PR is ready for review @yunwwang!

Figured out a way to make the EpisodeOfCare retrieve without the warning being thrown. This should also be a better solution, because a Pregnancy Episode of Care will be identified if the pregnancy code is present at any point within the type array, as opposed to only if it is present in the first element of the array. I will monitor the Zulip thread linked above, to add a more elegant solution if one is proposed.

All comments were resolved, so I'm going to go ahead and merge this. We can revisit if we eventually want to make the Episode of Care retrieve more efficient.


Using busboy as a middleWare

Hi. I am new to NodeJs. How can I use busboy as a middleware. I need s3 url of file after it was uploaded. Because I will do some database operations with the url.
var s3UploadService = require('../util/Aws-Streaming');
app.post("/testUpload", function (req, res) {
  return s3UploadService(req, function (err, url) {
    // Do some database operations.
  });
});

var s3UploadService = require('../util/Aws-Streaming');
app.post("/testUpload", s3UploadService, function (req, res) {
  // I can not pass req parameter to s3UploadService this way.
});

Either way I couldn't use busboy as middleware. If you can help, I will be appreciated. Thank you.

Don't use it as a middleware, just use it in your route like here : https://github.com/thaume/s3-streaming/blob/master/routes/index.js#L7
Then you can get the URL back from the callback.

Thank you so much.


Make MySQL an optional dependency of InLong-Manager

Description
The parent issue is:https://github.com/apache/incubator-inlong/issues/2281
Based on the parent requirements, according to the requirements of ASF 3RD PARTY LICENSE POLICY, we need to replace them or change the current non-standard third-party mandatory dependencies into non-mandatory dependencies. The InLong-Manager module is forced to rely on MySQL, which needs to be adjusted.
This issue tracks modification handling in this regard.
InLong Component
InLong Manager
Are you willing to submit PR?

[ ] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

duplicated with https://github.com/apache/incubator-inlong/issues/2784, close it.


fix(pinia-orm): Data was wrong for belongsTo & hasOne with composite key

🔗 Linked issue
Resolves #1903

📚 Description

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

Project coverage is 99.72%. Comparing base (93359fd) to head (d841a52).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1904   +/-   ##
=======================================
  Coverage   99.72%   99.72%
=======================================
  Files          90       90
  Lines        6265     6272    +7
  Branches      528      529    +1
=======================================
+ Hits         6248     6255    +7
  Misses         14       14
  Partials        3        3

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


added different hosts and binary uploads

Adding Ads API specific info according to the thread here:
https://twittercommunity.com/t/setting-up-twurl/35390

This PR only touches the README so should not have affected the build.

lgtm but I'm not sure why it affected the Travis build

should i resubmit this?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Manually added to #133


Do not bind  hosts at all and do not install at new iis  on win 11

Before you post
Make sure to search through the issue list to check if others have reported the same bug already. If you find an existing issue, make your voice heard there. Your case may provide valueable extra clues towards a solution and will increase its priority.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce

Run with command line ...
Pick menu options A, B, C
See error

Expected behavior
A clear and concise description of what you expected to happen.
Log
Please add any available log output. You may copy it from the console window, get it from the Windows Event Viewer, or the log files on disk, which can typically be found in a sub folder of %programdata%\win-acme.
Platform:

OS: [e.g. Windows 2016, English]
Version: [e.g. 2.1.0, 64-bit, pluggable]

Additional context
Add any other context about the problem here, for example possible network issues (firewalls, proxies, NAT) that might play a role.

You've literally provided no information other than the title of this issue. Please try again when you have some log output and a more thourough description of your issue.

at that moment, I cant open log file because of permission error  and can't change permissions under administrator rights. I'll try later

How I can upload and add file, or i should make excerpts from a file


rename file always failed

We have written the needed data into your clipboard because it was too large to send. Please paste.

@LitterB It looks like the data needed to be pasted into the issue. Can you please edit the issue to include information about the problem you're seeing?


[ Layer ] More Parallelization along batch direction

This patch includes more parallelization along batch direction.

Forward function of Pooling Layer
calGradient function of LSTM Layer

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1979. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


[WIP] Rehaul v2

This will need one/two follow-ups but I think we better start merging back in

🎉

@barambani not sure why snyk now reports again two issues from the aws sdk that previously did not report… anyhow… happy to merge?

I'm not sure why Snyk didn't fail on this week's PRs. It could be a glitch. I actually was expecting it to keep failing as we still depend on an affected version. Anyway let's merge, yes.


Can't update token info in solana.tokenlist.json file

I'm submitting a ...
[X ] bug report
[ ] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary
I am trying to update the logo path in the solana.tokenlist.json file but get the error "Error: Pull request should not contain any deletions." I believe that this is because that file doesn't allow any edits to be made once the original info is merged.

This means that if you make a mistake in the tokenlist file, you can't fix it. If the URL to your token or some other info changes, they you are stuck and would have to re-mint your token. This seems very restrictive.
Can someone please let me know if there's a work-around?
Thanks.

Other information (e.g. detailed explanation, stack traces, related issues, suggestions how to fix, links for us to have context, eg. StackOverflow, personal fork, etc.)

Please read the SPL-token README, and ensure your SPL-token info uses the correct Uniswap format.

You are correct, this requires manual review. Please link a PR


Noob question, what's a model?

Hey, in your docs, you keep loading a model but never show us what a User model looks like. I can't really figure out how to use this library without knowing what the model needs to look like (from which we would build the model resource like: UserResource)
Any more details would be great, thanks!

@anlek @edmundo096 A Model is an object that defines the necessary operations for database retrieval/storage of resource objects. Currently, only mongoose models are supported, but I planned jsonapify to be backend agnostic somewhere in the future (maybe).
Regarding usage, do you find the information in the readme insufficient? What else would you like to see? As a workaround, you can always take a look at the test folder, though that's not an excuse for proper documentation, I know :sweat_smile:

@alex94puchades Indeed, looking at the test directory will help understanding the actual implementations and uses.
The Readme has a good introduction but for a quick-look search for "why I should use this library" or "how this library will help me", was not enough in my humble opinion.
I will take a look to this library a bit more in the future. Thanks for the reply!


What about ios compatibility. Will app store let me to publish it?

Is there anyone that have published app in app store with this plugin?

@armsofter
hi
did you try to publish an app with this plugin? was it approved?

@agrieve, @jpchase, @armsofter, @vahidreza
Have you a response of this question?
Thanks


Dynamic rescaling of HR resources

This PR builds on #1241 to include a dynamic rescaling of HR resources taking place at the start of every year.
This scaling can be:

due to a constant scaling factor, controlled by parameter "dynamic_HR_scaling_factor" (e.g. set this parameter to 1.05 if considering a 5% yearly capability expansion);
proportional to the population growth compared to the previous year, by setting "scale_HR_by_popsize" to TRUE (i.e. rescaling factor = pop_size_this_year/pop_size_previous_year);
a combination of both (i.e. can consider fixed expansion in addition to pop growth);

@tbhallett appropriate test (test_dynamic_HR_scaling) now included. Note that there is a small precision error (<1e-10) which builds up over a 2 year period. I think we should be ok for the timescales considered, but it's a bit annoying that it's happening.
Note: I tried to include the last_year_pop_size as you suggested in the class definition of the regular event, but python insisted that it was a local variable/each instance of that class would have a new copy, so I moved last_year_pop_size to the healthsystem as I mentioned earlier.

Thanks @marghe-molaro

@tbhallett appropriate test (test_dynamic_HR_scaling) now included. Note that there is a small precision error (<1e-10) which builds up over a 2 year period. I think we should be ok for the timescales considered, but it's a bit annoying that it's happening.

I think this wasn't floating point error but due the number of time the updating happened.

Note: I tried to include the last_year_pop_size as you suggested in the class definition of the regular event, but python insisted that it was a local variable/each instance of that class would have a new copy, so I moved last_year_pop_size to the healthsystem as I mentioned earlier.

You had saved it on the instance of the module (HealthSystem) rather than the instance of DynamicRescalingHRCapabilities. I think these commits accomplish it.


chore(deps): upgrade @google-cloud/paginator to v0.1.2

Addresses system test error caused by a bug in the paginator.

Looks like they're still failing :/


livestreamer and old livestreame.com website

Hi this still an issue with livestreamer using the old livestream.com website. It plays the script then stops in a few  seconds.

I get " [cli][error] No data returned from stream"   i guess they have yet again changed something


Adds ability to pass a middleware to CLI for instrumentation

Great! Do you think you could provide some sample usage or examples in the readme?

Yes most definitely.


baselineAutoAdjust related issue

Hello, community.
Version jdk 17, ignite 2.15.0
When automatic baseline adjustment is enabled, some nodes cannot be added to the baseline topology when the VM where the cluster node resides restarts. You must manually add nodes to the baseline topology. By the way, this problem may occur when the cluster is forcibly restarted, like deleting all ignite pods from Kubernetes.
The following figure shows the example:

Thanks
Wei Cheng

It isn't clear which IP finder you are using. Please share the relevant configuration to help us understand how your cluster is doing discovery.


Missing spanish translation for Koch Curve, Stepping Feet and Bubble sort examples

Actual Behaviour
Missing spanish translation
Expected Behaviour
Spanish translation.
Steps to reproduce it
Screenshots of the issue
Would you like to work on the issue?
Yes, and will submit a pull request shortly

this issue was resolved with #626


Tycmd compatibility with Unity3D

I was able to get some position data printed on tycmd. Is there a way to integrate that data into unity3D?
I have tried with a c# code. But the problem occurs when both Tycmd and Unity accessing the same port.
Please help me in solving this problem. Thank you!!!

Serial ports are always exclusive for one to one connections. So, on the PC side, it’s either Tycmd, or a “classic” terminal program, or another program accessing the serial port.
The most common way if you have an embedded MCU sending serial data to another application is to use two serial ports. One which sends data to your app and another which sends debug data and information to a serial monitor like Tycmd.

Thank you for your answer..
I have tried only with a c# code using serial port. But I received a timed out error.
Is there a way to access data from Unity instead of printing on tycmd?

If your MCU (you even don’t write which one you use, Teensy 3.x?) sends data over serial, it can basically be read and be captured by whatever program which can access and read the serial port. If you see data in Tycmd, that’s the proof that your MCU sends data correctly. Next step would be a check with a known working terminal program which naturally could only access the serial port if Tycmd is closed and the serial port is free. In a third step, you might try to write your own program which could read serial data, still under the condition that nothing else blocks the port.
Since there have been people acquiring serial data from a Teensy with Python and MATLAB, it should also be possible with other stuff. But this is definitely not a Tycmd question or issue! You are asking in the wrong place.
You register on the Teensy forums https://forum.pjrc.com and use the search function there to find working examples. Or you ask in a C# developer forum to learn how to read data from a serial port. I can’t help you for that, as far as I know, C# is for Windows, which is not my world.

Thank you very much for your support...

Seems like this issue is resolved (thanks Theremingenieur), I'm closing it.


Expect value for staticPath different than expected in Debug Panel

AH!
This appears to happen a lot. Whenever I start coding again, the ghost in the machine interferes with my app. I don't want to return to work.
HA!

VSCode Version: 1.13.0
OS Version: Windows 7 Pro 64bit

Steps to Reproduce

git clone Rescue Shelter Demo
Open with VSCode
npm install & npm run compile-ts
Select the app.js file located in build\src folder
Add break point and step into server.js.
Review the Debug panel and hover over staticPath variable

This is just how we decided to show whitespace in different areas, in variables and watch expressions since the space is limited we decided to show \r \n instead of newline. In the hover since we have more space we have decided to instead show the real meaning of those characters, so we are by design replacing \r with nothing.
The bug in your program is that you are a doing
let staticPath = __dirname + "\react";
instead of
let staticPath = __dirname + "\\react";
Or even better maybe a path library so this code is not os dependent
Closing as designed. For more reasons on why we present whitespace like this you can search our old issues.

Ah! I should have used forward slash instead. I didn't think about escape characters at that time. Ha!
I thought this would be a quick app to write. React said it was designed for easy of use. So, the plan was get the app run and refactor. The create-react-app module was taking up most of my time with errors executing in the terminal.
Thanks for the quick peer review.
@isidorn @weinand @vscodebot


Virtual Gesture Keyboard using Python

Virtual Gesture Keyboard using Python #4372 as the part of GSSOC'23
Close #4372
Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@pranjay-poddar , please merge the project, of let me know of any necessary changes


Environmental Settings on Readme needs listing for PW_GW_PWD

Took a while to figure out that the PW_PASSWORD was not the same as PW_GW_PWD.  Yes, these were a bit outdated. I updated the docs. Let me know if this makes more sense. Feel free to suggest more or submit a PR. Thank you! 🙏
https://github.com/jasonacox/pypowerwall/tree/main/proxy

Will do. Thanks!


Prepare harvester-csi-driver chart for 2.7.0

harvester-csi-driver chart needs to be updated for 2.7 by

File a PR removing the doNotRelease flag form the package.yaml; in this PR, you will also want to make any changes that need to be made to update the chart to the version you intend to release in 2.7. For example, for a charts like rancher-aks-operator / rancher-gke-operator / rancher-eks-operator, you will need to create a new branch in upstream for Rancher 2.7, cut the first release for that chart, and introduce those changes in to the package.yaml as well before removing the doNotRelease flag
To make your chart available for 2.7, almost every chart will also have to modify the Rancher Version annotations that scope it to 2.6, i.e. https://github.com/rancher/charts/blob/2e058e93ee01f2980e4750148842dd7f04262b38/charts/fleet-agent/100.0.5%2Bup0.3.11/Chart.yaml#L8
Once the PR is merged, ensure that the brand new chart is smoke tested by QA/dev

Based on TachunLin closing the issue  https://github.com/rancher/rancher/issues/38976#issuecomment-1300433758


Fix move regular expression to avoid random skipping.

Allow flipping board view (Black's perspective)
Limit the move forward/backward action to the current game.

I promise we will review quickly this time :laughing:

I create a new pull request


Propagate the context arg.

If the incoming request has distributed trace information, we need to pass it down to the logger instead making a new default context.

Hello @jum , thanks for the PR. I'll review it in the next days


fix(web): handle parameter definition exceptions

There is a UI bug when a user provides malformed parameter definitions. An exception is thrown internally, but the UI gets stuck with the message:

processing, please wait

This PR catches exceptions thrown inside the designParameters function. If an exception is thrown, it gets added to the status object so it can be displayed to the user. After applying this PR, the user message becomes more helpful:

ERROR: Definition of choice parameter (tooling) should include a 'values' parameter

Example script:
const { cuboid } = require("@jscad/modeling").primitives

function getParameterDefinitions() {
  return [{name: "tooling", type: "choice", caption: "Tooling"}] // Oops forgot "values"
}

function main(params) {
  return cuboid({})
}
module.exports = { getParameterDefinitions, main }

All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Does your submission pass tests?

@platypii This is great! So many users are still complaining about the stack traces, error messages, etc. This is going to help many many users. SUPER


One-Bladed Turbine

Hi,
I did a series of experiments on a one bladed turbine. The rotor consists of 1 aerodynamic blade and 2 counterweights that are just cylindrical rods. The rotor diameter is 3.6m.
I was reading through the manual and found that Aerodyn can run a one-, two-, or three-bladed rotor atop a single tower. But somewhere else in the report (overview section) it says that OpenFast can run two- or three-blade horizontal-axis rotor.
I started with the UAE example and changed NumBl       - Number of blades (-) from 2 to 1 in the elastodyn .dat file. I ran OpenFast again and I got this error:
Running ElastoDyn.
FAST_InitializeAll:ED_Init:ED_ValidateInput:ValidatePrimaryData:NumBl must be either 2 or 3.
So my question is can I run openfast on a one bladed turbine? if so how can I solve the issue I got. So I have to change something else in the code?
Thanks for your help

You can't model a 1-bladed turbine with ElastoDyn, but you can model a 1-bladed turbine with a BeamDyn+ElastoDyn model.

Thanks for the quick reply. I am really new to OpenFast and I have been using it for a few days now.
I went to the .fst file and I changed CompElast   from 1 to 2 so I could run ElastoDyn + BeamDyn. I found a BeamDyn.dat example in the r-test folder and used it. I changed the input files to refer to DeamDyn.dat. I ran it again and I got this error:
Running ElastoDyn.
FAST_InitializeAll:ED_Init:ED_CalcContStateDeriv:LAPACK_DGETRF: U(           7,           7)=0.
Factor U is exactly singular.

I have been thinking about the problem some more. I think I am able to change the shape and properties of each blade in openfast. Am I correct? If so I can modify the blade to be a very thin cylindrical blade with no aerodynamic effects. So I will have one aerodynamic blade and one cylindrical blade with a very small diameter.
This way the second blade will have no influence on the induction factor or rotor performance. The results should be similar to a single bladed design. Does OpenFast allow me to do so?
This way I will avoid running BeamDyn.

You can certainly define a two-bladed turbine and set the blade mass and aerodynamic coefficients (lift, drag, pitch) to zero for the "extra" blade, but you'd have to disable blade degrees-of-freedom (DOFs) in ElastoDyn to do so.  This is because if you enable a blade DOF in ElastoDyn, ElastoDyn will be unable to solve the equations of motion because the mass matrix for the enabled "extra" blade DOFs will be zero.  You could get around this by modifying the ElastoDyn source code so that you only enable the blade DOFs for the blade whose mass is nonzero.
Best regards,

Thanks so much for your input. What I ended up doing was simulating a 3 bladed turbine where 1 blade is aerodynamic and the other 2 are just cylinders with 0 lift coefficient. This way the model resembles my experimental setup. That seems to have worked pretty well and the data I got from the model resembles the experimental data.
I am in the process of comparing and discussing the differences between the model and the experimental results. I read the report that you wrote back 2003 (Modeling of the UAE Wind
Turbine for Refinement of FAST_AD) where you modeled the UAE in fast and compared experimental data to the model. You mentioned that discrepancies in the model are due to tip loss, stall delay, post stall, and dynamic stall. The unsteady aerodynamic in Fast is based on the Beddoes-Leishman model.
I then read over the Mexnext Phase 1 report (2012) where they compared Fast to the MEXICO experiments. They mentioned unsteady aerodynamics especially when the airfoil is stalled were hard to model in Fast.
I am looking at my data and also notice few discrepancies between the model and experiments specially when the turbine is yawed to 30 degrees.
Was there any modification to the unsteady aerodynamic modeling in Fast in the last few years? Am I safe to say that these discrepancies that I see are due from unsteady aerodynamic modeling? Because when the turbine is not yawed the model and experiments align within reason.
Do you also know of other reports I could refer to where they compared experimental data to the FAST model especially on yawed turbines?
Thank you very much for your help
Farid

Dear @fsamara12,
There were a number of improvements to the Unsteady Airfoil Aerodynamics modeling in FAST v8 with the introduction of AeroDyn v15, which is what I assume you are using.  See the AIAA SciTech 2016 paper by Damiani et al for more information: https://arc.aiaa.org/doi/10.2514/6.2016-1007.  That said, the accuracy of this model is determined by the appropriate settings of the unsteady airfoil aerodynamic input parameters, which can be calibrated to match known / measured stall hysteresis loops.  How did you set these parameters for your case?
The most recent validation of FAST under yawed-flow conditions was also performed by Damiani et al--see: https://www.wind-energ-sci-discuss.net/wes-2017-38/wes-2017-38.pdf.
Best regards,

Dear Jonkman,
Yes I am using AeroDyn v15. I set the airfoil parameters by looking at the S809 airfoil from the UAE example first. I then changed the Cl, Cd, Cm parameters so it matches the S833 airfoil that I am using. I changed stall angle, slope of the 2D airfoil. I noticed that the Cn1 parameter is important to set correctly. I am reading the manual and not sure what should the value be. I am overwhelmed by the number of variables to set and relied on the S809 airfoil example.
I did some dynamic stall experiments on my airfoil in a 2d wind tunnel and I have the hysteresis and dynamic stall loops. But I am not sure how to use that data to import it to OpenFast.
Best Regards
Farid
Mod_S833_FS_V2_copy.txt

Dear @fsamara12,
Indeed there are many unsteady airfoil aerodynamics parameters to set.  And I'm no expert on how to set them properly.  The Cn1 value can be derived from the plot of normal force coefficient versus angle of attack (derivable from the lift and drag coefficients).
If you have measured hysteresis and dynamic stall loops, I would run the unsteady airfoil aerodynamics module in standalone mode (there is a standalone driver for that) in order to plot the hysteresis loops from AeroDyn v15 for comparison with your measured data.  The airfoil parameters can be then be adjusted to better match the data.
Best regards,

Dear Jonkman,
Thats a good idea.
I am trying to find documentation to run unsteady airfoil aerodynamics model in standalone just like you suggested. Is there a compiled version of the standalone model? or could you refer me to documentation to be able to run the model? My programming skills are very minimal.
Thanks
Farid

OpenFAST creates an unsteady aerodynamics driver by default. https://github.com/OpenFAST/openfast/blob/dev/modules/aerodyn/CMakeLists.txt#L62
https://nwtc.nrel.gov/AeroDyn should contain some examples of an input file for the Unsteady Aero driver.
The unsteady aerodynamics coefficients for each airfoil should be modified in the input file containing the aerodynamic data for the corresponding airfoil like here
https://github.com/OpenFAST/r-test/blob/master/glue-codes/openfast/5MW_Baseline/Airfoils/DU21_A17.dat

To compile the unsteady aero driver, make sure you're on the dev branch and run cmake .. && make unsteadyaero_driver.

I would like to thank you again for your amazing help while I try and figure out how to use this great code.
I was able to use the example compiled code from the aerodyn link that you gave me.
Here is what I got. I think the agreement looks good. I compared my experimental data to the UA model for a mean of 10 and 20 deg

with amp 10 deg, and k=0.1.
Thanks again
Farid

I think this is the final problem that I have been facing. The code seems to work smoothly and reads all the input files correctly. But so far I have been running with no DOF.
When I turn on the DOF in the Flap and edge wise direction, I keep getting this error attached at the end. I have looked at the all options again and again trying to figure out where I am going wrong.
As a blade input file to aerodyn I am using the UAE Ames blade and modified the twist.
Am I missing something? What could I do to fix it? Also will having a DOF effect my results if I have a 1.7 m blades?
Thanks
Farid
UAE_Ames_Blades_UW1.txt
Error:
Warning: SkewedWakeCorrection encountered a large value of chi (103.35 deg), so the yaw
correction will be limited. This warning will not be repeated though the condition may persist.
See the AD15 chi output channels, and consider turning off the Pitt/Peters skew model (set
SkewMod=1) if this condition persists.
FAST_Solution:FAST_AdvanceStates:ED_ABM4:ED_CalcContStateDeriv:SetCoordSy:Small angle assumption
violated in SUBROUTINE SmllRotTrans() due to a large blade deflection (ElastoDyn SetCoordSy). The
solution may be inaccurate. Simulation continuing, but future warnings from SmllRotTrans() will
be suppressed.
Additional debugging message from SUBROUTINE SmllRotTrans(): 6.00000E-03 s
FAST_Solution:CalcOutputs_And_SolveForInputs:SolveOption2:InflowWind_CalcOutput:CalcOutput:IfW_Uni
formWind_CalcOutput:GetWindSpeed:Height must not be negative.
IfW_UniformWind_CalcOutput: Error calculating the wind speed at position (0.47079, 0.73189,
-0.13626) in the wind-file coordinates
FAST encountered an error at simulation time 4.50000E-03 of 60 seconds.
Simulation error level: FATAL ERROR
Aborting OpenFAST.

Dear @fsamara12,
The SmllRotTrans() error due to large blade deflection at the beginning of your simulation points to the problem.  When you enable DOFs, you are introducing natural frequencies in your system that must be resolved in time and your time step is too large to resolve them, resulting in a numerical instability.   The rule of thumb for selecting the time step for the ElastoDyn structural module is given in the following topic on our forum: https://wind.nrel.gov/forum/wind/viewtopic.php?f=4&t=351&p=1025.
It is important to enable structural DOFs if there is a potential that the excitation frequencies will coalesce with the natural frequencies or if there is a potential for impactful structural deflection or instabilities.
Best regards,

Closing this issue due to inactivity.  @fsamara12, please reopen if your issue has not been fully solved.


As a server admin, I want to share or not share posts with users on other servers.

https://github.com/Zhipeng-Chang/CMPUT404-Winter-2019-GroupProject/pull/112


SOP for adding new HL7 spec

This PR adds an SOP describing the process of adding a new HL7v2 message type.
Linked Issues

#13024

Another comment: You have this main section:
# Adding a New Conformance Profile in ReportStream
But what about ALSO including info on this:
# Updating an existing Conformance Profile in ReportStream
This may require some restructuring of the doc as there is some overlap probably. Maybe they are not separate sections.


jenkins-master is slowly running out of disk space

6.8G remains and the growth is gradual, but it won't be much longer
/var/lib/jenkins/jobs/scala-2.11.x-integrate-ide is using 41G (of the 87G total current usage) so that seems like the main culprit
additionally there are 1215 subdirectories under /var/lib/jenkins/jobs/dotty-master-validate-partest/builds, and 308 more under dotty-master-validate-partest-bootstrapped. I tried to do:
% cd /var/lib/jenkins/jobs
% du -sh dotty*partest*

but after 10 minutes I got tired of waiting. but recent builds are taking 27M each, so 1215 times 27M equals ~30G, so I think that's our #2 culprit

[ ] limit how many jobs we retain log files for
[ ] make these jobs less verbose

27G available currently after:

adriaan [3:23 AM]
i "permanently archived" a bunch of old scala-ide logs, so that we should be good for a while in the disk space department
[3:23]
we should really make logging for that job a bit more concise

the real problem was that that partition had run out of inodes, because the dotty partest jobs archive 5000 files per job
I think we need something like https://wiki.jenkins-ci.org/display/JENKINS/Compress+Artifacts+Plugin, or perhaps hardcode it in the job to tar most of the files. (See thread on https://gitter.im/lampepfl/dotty just now)

today @adriaanm suggested we stop archiving (or stop archiving so many) the scala-2.11.x-integrate-ide target directories — they are large (69M each) and nobody looks at them. at most we might look at a recent failure

not pressing — I check the space on all the Jenkins nodes every 4 days and manually blow away caches as needed

the worst problems have been fixed; addressing every space leak isn't worth the effort. there is now doc at https://github.com/scala/scala-jenkins-infra/blob/master/doc/maintenance.md about where to look when space starts running low


DI-273 update scratch config to use new us multi region bucket

Why
DI-273
This PR
The diff run during the load portion of the HCA ingest pipeline does an external join against the buckets and bq is balking because BigQuery & Dataflow were not touched during this move, leaving them in US multi-region.
The suggested solution is to create a new multi-region scratch bucket. This will contain only the scratch data we process during this interim period, thus still garnering the cost savings on the now single-region bucket, while unblocking this production workflow.
Checklist

[X] Documentation has been updated as needed.

Not sure why the IDE had old commit in here... I'll fix those when resolving the merge conflicts and makes sure to clear out my IDE


Load TLS certificates only if we need HTTPS or WSS

Now we can see this if application/cert do not contains key.pam and cert.pem
11:30:58  W1   error   Can not load TLS certificates

I get this error on startup:
18:28:02  W1   error   Can not load TLS certificates
18:28:02  W0   error   Can not start server


feat(opentelemtry): Initial implementation of OpenTelemetry

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 1.0.0
download it from Artifactory here

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

1.0.0
GitHub release

Your semantic-release bot :package::rocket:


Combining routers

Right now, I have to do app.use with the routes() method on each of my routers. Example:
app.use(rootRouter.routes())
app.use(userRouter.routes())
Is there any way to combine the above into one router and then just be able to do something like
app.use(combinedRouter.routes()) ?
Here is my current code:
app.js
const Koa = require('koa');
const rootRoutes = require('./routes/index')
const userRoutes = require('./routes/user')

const app = new Koa()

app.use(rootRouter.routes())
app.use(userRouter.routes())

app.listen(3000)

export default app

routes/index.js
import Router from 'koa-router'
const router = new Router()

router.get('/', async ctx => {
  await ctx.render('index')
})

router.get('/about', async ctx => {
  await ctx.render('about')
})

export default router

routes/user.js
import Router from 'koa-router'
const router = new Router({ prefix: '/user' })

router.get('/:name', async ctx => {
  const user = ctx.params.name
  await ctx.render('user', { user })
})

export default router

Looking for doing the same. There are actually double more app.use, because there is routes but also allowedMethods for each Router.
This makes "hot-reloading" like described here much more challenging.

I think koa-compose could do the trick.
app.use(compose([router1.routes(), router2.routes()]))
That's what koa is doing under the hood anyways

^ gingerich is right.
For convenience, I created a separate module which is just a wrapper around koa-compose to combine instances of koa-router. Instead of doing something like this:
import compose from 'koa-compose'
import router1 from './router1'
import router2 from './router2'
import router3 from './router3'

const router = compose([
  router1.routes(),
  router1.allowedMethods(),
  router2.routes(),
  router2.allowedMethods(),
  router3.routes(),
  router3.allowedMethods()
])

export default router

You'll be able to do this:
import combineRouters from 'koa-combine-routers'
import router1 from './router1'
import router2 from './router2'
import router3 from './router3'

const router = combineRouters([
  router1,
  router2,
  router3
])

export default router

Avoids a bit of repetition.

Thanks! I would like for there to be some way to do this in koa-router itself, but for now I'll go with using your package.

I am not sure if this is the correct place or what I am trying to achieve is aligned with the above
const Koa = require('koa');
const app = new Koa();

const Router = require('koa-router');

const foo = new Router({ prefix: '/foo' });
foo.get('/', ctx => ctx.body = '/foo' );

const bar = new Router({ prefix: '/bar' });
bar.get('/', ctx => ctx.body = '/bar' );

const dup = new Router({ prefix: '/dup' });
dup.get('/', ctx => ctx.body = '/dup' );

bar.use('/', dup.routes(), dup.allowedMethods());
foo.use('/', bar.routes(), bar.allowedMethods()); // explGET /foo/bar, GET /foo/bar/dup
foo.use('/', dup.routes(), dup.allowedMethods()); // GET /foo/dup

app.use(foo.routes()).use(foo.allowedMethods());

app.listen(3000);

This is my code but I get following results in matters of status codes for the above urls
$ curl localhost:3000/foo
/foo
$ curl localhost:3000/foo/bar
Not Found
$ curl localhost:3000/foo/dup
Not Found
$ curl localhost:3000/foo/bar/dup
Not Found

What I want is to have the same routes / methods under /foo/dup and /foo/bar/dup path


feat: Motasim.md

Great Having you @motasimmakki

Great Having you @motasimmakki
Great to be a part of it, kindly invite me to join the organization too. 🙃

Great Having you @motasimmakki
Great to be a part of it, kindly invite me to join the organization too. 🙃

Join Our Discord!!
We are inviting people soon!!

Great Having you @motasimmakki
Great to be a part of it, kindly invite me to join the organization too. 🙃

Join Our Discord!!
We are inviting people soon!!

sure, looking forward to dig more into open-source with org.


snapToInterval not implemented on android for vertical scroll view

Environment
OS: macOS High Sierra 10.13.4
Node: 9.11.1
Yarn: 1.0.2
npm: 5.6.0
Watchman: 4.9.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 3.1 AI-173.4720617
Packages: (wanted => installed)
react<EMAIL_ADDRESS>=> 16.3.2
react-native: 0.55.0 => 0.55.0
Description
snapToInterval is not implemented on android for vertical scroll view.
https://github.com/facebook/react-native/blob/8dc3ba0444c94d9bbb66295b5af885bff9b9cd34/Libraries/Components/ScrollView/ScrollView.js#L511
The official doc states the feature is actually implemented.
https://facebook.github.io/react-native/docs/scrollview.html#snaptointerval

How to merge the above commit to RN 0.56?


A copy of your changes has been stored to "/tmp/kubectl-edit-4jyoa.yaml" error: Edit cancelled, no valid changes were saved.

Environment
Installation method:
Kubernetes version: client : 1.14, server 1.14
Dashboard version:
Operating system:
Node.js version ('node --version' output):
Go version ('go version' output):
Steps to reproduce
When i want to edit my file to use the public IP of my server, I get the error in the tittle:
sudo kubectl -n kubernetes-dashboard edit service kubernetes-dashboard
Observed result
A copy of your changes has been stored to "/tmp/kubectl-edit-4jyoa.yaml" error: Edit cancelled, no valid changes were saved.
Expected result
Comments

Had to reinstall all the things :(

thanks.  I've gotten past that error but discovered my AKS cluster has been upgraded past the point kube-lego is working so now I'm trying to figure out how to get cert-manager working.

Make sure the indentations are right especially where the hyphens "-" are found


Question: is there a way to automate Publish>Deploy as API executable...

Currently when working on REST script execution targets, after running gulp upload-latest --env dev I have to go to the script project in a browser and run Publish>Deploy as API executable..., and name the version.
Is there any way to automate this as part of pushing the code up?

Is there an API for this?

There was but it was deprecated. There is no API for this currently.

Actually, that deprecated method doesn't relate to the API Executable.
Currently, there are no APIs for many of the functions that you need to perform in the script editor, such as publishing (web apps, add-ons, API executables).

You're correct of course. My comment referenced Publish>Deploy as web app... and the OP asked about Publish>Deploy as API executable....

Is this still true? Constantly having to navigate the Publish>Deploy as API executable... interface really makes working on google app scripts a drag.

Related: API to deploy google apps script as WebApp

Don't actually know if this is helpful for your use case but I have devMode: true set and it allows the remote execution API to run the latest version of the script without republishing. It only does this for the developer, not users.

Hi !
@lastobelus @oshliaer @peterherrmann @hess-g @gunar @sbgood
I have a google sheet containing script, I want to execute this script programmatically for any user (public). Can I perform  "Publish>Deploy as API executable" programmatically?

Hi @asadmkhan
It's not possible.

HI. Any updates on this? I've followed the links but it seems everything is still on the same page


I can't open Object Builder, the app closes instantly after open

I can't open Object Builder, the app closes instantly after open
Please help! Someone know why?
https://user-images.githubusercontent.com/19294710/153526851-1f260964-79b4-41fe-9ac4-a25094d62f19.mp4

Hello, I've updated the wiki pages, please follow step by step.
From now on, use Moonshine SDK Installer to install Apache Flex


Getting multiple replies from chatbot sometime while integrated with slack

Rasa Core version: 0.11.10
Rasa NLU version:  0.13.6
Python version: 3.6.3
Operating system (windows, osx, ...): windows
Issue:
Hi I am getting multiple replies from chat bot, is this due to incorrect mapping or slack integration/ ngrok issue. Please suggest, how i can improve this.

Content of domain file (if used & relevant):
 - greet
 - mood
 - goodbye
 - product_inform
 - product_start_inform
 - forget_password
 - account_access_issue
 - product_upgrade
 - product_issue
 - product_utility
 - chatbot_name
 - chatbot_info

entities:
 - products
 - product_utility_entity
 - product_start_inform_entity
 - forget_password_entitiy
 - product_upgrade_entity
 - product_issue_entity
 - account_access_issue_entity

templates:
  utter_greet:
    - 'Hello! How can I help?'
  utter_mood:
    - 'I am good. How can I help you?'
  utter_goodbye:
    - 'Talk to you later.'
    - 'Bye bye :('
    - "Goodbye :("
  utter_product_inform:
    - 'We provide three type of products named A, B and C.'
  utter_product_start_inform:
    - 'You can read our getting started menu which you must have got while purchasing the product
       or you can contact us for any asistence at<EMAIL_ADDRESS>  utter_forget_password:
    - 'You can reset your password by clicking forgot password link available on home page of telldus'
  utter_account_access_issue:
    - 'you need to contact support for troubleshooting your issue<EMAIL_ADDRESS>  utter_product_upgrade:
    - 'you need to contact support for product upgrade at<EMAIL_ADDRESS>  utter_product_issue:
    - 'you need to contact support for troubleshooting your issue<EMAIL_ADDRESS>  utter_product_utility:
    - 'Our products provide features to make your home smart:
		1. ASSURANCE
		2. COMFORT
		3. CHILD SAFETY
		4. SCARE AWAY THE THIEVES
		5. COMFORT'
  utter_chatbot_name:
    - 'I am Johny, How can i help you?'
  utter_default :
    - 'Sorry I can not get you.'
  utter_chatbot_info :
    - I provide information about Telldus products.
actions:
 - utter_greet
 - utter_mood
 - utter_goodbye
 - utter_product_inform
 - utter_product_start_inform
 - utter_forget_password
 - utter_account_access_issue
 - utter_product_upgrade
 - utter_product_issue
 - utter_product_utility
 - utter_chatbot_name
 - utter_chatbot_info

Thanks for raising this issue, @ricwo  will get back to you about it soon.

@vikingsvivek Would you mind sharing your logging output when those repeated replies happen? The lines should start with Received retry #X request from Slack due to Y


[Feature] Ability to set cursor style

Right now it looks like we've got no choice but to use the "block" cursor.
It would be pretty sweet if we could have the option to set the cursor style to IBeam or Underline.
It would be even sweeter if the default could automatically be detected from the current terminal. I have no idea if that's possible though!
Version: 1.4.1
Commit hash: 1856891
Compiled on August 10, 2018
OS: Ubuntu 18.04
Terminal: Tilix

It would be even sweeter if the default could automatically be detected from the current terminal. I have no idea if that's possible though!

This seems to already be the case (maybe it wasn't in 2018). But it might stil be useful to override this behavior.


Reduce proxies memory consumption  [7039]

This is a port of #916 from 1.9.x

Build status:

Linux
Mac
Windows


Could not open ilastik GUI in windows

/opt/ilastik/ilastik-meta/lazyflow/lazyflow/classifiers/init.py:20: UserWarning: init: Could not import tiktorch classifier
warnings.warn("init: Could not import tiktorch classifier")
Starting ilastik from "/opt/ilastik".
QXcbConnection: Could not connect to display
Fatal Python error: Aborted
Thread 0x00007f6b2bfff700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 300 in wait
File "/opt/ilastik/lib/python3.7/threading.py", line 552 in wait
File "/opt/ilastik/lib/python3.7/threading.py", line 1156 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b4a3cd700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b4abce700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b4b3cf700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b4bbd0700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b630e7700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b608e6700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b5e0e5700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Thread 0x00007f6b5b8e4700 (most recent call first):
File "/opt/ilastik/lib/python3.7/threading.py", line 296 in wait
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 154 in _get_next_job
File "/opt/ilastik/ilastik-meta/lazyflow/lazyflow/request/threadPool.py", line 99 in run
File "/opt/ilastik/lib/python3.7/threading.py", line 917 in _bootstrap_inner
File "/opt/ilastik/lib/python3.7/threading.py", line 885 in _bootstrap
Current thread 0x00007f6b6b41c740 (most recent call first):
File "/opt/ilastik/ilastik-meta/ilastik/ilastik/shell/gui/startShellGui.py", line 61 in startShellGui
File "/opt/ilastik/ilastik-meta/ilastik/ilastik_main.py", line 176 in main
File "/opt/ilastik/ilastik-meta/ilastik/ilastik.py", line 128 in main
File "/opt/ilastik/ilastik-meta/ilastik/ilastik.py", line 134 in
/opt/ilastik/run_ilastik.sh: line 77:    38 Aborted                 "${PREFIX}/bin/python" "${PREFIX}/ilastik-meta/ilastik/ilastik.py" "$@"

Hi @ReethikaC
Have you seen the heading, Graphical user interfaces on Windows under the installation section of the steinbock documentation?

Hi,
I have seen the heading, GUI on Windows. I have downloaded the VcXsrv but I
still cannot run the command. I see that it is untested, I wanted to raise
an issue so that it will help you know the problem. And I also wanted to
see if you have a solution for it.
Regards,
Reethika C
On Tue, Aug 29, 2023 at 11:37 AM Milad4849 @.***> wrote:

Hi @ReethikaC https://github.com/ReethikaC
Have you seen the heading, Graphical user interfaces on Windows under the installation
section
https://bodenmillergroup.github.io/steinbock/latest/install-docker/ of
the steinbock documentation?
—
Reply to this email directly, view it on GitHub
https://github.com/BodenmillerGroup/steinbock/issues/219#issuecomment-1697693412,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCGBOZBMTY67ULDS7FNYRRTXXYEEFANCNFSM6AAAAAA4BFLG2Q
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi,
I get the same problem after installing VcXsrv did you manage to solve the issue somehow?

I am afraid this remains untested at the moment. With the next release we will make the decision to whether test and support this particular feature.


Fix for study area chunk metadata initialized too early due to error/retry

The root cause we're trying to solve in this PR is the following. Before this PR we had a logic avoiding retry for build_feature_matrix CF if the chunk metadata is already present. But if the error happens in this CF we do create chunk metadata object with just error field populated which now blocks retry (it does detect the presence of chunk metadata and stops but it shouldn't in case of previous error).

Can you describe the root problem you're trying to solve? Some descriptions in this PR or code comments would be helpful for understanding the changes and what issues they address.

Can you describe the root problem you're trying to solve? Some descriptions in this PR or code comments would be helpful for understanding the changes and what issues they address.

Added more details PR description.
How about this explanation in the code? https://github.com/UrbanSystemsLab/climateiq-cnn/pull/117/commits/aaced9bccb383b0e613cb8a02cc92d427fde38be

logic avoiding retry for build_feature_matrix CF if the chunk metadata is already present

Is that logic something we need? If the function is idempotent, is it okay to rebuild the matrix again?
Could that logic prevent us from performing re-runs that we do want -- re-runs due to code changes, failure retries, etc?

logic avoiding retry for build_feature_matrix CF if the chunk metadata is already present

Is that logic something we need? If the function is idempotent, is it okay to rebuild the matrix again? Could that logic prevent us from performing re-runs that we do want -- re-runs due to code changes, failure retries, etc?

Yeah, I added that check as a result of a problem when we saw build_feature_matrix CF succeeded but got retried because it was just a bit over the 1 minute threshold. The retry loop produced a lot of events that triggered many rescaling passes. That was last needed chunk (longest to process) that was needed to start rescaling so with it being done in every retry rescaling logic started full check again and again.
This check will prevent re-runs due to code changes - in this case we would have to re-upload from scratch forcing initial cleanup in uploader. But this wouldn't prevent failure retries since chunk metadata state update is the last action of the CF building unscaled feature matrix (https://github.com/UrbanSystemsLab/climateiq-cnn/blob/96563734d16d909c6fc872bedb636f5b7ffc6c3f/usl_pipeline/cloud_functions/main.py#L395).


TypeScript: source file resolution (over HTTP)

Hi,
I apologise in advance as this may have been asked before, but I am having some difficulty understanding file resolution / acquisition for multiple TypeScript source files within monaco.
For some background, I am currently looking to embed monaco inside a small web application and the application will be backed by a REST web service which will serve TypeScript source files over HTTP. So I am curious how to approach resolving source files on behalf of monaco (or typescript) in this scenario.
My first thought would be that there might be a IEnvironmentHost interface or similar that could be implemented, something that monaco could call out to when attempting to resolve a file. Something that would execute when monaco discovers a import or <reference> directive inside a TypeScript source file, but I am not exactly sure where to look, or indeed if such a function exists.
I guess the end goal would be to automatically resolve the red squiggles for imports, and to avoid having to pull down the codebase on page load, instead leaving monaco to request the files it needs from the environment when discovering a import.
If anyone could provide guidance on this it would be greatly appreciated.
Many Thanks
sinclair

Any news on that?

Sorry, but we have no plans of doing that. From the monaco-typescript:

Note that this project focuses on single-file scenarios and that things like project-isolation, cross-file-features like Ranme etc. are outside the scope of this project and not supported.

Closing, as question


documentation_request.md

Change type

[x] Bugfix (code changes that DO NOT CHANGE the business rule)
[ ] New Feature (code additions that DO NOT CHANGE the business rule)
[ ] Code improvement (code changes that DO NOT CHANGE the business rule)
[ ] Fixes or new features that CHANGE the business rule

Proposed changes/Description
Describe your changes briefly and objectively.
<< Add here Pull Request. >>
Current behavior

Please describe the current behavior that you are modifying

<< Currently the component does... >>
New behavior

Please describe the behavior or changes this PR adds

Change one here..
Change two here..
Change three here..

Is this a breaking change (Yes/No):

If Yes, please describe the impact and migration steps for existing users.

Additional Information
<< Add here any additional info or N/A. >>
Task
This Pull Request is associated with the following tasks in Jira:
LINK_TO_JIRA_ISSUE
Proposed changes
Describe your changes briefly and objectively.

Foo
Bar
Baz

Checklist

[ ] Have you written tests for your changes?
[ ] Have you successfully ran tests with your changes locally?
[ ] Have you lint your code locally prior to submission?
[ ] Have you updated the component documentation with the changes?

tradução documentation_request.md


this is a github actions test

Just testing how the workflow works :)

@righettod did you disable github actions on this repository? Could you enable them? I'd like to see the logs

Hi, no it is enabled but you need to modify a TXT file (a dict in fact) to trigger it 😃


Netlify Deploy fails on next build due to a webpack error

Describe the bug
My deploy seems to be failing on next build while locally I can't seem to replicate the issue
11:45:38 PM:
11:45:38 PM: ./node_modules/clone-deep/utils.js
11:45:38 PM: Cannot statically analyse 'require(…, …)' in line 11
11:45:38 PM: > Build error occurred
11:45:38 PM: Error: > Build failed because of webpack errors
11:45:38 PM:     at /opt/build/repo/node_modules/next/dist/build/index.js:397:19
11:45:38 PM:     at async Span.traceAsyncFn (/opt/build/repo/node_modules/next/dist/telemetry/trace/trace.js:60:20)
11:45:38 PM:     at async Object.build [as default] (/opt/build/repo/node_modules/next/dist/build/index.js:77:25)

Netlify Deploy Log: https://app.netlify.com/sites/looter-io/deploys/613ac93454eb7c000838fdba?utm_source=github&utm_campaign=bot_dl
Code Being Deployed: https://github.com/nrodriguez/looter-io/pull/16
Relevant Netlify Forum Post: https://answers.netlify.com/t/next-js-deploy-fails-on-webpack-build/43921/2
Next.js Version: Next.js v11.1.2

@nrodriguez hey! this seems to be an issue with our plugin defaulting to target: serverless. can you set the target property in your next.config.js to experimental-serverless-trace?

@nrodriguez hey! this seems to be an issue with our plugin defaulting to target: serverless. can you set the target property in your next.config.js to experimental-serverless-trace?

Hi @lindsaylevine , I've added the change as well as changing my plugin-nextjs to use the latest version (v3.9.0). It worked in passing the next build step but I do see an error showing up at the end
Build Log: https://app.netlify.com/sites/looter-io/deploys/614213ac22a2320007e8d262
11:42:51 AM: ❯ Uploaded redirects
11:42:51 AM:   # Next-on-Netlify Redirects
11:42:51 AM:   /  /.netlify/functions/next_index  200
11:42:51 AM:   /_next/data/build/index.json  /.netlify/functions/next_index  200
11:42:51 AM:   /api/search-results  /.netlify/functions/next_api_searchresults  200
11:42:51 AM:   /_next/image*  url=:url w=:width q=:quality  /nextimg/:url/:width/:quality  301!
11:42:51 AM:   /api/scrape/:marketplace  /.netlify/functions/next_api_scrape_marketplace  200
11:42:51 AM: Starting to deploy site from 'out'
11:42:52 AM: Creating deploy tree
11:42:53 AM: Creating deploy upload records
11:42:53 AM: 2 new files to upload
11:42:53 AM: 3 new functions to upload
11:43:02 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:05 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:06 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:09 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:12 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:16 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:22 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:31 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:38 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:43:55 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:44:14 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:44:33 AM: Request must be smaller than 69905067 bytes for the CreateFunction operation
11:44:33 AM: Failed to upload file: next_api_scrape_marketplace
11:44:33 AM:   /nextimg/*  /.netlify/functions/next_image  200

Is this related to the plugin or is this more of a netlify itself issue?

@nrodriguez it's a symptom of experimental-serverless-trace. if you add node_bundler as esbuild in your netlify.toml it should fix this issue. see https://docs.netlify.com/configure-builds/file-based-configuration/#functions

@nrodriguez it's a symptom of experimental-serverless-trace. if you add node_bundler as esbuild in your netlify.toml it should fix this issue. see https://docs.netlify.com/configure-builds/file-based-configuration/#functions

I've added the proposed value to my netlify.toml file but still seem to see an error
https://app.netlify.com/sites/looter-io/deploys/61422216b668ff00070996db

I'm also using puppeteer and puppeteer-extra, is this any cause of the issues as well?

@nrodriguez hey - i pinged people earlier today internally at netlify who specialize in functions bundling. hopefully they can give us some more insight tomorrow!

@nrodriguez hey - i pinged people earlier today internally at netlify who specialize in functions bundling. hopefully they can give us some more insight tomorrow!

Awesome to hear, thanks for the follow up!

The problem with puppeteer is that it's probably bundling the whole of chromium in the function. Do you need it at runtime? You can look in .netlify/functions/next_index.zip and see what's in there that's making it so large, but my guess is that puppeteer is the culprit. If you don't need it, try adding PUPPETEER_SKIP_CHROMIUM_DOWNLOAD=1 to your env vars in the site UI

Ah yes sorry. Download the CLI and run netlify build

@ascorbic  I've update my code to use puppeteer-core and import the chromium-aws-lambda package per the advice of (https://github.com/netlify-labs/netlify-functions-headless-chrome) but it still failed.
I tried even adding that env variable you suggested but it still fails on the builds.
I built it locally but unfortunately I can't see in the zip the size of the internal components

Try zipinfo .netlify/functions/next_index.zip if you're on something unixy

@ascorbic After moving to try to use the suggested code in the netlify-labs repo (also mentioned here: https://bitsofco.de/how-to-use-puppeteer-in-a-netlify-aws-lambda-function/), I see that chromium is by far the largest from the chrome-aws-lambda package.
However, I do need to use puppeteer to scrape a page at runtime. Is there any other ways to get around netlify generating that function?

I just found https://github.com/soofka/netlify-plugin-chromium, would this be a suitable replacement for using chromium in netlify at runtime?

Good fix! That does sound like a different error. It's almost always two copies of React. Try npm ls react or yarn why react

@ascorbic  Hm, when I run it locally (where I don't see the error occur) I see
λ JustNeph-Desktop looter-io → λ git feat/add-poshmark* → npm ls react
looter-io@ /home/Projects/looter-io
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ ├──<EMAIL_ADDRESS>deduped
│ ├─┬<EMAIL_ADDRESS>│ │ └──<EMAIL_ADDRESS>deduped
│ └─┬<EMAIL_ADDRESS>│   └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├─┬<EMAIL_ADDRESS>│ └──<EMAIL_ADDRESS>deduped
├──<EMAIL_ADDRESS>└─┬<EMAIL_ADDRESS>  └──<EMAIL_ADDRESS>deduped

λ JustNeph-Desktop looter-io → λ git feat/add-poshmark* → yarn why react
yarn why v1.22.11
[1/4] Why do we have the module "react"...?
[2/4] Initialising dependency graph...
[3/4] Finding dependency...
[4/4] Calculating file sizes...
=> Found<EMAIL_ADDRESS>info Has been hoisted to "react"
info This module exists because it's specified in "dependencies".
info Disk size without dependencies: "356KB"
info Disk size with unique dependencies: "404KB"
info Disk size with transitive dependencies: "432KB"
info Number of shared dependencies: 3
Done in 0.44s.

And when I run it in netlify (which is where I see that error occur):
9:46:18 AM: $ NODE_OPTIONS=--trace-deprecation yarn build
9:46:19 AM: yarn run v1.22.10
9:46:19 AM: $ npm ls react && yarn why react && next build
9:46:22 AM: /opt/build/repo
9:46:22 AM: └──<EMAIL_ADDRESS>9:46:22 AM: [1/4] Why do we have the module "react"...?
9:46:22 AM: [2/4] Initialising dependency graph...
9:46:22 AM: [3/4] Finding dependency...
9:46:22 AM: [4/4] Calculating file sizes...
9:46:22 AM: info
9:46:22 AM: => Found<EMAIL_ADDRESS>9:46:22 AM: info Has been hoisted to "react"
9:46:22 AM: info This module exists because it's specified in "dependencies".
info Disk size without dependencies: "356KB"
9:46:22 AM: info Disk size with unique dependencies: "404KB"
9:46:22 AM: info Disk size with transitive dependencies: "432KB"
9:46:22 AM: info Number of shared dependencies: 3

I've also tried adding a linter rule for the hooks (https://www.npmjs.com/package/eslint-plugin-react-hooks) but I really don't think this is a locally occurring issue as the linter doesn't show any errors.
It seems like this issue is solely occuring on netlify builds.

Have you tried a clear cache and deploy?

Have you tried a clear cache and deploy?

Yup, a couple different times. The build doesn't show me any glaring issues: https://app.netlify.com/sites/looter-io/deploys/6149e0ba0c6a0f00b97fb364
The preview still fails unfortunately: https://6149e0ba0c6a0f00b97fb364--looter-io.netlify.app/

@nrodriguez hey! i know its been a few weeks, we apologize for the delay and appreciate your patience!! we JUST released the beta of our full plugin rewrite. we anticipate it will fix your issue(s). would you be open to testing it out? instructions inside our discussion here. let us know! 🙏

CLosing because it doesn't apply with the new plugin. If you can upgrade the plugin to 4.0.0 or later, and if you still have trouble please open a new ticket. Thanks!


Add some utilities for testing util::Status.

Converted one test to use it, as an example.

Nice!


Submitting forms

Hi @dai-shi!
Just found myself puzzled with how to organize sending forms along with react-use-form-state.
That would simplify work if there was some example of that use-case.

Oh! There is undocumented start property returned from useAsyncTask! That's what I need.

Yeah, you can use start. The example is here.
The reason why it's not documented is it's not the main use case.
If you are only calling an async function from an event handler, you can just define a function. There's nothing special this library can help except for combining multiple async functions.
So, I'm interested in your use case. Please let me know how you used the library. Thanks.

I basically submit form in onSubmit with react-use-form-state hook:
import React from 'react'
import { connect } from 'unistore/react'
import { useFormState } from 'react-use-form-state'
import { useAsyncTask, useAsyncRun } from 'react-hooks-async'

import { actions } from '../store'

export default connect(null, actions)(function SignIn (props) {
  const [formState, {text, email, password}] = useFormState()

  // NOTE: in some reason, if not to pass `[formState.values]`, useAsyncTask causes infinite rerendering
  const signInTask = useAsyncTask(() => props.signIn(formState.values), [formState.values])

  return (
        <form
          className="sign-in-form"
          autoComplete='on'
          method='POST'
          action="/login"
          onSubmit={(e) => {
            e.preventDefault()

            // NOTE: If I get it, there should be useAsyncRun, but react raises error
            // So, documenting that should be apparent, that is quite standard use-case
            signInTask.start()
          }}
        >
          <TextField
            id="merchantAccountNumber"
            autoComplete="username"
            label={"Merchant Id"}
            required
            margin="normal"
            variant="outlined"
            helperText={formState.errors.merchantAccountNumber}
            {...text('merchantAccountNumber')}
            disabled={signInTask.started && signInTask.pending}
          />
          <TextField
            id="email"
            autoComplete="email"
            label={"Email"}
            required
            margin="normal"
            variant="outlined"
            helperText={formState.errors.email}
            {...email('email')}
            disabled={signInTask.started && signInTask.pending}
          />
          <TextField
            id="password"
            autoComplete="current-password"
            label={"Password"}
            required
            helperText={formState.errors.password}
            margin="normal"
            variant="outlined"
            {...password('password')}
            disabled={signInTask.started && signInTask.pending}
          />
          <Button
            variant="contained"
            color="primary"
            size="large"
            type="submit"
            disabled={!!signInTask.error || (signInTask.started && signInTask.pending)}
          >
            Sign In
          </Button>
        </form>
  )
})

If you could comment the solution, I'd greatly appreciate @dai-shi

Well, I wouldn't use react-hooks-async for this purpose. It's much simpler without it.
  const [formState, {text, email, password}] = useFormState();
  const [signInStatus, setSignInStatus] = useState('ready');
  const isMounted = useRef(true);
  useEffect(() => () => { isMounted.current = false; }, []); // it's a bit tricky
  const startSignIn = async () => {
    setSignInStatus('pending');
    try {
      await props.signIn(formState.values);
      if (isMounted.current) setSignInStatus('done');
    } catch (e) {
      if (isMounted.current) setSignInStatus('error');
    }
  };

You can make this as a custom hook.
The whole point of react-hooks-async is abortability. So, useAsyncTask should be used with AbortController.

But isn't this almost the same as useAsyncTask? I believe useAsyncTask provides nicer abstraction over handling flags/errors/status, less congnitive load with implementation details in code - hardly can imagine managing ~3-4 sets of flags for 4 requests.

For me useAsyncTask seems nicer abstraction, hiding flags/implementation details. That's less error-prone. If we have 10 forms with junior devs, that can be implementation mess.

nicer abstraction

Yeah, I actually agree with that.

So, what you pointed out is:

a) this library is usable even without AbortController
b) this library is usable without useEffect (which is in the example, though)

I wonder how I should describe this.
There's a minor issue which might be critical. A task created by useAsyncTask is empty at first.
By empty, I mean task.start is null. It's available only in the second render. So, technically without checking the existence of task.start, it may cause an error.
What you can do is:
const signInTask = useAsyncTask(...);
if (!signInTask.start) return null; // avoid showing the form in the first render

and it's safe.

Oh, I see. That’s a bit unexpected.
Is there a reason that can’t be fixed? This and infinite loop are quite
annoying details.
On Wed, Apr 3, 2019 at 7:05 AM Daishi Kato<EMAIL_ADDRESS>wrote:

So, what you pointed out is:

a) this library is usable even without AbortController
b) this library is usable without useEffect (which is in the
example, though)

I wonder how I should describe this.
There's a minor issue which might be critical. A task created by
useAsyncTask is empty at first.
By empty, I mean task.start is null. It's available only in the second
render. So, technically without checking the existence of task.start, it
may cause an error.
What you can do is:
const signInTask = useAsyncTask(...);if (!signInTask.start) return null; // avoid showing the form in the first render
and it's safe.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dai-shi/react-hooks-async/issues/6#issuecomment-479442991,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AASUI0JjGRUK6Ozsvihk302bmnpEaQo3ks5vdIrugaJpZM4cXKJp
.

Is there a reason that can’t be fixed?

Past implementation dealt with it. However, it was a very complex implementation with useRef, and I felt like it may not work correctly in concurrent mode in some edge cases. The current implementation is much cleaner in that sense. This library is meant to use with useEffect (= useAsyncRun), and it's not a big problem with that use case.

infinite loop

Deps is required somewhere anyway for the useEffect use case.

nicer abstraction

So, that means there should be a better abstraction for your use case.
const initialState = {
  started: false,
  pending: true,
  error: null,
  result: null,
};

const reducer = (state, action) => {
  ...
};

const useAsyncTask = (func) => {
  const [state, dispatch] = useReducer(reducer, initialState);
  const start = async () => {
    ...
  };
  return { ...state, start };
};

Yes, AsyncTask abstraction is self-sufficient, just checked Android AsyncTask/Q API, the flow is quite similar.
Something like:
let {result, error, start, abort, started, pending, resolved, rejected, aborted} = useAsyncTask(promise, deps)

As I got several requests since then, I changed my mind and redesigned it.
Now, it will support the non-useEffect use case.
If you are still interested, please try #18.
Discussion should be merged to #17. Closing this issue.


Remove 'type_ann' field from Pattern

The type_ann field was moved from Pattern to Expr::Let(Let).  This makes more sense since Pattern is recursive and we only allowed type annotations at the top-level.

Codecov Report

Merging #215 (941dbaa) into main (6b01eff) will decrease coverage by 0.06%.
The diff coverage is 93.67%.

:exclamation: Current head 941dbaa differs from pull request most recent head 85e7ecb. Consider uploading reports for the commit 85e7ecb to get more accurate results

@@            Coverage Diff             @@
##             main     #215      +/-   ##
==========================================
- Coverage   92.61%   92.55%   -0.07%
==========================================
  Files          32       32
  Lines        3943     3935       -8
==========================================
- Hits         3652     3642      -10
- Misses        291      293       +2

Impacted Files
Coverage Δ

crates/crochet_ast/src/pattern.rs
89.65% <ø> (-3.90%)
:arrow_down:

crates/crochet_ast/src/expr.rs
78.12% <75.00%> (-0.45%)
:arrow_down:

crates/crochet_infer/src/infer_pattern.rs
89.02% <75.00%> (+1.92%)
:arrow_up:

crates/crochet_infer/src/infer_type_ann.rs
96.77% <84.61%> (-3.23%)
:arrow_down:

crates/crochet_parser/src/decl.rs
98.41% <93.75%> (+0.19%)
:arrow_up:

crates/crochet_infer/src/infer.rs
97.29% <100.00%> (-0.14%)
:arrow_down:

crates/crochet_infer/src/infer_expr.rs
95.77% <100.00%> (+0.02%)
:arrow_up:

crates/crochet_parser/src/expr.rs
99.64% <100.00%> (+0.01%)
:arrow_up:

crates/crochet_parser/src/pattern.rs
98.86% <100.00%> (+0.94%)
:arrow_up:

crates/crochet_parser/src/type_ann.rs
100.00% <100.00%> (ø)

... and 2 more

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


Improve ambigious base method extension

https://github.com/YongHee-Kim/JSONPointer.jl/blob/105ed61137dff9c933f8a0448844e2ddc497709c/src/pointer.jl#L145

Use macro

This would allow usage of JSONPointer on every dictionary, but won't be able to use default syntax with it (d[:key] = value ...)
https://juliahub.com/ui/Packages/SymDict/uRfhr/0.3.0

2.Wrap dictionary with custom struct
https://github.com/JuliaCollections/PropertyDicts.jl/blob/master/src/PropertyDicts.jl

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/67925
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.4.0 -m "<description of version>" b3f0532156e96cd7ed104d93b3d58f83ed8c463f
git push origin v0.4.0


Document our monitoring setup

Signed-off-by: Stephen Kitt<EMAIL_ADDRESS>

@tpantelis looks good to me, can you have an eye to skitt comments ^ ?