Command make serve in development env may fail due to too many open files

This is a Bug Report

Problem:
Command make serve fails on my laptop (come with macOS Catalina version 10.15.6 (19G73)) due to too many open files after running command make serve.
Built in 16332 ms
Watching for changes in /path/to/website/{archetypes,assets,content,data,i18n,layouts,static,themes}
Watching for config changes in /path/to/website/config.toml, /path/to/website/themes/docsy/config.toml, /path/to/website/go.mod
Environment: "development"
Serving pages from memory
Running in Fast Render Mode. For full rebuilds on change: hugo server --disableFastRender
Web Server is available at http://localhost:1313/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop
ERROR 2020/08/01 19:09:18 Error: listen tcp <IP_ADDRESS>:1313: socket: too many open files
make: *** [serve] Error 1

I've tried to change the maxfiles limit to 10240 by command sudo launchctl limit maxfiles 10240 unlimited, rerunning make serve got
Error: listen tcp <IP_ADDRESS>:1313: socket: too many open files in system

Seems 10240 exceeds the kernel ulimit. Then I changed my ulimit to 640000/1280000 according to https://superuser.com/a/1171028 , still got no help.
After append options --watch=false to hugo server --buildFuture command of target serve in Makefile, it works, but without hot reloading.
For someone who works on macOS Catalina, the following link may help:
https://discourse.gohugo.io/t/error-hugo-0-52-too-many-open-files/17653
https://superuser.com/a/1171028
It would be better if this issue could be well-documented, for example what's the good value of ulimit should I set.
Proposed Solution:

Provide two targets. make serve-live with option --watch=true to provide hot reloading, while make serve with option --watch=false not.
Well-documented guidelines to let new contributors know this issue, better if along with possible solutions.

Page to Update:
N/A

Maybe PR #22748 is related

Maybe PR #22748 is related and #22515 tries to handle this issue?

Great. Partial serve would be a good solution to this issue. This one could be closed, thank you!

Nice to hear that! 😊

