Support for tmux?

Hi, I am using tmux. The left pane is my vim pane writing python, the right is ipython pane. When I use term-image's image.show() funtion. A very vague image printed in the right pane. how to make it more sharp? thx

Hi when  I use term-image command to open a image file in iTerm2, a high-resolution image appears. But when I type tmux into a tmux session, the image turns into vague. Why? Is there any solution? thx

Hello!
Thanks for the suggestion.

A very vague image

Well... that's because the image is represented using Unicode blocks with ANSI color escape sequences. In other words, what you're seeing is basically colored text.
This is called the "block" render style within the context of this project.

how to make it more sharp?

Well... Tmux itself doesn't support any graphics protocol, cos it just technically can't but there might be a way forward, I'll get to that in a moment.

But when I type tmux into a tmux session, the image turns into vague. Why?

The iTerm2 inline image protocol is implemented by the iTerm2 terminal emulator (the GUI program whose icon shows on your dock).
Tmux is a terminal multiplexer... basically, that means a program that provides and manages multiple terminal sessions within another terminal emulator.
Tmux intercepts escape sequences interpretes them and decides if and how they should be passed on to the actual terminal emulator.
Now, the problem is Tmux doesn't implement/support the image protocol i.e it can not understand the escape sequences used by the image protocol.
By default, term_image automatically detects the best image render style (or protocol) supported within the environment it is running.
So, since the iTerm2 protocol is not supported within Tmux, it uses the block render style which is supported within Tmux.

Is there any solution?

Well... more of a workaround.
Tmux provides a means by which escape sequences can be passed directly to the terminal emulator... called the Tmux passthrough escape sequence.
I've known about this for a while and had it in mind to add support for this but I've been reluctant to do this because it's a "workaround" meaning many things won't work as expected... e.g images won't scroll along with text as they normally would within the actual terminal emulator, amongst other issues.
The best option you have (and my advice) is to use use the terminal emulator directly and I'll recommend Kitty, it probably has most of the features of Tmux you need.

Please note that I'm not promising support for Tmux passthrough, I'm still considering it.
Thank you!

Hi when I use term-image command to open a image file in iTerm2, a high-resolution image appears. But when I type tmux into a tmux session, the image turns into vague. Why? Is there any solution? thx

Hi! Thanks a lot for your very detailed reply and instruction. After one day's work, I turn from tmux to kitty. And indeed the image appears very clear (high-resolution)!  Thx!!!
However, I found term-image is the only tool which can show image in tmux pane to my best knowledge despite its vague feature. But if the support is available in future version of term-image. I think it's very promising tool for wide python coders using tmux.

You're welcome :smiley:
I'll look more into and actually test it out soon i.e support for Tmux passthrough.
If I do add support at some point in the future, I'll simply note the quirks and maybe add some tips on how to best use it without much issues.
Once again, thanks for the suggestion.

By the wsy, I'll add support for matplotlib backend and IPython magic soon.
I just thought I should let you know cos they might be useful to you. :smiley:

Hi thanks for your sincere help! I am very delighting to see these new features! I really like term-image, an awesome tool!
BTW, is there a solution for term-image to draw the object of matplotlib figure?
like
from term-image.image import from_mpl # ideal function
fig = plt.figure()
ax = fig.subplot()
ax.plot()

image = from_mpl(fig)
image.draw()

I know I can use fig.savefig() to a file, a use from file() to read it back and then draw this image()
But I think this feature may be more convenient for users.

I personally don't use matplotlib but I'll look around and get back to you.
In the mean time, you can simply save the image to a buffer instead of a file i.e
import io
from term_image.image import AutoImage

buf = io.BytesIO()
fig.savefig(buf)
image = AutoImage(Image.open(buf))

I don't think there'll be any significant performance difference but it'll help avoid writing to disk.

from term_image.image import AutoImage
fig.canvas.draw()
img = Image.frombytes('RGB',fig.canvas.get_width_height(),fig.canvas.tostring_rgb())
image = AutoImage(img)
img.draw()

Hi thank your for your reply. I found a solution that seems to be much faster than savefig as the answer said (2ms vs 35~40ms)
See:  This link

Yeah, thanks.
Beware though... only the AGG backend(s) implement(s) the tostring_rgb() method, so you probably want to create a new canvas first i.e
from matplotlib.backends.backend_agg import FigureCanvasAgg
canvas = FigureCanvasAgg(fig)

After spending some time going through the package's docs (and a little through the source code) yesterday... I found FigureCanvasAgg.buffer_rgba() to be the best option (based on the implementation).
I had not yet mentioned it because I'm yet to test it but I plan to use this in implementing a from_mpl_figure() helper method later.

Now, back to the issue at hand...
After reading through https://gitlab.com/gnachman/iterm2/issues/3898 and after some experimentation and thought... I've come to conclude that tmux support for any graphics protocol-based style just isn't workable.
One could make the image display, no much problem with that... but due to the fact that tmux is in no anyway aware of the image or it's size, the whole thing becomes literally useless and in some cases even problematic.
See also: https://github.com/kovidgoyal/kitty/issues/2457
A few amongst the problems I've realised are:

For graphics protocols wherein image cells are overwritten by text (e.g iTerm2), any screen redraw or update by tmux will make the image disappear.
For graphics protocols wherein image cells are not overwritten by text (e.g kitty), more output or scrolling in tmux will either make an image cover up text or make text overlay an image.
Images will never scroll along with tmux' output.
In a multi-window setup, images wider or taller than the window in which they were drawn can extend into neighbouring windows.

In short... Terminal multiplexers are a pain in the ass... if you need terminal graphics, simply use the terminal emulators that support them!
Thank you! :smiley:

Thanks a lot for your valuable advice! I will listen to you!


support finding dependencies of a variable

I think we discuss this a long time ago in slack @MikeInnes , by dispatching pullbacks or other contextual calls to only variables dependent on the result, it will at least eliminate some annoying errors for functions that does not involve global references etc.
but I'm not sure this is the right way to implement it. I guess you might have some comments. especially, I'm not sure this is right for branches.

This seems like a reasonable approach; it won't yet work for branches because ir[v] doesn't exist for block arguments. You instead have to enumerate each branch to the block and include all passed argument variables as dependencies.
It may be better to rework this to use a dataflow-based approach, which is basically like doing type inference. You walk forward over all variables keeping track of the set of dependencies (which is easy to calculate as the union of dependencies of variables in the Expr). When you get to the end of the block, you check each branch to see if its dependencies have changed, and if so add the target block to a work queue. That will correctly handle things like loops where dependencies can be complex.
It might be useful to have a little tutorial on doing data flow analyses in the docs, since this is a good approach to almost every non-trivial task on IR.

Thanks! @MikeInnes by dataflow-based approach do you mean something like Mjolnir? Or should I PR this feature into Mjolnir instead of IRTools?

No, "data flow analysis" is just a general approach to writing algorithms on IR like this; see for example https://en.wikipedia.org/wiki/Data-flow_analysis. The clearest example in IRTools is probably this function which turns all implicit inter-block variable usage into explicit block arguments. So I'm suggesting that you rework the structure of your algorithm to do a dataflow analysis in the way that I described above.

@MikeInnes I just rework this, should be ready for a review.

err... Not sure why this fails on 1.0

I think this is because the IR on 1.0 is different from later Julia versions due to different implementation of @info. So I mark that as broken.
@MikeInnes any chance to review this?

@MikeInnes bump


Add ET Releases to Collector models and recreate models

Continuation of /pull/645, this time recreating the models on each run of the load_et_products task.

These changes where approved by @juspence in pull/645.


Please confirm these before moving forward

[x] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[x] I have not been informed if the issue is resolved in a preview version of the winget client.

Category of the issue
Installation issue.
Brief description of your issue
DependenciesGui.exe was installed under the WinGet\Links directory but it cannot be invoked from there. I need to use the file found under WinGet\Packages directory.
Steps to reproduce

Install the package with winget install lucasg.Dependencies.
Check there is DependenciesGui.exe in the WinGet\Links directory.
Try to invoke the command from there.

Actual behavior
Nothing happens.
Expected behavior
Its GUI window should open.
Environment
v1.9.25200 の Windows パッケージ マネージャー
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.22631.4460
システム アーキテクチャ: X64
パッケージ: Microsoft.DesktopAppInstaller v1.24.25200.0

Screenshots and Logs
No response

Probably, ArchiveBinariesDependOnPath is required in the manifest, isn't it?


Installation error

I've installed Tidal and Supercollider following this tutorial: https://tidalcycles.org/index.php/Linux_installation
And vim-tidal following the README (manual install), when I launch Tidal I get:
➜  tidal
GHCi, version 8.2.2: http://www.haskell.org/ghc/  :? for help

<interactive>:8:26: error:
    Variable not in scope: cpsUtils' :: IO (a0, b0, c0)

<interactive>:10:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:10:29: error: Variable not in scope: getNow

<interactive>:11:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:11:29: error: Variable not in scope: getNow

<interactive>:12:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:12:29: error: Variable not in scope: getNow

<interactive>:13:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:13:29: error: Variable not in scope: getNow

<interactive>:14:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:14:29: error: Variable not in scope: getNow

<interactive>:15:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:15:29: error: Variable not in scope: getNow

<interactive>:16:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:16:29: error: Variable not in scope: getNow

<interactive>:17:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:17:29: error: Variable not in scope: getNow

<interactive>:18:12: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:18:29: error: Variable not in scope: getNow

<interactive>:19:14: error:
    Variable not in scope: superDirtSetters :: t0 -> IO (a0, b0)

<interactive>:19:31: error: Variable not in scope: getNow

<interactive>:21:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:21:25: error: Variable not in scope: getNow

