Combine stdout and stderr when getting version from requried commands

What issue type does this pull request address? (keep at least one, remove the others)
/kind enhancement
What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)
Some required commands output their version on stderr instead of stdout. Devspace is not handling stderr when running command version so it is not possible to assess that the command is installed with its right version. This change changes the behavior so stdout and stderr are combined and used the same to look for the version string/
Please provide a short message that should be published in the DevSpace release notes
Combine stdout and stderr for required commands when getting their version
What else do we need to know?
This can potentially break compatibility since some programs can mix version data between stdout and stderr output. I assume this is an edge case that should not be common at all but is still possible.

Hi @lizardruss, here're some examples I'm working on:
Full disclosure: I'm also contributing to this PR which adds Tanka support to devspace.

Tanka: tk --version
jsonnet-bundler: jb --version

I tend to agree with you that combining stdout and stderr is not in general a good thing to do, I've written my opinion on that in my initial comment. But I think that in this case is the less dangerous one. We are using a well-known particular switch (version, --version, -V, ...) to get the version, and stdout and stderr mixing should not be something that might collide with each other in this case. Most likely, either there is stdout only or stderr only. That is at least my rationality.
As an alternative, we might add another field under require.command[] that specifies which output we want to capture, but I thought that can be overkill for such a task. I can develop it though if you think is better.

I missed this in my comment:

It's possible to add a small script to invoke the command and redirect stderr to stdout

I guess you're referring on having a wrapper to capture the stderr and throw it into stdout. That is possible, although

either a wrapper with the name of the original tool needs to created per tool (since we use require.commands[].name as the executable to run)

require:
  commands:
    - name: tk_wrapper.sh
      versionArgs: ["--version"]
    - name: jb_wrapper.sh
      versionArgs: ["--version"]

or use the same name for all commands with different versionArgs:

require:
  commands:
    - name: wrapper.sh
      versionArgs: ["tk", "--version"]
    - name: wrapper.sh
      versionArgs: ["jb", "--version"]