Seems this issue needs to be reopened. I've tried Partial serve ，but make partial-serve LANG=en got a lot of errors such as
ERROR 2020/08/02 20:17:33 [en] "docs/reference/glossary/code-contributor.md": "sig" is not a valid glossary term_id, see ./docs/reference/glossary/* for a full list
ERROR 2020/08/02 20:17:33 [en] "docs/reference/glossary/daemonset.md": "node" is not a valid glossary term_id, see ./docs/reference/glossary/* for a full list
ERROR 2020/08/02 20:17:33 [en] "docs/reference/glossary/developer.md": "platform-developer" is not a valid glossary term_id, see ./docs/reference/glossary/* for a full list
ERROR 2020/08/02 20:17:33 [en] "docs/reference/glossary/csi.md": "storage-class" is not a valid glossary term_id, see ./docs/reference/glossary/* for a full list

or
Error: Error building site: "/path/to/website/content/de/docs/tutorials/hello-minikube.md:41:1": failed to render shortcode "codenew": failed to process shortcode: "/path/to/website/layouts/shortcodes/codenew.html:14:16": execute of template failed: template: shortcodes/codenew.html:14:16: executing "shortcodes/codenew.html" at <readFile $filename>: error calling readFile: file "content/en/examples/minikube/server.js" does not exist
make: *** [partial-serve] Error 255

Different language arguments leads different error messages.

/kind bug

Hi @blueabysm have you try doing make container-partial-serve LANG=en ?

Hi @blueabysm have you try doing make container-partial-serve LANG=en ?

Hi @irvifa . I've tried make container-partial-serve LANG=en, but got
➜  website git:(master) ✗ docker login
Authenticating with existing credentials...
Login Succeeded
➜  website git:(master) ✗ make container-partial-serve LANG=en
docker run --rm --interactive --tty --volume /Users/sailing/repo/open-source/website:/src --mount type=tmpfs,destination=/src/resources,tmpfs-mode=0755 -p 1313:1313 kubernetes-hugo:v0.74.3-477fda0143e3 hugo server --buildFuture --bind <IP_ADDRESS> --contentDir content/en
Unable to find image 'kubernetes-hugo:v0.74.3-477fda0143e3' locally
docker: Error response from daemon: pull access denied for kubernetes-hugo, repository does not exist or may require 'docker login': denied: requested access to the resource is denied.
See 'docker run --help'.
make: *** [container-partial-serve] Error 125

Before doing, I've pulled from this repo to ensure that the main branch of my fork points to the same as the upstream.

Seems like the Dockerimage doesn’t exist. You haven’t build that on your local yet. I need to investigate the one without Docker though.

Seems like the Dockerimage doesn’t exist. You haven’t build that on your local yet. I need to investigate the one without Docker though.

Appreciated for your work! 😃

@blueabysm can you take a look on https://gist.github.com/tombigel/d503800a282fcadbee14b537735d202c#gistcomment-3288161 This works for both of Catalina and Mojave

@irvifa Thanks for your reply. It works!
According to https://gist.github.com/tombigel/d503800a282fcadbee14b537735d202c#file-load-sh this script, what I missed is limit.maxproc.plist file.
BTW, I'll use this new account mainly from now on. The previous account @blueabysm will still receive email notifications.
How would you think if I write this down into https://github.com/kubernetes/website/blob/master/README.md?

Okay I'll write it down in my next PR. I'm working on it right now.

@rayw000 I put it on https://github.com/kubernetes/website/pull/23207

@rayw000 I put it on https://github.com/kubernetes/website/pull/23207

Haha you've done exactly what I meant to do. Thanks a lot! 😄

/close
via https://github.com/kubernetes/website/pull/23207


ButtonHelper "over" and "out" event swaps after changing enabled property of it

I am using button helper in my code for creating buttons and after clicking I use the enabled property of the button helper to disable the button and after few seconds I set the enabled property to true but after setting the enabled property to true the properties I had set before clicking gets swap
I have attached images depicting the problem I faced
Image MathFact_1: Depicts my original animation I need at "out"
Image MathFact_2: Shows the over animation
Image MathFact_3: Shows the down animation
Image MathFact_4: Shows the over animation after the click event is fired
Image MathFact_5: Shows the out animation after the click event is fired

Here's the code for the same
class BaseButton extends createjs.ButtonHelper{
currentListener:any;
currentState: GameButtonState = GameButtonState.INACTIVE;
preOverState: GameButtonState;
keyBinds: number[] | null = null;
public KBListener: any;
isListening: boolean = false;
constructor(public target: createjs.MovieClip){
    super(target,"up", "over", "down",false);
    this.target.on("click",this.buttonEvent.bind(this));
}

protected addListener(){
    if(!this.isListening)
    {
        console.log(this.currentListener);
        this.isListening=true;
        this.enabled = true;
        this.target.gotoAndStop("up");
    }
}

protected buttonEvent(event):boolean{
console.log(event);
debugger;
this.removeListener();
setTimeout( () => {
this.addListener();
}, 5000)
return true;
}
protected removeListener():void{
this.isListening=false;
this.enabled = false;
this.setMouseDefault();
}
protected setMouseDefault():void{
let element: HTMLElement | null = window.document.getElementById("canvas");
element!.style.cursor = "default";
}
}

the images are in order 4,5,1,2,3


Firebase codelab is missing step to enable storage?

RE: https://codelabs.developers.google.com/codelabs/flutter-firebase/#9
We could not get this step to work. After running the app, and selecting the image, the app failed with an "-13000" error in the log file.
Going to the Firebase console, opening the Storage section, and selecting "Get Started" seemingly fixed this.
This bug was found during a "dryrun" of the Firebase codelab in AAR by @floitschG, @mravn-google, and @mit-mit.

@collinjackson does this ring a bell? is this a required step?

This is a duplicate of #35 in the friendlychat-steps repo, where we've been tracking other codelab bugs.

(I believe it has been fixed)

It has been fixed.
@mit-mit @floitschG @mravn-google
Feel free to ping before dry running through any codelab. There are likely fixes that you can't see... and a update process you can plug into to stay in sync. Thanks!


Typo in Eq (4.57), pg.161, version 2023-08-15

the middle side of the equation (eq.4.57) needs to close parenthesis for the marginal likelihood: $\log{\sum_{n=1}^{N}p\left(\boldsymbol{x}_{n}\mid\boldsymbol{\theta}\right)}$

fixed


flash player: event "progress" still fired from a previous a source after an update of source

Tested on firefox 19 on debian with only the MediaElement class (not the player)
Symptom: after a source update, the progress gauge display both the previous download progress and the new one.
To be sure this is a flash issue, here is a log i have added to mejs.MediaPluginBridge.fireEvent
    if (eventName === "progress") {
        console.log(eventName , values.bufferedBytes , values.bytesTotal, values.src);
    }

I  first play a "a.mp3" and then change source to b.mp3, during a noticable time, the event "progress" is still fired from "a.mp3" load with its data and with "b.mp3" as source, so, no way to filter on source to avoid it. The only ugly way should be to test the bytesTotal :/
Here is an extract  of my console

load a.mp3:

13056  4334446 http://<IP_ADDRESS>/media/a.mp3
...
1338922  4334446 http://<IP_ADDRESS>/media/a.mp3
...

change source to b.mp3

55494 8135779 http://<IP_ADDRESS>/media/b.mp3
1620556 4334446 http://<IP_ADDRESS>/media/b.mp3   << previous buffer + new source
63210 8135779 http://<IP_ADDRESS>/media/b.mp3
1684856 4334446 http://<IP_ADDRESS>/media/b.mp3   << previous buffer + new source
86358 8135779 http://<IP_ADDRESS>/media/b.mp3

I wish to have debug the flash, but it's not in my skills right now :/

@jbdemonte Closing old issues. If you'd like to reopen this, let us know. Thanks!


Custom card number and holder name text style

Hello!
Thanks a lot for this very useful package!
I am trying to use it on a mobile banking app and would like to customize text (text size especially). What about adding new features according to card's text customization
Regards

I initially had plans to add that, I guess I forgot about adding that. Thanks for reminding me. I'll add that next.

Hello may I add the capability to center the number card?
thank you


Skip nonexisting target

You can follow the discussion on issue #103

Thanks for the fix! Merged with minor changes in 4468fff67a9e9d9f611d5dade4a9486e75dafa1c.

On 11/19/2016 05:28 PM, Anish Athalye wrote:

Thanks for the fix! Merged with minor changes in 4468fff
https://github.com/anishathalye/dotbot/commit/4468fff67a9e9d9f611d5dade4a9486e75dafa1c.

I didn't get why you removed the Skipping word from prefix. I think it
was a good way to inform the users that one of the entry has been skipped
--
tommy.ael
gpg key: 0xEBD8B3682C4C4645

Oh, just to match previous behavior: https://github.com/anishathalye/dotbot/blob/master/plugins/link.py#L163

On 11/19/2016 05:56 PM, Anish Athalye wrote:

Oh, just to match previous behavior:
https://github.com/anishathalye/dotbot/blob/master/plugins/link.py#L163

It's fine then :)


Improperly Formatted NTRIP sourcetable response

Improperly Formatted NTRIP sourcetable response
Describe the bug
The NTRIP sourcetable HTTP response from socket_server.py  is missing the \r\n separator line that should exist between the http header section and the body.  This causes some http clients to treat the sourcetable data as a header, and can result is some clients hanging and timing out while they try to read the additional bytes specified in the Content-Length: header.
Running pygnssutils version 1.0.21, but issue exists in main branch
To Reproduce
Steps to reproduce the behaviour:

Start the socket_server in NTRIP mode
Use a HTTP client such as cURL to request the sourcetable

$ curl http://<IP_ADDRESS>:2101/ -H 'Connection: close' -H 'User-Agent: NTRIP pygnssutils' -H 'Accept: */*' -vvv -m 3