<interactive>:22:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:22:25: error: Variable not in scope: getNow

<interactive>:23:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:23:25: error: Variable not in scope: getNow

<interactive>:24:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:24:25: error: Variable not in scope: getNow

<interactive>:25:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:25:25: error: Variable not in scope: getNow

<interactive>:26:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:26:25: error: Variable not in scope: getNow

<interactive>:27:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:27:25: error: Variable not in scope: getNow

<interactive>:28:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:28:25: error: Variable not in scope: getNow

<interactive>:29:13: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:29:25: error: Variable not in scope: getNow

<interactive>:30:15: error:
    Variable not in scope: dirtSetters :: t0 -> IO (a0, b0)

<interactive>:30:27: error: Variable not in scope: getNow

<interactive>:33:31: error:
    • Variable not in scope: c1 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:34: error:
    • Variable not in scope: c2 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:37: error:
    • Variable not in scope: c3 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:40: error:
    • Variable not in scope: c4 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:43: error:
    • Variable not in scope: c5 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:46: error:
    • Variable not in scope: c6 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:49: error:
    • Variable not in scope: c7 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:52: error:
    • Variable not in scope: c8 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:55: error:
    • Variable not in scope: c9 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF’ (imported from Sound.Tidal.Context),
        ‘cI’ (imported from Sound.Tidal.Context),
        ‘cP’ (imported from Sound.Tidal.Context)

<interactive>:33:58: error:
    • Variable not in scope: c10 :: Pattern a0 -> m b0
    • Perhaps you meant one of these:
        ‘cF0’ (imported from Sound.Tidal.Context),
        ‘cR0’ (imported from Sound.Tidal.Context),
        ‘cT0’ (imported from Sound.Tidal.Context)

<interactive>:33:62: error:
    Variable not in scope: d1 :: Pattern a0 -> m b0

<interactive>:33:65: error:
    Variable not in scope: d2 :: Pattern a0 -> m b0

<interactive>:33:68: error:
    Variable not in scope: d3 :: Pattern a0 -> m b0

<interactive>:33:71: error:
    Variable not in scope: d4 :: Pattern a0 -> m b0

<interactive>:33:74: error:
    Variable not in scope: d5 :: Pattern a0 -> m b0

<interactive>:33:77: error:
    • Variable not in scope: d6 :: Pattern a0 -> m b0
    • Perhaps you meant ‘Chords.m6’ (imported from Sound.Tidal.Chords)

<interactive>:33:80: error:
    Variable not in scope: d7 :: Pattern a0 -> m b0

<interactive>:33:83: error:
    Variable not in scope: d8 :: Pattern a0 -> m b0

<interactive>:33:86: error:
    • Variable not in scope: d9 :: Pattern a0 -> m b0
    • Perhaps you meant ‘Chords.m9’ (imported from Sound.Tidal.Chords)

<interactive>:33:89: error:
    Variable not in scope: d10 :: Pattern a0 -> m b0

<interactive>:34:17: error: Variable not in scope: hush :: m a0
Loaded GHCi configuration from /home/alex/Downloads/vim-tidal-0.3.3/bin/../Tidal.ghci
tidal>
Leaving GHCi.
➜  cabal install tidal
Resolving dependencies...
All the requested packages are already installed:
tidal-1.0.10

Similiar to #16, I have the proper version.

Hi @dubreuia can you try updating the plugin? I updated the boot script to match latest Tidal version.

It seems to be working now that Tidal 1.0.11 was just released, just run cabal update and cabal install tidal --reinstall. Feel free to reopen if you still have problems.


The new project also has the previous project setting

Repro steps:

Navigate to https://pxt.microbit.org/beta
Click "Slow-Mo" on the Simulator, or choose "More..."-->"High Contrast on"
Then delete the project

Results:  (the "Slow-Mo" and "High Contrast on" still checked)

By design, these flags are editor specific - not tied to the project.

Yes by design


NFT card broken

https://beta.kodadot.xyz/bsx/gallery/3134362679-21?page=1
this item seems to be breaking the nft cards

👋


Incorect distance

Some activities shows incorrect distance when move right slider and return it back at the end:
ActivityID:
17272075001

Only happens with the right slider:


Add detector name to the ROOT output

Partial fix of #132 (was doing it for something else, thought it'd be useful to pull request for e.g. hyperk/hk-BONSAI#2). I've added it to WCSimRootGeom, because that seems appropriate.
Standard validation: ks tests are 1

@tdealtry I think this is now solved with PR #198, right?

Yeah, it's resolved via #198


Fix update issue in internalGetUpdates

My bot sometimes hangs in network issue and doesn't update anymore!
I think it's because the callback function doesn't call
I just move the schedule out of the callback
https://github.com/mast/telegram-bot-api/commit/362b907862bbbc423c75b6046d0d216aeefa116e
And for easiest contributing with same editor config I added the editorconfig file
https://github.com/mast/telegram-bot-api/commit/cc5791f5168835ca3bc59417c5f1739b392438cf
And trim lines and change the EOL's

Please make new tage for v0.3.3 after merge
Thank you in advance

Close temporary for better fixing the the issue


Identifier '__vite__injectQuery' has already been declared

Describe the bug
I do not change anything related to Vite this error appears on my browser's console. It is now blocking my application to be loaded.
Reproduction
https://example.com
Steps to reproduce
npm run dev
System Info
There is only standard console output in VS Code

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

It was due to an import line from Vite library. Everything works as expected.


[WIP] Construct

Throws in some constructors to make it easier for downstream consumers.
Will test against https://github.com/ossf/allstar/pull/114.
cc: @jeffmendoza

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] PR title follows the guidelines defined in  https://github.com/ossf/scorecard/blob/main/CONTRIBUTING.md#pr-process

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:

Codecov Report

Merging #1645 (0e0ac01) into main (ba503c3) will increase coverage by 2.83%.
The diff coverage is 0.00%.

:exclamation: Current head 0e0ac01 differs from pull request most recent head 29f9cd8. Consider uploading reports for the commit 29f9cd8 to get more accurate results

@@            Coverage Diff             @@
##             main    #1645      +/-   ##
==========================================
+ Coverage   56.10%   58.94%   +2.83%
==========================================
  Files          72       72
  Lines        6495     6496       +1
==========================================
+ Hits         3644     3829     +185
+ Misses       2606     2416     -190
- Partials      245      251       +6


Environment not being properly activated

See the run here. It cannot find pip from conda on the macOS runner and uses the system one, even though it is explicitly listed in the deps in environment.yml.
The config for this action is as follows:
- uses: conda-incubator/setup-miniconda@v2
  with:
    activate-environment: sparse-dev
    allow-softlinks: true
    environment-file: ci/environment.yml
    python-version: ${{ matrix.python }}
    miniforge-variant: Mambaforge
    use-mamba: true

Would it work if you use python -m pip instead of pip directly?

It seems I needed to manually specify a shell, and that this is documented.

It seems I needed to manually specify a shell, and that this is documented.

Indeed, by default github actions do not use the bashrc/etc config, so we need to specify the shell: bash -l {0} you can specify this globally and not on every step. See https://github.com/conda-incubator/setup-miniconda#use-a-default-shell
Glad you find the solution @hameerabbasi !
Closing


Return entities as endpoints

Addresses #19

@paulmach Updated!

pings @paulmach for follow up. :smiley:


[MM-45686] Add display name to add members to channel modal

Summary
Add the display name of members in the add member to channel modal. The display name depends on the user's setting for teammate display style, e.g., show username or show full name. The behaviour and style matches with the existing channels members list.
Ticket Link
Jira ticket
Screenshots

before
after

For reference, the member list style:

Release Note
NONE

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

I have pushed a commit, so that now the username is only shown if the display name is distinct. Example:

Let me know if I should update the members list in this PR as well, or if we should fix that (along with inconsistencies in other areas) in a different PR.

https://mattermost-webapp-pr-11136.test.mattermost.cloud

I think we can update the members list in this PR!

I have pushed a commit, so that now the username is only shown if the display name is distinct. Example:
Let me know if I should update the members list in this PR as well, or if we should fix that (along with inconsistencies in other areas) in a different PR.

I think we can update the members list in this PR!

/update-branch

/e2e-test

Successfully triggered e2e testing!
https://git.internal.mattermost.com/qa/cypress-ui-automation/-/pipelines/247515


Made the navbar of home page responsive

the navbar was not responsive i have made it responsive and add some intresting styling to it and made some changes in page content

This issue is already solved.
Try to contribute to different issues.


Popupmenu is in the wrong place again (v8.0.1529)

Expected position (v8.0.1489):

In v8.0.1489, the popupmenu is adaptive to the cursor's position. The left edge of
popupmenu corresponds to the cursor.
Actuall position (v8.0.1529):

But in v8.0.1529, popupmenu's left edge is not in the same column of cursor. I need move my eyes away from the cursor position to the left side to see the completion candidates, which is not convenient.
So, can popupmenu show itself in the old way ?

						*'pumwidth'* *'pw'*
'pumwidth' 'pw'		number	(default 0)
			global
			{not available when compiled without the
			|+insert_expand| feature}
			{not in Vi}
	Determines the minium width to use for the popup menu for Insert mode
	completion.  When zero the default of 15 screen cells is used.
	|ins-completion-menu|.

https://github.com/vim/vim/issues/2314

I have set pumwidth to 0 or 8, and got the same result.

In my second screen capture, the actual width of the popup menu is definitely more than 100, no matter I set the pumwidth to zero or something else.

I don't care the minimum width of popupmenu, but I really care having to move eyes left and right repeatly and that makes me dizzy and became less productive.

