[i2c] Bump version to 2.0.0, set to D1/V1

Since i2c was versioned at 1.1.0 (in 0a8d1b27eba), we made multiple changes that change the HW and SW interfaces of this HW IP block in a backward-incompatible way. git diff 0a8d1b27eba..<this commit>^ -- hw/ip/i2c/data/i2c.hjson gives a good overview:
diff --git a/hw/ip/i2c/data/i2c.hjson b/hw/ip/i2c/data/i2c.hjson
index 9e62d5095f..f316e58936 100644
--- a/hw/ip/i2c/data/i2c.hjson
+++ b/hw/ip/i2c/data/i2c.hjson
@@ -40,13 +40,16 @@
   // INTERRUPT pins
   interrupt_list: [
     { name: "fmt_threshold"
-      desc: "host mode interrupt: raised when the FMT FIFO depth is less than the low threshold."
+      desc: "host mode interrupt: asserted whilst the FMT FIFO level is below the low threshold. This is a level status interrupt."
+      type: "status"
     }
     { name: "rx_threshold"
-      desc: "host mode interrupt: raised if the RX FIFO is greater than the high threshold."
+      desc: "host mode interrupt: asserted whilst the RX FIFO level is above the high threshold. This is a level status interrupt."
+      type: "status"
     }
-    { name: "fmt_overflow"
-      desc: "host mode interrupt: raised if the FMT FIFO has overflowed."
+    { name: "acq_threshold"
+      desc: "target mode interrupt: asserted whilst the ACQ FIFO level is above the high threshold. This is a level status interrupt."
+      type: "status"
     }
     { name: "rx_overflow"
       desc: "host mode interrupt: raised if the RX FIFO has overflowed."
@@ -74,14 +77,15 @@
       '''
     }
     { name: "tx_stretch"
-      desc: "target mode interrupt: raised if the target is stretching clocks for a read command.  This is a level status interrupt."
+      desc: "target mode interrupt: raised if the target is stretching clocks for a read command. This is a level status interrupt."
       type: "status"
     }
-    { name: "tx_overflow"
-      desc: "target mode interrupt: raised if TX FIFO has overflowed."
+    { name: "tx_threshold"
+      desc: "target mode interrupt: asserted whilst the TX FIFO level is below the low threshold. This is a level status interrupt."
+      type: "status"
     }
     { name: "acq_full"
-      desc: "target mode interrupt: raised if ACQ FIFO becomes full.  This is a level status interrupt."
+      desc: "target mode interrupt: raised if ACQ FIFO becomes full. This is a level status interrupt."
       type: "status"
     }
     { name: "unexp_stop"
@@ -100,10 +104,21 @@
   ],
   param_list: [
     { name: "FifoDepth",
-      desc: "Depth of FMT, RX, TX, and ACQ FIFOs",
+      desc: '''
+            Depth of FMT, RX, and TX FIFOs.
+            The maximum supported value is 2^12-1, although much lower values are recommended to keep area requirements reasonable.
+            ''',
       type: "int",
       default: "64",
     }
+    { name: "AcqFifoDepth",
+      desc: '''
+            Depth of ACQ FIFO.
+            The maximum supported value is 2^12-1, although much lower values are recommended to keep area requirements reasonable.
+            ''',
+      type: int
+      default: "268",
+    }
   ],
   features: [
     { name: "I2C.MODE.HOST",
@@ -209,27 +224,27 @@
       ]
     }
     { name:     "STATUS"
-      desc:     "I2C Live Status Register"
+      desc:     "I2C Live Status Register for Host and Target modes"
       swaccess: "ro"
       hwaccess: "hwo"
       hwext:    "true"
       fields: [
         { bits: "0"
           name: "FMTFULL"
-          desc: "FMT FIFO is full"
+          desc: "Host mode FMT FIFO is full"
         }
         { bits: "1"
           name: "RXFULL"
-          desc: "RX FIFO is full"
+          desc: "Host mode RX FIFO is full"
         }
         { bits: "2"
           name: "FMTEMPTY"
-          desc: "FMT FIFO is empty"
+          desc: "Host mode FMT FIFO is empty"
           resval: "1"
         }
         { bits: "5"
           name: "RXEMPTY"
-          desc: "RX FIFO is empty"
+          desc: "Host mode RX FIFO is empty"
           resval: "1"
         }
         { bits: "3"
@@ -244,20 +259,20 @@
         }
         { bits: "6"
           name: "TXFULL"
-          desc: "TX FIFO is full"
+          desc: "Target mode TX FIFO is full"
         }
         { bits: "7"
           name: "ACQFULL"
-          desc: "ACQ FIFO is full"
+          desc: "Target mode receive FIFO is full"
         }
         { bits: "8"
           name: "TXEMPTY"
-          desc: "TX FIFO is empty"
+          desc: "Target mode TX FIFO is empty"
           resval: "1"
         }
         { bits: "9"
           name: "ACQEMPTY"
-          desc: "ACQ FIFO is empty"
+          desc: "Target mode receive FIFO is empty"
           resval: "1"
         }
       ]
@@ -277,7 +292,7 @@
              "excl:CsrAllTests:CsrExclCheck"]
     }
     { name: "FDATA"
-      desc: "I2C Format Data"
+      desc: "I2C Host Format Data"
       swaccess: "wo"
       hwaccess: "hro"
       hwqe: "true"
@@ -324,58 +339,6 @@
           name: "FMTRST"
           desc: "FMT fifo reset. Write 1 to the register resets FMT_FIFO. Read returns 0"
         }
-        { bits: "4:2"
-          name: "RXILVL"
-          desc: '''Trigger level for RX interrupts. If the FIFO depth exceeds
-                this setting, it raises rx_threshold interrupt.
-                '''
-          enum: [
-            { value: "0",
-              name: "rxlvl1",
-              desc: "1 character"
-            },
-            { value: "1",
-              name: "rxlvl4",
-              desc: "4 characters"
-            },
-            { value: "2",
-              name: "rxlvl8",
-              desc: "8 characters"
-            },
-            { value: "3",
-              name: "rxlvl16",
-              desc: "16 characters"
-            },
-            { value: "4",
-              name: "rxlvl30",
-              desc: "30 characters"
-            },
-          ]
-        }
-        { bits: "6:5"
-          name: "FMTILVL"
-          desc: '''Trigger level for FMT interrupts. If the FIFO depth falls below
-                this setting, it raises fmt_threshold interrupt.
-                '''
-          enum: [
-            { value: "0",
-              name: "fmtlvl1"
-              desc: "1 character"
-            },
-            { value: "1",
-              name: "fmtlvl4",
-              desc: "4 characters"
-            },
-            { value: "2",
-              name: "fmtlvl8",
-              desc: "8 characters"
-            },
-            { value: "3",
-              name: "fmtlvl16",
-              desc: "16 characters"
-            }
-          ]
-        }
         { bits: "7"
           swaccess: "wo"
           name: "ACQRST"
@@ -388,25 +351,81 @@
         }
       ]
     }
-    { name: "FIFO_STATUS"
-      desc: "I2C FIFO status register"
+    {
+      name: "HOST_FIFO_CONFIG"
+      desc: "Host mode FIFO configuration"
+      swaccess: "rw"
+      hwaccess: "hro"
+      hwqe: "true"
+      fields: [
+        { bits: "11:0"
+          name: "RX_THRESH"
+          desc: '''Threshold level for RX interrupts. Whilst the level of data in the RX FIFO
+                is above this setting, the rx_threshold interrupt will be asserted.
+                '''
+          resval: "0"
+        }
+        { bits: "27:16"
+          name: "FMT_THRESH"
+          desc: '''Threshold level for FMT interrupts. Whilst the number of used entries in the
+                FMT FIFO is below this setting, the fmt_threshold interrupt will be asserted.
+                '''
+          resval: "0"
+        }
+      ]
+    }
+    {
+      name: "TARGET_FIFO_CONFIG"
+      desc: "Target mode FIFO configuration"
+      swaccess: "rw"
+      hwaccess: "hro"
+      hwqe: "true"
+      fields: [
+        { bits: "11:0"
+          name: "TX_THRESH"
+          desc: '''Threshold level for TX interrupts. Whilst the number of used entries in the
+                TX FIFO is below this setting, the tx_threshold interrupt will be asserted.
+                '''
+          resval: "0"
+        }
+        { bits: "27:16"
+          name: "ACQ_THRESH"
+          desc: '''Threshold level for ACQ interrupts. Whilst the level of data in the ACQ FIFO
+                is above this setting, the acq_threshold interrupt will be asserted.
+                '''
+          resval: "0"
+        }
+      ]
+    }
+    { name: "HOST_FIFO_STATUS"
+      desc: "Host mode FIFO status register"
       swaccess: "ro"
       hwaccess: "hwo"
       hwext: "true"
       fields: [
-        { bits: "6:0"
+        { bits: "11:0"
           name: "FMTLVL"
           desc: "Current fill level of FMT fifo"
         }
-        { bits: "22:16"
+        { bits: "27:16"
           name: "RXLVL"
           desc: "Current fill level of RX fifo"
         }
-        { bits: "14:8"
+      ]
+      tags: [// Updated by the hw. Exclude from write-checks.
+             "excl:CsrNonInitTests:CsrExclWriteCheck"]
+    }
+    { name: "TARGET_FIFO_STATUS"
+      desc: "Target mode FIFO status register"
+      swaccess: "ro"
+      hwaccess: "hwo"
+      hwext: "true"
+      fields: [
+        { bits: "11:0"
           name: "TXLVL"
           desc: "Current fill level of TX fifo"
         }
-        { bits: "30:24"
+        { bits: "27:16"
           name: "ACQLVL"
           desc: "Current fill level of ACQ fifo"
         }
@@ -551,7 +570,10 @@
       ]
     }
     { name: "TIMEOUT_CTRL"
-      desc: "I2C clock stretching timeout control"
+      desc: '''
+            I2C clock stretching timeout control.
+            This is used in I2C host mode to detect whether a connected target is stretching for too long.
+            '''
       swaccess: "rw"
       hwaccess: "hro"
       fields: [
@@ -599,7 +621,7 @@
           name: "ABYTE"
           desc: "Address for accepted transaction or acquired byte"
         }
-        { bits: "9:8"
+        { bits: "10:8"
           name: "SIGNAL"
           desc: "Host issued a START before transmitting ABYTE, a STOP or a RESTART after the preceeding ABYTE"
           enum: [
@@ -619,6 +641,14 @@
               name: "RESTART",
               desc: "ABYTE contains junk, START with address will follow"
             },
+            { value: "4",
+              name: "NACK",
+              desc: "ABYTE contains either the address or data that was NACK'ed"
+            },
+            { value: "5",
+              name: "NACKSTART",
+              desc: "ABYTE contains the I2C address which was ACK'ed, but the block will continue and NACK the next data byte that was received: this only happens for writes"
+            },
           ]
         }
       ]
@@ -643,5 +673,35 @@
         { bits: "31:0" }
       ]
     }
+    { name: "TARGET_TIMEOUT_CTRL"
+      desc: '''
+            I2C target internal stretching timeout control.
+            When the target has stretched beyond this time it will send a NACK.
+            '''
+      swaccess: "rw"
+      hwaccess: "hro"
+      fields: [
+        { bits: "30:0"
+          name: "VAL"
+          desc: "Clock stretching timeout value (in units of input clock frequency)"
+        }
+        { bits: "31"
+          name: "EN"
+          desc: "Enable timeout feature and send NACK once the timeout has been reached"
+        }
+      ]
+    }
+    { name: "TARGET_NACK_COUNT"
+      desc: '''
+            Number of times the I2C target has NACK'ed a new transaction since the last read of this register.
+            Reading this register clears it.
+            This is useful because when the ACQ FIFO is full the software know that a NACK has occurred, but without this register would not know how many transactions it missed.
+            '''
+      swaccess: "rc"
+      hwaccess: "hrw"
+      fields: [
+        { bits: "31:0" }
+      ]
+    }
   ]
 }

Thus we need to give I2C a major version bump and reset its D and V stages.
Issues #20971 and #22108 track signing i2c off at D2(S) and V2(S) again, respectively.

Should we wait for the other design changes to go in? There are at least 4 PRs that would still need to be put in.

Not necessary for D1. For D2, we should wait for those design changes and the D2 review, though.

Also, have we done a formal V1 sign-off? Maybe we can change the verification stage to V0?

Good point; I changed to V0.


No Mobile Controls!

The game runs flawlessly on an iOS browser, but there’s no way to send any controller input to the game. Perhaps there should be an optional control pad overlay, or at the very least have a way for the keypad to appear when playing.

This is on the to do list.  It would happen quicker if someone sends a PR though


Sorted The Repo Alphabetically and Removing Duplicate Project

Description
Fixes:  #717

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Don't Merge, I have to push some more commits


refactor(simapp): use app v2 for core sdk modules

Description
Spike, feel free to merge if the IBC team find that useful.
This PR makes the Core SDK Modules use app wiring.  Given that IBC modules do not support app wiring yet, this is then an example of a mixed configuration.
The advantage for IBC to have such mixed app.go, is still to benefit of app v2 from the Cosmos SDK modules, and hence a smaller app.go; as well as not having any breaking change in app wiring for Core SDK module in the future.
As well as demonstrating other app developers how to use a mixed app v2 (we have docs https://docs.cosmos.network/main/building-apps/app-go-v2#registering-non-app-wiring-modules, but this can still be a good reference) until you migrate.
Commit Message / Changelog Entry
refactor(simapp): use app v2 for core sdk modules

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Looks great @julienrbrt, if I have some extra time I might push some commits to this if that's cool?
Would be great to see if we could get e2e's running with this, but now that I think of it, I think we may still be waiting on interchaintest v0.50 compat.

This is very useful; thank you very much, @julienrbrt! I had a quick look and was thinking if there was a reason why you replaced the legacy app.go with the v2 one, instead of keeping the legacy and adding an app_v2.go, as you are doing in the SDK. What are you plans for the legacy app.go in the SDK?

This is very useful; thank you very much, @julienrbrt! I had a quick look and was thinking if there was a reason why you replaced the legacy app.go with the v2 one, instead of keeping the legacy and adding an app_v2.go, as you are doing in the SDK. What are you plans for the legacy app.go in the SDK?

The idea is that given that we verify the manual wiring of the SDK modules directly in the SDK, you do not need to do it here too. I guess once you fully use depinject for your app v2 it will make sense to do the same as the SDK for the IBC modules.
So this is sort of your legacy app.go, which as well demonstrate a mixture of dependency injection and manual wiring.

Love this.  Thank you @julienrbrt !

Looks like the pr got automatically closed after meging the feature branch. I'm not able to reopen or rebase. You can reopen against main

Sure, will do.


Add an example of using the consumer

Somehow we never added this.
@wvanbergen @i19
Fixes #556.

:+1:, this is a good start. (I could have sworn we already had something like this, but apparently not).
Might be nice to also have a more complete example that consumes all partitions of a topic. Maybe that is better suited for the examples folder though.

Thx a lot.
I found that sarama doesn't have connection to ZooKeeper. So I wonder how you guys perform the rebalance and how to scale the consumer?

Zookeeper logic is in a separate package: https://github.com/wvanbergen/kafka

Cool ! Thx again !


Cursor jumps back to original location when closing quickfix window after opening any item

Neovim version (nvim -v | head -n1)
NVIM v0.6.1
Operating system/version
macOS 11.6.2 (20G314)
How to reproduce the issue

Open any new file with any context
Find a word with grep :grep window .
Open a quickfix window with :copen
Navigate to any new item (not the first item)
Press enter to go to new location where that item is
Use :cclose to close the quickfix window

Expected behavior
The cursor still stays at its current location after closing the quickfix window
Actual behavior
The cursor still jumps back to the previous location

Thanks, fix now.


[Editor] Don't add the keyboard listener on the color-picker each time the dropdown menu is shown

and display/hide the dropdown menu when pressing arrow keys.

/botio integrationtest


Youtube directlink does not work

Hello,
I have tried different sub options but never got it to run properly.
Youtube-dl -g (URL)
I am getting links like:
https://rr3---sn-5hne6ns6.googlevideo.com/videoplayback?expire=1668118744&ei=eCRtY7yAE6WJ6dsPidS2-AI&ip=<IP_ADDRESS>&id=o-APoqV2lngEXIaeTL4OdftNnBE2QFaVmFluCFPhia2mbF&ita
g=251&source=youtube&requiressl=yes&mh=Bh&mm=31%2C29&mn=sn-5hne6ns6%2Csn-5hneknee&ms=au%2Crdu&mv=m&mvi=3&pl=22&initcwndbps=793750&spc=SFxXNngyHKQ2iZLdBxnnussEtnJ-AZo&vprv=
1&svpuc=1&mime=audio%2Fwebm&gir=yes&clen=21803127&dur=1292.481&lmt=1650693138260344&mt=1668096796&fvip=5&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=4532434&spara
ms=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhALE8E3rHoaMgfgSV0yuI0k_2Sg2Fvbn1P2a7QzoNgOtcAiBI
a5XRN8FgOb6-6RbsQ-i0AaWcZl1gcgiAcHLKMathGw%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgQBVdK05gZHqWmsSauoP05UO1gJu-Qqqznvb30ofbL7gCIHhb
n5a7Q_x8Io1ds2h_7cxVIow22MNmvE0TnQQdcBTT
Problem: I cannot copy this link to my browser or other downloaders. It just reports a blank page.
Question: How can I use youtube-dl to get a direct link to any youtube video?
Thanks!

direct link to any youtube video

Hello, I have tried different sub options but never got it to run properly. Youtube-dl -g (URL) I am getting links like: https://rr3---sn-5hne6ns6.googlevideo.com/videoplayback?expire=1668118744&ei=eCRtY7yAE6WJ6dsPidS2-AI&ip=<IP_ADDRESS>&id=o-APoqV2lngEXIaeTL4OdftNnBE2QFaVmFluCFPhia2mbF&ita g=251&source=youtube&requiressl=yes&mh=Bh&mm=31%2C29&mn=sn-5hne6ns6%2Csn-5hneknee&ms=au%2Crdu&mv=m&mvi=3&pl=22&initcwndbps=793750&spc=SFxXNngyHKQ2iZLdBxnnussEtnJ-AZo&vprv= 1&svpuc=1&mime=audio%2Fwebm&gir=yes&clen=21803127&dur=1292.481&lmt=1650693138260344&mt=1668096796&fvip=5&keepalive=yes&fexp=24001373%2C24007246&c=ANDROID&txp=4532434&spara ms=expire%2Cei%2Cip%2Cid%2Citag%2Csource%2Crequiressl%2Cspc%2Cvprv%2Csvpuc%2Cmime%2Cgir%2Cclen%2Cdur%2Clmt&sig=AOq0QJ8wRQIhALE8E3rHoaMgfgSV0yuI0k_2Sg2Fvbn1P2a7QzoNgOtcAiBI a5XRN8FgOb6-6RbsQ-i0AaWcZl1gcgiAcHLKMathGw%3D%3D&lsparams=mh%2Cmm%2Cmn%2Cms%2Cmv%2Cmvi%2Cpl%2Cinitcwndbps&lsig=AG3C_xAwRAIgQBVdK05gZHqWmsSauoP05UO1gJu-Qqqznvb30ofbL7gCIHhb n5a7Q_x8Io1ds2h_7cxVIow22MNmvE0TnQQdcBTT
Problem: I cannot copy this link to my browser or other downloaders. It just reports a blank page.
Question: How can I use youtube-dl to get a direct link to any youtube video? Thanks!

try with youtube-dl --get-url "https://www.youtube.com/watch?v=ZuRLOlB4N8U"

That looks like what OP did. But the links expire. If you select a format with separate video and audio streams, you get two URLs.
When passing the URL(s) to other command-line applications, ensure the URL is quoted to avoid the embedded & being interpreted by the shell (Windows cmd: "URL"; POSIX: 'URL').

Closing for lack of response. Re-open with new info.


Vectored IO for Windows HANDLE's

We can use WriteFileGather to write lazy bytestring into HANDLEs - for both files and named pipes.

Addressed in PR #1552

#1552 merged to master


Invitation token is invalid when entering wrong length password

There might be a possible bug with the invitable gem.
Way to reproduce:

Click on the invitelink in the mail and fill out just the first password field with a long enough password.
A validation error is show
Enter a correct validated password
You now get the error message: "Invitation token is invalid".

I am using devise 3.4.1, devise_invitable 1.3.5

That case is tested here and it works:
https://github.com/scambra/devise_invitable/blob/master/test/models/invitable_test.rb#L194
Did you override some method in model, or controller or view?

I had two overloads in the user.rb model, but I have removed them for testing purpose and the token in the url is still removed when typing the wrong password.

You could try to update devise_invitable, although I think nothing related has changed since 1.3.5
I have checked accept_invitation! method and find_by_invitation_token and original token is restored after find, so it should work:
https://github.com/scambra/devise_invitable/blob/v1.3.5/lib/devise_invitable/model.rb#L285
I would need a test case to be able to fix it

@phatzor sounds like your new-user registration form is missing a hidden parameter; like <%= form.hidden_field :invitation_token, value: params[:invitation_token] %>

+1 .. I am facing the sane issue.

@aruprakshit did you change something on controller? I don't see failing in tests. Also, I don't think you need this value: f.object.invitation_token, although it shouldn't change anything

@scambra  Yes, I don't need it. I added it to debug. No, I didn't override or modify edit or update action. Also you can see in the logs, which I had added, the token is missing. But, if you add bring down hidden_field inside the https://gist.github.com/aruprakshit/fdd03c6a74e255393b35#file-edit-html-erb-L10 , everything just works like a charm. This is the workaround I have now to push my feature up. But I would still like to hear back why the error was coming in my added examples.

I don't know why rails doesn't display the hidden field, as you see, devise invitable is setting field on f.object

I had the same problem. When the validation on accepting an invitation fails, then the edit form is rendered rather than redirecting to the edit action and including the url param.
However, I realise that, while editing the form to be a .slim template, I had lost the hidden invitation_token field so it was silently creating a new User rather than loading by the token. All working fine now, but maybe something to put into Gotchas or FAQ?

We are having the same issue but i'm pretty sure we are using the token.  The user in question has already accepted the invitation and the model reflects that.


Error: Head "http://<IP_ADDRESS>:11434/": EOF (Windows 10)

Thank you for the OLLAMA.
So far, I've been using ollama in WSL2, and when the windows version came out, I experienced it right away. But..
Microsoft Windows [Version 10.0.19045.4046]
C:\Users\Name>ollama pull nous-hermes:13b-llama2-q6_K
Error: Head "http://<IP_ADDRESS>:11434/": EOF
C:\Users\Name>ollama list
Error: Head "http://<IP_ADDRESS>:11434/": EOF
ollama help provides a normal help
from app.log:
...
time=2024-02-17T13:20:54.375+03:00 level=WARN source=server.go:109 msg="server crash 16 - exit code 1 - respawning"
time=2024-02-17T13:20:54.875+03:00 level=ERROR source=server.go:112 msg="failed to restart server exec: already started"
time=2024-02-17T13:21:10.884+03:00 level=WARN source=server.go:109 msg="server crash 17 - exit code 1 - respawning"
time=2024-02-17T13:21:11.385+03:00 level=ERROR source=server.go:112 msg="failed to restart server exec: already started"
from server.log:
...
Error: listen tcp <IP_ADDRESS>:11434: bind: An attempt was made to access a socket in a way forbidden by its access permissions.
Error: listen tcp <IP_ADDRESS>:11434: bind: An attempt was made to access a socket in a way forbidden by its access permissions.
Error: listen tcp <IP_ADDRESS>:11434: bind: An attempt was made to access a socket in a way forbidden by its access permissions.

I had this problem and I discovered that it was because I had added a portproxy to be able to access the ollama API from Windows that was running on WSL2.
To see if this is it, run this command in cmd:
netsh interface portproxy show all
If so, you need to remove it with this command:
netsh interface portproxy delete v4tov4 listenport=11434 listenaddress=<IP_ADDRESS>
or
netsh interface portproxy delete v4tov4 listenport=11434 listenaddress=<IP_ADDRESS>

I had this problem and I discovered that it was because I had added a portproxy to be able to access the ollama API from Windows that was running on WSL2.
To see if this is it, run this command in cmd:
netsh interface portproxy show all
If so, you need to remove it with this command:
netsh interface portproxy delete v4tov4 listenport=11434 listenaddress=<IP_ADDRESS>
or
netsh interface portproxy delete v4tov4 listenport=11434 listenaddress=<IP_ADDRESS>

Thanks, it solved my problem.

Same error, no proxy

Hello I solved the same issue see here


[BUG] Error in pipeline when running update

Describe the bug
When i want to push an update to a new tenant, I get the following error on update_enrollmentConfigurations.py. "Tenant is not Global Admin or Intune Service Admin. Patch operation is restricted"
If I run the backup with --exclude EnrollmentConfigurations the update then runs fine.
I've even removed every custom setting under enrolment in the tenant where the backup comes from, but still no luck.
In previous test setups, I've never encountered this issue before.
Any insights are welcome.
To Reproduce
Not sure what causes the issue, like I've written above, I've removed every custom setting in the enrollment page.
Expected behavior
apply configuration with is does for to some extend see Pic2
The naming convention I've set gets applyed before the error.
Screenshots
Pic 1:

Pic 2:

Pic 3: App rights

Pic 4: Update pipeline

Pic 5: Run when --exclude EnrollmentConfigurations is applyed to backup

Pic 6: Run when no excluded is applyed to backup

Run type (please complete the following information):

Mode: 0
Client Pipeline
Version latest

Additional context
Add any other context about the problem here.

How is the app registration setup, is it a multi-tenant app setup in the upstream tenant and added to the downstream?

Somewhere in the chain you are running in to permission issues

Also, given the log outputs, you are not running the latest version of IntuneCD (2.3.0), then the output would be in a different format

Hi
The tenant where the app is created is not a multi-tenant.
The app is created through PowerShell, as I have been doing for all the other apps I've created.
With other tests I've done with the same approach for the app, there were no problems.
I've recreated the app on both tenants where the problem occurs. But the problem is still there.
I've now created a new demo tenant, ran the script, and now it just works.
After some more searching, I found where the problem occurs, in the settings for Windows Hello for Business on the enrollement page .
The first time, this had custom settings for the policy.
Later I've tried to just disable the settings by putting it back on Not Configured. But I left in the custom settings in the policy.

After putting everything back to default and running a new backup, the update then ran correctly.
but this was after the test with the new tenant, so not sure why it worked with those two tenants and not with these two.
Also is there a parameter to give to make sure that the pipeline uses version 2.3.0?
Thanks in advance

I will see if there is any way that I can replicate the behaviour you have described to understand what is happening.
Regarding the use of the latest version, all that is required is to run the pip install IntuneCD command without any version set like IntuneCD==2.2.3.

I've done some more testing, and the error appears when I enable Windows Hello under Windows enrollment.

Just enabeling it is enough to fail the update. Even when keeping the default values, it will still fail.
As for the version of IntunceCD when i run the pipeline the command is pip install IntuneCD but it wil install version 2.2.0

I managed to replicate and find out what is going on.
Windows Hello For Business settings require delegated permissions when updating the values, i.e. you must use interactive authentication as application permissions won't work. I will have to add a check for this payload and output that updating Windows Hello For Business is only possible when running with interactive auth.
Regarding the IntuneCD version, can you try this command and see if it successfully installs the newest version? pip3 install IntuneCD==2.3.0

Ok, no problem, good to know. Then this will be a manual setup for Windows Hello for us.
As for running the command, it produces the following error:
2024-04-05T07:50:59.4359335Z ##[section]Starting: Install IntuneCD
2024-04-05T07:50:59.4364256Z ==============================================================================
2024-04-05T07:50:59.4364392Z Task         : Command line
2024-04-05T07:50:59.4364461Z Description  : Run a command line script using Bash on Linux and macOS and cmd.exe on Windows
2024-04-05T07:50:59.4364577Z Version      : 2.237.1
2024-04-05T07:50:59.4364640Z Author       : Microsoft Corporation
2024-04-05T07:50:59.4364729Z Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/command-line
2024-04-05T07:50:59.4364856Z ==============================================================================
2024-04-05T07:50:59.8115582Z Generating script.
2024-04-05T07:50:59.8125104Z Script contents:
2024-04-05T07:50:59.8125280Z pip3 install IntuneCD==2.3.0
2024-04-05T07:50:59.8125472Z ========================== Starting Command Output ===========================
2024-04-05T07:50:59.8143741Z [command]/usr/bin/bash --noprofile --norc /home/vsts/work/_temp/92365335-c749-4866-a60e-e1526a4ce5ed.sh
2024-04-05T07:51:02.4027211Z ERROR: Could not find a version that satisfies the requirement IntuneCD==2.3.0 (from versions: 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.0.5, 1.0.6, 1.0.7, 1.0.8, 1.0.9, 1.1.0, 1.1.1, 1.1.2, 1.1.3, 1.1.4, 1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.2.4, 1.2.5, 1.2.6, 1.2.7, 1.2.8, 1.2.9, 1.3.0, 1.3.1, 1.3.2, 1.3.3, 1.3.4, 1.3.5, 1.4.0b1, 1.4.0b2, 1.4.0, 1.4.1b1, 1.4.1, 1.4.2, 1.4.3, 1.4.4, 1.4.5b1, 1.4.5b2, 1.4.5, 1.4.6b1, 1.4.6, 1.4.7b1, 1.4.7, 1.4.8b1, 1.4.8, 1.4.9, 1.5.0, 1.5.1, 2.0.0b1, 2.0.0b2, 2.0.0b3, 2.0.0b4, 2.0.0b5, 2.0.0, 2.0.1, 2.0.2b1, 2.0.2, 2.0.3b1, 2.0.3b2, 2.0.3, 2.0.4b1, 2.0.4b2, 2.0.4b3, 2.0.4, 2.0.5b1, 2.0.5b2, 2.0.5, 2.0.6, 2.0.7, 2.0.8b1, 2.0.8, 2.0.9b1, 2.0.9b2, 2.0.9b3, 2.0.9b4, 2.1.0b1, 2.1.0, 2.1.1, 2.1.2b1, 2.1.2b2, 2.1.2b3, 2.1.2, 2.2.0b1, 2.2.0b2, 2.2.0b3, 2.2.0b4, 2.2.0b5, 2.2.0b6, 2.2.0b7, 2.2.0, 2.3.0b1, 2.3.0b2, 2.3.0b3, 2.3.0b4, 2.3.0b5, 2.3.0b6, 2.3.0b7, 2.3.0b8, 2.3.0rc1, 2.3.0rc2)
2024-04-05T07:51:02.4029561Z ERROR: No matching distribution found for IntuneCD==2.3.0
2024-04-05T07:51:02.4637412Z
2024-04-05T07:51:02.4686814Z ##[error]Bash exited with code '1'.
2024-04-05T07:51:02.4713499Z ##[section]Finishing: Install IntuneCD

I just tried in an Azure DevOps pipeline using the ubuntu-latest image and for me it is installing 2.3.0
Try run an update of pip before installing IntuneCD: pip3 install --upgrade pip

Very interesting, latest should be ubuntu-22.04..


CRQ-8069 Change log level from debug to trace to prevent OutOfMemoryE…

CRQ-8069 Change log level from debug to trace

Seems fine


WW-4634 for support-2-3

Cherry-pick PR for support-2-3

As far I understand the issue was in 2.5 only, 2.3 works as expected.

But it is using the parameter align instead of a css class. the align parameter is deprecated in html 5.

That's why people should migrate to 2.5 ;-) I think we can ask reported if he wants the same in 2.3

OK

PR remove, it wasn't requested


Fix vulnerabilities

Some packages, such as lxml are out dated and ranked as severe vulnerable. However, upgrading to the latest version will bring in breaking changes since it requires the change of Django and Spyne.
It may require a major change to the code base in order to fix all these vulnerabilities.

The problem is that on the production server pcons1.scilifelab.se, lxml version later than 4.2.5 is not working.
The solution could be containerize the servers (topcons2, proq3, scampi2, boctopus2)
Known blockers for containerization.

How to submit jobs to the same queue (slurm or something else) from multiple containers
May need to use another production server since the port 80 and 443 on pcons1 are also used by +10 other web servers

OS info for the production server
$ lsb_release -a
LSB Version:    :core-4.1-amd64:core-4.1-noarch:cxx-4.1-amd64:cxx-4.1-noarch:desktop-4.1-amd64:desktop-4.1-noarch:languages-4.1-amd64:languages-4.1-noarch:printing-4.1-amd64:printing-4.1-noarch
Distributor ID: CentOSStream
Description:    CentOS Stream release 8
Release:        8
Codename:       n/a


Addon tries to call Dragonriding mount in Pandaria

/dump C_MountJournal.GetCollectedDragonridingMounts() returns { 1563, 1589, 1590, 1591 }
/dump C_MountJournal.GetMountInfoByID(1563) returns { "Highland Drake", 360954, ... }
/dump IsUsableSpell(360954) returns { true, false }
Mounty Why output:
durability => 50% > dragonflight area > summon dragon > all 4 usable > random pick -> Dragon - Cliffside Wylderdrake

Prior to switching to Mounty, I had my own addon that tried to do this, and I had to settle on a zone string match to determine if Dragonriding was allowed:
function dragonridingEnabled(zone)
	return zone == "Dragon Isles" and (not IsInInstance() or zone == "The Nokhud Offensive")
end

Hi,
thank you very much for the very detailed issue!!!
I'd like to reconstruct the issue.
Where exactly are you so GetBestMapForUnit returns 424/Pandaria instead of a map of higher level?
Your solution lacks the check of being in Dragonflight but inside of buildings, where IsInside () or IsOutside () don't seem to be fully reliable, too, as the web tells me.
But let's find a perfect way together, please.
#tlv

I tried to reproduce, and the map ID may be the root cause.  I had tried to use Mounty magic right after using Wormhole Generator: Pandaria.  One of the spots that can take you is the lake in Kun-Lai Summit at 43,84 which is where I was when I used Mounty.
Unfortunately the place is random.  I tried porting again and ended up in Town-Long Steppes, but even immediately after porting the MapID was 388 (which is correct).  I tried to fly to the Kun-Lai lake, but entering it from within the zone results in the expected MapID (379).
I suspect the issue may be because that particular port spot drops you off way in the air, so until you change subzones, perhaps the client only knows the continent rather than the zone.  I'll try to reproduce (toy has a 4h cooldown and there are 9 possible spots, so it might take a few days), but if I can reproduce the 424 map ID immediately after getting ported to the lake, I think this is an understandable edge case and not worth addressing in Mounty for the small subset of engineers who regularly travel to Pandaria these days.

That's what I thought. Also happened every time, when ported back to Oribos. Before touching ground, it was flyable for a split second. ^^

But now I also implemented a continent check, so if not on Dragon Isles, Mounty won't summin dragons at all. Coming soon.
Hopefully they won't allow dragon riding on other continents soon. ^^

Ahhh, maybe I swith from ADDON_LOADED to PLAYER_ENTERING_WORLD. Worth a try.

On the right track. When porting to Oribos and spamming C_Map.GetBestMapForUnit("player") this happens:
[09:52] Mounty #TLV v<IP_ADDRESS>: map id: 1550.
[09:52] Channel gewechselt: [1. Allgemein - Oribos]
[09:52] Channel gewechselt: [3. LokaleVerteidigung - Oribos]
[09:52] Mounty #TLV v<IP_ADDRESS>: map id: 1670.

This is how I fixed it: Before starting the magic process I let Mounty check first, if the player is in a true zone. This work's well, having been tested by using portals between Stormwind and Oribos.
function TLVlib:IsInTrueZone()
    local mapID = C_Map.GetBestMapForUnit("player")
    if (mapID) then
        local info = C_Map.GetMapInfo(mapID)
        if (info) then
            return info['mapType'] > 2
        end
    end
    return false
end

Thanks for your input. Helped a lot!
#tlv

I was able to reproduce the MapID issue with the lake.  Interestingly if you log out in the middle of the lake and then log back in, you'll get mapID 424 consistently until you swim about ~10y from the center point, at which point you start getting mapID 379.
Thanks for the quick response and fix to the addon.

yw!
#tlv


Omnisharp doesn't recognize when assembly referenced by .csx file is changed

Issue Description
I've got a csx  file referencing an assembly. When it's been rebuilt  in the background I want intellisense to update, and it doesn't, not showing me new static members of the public classes.
Steps to Reproduce
Have a .csx file referencing an assembly.
In that assembly add a new static member to a static class
Rebuild said assembly in external editor or from command line (I use dotnet build)
Try referencing said new static member
Expected Behavior
Seeing autocompleted list of members with the new one in it
Actual Behavior
Seeing the member list when omnisharp first saw the assembly. I have to restart omnisharp to get the new members.
 I have to restart omnisharp every time I rebuild protobuf dll


Add thread_command commands

In this PR I added a file with the allowed commands for ThreadCommands.
The work is almost done (ThreadCommands from viewer UI are still missing).
What do you think?

I like it.  However we could go one step further by also defining and enforcing the type the attribute should take for each command.


avx is true by default in 0.7.0 config.json

Yes that's intended behavior as it now checks if the CPU is going to crash on it or not. If your CPU supports AVX it will work so no need to patch it away


NIFI-8471 Parameter Contexts - show referencing process groups

Description of PR
https://issues.apache.org/jira/browse/NIFI-8471
Improve the UI for list of all the process groups that are referencing this parameter context

For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[X] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

@pvillard31 I’m sorry I didn’t make it clear in the description of my changes.
the change show in the picture below.
I display all reference processors on the SETTINGS tab, mainly I think if add them in PARAMETERS tab that will be too mass
the bug you mentioned seems cause by another commit, but easy to fix, should I fix it directly or create another JIRA to fix it?

Oh my bad :-D
Makes much more sense now! Will have another look.
For the bug I mentioned another JIRA/PR would be preferred.
Thanks!

@pvillard31 I have created another JIRA and the pull request for the bug https://github.com/apache/nifi/pull/5060
Thanks

I had another look today and it's looking good to me. Maybe someone like @mcgilman or @moranr can also have a quick look (as I'm not familiar with the UI code) just to confirm I didn't miss anything obvious.

@s9514171 thanks for this commt. I am a +1!


\marginfigure and \margintable overlapping with other elements

The fixes in #25 are great! How about \marginfigure and \margintable? Are those not defined as floats? For example, my \sidenotes and \marginnotes are appearing on top of \marginfigures and \margintables that they are defined close enough to.

In principle, setting \marginskip to a higher value, like 1cm, should work, but I found that it has no effect. I'll dig into the template and see what I can find out.

So, apparently I misunderstood what \marginskip is supposed to do. It turns out, as everybody knows, that it does not set globally the space between \marginpars, but rather it should be called whenever it is needed, at any point of the document.
Given this, I modified the \marginfigure, \margintable, and \marginlisting commands so that thy call \marginskip as the first and last things they do. This adds a space between the figure/table/list and any other floating element. The spaces above and below are controlled by two lengths and can be modified with, e.g.:
\setlength{\kaomarginskipabove}{1cm}
\setlength{\kaomarginskipbelow}{1cm}

in the main.tex. By default, both lengths are 3mm.

Nice! It looks like it works as long as I don't include the offset argument. However, when I include an offset (e.g. of 0pt), they still overlap.

Yes, this is indeed the expected behaviour, since the offset prevents the elements to float freely, and the \marginskip only works with floating elements. However, now that the automatic placement prevents the overlaps, setting an offset should be needed only occasionally, and for that occasions, it may be manually set so as to prevent the overlaps, and indeed so as to improve the design of the page. As you know I am always open to new ideas, so if you have one to further improve on this issue I'll try and implement it :)

Sounds good 👍

I agree that it would be difficult, in fact right now I wouldn't know how to do it. I have made some attempts, but still to no avail. Hopefully I'll find a way in the future... in the meantime I have set the help-wanted flag.


"set zlib back to system ignore list/revert pr #871" commit make raspberry pi cannot access to web ui IP

@Paulchen-Panther I confirm this commit make raspberry pi cannot access to web ui IP. i tested it many time and when i reverted it, the issue was solved. For linux, windows, macos, this commit works well
Originally posted by @tihoangyeudau in https://github.com/hyperion-project/hyperion.ng/pull/904#issuecomment-666352703

This is the last time I warn you, use the template. If you fool the next time, you will get a ban...

sorry I forgot!


(readme): fixed code in examples

Thanks. fixed by https://github.com/pshihn/rough-notation/pull/1


Setting TTL to session without cookie_lifetime

Hello,
I would like to add the ability to set TTL on Redis session keys, but without having to set the cookie_lifetime option of the framework.
The reason is that setting an expiration date on the session cookie could be a bad idea. For instance, a client who have an "early date" (in GMT -- many people change the time of their computer without selecting the right timezone) will not have any session because his browser will consider the cookie expired.
For that reason I prefer to leave the cookie_lifetime to 0. But I still want to set a TTL in redis (for security and storage purposes). As this StackOverflow comment points out, a "timeout should be implemented on your own", not with session.gc_maxlifetime or  session.cookie_lifetime.
So the idea is to add a parameter in the configuration which will override the cookie_lifetime parameter of the framework config. Something like:
snc_redis:
    ...
    session:
        client: session
        prefix: foo
        cookie_lifetime: 1440 # override framework.cookie_lifetime for Redis TTL

I would like your opinion before implementing this solution.

@MattKetmo do you recommend having a cookie with expiration time or a session cookie for login user? do you have any recommendation for the redis session ttl value?

Hello guys, @snc @MattKetmo   which is better when the user has a session cookie, put a ttl of one day (snc_redis.session.ttl:86400) or leave the value blank ?


Remove spectate as a dependency

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

No probs :)

No probs :)

Ah, yeah, have to release a new version of ipyigv that actually doesn't use it. Will do it later today!

Ah, yeah, have to release a new version of ipyigv that actually doesn't use it. Will do it later today!


Build static dapp

Add build:dapp to build static website for IPFS hosting

works with fleek.co with command "yarn && yarn build:dapp" on nextjs env


AsciiDoc and Manpages for odgi

This will help to document what is actually possible with odgi.
I will create one document for odgi itself, listing all subcommands, its general purpose and related tools maybe. There will be a manpage for each subcommand, too.

The nice thing is, if we stick to https://asciidoctor-docs.netlify.com/asciidoctor/1.5/manpage/convert-to-man-page/, we can always generate a manpage out of the AsciiDoc.

And I hope that we will be able to link into the manpages directly while building the conda package. For example man odgi build would produce a manpage compliant with https://liw.fi/manpages/ standards.

Fixed via #92


fixes for Firefox

SVGs don't have height in Firefox, so we have to fallback to the
parentElement's height in order to calculate scale correctly. When the
drag handler was used on a .handle element that was an SVG, the
value of scale was always zero. Now it is the correct value (usually
1).
Use the transitionend event to wait for events to finish. This prevents race conditions versus trying to extract the time from the styles. In chrome,
these animations were scheduled in such a way that the dom elements were
technically in the right order but visually inaccurate if we slowed the
transitions down. In Firefox, the smoke tests failed due to the elements
being out of order. In addition to using the transitionend event
(which is supported by all browsers we
support(https://developer.mozilla.org/en-US/docs/Web/Events/transitionend#Browser_compatibility),
when DEBUG mode is active (e.g. development or test builds, not
production builds`, we emit a custom event on the document and use a
custom waiter to prevent Ember from advancing the test suite too fast
before the transitions are done (the transforms need to finish so that
the elements are in the correct order in the dom).
Add Firefox to our .travis.yml and testem.js files so that we test
against Firefox in CI.

Released in 1.12.4

Breaks all drag & drop usages on my applications .. (tested on Safari, Chrome and Firefox) ...
All worked fine before 1.12.4 ....
Now after dragging an element, the onUpdate action is not called and the dragged element is not draggable anymore ...

@BnitoBzh very sorry to hear that. Are you able to give us any code snippets that will help us figure out what happened here? Are you OK to downgrade for the time being?

@jgwhite I have no code snippet, i am using ember-sortable in its basic way, (readme example).
I have downgraded and blocked ember-sortable to its version 1.12.3.
I have not a lot of time now to debug this issue, sorry

@BnitoBzh no problem. We’ll look into it and make a note on this thread when a fix lands.

@BnitoBzh Can you open this as a new issue? Thanks

Am having the same bug where transitionend never fires, I'll open an issue with what I know


New feature: commit confirmed

See details in the parent issue napalm-automation/napalm-base#214

@ppieprzycki would be possible to implement this on VyOS?

@mirceaulinic Hi, we can try implement this.
However it works on vyos a bit differently than eg. on junos.
It assumes that rollback method is through restart  :)
Where I can find details eg. how optional variable should be named or even better some example for implementation. ?
vagrant@vyos2# commit-confirm
commit confirm will be automatically reboot in 10 minutes unless confirmed
Proceed? [confirm][y]
vagrant@vyos2# confirm

@ppieprzycki here: https://github.com/napalm-automation/napalm-base/pull/213/files
However, the feature you mention seems something different IMHO. I'd advise to add a huge note in the "caveats" section in the documentation and, also, to lock the feature so it can't be used unless the user has unlocked it (for example, using an optional_arg).


asset size

somehow bundle size is ~1.7MB with webpack4 which seems really big...
previous bundle size was very small (a couple of kb)
images should also be resized/optimized

fixed the bundle size; source maps were included

i think we can close this for now, performance isn't an issue anyway 🤷♂


How to auto close tooltip when Route Change?

How I config auto close tooltip when the route changed?

@tagmetag Why did you close? Did you solve it? How?


Changed determination nvm in system

As noted by @deepsweet in #2236, check if ~/.nvm/nvm.sh exists is not the best way to determine is there an nvm command or not.
I replaced it with command -v.

I’ve just rebased the branch from master.

Hey @isquariel, on a second thought, maybe the best way is using whence -p, in case nvm is an alias.

@mcornella, whence -p nvm returns 1 because nvm is just a shell function may be it would be good to check whence -w nvm return 0 or whence -w nvm | grep function return 0? Which case is better?

Nevermind, in that case it's better to use command which will return 0 whatever the type of nvm is.
:+1:
/cc @robbyrussell

Just rebase.

There seem to be a couple issues with this.
Using command to detect the nvm installation location won't work either in the general case. Because nvm is a shell function defined in nvm.sh, you can't use it to locate nvm.sh, because it is not defined until after you have already located and run nvm.sh. (Also, the output will just give you the body of the function, not tell you what file it was loaded from.) Using which nvm or command nvm is begging the question.
This change also seems to require the user to set $NVM_DIR before loading OMZ. Normally, it is nvm.sh which sets $NVM_DIR based on where you load it from. And after the plugin runs, $NVM_DIR will be set regardless, so the prompt info function could just use that instead of doing the search each time. As is, it'll fall back to using ~/.nvm, regardless of where you actually loaded it from. Seems like this should be optional, to allow the user to force a particular location that isn't one of the well-known installation locations.
I think the thing to do instead is:

Have the nvm plugin locate nvm by looking in a few well-known installation locations for nvm.sh
$NVM_DIR, if it is already defined
`$HOME/.nvm
$(brew --prefix nvm), if brew is installed
(some other location(s) from other platforms' package systems?)
Have the nvm_prompt_info function in lib/nvm.zsh use only $NVM_DIR.
Or have it not use the filesystem at all, and just check which nvm to see if nvm is loaded in this session.

Because lib/nvm.zsh does not express a dependency on plugins/nvm, I think it should probably be written to be compatible with as many different nvm setups as possible, not just the ones that plugins/nvm knows how to load. Using which nvm is probably the best test: it's possible that a user could define a custom $NVM_DIR, but not actually load nvm in all of their shell sessions.

Closed in favor of #4338


A question about running goal_picker main.py

When I run roslaunch robot_controller start.launch ,I encounter the following error.So I ran the commands in the launch file one by one and found that I had problems running rosrun goal_picker main.py
rosrun goal_picker main.py
Goal picker node connected

Serving Flask app "main" (lazy loading)
Environment: production
WARNING: This is a development server. Do not use it in a production deployment.
Use a production WSGI server instead.
Debug mode: on
Goal picker node connected
shutdown request: [/goal_picker] Reason: new node registered with same name

Does it affect navigation?  How can I fix it? Thank you for your help.

Only one instance of the goal_picker should be enough. The error should not be an issue; I guess, as long as the goal_picker works (you are able to reach it on the specified port).


Linux Update && Upgrade error

root@w3w:~# apt-get upgrade
Reading package lists... Done
Building dependency tree
Reading state information... Done
Calculating upgrade... Done
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.
1 not fully installed or removed.
After this operation, 0 B of additional disk space will be used.
Do you want to continue? [Y/n] y
Setting up python3.4 (3.4.2-1+deb8u3) ...
File "/usr/lib/python3.4/http/client.py", line 1014
raise InvalidURL(f"URL can't contain control characters. {url!r} "
^
SyntaxError: invalid syntax
dpkg: error processing package python3.4 (--configure):
subprocess installed post-installation script returned error exit status 1
Errors were encountered while processing:
python3.4
E: Sub-process /usr/bin/dpkg returned an error code (1)

Please help me understand this error and if possible provide useful advice

@ingahaindongo
Just follow this thread on Reddit: https://www.reddit.com/r/debian/comments/c53ef1/problem_update_python_34_debian_8/
The problem is related to the Python 3.4 patch, not the code in this repo.
BTW, Python 3.4 had reached EOL in 2019-03-16
FYI, https://devguide.python.org/devcycle/#end-of-life-branches


An unexpected highlighted word that matches in filename in v0.6.0

```js filename="a/hello/file.js"
console.log("hello")
```

originally reported in https://github.com/shuding/nextra/issues/1088

What is filename from? The check internally uses title to filter it out so maybe that's why:
https://github.com/atomiks/rehype-pretty-code/blob/8d6f00dcb2bd5abf17549879eef2f5748b69587b/src/index.js#L170-L173

filename is our custom metadata prop that will be rendered as header for code block


thrift_router: refactor and simplify hosts filtering and ranking

Motivated by reviewing https://github.com/pinterest/rocksplicator/pull/554, where our hosts selection logic in thrift_router is bloated and causing lots of confusions, e.g. RankHostsByGroupPrefixLengthAndShrinkTo is called 3 times in a single function. https://github.com/pinterest/rocksplicator/pull/554 had to check AZ in multiple places -- error prone and make things even more un-readable.
This PR attempts to simplify the host selection logic, by separating it into two phases in sequential: (1) filtering, and (2) sorting.
There shouldn't be any perf impact because we still do two pass of the host list: one for filtering by role & AZ, and another for sorting the list.
No production logic change, and passing existing test should suffice.

Reviewed this in detail, and it looks good. What is the test plan for this? Since this is a crucial piece of the the routing framework, we would want to ensure there are no bugs.

it's a pure refactor so existing tests should cover it (locally, I intentionally injected a few bugs to make sure unit test work as expected and they did). That said, I can double check to see if we miss any test cases.
We could also test with a private build ---or we could also do that when we bump the rocksplicator hash in prod repo

Reviewed this in detail, and it looks good. What is the test plan for this? Since this is a crucial piece of the the routing framework, we would want to ensure there are no bugs.

it's a pure refactor so existing tests should cover it (locally, I intentionally injected a few bugs to make sure unit test work as expected and they did). That said, I can double check to see if we miss any test cases.
We could also test with a private build ---or we could also do that when we bump the rocksplicator hash in prod repo

Didn't realize we already have unit tests, gives more confidence :)
If possible, lets still try to try it with a private build. Or be mindful when we roll it to prod.

Reviewed this in detail, and it looks good. What is the test plan for this? Since this is a crucial piece of the the routing framework, we would want to ensure there are no bugs.

it's a pure refactor so existing tests should cover it (locally, I intentionally injected a few bugs to make sure unit test work as expected and they did). That said, I can double check to see if we miss any test cases.
We could also test with a private build ---or we could also do that when we bump the rocksplicator hash in prod repo

Didn't realize we already have unit tests, gives more confidence :) If possible, lets still try to try it with a private build. Or be mindful when we roll it to prod.

yeah,  also just deployed a test build based on the new hash (https://phabricator.pinadmin.com/D816082) into pininfoservice test stage, monitored all metrics (SR, latency, QPS to other service) and didn't observe any changes.
Will also monitor carefully when we release this in production


Zookeeper - KV connection error: Failed to test KV store connection

Hi,
Traefik (in a docker) can't discuss with my zookeeper (in a docker too).
This is my traefik.toml :
traefikLogsFile = "/tmp/traefik.log"
accessLogsFile = "/tmp/access.log"
logLevel = "DEBUG"
[web]
address = ":8081"
[docker]
endpoint = "unix:///var/run/docker.sock"
domain = "docker.localhost"
watch = true
[zookeeper]
endpoint = "<IP_ADDRESS>:2181"
watch = true
prefix = "/traefik"
And the docker run command :
docker run -d -p 8081:8081 -p 80:80 -h traefik -v ~/ContainersLocalFiles/traefik/traefik.toml:/etc/traefik/traefik.toml -v /var/run/docker.sock:/var/run/docker.sock -v /tmp/traefik.log:/tmp/traefik.log --name traefik traefik
I use and run the jplock/zookeeper as this :
docker run -d -p 2181:2181 -v ~/ContainersLocalFiles/zookeeper:/tmp/zookeeper --name zookeeper jplock/zookeeper start-foreground
and I have created sample datas as describe here
And this the logs when I start traefik :
time="2016-06-13T09:55:35Z" level=info msg="Traefik version v1.0.0-rc2 built on 2016-06-08_12:35:41PM"
time="2016-06-13T09:55:35Z" level=info msg="Using TOML configuration file /etc/traefik/traefik.toml"
time="2016-06-13T09:55:35Z" level=debug msg="Global configuration loaded {"GraceTimeOut":10,"Debug":false,"AccessLogsFile":"/tmp/access.log","TraefikLogsFile":"/tmp/traefik.log","LogLevel":"DEBUG","EntryPoints":{"http":{"Network":"","Address":":80","TLS":null,"Redirect":null}},"Constraints":[],"ACME":null,"DefaultEntryPoints":["http"],"ProvidersThrottleDuration":2000000000,"MaxIdleConnsPerHost":200,"Retry":null,"Docker":{"Watch":true,"Filename":"","Constraints":null,"Endpoint":"unix:///var/run/docker.sock","Domain":"docker.localhost","TLS":null},"File":null,"Web":{"Address":":8081","CertFile":"","KeyFile":"","ReadOnly":false},"Marathon":null,"Consul":null,"ConsulCatalog":null,"Etcd":null,"Zookeeper":{"Watch":true,"Filename":"","Constraints":[],"Endpoint":"<IP_ADDRESS>:2181","Prefix":"/traefik","TLS":null},"Boltdb":null,"Kubernetes":null}"
time="2016-06-13T09:55:35Z" level=info msg="Preparing server http &{Network: Address::80 TLS: Redirect:}"
time="2016-06-13T09:55:35Z" level=info msg="Starting provider *provider.Docker {"Watch":true,"Filename":"","Constraints":null,"Endpoint":"unix:///var/run/docker.sock","Domain":"docker.localhost","TLS":null}"
time="2016-06-13T09:55:35Z" level=info msg="Starting provider *main.WebProvider {"Address":":8081","CertFile":"","KeyFile":"","ReadOnly":false}"
time="2016-06-13T09:55:35Z" level=info msg="Starting provider *provider.Zookepper {"Watch":true,"Filename":"","Constraints":[],"Endpoint":"<IP_ADDRESS>:2181","Prefix":"/traefik","TLS":null}"
time="2016-06-13T09:55:35Z" level=info msg="Starting server on :80"
time="2016-06-13T09:55:35Z" level=error msg="KV connection error: Failed to test KV store connection: zk: could not connect to a server, retrying in 682.242338ms"
time="2016-06-13T09:55:35Z" level=debug msg="Docker connection established with docker 1.11.1 (API 1.23)"
time="2016-06-13T09:55:35Z" level=debug msg="Filtering container with more than 1 port and no traefik.port label /traefik"
time="2016-06-13T09:55:35Z" level=debug msg="Filtering container with more than 1 port and no traefik.port label /zookeeper"
time="2016-06-13T09:55:35Z" level=debug msg="Configuration received from provider docker: {"backends":{"backend-mongo":{"servers":{"server-mongo":{"url":"http://<IP_ADDRESS>:27017","weight":1}}},"backend-vertxjs":{"servers":{"server-vertxjs":{"url":"http://<IP_ADDRESS>:8082","weight":1}}}},"frontends":{"frontend-Host-mongo-docker-localhost":{"backend":"backend-mongo","routes":{"route-frontend-Host-mongo-docker-localhost":{"rule":"Host:mongo.docker.localhost"}},"passHostHeader":true,"priority":0},"frontend-Host-vertxjs-docker-localhost":{"backend":"backend-vertxjs","routes":{"route-frontend-Host-vertxjs-docker-localhost":{"rule":"Host:vertxjs.docker.localhost"}},"passHostHeader":true,"priority":0}}}"
time="2016-06-13T09:55:35Z" level=debug msg="Last docker config received more than 2s, OK"
time="2016-06-13T09:55:35Z" level=debug msg="Creating frontend frontend-Host-mongo-docker-localhost"
time="2016-06-13T09:55:35Z" level=debug msg="Wiring frontend frontend-Host-mongo-docker-localhost to entryPoint http"
time="2016-06-13T09:55:35Z" level=debug msg="Creating route route-frontend-Host-mongo-docker-localhost Host:mongo.docker.localhost"
time="2016-06-13T09:55:35Z" level=debug msg="Creating backend backend-mongo"
time="2016-06-13T09:55:35Z" level=debug msg="Creating load-balancer wrr"
time="2016-06-13T09:55:35Z" level=debug msg="Creating server server-mongo at http://<IP_ADDRESS>:27017 with weight 1"
time="2016-06-13T09:55:35Z" level=debug msg="Creating frontend frontend-Host-vertxjs-docker-localhost"
time="2016-06-13T09:55:35Z" level=debug msg="Wiring frontend frontend-Host-vertxjs-docker-localhost to entryPoint http"
time="2016-06-13T09:55:35Z" level=debug msg="Creating route route-frontend-Host-vertxjs-docker-localhost Host:vertxjs.docker.localhost"
time="2016-06-13T09:55:35Z" level=debug msg="Creating backend backend-vertxjs"
time="2016-06-13T09:55:35Z" level=debug msg="Creating load-balancer wrr"
time="2016-06-13T09:55:35Z" level=debug msg="Creating server server-vertxjs at http://<IP_ADDRESS>:8082 with weight 1"
time="2016-06-13T09:55:35Z" level=info msg="Server configuration reloaded on :80"
time="2016-06-13T09:55:36Z" level=error msg="KV connection error: Failed to test KV store connection: zk: could not connect to a server, retrying in 741.756464ms"
time="2016-06-13T09:55:37Z" level=error msg="KV connection error: Failed to test KV store connection: zk: could not connect to a server, retrying in 708.825532ms"
time="2016-06-13T09:55:37Z" level=error msg="KV connection error: Failed to test KV store connection: zk: could not connect to a server, retrying in 1.002564428s"
Etc..
Note : I can connect to ZK and managing datas from any hosts.
Thx

Anybody ? :)

Hi, it seems traefik cannot connect to zookeeper. Are you sure this is not a network issue?

Hi,
the 2 dockers are running on localhost, so I'm pretty sure it's not a network issue :)

@raclepoulpe it depends how you launched your containers.
Could you provide more details on this ?

@raclepoulpe you configured traefik to talk to Zookeeper on <IP_ADDRESS>:2181. As you launched traefik container in bridged mode, <IP_ADDRESS> will not ends on zookeeper container.
This issue comes from a bad networking configuration of your containers.
You can create a link between these two containers to fix that.

@emilevauge thx.
I plugged traefik on an another zk, and the connection is Ok.
This issue can be solved


Add freeze_tensorrt and support loading frozen graphs in python

This is faster. thought it's not clear how much faster

@sethtroisi what are we doing with this?  @chsigg mentioned he's got this in his batching stuff as well -- what's the plan here?

/retest

The TensorRT backend in the cc player loads an .uff graph, so freezing the graph in the python code is done with the tf2uff converter, not the TensorRT integration of TensorFlow.
Here is what freezing a graph to uff looks like:
def freeze_graph(model_path):
    n = DualNetwork(model_path)
    outputs = ["policy_output", "value_output"]
    out_graph = tf.graph_util.convert_variables_to_constants(
        n.sess, n.sess.graph.as_graph_def(), outputs)
    with tf.gfile.GFile(model_path + '.pb', 'wb') as f:
        f.write(out_graph.SerializeToString())
    with tf.gfile.GFile(model_path + '.uff', 'wb') as f:
        f.write(uff.from_tensorflow(out_graph, outputs))

I was planning to just add that small modification as a patch to MiniGo when importing it for MLPerf. Or not even that and just let submitters figure it out themselves if they want to use TensorRT.


Automatic ID inference issue

Describe the bug
After #1788 was fixed, I gave my dataset another try and it's failing again.
It looks like it's related to reordering the keys which was introduced for #1788.
However, when trying to create a reproduction document, I observed some strange behavior which I don't understand yet: It seems like there's a chance the below either works or not, even without (knowingly) changing anything. It "feels" like it's failing more often when using a file instead of just providing the json directly in the data param which doesn't make sense.
To Reproduce
The following test case works against 0.23.0rc2 but (often?) fails against 0.23.0rc3 and 0.23.0:
fail-document-identifier-invalid.json:
[{"id":19753,"Source of incident":["a"]}]

Steps to reproduce the behavior:

Ensure fresh index: curlie -X DELETE 'http://<IP_ADDRESS>:7700/indexes/test1'
Add test doc: curlie -X POST 'http://localhost:7700/indexes/test1/documents' --data-binary @fail-document-identifier-invalid.json  -H 'Content-Type: application/json'
curlie 'http://<IP_ADDRESS>:7700/indexes/test1/updates' (run it possibly multiple times, until the previous POST appears and continue until it finished
observe status failed with message "document identifier is invalid [\"a\"], a document id can be of type integer or string only composed of alphanumeric characters (a-z A-Z 0-9), hyphens (-) and underscores (_)"

Expected behavior
status: processed
MeiliSearch version: v0.23.0rc3, v0.23.0
Additional context
This can be worked around by providing the primaryKey explicitly.
I observed lots of strange results and non-repeatable behavior. E.g. status updates not being available directly after adding documents, errors that document doesn't have an identifier when the document stats with "id" (when providing all documents at once but not explicitly specifying id):
...
  "type"=>{"name"=>"DocumentsAddition"},
  "message"=>
    "document doesn't have an identifier {\"id\":19472,\"title\": ...
  "errorCode"=>"missing_document_id",
...

By the way, add-documents processing time significantly increased: v0.22.0: ~3s, v0.23.0: ~27s.

So the first issue shoud be adressed by https://github.com/meilisearch/milli/pull/388.
The performance issue, on the other hand doesn't come from where I expected. I though it might be coming from the document formats transformations. This is a 2 phases process:

The documents are transformed from a textual format (json, jsonl, csv) to a binary format (obkv). This is done synchronously so payload error can be returned to the user right away.
The obkv fields are remapped: the obkv maps the fields names to field ids and store them in an index. When we are about to process the documents, we need to remap the documents fields ids to the one already present in the index. For example if "id" was mapped to 19 in the addition obkv, and it is mapped to 0 in the index, then we remap it to 0 in the addition. This process is asynchronous, and is done right before the beginning of the indexing phase.

From there the indexation can take place. I have taken some measurements:

dataset
phase 1 transform (ms)
phase 2 transform (ms)
indexing (ms)

movies.json(9.1MB)
222ms
90ms
11352ms

movies.json (200MB)
8683ms
2582ms
249875ms

As we can see the phase1, which is the longest, accounts for 1-3% of the total indexing time. Overall the transforms are pretty negligible compared to indexing, so optimizing this isn't a real gain in term of indexing speed, and optimizing indexing remains the priority. @ManyTheFish Is there anything else that may explain the performance regression?
That being said, now that the serialization of the documents is done synchronously, there is be more latency between the moment the documents are pushed and the moment the update is registered. @bb  if this is what you observed, I think there is room for improvement in the phase 1, but I judge it non critical for the release @curquiza.

Fixed with #1801 and v0.23.1 release


Add Banco Atlântico Europa

@marmelo Seems like 3 links are failing in CI
https://www.weareforeteller.com/ is gone, but it's previous domain http://www.growininnovation.com/ is still up, not sure if they are gone or the website is down temporarily

Please re-check the contributing guide https://github.com/marmelo/tech-companies-in-portugal/blob/master/CONTRIBUTING.md
You missed the point regarding a short description, go for less than 70 chars, and that the company must the placed in order, this one being at the top place.

Added issue #151 for the issue you raised https://github.com/marmelo/tech-companies-in-portugal/pull/147#issuecomment-565751962

@vpereira01, I corrected the issue for this PR and will do the same regarding order in others

Thanks @shizpi and @vpereira01!


don't break lineage if user forces stale execution

This breaks the dynamic slicer. cc @shreyashankar

@shreyashankar let me take this one and instead assign you the one about live refs; I did some initial scoping and I think it makes more sense for me to take this one on.

This seems to be working after @shreyashankar's fix for #54.


[Extension Project] Generating Box Prompts with Zero-Shot Detector for Segment-Anything

Hi! Thanks for releasing such impressive work! We find an interesting extension for this great work by combining SoTA zero-shot detector with Segment-Anything which can generate high-quality box and mask annotations with text inputs! The new project is here, we simply named it Grounded-Segment-Anything: https://github.com/IDEA-Research/Grounded-Segment-Anything
We take Grounding-DINO as the zero-shot detector to generate box prompts for segment-anything, and our visualization results are as follows:

We hope to maintain this project as a sub-project of segment-anything. We're also explore to combing Grouned-SAM with diffusion models for controllable image editing as well~

Combining Grounded-SAM with Stable-Diffusion Inpainting!

excellent~

nice~

I come again ...
It's an excellent work above via grounding box. While we provide a simpler solution via CLIP's explainability.
Our work can achieve text to mask with SAM using CLIP model only, without any fine-tuning or extra supervisions to generate the boxes:.
https://github.com/xmed-lab/CLIP_Surgery
Besides, it enhances many open-vocabulary tasks, like segmentation, multi-label classification, multimodal visualization.
This is the jupyter demo:
https://github.com/xmed-lab/CLIP_Surgery/blob/master/demo.ipynb


License

I noticed the repository doesn't have a license listed. Is this on purpose or you just didn't add one yet @Garrett92 ?

No reason. I'm a total noob when it comes to git and this project was the
first I've posted to github. Feel free to make any changes, let me know if
I need to change something to make it easier.
On Dec 6, 2017 1:50 PM, "Wouter"<EMAIL_ADDRESS>wrote:

I noticed the repository doesn't have a license listed. Is this on purpose
or you just didn't add one yet @Garrett92 https://github.com/garrett92 ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Garrett92/WurmMapGen/issues/2, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABavLinhSqH9n7er0rpwMncynw2UrJRKks5s9wwWgaJpZM4Q4pLe
.

Might be best if you add the license to the repo, that way you can choose a license you like.

@Garrett92 any progress on this? I'm hesitant to make changes if you haven't added a clear license to your project.
FYI, having no license essentially means "no rights given to anyone". You can add a license by simply adding a LICENSE.txt file to your repository with the necessary license text. If you're not sure which license to use, try choosealicense.com, one of those is usually good for code on Github.

Sorry about that, the licence is there now.
On Dec 13, 2017 1:46 AM, "Wouter"<EMAIL_ADDRESS>wrote:

@Garrett92 https://github.com/garrett92 any progress on this? I'm
hesitant to make changes if you haven't added a clear license to your
project.
FYI, having no license essentially means "no rights given to anyone". You
can add a license by simply adding a LICENSE.txt file to your repository
with the necessary license text. If you're not sure which license to use,
try choosealicense.com, one of those is usually good for code on Github.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Garrett92/WurmMapGen/issues/2#issuecomment-351338419,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABavLinqsahYaw9refWEtYO1CrCW_-u2ks5s_5zrgaJpZM4Q4pLe
.

Awesome, thanks!


Install from source needed for dev functionality

In order to be able to run cudapoa, it is necessary to install the pyclaragenomics package from source. TODO: Updated documentation on this.

Updated with #530 and #533 . Closing.


What's the relationship with pion/stun

hi @ernado , I find the source code of pion/stun is so similar to gortc/stun. Can you tell me what's the relationship between them?  is it to mean that you will  give up maintaining this project and focus on pion/stun.

Hi, I've merged gortc/stun to pion/stun and will focus on pion development.

get it, thanks.


unreleased-commits - Fix icon alignment

Small cosmetic tweak
Test URLs
https://github.com/refined-github/refined-github
Screenshot

Before
After

Can you resend this?

align + icon
align tag icon only if count is missing

Because this does look misaligned:


manual body direct reader example

There should be an example where the body of a message is simply mutable_buffer and the caller can provide their own buffer for incremental reading of a message body.

Done


ClickHouseClient.load reports an error: Authentication failed

I have the following code 1:
ClickHouseNode server = ClickHouseNode.builder()
    .host("X.X.X.X")
    .port(18123)
    .database("default")
    .credentials(ClickHouseCredentials.fromUserAndPassword("default", "password")).build();
ClickHouseClient.load(server, "table_name",
    ClickHouseFormat.CSV, ClickHouseCompression.NONE, "csv_name.csv").get();

it will prompt：
java.util.concurrent.ExecutionException: com.clickhouse.client.ClickHouseException: Code: 516. DB::Exception: default: Authentication failed: password is incorrect or there is no user with such name. (AUTHENTICATION_FAILED) (version <IP_ADDRESS> (official build))
, server ClickHouseNode [uri=http://X.X.X.X:18123/default]

However, the following code 2 work fine:
ClickHouseNode server = ClickHouseNode.of("http://X.X.X.X:18123/default?user=default&password=password");
ClickHouseClient.load(server, "table_name",
    ClickHouseFormat.CSV, ClickHouseCompression.NONE, "csv_name.csv").get();

The clickhouse-jdbc version is 0.3.2-patch11, X.X.X.X is a node in the CK cluster.
I guess there are some problems with ClickHouseNode.Builder or ClickHouseClient.load.

Thank you @mywrong it does look like a bug and sorry for the inconvenience at your end. Will fix it in next release.


compering, editorializers

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 5
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[c,e]

/trunk merge


CLN: cleaning core/common.py

partial on #12503

create pandas/types
moved some dtype to pandas/types
move some missing value utils to pandas/core/missing
move .take and .diff to core/algorithms
create formats with core/formats,style moved and printing.py (form core/common.py)
removed some non-used functions

cc @wesm
still more to do, but this is a start.

@shoyer @jorisvandenbossche @sinhrks @TomAugspurger
any comments. don't want to leave this out too long as it touches lots of things.

Not objections to your description.
Not a big deal, but does formatting/ make more sense than formats/?

its slightly longer, but no big deal.

assuming you're just moving stuff around, this seems sane to me

I'll look later this AM (pacific time)

This all looks fine to me.
Aside: I'm on vacation most of this month but planning to spend some time in May / June / July on refactoring to look toward retooling DataFrame internals (it would be nice to plot a course to removing the BlockManager, deprecating Panel, and some of the other under-the-hood internals work we've been discussing). In the meantime, anything that can be done to get organized and compartmentalize non-user code that will be affected will help this proceed more quickly. I think this patch helps us along this path.

np on schedule. will be a bit more internal refactoring soon.

@TomAugspurger do you feel strongly on the formats -> formatting?

Not at all, especially since this is all internal and not user facing.

@jreback : Are you missing something in setup.py?  When I try installing the master branch, the formats folder does not get installed, leading to the error "No module named formats.printing"

I pushed a change.
though I installed fine from a fresh clone (this had to do with package setup)

@gfyoung you prob got somethign like this: https://ci.appveyor.com/project/jreback/pandas-465/build/1.0.409/job/fnp6uhksqcp039g4

@jreback : Indeed, I did, and I'm happy to say that with your most recent commit, everything installs just fine now!  Thanks!

great not sure why our install test on Travis didn't fail
but I guess it's not actually testing the built version but just creating it
in any event appveyor catches it because of the way it builds (it's a full conda build) not just an inplace build

@TomAugspurger I think the doc build broke: lots of Styler warnings now. https://travis-ci.org/pydata/pandas/jobs/121316764.
building locally so far so good though (not done yet)

Yes, just wanted to mention the same. It's the same error again as before. I can't remember anymore what solved it before

Mmm, I think it magically fixed itself last time. Will look later.

The .. currentmodule:: pandas.core.style in api.rst should probably updated?

yep, just pushed it

https://github.com/pydata/pandas/commit/8250d7c7ddfd321f91cf7ccd44cb906cdc63c0a0

ok, this now built ok for me locally.
Still these but IIRC they were there before
None:None: WARNING: toctree contains reference to nonexisting document u'generated/template:'


the collapse of running

When I try to use this package shown in the link to connect to wrds in matlab,  I run
wrds_install;
w = wrds('username','password');

my matlab (2015Ra) in laptop collapes immediately.
It prints in the command window:
Adding Ganymed-ssh2 to the java path by running
javaaddpath('/Users/huangshan/Documents/MATLAB/wrds-master/external/ssh2/ganymed-ssh2-build210/ganymed-ssh2-build210.jar')

Just added Ganymed-ssh2 to Matlab's dynamic java Classpath.
Connected.
Retrieving SAS library names (libref).
Request submitted to WRDS servers.

And then immediately a Matlab system error appears. It says:
matlab has encountered an internal problem and needs to close. The unsaved information you were working on may be lost. We are sorry for the inconvenience.

Thanks for reporting the issue. What Java version are you using (you can check on https://www.java.com/en/download/installed.jsp)? And what operating system?
In order, you can try to update:

wrds to most recent version
your java version
increase java heap space
Matlab's version to R2015b

I use a mac laptop, I checked the java, it says
Verify Java Version
We are unable to verify if Java is currently installed and enabled in your browser.
If you have installed Java and there is an error with the verification, there could be a configuration issue (eg. browser, Java control panel, security settings) or the Java plug-in is blocked by the browser. Try restarting your browser before trying to verify the installation again, and check that the browser allows Java to run.
Moreover, I find the dump document, but I can not upload it here as I find no attachment function.

I think you need to solve the Java issue first.  A link to start from is https://java.com/en/download/help/version_manual.xml. If you have issues with that, you might want to check on https://www.java.com/en/download/help/mac_java_update.xml.
To update files here, you would usually upload them on a file sharing app like dropbox and then share a link to the file.

thanks, I first sent you the dump. I am checking the problem of java. I am
not familiar with computer syntax. I even do not know what java is used
for....
2016-03-01 22:31 GMT+08:00 Oleg Komarov<EMAIL_ADDRESS>
Thanks for reporting the issue. What Java version are you using (you can
check on https://www.java.com/en/download/installed.jsp)? And what
operating system?
In order, you can try to update:

wrds to most recent version
your java version
increase java heap space
http://www.mathworks.com/matlabcentral/answers/92813-how-do-i-increase-the-heap-space-for-the-java-vm-in-matlab-6-0-r12-and-later-versions
Matlab's version to R2015b

—
Reply to this email directly or view it on GitHub
https://github.com/okomarov/wrds/issues/7#issuecomment-190742644.

I upload the java to version 8 update 73 and run the code above again. A dump receives as before.
https://www.dropbox.com/s/0oib7nsh0xw2oqs/matlab_crash_dump.1597-1?dl=0

@Shine119 The link is invalid.

@Shine119 I looked into the crash dump and cannot narrow down the cause. I suggest to submit the crash dump to Mathworks technical support maybe even with the link to this issue. You can also try to clean re-install your Matlab version to see if that fixes.

@Shine119 Have yuo had any luck in solving the issue? I would be interested myself to know if the cause is still to be found in the submission.

Thanks for your caring, the problem is still there, crash every time.
So in order to get data from WRDS, I just logged in its web set and download data there as xlsx document and then use Matlab to read it.
Sent from my iPhone

On 3 Mar, 2016, at 17:51, Oleg Komarov<EMAIL_ADDRESS>wrote:
@Shine119 Have yuo had any luck in solving the issue? I would be interested myself to know if the cause is still to be found in the submission.
—
Reply to this email directly or view it on GitHub.


Fix check for whether host's /home is a symlink.

Check in create() if /home on the host redirects to /var/home,
and modify the container accordingly in init-container(). This
restores the logic introduced in commit 66e982a which was
accidentally undone by commit 8b845e.
Closes #185

I took the liberty to make the above adjustments.

Thank you very much for your contribution!


Some jobs are forever in 'Running' State with WorkflowTaskFailed Error

Some of the workflows remain in Running state while WorkFlowTask failed activity is triggered with the following error. This seems to happen only when I trigger multiple (20-100) instances of the workflow at the same time so it is a little sporadic in nature. The same workflow that works normally, hangs in ‘running’ mode with the error below:
Expected Behavior
All runs finish with 'complete' status
Actual Behavior
Some runs fail with the following bug
15:25:20.731 [Workflow Executor taskQueue="Classify", namespace="default": 1568] ERROR i.t.internal.worker.PollerOptions - uncaught exception
java.lang.RuntimeException: Failure processing workflow task. WorkflowId=438cb389-3973-49cf-b298-64cee904ad44, RunId=6810e3eb-be56-4e3b-b810-3c12bf44f76e, Attempt=798
	at io.temporal.internal.worker.WorkflowWorker$TaskHandlerImpl.wrapFailure(WorkflowWorker.java:349)
	at io.temporal.internal.worker.WorkflowWorker$TaskHandlerImpl.wrapFailure(WorkflowWorker.java:279)
	at io.temporal.internal.worker.PollTaskExecutor.lambda$process$0(PollTaskExecutor.java:79)
at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
	at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: io.temporal.internal.replay.InternalWorkflowTaskException: Failure handling event 302 of 'EVENT_TYPE_WORKFLOW_TASK_STARTED' type. IsReplaying=false, PreviousStartedEventId=302, workflowTaskStartedEventId=302, Currently Processing StartedEventId=302
	at io.temporal.internal.statemachines.WorkflowStateMachines.handleEvent(WorkflowStateMachines.java:188)
	at io.temporal.internal.replay.ReplayWorkflowRunTaskHandler.handleEvent(ReplayWorkflowRunTaskHandler.java:140)
	at io.temporal.internal.replay.ReplayWorkflowRunTaskHandler.handleWorkflowTaskImpl(ReplayWorkflowRunTaskHandler.java:180)
	at io.temporal.internal.replay.ReplayWorkflowRunTaskHandler.handleWorkflowTask(ReplayWorkflowRunTaskHandler.java:150)
	at io.temporal.internal.replay.ReplayWorkflowTaskHandler.handleWorkflowTaskWithEmbeddedQuery(ReplayWorkflowTaskHandler.java:201)
	at io.temporal.internal.replay.ReplayWorkflowTaskHandler.handleWorkflowTask(ReplayWorkflowTaskHandler.java:114)
	at io.temporal.internal.worker.WorkflowWorker$TaskHandlerImpl.handle(WorkflowWorker.java:319)
	at io.temporal.internal.worker.WorkflowWorker$TaskHandlerImpl.handle(WorkflowWorker.java:279)
	at io.temporal.internal.worker.PollTaskExecutor.lambda$process$0(PollTaskExecutor.java:73)
	... 3 common frames omitted
Caused by: java.lang.RuntimeException: WorkflowTask: failure executing SCHEDULED->WORKFLOW_TASK_STARTED, transition history is [CREATED->WORKFLOW_TASK_SCHEDULED]
	at io.temporal.internal.statemachines.StateMachine.executeTransition(StateMachine.java:140)
	at io.temporal.internal.statemachines.StateMachine.handleHistoryEvent(StateMachine.java:91)
	at io.temporal.internal.statemachines.EntityStateMachineBase.handleEvent(EntityStateMachineBase.java:63)
	at io.temporal.internal.statemachines.WorkflowStateMachines.handleEventImpl(WorkflowStateMachines.java:205)
	at io.temporal.internal.statemachines.WorkflowStateMachines.handleEvent(WorkflowStateMachines.java:173)
	... 11 common frames omitted
Caused by: java.lang.ClassCastException: class scala.runtime.NonLocalReturnControl cannot be cast to class java.lang.Exception (scala.runtime.NonLocalReturnControl is in unnamed module of loader 'app'; java.lang.Exception is in module java.base of loader 'bootstrap')
	at io.temporal.serviceclient.CheckedExceptionWrapper.wrap(CheckedExceptionWrapper.java:56)
	at io.temporal.internal.sync.WorkflowInternal.wrap(WorkflowInternal.java:412)
	at io.temporal.internal.sync.DeterministicRunnerImpl.runUntilAllBlocked(DeterministicRunnerImpl.java:211)
	at io.temporal.internal.sync.SyncWorkflow.eventLoop(SyncWorkflow.java:148)
	at io.temporal.internal.replay.ReplayWorkflowExecutor.eventLoop(ReplayWorkflowExecutor.java:74)
	at io.temporal.internal.replay.ReplayWorkflowRunTaskHandler$EntityManagerListenerImpl.eventLoop(ReplayWorkflowRunTaskHandler.java:329)
	at io.temporal.internal.statemachines.WorkflowStateMachines.eventLoop(WorkflowStateMachines.java:400)
	at io.temporal.internal.statemachines.WorkflowStateMachines.access$500(WorkflowStateMachines.java:71)
	at io.temporal.internal.statemachines.WorkflowStateMachines$WorkflowTaskCommandsListener.workflowTaskStarted(WorkflowStateMachines.java:820)
	at io.temporal.internal.statemachines.WorkflowTaskStateMachine.handleCompleted(WorkflowTaskStateMachine.java:121)
	at io.temporal.internal.statemachines.WorkflowTaskStateMachine.handleStarted(WorkflowTaskStateMachine.java:113)
	at io.temporal.internal.statemachines.FixedTransitionAction.apply(FixedTransitionAction.java:45)
	at io.temporal.internal.statemachines.StateMachine.executeTransition(StateMachine.java:137)
	... 15 common frames omitted

Steps to Reproduce the Problem

kick-off 20-100 jobs of a custom workflow
Most of them would complete successfully while few (1-9) would be in forever running state with workflowTaskFailed Activity

Specifications

Version: Temporal Client SDK version 1.3.1
Platform: IntelliJ

Hi @gosandhya

Failure handling event 302 of 'EVENT_TYPE_WORKFLOW_TASK_STARTED'
It looks like we at least have workflows with some pretty long histories here.

Any chance you can provide some isolation or reproduction of the problem?
If you can't, maybe you can provide some additional details or characteristics about the type of workflows you run.

Hello @Spikhalskiy
We run a classification workflow that has multiple activities that perform fetching the data, disk operations, and finally interacting with various APIs to get back the results.
Maxim pointed out there might be an edge case unhandled in temporal Java SDK particularly NonLocalReturnControl exception extending Throwable directly in Scala which is not ideal. So was hoping this issue could be fixed.
Caused by: java.lang.ClassCastException: class scala.runtime.NonLocalReturnControl cannot be cast to class java.lang.Exception (scala.runtime.NonLocalReturnControl is in unnamed module of loader 'app'; java.lang.Exception is in module java.base of loader 'bootstrap')
	at io.temporal.serviceclient.CheckedExceptionWrapper.wrap(CheckedExceptionWrapper.java:56)
	at io.temporal.internal.sync.WorkflowInternal.wrap(WorkflowInternal.java:412)

Class cast exception will be fixed in 1.4.0, but the behavior itself (workflow stuck in Running) will not change.
Temporal treats only subclasses of TemporalFailure as workflow execution failure, other exceptions or throwables are treated like workflow bugs, and Temporal continues to retry them.
In your case workflows are not stuck, Temporal actually keeps retrying them and they will succeed if you fix the exception and redeploy the workers.
If you want the scala.runtime.NonLocalReturnControl Throwable be treated as a workflow execution failure, wrap it into TemporalFailure class or use WorkflowImplementationOptions#setFailWorkflowExceptionTypes https://github.com/temporalio/sdk-java/blob/657a392333e6d623fda91b1d7a0ea1f152f2eb9b/temporal-sdk/src/main/java/io/temporal/worker/WorkflowImplementationOptions.java#L68 to specify scala.runtime.NonLocalReturnControl  as a type that causes Workflow fail.

Hi, thank you for responding to this promptly! Appreciate it.
Do you have any time estimate for when will version 1.4.0 be released?

We are currently targeting 5 October, but it's normal for us to delay releases if we are unsure about safety.


Error: Please install chromadb as a dependency with, e.g. npm install -S chromadb

I get this error every time I try to run my workspace, i tried to run "npm install -S chromadb" but I get this error every time

This should be fixed in the latest flowise version


WebsiteMap ignores port

When the Base_Url contains a port, the baseUrlToWebsiteMap doesn't want a port.
We should be consistent and the port is part of the base url, therefore it should be part of the website map.

Reproduce: Add port to website map and it doesn't work as expected (hope I remember correctly)


Mentioning default bitness as 32-bit for App Services created from th…

The documentation should mention the default bitness as 32-bit for App Services created from the portal. This can be added to the 'bitness' field under Platform Settings (https://docs.microsoft.com/en-us/azure/app-service/configure-common#configure-general-settings)

@anandanthony : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Show Deploy results to user

Right now, the picoG Desktop software does not indicate whether a deploy succeeded or failed. It would be create to see why the deploy failed.
Ideas:

Option to save the generated VIA somewhere that the user can look at
Option to save a log file with of all the serial communication (VIA commands/responses sent to/from device) during the deploy

The work-around is to make sure that the app has a blinking (heartbeat) functionality, and this only tells whether the deploy succeeded or failed (and not why).

Will be implementing a terminal feature that can remain active for all connected devices. The terminal feature will be used by the deployment tool as well as allow viewing output from a running app / interacting via picoG's stdin functions. Need to refactor the Resource class to support more of this stateful connection and to facilitate a read loop that runs in the background.

This will be available on the device detail panel. A user can manually start the terminal connection if not yet connected, such as after a deploy process. Once open, a terminal connection will remain active even when navigating away from the device until the app closes, the user stops the terminal, or some other action in the application needs to stop the terminal.