Here is the current output when running in verbose mode (-vvv):
:~$ curl http://<IP_ADDRESS>:2101/ -H 'Connection: close' -H 'User-Agent: NTRIP pygnssutils' -H 'Accept: */*' -vvv -m 3
*   Trying <IP_ADDRESS>:2101...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 2101 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>:2101
> Connection: close
> User-Agent: NTRIP pygnssutils
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Ntrip-Version: Ntrip/2.0
< Ntrip-Flags:
< Server: pygnssutils_NTRIP_Caster_1.0.21/of:10 Apr 2024
< Date: Wed, 10 Apr 2024 20:17:32 UTC
< Connection: close
< Content-Type: gnss/sourcetable
< Content-Length: 241
< STR;pygnssutils;PYGNSSUTILS;RTCM 3.3;1005(5),1077(1),1087(1),1097(1),1127(1),1230(1);0;GPS+GLO+GAL+BEI;SNIP;SRB;40.0;-105.0;0;0;sNTRIP;none;B;N;0;
< NET;SNIP;pygnssutils;N;N;pygnssutils;<IP_ADDRESS>:2101;info@semuconsulting.com;;
< ENDSOURCETABLE
* Operation timed out after 3001 milliseconds with 0 out of 241 bytes received
* Closing connection 0
curl: (28) Operation timed out after 3001 milliseconds with 0 out of 241 bytes received

The sourcetable content can be seen in the response headers, and cURL times out trying to read data that will not arrive.
Expected Behaviour
I confirmed in the NTRIP 2.0 Spec that the expected output is the sourcetable in the HTTP body.
Running cURL in verbose mode (-vvv)  the expected output would be
:~$ curl http://<IP_ADDRESS>:2101/ -H 'Connection: close' -H 'User-Agent: NTRIP pygnssutils' -H 'Accept: */*' -vvv
*   Trying <IP_ADDRESS>:2101...
* TCP_NODELAY set
* Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 2101 (#0)
> GET / HTTP/1.1
> Host: <IP_ADDRESS>:2101
> Connection: close
> User-Agent: NTRIP pygnssutils
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Ntrip-Version: Ntrip/2.0
< Ntrip-Flags:
< Server: pygnssutils_NTRIP_Caster_1.0.21/of:10 Apr 2024
< Date: Wed, 10 Apr 2024 20:14:40 UTC
< Connection: close
< Content-Type: gnss/sourcetable
< Content-Length: 241
<
STR;pygnssutils;PYGNSSUTILS;RTCM 3.3;1005(5),1077(1),1087(1),1097(1),1127(1),1230(1);0;GPS+GLO+GAL+BEI;SNIP;SRB;40.0;-105.0;0;0;sNTRIP;none;B;N;0;
NET;SNIP;pygnssutils;N;N;pygnssutils;<IP_ADDRESS>:2101;info@semuconsulting.com;;
ENDSOURCETABLE
* Closing connection 0

In the above example, the sourcetabe data is not returned in the headers, and the client does not timeout waiting for data.
Desktop (please complete the following information):

Windows 11 with WSL2

Thanks for the quick response!  I was able to test it out and your code change is working perfectly!  Would you like me to close the issue from my side, or would you like to close it?

Thanks @dclemmon . The issue wil be closed automatically once the branch is marged with main, which will hopefully be in the next week or so.
Thanks again for the comprehensive issue report - most helpful.


Loading error

The expense app upon installing opens fine. However, if I try to edit its throwing an error "Something went wrong while opening your app". There's no further details.

Regards,
Praveen

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

ID: 379862e0-89e8-8e6b-b686-77d6836f0434
Version Independent ID: 29fc7f25-b6b5-26d2-69f3-eada7fff64b3
Content: Install and configure the Expense Report sample for canvas apps - Power Apps
Content Source: powerapps-docs/maker/canvas-apps/expense-report-install.md
Service: powerapps
GitHub Login: @matthewbolanos
Microsoft Alias: mabolan

Thank you for reaching out. Documentation has been updated recently, please refer to the updated documentation.


scripts entry in package.json

There is an entry in scripts in package.json:
"scripts": {
    "update-grammar": "node ../../build/npm/update-grammar.js grammars/ejs.json ./syntaxes/ejs.json"
}

However, update-grammar is not a standard script, at least it's not listed in the official docs (which list one script, and point to the npm docs for other standard npm scripts, where it's also not listed). I therefore think it should be removed entirely. Of course, unless there is a purpose that I can't find!

This is run using npm run update-grammar.
As for its purpose, the author of this package uses it to update the ejs.json TextMate language definition.


Selectize wait for options

I have found a case where the set_selectize utility can try to set the value before the options have been created by the selectize library.
Adding a wait_for_visible(selectize_control + ' .has-options') makes sure the options are available before filling in the control.

The travis build is showing the "Address already in use" error. It looked like the flag in the custom class was supposed to fix that?

@spockNinja I didn't test it with multiple test files. :-/ I don't know why it's doing that. We might want to go flask for testing after all.

Or run it as a process instead of a thread so we can kill it after.

@spockNinja You'll want to bump the version number in setup.py to be 1.4.2 as well

I'm going to go ahead and merge this and I'll tweak it a bit.


[BACKLOG-25488] Updates Kettle Trans status page to match new styles

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetJobStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/resources/org/pentaho/di/www/messages/messages_en_US.properties
  engine/src/test/java/org/pentaho/di/www/GetTransStatusServletTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.checkdbconnection.JobEntryCheckDbConnections

Branch Change: -1.3% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.1% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -3.1% :small_red_triangle_down:
Complexity Change: -3.1% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -0.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.www.GetTransStatusServlet

Branch Change: -4.6% :small_red_triangle_down:
Instruction Change: -31.4% :small_red_triangle_down:
Line Change: -25.4% :small_red_triangle_down:

:new: org.pentaho.di.www.StatusServletUtils