I believe the problems lays with the maximum width of the popup menu. It does not make much sense to move it leftward past the completion point by 1000s of characters in the case of a STL completion.

I have just tested v8.0.1538 immediately, but it appeared that still not fixed on my linux box:

anybody can reproduce this in v8.0.1538 ?

Looks like this still happens if the entries in the popup menu are wider than the window.

Actually, it's when the space available is more then 'pumwidth'.

Thanks, it seems to be fixed

Linwei wrote:

Thanks, it seems to be fixed

Keep an eye open for other problems, I'm still not happy about the logic
of the code.  At least we now have tests to check for any regressions.
--
How do you know when you have run out of invisible ink?
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


pull request for e2e test at 12/30/2016 09:45:07

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Hints for starting certmgr in current user scope

Added troubleshooting in case people cannot find certificate manager in current user scope

@cherylmc - Would you review the proposed changes? Thanks

@fbeltrao - Sounds good. I'll get that merged in. Thanks!

#sign-off


Mount /sys/fs/cgroup into the container

Then mount all of the config directories specific to the container.
This pull request grabs some of the ideas from https://github.com/docker/libcontainer/pull/528 and mounts them at the standard location under /sys/fs/cgroup.
One of my goals is to get systemd to run within a container without having to to a -v /sys/fs/cgroup:/sys/fs/cgroup:ro.
Up til now I have not figured how to fool it without mounting the file system in.  The only problem I see with mounting it in is a leak of information about other processes on the system.
Docker-DCO-1.1-Signed-off-by: Dan Walsh<EMAIL_ADDRESS>(github: rhatdan)

A few useful links about systemd in CT:
https://lwn.net/Articles/624767/
http://www.freedesktop.org/wiki/Software/systemd/ContainerInterface/
http://lists.freedesktop.org/archives/systemd-devel/2015-February/028772.html

Yes I am very familiar with these articles/problems, and have talked to Lennart fairly regularly.

I think this should be closed by https://github.com/docker/libcontainer/pull/553

Yes although this does not satisfy the need for /sys/fs/cgroup mounted into systemd based containers.
I will have to work on separate patches for that.


Deploy new yaml file generation feature

Deploy without SLURM migration changes in first instance

My Plan for this task

Rollback Changes in dev and master:

Checkout to dev branch: git checkout dev.
Create a backup branch: git branch backup_dev.
git reset --hard 1ada7b9636b3c4c9b6ff73e9a17f389c9d4000ac to rollback to this commit https://github.com/EBISPOT/gwas-sumstats-service/commit/1ada7b9636b3c4c9b6ff73e9a17f389c9d4000ac
Repeat these steps for the master branch.

Merge Feature Branch to dev:

git checkout dev.
Merge feature branch: git merge new_yaml_from_master.

Test in sandbox:

Conduct our tests.

Merge dev to master:

Checkout to master: git checkout master.
Merge dev: git merge dev.

Briefly test in prod:

Make sure that everything is up and running

Merge Rolled Back Changes to dev:

Manually re-apply the changes in backup_dev.

My Plan for this task

[x] Rollback Changes in dev and master:

Checkout to dev branch: git checkout dev.
Create a backup branch: git branch backup_dev.
git reset --hard 80437fd6dff05c855063af597d27d568e229219c to rollback to this commit https://github.com/EBISPOT/gwas-sumstats-service/commit/80437fd6dff05c855063af597d27d568e229219c
Repeat these steps for the master branch.

[x] Cherry pick changes to dev:

git checkout dev.
Cherry pick: git cherry-pick 7847851 bb3ddb7.

[ ] Test in sandbox:

Conduct our tests.

[ ] Merge dev to master:

Checkout to master: git checkout master.
Merge dev: git merge dev.

[ ] Briefly test in prod:

Make sure that everything is up and running

[ ] Merge Rolled Back Changes to dev:

Manually re-apply the changes in backup_dev.

Test OK in sandbox.

Merged dev to master. https://github.com/EBISPOT/gwas-sumstats-service/pull/284

Created tag v1.3.9 https://github.com/EBISPOT/gwas-sumstats-service/releases/tag/v1.3.9

Published the release v1.3.9. https://github.com/EBISPOT/gwas-sumstats-service/releases/tag/v1.3.9

Released to prod now. https://gitlab.ebi.ac.uk/gwas/gwas-sumstats-service/-/pipelines/485458

I confirm that yaml files for the new submission are in the new format. I will suggest closing this ticket.


Missing Linux AArch64arm64 wheels on PyPi

Expected behavior and actual behavior.
Missing Linux AArch64 wheels on PyPi. The command: pip install fiona should be able to download the wheel from PyPi and install it.
Currently pip tries to build the wheel from source code and fails.
Steps to reproduce the problem.
On Linux AArch64, run: pip install fiona
Operating system
Linux
Fiona and GDAL version and provenance
Not applicable

Can you please let me know which CI/CD you are using to build and deploy the wheels. I found a repository but it look to be outdated now. Can you please point out the current wheel builder?
Or, you are building and uploading the wheels manually?

@sgillies, can you please let me know your interest and point out the wheel builder?
Building and uploading wheel on PyPi can be useful as -

The Arm architecture has been available on AWS (Graviton2) for about 2 years now and has seen a lot of adoption by various organizations. The install base of Arm users is growing at a rapid pace.
Oracle cloud will be releasing Arm based Compute Shapes in the future. Please see here
Generally, there is a lot of momentum behind various products switching to Arm based CPUs. For example, the Microsoft surface Pro X is an Arm based device.

@sgillies, can you please let me know your thoughts on releasing Linux AArch64 wheels? Thanks

I second this desire. Currently one cannot install fiona (and hence e.g. geopandas) on arm linux or M1 macs without using anaconda. I would very much like to see wheels for these two archs.

Fiona 1.8.21 wheels for x86_64 manylinux2014 and macos (10.10) only were built on GitHub actions using https://github.com/sgillies/fiona-wheels.
There are no current plans to build arm64 wheels for Linux or macOS. While I understand how great it would be to get software for free, that you don't even have to compile (better-than-free!), that you can run on, for example, fancy new AWS instances (not free), I believe arm64 wheels yield diminishing returns for the Fiona project. In other words, I don't see that this project would get much back in the way of bug fixes or other contributions in return for work in getting GDAL and all its dependencies to compile on what looks to me to be a production-only platform.
In the M1 (also not free!) case, we're mainly blocked by the lack of free CI with native M1 workers. Cross-compilation is, in theory, possible, and I've done it for the GEOS library, but there are a lot of unknowns for the deep stack of libraries that GDAL depends on.
For the time being: use packages from conda-forge. The conda-forge project's mission is to build packages. You can even sponsor the project with money to do it. The Fiona project is not currently on a mission to build packages for every available platform.


Adding -h to print usage, just to avoid the error message

Previously "./quickstarh.sh -h" would print an error about it being an invalid parameter and print the usage, now it'll just print the usage.
Also means in future it could have an expanded help if necessary.

LGTM. Thanks @nickdgriffin.
`$ ./quickstart.sh -h
.....
.....
Usage:
./quickstart.sh
-t local
[-m <MACHINE_NAME>]
[-u <INITIAL_ADMIN_USER>]
[-p <INITIAL_ADMIN_PASSWORD>]
    ./quickstart.sh
    -t aws
    -m <MACHINE_NAME>
    -c <AWS_VPC_ID>
    -r <AWS_DEFAULT_REGION>
    [-z <AVAILABILITY_ZONE_LETTER>]
    [-a <AWS_ACCESS_KEY>]
    [-s <AWS_SECRET_ACCESS_KEY>]
    [-u <INITIAL_ADMIN_USER>]
    [-p <INITIAL_ADMIN_PASSWORD>]

`


Creating a pull request sidebar view does not update when switching branches

Refs https://github.com/microsoft/vscode/issues/122245

Steps to Reproduce:

Open a repo that's on the default branch
Start creating a pull request
Switch branches
:bug: sidebar view "merge changes from" does not update

GH PR Extension Version: v2021.4.39423
Version: 1.56.0-insider (user setup)
Commit: 13f1afffab09548c10ddb3bc5d5db52587876ad8
Date: 2021-04-28T04:52:06.990Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

The same applies to marking a PR as "Ready for review" in the webview and the sidebar still shows a "Ready for review" button:


Check that header exists instead of contents of header in build iOS module test

Description
https://github.com/flutter/engine/pull/21193 changed the import.  Check that the header exists, instead of checking the contents.  This will avoid a manual engine roll.
Related Issues
https://github.com/flutter/flutter/issues/60025#issuecomment-693612293

Task result:
{
  "success": true,
  "data": null,
  "benchmarkScoreKeys": []
}

Linux hostonly_devicelab_tests was a Gradle download flake, it passed at https://ci.chromium.org/p/flutter/builders/try/Linux hostonly_devicelab_tests/2531


[Bug] Field colours are inconsistent and don't change in view or edit more

Description
Field colours are inconsistent on Google Sheets query page , i.e. cannot differentiate mandatory/optional fields and the background colour does not change between edit and view modes
Steps to reproduce the behaviour:

Create a new query for Google Sheets source and observe the variations in field colours on the page and also when you click on each field to edit and when field is in view mode - expectation is to follow designs on figma/other datasources
Mandatory fields like row offset and row limit are not marked with an asterisk symbol as seen on the Method field

Important Details

Version: Cloud
OS: Win10
Browser: Chrome
Environment: Production

This will get resolved with #10235

@prapullac adding to your queue since this should already be fixed now.

Missing * is still on release

Hey @yatinappsmith! Since this is a fix that is coming in with UQI, could you please test this on the deploy preview instead?