On option 1 I think it can be too cumbersome to maintain. Option 2 I have doubts it might work (right now I don't know if name collision is checked). In any case, none of these look to me elegant solutions, although I agree it will maintain compatibility (if there is any non-compatibility on my original approach).


Mekko charts III: The Return Of The Overlap - Browser compatability

One thing to test: browser compatibility...
Chrome/Safari 😄

Firefox 48.0.1 😢

I think the Firefox thing has to do with this PR because I checked out some commits from way back when and it started working again... it's a bit odd.
IE9

Returns a D3 error
IE11, Edge
Are a bit untesteable on a localhost machine due to CORS errors with localhost.

Hmm, this is deploy-critical as the ETM freezes on IE9 when you open a mekko. @grdw is looking into this. Do you have any ideas, @antw ?

I managed to fix Firefox problems 👍
IE9 is up next.

I'm close though. The problem is in the fitHeightToLabels() method. I'm sort of commenting out lines and adding them back in again to see what causes the D3 error.

IE11, Edge

IE11 and Edge both look fine on my Windows machine.
I don't have IE9 available. I can get it, but probably not soon enough to be any help. 😬

The problem is in the fitHeightToLabels() method.

Perhaps fitHeightToLabels: -> needs to be fitHeightToLabels: => (fat arrow) for IE9?

No, it's specificly in this line that it breaks ... now I need to figure out the details 😅

I see why. Appereantly IE9 doesn't know how to select a [object SVGSVGElement]. Which causes an argument error on the side of D3. I think that's what somebody at D# already found out.
So I guess we would have to this with setAttribute (it fixes the problem):
// Instead of...
d3.select(real_svg).style('height', g_height + transform)
// ... we'd use:
real_svg.setAttribute('height', g_height + transform)

So I guess we would have to this with setAttribute (it fixes the problem):

That's probably better anyway. :+1:

It does suck at wrapping the labels though... but it fits.


Feature/event listener

Coverage decreased (-0.2%) to 99.599% when pulling 911911cc8ae30032bc2843cf6e2b5b006979266b on msohailhussain:feature/eventListener into ef897e2932c534f9d0853c7ae67694a21ff55261 on optimizely:master.

Coverage decreased (-0.2%) to 99.599% when pulling f36e9be1560c81aa158e6899064a3d60b150709b on msohailhussain:feature/eventListener into fc5731e6c0277d6072353bb85a336e4e12be8945 on optimizely:master.

Coverage decreased (-0.2%) to 99.597% when pulling 745b821b7ceef2fa5c86c41c320291ba2ce3d194 on msohailhussain:feature/eventListener into fc5731e6c0277d6072353bb85a336e4e12be8945 on optimizely:master.

Coverage decreased (-0.2%) to 99.597% when pulling 1a56a484a895817f93bbc962887644040842147e on msohailhussain:feature/eventListener into fc5731e6c0277d6072353bb85a336e4e12be8945 on optimizely:master.

E2E tests are passing @thomaszurkan-optimizely lemme know if you want to take a look at this before I merge it in!


Paste/delete multiple items

First of all, thanks for creating Maccy and all the work that has been put in so far.
My request is the following. I'd like to be able to copy many individual items to the clipboard history, invoke Maccy, select multiple items and paste them. I imagine all the items would be pasted in one go, joined by \n or something similar.
This is handy in situations, in which the source is spread over multiple places, e.g. distinct address fields (Street, Zip, City, etc.) that need to pasted as one element later.

I will consider merging PR which adds support for this (e.g. select items by pressing Shift+arrow and then they all are pasted), but I don't have time to design the UX for this and implement it right now.

I came here to find out if this is possible as well. For my use case, I would just like to able to dump the whole clipboard history at once (something like Shift+A to select everything and then Cmd+V to paste?, or whatever other way would work best). I don't think any other clipboard manager has this function, or at least none that I can find, so I was wondering if there was some technical reason for that.

Why would you want to copy and save all clipboard history? This looks shady to me as it allows anyone with access to computer quickly save all the history.

It's just a quicker way to do what the original poster in this thread suggested, in the case that you need all of the items from your session, and not just the last 5-6.
In my case it would be to save words from a book or a document that I'm reading in a separate document. At the moment, if I need to save 50 words—I need to perform 50 paste operations. I would prefer just copying as I read and then pasting everything at once when I'm done, but admittedly I hadn't thought of the security concerns.
The original suggestion, and your response, would be a big enough improvement for me anyway (Shift+arrowing the last 50 from maccy's history, then pasting them with one operation, if I understood it correctly), so if you think that that is less shady, I would be happy to be able to do it that way.

I've heard this called "sequential paste" and it's very useful. Something like ctrl+cmd+c then cmd-c to push a bunch of things onto the sequential paste queue, then ctrl-cmd-v to pop them off the paste queue (hitting return in between, if you'd like).

@sfc-gh-afedorov Interesting, can you share any links or examples where this "sequential paste" is implemented?

I have built a dummy prototype of this feature where it's possible to mark multiple items (shift+click or shift+arrow key) and then delete/pin them at once. It's also possible to batch their pasting by iterating over the list, copying item to the system clipboard and then emitting Cmd+V keystroke.
It quickly reveals the problem that after each paste, there is no newline added and all items are pasted in a single line (or single cell in case of Numbers.app). I don't know how to overcome this problem because:

I cannot emit Enter keystroke - it would cause different behavior depending on the app.
I don't think I can construct new pasteboard entry which contents is joined by all the contents of every item in the list + "\n" character between. It would work for simple content such as strings, but won't work for rich formatting contents such as formatted text or images. Making this work without formatting strings would be quite a poor UX in my opinion.

For now I'm stuck as I can't imagine any solution to this problem. Looking forward to hear any suggestions!

no newline seems fine, if there is no newline in the input, no need to change anything
don't paste all the contents, just paste one item at a time, just as it was copied. this is also useful if you want to paste them in a slightly different order than you copied.
e.g. from pastebot — https://user-images.githubusercontent.com/42752788/154008231-234ee656-bf2d-409a-b0e0-5a1da203fbb7.mov

no newline seems fine, if there is no newline in the input, no need to change anything
don't paste all the contents, just paste one item at a time, just as it was copied. this is also useful if you want to paste them in a slightly different order than you copied.

Then I guess it's useless for the original author of this issue.

e.g. from pastebot — https://user-images.githubusercontent.com/42752788/154008231-234ee656-bf2d-409a-b0e0-5a1da203fbb7.mov
Can't load this movie :(

I can see how a sequential paste may be useful in other use cases (e.g. filling out forms with multiple fields). Without the new-line, however, the user has to manually type enter before every paste operation.
It would still be an improvement over the current solution because it would remove the need for selecting the next entry and remembering which entries have been already used.

Then I guess it's useless for the original author of this issue.

A bit strong, I think he can just hit "return" between pastes?

Can't load this movie :(

https://youtu.be/uDQrRbXacEk

I think he can just hit "return" between pastes?

It won't be possible to hit return between pastes because they are performed in a row with no delay.

https://youtu.be/uDQrRbXacEk

Ok, now I understand what this is better. It is different from what I originally thought of because it requires "manual" pasting, while I implemented the automatic pasting.
I will need to think about this more!

I agree both the ability to paste multiple items and sequential paste would be very useful. However, the challenge of how to integrate them into the app without spoiling Maccy's simple UI is considerable.
Starting with pasting multiple items first. The ideal would be a system like you mentioned earlier where a hokey (such as opt-cmd-down) is used to select multiple items on Maccy's clipboard and then paste the whole selection as usual, with a choice in settings as to whether each item is to be separated by a new line,  a space or nothing at all. But it seems you were running into issues with that. An alternative implementation could be one more similar to how Alfred's clipboard manager works whereby you append an item to the most recent entry on the clipboard with a hotkey, making them into one entry. This is effectively the same as deleting the most recent clipboard entry and re-copying a replacement with a single clipboard entry consisting of both entries (with the entries separated as specified in the settings). The Alfred settings panel sets it out as follows:

Now coming on to sequential paste, I have seen it implemented in Pastebot and Paste in almost exactly the same way as seen in the video above. This requires opening a secondary clipboard pane. One possible implementation of sequential paste that achieves a similar goal but without changing Maccy's UI too much would be to have a hotkey whose behaviour is to both paste and delete the most recent entry in the clipboard. This has the obvious disadvantages of deleting the selection after use and requiring you to copy the selections in the 'reverse' order as you intend to use them. But I can't see another way of implementing this behaviour without adding a secondary clipboard or some other UI complexity.

What do you guys think if I'll implement some support for selecting multiple entries and a custom Shortcut action that allows fetching the selected items? This would allow advanced users to build highly customized workflows to iterate over selected entries and paste them as they wish.
For example, a custom workflow could be run after selecting multiple entries in Maccy to paste them one by one and press "Enter" between pastes:

I was about to open an enhancement request to do something similar when I found this.  I'd like to suggest a slight generalization; instead of just inserting a \n between items, allow the user to select a seperator as specified in joined().
My specific use case is picking out individual words from wikipedia pages and pasting them as a list.  Sometimes I want word1, word2, word3, word4 when it's in running text.  Sometimes I want word1|word2|word3|word4 when I need them as template arguments.  And sometimes even:
* word1
* word2
* word3
* word4

I've been looking at the code, but haven't done any Mac development in eons (and never in Swift).  I'd be happy to work with somebody on this, however.

What do you think about https://github.com/p0deje/Maccy/issues/239#issuecomment-1233467000? In Shortcuts it's possible to ask for input (in your case it would be separator), then iterate over selected items in Maccy, and paste each one followed by the input.

It seems like a plausible idea, but I'm afraid I don't understand AppleScript well enough to understand how this would work from a UX perspective.

I'll try drafting something and show.

Just to add -- for Windows, Ditto clipboard supports multiple paste. It has a setting for user-defined character to join the strings (e.g., comma, newlines, etc.).
It's done using Shift+Arrow keys to select the contents, and then Enter.

Ditto does it perfetcly on Windows. We can follow the same path here.

Separator configurable via settings. \n would be a good default I believe.
Shift + Up/Down for selecting items in the list (Command + Click for cherry-picking would also be handy)
Enter pastes it (or just copies depending on your setting)

I want this feature! Installed Maccy now hoping it would have this feature 😢 .
My use-case was to copy many colors from Figma in order and paste them in my editor.
But I wrote a quick shell script to solve my use-case instead of buying the Paste app.
Here's the script and how to use it, if someone is interested:
https://gist.github.com/scmx/77b84b09ab797d901978b5e4ce61623e

First time commenting in GitHub and following along. 👀 I was emailing you a bit @p0deje and realized when checking into this that this was an open case.  🥳

In 0.29 it's possible to create a simple Shortcuts.app workflow that will do sequential paste for you. It would look smth like this:

I am running this script in my terminal to dump all entries from maccy to a file:
rresult=$(sqlite3 ~/Library/Containers/org.p0deje.Maccy/Data/Library/Application\ Support/Maccy/Storage.sqlite "SELECT ZTITLE FROM ZHISTORYITEM ORDER BY ZFIRSTCOPIEDAT;")

echo "$result" > dump.maccy

Here is a sqlite3 query you can run in a terminal to get the most recent copied items
sqlite3 ~/Library/Containers/org.p0deje.Maccy/Data/Library/Application\ Support/Maccy/Storage.sqlite \
"SELECT ZVALUE FROM ZHISTORYITEMCONTENT WHERE ZTYPE = 'public.utf8-plain-text' ORDER BY Z_PK DESC LIMIT 10;"

Thanks for the sqlite commands! Added a zsh shortcut to use fzf to combine commands into the system clipboard: https://github.com/gibfahn/dot/commit/9b945d679a663af1a39afece0b2e93ae6a2badd7
Not quite as nice as a "select a bunch of entries and paste them" feature, but better than nothing.

I would personally love to see this feature implemented! Is your test implementation still being updated @p0deje? I might be able to make some time to contribute in this regard if I know where to look haha
Thank you for putting together such a nice app!

@philocalyst I'd recommend looking into branch 2.x - I'm about to ship a major update and implementing a multiple selection is more straightforward there.

I would be even happy to donate money for implementation of this feature. This is the only thing I am missing terribly since moving from windows laptop. Whenever I am working on numerical data, I have to copy and paste multiple items. Currently, my flow looks something like copying all items one by one from multiple sources and then paste it same way. Ditto on Windows has a feature where you can select the copied items and then paste all of them with just a single paste command. I think this is a very useful feature for people who copy contents from multiple sources with a single destination to paste.

CopyQ (the clipboard manager I used before Maccy) uses newlines to separate pastes (at least by default).

@p0deje it uses newline as a separator by default.

@philocalyst I'd recommend looking into branch 2.x - I'm about to ship a major update and implementing a multiple selection is more straightforward there.

Is it available in 2.x? How do you select multiple items?

you could just stick the following in your shell rc file
savecb() {
	sqlite3 ~/Library/Containers/org.p0deje.Maccy/Data/Library/Application\ Support/Maccy/Storage.sqlite "SELECT ZTITLE FROM ZHISTORYITEM ORDER BY ZFIRSTCOPIEDAT;" | sed 's/\xE2\x8F\x8E/\n/g'
}

and then call it like this
savecb > afile

In addition to extracting all the content of clipboard, it converts Unicode character ⏎ to an actual carriage return.


ZPM to support %SYS module install

Hi,
Issue:
Many applications need some initial % data pointers (global nodes) and initial routine(s) to check the environment or to start a background job, etc.
InterSystems IRIS offers the %SYS database with its %ZSTART, ZSTU and ZMIRROR.
Enhancement Request:
ZPM should allow a user to define %SYS installation modules so that if I needed to set a couple of application settings in a ^Global and run some initial code as the system comes up (ZSTU, %ZSTART, ZMIRROR), I could wrap those things in a ZPM module for %SYS.
Some links that may be useful:
ZSTU
%ZSTART and %ZSTOP
ZMIRROR

This ends up becoming a release blocker because there are a lot of community packages that should be described in this way (the main set of differences between the master and v1 branches)

The v0.9.0 implication here ends up being covered better by #470 - pushing the actual SystemRequirements aspect of this to v1.


Resolve windows 3.8 issue

Adds the Python matrix to the non-default settings test. This would have uncovered the issue from #94. Then it resolves #94 by avoiding the official installation script for Windows targets.
A few things to consider:

It would have been nice to limit the logic to Windows for Python <= 3.8, but we cannot reliably know what Python version is active, can we? I believe using setup-python after this action works most of the time, if not all of the time.
This just reverts to the old installation script. Are there any reasons to consider using a newer version of it?

@miigotu looking at the poetry upstream issue, they recommend using powershell (default windows shell) instead of bash. Perhaps we should spend a little effort looking into whether we can create an ergonomic way of setting up an os-matrix with powershell as the shell for windows.
We could even go as far as outputting specific errors for windows bash users, telling them to switch to powershell on python 3.7 and 3.8, though perhaps that should be considered a breaking change.
Just thinking out loud 🤔

I think we should support both methods. I can look a bit at it later, tonight.
I do have a huge project coming up (with a new developer joining) on sickchill. Rewriting the entire UI with jinja2, webpack, reactjs, and a flask backend on top of reorganizing the entire user flow for shows and settings.
So I'll be on to get a wire frame up for him and take a look at poetry PowerShell.

Yeah it now seems like the bash bug could be fixed, so yeah sounds better to support both. Maybe we should merge this in the meantime to fix v1

Yes I believe so. And maybe a refactor to support both bash, cmd, and PowerShell should be a major version bump, even though any shell they have set should work then.

Does the fix in 96 not work with the newest poetry installer script?
I'd much rather not pin an old installer, as there are a bunch of new configuration settings we can expose with it


Wrong risk values for old android users.

Describe the bug
some time ago "Diagnosis Keys Upload: Transmission Risk Levels are applied in reverse order"
has been fixed. (https://github.com/corona-warn-app/cwa-app-android/issues/678)
So infected users with a newer Android version send the risk values in the right order.
I assumed, with the the fix on the client side, the problem goes away.
But when you lock on, for example
https://svc90.main.px.t-online.de/version/v1/diagnosis-keys/country/DE/date/2020-06-28
, you will see there are still users that submit the risk in the wrong order.
21-06-20 10 key with risk 6
22-06-20 10 key with risk 8
23-06-20 10 key with risk 8
24-06-20 10 key with risk 8
25-06-20 10 key with risk 5
26-06-20 10 key with risk 3
27-06-20 10 key with risk 1
https://svc90.main.px.t-online.de/version/v1/diagnosis-keys/country/DE/date/2020-06-27
17-06-20 10 key with risk 6
18-06-20 10 key with risk 7
19-06-20 10 key with risk 8
20-06-20 10 key with risk 7
21-06-20 10 key with risk 5
22-06-20 10 key with risk 3
23-06-20 10 key with risk 2
24-06-20 10 key with risk 1
25-06-20 10 key with risk 1
26-06-20 10 key with risk 1
With this wrong submission  a user with contact to the key "26-06-20 10 key with risk 1" will get a low risk info. But with the right risk value of 6 it would be much higher. On the other hand a user with contact to
"17-06-20 10 key with risk 6" gets high risk info, but it should be low.

Expected behaviour

To fix the client side bug, also a server side fix is needed.
for 2020-06-28 in the above example it should be:
21-06-20 10 key with risk 1
22-06-20 10 key with risk 3
23-06-20 10 key with risk 5
24-06-20 10 key with risk 8
25-06-20 10 key with risk 8
26-06-20 10 key with risk 8
27-06-20 10 key with risk 6
for 2020-06-27 in the above example it should be:
17-06-20 10 key with risk 1
18-06-20 10 key with risk 1
19-06-20 10 key with risk 1
20-06-20 10 key with risk 1
21-06-20 10 key with risk 3
22-06-20 10 key with risk 5
23-06-20 10 key with risk 8
24-06-20 10 key with risk 8
25-06-20 10 key with risk 8
26-06-20 10 key with risk 6
So to fix the client side change in the profile (5,6,7,8,7,5,3,2,1) to (5,6,8,8,8,5,3,1,1)
and the "Diagnosis Keys Upload: Transmission Risk Levels are applied in reverse order"  bug
a server side fix is also needed.
Steps to reproduce the issue

Technical details

Host Machine OS (Windows/Linux/Mac):

Possible Fix

Additional context

Most Android clients have already 1.0.4 running - unfortunately, roughly 10% are still running the older 1.0.2 version which is affected. We will push out release 1.0.9 hopefully today, which will enforce an update on older Android clients. This will solve the problem.

A fix on the client side will not fix it for users that are running and old version?
But, if it is possible to "force" an update on client side, even the user is running an old version,
than that would fix the problem.
I do not know if that is possible.
Does that mean an old version (1.0.2) can not transmit values after 1.0.9 is published?
On the other hand it would be very easy to change the risk values immediately after a user has send them to the server and before they are published.

When the 1.0.9 server patch is delivered, all Android CWA apps which are not 1.0.4, will be forced to update to 1.0.4. If they are not updating, they will not be able to use the app.

Sound good.
That will fix it.
Thanks.

@christian-kirschnick
https://svc90.main.px.t-online.de/version/v1/diagnosis-keys/country/DE/date/2020-06-30 and
https://svc90.main.px.t-online.de/version/v1/diagnosis-keys/country/DE/date/2020-06-30/hour/17
still contain 5,6,7,8,7,5,3,2,1,1,1,1,1,1 in wrong order.
So how can I check:

that server version 1.0.9 is online now?
and, if 1.0.9 is online, the time it gets online?

If it was online before 17:00 UTC, the server fix does not work.
Since "release" shows version: 1.0.9 and 3 days ago.
Or is "release" the next release, that will be pushed.

I will close this ticket when 1.0.9 is online. At the moment 1.0.8 is still deployed.

1.0.9 is now live - therefore all Android clients below 1.0.4 are now deprecated and will be forced to update to 1.0.4.


There is no localhost adress on cli

When i type npm run dev it's working. But there is no localhost port number. so i couldn't find where the app working.

Its localhost:9090

Spent a while looking for this. Not being familiar with developing extensions didn't help.
This should be in the CLI output, or at least in the Readme.
Thanks

You can close the issue.


Trigger focus event before element value clear

Some JS libraries including redux-form reads element's value on focus event. It does not work as expected with Poltergeist because the value of the element is already updated to '' when focus is triggered.
Triggering focus before value clear looks better simulation of user editing to me.

This makes sense to me, but can anyone confirm whether it actually fixes issues with redux-form, react, etc?

@twalpole i don't know what about other guys, but it work for me.
fix this => #666

Change log entry is appreciated :)


Could not pass event PlayerJoinEvent to LuckPerms v5.3.20

Description

Could not pass event PlayerJoinEvent to LuckPerms v5.3.20
java.lang.IncompatibleClassChangeError: Found interface us.myles.ViaVersion.ViaManager, but class was expected

Reproduction steps

when player login in server
See error

Expected behaviour

https://pastebin.com/kuBxDpWf
Environment details

Server type/version: Paper running version 1.12.2 build 1618
LuckPerms version: v5.3.20
Proxy: Waterfall

Any other relevant details

Viaversion, Viabackwards, Viarewind, Protocllib are latest dev builds.

Viaversion, Viabackwards, Viarewind, Protocllib are latest dev builds.

Is this reproducible on latest stable for these? This looks like an issue on their end, not LP.

I think I have seen this before, it's probably due to their latest dev build having API breakages since the release version works from what I know.

Should be fixed in v5.3.24 and newer :)


Added LVM pool alerts

This PR adds alert rules for LVM pools(thick and thin). Three rules are added:

LVMVolumeGroupMissingPhysicalVolume- alerts if any underlying PV is missing for the volume group
LVMVolumeGroupCapacityLow- alerts if the volume group is running low on capacity
LVMThinPoolCapacityLow- alerts if thin pool is running low on capacity

Please resolve the conflict cause by the previous merge.


libxaw3d: new formula

Contributing to homebrew-xorg:
You can erase any parts of this template not applicable to your Pull Request.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Does your submission pass
brew audit --strict <formula> (where <formula> is the name of the formula you're submitting)?
[x] Have you built your formula locally prior to submission with brew install <formula>?

new libXaw3d formula

==> /home/linuxbrew/.linuxbrew/bin/gcc-5 test.c -o test
.
Last 150 lines from /home/ubuntu/.cache/Homebrew/Logs/libxaw3d/test.01.gcc-5:
2017-11-28 18:46:56 +0000

/home/linuxbrew/.linuxbrew/bin/gcc-5
test.c
-o
test

/home/linuxbrew/.linuxbrew/Cellar/gcc/5.4.0/libexec/gcc/x86_64-unknown-linux-gnu/5.4.0/cc1: error while loading shared libraries: libmpfr.so.4: cannot open shared object file: No such file or directory
Error: linuxbrew/xorg/libxaw3d: failed
Failed executing: /home/linuxbrew/.linuxbrew/bin/gcc-5 test.c -o test

Merged. Thanks, Maxim!


Nullreference exception when trying to read a sharedstorage.

Hello,
I tried accessing a d2i file (created through a offline playthrough in D2R). But I get a nullreference exception
when trying to read the data
    byte[] bytes = File.ReadAllBytes("test.d2i");
    D2I save = Core.ReadD2I(bytes, 0x61);

The project is a .netcore 3.1 (lts) console application. My diablo 2 R installation is up to date.
Any idea what could cause this?
I tried running the unit test using my shared storage file :
Message:
    Test method D2SLibTests.D2ITest.VerifyCanReadSharedStash115 threw exception:
    System.NullReferenceException: Object reference not set to an instance of an object.
  Stack Trace:
    Item.ReadComplete(BitReader reader, Item item, UInt32 version) line 313
    Item.Read(BitReader reader, UInt32 version) line 152
    ItemList.Read(BitReader reader, UInt32 version) line 62
    D2I.Read(Byte[] bytes, UInt32 version) line 18
    Core.ReadD2I(Byte[] bytes, UInt32 version) line 49
    D2ITest.VerifyCanReadSharedStash115() line 19`

Taking a look at the callstack the itemList received strange values but I still don't know why
        public static ItemList Read(BitReader reader, UInt32 version)
        {
            ItemList itemList = new ItemList();
            itemList.Header = reader.ReadUInt16(); // <====== yield 43605
            itemList.Count = reader.ReadUInt16(); // <====== yield 43605
            // ...

After further analysis, I've come to realize that the serializer may have been outdated? I've tried several operations on the storage and each storage page begins with 55 AA 55 AA (43605 43605)
Regards
I shared my storage file in case you want to test it: https://mega.nz/file/t1sRCKoR#UaGEkK6jIfLLHBtwaIhMob0atSE_AVajSSuVgqFdNDo

Do you use any mods like bigger inventory? Basically anything that changes the base inventory/stash sizes?

I can't recall If I have done all my tests on modified storage but yes It was intended to handle custom sized storages operations.


docz command not found after successful install

Question
After upgrading node to version 8.9 (required by dependencies), I was able to successfully add docz to my dev environment. However, the docz command is not being found, even after restarting terminal.

I installed via command yarn add docz --dev
Attached screenshot is partial output of yarn add docz --dev and docz commands:

Node version 8.9.4
NPM version 6.1.0
Here is package.json:
{
    "name": "wordpress-child-theme-starter",
    "private": true,
    "devDependencies": {
        "autoprefixer": "^6.3.3",
        "babel-core": "^6.26.3",
        "babel-eslint": "^8.2.3",
        "babel-preset-env": "^1.6.1",
        "bootstrap": "^4.1.0",
        "browser-sync": "^2.11.1",
        "css-mqpacker": "^4.0.0",
        "cssnano": "^3.5.2",
        "del": "^2.2.0",
        "docz": "^0.5.0",
        "gulp": "^3.9.1",
        "gulp-babel": "^7.0.1",
        "gulp-cache": "^0.4.2",
        "gulp-concat": "^2.6.0",
        "gulp-eslint": "^2.0.0",
        "gulp-html-replace": "^1.5.5",
        "gulp-imagemin": "^2.4.0",
        "gulp-load-plugins": "^1.5.0",
        "gulp-notify": "^3.2.0",
        "gulp-postcss": "^6.1.0",
        "gulp-rename": "^1.2.2",
        "gulp-sass": "^2.2.0",
        "gulp-sass-glob": "^1.0.8",
        "gulp-sourcemaps": "^1.6.0",
        "gulp-uglify": "^1.5.3",
        "popper.js": "^1.14.3",
        "postcss-pxtorem": "^3.3.1",
        "postcss-uncss": "^0.16.1",
        "run-sequence": "^1.1.5",
        "stylefmt": "^6.0.0",
        "stylelint": "^9.1.3",
        "stylelint-config-recommended": "^2.1.0",
        "uncss": "^0.16.2",
        "webpack": "^4.5.0",
        "webpack-stream": "^4.0.3"
    },
    "browserslist": [
        "last 2 versions",
        "ie >= 11"
    ],
    "dependencies": {
        "jquery": "^3.3.1"
    },
    "scripts": {
        "dev": "docz dev",
        "build": "docz build"
    }
}

To access the docz command directly on your terminal you need to install it globally or use something like npx, when you install some package just by run yarn add <something> the package binary will be available just to run commands using your package.json script...
So, you don't need to install docz globally, just install as you did, but to run it use these script:
"scripts": {
  "dev": "docz dev",
  "build": "docz build"
}

Like that
yarn dev
yarn build

Understood. Thanks!


[WIP] feat(measurements): show untracked measurements in measurement panel under additional findings

Context
Update the following measurement types to be displayed in the measurement panel as “NON ACQUISITION MESUREMENTS”:
Angle
ArrowAnnotate
Bidirectional
CircleROI
EllipticalROI
Length
PlanarFreehandROI
Probe
RectangleROI
CSV export will export all the details for non acquisition measurements, SR export filters them out.
Pending design @sedghi @dan-rukas

https://github.com/cornerstonejs/cornerstone3D/pull/1281
Needs this then everything should work fine, I fixed button name and sub panel name, and rounding for freehand


Should we have an option not to clear the input on error?

Hi, firstly I would like to thank you for your effort in building this great library.
I would like to use this library with an option not to clear the input on error. Hence, the focus should be on the last character.
https://github.com/user-attachments/assets/1b887c39-25c6-432a-95be-5a35e9266d2b
I'm thinking of adding a new flag called clearInputOnError: Boolean = true to support this feature. I will make a contribution soon 😄

Hey Lucas thank you so much for your kind words! It makes sense to me. Waiting for the contribution :)

@tarikyasar here you are https://github.com/composeuisuite/ohteepee/pull/31 🧑🍳

Hi @tarikyasar, I still have some ideas for adding more options for my usage. Please wait a bit for more contributions 🙏


cd to dist directory before running docopt-completion

docopt-completion writes file to pwd, so cd before running command so output is in the dist directory.

Fixes #169

LGTM!


[🐛 in esl-image]: esl-image in inline svg mode accept any content in request which leads to XSS valrnurability

Add mechanism to sanitize html content at esl-image in inner-svg mode to prevent DOM XSS attacks.
esl-image allows loading SVG that can contain malicious attributes or elements which can lead to account impersonation, observing user behavior, loading external content, stealing sensitive data, and more.

:tada: This issue has been resolved in version 4.0.0-beta.9 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for BLE Nano V2  i.e. nRF52832

The latest Redbear BLE Nano (V2) uses nRF52832.
Can you add support for this chip please

It appears that this board is supported via
Arduino Core for Nordic Semiconductor nRF5 based boards
https://github.com/sandeepmistry/arduino-nRF5#supported-boards
I will try this process once the V2 board arrives.
Question "Can the DAPLink program RedBear provides in it KIT be used to Flash the SoftDevice instead of the CMSIS-DAP"?
if so should i use the RBL_DAPLink or the DAPLink  programmer?

See pull request #185.

@drmpf

Question "Can the DAPLink program RedBear provides in it KIT be used to Flash the SoftDevice instead of the CMSIS-DAP"?

Yes, I think CMSIS-DAP and DAP link are the same.

I confirm pull request #185.
works with RedBear Nano V2 and BLEPeripheral V0.4.0
I needed to add
  #include <WStream.h>
  #define Stream WStream

to a number of my files
Currently having trouble finding a define that correctly includes these two lines

I get the following warning but code works as expected
C:\Users\matthew\Documents\Arduino\SampleRedbearBLENanoScreens\SampleRedbearBLENanoScreens.ino:48:98: warning: non-static data member initializers only available with -std=c++11 or -std=gnu++11 [enabled by default]

     BLEService               uartService          = BLEService("6E400001B5A3F393E0A9E50E24DCCA9E");

#if defined( NRF52 )
  #include <WStream.h>
  #define Stream WStream
#else
  #include <Stream.h>
#endif

seems to work
#if defined( ARDUINO_RBL_nRF52832 )   DOES NOT


kinect2_bridge using opencl? but switches to opengl when client connected.

hello,
when i start kinect2_bridge.launch everything seems fine. opencl chooses my dedicated graphic card and the bridge is waiting for clients:
ben@ben-laptop:~/libfreenect2/build/bin$ roslaunch kinect2_bridge kinect2_bridge.launch
... logging to /home/ben/.ros/log/4d754ebe-02f5-11e6-9763-ac811248baa3/roslaunch-ben-laptop-6562.log
Checking log directory for disk usage. This may take awhile.
Press Ctrl-C to interrupt
Done checking log file disk usage. Usage is <1GB.

started roslaunch server http://ben-laptop:39118/

SUMMARY
========

PARAMETERS
 * /kinect2_bridge/base_name: kinect2
 * /kinect2_bridge/base_name_tf: kinect2
 * /kinect2_bridge/bilateral_filter: True
 * /kinect2_bridge/calib_path: /home/ben/catkin_...
 * /kinect2_bridge/depth_device: -1
 * /kinect2_bridge/depth_method: default
 * /kinect2_bridge/edge_aware_filter: True
 * /kinect2_bridge/fps_limit: -1.0
 * /kinect2_bridge/jpeg_quality: 90
 * /kinect2_bridge/max_depth: 12.0
 * /kinect2_bridge/min_depth: 0.1
 * /kinect2_bridge/png_level: 1
 * /kinect2_bridge/publish_tf: False
 * /kinect2_bridge/queue_size: 5
 * /kinect2_bridge/reg_device: -1
 * /kinect2_bridge/reg_method: default
 * /kinect2_bridge/sensor:
 * /kinect2_bridge/use_png: False
 * /kinect2_bridge/worker_threads: 4
 * /kinect2_points_xyzrgb_hd/queue_size: 5
 * /kinect2_points_xyzrgb_qhd/queue_size: 5
 * /kinect2_points_xyzrgb_sd/queue_size: 5
 * /rosdistro: indigo
 * /rosversion: 1.11.16

NODES
  /
    kinect2 (nodelet/nodelet)
    kinect2_bridge (nodelet/nodelet)
    kinect2_points_xyzrgb_hd (nodelet/nodelet)
    kinect2_points_xyzrgb_qhd (nodelet/nodelet)
    kinect2_points_xyzrgb_sd (nodelet/nodelet)

auto-starting new master
process[master]: started with pid [6574]
ROS_MASTER_URI=http://localhost:11311

setting /run_id to 4d754ebe-02f5-11e6-9763-ac811248baa3
process[rosout-1]: started with pid [6587]
started core service [/rosout]
process[kinect2-2]: started with pid [6602]
process[kinect2_bridge-3]: started with pid [6605]
process[kinect2_points_xyzrgb_sd-4]: started with pid [6606]
[ INFO] [1460716327.713504237]: Loading nodelet /kinect2_bridge of type kinect2_bridge/kinect2_bridge_nodelet to manager kinect2 with the following remappings:
process[kinect2_points_xyzrgb_qhd-5]: started with pid [6607]
process[kinect2_points_xyzrgb_hd-6]: started with pid [6620]
[ INFO] [1460716327.724265773]: waitForService: Service [/kinect2/load_nodelet] has not been advertised, waiting...
[ INFO] [1460716327.752445131]: Initializing nodelet with 4 worker threads.
[ INFO] [1460716327.770637638]: waitForService: Service [/kinect2/load_nodelet] is now available.
[ INFO] [1460716328.032658979]: [Kinect2Bridge::initialize] parameter:
        base_name: kinect2
           sensor: default
        fps_limit: -1
       calib_path: /home/ben/catkin_ws/src/iai_kinect2/kinect2_bridge/data/
          use_png: false
     jpeg_quality: 90
        png_level: 1
     depth_method: default
     depth_device: -1
       reg_method: default
       reg_device: -1
        max_depth: 12
        min_depth: 0.1
       queue_size: 5
 bilateral_filter: true
edge_aware_filter: true
       publish_tf: false
     base_name_tf: kinect2
   worker_threads: 4
[Info] [Freenect2Impl] enumerating devices...
[Info] [Freenect2Impl] 12 usb devices connected
[Info] [Freenect2Impl] found valid Kinect v2 @2:3 with serial 000000000
[Info] [Freenect2Impl] found 1 devices
[ INFO] [1460716330.700342172]: [Kinect2Bridge::initDevice] Kinect2 devices found:
[ INFO] [1460716330.700463133]: [Kinect2Bridge::initDevice]   0: 000000000 (selected)
[Info] [Freenect2DeviceImpl] opening...
[Info] [Freenect2DeviceImpl] transfer pool sizes rgb: 20*16384 ir: 60*8*33792
[Info] [Freenect2DeviceImpl] opened
[ INFO] [1460716330.797604728]: [Kinect2Bridge::initDevice] starting kinect2
[Info] [Freenect2DeviceImpl] starting...
[Info] [Freenect2DeviceImpl] submitting rgb transfers...
[Info] [Freenect2DeviceImpl] submitting depth transfers...
[Info] [Freenect2DeviceImpl] started
[ INFO] [1460716331.132894109]: [Kinect2Bridge::initDevice] device serial: 000000000
[ INFO] [1460716331.132944647]: [Kinect2Bridge::initDevice] device firmware: 2.3.3913.0
[Info] [Freenect2DeviceImpl] stopping...
[Info] [Freenect2DeviceImpl] canceling rgb transfers...
[Info] [Freenect2DeviceImpl] canceling depth transfers...
[Info] [Freenect2DeviceImpl] stopped
[ WARN] [1460716331.373685615]: [Kinect2Bridge::initCalibration] using sensor defaults for color intrinsic parameters.
[ WARN] [1460716331.373783457]: [Kinect2Bridge::initCalibration] using sensor defaults for ir intrinsic parameters.
[ WARN] [1460716331.373871037]: [Kinect2Bridge::initCalibration] using defaults for rotation and translation.
[ WARN] [1460716331.374002613]: [Kinect2Bridge::initCalibration] using defaults for depth shift.
[ INFO] [1460716331.493728019]: [DepthRegistration::New] Using OpenCL registration method!
[ INFO] [1460716331.493855746]: [DepthRegistration::New] Using OpenCL registration method!
[ INFO] [1460716331.686222773]: [DepthRegistrationOpenCL::init] devices:
[ INFO] [1460716331.686304399]: [DepthRegistrationOpenCL::init]   0: Turks (GPU)[Advanced Micro Devices, Inc.]
[ INFO] [1460716331.686333763]: [DepthRegistrationOpenCL::init]   1: Intel(R) Core(TM) i5-2410M CPU @ 2.30GHz (CPU)[GenuineIntel]
[ INFO] [1460716331.686368373]: [DepthRegistrationOpenCL::init] selected device: Turks (GPU)[Advanced Micro Devices, Inc.]
[ INFO] [1460716331.832333485]: [DepthRegistrationOpenCL::init] devices:
[ INFO] [1460716331.832410500]: [DepthRegistrationOpenCL::init]   0: Turks (GPU)[Advanced Micro Devices, Inc.]
[ INFO] [1460716331.832451810]: [DepthRegistrationOpenCL::init]   1: Intel(R) Core(TM) i5-2410M CPU @ 2.30GHz (CPU)[GenuineIntel]
[ INFO] [1460716331.832489430]: [DepthRegistrationOpenCL::init] selected device: Turks (GPU)[Advanced Micro Devices, Inc.]
[ INFO] [1460716331.997848938]: [Kinect2Bridge::main] waiting for clients to connect

when i run the example for the viewer rosrun kinect2_viewer kinect2_viewer sd cloud the bridge output shows, that now openGL is used. the viewer windows is just black.
[Info] [Freenect2DeviceImpl] starting...
[Info] [Freenect2DeviceImpl] submitting rgb transfers...
[Info] [Freenect2DeviceImpl] submitting depth transfers...
[Info] [Freenect2DeviceImpl] started
[Error] [RgbPacketStreamParser] packetsize or sequence doesn't match!
[Info] [DepthPacketStreamParser] -2 packets were lost
[ INFO] [1460716745.353498318]: [Kinect2Bridge::main] depth processing: ~11.442ms (~87.397Hz) publishing rate: ~29.6352Hz
[ INFO] [1460716745.353587121]: [Kinect2Bridge::main] color processing: ~3.06391ms (~326.381Hz) publishing rate: ~17.6479Hz
[Info] [OpenGLDepthPacketProcessor] avg. time: 12.0005ms -> ~83.3298Hz
[Info] [TurboJpegRgbPacketProcessor] avg. time: 32.8789ms -> ~30.4146Hz
[ INFO] [1460716748.353881401]: [Kinect2Bridge::main] depth processing: ~10.3305ms (~96.8007Hz) publishing rate: ~29.9961Hz
[ INFO] [1460716748.353980375]: [Kinect2Bridge::main] color processing: ~3.24659ms (~308.016Hz) publishing rate: ~19.3308Hz
[Info] [OpenGLDepthPacketProcessor] avg. time: 11.3232ms -> ~88.314Hz

depth_method and reg_method are by default set to opencl. when i force this arguments to be opencl in the launchfile, kinect2_bridge start fails with:
...
[ INFO] [1460717026.790307537]: [Kinect2Bridge::initialize] parameter:
        base_name: kinect2
           sensor: default
        fps_limit: -1
       calib_path: /home/ben/catkin_ws/src/iai_kinect2/kinect2_bridge/data/
          use_png: false
     jpeg_quality: 90
        png_level: 1
     depth_method: opencl
     depth_device: -1
       reg_method: opencl
       reg_device: -1
        max_depth: 12
        min_depth: 0.1
       queue_size: 5
 bilateral_filter: true
edge_aware_filter: true
       publish_tf: false
     base_name_tf: kinect2
   worker_threads: 4
[ERROR] [1460717026.790443675]: [Kinect2Bridge::initPipeline] OpenCL depth processing is not available!
[ERROR] [1460717026.790475563]: [Kinect2Bridge::start] Initialization failed!
[FATAL] [1460717026.871391937]: Service call failed!

i can't see any depth images with integrated viewer or with rviz :( please help me. any suggestions?
why openCL seesm to be fine when using default and fails when using opencl?

You have to compile libfreenect2 with OpenCL support. Your current libfreenect does not have OpenCL enabled. I guess you installed OpenCL after compiling libfreenect2, since kinect2_bridge detected OpenCL. Remove your libfreenect2 installation, remove the libfreenect2 build folder and run cmake and make install again. make should output that it is compiling opencl_depth_processor.cpp. Then rebuild kinect2_bridge.

I thought it was compiled with OpenCL suppport...
Nevertheless I reinstalled my graphic card driver and made a fresh libfreenect2 build; now everything is fine with kinect2_bridge.
-- Feature list:
--   CUDA    no
--   CXX11    yes
--   Examples    yes
--   OpenCL    yes
--   OpenGL    yes
--   OpenNI2    no
--   TegraJPEG    no
--   Threading    std::thread
--   TurboJPEG    yes
--   VAAPI    no
--   VideoToolbox    no (Apple only)
-- Configuring done

and
[ 84%] Building CXX object CMakeFiles/freenect2.dir/src/opengl_depth_packet_processor.cpp.o
[ 88%] Building CXX object CMakeFiles/freenect2.dir/src/opencl_depth_packet_processor.cpp.o
Linking CXX shared library lib/libfreenect2.so


Support calico networking

add calico daemons into package
tap device creation (save info about it somewhere in sandbox)
inform calico about new workload/endpoint
pass tap device to libvirt as network device
pass addressing data from calico in PodSandboxStatus

Progress could be monitored in #46

It's no longer valid.


⚠ Processors has degraded performance

In e700490, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 60 ms

Resolved: Processors performance has improved in b924401.


Annotate HTML output with template file names by default

Rails 6.1 added support to annotating HTML output with template file names. This adds a comment to the rendered HTML with the file name of the view used to generate that particular template portion.
That is a very valuable tool during development.

I like this. Makes for a sensible default.

This is on by default in Rails main

@MatheusRich thank you for the update.
Closing this, but noting that #1149 may need to still implement this configuration since https://github.com/rails/rails/pull/50436 is not captured in the latest release of Rails.


Offset

I would we could to set an offset (in hex) instead of displaying byte from zero

Closing, no plans to implement this.

Of course, after I close this, with no other interest for 2 years, more requests come in :). I'm reopening, but still currently don't have plans for this. We can think about how it might work though.

I think one thing that would be important is if you opened a subset of a file, it needs to be clear you're working on a subset, and you cannot extended that file. So the subset file backing must enforce this.


using with react navigation and passing navigate props

navigate not working,appears null if it another tab selected.
class Chats extends React.Component {

   const { navigate } = this.props.navigation;
  renderItem={({item}) => <TouchableOpacity onPress={() => navigate('Chat')}><View style={{flex: 1,flexDirection:'row'}}>

> TypeError: undefined is not an object (evaluating 'this.props.navigation.navigate')
>
> This error is located at:
>     in Chats (at App.js:294)
>     in StaticContainer (at SceneComponent.js:11)
>     in RCTView (at View.js:71)
>     in View (at SceneComponent.js:10)
>     in SceneComponent (created by ViewPagerAndroid)
>     in AndroidViewPager (at ViewPagerAndroid.android.js:238)
>     in ViewPagerAndroid (at createAnimatedComponent.js:147)
>     in AnimatedComponent (at index.js:246)
>     in RCTView (at View.js:71)
>     in View (at index.js:387)
>     in ScrollableTabView (at App.js:289)
>     in HomeScreen (at SceneView.js:31)
>     in SceneView (at CardStack.js:412)
>     in RCTView (at View.js:71)
>     in View (at CardStack.js:411)
>     in RCTView (at View.js:71)
>     in View (at CardStack.js:410)
>     in RCTView (at View.js:71)
>     in View (at createAnimatedComponent.js:147)
>     in AnimatedComponent (at Card.js:26)
>     in Card (at PointerEventsContainer.js:55)
>     in Container (at CardStack.js:454)
>     in RCTView (at View.js:71)
>     in View (at CardStack.js:383)
>     in RCTView (at View.js:71)
>     in View (at CardStack.js:382)
>     in CardStack (at CardStackTransitioner.js:97)
>     in RCTView (at View.js:71)
>     in View (at Transitioner.js:192)
>     in Transitioner (at CardStackTransitioner.js:49)
>     in CardStackTransitioner (at StackNavigator.js:60)
>     in Unknown (at createNavigator.js:52)
>     in Navigator (at createNavigationContainer.js:212)
>     in NavigationContainer (at App.js:323)
>     in App (at renderApplication.js:35)
>     in RCTView (at View.js:71)
>     in View (at AppContainer.js:102)
>     in RCTView (at View.js:71)
>     in View (at AppContainer.js:122)
>     in AppContainer (at renderApplication.js:34)
> render
>     index.delta?platform=android&dev=true&minify=false:58868:53
> proxiedMethod
>     index.delta?platform=android&dev=true&minify=false:34324:37
> finishClassComponent
>     index.delta?platform=android&dev=true&minify=false:7731:43
> updateClassComponent
>     index.delta?platform=android&dev=true&minify=false:7712:38
> beginWork
>     index.delta?platform=android&dev=true&minify=false:7987:42
> performUnitOfWork
>     index.delta?platform=android&dev=true&minify=false:9780:31
> workLoop
>     index.delta?platform=android&dev=true&minify=false:9832:49
> _invokeGuardedCallback
>     index.delta?platform=android&dev=true&minify=false:2774:21
> invokeGuardedCallback
>     index.delta?platform=android&dev=true&minify=false:2736:39
> renderRoot
>     index.delta?platform=android&dev=true&minify=false:9887:36
> performWorkOnRoot
>     index.delta?platform=android&dev=true&minify=false:10478:40
> performWork
>     index.delta?platform=android&dev=true&minify=false:10415:30
> batchedUpdates
>     index.delta?platform=android&dev=true&minify=false:10560:26
> batchedUpdates
>     index.delta?platform=android&dev=true&minify=false:4107:37
> _receiveRootNodeIDEvent
>     index.delta?platform=android&dev=true&minify=false:4190:23
> receiveTouches
>     index.delta?platform=android&dev=true&minify=false:4220:34
> __callFunction
>     index.delta?platform=android&dev=true&minify=false:2361:49
> <unknown>
>     index.delta?platform=android&dev=true&minify=false:2131:31
> __guardSafe
>     index.delta?platform=android&dev=true&minify=false:2323:13
> callFunctionReturnFlushedQueue
>     index.delta?platform=android&dev=true&minify=false:2130:21
>

Hello, @barron9.
I don't see any mention of react-native-underline-tabbar in your stacktrace. There is a direct message that error is located in your code. I suspect issue is in the way how you use your Navigator. You pass props incorrectly. You may want to ask your question on StackOverflow.
P.S. Please, next time before creating an issue, make sure the problem is not on your side :)

cant you solve this14:09, 4 Şubat 2018, Konstantin Yakushin notifications@github.com:Hello, @barron9.
I don't see any mention of react-native-underline-tabbar in your stacktrace. There is a direct message that error is located in your code. I suspect issue is in the way how you use your Navigator. You pass props incorrectly. You may want to ask your question on StackOverflow.
P.S. Please, next time before creating an issue, make sure the problem is not on your side :)
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
-- Yandex.Mail mobil uygulamasından gönderildi

@barron9, What doest that mean?

i need to open a new stack after tap from tab inside. how can i do that15:09, 4 Şubat 2018, Konstantin Yakushin notifications@github.com:@barron9, What doest that mean?
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.
-- Yandex.Mail mobil uygulamasından gönderildi

@barron9, please, don't answer via email. It produces a lot of garbage in message.
I can not solve your problem. The ussue is on your side. Use debagger, think on your problem.

how can you go another page with this tell your solution?

i am at example code i want to open a new class when i tap somewhere

It doesn't relate to this component, but anyway.
Here you can read the documentation for react-native-scrollable-tab-view. The documentation says

page (Integer) - set selected tab(can be buggy see  #126

It means you can set selected tab by passing page prop to tab-view.
Here is an example
      <ScrollableTabView page={this.state.currentPageIndex}>
       /* this.state./currentPageIndex is just for example */
        <ReactPage tabLabel="React" />
        <FlowPage tabLabel="Flow" />
        <JestPage tabLabel="Jest" />
      </ScrollableTabView>


Bitbucket Integration

reference
#36 #50  #176 #229
Notes :-

bibucket returns uuids so i  had to add the migrations to change tables that contains provider and provider_id columns

without bitbucket token sent as a request header , it won't retrieve the current user private repos that he owns so i had to change the request helper method

i added the refresh_token column , cause bitbucket token expires after 1 hour as mentioned in the documentation , also i provided a screenshot of how the object will look like
refreshToken

in handleProviderCallback the user from socialite must be updated , to update his refresh_token and token

some of the methods uses version 1.0 of the api , because these end points aren't supported in version 2.0

in the readCollaborators if the repo owner is logged in and currently importing the repo into GitScrum the method will get all associated collaborators , otherwise , it won't get associated collaborators, and the reason for this as mentioned in the documentation priviliges  , Only the repository owner, a team account administrator, or an account with administrative rights on the repository can make this call

Hi @Te7a-Houdini,
Thanks for you PR!
Please, send your contact mail to<EMAIL_ADDRESS>Cheers,
Renato Marinho


Bad title position on the iPhone X emulator

Hi, thank you for great library!
I found following bug on the iPhone x emulator, look to screenshot:

Do you have any suggestions how to fix it?
Thank you in advance!

Hi @boyfox !
When I run the command pod try BFPaperTabBarController and get the default app up and running it works how iPhone X specifications state that it should:

Are you doing anything custom with this control or maybe some AutoLayout issues with the new Safe Zones?

HI @bfeher !, Sorry for later replying, i bought iphone x and checked again, same issuse:

I'm using CustomUITabBar class for setting fixed height:
-(CGSize)sizeThatFits:(CGSize)size {
    CGSize sizeThatFits = [super sizeThatFits:size];
    sizeThatFits.height = 54; // 54pt - design in iphone 6 plus

    return sizeThatFits;
}

and i'm using vector images such as pdf files for icons.
others devices works fine:

Do you have any suggestion, i need increasing tab bar height just on Iphone X? can i get standart device tabbar height for checking?

Hi @boyfox!
Hmm. This seems like an issue with the new Safe Areas.  It looks like the content of the bar itself is pushed up correctly with safe zones, but the background isn't.
I can't reproduce this so I can only guess. Do you get the same results if you use a standard UITabBarController?

Yes, @bfeher you are right, Safe area stayed turn off in my project. But i can't switch on it, because my deployment target is iOS 8.0 .
Do you have any suggestions to fix it?

When I updated my apps I had a problem where the entire UITabBar was below the Safe Area. I adjusted the Autolayout property and it worked perfectly. However what is strange in your case is that only the background and icons are in the original position while the bar itself is correctly positioned. Perhaps some of your auto layout constraints within the bar need new values when on the iPhone X? (Since you support iOS 8 and can't use Safe Area, maybe you can check if you are on the iPhone X and push up your constraints which are probably set to zero, to the height of the safe area?)

ok @bfeher thank you for suggestions, you can close or remove this issue, because this is not a bug of library. Thank you for great library!

@boyfox Sorry that I couldn't help more. If you find out that it is this library, or want me to look at some code, let me know :)

Thank you @bfeher, you are best programmer! I removed custom UITabBar and works fine. Also works if I will increasing custom ui tabbar height if device model iphone X.

 I'm glad you could solve it!


Update README.md with shorter clone command and improved instruction order

Supersedes #5978 and improves the order of instructions such that Node.js must be installed first (with dependencies) before telling the user to start a web server.

Looks good, thanks!


Image Submit doesn't work on first attempt

I've just completed the tutorial and noticed that nothing happens the first time I submit an image. However, once a "?#" is appended to the url and I attempt to submit an image a second time, it comes through fine (the second image appears, the first one seems to be lost/never makes it to the database).
https://chat-app-c3ea7.firebaseapp.com/

Hey @itsclarke I wanted to test now, sorry for the delay but it seems the app is now not working (looks like you were working on adding notifications in there). Let me know once you have a working app again and I'll be happy to have a look.

Seems the issue is not in FF but it is still appearing on Chrome.
Hello @nicolasgarnier
To reproduce:

Go to: https://chat-app-c3ea7.firebaseapp.com/ (Ingnore missing styles)
Click on the Image button on the right side of the page.
See the URL ("?#" will be added)

Test on FF and Chrome.

Fixed!


Transparent taskbar glitches

Describe the bug
Transparency of taskbar only glitches on second monitor when main monitor is being used/has tabs open. Transparency seems like it is constantly activating and deactivating very quickly
Desktop

OS: [e.g. Windows 10]
Applications also used: Wallpaper Engine

To be able to focus on major/current bugs I had to close all issues. The issues section has gotten very messy to a point that I couldn't follow it anymore. Most issues are duplicates, very old or are already fixed/solved. Feel free to reopen this issue if you still have this issue on the latest version of TaskbarX to get my attention. All feature requests have been put in a list here: 💡| Feature requests #870
I hope you understand ❤


Can you change the fake chat response text when testing?

It doesn't seem possible to fake a Chat response with custom output from what I can understand.
For example if we follow the docs to create a fake json response.
use OpenAI\Laravel\Facades\OpenAI;
use OpenAI\Responses\Chat\CreateResponse;
use OpenAI\Responses\Chat\CreateResponseChoice;

OpenAI::fake([
    CreateResponse::fake([
        'choices' => [
            [
                CreateResponseChoice::from([
                    'index' => 0,
                    'message' => [
                        'role' => 'assistant',
                        'content' => json_encode([
                            'Fruits' => [
                                'banana',
                                'guava',
                                'tangerine',
                                'dragon fruit'
                            ]
                        ]),
                    ],
                ]),
            ],
        ],
    ]),
]);

For example this will always return a OpenAI\Responses\Chat\CreateResponseMessage with content of Hello there, this is a fake chat response.

I have managed to sidestep this issue by loading all the attributes from the fixture that the fakeable uses and changing the bit I wanted.
It seems you need to match exactly, so this seems to work, using the base client fixture https://github.com/openai-php/client/blob/main/src/Testing/Responses/Fixtures/Chat/CreateResponseFixture.php
use OpenAI\Responses\Chat\CreateResponse;
use OpenAI\Testing\Responses\Fixtures\Chat\CreateResponseFixture;

$fakeOverrides = CreateResponseFixture::ATTRIBUTES;
$fakeOverrides['choices'][0]['message']['content'] = $jsonResponse;

$response = CreateResponse::fake($fakeOverrides);


out of order handling of tcp queries

Right now this is serial because: https://github.com/miekg/dns/blob/master/server.go#L567
If we put go  in front of that it is out of order.
However I'm not fully convinced the response.Writer deals ok with this; should there be a mutex there?

A simple diff works:
diff --git a/server.go b/server.go
index 685753f..8248508 100644
--- a/server.go
+++ b/server.go
@@ -564,7 +564,7 @@ Redo:
                        w.tsigRequestMAC = req.Extra[len(req.Extra)-1].(*TSIG).MAC
                }
        }
-       h.ServeDNS(w, req) // Writes back to the client
+       go h.ServeDNS(w, req) // Writes back to the client

 Exit:
        if w.tcp == nil {
@@ -712,7 +712,6 @@ func (w *response) Close() error {
        // Can't close the udp conn, as that is actually the listener.
        if w.tcp != nil {
                e := w.tcp.Close()
-               w.tcp = nil
                return e
        }
        return nil

but shows up in a test failure in CoreDNS (data-race on axfr), on setting specific dns.ResponseWriter fields; so that thing would indeed need a mutex, also for writing the reply back to the client.
Now this only comes up when using TCP and DNS over TLS and responding to out-of-order request, for UDP we dont' care.

@miekg This change is far more complex than adding a go  and a mutex. I can take a closer look in the morning.
(Has this come up as a bottleneck somewhere?)

[ Quoting<EMAIL_ADDRESS>in "Re: [miekg/dns] out of order handli..." ]

@miekg This change is far more complex than adding a go  and a mutex. I can take a closer look in the morning.
(Has this come up as a bottleneck somewhere?)

No, not directly, but with DNS-over-TLS this might speed up things. I'm pretty
worried that making it faster for the DNS-over-TLS will make it slower for UDP.
Thanks for taking a look!

@miekg I took a look at rewriting the serve method with the intention of allowing this change.
It's looking like it's impossible to make this work with the Hijack method. I'm not sure how useful the Hijack method is in general, or what the intention behind it was, but I doubt it's possible to remove it while keeping it backward compatible.

Can't be done in current architecture; also DoT is quite bad for performance etc. etc.
Closing.


The viterbi decoding implementation may be wrong

I look into the viterbi decoding implementation and find that when backtracing the best path, argmax is done based on the current emission probability + previous score + transition probability. If my understanding is correct, the argmax should be done only based on previous score + transition probability, which is also suggested by wikipedia and the official crf implementation crf. Please clarify this.

Hi @jiqiujia , it is not wrong. This implementation does not affect the final result. During the argmax,  the emission score for each tag is the same, so it does not affect the final result. It is just a different implementation.
You can also find this comment at the official CRF implementation
# We don't include the emission scores here because the max
# does not depend on them (we add them in below)


Remove Wrapper and inner() to allow package-private inner classes

Blocked as of 7/1/16 by models in network & compute that don't have fluent yet


Buggy version of Time 5 doesn't fail any tests

Hi,
The buggy version of Time 5 defect doesn't fail any tests.
File trigger_tests for Time 5 is blank.

Thank you for reporting this. It looks like there was an issue when we regenerated the metadata for Java 8 recently. I'll fix this as soon as possible.

Hi @manishmotwani3 - The latest commit fixes this issue. Thanks again for pointing this bug out.


Update README.md and create directory for docs and images

Adds logo to README
Adds go score card
Fixes capitalization and punctuation
Typos
Adds additional description to about section
Reorganizes table of contents
Rewords roadmap description
Adds directory for docs and images

Merging this for ControlPlane Zero Day at CNSecCon — I'll forward the Kubesec logo designer @wakeward and we can redesign the logo for wider public consumption :blush:


[Figma] Create Modal design doc

Create design doc for Modal

Include:

When to use
Usage Guideline
Best Practices
Accessibility
Ext / Mob links

Publish to Notion design doc
https://southern-bush-399.notion.site/Metamask-Design-System-f86ecc914d6b4eb6873a122b83c12940

Hey team! Please add your planning poker estimate with Zenhub @Akatori-Design @brianacnguyen @garrettbear

Added what I have so far to Notion & Figma
https://www.notion.so/Modal-1c358986fac74bdd84494473f992256f?pvs=4


VBA ComboBox display error

The VBA ActiveX Object ComboBox is not displaying what's written with the keyboard. However it reacts at what's typed.
I have used the DropDown.bas and programmed it on an Excel spreadsheet (with two Named range) to provide two types of search cells in two different columns (the search is dependent of the named range associated with the cell).
With the leftmost cell it works fine, but the other one stop the display. They're totally identical though.

A solution might be to create a ComboBox per defined range instead of deducing the defined range with the Target and assign it to the ComboBox ListFillRange automatically.
But it does not soung like a good way to really solve the problem.

I don't think I zill fix it so I will close it :wink:


Tldraw Not Opening as soon you embed

if you use the command: Create New drawings and embed as attachment. its leaves a empty frame on the notes unless you toggle on the setting:  (show background or show background dotted pattern). then you will see the actual embed. there is no way to open the drawing unless you find the links on either outgoing links or backlinks. i found a bit of hack of opening the drawings directly from the notes without using the outgoing links. you have to long click from ipad or right click on windows on the embeds itself. then it opens small dialog that says "Edit drawings (new tab)".
User like my self would appreciate a quality of life improving of allowing Tldraw to automatically open as soon we embed it as attachment. or have option to click the  link or click the embedded frame to open Tldraw. honestly i prefer not having the show background option on as i like my embeds with not background so they match my notes seamlessly.
Thank you for making this amazing plugin
see attached for your reference
With Background On

with no background - with dialog open

Good catch, here are some solutions to this I can think of implementing:

If it is a new drawing, then the embed should display an edit button
Add variations to the "Create New drawing and embed as attachment" command

"Create New drawing and embed as attachment (edit in new tab)"
"Create New drawing and embed as attachment (edit in new tab - split horizontal)"
"Create New drawing and embed as attachment (edit in new tab - split vertically)"

Let me know what you think!

Thats amazing Thanks you so much - Will the edit button disappear? it shouldn't interfere with drawings it self. I think excalidraw solved this as soon you embed it opens automatically. Also you can double click on the drawing to open it. if you can implement that. That will be awesome

Thats amazing Thanks you so much

No problem :slightly_smiling_face:

Will the edit button disappear?

Yes that is the plan :sunglasses:

Also you can double click on the drawing to open it. if you can implement that. That will be awesome

Will do :+1:

Thank you

@MohamedKelif Sorry it took me a while to get back to you on this issue, I have a video clip to present:
https://github.com/user-attachments/assets/e98bc2e6-c313-4b56-aaed-7540980a2e81
If you think this is acceptable I will create a release for it. One thing to note, I decided to display a message instead of a button since double clicking the embed will open the editor.

https://github.com/jon-dez/tldraw-in-obsidian/releases/tag/1.13.0

Hi there! Thanks a bunch for getting back to me so quickly. You're doing amazing work, and I really appreciate all your efforts.
I went ahead and tested it on three devices: Windows, iPhone, and iPad. Here's what I found:
For Windows Machine

Embedding the file works like a charm and opens up super fast! It would be awesome if we could skip the 'confirm page' since I've already set the location in the plugin settings. (Maybe something for the future?)
Reopening files works great too, and the double-click feature is spot on for Windows.

For iPad and iPhone

Embedding the file usually works well, but sometimes I get a "Failed to Open" message.
Reopening doesn't work unless you tap a few times. It's not a bug, just that iOS and iPadOS don't seem to have a double-click gesture. I even tried using my Apple Pencil, but no luck there.

Maybe a long-press option for mobile and tablets could help? No rush, though; we can always use long-press to bring up the dialog and edit. I'm not sure about Android tablets, but others might be able to test this out.
A Few Other Bugs I Found (Off-topic)
I think someone mentioned this in issue #66, but just wanted to let you know:

There are random streak lines when writing with the Apple Pencil. It might be an iPad thing, not sure.
The size and pen type keep going back to the original settings, no matter what you choose.

New Issue I Noticed
I can open a new issue if you'd like, just let me know. But here's a quick mention:
I noticed the embed size is a bit small. I tried following your "Documentation for Embeds," but it was a bit tricky for someone without a coding background. What worked for me was using ![[<tldraw_markdown_file>|<embed_size>]]. For example, ![[<tldraw_markdown_file>|<800x1000>]] made the size bigger, and that did the trick. I didn't add "size =" because changes weren't showing up for some reason.
I also tried debugging by toggling the background and dotted background. The dotted background stretches to the document's width, but the actual background is a bit short. I think it might have to do with the canvas position, but I haven't figured it out yet.
Maybe adding a sizing option to the plugin settings could help, so we can tweak it without messing with Obsidian modifiers.
Thanks so much for all your hard work!

Thank you for the kind words :slightly_smiling_face:
You can disable the confirmation pop-up in the settings, I think this is what you are looking for:

For iPad and iPhone, well really for devices that support touch gestures, the tap and hold gesture brings up the context menu for the embed. I think this is probably the best choice since it does allow for choosing to edit the drawing, plus other options that relate to the embed in the markdown document. I do think tough that it would make sense to add a double tap gesture for the embeds so that it navigates to an editor, so I'll keep that in mind.
The bugs that you mentioned:

The streak lines

I don't have an iPad, so I cannot test this. If possible, could you record a screen capture?

Size and pen type

Does this also occur in the editor on tldraw.com?
Also, just to double check what version of the plugin do you have installed?
The new issue that you notice:
I haven't been too focused on the sizing of embeds as much, but if you open a new issue I'll try to make time for it.

@MohamedKelif I created a new release that adds a new tool which assists with automatically setting the bounds, so you no longer have to set them manually.
https://github.com/jon-dez/tldraw-in-obsidian/releases/tag/1.14.0


Expiry input added

I have resolved all the merge conflicts and posted it as new PR
I have added the interface for getting Expiry month and year so that expired cards can be invalidated on that criteria also.
I'm working the logic of it...please do consider the PR till here, for now and assign suitable bounty points (just a reminder as it hasn't been labelled for a bounty.
Closes #44 , #77 and #78
Thank you

Go on and work on the logic ;)

Go on and work on the logic ;)

It's taking a lot of time as I'm a beginner in javascript.
I have made a function to generate current month and year. I'm working on equating the values with ones in input
If I cannot submit by the deadline, request you  consider till here and give suitable points.
Thank you

Yes, points will be alloted to you accordingly :) don't worry about that


TimeoutOverflowWarning when using @effect/cli with a file argument

What version of Effect is running?
2.0.3
What steps can reproduce the bug?
When using @effect/cli with a file argument, node prints:
(node:336148) TimeoutOverflowWarning:<PHONE_NUMBER>740991 does not fit into a 32-bit signed integer.
Timeout duration was set to 1.

repro with more information: https://github.com/leonitousconforti/effect-TimeoutOverflowWarning-repro
What is the expected behavior?
For the timeout to not overflow? It doesn't seem to cause any noticeable issues for me, the cli still runs and completes though
What do you see instead?
A warning about a timeout overflow from effect runMain
 (node:6365) TimeoutOverflowWarning:<PHONE_NUMBER>740991 does not fit into a 32-bit signed integer.
 Timeout duration was set to 1.
     at new Timeout (node:internal/timers:169:17)
     at setInterval (node:timers:234:19)
     at runMain (/workspaces/effect-TimeoutOverflowWarning-repro/node_modules/.pnpm/@effect+platform-node@0.41.0_@effect+platform@0.41.0_effect@2.0.3/node_modules/@effect/platform-node/src/internal/runtime.ts:10:21)
     at pipeArguments (/workspaces/effect-TimeoutOverflowWarning-repro/node_modules/.pnpm/effect@2.0.3/node_modules/effect/src/Pipeable.ts:295:21)
     at pipe (/workspaces/effect-TimeoutOverflowWarning-repro/node_modules/.pnpm/effect@2.0.3/node_modules/effect/src/internal/core.ts:164:25)
     at <anonymous> (/workspaces/effect-TimeoutOverflowWarning-repro/bad.ts:19:50)
     at Object.<anonymous> (/workspaces/effect-TimeoutOverflowWarning-repro/bad.ts:22:1)
     at Module._compile (node:internal/modules/cjs/loader:1376:14)
     at Object.S (/workspaces/effect-TimeoutOverflowWarning-repro/node_modules/.pnpm/tsx@4.7.0/node_modules/tsx/dist/cjs/index.cjs:1:1292)
     at Module.load (node:internal/modules/cjs/loader:1207:32)

Additional information
repro with more information: https://github.com/leonitousconforti/effect-TimeoutOverflowWarning-repro

ooops, thought I had searched discord and github for relevant terms well enough but I just came across https://github.com/Effect-TS/effect/pull/1905 when searching other places for MAX_SAFE_INTEGER


Update config.guess and config.sub files

The existing files failed to recognize loongarch64 machines. This patch pulls the latest config.guess and config.sub files from git.savannah.gnu.org. This brings in loongarch64 support.

Ah, missed that this was necessary.


[feature] - Add update apps for Truenas Scale

Proposed change

Add update apps for Truenas Scale
Type of change

[ ] Bugfix
[X] New feature
[ ] Code quality improvements to existing code or addition of tests
[ ] Documentation

Additional information

Checklist

[X] The code change is tested and works locally.
[X] The code has been formatted using Black.
[X] Tests have been added to verify that the new code works.
[X] Documentation added/updated if required.

Very good PR, just few small changes are necessery.


Donor page login redirects to a page with no component

We have to wait until the Donor login is resolved. (I think it would automatically fix this issue.)

https://github.com/LSFLK/MedicinesforLK/issues/183


Android - Gradle Sync Issue - Duplicate module in IML file

You want to:  Ask a question
Hello Guys,
While installing react-native-firebase and react-native-async-storage, I encountered the below issue.
Currently, I'm deleting each duplicate module manually; but this is not ideal.
Can you please suggest me what is the best way to fix this ?
Details:
ERROR: The modules ['async-storage-android-@react-native-community_async-storage', 'react-native-community_async-storage'] point to the same directory in the file system.
Each module must have a unique path.

Hey @avinashlng1080
What RN version are you using? I suspect that the AsyncStorage might be linked two times - manually and by auto-linking feature.

Hello @Krizzu ,
I'm on "react-native": "0.61.4",.

How does your MainApplication.java and settings.gradle look like?

Same problem, anyone solved it?

It finally solved out by installing the latest android studio, updating the gradle build plugin to 3.6.0.
Optionally, I also ran npx react-native link packageName and remove the imported package from the mainapplication.java file only while leaving the path inside the setting.grDle file.

It finally solved out by installing the latest android studio, updating the gradle build plugin to 3.6.0.

Thank you, updating Android Studio and gradle to 3.6.1 were fixed my problem


/api/user/{id} Endpoint Peculiar Behaviour

The groups schema for the /api/user/{id} endpoint stipulates that the groups property value is a dictionary with the group’s Id as the key and group obj as the value. This is true for the most part alas, not when the user is not enrolled in any groups.
If the user is enrolled in groups, the response looks like this:
{
	“groups”:  {
		“5213”: {...}
	}
}

If the user is not enrolled in any groups, it looks like:
{
	“groups”:  []
}

@armatronic

@Brahmah we'll be fixing this in the next two weeks: however, it won't be released until the next major Schoolbox release.
For what it's worth groups should be an array instead of an object: we'll fix that and update the docs to match.

Thanks, @armatronic!
Related: #86

Fixed, Thanks @tomcdonnell


Getting Error  xER41300

Hello, I Just installed the code following the instructions on your page. (git, composer and automatic installation). When trying to log into the page, I got the following error:
Server Error: Please contact site administrator at<EMAIL_ADDRESS>and relay this error - xER41300
I can still navigate from page to page but this error keep displaying.
Any idea why?
FYI: I am running a local  XAMPP on Windows 10 with php 7.2.7
PHP 5.6.3 would not run due to specific php7 code in some files ( authorizationhandler.php )

got the same issue, tho i had to use the manual installer (went fine) i can login and everything seams to work just fine tho. Documentation about xER41300?

According to what i can read, if email is not working, you get the error message?

am also getting same issue I am using the Automated installer

I faced same issue. Please check your credentials, especially spaces on first or last character. I had space in before first character.

well, even tho mail works for me now, i still have this issue persistent so im just gonna leave it commented out for now ^^


CAMEL-20776: Kinesis Consumer uses a separate thread to monitor shard changes

Description
Each time when camel kinesis connector requests records, it gets shard list first. This will increase E2E latency for each message. In most scenario, shard list won't change frequently. So, it's unnecessary to get shard list before every data request.
This PR adds a separate thread for Kinesis Consumer to monitor shard changes
Target

[x] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[x] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[x] I checked that each commit in the pull request has a meaningful subject line and body.

[x] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

Also we mostly use millis for values, so its better to use 10000 for that delay.

Resolved above comments. Please review again.

I would mark this new option as consumer,advanced as end users likely is not going to tweak this option as much

Sure, added "advanced", please review again. Thanks.


WIP: Fix FileNotFound when predicting gas stations

As assumed the error happens when we try to predict the price for a gas station where we don't have any data for it. @WGierke Do you have an idea how to best integrate the possibility that a prediction can return None, 0, nothing etc. without breaking the whole prediction and FixedPathGasStation logic?

I'm not quite sure about that. What's the plan if we don't have training data? Interpolate the prediction using the k-NN gas stations?

Just exclude it completely?

I don‘t think we have to incorporate gas stations with missing data into the price prediction along a route. In the case of predicting the price for a single gas station, the data would be provided.

@feeds Is this ready for reviewing? Then please remove the WIP label.


Fix spacetux

This mod currently suffers from "Could not find spacetux entry in zipfile to install"
This pull request updates the license and install stanza.

#rebuild


Cannot connect to HTTPS docker API environment

Issue
Error response from daemon: Client sent an HTTP request to an HTTPS server.
Expected behaviour
Should connect to docker API with https succesfully

@WaysonWei finally! thanks for looking into this 🙏
I had to mangle port :2376 on haproxy to listen on both http + https and redirect accordingly when a http request occurred (oddly it happened only on a handful of requests but enough to block frontend and make it unusable)
reference if anyone else comes across to similar problem https://discourse.haproxy.org/t/one-port-to-catch-http-and-https-requests-and-redirect-to-the-https-version/1703/5


Non standard elevation mechanism

Describe the bug
The elevation mechanism used (Elevate-Privileges SeTakeOwnershipPrivilege) is non standard, depend on the external module take-own.psm1, and is not compatible with Constrained Language Mode.
To Reproduce
Steps to reproduce the behavior:

Launch the script

Proposed solution
Use a generic elevation mechanism (such as RunAs):
$grp = [int](whoami /GROUPS | ForEach-Object { if ($_ -match '(.*)S-1-16-(.*)') { $matches[2] } } )
if (-Not ($grp -eq 12288)) {
  $list = "& $($myInvocation.MyCommand.Source) "
  $PSBoundParameters.Keys | ForEach-Object { $list += "-$($_)" }
  Start-Process powershell.exe -Verb RunAs -ArgumentList $list
  exit 0
}

Unfortunately the runas mechanism doesn't work in harden environments either.
We appreciate the notification. However our suggestion is to comment out those lines if you're working in constrained language mode.


[Fix][Zeta] Increase the log level when waitForJobComplete fails

Purpose of this pull request
Increase the log level when waitForJobComplete fails: info--->severe
Does this PR introduce any user-facing change?
no
How was this patch tested?
exists test
Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

Hi @xiaochen-zhou , please merge from dev to fix ci failed.

Hi @xiaochen-zhou , please merge from dev to fix ci failed.

done.


Delete .DS_Store file

removed .DS_Store file, as this should not be in git

manually removed .DS_store and added to .gitignore


Set expected Locale for cJSON and calibration blob

Fixes #883
Description of the changes:

Switching locale from process based to thread based and setting the locale based on the calibration raw data.

@avilleret can you verify this fix by chance? I was not able to install QT to confirm it actually fixed your issue.

I confirm that the simple program I posted here is now working with your PR.

I confirm that the simple program I posted here is now working with your PR.

Thank you very much!


[tool_crash] ProcessException: The directory name is invalid. Command: C:\flutter\bin\cache\dart-sdk\bin\pub.bat, OS error code: 267

Command
flutter create fing

Steps to Reproduce

...
...
...

Logs
ProcessException: The directory name is invalid.
Command: C:\flutter\bin\cache\dart-sdk\bin\pub.bat, OS error code: 267
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:390:33)
#1      Process.start (dart:io-patch/process_patch.dart:36:20)
#2      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:39:20)
#3      ErrorHandlingProcessManager.start.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:638:33)
#4      _run (package:flutter_tools/src/base/error_handling_io.dart:532:20)
#5      ErrorHandlingProcessManager.start (package:flutter_tools/src/base/error_handling_io.dart:638:12)
#6      _DefaultProcessUtils.start (package:flutter_tools/src/base/process.dart:472:28)
#7      _DefaultProcessUtils.stream (package:flutter_tools/src/base/process.dart:491:35)
#8      _DefaultPub.batch (package:flutter_tools/src/dart/pub.dart:282:34)
<asynchronous suspension>
#9      _DefaultPub.get (package:flutter_tools/src/dart/pub.dart:222:7)
<asynchronous suspension>
#10     PubDependencies.update (package:flutter_tools/src/cache.dart:774:5)
<asynchronous suspension>
#11     Cache.updateAll (package:flutter_tools/src/cache.dart:562:9)
<asynchronous suspension>
#12     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1109:7)
<asynchronous suspension>
#13     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1009:27)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     AppContext.run (package:flutter_tools/src/base/context.dart:149:12)
<asynchronous suspension>
#16     CommandRunner.runCommand (package:args/command_runner.dart:197:13)

[✓] Flutter (Channel stable, 2.0.5, on Microsoft Windows [Version 10.0.14393], locale en-GB)
    • Flutter version 2.0.5 at C:\flutter
    • Framework revision adc687823a (11 days ago), 2021-04-16 09:40:20 -0700
    • Engine revision b09f014e96
    • Dart version 2.12.3

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\hp\AppData\Local\Android\Sdk
    • Platform android-30, build-tools 29.0.2
    • ANDROID_HOME = C:\Users\hp\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] IntelliJ IDEA Community Edition (version 2019.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2019.3.4
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.55.2)
    • VS Code at C:\Users\hp\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.21.0

[✓] Connected device (2 available)
    • moto e6 (mobile) • ZY326SKJWG • android-arm    • Android 9 (API 28)
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 90.0.4430.85

• No issues found!

Flutter Application Metadata
No pubspec in working directory.

Closing this as duplicate of https://github.com/flutter/flutter/issues/80418


for sql writer, template/data/examples

Great work!
Can you update the readme and add a new task there?

Great work! Can you update the readme and add a new task there?

New commit added to the readme, I also added the example ipynb file to the colab and provided a link there


About computing optimality

Hi, Thanks for sharing this awesome code. I am wondering how can I compute the optimality reported in the paper.
I guess I should read the graph info pickle file to get the optimality answer and compute w.r.t to them?
It would be really helpful if you could share the related code.
Thanks!

Hi,
In short, yes, the 'optimum' solution for every graph (i.e. the best result any methods, including ECO-DQN and all baselines, as detailed in the supplementary material of the paper) are provided in the graphs/*/opts/ folder(s) and the approximation ratios reported in the paper are calculated with respect to these values.
Loading them is quite simple.  Here is an example script to load the list of optimum values for all 100 instances in the validation set for 100 vertex BA graphs.
import pickle

sols_pth = "_graphs/validation/opts/cuts_BA_100spin_m4_100graphs.pkl"
with open(sol_pth, 'rb') as f:
    sols = pickle.load(f)

print(sols)

@tomdbar Thanks for your reply. I tried to compute the optimality, but I found it is hard to reproduce the paper results.
For the test function, here is the parameters I used:

And for computing optimality:

It would be really helpful if you could help me reproduce the results.
Thanks!


Issues observed during the workshop

I'll edit this as we go along

Suggest introduce targets earlier, as it the material on syntax and system introspection makes more sense if students aren't being distracted by PRIVATE etc.
Graphic for https://enccs.github.io/cmake-workshop/hello-cmake/#what-is-cmake would be easier to understand if the "pages" that corresponding to source files were named also with words, and the cylinder was labelled as a binary executable.
users are not experts at unpacking tarballs, so we need to give them a command to unpack them. Using .zip is probably more friendly for new programmers than .tar.gz or .tar.bz2, as they may have seen those in other contexts
text should mention that CMakeLists.txt should be stored in git, but not the makefiles it generates
dereference operator is formally ${} but that's not how it looks when used, so an example would help clarify.
the first exercise set BUILD_SHARED_LIBS on the command-line, which might have been something a student would ask about. (Suggest we don't set it at all there, as it brings in questions about the cache and variables that haven't yet been covered in the syntax lesson and is a topic itself much later.) Then the second exercise said "Define the BUILD_SHARED_LIBS variable" but didn't say whether to put it on the cmake command line or in the CMakeLists.txt file. Perhaps all of this makes more sense if we teach the students to manage setting variables using a non-standard name, so that questions of over-writing the cache do not apply.
we should number the exercises in the sphinx lesson and use matching numbers in downloaded files and the subdirectories they contain
the dependent-option exercise solution can make a library of the same name in both STATIC and SHARED. That causes an error. We should rather expose a variable that the user can use to toggle between them, rather than toggle both of them on or off. See #20
"the build tree" is a sub-heading of "flow control" and probably should not be
ctest operates at a higher level than a unit-test framework like Catch2 or GoogleTest, or your own binary, or some python script, etc. Perhaps a diagram is good for that.

Exercise 16

exercise numbers need to appear also in the sphinx lesson titles, like "Exercise 16: Configure a file"
exercise 16 has too many variables to configure in config.h. A few values and a string is enough.
exercise 16 sets e.g. _HAS_FPU from querying cmake_host_system_information for HAS_FPU and then uses it in config.h to set preprocessor variable HAS_FPU. There's no need to introduce underscore prefixing.
teach where config.h will be written before students have to use that knowledge
provide URL links to docs for configure_file, target_include_directories as well as command lines to query for the offline docs

all exercises should be numbered to make them easier to find

It would be useful to go through the solution after listeners are back from BO rooms

maybe give more time for the first exercise

Ex. 21: the questions below the source tree should be connected to a specific folder. Though it may be only relevant to the simplified version of the exercise that wee have got:)

I've "unpacked" the list into separate issues. Some of them were fixed during the workshop already.


AttributeError: module 'towhee' has no attribute 'glob'

run this file "1_text_video_retrieval_engine.ipynb" get the error:
AttributeError: module 'towhee' has no attribute 'glob'.
And this issue https://github.com/towhee-io/towhee/issues/2504 can not help me,why not give the correct version,it makes users hard to execute scucess.

We have updated the example, try again.

Reopen if the issue persists in the latest version.


Process GROW NOM data

Process GROW NOM data.
Challenge is mapping ESS-DIVE to EMSL raw file.

@corilo do you have any updates on this issue?  I am trying to close out the sprint on time since I'll be off next week.  Thanks for your help.

Removing from sprint. @corilo please let me know if you plan to work on this in May.

I think this can be closed. NOM GROW data was processed for GSP 2023. @corilo  @mslarae13 please reopen if this is incorrect.


feature(wallet-mobile): portfolio UI

Description / Change(s) / Related issue(s)
Ticket
YOMO-

[!NOTE]
Please create the ticket if missing it.

@lephuochoai there is a flickering while scrolling up token details.


AccessKit Node Re-export is Confusing with Bevy's Node

Bevy version
main/0.15 RC
Description
The bevy_a11y re-exports AccessKit's Node, but that can be confusing since bevy_ui also defines a Node. Since #16234, there is now a way to use AccessKit without importing accesskit::Node. Now that it is optional, we should reconsider the way Bevy re-exports AccessKit. For example, only re-exporting Role.
Context
This was raised by @alice-i-cecile in this Discord message.

I think we should not re-export at all here, and state in the docs that the versions must match.


Pass binary data through rest proxy.

This PR:

Assures that BinaryDataContent type doesn't change when traveling via rest proxy from generated APIs.
This is to enable memory savings for async clients.

API change check
API changes are not detected in this pull request.