Branch Coverage: 0% :exclamation:
Complexity Coverage: 16.7% :exclamation:
Instruction Coverage: 16.1% :exclamation:
Line Coverage: 23.1% :exclamation:
Method Coverage: 50% :exclamation:

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change: -0.1% :small_red_triangle_down:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java

:heavy_exclamation_mark: Unused import - java.io.File. at Line: 25 Column: 8

:heavy_exclamation_mark: Unused import - javax.xml.parsers.DocumentBuilder. at Line: 39 Column: 8

:heavy_exclamation_mark: Unused import - javax.xml.parsers.DocumentBuilderFactory. at Line: 40 Column: 8

:heavy_exclamation_mark: Unused import - org.w3c.dom.Document. at Line: 48 Column: 8

/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java

:heavy_exclamation_mark: Unused import - java.io.File. at Line: 25 Column: 8

:heavy_exclamation_mark: Unused import - javax.xml.parsers.DocumentBuilder. at Line: 35 Column: 8

:heavy_exclamation_mark: Unused import - javax.xml.parsers.DocumentBuilderFactory. at Line: 36 Column: 8

:heavy_exclamation_mark: Unused import - org.w3c.dom.Document. at Line: 54 Column: 8

/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java

:heavy_exclamation_mark: '{' is not preceded with whitespace. at Line: 33 Column: 42

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetJobStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/resources/org/pentaho/di/www/messages/messages_en_US.properties
  engine/src/test/java/org/pentaho/di/www/GetTransStatusServletTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.7% :small_red_triangle_down:
Complexity Change: -3% :small_red_triangle_down:
Instruction Change: -0.7% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -0.5% :small_red_triangle_down:

org.pentaho.di.www.GetTransStatusServlet

Branch Change: -4.6% :small_red_triangle_down:
Instruction Change: -31.4% :small_red_triangle_down:
Line Change: -25.4% :small_red_triangle_down:

:new: org.pentaho.di.www.StatusServletUtils

Branch Coverage: 0% :exclamation:
Complexity Coverage: 16.7% :exclamation:
Instruction Coverage: 16.1% :exclamation:
Line Coverage: 23.1% :exclamation:
Method Coverage: 50% :exclamation:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java

:heavy_exclamation_mark: '{' is not preceded with whitespace. at Line: 33 Column: 42

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/resources/org/pentaho/di/www/messages/messages_en_US.properties
  engine/src/test/java/org/pentaho/di/www/GetTransStatusServletTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.job.entries.evalfilesmetrics.JobEntryEvalFilesMetrics

Branch Change: -0.4% :small_red_triangle_down:
Complexity Change: -0.5% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Method Change: -1% :small_red_triangle_down:

org.pentaho.di.trans.steps.ifnull.IfNullMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -1.1% :small_red_triangle_down:
Complexity Change: -1.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.www.GetTransStatusServlet

Branch Change: -4.6% :small_red_triangle_down:
Instruction Change: -31.4% :small_red_triangle_down:
Line Change: -25.4% :small_red_triangle_down:

:new: org.pentaho.di.www.StatusServletUtils

Branch Coverage: 0% :exclamation:
Complexity Coverage: 16.7% :exclamation:
Instruction Coverage: 16.1% :exclamation:
Line Coverage: 23.1% :exclamation:
Method Coverage: 50% :exclamation:

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java

:heavy_exclamation_mark: '{' is not preceded with whitespace. at Line: 33 Column: 42

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -Dsurefire.runOrder=alphabetical -Daudit -amd clean install

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/www/GetStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/GetTransStatusServlet.java
  engine/src/main/java/org/pentaho/di/www/StatusServletUtils.java
  engine/src/main/resources/org/pentaho/di/www/messages/messages_en_US.properties
  engine/src/test/java/org/pentaho/di/www/GetTransStatusServletTest.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes.


Some warning message, can we do dir check?

shiny::runApp('.', port=9877, launch.browser=T)
Loading required package: shiny
Warning: replacing previous import ‘limma::plotMA’ by ‘DESeq2::plotMA’ when loading ‘MetaDE’
Warning in dir.create(.Object@dir, recursive = T) :
'data/studies' already exists
Warning in dir.create(paste(".Database", name, "meta", sep = "/"), recursive = T) :
'.Database/studies/meta' already exists
Warning in dir.create(paste(".Database", name, "working", sep = "/"), recursive = T) :
'.Database/studies/working' already exists
Warning in<EMAIL_ADDRESS>recursive = T) :
'.Database/studies/active' already exists

Anzhe will improve default working directory to solve these problems.