Debounce decorators applied to async functions do not work correctly

Lodash decorators do not work correctly with async functions.

Fixed


Azure deployment issues

Describe the bug
When publishing to Azure, a Terraform error message is displayed, which refers to the allocation method of the public IP.  A warning is also issued which refers to an deprecated attribute. And last but not least, the Azure tutorial is confusing as it shows an partly outdated screenshot and does not mention the necessary change of a mandatory variable.
To Reproduce
Steps to reproduce the behavior:

Select Azure deployment and prepare repo accordingly
Replace Terraform variables with your own values
execute "terraform plan"
See warning
execute "terraform apply"
See error

Expected behavior
It is expected that terraform plan and apply will be executed without warnings or errors.
Screenshots
Warning (after terraform plan):

IP error (after terraform apply):

Software versions (please complete the following information):

OS: Windows 11
Terraform version: 1.9.8
Plugin version: Azure cli 2.65.0, cml2 2.7.2

Additional context
Related PR: #24

thanks @kellermatthis for bringing this up.  In fact, I saw this myself last week when running some demo VM in Azure.  I haven't had time to address this on GH so your contribution is most welcome.
The use_cache thing is purely cosmetic -- I dreprectated that option in the last Terraform release.  But besides showing that warning, there's nothing to worry about.

Thanks for approving the pull request


feat(storybook): Added storybook with web-components

PR adds apollo-web-app along with storybook to work on visual language of graphql schema representation of iroha ledger.

Codecov Report

Merging #5 (22d9c31) into main (033cab2) will increase coverage by 6.34%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main       #5      +/-   ##
==========================================
+ Coverage   90.32%   96.66%   +6.34%
==========================================
  Files           8        5       -3
  Lines         186      150      -36
==========================================
- Hits          168      145      -23
+ Misses         18        5      -13

Impacted Files
Coverage Δ

playground/domain.py
100.00% <0.00%> (ø)

playground/schema.py
100.00% <0.00%> (ø)

playground/mutators.py

playground/concurrency.py

playground/resolvers.py

playground/iroha.py
91.22% <0.00%> (+9.08%)
:arrow_up:

Continue to review full report at Codecov.

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


When Before returns an error, it is displayed twice

Hey!
With the following cli.Command:
cli.Command{
		Name:        "stuff",
		Usage:       "Print stuff",
		ArgsUsage:   " ",
		Before: func(c *cli.Context) error {
			if c.Args().Present() {
				return cli.NewExitError("no additional arguments accepted", 1)
			}
			return nil
		},
		Action: func(c *cli.Context) error {
			fmt.Println("stuff")
		}
}

If I add an additional argument, I get:

the error text (good)
the usage (good)
the error text again (bad)

It's a bit difficult to fix:
		err = c.Before(context)
		if err != nil {
			fmt.Fprintln(ctx.App.Writer, err)
			fmt.Fprintln(ctx.App.Writer)
			ShowCommandHelp(ctx, c.Name)
			HandleExitCoder(err)
			return err
		}

We could tell HandleExitCoder() to not display the error, but this would change the interface.

In fact, fixing #534 to not display error nor usage would help in this case since it would enable me to override OsExiter and handle the error myself.

Hey, noticed this here as well; what's the point of that first println? 	fmt.Fprintln(ctx.App.Writer, err). I suppose what's in HandleExitCoder is already enough, and desired, right?
Thx!

Hey, in my case i'd actually expect to see the error after the usage (as it's easier to spot if there's a big usage). In that case it'd mean just removing that first println i'd say


Bug: Loud noise

This app is good but sometime causing disturbing loud noise. For example connect bluetooth headset then disconnect it then computer speaker is disturbing loud noise. I can't explain detailed the problem but sometimes computer speaker disturbing for a small time (may be 0.5 second). I fear that this app will break down my laptop's speaker...
Hardware: Macbook Pro late 2019 16 inch intel i9 16gb ram.
Operating system: Monterey 12.3.1

Hi, this might be related to #594. I am closing this issue due to the similarity, please comment on the that issue if you have any further information to add. If you believe this is an mistake, please reopen this issue and comment on it.


Jorwoods/query endpoint paging

Tweaks the QuerySet added by @scuml to enable automatic pagination through all results. @scuml - Was the original intent to have users explicitly paginate through results? Would handling the pagination implicitly within the __iter__ break anything?
Also adds the ability to use QuerySet operations on Groups and Flows.

Was the original intent to have users explicitly paginate through results?

Yes - this looks good.  Thanks for the improvement.

@bcantoni Seems the slack notifier is failing when I push on this PR. The error message is empty so I am not sure what is needed to resolve it.

This to me is a pretty important PR that should definitely be merged before the next release. Any use of the QuerySet methods without this fix will only yield the first 100 items, regardless of how many exist.

I think I'm missing something here - what does this add to the existing Pager/Pagination functionality? In the sample list.py, I can retrieve over 100 views already.

Mostly it’s syntactic sugar. This corrects an issue with the QuerySet introduced in #615 .
It makes filtering easier than building aRequestOptions object to pass in. Searching for a given user, for example, becomes:
server.users.filter(name=jorwoods’)

@jacalata I should clarify a little more. The QuerysetEndpoint that was introduced by #615 would only handle the first page. This PR fixes the QuerysetEndpoint so that it can work on more than the first page.


Improving segment replacement/revert protocol

Add retention for original segments for REFRESH table after
the replacement protocol is successfully done. For now, we
use 24 hrs retention. We keep the original data for 24 hours
so that the user can revert back to the original segments
using revertReplaceSegments API.
Add the proactive clean-up in startReplaceSegment so that we
do not keep too much data snapshots for REFRESH table. For
now, we allow to keep at most 2 snapshots.
Added unit tests.

#7813

Codecov Report

Merging #7995 (aef1a71) into master (7030260) will decrease coverage by 56.91%.
The diff coverage is 66.66%.

@@              Coverage Diff              @@
##             master    #7995       +/-   ##
=============================================
- Coverage     71.23%   14.31%   -56.92%
+ Complexity     4218       81     -4137
=============================================
  Files          1595     1551       -44
  Lines         82743    80906     -1837
  Branches      12347    12153      -194
=============================================
- Hits          58942    11585    -47357
- Misses        19813    68465    +48652
+ Partials       3988      856     -3132

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
14.31% <66.66%> (+0.05%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...troller/helix/core/retention/RetentionManager.java
79.36% <56.25%> (-3.55%)
:arrow_down:

...ntroller/helix/core/PinotHelixResourceManager.java
61.08% <81.81%> (-3.44%)
:arrow_down:

...ain/java/org/apache/pinot/core/data/table/Key.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/spi/utils/BooleanUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/data/table/Record.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../java/org/apache/pinot/core/util/GroupByUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/config/table/FSTType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ava/org/apache/pinot/spi/data/MetricFieldSpec.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/spi/utils/BigDecimalUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...java/org/apache/pinot/common/tier/TierFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1271 more

Continue to review full report at Codecov.

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

LGTM


Add plugins support

It allow serenity to support custom plugins (create a new folder plugins in serenity root)
Example plugin:
package.json:

main example:

Console:

Ingame:

It is s good start for a plugin system, there are a few things I would like to rework. Looks good!


Issue after hosting locally

I have created a custom demo based on documentation. When I host my files locally it is giving the following error "CancellationToken. Timeout!" and I'm not getting any search suggestions from web service. What could be the issue?

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

ID: 2b2e1157-134d-cf7c-61f9-e70482eba311
Version Independent ID: 81cad11f-3081-4fc8-463b-3bfd0b52b80e
Content: Web API interface in the Knowledge Exploration Service API
Content Source: articles/cognitive-services/KES/WebAPI.md
Service: cognitive-services
GitHub Login: @bojunehsu
Microsoft Alias: paulhsu

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

@rajashekar2012 Since running a custom demo locally falls outside the scope of this document, please share your question on MSDN forums or Stack Overflow.
Can you please share the URL to your forum post so that we can track it?

I have opened the same issue in StackOverflow...  link below
stackoverflow link

@rajashekar2012 Thank you. We are now tracking the issue. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Can't seem to find a way to set LogLevel

The docs for (I'm assuming the none-kotlin version), mention there is a LogLevel that can be set when initilaizing the sdk: https://segment.com/docs/connections/sources/catalog/libraries/mobile/android/#adding-debug-logging
I would like to disable logging - because when the phone is offline, the logs are being spammed because it cannot reach api.segment.io
Is there a way to do this that I am missing?

hi @compscidr, in analytics-kotlin, you can customize your own flush policy to make segment not flush when the device is offline. please take a look at these samples: UnmeteredFlushPolicy and other policies

Sure, but don't you think there should still be a way to control whether segment is logging or not? That was the original request here.

so you mean the debug logs that segment prints to logcat? by default, it is disabled, but you can toggle it on/off through Analytics.debugLogsEnabled (see this line). also, such logs are not sent to api.segment.io. that's why I was assuming you were talking about disabling event flushing.
if you meant to disable segment completely to not track any events, we do have an internal ticket to track this feature request. but currently it's not available.

I'm talking about logging to the android logcat logger - I'm not worried about the logs being sent to api.segment.io - more concerned with the debug logs, I'm trying to debug other things in my app but the logs are flooded with exceptions from segment, I would think with the debugLogsEnabled set to false, it wouldn't show these types of exceptions but it still seems to.

For example this gets spammed continually:
2023-05-22 09:24:53.634 12947-12998 System.err              xyz.bumpapp.dev.debug                W  java.net.UnknownHostException: Unable to resolve host "api.segment.io": No address associated with hostname
2023-05-22 09:24:53.634 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.net.Inet6AddressImpl.lookupHostByName(Inet6AddressImpl.java:156)
2023-05-22 09:24:53.634 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.net.Inet6AddressImpl.lookupAllHostAddr(Inet6AddressImpl.java:103)
2023-05-22 09:24:53.634 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.net.InetAddress.getAllByName(InetAddress.java:1152)
2023-05-22 09:24:53.634 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.Dns$1.lookup(Dns.java:41)
2023-05-22 09:24:53.634 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.RouteSelector.resetNextInetSocketAddress(RouteSelector.java:178)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.RouteSelector.nextProxy(RouteSelector.java:144)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.RouteSelector.next(RouteSelector.java:86)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.StreamAllocation.findConnection(StreamAllocation.java:176)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.StreamAllocation.findHealthyConnection(StreamAllocation.java:128)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.StreamAllocation.newStream(StreamAllocation.java:97)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.HttpEngine.connect(HttpEngine.java:289)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.http.HttpEngine.sendRequest(HttpEngine.java:232)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.huc.HttpURLConnectionImpl.execute(HttpURLConnectionImpl.java:465)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.huc.HttpURLConnectionImpl.connect(HttpURLConnectionImpl.java:131)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.huc.HttpURLConnectionImpl.getOutputStream(HttpURLConnectionImpl.java:262)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.huc.DelegatingHttpsURLConnection.getOutputStream(DelegatingHttpsURLConnection.java:219)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.android.okhttp.internal.huc.HttpsURLConnectionImpl.getOutputStream(HttpsURLConnectionImpl.java:30)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.segment.analytics.kotlin.core.HTTPClientKt.createPostConnection(HTTPClient.kt:93)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.segment.analytics.kotlin.core.HTTPClient.upload(HTTPClient.kt:32)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at com.segment.analytics.kotlin.core.platform.EventPipeline$upload$1.invokeSuspend(EventPipeline.kt:129)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1137)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:637)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.lang.Thread.run(Thread.java:1012)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  Caused by: android.system.GaiException: android_getaddrinfo failed: EAI_NODATA (No address associated with hostname)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at libcore.io.Linux.android_getaddrinfo(Native Method)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at libcore.io.ForwardingOs.android_getaddrinfo(ForwardingOs.java:133)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at libcore.io.BlockGuardOs.android_getaddrinfo(BlockGuardOs.java:222)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at libcore.io.ForwardingOs.android_getaddrinfo(ForwardingOs.java:133)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	at java.net.Inet6AddressImpl.lookupHostByName(Inet6AddressImpl.java:135)
2023-05-22 09:24:53.635 12947-12998 System.err              xyz.bumpapp.dev.debug                W  	... 24 more