That's good. But meanwhile checking if wd is null is still helpful
On Friday, August 26, 2016, Zhiguang Huo (Caleb<EMAIL_ADDRESS>wrote:

Anzhe will improve default working directory to solve these problems.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/metaOmic/metaOmics/issues/9#issuecomment-242757956,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARPM5PXza0lvWvW2ENv3WxR3DC61cWLyks5qjv33gaJpZM4JteW7
.

@xiaoguang1988 this is not urgent, I will leave it for now

Sounds good.

@xiaoguang1988 I decided to enable all the warning messages. They are very important for debugging after all. I spent the whole morning and an hour in the afternoon debug for something I could have been found in warning message.

This is not urgent.
My suggestion is not to suppress warning message.
My suggestion is we may do dir check before creating dir. If dir exist, we won't bother create a new one.
Anyone, this is a minor issue.

But this is exactly the same behaviour as the default behaviour. The warning message is just so you know there is a directory, it won't create a new one anyway if the directory exists?

Alright, I see.


Add support for possible extra : in input names

This PR adds support to handle node/tensor names with : in their names in the NodesToKeep list.

@aaroey I added the missing space. Can you review?

Thanks @rmlarsen!


Loadable

To use it:
Loadable add to the children two props:

toggleLoading
stopLoading

    <Loadable>
      <ContentThatLoads />
    </Loadable>

When Loading:

When Loaded:

The loadable start with the spinner on

I would use a more explicit startLoading instead of toggleLoading so that we know each time what is the status of the component

OK, I can change this and add "startLoading".


Runestrike no longer executed for Tank DK sim

Describe the bug
The ability "runestrike" is no longer being executed when running a DK-tank sim, both noticable by looking at it missing in the "results" tab as well as generally way less DPS than before. This must have happened in the last few days/weeks, as with the new APL-options, runestrike initially worked as intended. While I have a manual APL-setup, it still happens even with the legacy or default-APL variants.
To Reproduce
Steps to reproduce the behavior:

Open the link I provided below of my sim
Click on "Simulate"
Open the "Results" tab
See runestrike missing

Expected behavior
Runestrike should be executed as part of the rotation normally
Sim Links and Screenshots
https://wowsims.github.io/wotlk/tank_deathknight/#eJztVVtoVEcYPv9/Nuuc30smk6gnY3Z7el5MFyKzu6xVaTlbX/oaLUIpfZGihNqHVAulgcIqXlqhpJTWy9LW0pYQfYhFEROKoOmFGgxNIaa0D1KzuG6T1garsOKDzrnssqEtKLFv/uy5zHf5///MDLO0miFHGxT2QC8UAAYBvgI4DzABUAGoAk4gFhA2OhxscEBBHnzpO4F02Nfhb4BVgA2GMOQA7YF492tb39q2kxEneQWomQ9W0B5IuvEvv8HyCJLFZ26ia17Qr0v4Cc2VEq5ZDkaXbqL9UUc4Erx6A+1qwrW0shz8aCkvacXBpBvTmEGcnyyh/WHSXRRKDF1sUuducAg+ph035mVp5t9r7FCHGx8fiTo6VsKwqsXPzEXNxfhsObj/6FumtOWA/ogwDRmaGLmFLunZa1JWt/mi+TneRjqPxuNYUAj1sI6+xQxsdKCHBoH2IKOYA50gjYPwlFK5nMqkc9msUvrKprNpPfTvuXRXNpNTSmW6tCgzDTb78xryTyfRPj2FzpFJ7OyfRHX0MlYgNgdWEVoorhf9u+vYsj2IKW8U3499BkCtxKTmWHEc3UXURGZXblcdrM7WwHQd5J+MNIBtkfLAHXSZX6Qrq3ZRgoTktIy9u8K5Vwuom8Ry2Uq6Kd/319emjJ7ClitoiUvEMj5ydihk7o6bQkq7gRnQjK7Fdu83HRDPyA2k5BpanmplMcFerXl1LYK18+CBOhx+8numSMhV1JJqZnFhjoHhw4RPq4Deew8FyfpEiHVyLaVEJ4kU1/KmDxAZhunSjVhUIh20uDdosZZl4m8UjkzW1DOADBrVV0766qTsiBRBS8GspbNhT9f3mXU+Ni9DyP9Swga+wZ8J+cOjpmiXK2mpu5isUxAtZ21Z9ZH4KySsLX4cO5dvifZovj18VjwnXkTDLZpWMC5sr0na8u3Fo35c9NaHyIT3ZG2nZWaw9+O3+6zurW+80vPmtp07nBdslvqfD5u53flHkaZw8duF5FHrUr3PPYo+/i2CmS4cms5HFSqe/mPbHJzjT+yb3TScvOxlgyh5oWDOW2jJ4aSfeMh7/Vn6+Yvea94fm3zkJ+9qX9vQ81fHvM2B4Ic6/195ws1y3Js//v0f+nALlb3+0dudL/dPL7j/Bw2hxi/5wfIP6njpoU/++wuzMjM=
Desktop (please complete the following information):

OS: Windows 11
Browser: Firefox
Version: 116.0.2

Additional context
Add any other context about the problem here.

It seems there were some changes in the tank targetsl. Can you try restoring your defaults and then selecting your tank target again? This fixed it for me.

Yes, restoring defaults fixed the same setting for me. Closing issue


OADP-612: Partially fail backup if VSB fails

Related: https://github.com/openshift/velero-plugin-for-csi/pull/40

closing for 1.9 branch


Pathway may appears downstream from gene selection, but with no data mapped to it-alpha

Gene selection from heatmap...choose pathway summary tool.  Select pathway.  Select map, and while the map loads, none of the genes appear on it.

I cannot reproduce this problem. Would you please provide url or some more detail (what genomes/genes selected?)

checked from both PF and transcriptomics heatmaps. Seems to be working fine.


UPDATE enable https traffic only as default

Hi all,

[x]  Please check this box and fill the data as below once you have run the unit tests.
Code coverage should keeps at 100%.

pushd src/bosh_azure_cpi
  ./bin/test-unit
popd

Changelog

Add supportsHttpsTrafficOnly (true) as default
If you check the documentation you will see that the property will be also true as default when you're using API Version > 2019-04-01
https://docs.microsoft.com/en-us/rest/api/storagerp/storageaccounts/create#storageaccountcreateparameters
Thats also the reason why I didn't add a parameter to disable / toggle the setting.

Kind regards

Checking their docs, it mentions that custom domains don't support https: https://docs.microsoft.com/en-us/azure/storage/common/storage-require-secure-transfer . I can't tell if it will ignore this property and default to http or fail entirely.

Hi @mrosecrance,
I will change the option as a parameter

Hi @mrosecrance,
I changed to options to pass a config value to enable / disable that option.
Can you please check that again?
Kind regards

Hello Tobi -
There are two broad concerns I have with this feature.

We've been pushing everyone away from StorageAccounts towards Managed Disks. I know that not every SA feature is yet available in managed disks (for example, customer-provided encryption keys were missing last I checked). What is keeping you from moving to Managed Disks?

The CPI's primary job isn't to create storage accounts. This type of convenience feature does not exist on other CPIs, and if we could easily drop it at this point we would. For folks wishing to customize the creation of their "convenience resources", we want you to pre-create these in their already desired configuration. Would that be an acceptable solution?

We're not saying no 100%, let us know the answers to the questions above, but it's likely this doesn't make sense to merge. Our apologies for the back and forth, we're just internally teaching each other more about Azure on the team.


fix(gatsby-theme-minimal-blog): Add post slug to the url meta tag

Otherwise, when sharing a post, clicking on the rendered link always directs user to the blog frontpage

Thanks!


Missing features: count by author and filter by time range

By author might be a good feature. But for the time range, I think, don't be too specific (I can guess you are comparing with npm-stat.com) 1 2 3 weeks/1 3 6 months/1 year is a good option. I don't think there is need to add a time picker, just options

Yeah, product  balance.


IPv4-in-IPv6 root node is wrong

The definition in question:
https://github.com/g-andrade/locus/blob/8d0019d3010fdd7a1bf6db68488d7be36eda7664/src/locus_mmdb_tree.erl#L48-L49
Following tentative PR #39 a few months ago and, more recently, evidence of this in #43 .
As per the spec, the root node seems to be ::/96 rather than ::ffff:0:0/96 (which is for IPv4-mapped IPv6 addresses) and I had read it wrong all this time:
https://maxmind.github.io/MaxMind-DB/#ipv4-addresses-in-an-ipv6-tree
Official Python implementation mentioned by @nickjacob in #39, which further suggests this in the way it walks over the IPv6 tree to get the IPv4 root node:
https://github.com/maxmind/MaxMind-DB-Reader-python/blob/b59335627a27b96c6e5a3178632ed0ab77d53cfb/maxminddb/reader.py#L118-L127
This being true, the existing behavior worked purely out of chance: all the databases I had tested so far happened to account for IPv4-mapped IPv6 addresses, and therefore I didn't run into the bug for 6+ years.

Fixed in 75e38f3.

Released in 2.3.8, pushed to Hex

If there is a clarification to the specification that can be made, we would be happy to include that. The root node for IPv4 is at ::/96 with the other mapped locations being (optional) pointers to that. All the MaxMind databases are build with these pointers, but third-party databases may have disabled them.


Add API to play local gif file by creating an new animation.

Will nee options to control the fit, such as CenterAndCrop / StretchToFit.

command is done by 5724dec146f8184c18ff02bc5dff23c219f42f22. but no flag support yet.


Display video title and poster when playing

This is great.  It's something I've been meaning to do for a while and never got around to.


remix-serve isn't load balancer friendly

What version of Remix are you using?
1.7.1
Steps to Reproduce
Configure Remix to run inside an ECS container.
Setup an Application Load Balancer instance and point this at Remix.
Thrash the server, eventually (not very long after) you'll start seeing 502 errors!
This causes a failed health check from ALB and the container dies, or in this case is savagely murdered by the load balancer.
Expected Behavior
These 502s shouldn't occur.
Remix should have an option to configure the default keepAliveTimeout.
Actual Behavior
The socket gets closed by Remix, which then causes the load balancer to panic!
Silly brittle things they are.

For more info and a proposed solution:

This is a known issue in NextJS too.
https://nextjs.org/docs/api-reference/cli#keep-alive-timeout
While I recognise this framework tends towards functions, it does support fixed compute too of course. Some non-AWS services don't have configurable upstream timeouts so it should be able to be implemented without having to use the ejector seat. i.e. a custom server.

I propose a config key and a CLI flag. I'll try to implement this into remix-serve and add a caveat to the gotchas page.

Might take me a few days as I'm pretty busy, but I'll get to it if I can.
Feel free to swoop in if you have more time 🙂

As if this a feature request, I'm going to convert it to a Proposal


unknown type: image

get the following in iobroker:
openhab.0 | 2022-05-26 10:52:32.989 | warn | oh2iob - Unknown type: image
fixable?

Hi,
Anything is fixable!
Is it fixable with these many information you provided... most likely not :-)
Please enable the debug loglevel for the openhab instance on "instance" tab (with active export mode) and post the full log from adapterstart. Please get log from logfile on disk and not from admin ... /opt/iobroekr/log/...)
There is an object from OpenHab with type "image" which is not supported currently to map to iobroker ... so we need to investigate it in depth

Sorry for late answer - kicke openhab out of my iobroker .. so no logs from me sorry ...


Script crashes if line shares a polygon edge

The following causes the script to crash, with:
#
# Fatal error in , line 0
# Fatal JavaScript invalid size error 169220804

Features to replicate:
const line = {
  type: 'Feature',
  geometry: {
    type: 'LineString',
    coordinates: [
      [80, 90],
      [80, -90]
    ]
  }
};

const poly = {
  type: 'Feature',
  geometry: {
    type: 'Polygon',
    coordinates: [
      [
        [70, 20],
        [90, 20],
        [90, 10],
        [80, 10],
        [80, 0],
        [70, 0],
        [70, 20]
      ]
    ]
  }
};

const output = polygonSplitter(poly, line);

This seems to happens every time an exact edge is shared.
If a fix is not possible, any ideas for a usable workaround?

Thanks for the reproducible test case @danielfdsilva
I've done a little digging and have surfaced a few issues

currently the algorithm is generating 5 intersection points

looking at the picture above we'd expect 3, but we get 2 extra because 2 of the intersections occur on polygon corners

Currently the algorithm expect an intersection point to be associated with a single polygon edge, but in those cases that occur on a corner an intersection point is associated with 2 edges

I've tried to implement a few work arounds for those issues, but even then the shared edges are proving problematic at the moment... I'll keep giving it some thought but also open to external suggestions :)


QueryBuilder work with StringBuilder

QueryBuilder changed to work with only StringBuilder to optimize query generation

Thanks for this optimization mate 😄


Make app fixture session scope to speedup tests

Gives ~30% speed improvement on my local machine without xdist

Codecov Report

Merging #278 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #278      +/-   ##
==========================================
- Coverage   99.91%   99.91%   -0.01%
==========================================
  Files          19       19
  Lines        1184     1175       -9
  Branches      187      187