I'm sure, as you said if I configure the flush policy and everything such that it doesn't try to flush it will solve it - but I'd expect that if debuglogs are disabled it would also not show up.

ah. I see. yeah. I do see a line that's print out the stack trace disrespect the segment log settings. we will get it removed. thanks for reporting this. in the meanwhile, please try the flush policy to avoid run into this issue.


🐞SPFx - Update package-solution.json's schema to include optional properties for webApiPermissionRequests

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
No response
What browser(s) / client(s) have you tested

[ ] 💥 Internet Explorer
[ ] 💥 Microsoft Edge
[ ] 💥 Google Chrome
[ ] 💥 FireFox
[ ] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[X] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

SPFx v1.15.2 projects

Describe the bug / error
The schema for ./config/package-solution.json, https://developer.microsoft.com/json-schemas/spfx-build/package-solution.schema.json, is missing optional properties on objects in the webApiPermissionRequests array introduced in SPFx v1.15.2: appId & replyUrl.
This means developers won't get IntelliSense with included docs when editing this file. In fact, the file will fail validation as it currently contains additionalProperties: false which will say the properties aren't allowed:

Steps to reproduce

Create new project

Use any type of component when prompted

Open said project in VSCode

Open ./config/package-solution.json

Add the following code after the developer property:
"webApiPermissionRequests": [
  {
    "scope": "", "resource": "", "appId": "asd", "replyUrl": "https://foo.com/endpoint"
  }
],

Developer curses at VSCode and 😭 in solitude... kittens rush for safety to avoid being kicked by said developer.

Expected behavior

Create new project

Use any type of component when prompted

Open said project in VSCode

Open ./config/package-solution.json

Add the following code after the developer property:
"webApiPermissionRequests": [
  {
    "scope": "", "resource": "", "appId": "asd", "replyUrl": "https://foo.com/endpoint"
  }
],

Observe IntelliSense when modifying values under the included webApiPermissionRequests

The Microsoft engineer who took the time to update this file, on their way home from work to enjoy a good streaming movie, when picking up their take-out tacos 🌮, comes to find out someone paid it forward as thanks and now they enjoy free 🌮 for their contribution to developers across the globe.

Thanks @andrewconnell for reporting it!
the issue is now fixed.


Docker Ip Address working in Linux

I see that you implements a ip localization. It will be work very fine in windows and osx but in linux isn't needed.
I try to use my localhost ip but it doesn't works.
Which have to be docker ip address in a linux dev environment?

@maxfields2000 Yes, this give me an error:
I put originally the localhost host: <IP_ADDRESS>, but it didn't works, so i decided review in internet for get my host ip:
 ip r | awk '/default/{print $3}'                    ✓  1778  15:40:02
<IP_ADDRESS>
 salahaddin@TulipanROG  ~/Proyectos/UzmanTech/jenkins/jenkins  master ●   netstat -nr | grep '^0\.0\.0\.0' | awk '{print $2}'
<IP_ADDRESS>

I put then this ip but it doesn't work yet:
And now with other options:

I'm hitting this same issue with Mac. I can see that the Docker API is available because I get a response in my browser with http://localhost:3375/images/json.
How are others not getting this issue?

For those of you who end up here (like me).... one way you can find the host ip is running the following command:
docker run --net=host debian:jessie ip addr show | grep 192
You will get an output like:
inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global eth0

My ip in this case would be <IP_ADDRESS>. If you use that in the Jenkins configuration and test, you should get a successful connections.

@willis7 I will test it, thank you.

I have a fix for this coming soon in my branch that will no longer require you to use a local IP like this by leveraging docker networking, sorry for the pain folks that this isn't obvious.

This should now be fixed in the latest version on master, no need to provide a local IP (unless running on Docker for Windows and then only in the jenkins config for the Dockerhost).


Fix upgrade test by using newer metrics server

Description
k8s 1.22 deprecated v1beta1 of apiregistration and it is now v1. So just use the newer version of the components.yaml from the metrics server which has the fix
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
[ ] Evaluated and added CHANGELOG entry if required
[ ] Determined and documented upgrade steps
[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Upgrade tests should pass :)

/test

/test gke-upgrade-tests

/test gke-upgrade-tests

/test gke-upgrade-tests

/retest

Thanks, this was also breaking the Max PJ Rehearsal, glad to see it's not go1.18 related

/test gke-upgrade-tests

/test gke-upgrade-tests

Yup that did the trick


Whoops - JXHTTP is gone.

By way of a failed "pod install" this morning, it looks the jstn/JXHTTP dependency has vanished (https://github.com/jstn/JXHTTP) so this framework needs to be updated to build properly.

Yikes. I’ve pointed the JXHTTP pod at our Tumblr fork, so this is now working again. Thanks for flagging it up!


Small stylesheet fix for the latest atom update

Work that must be done :smile:

Thanks! This is now fixed in 0.11.

Np, have a nice day :).


Basic set of Augments

A list of augments for the bare minimum variety.
Additional descriptions can be found for each augment at the milanote page.
Bodies (Verb)

[ ] "Pump"-action (hopp)
[x] #119
[x] #112
[x] #175
[ ] bullet Pachinko/labyrinth

Barrels (Subjektiv)

[x] #79
[ ] #83
[ ] #180
[x] #173
[ ] #174
[x] #190

Extensions (Adjektiv)

[x] #130
[ ] #85
[ ] #84
[x] #251

3 of each satisfied with newest build


VirtualList debug mode error

🐛 Bug Report
Get errors when setting FlatList - debug={true}

To Reproduce
cd Destop && react-native init VirtualListDebugDemo && cd VirtualListDebugDemo && react-native run-ios

Expected Behavior
debug mode work fine
Code Example
Edit the App.js:
import React, { Component } from 'react'
import { Text, FlatList } from 'react-native'

export default class App extends Component {
  render() {
    return (
      <FlatList
        //  ❌ Un-commit this line to see error
        // debug={true}
        data={[1, 2, 3, 4, 5, 6]}
        renderItem={_ => {
          return <Text>123</Text>
        }}
        keyExtractor={item => item.toString()}
      />
    )
  }
}

Environment
info
  React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
      Memory: 817.49 MB / 16.00 GB
      Shell: 5.3 - /bin/zsh
    Binaries:
      Node: 11.3.0 - /usr/local/bin/node
      Yarn: 1.13.0 - /usr/local/bin/yarn
      npm: 6.9.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 19, 23, 25, 26, 27, 28
        Build Tools: 23.0.1, 23.0.3, 25.0.2, 26.0.2, 27.0.3, 28.0.3
        System Images: android-23 | Intel x86 Atom, android-23 | Intel x86 Atom_64, android-23 | Google APIs ARM EABI v7a, android-23 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom_64
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>64788
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3
      react-native: 0.59.1 => 0.59.1
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-create-library: 3.1.2
      react-native-git-upgrade: 0.2.7

Hi there, thanks for reporting this issue! I had a quick look and I found the problem in the VirtualizedList. Once #24058 is merged, this should be fixed.


Self relation in graphql v2