==========================================
- Hits         1183     1174       -9
  Partials        1        1

Flag
Coverage Δ

#unittests
99.91% <ø> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/extra/pytest_plugin.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/runner/events.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/models.py
100% <0%> (ø)
:arrow_up:

src/schemathesis/lazy.py
100% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Allow creating Darwin controllers via alloc/initWithParameters.

Fast-tracking platform-specific fix with platform owner review.


Access Denied [PCSE01356]

App summary

App name: Access Denied
App serial: PCSE01356
App version: 1.0

Vita3K summary

Version: v0.1.5
Build number: 2786-7ac13c3c
Commit hash: https://github.com/Vita3K/Vita3K/commit/7ac13c3c3ce219a7548a815a1641e45863da50f3
CPU backend:Dynarmic

Test environment summary

Tested by: benderscruffy
OS: Windows 10
CPU: Intel Core i7 920
GPU: AMD RX 580
RAM: 16 GB

Issues
Nothing
Screenshots

Log
vita3k.log
Recommended labels

Nothing

Vita3K summary

Version: v0.1.6
Build number: 2840
Commit hash: https://github.com/vita3k/vita3k/commit/4a2c6582
CPU backend: Dynarmic

Test environment summary

Tested by: othmanead
OS: Windows 11
CPU: Intel i5-10300H
GPU: RTX 3060 mobile
RAM: 16 GB

Issue:
Game plays with no issues
Screenshots:

Log
vita3k.log
Recommended labels

Playable


Do Not require an enter/Done on values

If I want to Award or adjust DKP or Change DKP value of items in Award Screen or If i want to bid a DKP value Not prefilled the UI requires an enter/Done. To use the UI in an easier way there should Not be a necessary "enter" on values

This will not be changed unless we change the GUI engine which wont come soon.
Remember that pressing Enter after putting value in also works


IMXRT USB speed may need to be forced to FULL when CONFIG_USBDEV_DUALSPEED is not configured

I am working with a Teensy4.1 and had a consistent hardfault whenever I connected QGC to Mavlink over USB.  I eventually realized that I did not have CONFIG_USBDEV_DUALSPEED enabled, but line 1975 of imxrt_usbdev.c (https://github.com/apache/incubator-nuttx/blob/master/arch/arm/src/imxrt/imxrt_usbdev.c#L1975) was being executed at startup.  The iMXRT1062 (Teensy4.1 SOC) does support HIGH speed, so I enabled CONFIG_USBDEV_DUALSPEED and the crash disappeared.
It seems there should be some coordination between CONFIG_USBDEV_DUALSPEED and the link's speed at startup.  UPDATE: I thought I had something working consistently, but further testing fails.  There are a few iMXRT1062-register bits that apparently force the link to FULL speed, but I haven't been able to consistently use any of those to avoid execution of line 1975 (imxrt_usbdev.c) at startup.
Any thoughts on this?

LOL!  I wish I did have a friend at NXP!  I was just posting an update as you added your comment...
After a few days of testing, my crash is not resolved.  Prior to enabling CONFIG_USBDEV_DUALSPEED, I was not able to do any sensor calibration through QGC (because of the crash); then after enabling it I was able to calibrate (accelerometer)  so I was off on a tangent for a few days.  Now, I am seeing that it just crashes at a different point (still related to USB/QGC somehow).
I also added a line of code to imxrt_usbreset() that will keep line 1975 from being called when CONFIG_USBDEV_DUALSPEED is not enabled, but at this point I don't know if that has any value (to include a possible negative effect).  Bottom line... I'm back where I started, still trying to understand this crash.

@edsut did you try to ask help from PX4 community as well since it is also related to QGC? I think there are more people using it and they will help to validate if your modification. After that I think it could submit a PR here with the right solution. I have iMXRT board but I never used QGC before.

I am ooo til 7-5. I will look at it ASAP when I return

@acassis  I did post the question there prior to this... https://discuss.px4.io/t/mavlink-hardfault-on-usb-connection/27856
No response.
My gut says its not QGC specific (my hunch is still that it is deep in the USB driver).
Prior to enabling CONFIG_USBDEV_DUALSPEED, I was able to cause the crash with a snippet of code that just wrote a large chunk of data to USB (see https://discuss.px4.io/t/possible-bug-in-imxrt-usb-interface/28005).  Now (with CONFIG_USBDEV_DUALSPEED enabled, that reproduction snippet doesn't crash, so at the moment my only way to reproduce it is with certain interactions with QGC.
One new bit of information...
I wrote a Linux program that monitors the connection between QGC (on laptop) and my Teensy4.1.  With that inserted between the two devices the above crash does NOT occur!!  Clearly with that inserted, buffering is different from end-to-end, but data (messages sent between the two) should be identical.

The details of this issue are scattered (my fault) across a few forum posts and github issues so I'll try to recap here...
Note in all cases, "crash" refers to a hardfault-CPU-crash, not a physical crash of a drone...
I first noticed a consistent crash while doing calibration tests with QGC and PX4 (running Mavlink over USB).  Unfortunately the address of the fault was inconsistent, so it was not an easy event to trace back to.  Eventually I found I was able to reproduce the crash by simply writing a large block of data to the USB interface running just NuttX (i.e.  this is not a PX4/Mavlink problem).  After more testing/analysis I discovered what I thought might be the problem (refer to my initial post in this issue), so I enabled CONFIG_USBDEV_DUALSPEED, and after that I was not able to cause the crash by writing a large block of data to USB... While I think that is a valid change, after further investigation, it appears that the bug is still lurking.   I know this because now I can consistently cause a crash at a different point in my QGC<->PX4 interaction.  Apparently enabling CONFIG_USBDEV_DUALSPEED just caused the bug to show itself in a different way.
All that being said, the easiest way to reproduce this problem is to leave CONFIG_USBDEV_DUALSPEED disabled and write a large (1000 bytes) block of data to USB.  I've reproduced the problem with NuttX-10.2.0 and NuttX-10.3.0 (master).  While I can reproduce the crash with the standard teensy-4.x:nsh-4.1 configuration, that doesn't show the dump of the hardfault.  I modified the configuration to run the console on a UART so that the USB driver was not involved with the console.  In this case the dump from the hardfault-crash is shown on the UART-console (but doesn't show a useful backtrace).
All it takes is this code (I just created an example app to invoke this)...
int fd = open("/dev/ttyACM0", O_RDWR | O_NOCTTY));
write(fd,(uint8_t *)0x20200000,1000);
close(fd);

Note that this code ONLY causes the crash when CONFIG_USBDEV_DUALSPEED is not enabled.  When I enable CONFIG_USBDEV_DUALSPEED, the reproduction scenario gets a bit more complicated; however I believe they are one-and-the same bug.  If anyone has a clue (I have yet to dig deep into the guts of iMXRT1062 USB/Teensy4.1 driver; but I suspect that is where the problem lies)

More information...
Thanks to suggestion by @acassis, I used the usbserial application to cause the crash.
Its tough to work with unless a new defconfig is used to allow a UART to run as the console.
Note also that when I enabled CONFIG_USBDEV_DUALSPEED, then usbserial did not crash.
@davids5 any thoughts on this?

More information:
I now have DCACHE turned off and CONFIG_USBDEV_DUALSPEED enabled.  Crash still occurs...
Running usbserial app, consistently takes a hardfault, and I believe the value of privep->head at line 908 of imxrt_usbdev.c has a corrupt pointer at the time of the crash.  I will be OOO from now till Monday.  Will continue analysis then.

@davids5  I've got to be getting close here...
I now see that just prior to the hardfault, the value of bytesleft in imxrt_usbdev.c is negative.  I'll start digging into that, but if you have any thoughts,  I'd sure appreciate it...

I've pulled in NuttX10.3.0's imxrt_usbdev.c code, and so far (two days of testing) it has not hard-faulted.  I -thought- I had done this a few weeks back, but apparently I didn't so I'll close this issue.


Update to godot 4

At the moment this addon doesn't work in godot 4.
Do you know which changes needs to be done ?

Same here, it would be nice to update it to 4.2.
I checked and got some errors on variables declaration and variable type creation (PDStream) in AudioStreamPD.gd.
Sadly I'm not competent enough to update it myself.

I could look into an update; the project isn't really in my focal area right now, but I'll check out the scope of updating it and give it a shot if it's not too much effort. Not sure on a timeframe for that.On 28 Apr 2024, at 8:29 PM, Nomys @.***> wrote:
Same here, it would be nice to update it to 4.2.
I checked and got some errors on variables declaration and variable type creation (PDStream) in AudioStreamPD.gd.
Sadly I'm not competent enough to update it myself.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

Hi @magogware , thanks so much for the effort on this repo. I'm also trying to see if I can get it working on Godot 4.3. As someone mentioned above, the problem I'm facing right now if with the PdStream type in AudioStreamPD.gd script. Looks like this PdStream object is saved as a .tres resource on the root of the addon, but it shows as a MissingResource (not sure if problematic). But it does look like it fails to create this special type that your script references.
The export syntax from Godot 3 is easily updated as seen below, but I can't figure out the PdStream issue.
@export_file("*.pd") var patch : String
@export var interval : float

Thank you!

Hey @ribponce,
Have you tried grabbing my godot-pdstream repo, running the makefile, and using the new libraries it generates? I'm not sure what it'll take to update this and the backend native code, but that library would almost certainly need updating.

Okay, I've looked into what steps will need to happen to get this updated.

Turn godot-pdstream into a GDExtension module, probably using the godot-cpp-template repo
Update the tool script syntax in this repo
Update the resource files in this repo to use the GDExtension module

I'm not really sure I have the interest in prioritising time to do step 1 at the moment, but I'll be happy to assist anyone that takes up the task.

I almost got it working with a different repo that already has the GDextension module approach (I found it after yours), as it was done more recently for 4.2: https://github.com/kumikohime/gdpuredata
But I ran into an issue with building with scons, which I posted there as well. I the repo takes care of step 1 you listed? Perhaps if you have interest you can try their repo too.


Show metrics results in notebooks

This is a small change to the demo notebooks (mean climate, monsoon_wang, and mov). Instead of showing all the json information, only show the RESULTS field from the JSON. This will shorten the notebook and help users see the most relevant metrics data.
In the mean climate notebook, I added a "regions" field to the parameter file so that only Global is run in the default example. This also reduces the notebook length by not running multiple default regions.

@gleckler1 @lee1043 Not an urgent fix, but I found a couple of ways to make the notebooks a little shorter and tidier. Let me know what you think.

@acordonez  Looks like this will present things much cleaner - great!   A few min ago merged a PR which means this and another required update and thus CI is rerunning.   I'll keep an eye on this.


LICENSE

Hi, I see you mention MIT in the metadata, but there's no actual license text with attribution to copy. Can you add a LICENSE file to this repo? Thanks!

Done, may I ask why you are interested?


Broken link : example machine learning workflows

Broken link : example machine learning workflows on this page https://pachyderm.readthedocs.io/en/latest/cookbook/ml.html#examples on clicking some example pipelines it opens up http://docs.pachyderm.io/en/latest/examples/readme.html#machine-learning which says " SORRY This page does not exist yet."

Thanks for pointing this out! We'll definitely work on getting that fixed @Nick-Harvey


Not working at Angular 4

I got this error in angular 4
ERROR in Error: Metadata version mismatch for module ..... found version 4, expected 3, resolving symbol AppModule in.....
Can you please help me in angular 4. I can't upgrade angular version because It's an old project.

Sorry, this project only supports angular 5+


Do not set the privacy of the email attr from Idp.

Coverage remained the same at 89.501% when pulling 2e030faa2f2317bfac57eb099d23860751292874 on akatsoulas:primary-email-privacy into 504c4cff7643661feabe18e552d411d23fa29823 on mozilla:master.