I can't get children of parent category? It is empty array. Here is my schema:
`type Category
  @model
  @auth(
  rules: [
    { allow: groups, groups: ["Admin"] }
    { allow: owner, operations: [create, update, delete, read] }
    {  allow: private, operations: [read] }
    { allow: public, operations: [read] }
]
) {
id: ID!
parentID: ID!
title: String!
order: Int
type: String! @index(name: "categoriesByOrder", queryField: "categoriesByOrder", sortKeyFields: ["order"])
parent: Category @belongsTo(fields: ["parentID"])
children: [Category] @hasMany
articles: [Article] @hasMany(indexName: "byCategory", fields: ["id"])
}`

I try to make categories with nested subcategories. Is this schema good for this situation?
Which Category is your question related to?
Graphql API
Amplify CLI Version
5.1.1

Is there some solution with appsync relationships or I need on frontend create subcatogories by grouping by parentId?


1019/1017-grouping_days_since_covid - changes after review

Pre-review checklist
I have ensured that:

[x] Code runs successfully in the development environment.
[x] New code is tested to prove that it works. If not, justify reasoning.
[x] New code is documented using type hinting and following the numpydoc docstring format.
[x] New code follows the project naming standards.

Overview of changes

Post-review checklist
As I merge these changes, I have:

[x] Updated the related Jira ticket, including approximate time spent (including review).
[X] Moved the related Jira ticket to Done.

Function and test look good, but it doesn't appear to get called in the survey responses processing. Does this need adding?

Please add a call to the function in the input processing for survey responses


Change 'Next' and 'Finish' button text

Is there a way to change the 'Next' button and 'Finish' button text? I don't see anything listed for props or slots

This would be great

node_module/src/Translations.js


Constrainted datetime based on timezone offset / UTCDateTime

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
This is basically the same request as #4385 but that was closed as duplicate of #4330 (which is closed now too) but several things have changed since then so creating a new issue.
Summary of the request: being able to add a constraint on the timezone offset of an AwareDatetime.
Support for constraining the timezone offset was added in pydantic-core in May 2023 with
https://github.com/pydantic/pydantic-core/pull/653
So far I can see two ways to achieve this:

Levering the tz_constraint option from pydantic-core by extending from AwareDatetime and overriding __get_pydantic_core_schema__,

Example code
from typing import Any
from datetime import datetime, timezone, timedelta
from pydantic import BaseModel, AwareDatetime
from pydantic_core import core_schema

# Load GetCoreSchemaHandler via try/except because it was made public in v2.4.0
try:
    from pydantic.annotated_handlers import GetCoreSchemaHandler
except:
    from pydantic._internal._annotated_handlers import GetCoreSchemaHandler

class UTCAwareDatetime(AwareDatetime):
    @classmethod
    def __get_pydantic_core_schema__(
        cls, source: type[Any], handler: GetCoreSchemaHandler
    ) -> core_schema.CoreSchema:
        if cls is source:
            # used directly as a type
            return core_schema.datetime_schema(tz_constraint=0)
        else:
            schema = handler(source)
            _check_annotated_type(schema["type"], "datetime", cls.__name__)
            schema["tz_constraint"] = 0
            return schema

    def __repr__(self) -> str:
        return "UTCAwareDatetime"

class TestModel(BaseModel):
    created_at: UTCAwareDatetime

now = datetime.now(tz=timezone(timedelta(seconds=1800)))

model = TestModel(created_at=now)
print(model.model_dump_json())

Using an AfterValidator to do the extra validation

Example code
from typing_extensions import Annotated
from datetime import datetime, timezone, timedelta
from pydantic import BaseModel, AwareDatetime
from pydantic.functional_validators import AfterValidator

def check_tz(v: datetime):
    assert v.utcoffset() == timedelta(seconds=0)
    return v

UTCAwareDatetime = Annotated[AwareDatetime, AfterValidator(check_tz)]

class TestModel(BaseModel):
    created_at: UTCAwareDatetime

now = datetime.now(tz=timezone(timedelta(seconds=1800)))

model = TestModel(created_at=now)
print(model.model_dump_json())

In both cases there is validation error (as expected), and the error from pydantic-core is superior.
Now back to the future request: Make it easier to use the tz_constraint option of pydantic-core to restrict the tz offset of a datetime.
Could be via:

an extra type? (UTCDatetime? RestrictedTzOffsetAwareDatetime?)
extend Field to support tz_constraint?
...

Affected Components

[ ] Compatibility between releases
[X] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[X] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[ ] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

I think we could make use of annotated-types' Timezone constraint. It might already be supported, but I can't find any reference in Pydantic docs

I'm happy to add this via full support for the annotated-types type.
@bram-tv by the way, you can use custom errors to make the errors nicer.

@bram-tv,
Any interest in opening a PR to add support for this?

I would appreciate it if @sydney-runkle , @bram-tv  could provide me with some context regarding the implementation of this Feature. Specifically, I am interested in understanding the various Cases that need to be covered and the desired outcomes we aim to achieve by implementing these constraint types.

@Zidane786  @samuelcolvin  @bram-tv It would be great if this constraint will support not only specifying one specific timezone, but also specifying a UTC offset range or a list of supported timezones. Example of use: postgres supports timezones with UTC offset from -12:00 to +13:00. If we pass a datetime with a utc offset that goes beyond this limit, we get an error. For myself, so far I've done something like this
from datetime import datetime, timedelta
from typing import Annotated

from pydantic import AfterValidator

def check_timezone_utcoffset_range(v: datetime) -> datetime:
    utc_offset = v.utcoffset()
    if utc_offset is not None and (
        utc_offset < timedelta(days=-1, seconds=43200)
        or utc_offset > timedelta(seconds=46800)
    ):
        raise ValueError("timezone utc offset should be in range from -12:00 to +13:00")

    return v

DatetimeRestrictedUtcOffset = Annotated[
    datetime, AfterValidator(check_timezone_utcoffset_range)
]

At PyCon2024, taking a look at this.

@Zac-HD,
We're looking at adding support for the Timezone class from annotated-types. I see that type doesn't support integer time offsets - what would you think about adding support for that to annotated-types? Would it make more sense to add that in a new offset type? Thanks for your feedback!
Also happy to chat in person if you'd like, we're in room 321.

Here's the approach we talked about :). This is a super cool way to combine
from pydantic import TypeAdapter, GetCoreSchemaHandler
from typing_extensions import Annotated
from datetime import datetime
from pydantic._internal._core_utils import pretty_print_core_schema
from dataclasses import dataclass
from typing import Any
from pydantic_core import CoreSchema, core_schema
from functools import partial

def my_validator_function(tz_constraint: str, value: datetime, handler):
    # pre logic

    validated_dt = handler(value)

    # post logic

    return validated_dt

@dataclass(frozen=True)
class MyDatetimeValidator:
    tz_constraint: str

    def __get_pydantic_core_schema__(self, source_type: Any, handler: GetCoreSchemaHandler) -> CoreSchema:
        return core_schema.no_info_wrap_validator_function(
            partial(my_validator_function, self.tz_constraint), handler(source_type)
        )

ta = TypeAdapter(Annotated[datetime, MyDatetimeValidator('UTC')])
print(pretty_print_core_schema(ta.core_schema))
"""
{
    'type': 'function-wrap',
    'function': {
        'type': 'no-info',
        'function': functools.partial(<function my_validator_function at 0x104a73d90>, 'UTC')
    },
    'schema': {'type': 'datetime', 'microseconds_precision': 'truncate'}
}
"""

my_dt = ta.validate_python(datetime.now())
print(my_dt)
# > 2024-05-20 14:42:15.066963

We're looking at adding support for the Timezone class from annotated-types. I see that type doesn't support integer time offsets - what would you think about adding support for that to annotated-types?

Offsets are not timezones, so I'd oppose such a change.

Would it make more sense to add that in a new offset type?

...and offsets are very very very rarely the right solution to a problem, making an Offset type into an attractive nuisance of an interface.  I'd oppose adding one to annotated-types, and would recommend that Pydantic require would-be users of offsets to implement that for themselves.

It would be great if this constraint will support not only specifying one specific timezone, but also specifying a UTC offset range or a list of supported timezones. Example of use: postgres supports timezones with UTC offset from -12:00 to +13:00. If we pass a datetime with a utc offset that goes beyond this limit, we get an error.

IMO this is a nasty bug in Postgres; there are timezones with a +14 UTC offset, and other historical examples still might need to be represented in a database.  I recommend reporting a bug to your database vendor.
More generally, supporting only some timezones is somewhere between an attractive nuisance and a code smell.  If you really really need to, you can use a union of types: Annotated[datetime, Timezone("Australia/Sydney")] | Annotated[datetime, Timezone("America/Los_Angeles")] | ... - the inelegant syntax accurately reflects underlying design problems.

Going to close this as completed now - the given example does a great job at showing how to get the desired behavior.


[Feature][UI]make status and schedule publish status display together (issue-12834)

make status and schedule publish status display together in workflow definition list page (issue-12834)

Purpose of the pull request
close #12834
Currently,status and schedule publish status display together

Brief change log
Modified the order of headers, no backend changes
Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

Merging #12922 (8791ea4) into dev (f20c9b3) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #12922      +/-   ##
============================================
- Coverage     39.48%   39.46%   -0.03%
+ Complexity     4251     4244       -7
============================================
  Files          1055     1055
  Lines         39885    39853      -32
  Branches       4583     4569      -14
============================================
- Hits          15750    15726      -24
+ Misses        22356    22349       -7
+ Partials       1779     1778       -1

Impacted Files
Coverage Δ

...erver/master/processor/queue/TaskEventService.java
69.64% <0.00%> (-10.72%)
:arrow_down:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
42.55% <0.00%> (-2.13%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...hinscheduler/plugin/alert/script/ScriptSender.java
70.58% <0.00%> (-1.64%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
51.38% <0.00%> (-1.39%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
51.19% <0.00%> (-1.20%)
:arrow_down:

...cheduler/plugin/alert/dingtalk/DingTalkSender.java
34.13% <0.00%> (-0.78%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.86% <0.00%> (-0.55%)
:arrow_down:

...inscheduler/plugin/registry/etcd/EtcdRegistry.java
50.69% <0.00%> (-0.35%)
:arrow_down:

... and 20 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@SbloodyS  Please do a CR when you are free, or help to see if I need to add something

@Amy0104 Can these commits be merged and the pr closed now? what else do i need to do


Api Refactoring

Issue:

[ ] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
Refactoring APIs based on feedback from creating client

Split Player Management controllers for easier navigation of methods. Enables applying authorization at the controler etc
Tweak the API shape (e.g. /playerextended to /player/state)

Test:
Run Nether.Web and integration tests

Hi @stuartleeks, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor. If you're full-time or an intern, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Feels like we are dealing with quite a lot more than "just API Refactoring" but I'll keep reviewing

It's refactoring the external API, and splitting out some of the large controllers to simplify them as I go :-)

I've looked at the changes from a helicopter perspective and it looks good. I think we should go ahead and merge.

Reviewing last changes


The generator implicitly converts integers to string in some cases.

Consider the following two profiles. They are identical, except for the differences in bold.
First version:

{
  "schemaVersion": "0.1",
  "fields": [
    {
      "name": "Column 1"
    }
  ],
  "rules": [
    {
      "rule": "Rule 1",
      "constraints": [
        {
          "allOf": [
            {
              "anyOf": [
                {
                  "field": "Column 1",
                  "is": "ofType",
                  "value": "string"
                },
                {
                  "field": "Column 1",
                  "is": "ofType",
                  "value": "integer"
                }
              ]
            },
            {
              "not": {
                "field": "Column 1",
                "is": "null"
              }
            },
            {
              "if":
                {
                  "field": "Column 1",
                  "is": "ofType",
                  "value": "integer"
                },
              "then":
                {
                  "field": "Column 1",
                  "is": "equalTo",
                  "value": "a"
                },
              "else":
                {
                  "field": "Column 1",
                  "is": "equalTo",
                  "value": "b"
                }
            }
          ]
        }
      ]
    }
  ]
}

Second version:

{
  "schemaVersion": "0.1",
  "fields": [
    {
      "name": "Column 1"
    }
  ],
  "rules": [
    {
      "rule": "Rule 1",
      "constraints": [
        {
          "allOf": [
            {
              "anyOf": [
                {
                  "field": "Column 1",
                  "is": "ofType",
                  "value": "string"
                },
                {
                  "field": "Column 1",
                  "is": "ofType",
                  "value": "integer"
                }
              ]
            },
            {
              "not": {
                "field": "Column 1",
                "is": "null"
              }
            },
            {
              "if":
                {
                  "field": "Column 1",
                  "is": "ofType",
                  "value": "integer"
                },
              "then":
                {
                  "field": "Column 1",
                  "is": "equalTo",
                  "value": 0
                },
              "else":
                {
                  "field": "Column 1",
                  "is": "equalTo",
                  "value": 1
                }
            }
          ]
        }
      ]
    }
  ]
}

First Version Discussion
In plain English, the profile will produce data that is never null, with a type of either string or integer. If the type is integer, then only values of "a" will be produced else only values of "b" will be produced.
Because the string "a" is not an integer, the only generation output to CSV is rows of "b".
Second Version Discussion
In plain English, the profile will produce data that is never null, with a type of either string or integer. If the type is integer, then only values of 0 will be produced else only values of 1 will be produced.
Because the integer 0 is not a string, the only generation output to CSV is rows of 1.
This is not what happens. The second version produces 0s and 1s.
This is because there are implicit type conversions from integer to string.
When we get to the 0 case, we know that the field is a string and is equal to 0 (and is not null, from another condition). The system implicitly converts the 0 to the string "0", which matches all criteria, and therefore 0s are allowed outputs.
Pros of this type conversion

Data may be inputted into the generator from a format that does not distinguish between the string "0" and the integer 0. Therefore, the generator should not distinguish between them.
Users may expect this conversion.

Cons of this type conversion

Users may not expect this conversion (as I didn't expect it, which is why I raised this issue in the first place).

Moving Forward (if we choose to keep the current behaviour)

Perhaps someone could provide an example profile justifying why this type conversion could be helpful, if the team decides it would be helpful?
This behaviour should be documented in detail. Perhaps in a "gotchas" section for profiles?
For the generator's CSV output, all strings are enclosed by double quotes ("), e.g. the "a"s and "b"s for the first profile here are enclosed by quotes. However, the 0s implicitly converted to strings are not given enclosing quotes. This should be fixed to make it clearer to the user that this type conversion exists.
Consider giving a console warning when an implicit type conversion is made.

Moving Forward (if we choose to remove the implicit type conversion)

Consider first whether the potential problem (the first bullet in the "Pros of this type conversion" section) is actually a problem, and how it may need to be solved.

The Cartesian walker does not do these type conversions, so only outputs zeros. Therefore one of the two must be wrong (mentioned by @willsalt-sl ).

I think the root issue isn't to do with type conversions - my suspicion is that the reductive solver isn't applying the conditional constraint correctly, hence both options being outputted.

as we are moving towards strongly typed fields i am closing this


Unable to use locations() function in gimli::read::LocationLists with object files using DWARF4

Hi,
I am trying to read location lists from the .debug_loc section in object files having debugging information in DWARF4.
While using the locations() function from gimli::read::LocationLists (https://docs.rs/gimli/0.18.0/gimli/read/struct.LocationLists.html#method.locations), I noticed that it takes debug_addr and debug_addr_base parameters, as follows.
pub fn locations(
    &self,
    offset: LocationListsOffset<R::Offset>,
    unit_encoding: Encoding,
    base_address: u64,
    debug_addr: &DebugAddr<R>,
    debug_addr_base: DebugAddrBase<R::Offset>
) -> Result<LocListIter<R>>

But, these parameters correspond to the .debug_addr section, which was added in DWARF5.
So, I am not able to use this function to read the location lists for an object file that uses DWARF4.
Could you please suggest something?
Thanks!

It doesn't matter what those parameters are in that case because there won't be any attributes that use them, so default them to empty/zero.

It doesn't matter what those parameters are in that case because there won't be any attributes that use them, so default them to empty/zero.

Ok, I see. Thanks! I have done that and it works.

Ok, I see. Thanks! I have done that and it works.


fix a bug in the docs for using schema downgrading

The documentation left out a required argument for using the downgrading system from python in one of its examples. Also clarifies how to get core schemas from python.

Codecov Report

Merging #1641 (c7a5193) into main (45c6cfd) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1641      +/-   ##
==========================================
- Coverage   79.91%   79.91%   -0.01%
==========================================
  Files         197      197
  Lines       21731    21731
  Branches     4339     4339
==========================================
- Hits        17367    17366       -1
  Misses       2213     2213
- Partials     2151     2152       +1

Flag
Coverage Δ

py-unittests
79.91% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


In redmine 4.1.0, it's difficult do download a file in File tab

I'm experiencing s strange behavior: with purplemine2 theme applied and redmine 4.1.0 running, I cannot download a file from project File tab: to start the operation I need to click multiple time on download icon. With default theme applied I haven't this issue.
Thanks in advance.

I'm closing this. If you are still experiencing the issue, please don't hesitate to reply.

Hi @mrliptontea, sorry for the big delay : I attach here a video to show the steps to reproduce the issue. It seems related to row highlight feature. I can download the file only if I'm exactly on the download icon.
Thanks in advance.

Hi @pasqualedante
Now I see what you mean. Can you update the theme and check if this is still an issue? I think this was addressed for #177.

Hi @pasqualedante
Now I see what you mean. Can you update the theme and check if this is still an issue? I think this was addressed for #177.

Yep, you are right! Thanks.

Awesome! 🎉


Copy only required bot assets from "python-zulip-api".

This is a follow-up for #25888.
Right now through generate_zulip_bots_static_files.py -- we are copying assets from python-zulip-api/zulip_bots/zulip_bots/bots for all bots. But we only require assets of the bots that are listed in  BOT_INTEGRATIONS in zerver.lib.integrations.
This is leading to assets of ~35 bots being copied -- out of which right now only 2 are required.
We need to fix this and make sure only the required assets are copied.

@zulipbot add "area: bots" "area: tooling"


Error loading cache from host.db

We are trying to upgrade to 7.1.1 .
And when I do start ATS, I am seeing following warning in diags.log file
"WARNING: Error loading cache from /opt/ats/var/trafficserver/host.db: -1"
I haven't overridden any of the properties related hostdb in records.config.
Can some one please help on this.

We have been seeing that, but as far as we can tell it's innocuous. Please let us know if you observe any negative effects. In our experiments this causes the old host DB file to be deleted and the HostDB repopulated with no observable problems.

Can you please also help me on, whether there is any known issue in hostdb.config file not getting generated? I am able to see hostdb.config getting in 6.1.0 while I don't find it in 7.1.1 .

If this is still an issue please reopen.


Add physical address as an option in follow/contact section

If the theme is used by someone retro enough to post the address, then this option provides a generalized way to put a standard html address on the page. It might be a useful option in case someone wants to use it, but it is easily disabled by just commenting out the section.
I used the 6px padding and text small, so that it is a bit separated from the description text.

@apvarun any feedback on this Pull request?


expose Readable & Writable

fixes #1

There is a way to add streams to unionfs, but it is not trivial.
Your implementation is obviously invalid, all it does is it creates some "placeholders".
What is the reason for doing so?

graceful-fs tries to get those streams (fs.prototype.ReadStream, fs.prototype.WriteStream), if they are not there it fails hard.
yes it seems wrong as it still doesnt work correctly here (just a few steps later)

I am OK including this hack. However this introduces a dependency:
import {Readable, Writable} from "stream";

Which may break browser users. It has to be done somehow as to not break browser builds.

Maybe something like this
const isBrowser = typeof __filename === 'undefined';

?

yeah made it conditional...
another question here:
i think im using it alltogether somehow wrong:
const {ufs} = require('unionfs');
const {Volume} = require('memfs');
const fs = require('fs');

ufs
.use(fs)
.use(Volume.fromJSON({"foo.js": ""}, "/tmp"))

console.log(ufs.existsSync(__filename)); // false
console.log(fs.existsSync(__filename)); // true
console.log(ufs.existsSync("/tmp/foo.js")); // true

why does it fail when trying to stat an existing file? the in memory volume is mounted somewhere else __filename shouldnt be affected?!

@streamich any idea on this one?

I will take a look at it this evening.

@streamich thanks for merging and fixing! did you have some time to look into my example above?

@digitalkaoz Sorry, I keep forgetting about this, will create an issue.

@digitalkaoz Should be fixed here: https://github.com/streamich/unionfs/issues/10#issuecomment-350269633
Thanks for the find.


⚠ Mobitopup has degraded performance

In 2bc7eae, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 4064 ms

Resolved: Mobitopup performance has improved in 0f32c27.


Missing Camera provider on iOS

Hello,
There seems to be no Camera provider for iOS, but I have seen there used to be one based on AVFoundation.
I am very familiar with AVFoundation, but not at all with the Kivy codebase. Can anyone give me some pointers about where to start looking and why it was disabled? I'll try to fix it if possible.
Thanks!
Versions

Python: 2.7.1
OS: MacOS, iOS
Kivy: 1.10
Kivy installation method: pip

Code and Logs
[CRITICAL] [Camera      ] Unable to find any valuable Camera provider.

Hi,
Kivy installation on your computer doesn't matter for the Kivy used for iOS. Cross compilation happen, and that's not the same Kivy (the Kivy-iOS toolchain will clone and cross compile Kivy, it doesn't matter what you installed).
As for AVFoundation, the camera code is located at https://github.com/kivy/kivy/blob/master/kivy/core/camera/camera_avfoundation.pyx, and implementation at https://github.com/kivy/kivy/blob/master/kivy/core/camera/camera_avfoundation_implem.m
Look like in the comment, i wrote to test it on iOS, but i never did.
To activate the provider, at the end of https://github.com/kivy/kivy/blob/master/kivy/core/camera/init.py, add a test to check for iOS and add the camera provider for it. Then change the setup.py to activate "use_avfoundation" on iOS (and make change whatever needed to be able to compile on both OSX and iOS.)
That's it, good luck :)

Thanks a lot! Someone on IRC pointed me to the right direction, but I was missing the setup.py part.
Because I am not familiar with the build system, I don't know how to alter the code that will be cross-compiled. toolchain.py build kivy does not see the changes and does not rebuild, and toolchain.py clean kivyfollowed by toolchain.py build kivy re-downloads the github archive and overwrites my changes.
I don't see any option to rebuild without re-downloading.
Or can I change the url attribute in the recipe to point to local files instead of GitHub?

OK, I discovered the DIR_KIVY environment variable that lets me compile local code.
Then I ran into the same problem as https://github.com/kivy/kivy/issues/3965
Replacing         self._format = 'bgra'
with         self._format = 'rgba'
is enough to make the app run, but of course the colors are wrong.
The trouble is iOS camera will not output RGBA, only BGRA and several planar Lum+Chrominances formats.
So can the texture use BGRA or some YUV variant? Or can _copy_to_gpu() do the conversion?

OK, last update for today:
            self._texture = Texture.create(self._resolution, colorfmt='bgra')
            self._texture.flip_vertical()
            self.dispatch('on_load')

        self._buffer = <bytes>data[:rowsize * height]
        self._format = 'bgra'
        self._copy_to_gpu()

Works (at least on a recent iOS device with BGRA support).

Nice work!
A https://github.com/kivy-garden/garden.zbarcam user/developer confirms (https://github.com/kivy-garden/garden.zbarcam/issues/17#issuecomment-397844165) this fixes the black screen on his iOS device too. Can we bring a fix upstream?

Issue looks fixed on master branch. Thank you!


Network consolidation

Summary
This PR attempts to consolidate all network related logic by pulling network connection listening, app state listening, and websocket initialization/starting/stopping into one component.

@jarredwitt build failures

@enahum @hmhealey pushed some fixes so that this PR should build. I'm working on incorporating the other changes now.

@enahum @hmhealey refactored and removed the animation from the state. I'm still trying to solve one case, but wanted to get your eyes on these changes quickly.

@enahum I didn't notice any issues with the connection bar showing at app launch. I also added in a retry after 30 seconds. Can obviously change that, but here's a screenshot. I didn't do a countdown in order to keep from adding the component state back in.

@jarredwitt is there anything else pending on this PR ?

@jarredwitt is there anything else pending on this PR ? also I'm not seeing where this.connectionRetryTimeout is being set

@enahum I don't think there's anything else pending on my end. Other than a solid testing round, I think it should be good to go.

@jarredwitt ok lets merge it and see ;)


the obvious: add /.idea/

get up and runing ;-)

Hmm, why? I don't know what .idea even is :-)
(Any reason why you cannot add local/custom ignores to .git/info/exclude?)

it's the pycharm project settings folder...

I can easily setup a global core.excludesFile...learnt another thing...up to you to close or merge...

Thanks again. I'm going to reject this. People should set set up their own ignore files for their editors.


removes move field from UCTNode

it was redundant.

@oscardssmith Did you test this? Doesn't work for me.

I might have a typo somewhere. I did this pretty quickly.


GPO doesn't make sense

"Turn off auto-restart notifications for update installations" - the default setting is not to show any notification, when you enable it, it doesn't show any notification, when you disable the setting, it doesn't show any notification again... When you leave it undefined, I guess  you know what happens

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

ID: 0184809a-63ff-ecc2-ab67-9c9f4df900f8
Version Independent ID: ac25ee6c-4c57-58b3-1313-e4a53f53c5a1
Content: Manage device restarts after updates (Windows 10)
Content Source: windows/deployment/update/waas-restart.md
Product: w10
Technology: windows
GitHub Login: @jaimeo
Microsoft Alias: jaimeo

@officedocsbot assign @mypil

@CoMMyCZ - Thank you for submitting feedback.
I will get this issue over to the Win10 ITPro writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

Dear @CoMMyCZ ,
the GPO works following way:
enabled: notifications do not appear.
disabled or not configured: notifications for auto-restart appear.
if you experience different behavior, I would recommend following: run rsop.msc to check the resulting policy and make other GPOs are not affecting this setting.
Thank you

@CoMMyCZ - From our understanding, the issue you raised has been answered by MaratMussabekov. If you feel it hasn't been answered, please re-open this issue.
Thank you for your contribution to make the docs better! Much appreciated!

@officedocsbot close


Shadow size too big

I think its best to lower the shadow size in the settings.

Since this is project is for showing how a FPC might work in Godot the graphics don't really matter. I havn't had any problems with the shadows on my Linux machine. Having any problems on your end, like any glitches? Is it just the resolution being too high or low?

Thx, it just gives more fps. Great documented tut. :)


Reset multi-currency expense's currencies when changing payout method

Part of #5146
Followup on https://github.com/opencollective/opencollective-frontend/pull/7421#pullrequestreview-872672318
Steps

Submit an expense to a USD collective in EUR
Edit the expense, set up a new payout method in GBP
Go next, the amount is still in EUR while the payout method is in GBP - which is not wrong in itself (the alternative of changing the currency without converting is far worse) but will usually result in a payment failure when paying with transferwise

Expected behavior
When changing the payout method, if expense.currency is not the same as collective.currency, nullify the currency and force the expense submitter to fill it again.

@Betree : I'll try to fix this issue this week if no objections. 🙂


Keep getting "Had exception during #Watir::LI:0x007ffdbc020028" error

I can launch the script however I kept getting "Had exception during #Watir::LI:0x007ffdbc020028" error after several successful invitation.

The problem has been solved by slowing down the speed of sending invitation.


Game doesn't end on players death if there is only one player in the game.

Exists in both singleplayer and a server browser.

This also happens if players have joined and later left the game.


using compdb with cmake's "include_directories()" results in clangd errors

I use include_directories(src) with cmake so I can include absolute paths from the src/ directory for C++ files. It compiles fine, but clangd yells at me that it can't find absolute paths include files
structure:
src
├── Engine
│   ├── Renderer.cpp
│   ├── Renderer.h
├── Game
│   ├── Chunks.cpp
│   ├── Chunks.h
└── Main.cpp
I get 'Engine/Renderer.h' file not found from clangd when included this way:
#include "Engine/Renderer.h"
It works fine relatively:
#inlude "../Engine/Renderer.h"
Is this a problem with compdb or with how cmake exports compile_commands.json?

I am interested.
Have you found a solution? :)


Problem to update packages

When I want to download it takes more than half an hour and not download anything.

this is probably an npm issue -- what version are you using? supposedly 3 is a lot slower, I would stick to 2.
either way this is good feedback for npm team, but we cant do much for this

it is correct. I update npm and work perfectly