METRON-1346 Add new PMC members to site

Add new PMC members @mmiklavc, @justinleet , @ottobackwards
##Testing
Follow steps at https://cwiki.apache.org/confluence/display/METRON/Website+PR+Merge
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

+1 by inspection, lgtm


Syntax doesn't work when surrounded by quotes

For example
<img class="__post-cover-photo" src="<%= thumbnail %>" />

It's a solid color inside of the quotes. It should have different colors for things like <%= and %> and thumbnail

Sadly the is no look ahead in regex, therefore there is no way for it to know that there is a script inside this string.

Hm, out of curiousity then how do the other templating languages do this? For example that identical code works in Mustache just changing it to {{ and }} instead of <%= and %>

It's a scoping mechanism. Those symbols determine that code begins and ends, code which has it's own syntax. For strings there is no further syntax to be parsed through, you can just have a string.
There is no way to differentiate when `"<%= ... %>" is meant for code and when it coincidentally happens as a string, therefore the syntax compromises and says it's always a string because it doesn't know any better.
If my explanation does not suffice as it might be confusing, try to learn more about regular expression or even how compilers and interpreters "tokenize" your code. Hope this helped any way.


Make functions with if and dtype attributes trace_only

According to Rama, if branches needs to produce the same type and this conflicts with some logic we have in the atenlib

:( More and more trace_only ops... Is there a way to automate the decision if trace_only should be applied?

I think it's still good if we make a conscious choice. I am going to create a guide #388 for it.


Fix safeArea for database content view

Currently, if you use iPhone X or any iPhone with a notch to view database, the column names will be hidden under the navigation bar. This is because the safeArea is not respected. This commit contains a simple fix to that.

Added compiler flag to support older SDKs


fix(codesnippet): Design audit changes

Closes #1888

example of fadeout/overflow with long text

This looks great, one final comment:
can we add the fade to the single line code snippet since terminal code snippet has it?
Otherwise LGTM!

Hey @jnm2377 thanks for making the change! Lovely fade for overflow 💯 ❤
A few hopefully small issues:

[ ] expansion animation is missing when clicking on "show more" (but closing animation goods great)
[ ] multiline horizontally scroll didn't work for me (chrome)

@shixiedesign @jeanservaas made those changes! 👍


Feat  real anon check

Adds the "real" check for isAnonymous on the Owner backend, and adds frontend handling of anonymous users (-> do not show the emailVerification-Slide-in if an anonymous user is logged in)

This PR is so we can (finally) start implementing handlers to avoid sending anonymous Users emails into the void


Fix small ui issues

I think easiest for you would be to try the branchs' ui your-self

Looks nice, do you want to look into also fixing the hover effect on the graphs?
The text is a little hard to read and I think the values should be rounded.

Also saw that the play box is buggy:

That one is on me but if you want to fix it would be greatly appreciated ^^

And yes didn't put any time on scaling the play box on window size but can be done in a future PR

I hate the librarys tooltip functionality, if you want to look in to changeing the colors or the background color pkeas do, I also want it fixed

Cool cool, create issues for tooltip functionality if that bugs you.


Repeatedly open tabs

In latest version 1.0.24

Thank you for catching this! It looks like it's occurring on notes within folders; sorry I didn't initially see this, my vault has a flat structure. Working on a fix now! Thanks again for spotting this.

Fixed by 1.0.25. Please let me know if you see anything else. Thanks again!


Show multiple interactive series on the Ready for Swift 6 charts

Still very much a WIP but I'd like to get it on staging for feedback.

Styling updated along with a mobile layout and a hacky dark mode layout.


Battery drains really fast while using for some reason,

This appears to be an iOS 16 thing as @RoootTheFox doesn't have this issue.
I'm 99% sure it's caused by the UIColorFromRGB macro having a bunch of expensive math operations.

is that still happening with the new tweak version btw? if not I'd like to close this

since this uses your ColorHelper for theming, i think this should no longer be an issue so I'll close this for now - if the issue persists, please reopen !!


New version: JetBrains.CLion version 2024.1.2

Created by 🥟 Dumplings in workflow run #5479.
Logs
Updated: 2024.1.1 → 2024.1.2
Submitting WinGet manifests

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run

This package installs and launches normally on a Windows 10 VM.

This package installs and launches normally on a Windows 10 VM.


add a settings tab

This might require a whole rethinking of how the extension works. Instead of activating the script by pressing the extension icon, there would be a new spotify-esk icon at the bottom, near the volume button, similar to the lyrics button.
Then press the extension icon for settings. The settings menu might contain what filters the search has, a checkbox to take the user to the highest rated tab on the list, etc.

#7


.scss @use directives don't appear to work.

I am using Yew.

Directory
index.html
styles/radial.scss
styles/window.scss
styles/styles.scss

index.html
<!DOCTYPE html>
<html class="no-js" lang="en">

<head>
  <meta charset="utf-8" />
  <title></title>
  <meta name="description" content="">
  <meta name="viewport" content="width=device-width, initial-scale=1">
  <link data-trunk rel="scss" href="styles/styles.scss" />
  <link data-trunk rel="copy-file" href="assets/favicon.ico" />
</head>

<body></body>

</html>

styles/styles.scss
@use 'radials';
@use 'window';

if I supply my regular scss all in the same file everything works fine.

Resulting css file.
@use 'radial_menu';@use 'rrr_window'

Are there any complex examples or demos of a modern yew website using scss that I could follow?
Thanks.

After doing digging, I see that this is because we aren't using a library that supports dart.
#13
Unfortunately it looks like grass is still missing these features:
indented syntax
css imports
@forward
compressed output

one of which was the reason you switched away from grass in the first place.
In the meantime, I learned something about css and switched over to @import tag.

I have no opinion, but there are a few more maintained options over sass-rs.
https://github.com/kaj/rsass
https://github.com/connorskees/grass

@Zageron agreed that sass-rs needs some love; however, unfortunately it would seem that it is currently the best option that we have.
@Zageron before we close this issue, do you mind actually updating the title of this issue back to what it was originally? I suspect that your diagnostics trail here may actually help other folks that might run into this issue.
Also, thanks for the report nonetheless!


Error during processing after detecting 20 diacritics

Current Behavior
I have a pdf that convert with convert from ImageMagick with the command convert -density 300 -quality 100 -alpha remove file.pdf file.jpg
Then when I make this command : tesseract 'file.jpg' 'file' pdf , the command fails with this logs :
Estimating resolution as 241
Detected 20 diacritics
Error during processing.
The pdf is a packaging with several language, similar to this image : https://static.wixstatic.com/media/8b953d_6a2242e1649a4e7ca75f3fa0a93ff95c~mv2.png
Expected Behavior
A pdf in result or more logs to precise the error.
Suggested Fix
No response
tesseract -v
tesseract 5.3.0
leptonica-1.78.0
libgif 5.1.4 : libjpeg 8d (libjpeg-turbo 1.5.2) : libpng 1.6.34 : libtiff 4.0.9 : zlib 1.2.11 : libwebp 0.6.1 : libopenjp2 2.3.0
Found AVX2
Found AVX
Found FMA
Found SSE4.1
Found OpenMP 201511
Found libarchive 3.2.2 zlib/1.2.11 liblzma/5.2.2 bz2lib/1.0.6 liblz4/1.7.1
Found libcurl/7.58.0 OpenSSL/1.1.1 zlib/1.2.11 libidn2/2.0.4 libpsl/0.19.1 (+libidn2/2.0.4) nghttp2/1.30.0 librtmp/2.3
Operating System
No response
Other Operating System
Linux Mint 19 Tara
uname -a
No response
Compiler
No response
CPU
No response
Virtualization / Containers
Directly on my computer
Other Information
Tesseract works with other pdf with the same proccess, convert then tesseract in the same environnement.

After I check, I can show the image.
So in the attachments, the pdf source and the image.
Thanks,
Jérémie

3276000700067-DUSTBINS REMIX SLIM SENSEA BLACK-FUCHENG-ARTWORK.pdf

The error is throwns because that image has four color components:
253006252-0bcbd732-d0e3-4841-b1f6-0e35c1742112.jpg: JPEG image data, JFIF standard 1.01, aspect ratio, density 300x300, segment length 16, baseline, precision 8, 3451x2058, components 4

The function tesseract::TessPDFRenderer::imageToPDFObj currently only handles images with one (DeviceGray color space) or three (DeviceRGB color space) color components. DeviceCMYK color space is not implemented.
With modified code, it creates this PDF:
out.pdf

Thank you for your response.
I test to use convert with the -colorspace RGB with jpg and it's working. I also test with extensions and tiff and it's working too.
So my problem it's solved.


[Bugfix] Attempt to fix broken ctrl-c behavior

Description
This PR tries to fix the bug mentioned in #812 and #1027. If you hold crtl-c the prompt build stops before evaluating the functions in $PROMPT and leaves the raw strings behind.
I approach this by evaluating the functions inside $PROMPT, i.e. __p9k_build_left_prompt, in function __p9k_prepare_prompt and writing it to a newly introduced global variable $__p9k_unsafe_PROMPT (this is probably against the style guide since I wrote part of the varibale in capital letters. I will change it if requested, of course).
The unsafe variable is then put into $PROMPT with single quotes: PROMPT='$__p9k_unsafe_PROMPT'. This is done to prevent evil code to be executed if the contents of the unsafe variable is tainted after evaluating __p9k_build_left_prompt. The same is done for $RPROMPT
The benefit is, that since the function __p9k_prepare_promt is run as a precmd hook, it can be interrupted without leaving $PROMPT unchanged/raw. This also means it could be out of date, but this is still better than no prompt.
Question/Problem
Currently I'm having trouble running tests since I get lots of errors regarding background_jobs on stderr. Example:
function setUp() {
  export TERM="xterm-256color"
  source powerlevel9k.zsh-theme
}
function testFoo() {
  cd /
}
source shunit2/shunit2

Running this test yields
 ❱ zsh foo.spec
testFoo
prompt_background_jobs:2: bad math expression: operand expected at end of string

Ran 1 test.

OK

I think this is the important part (run with set -xv right before the cd /)
+__p9k_build_right_prompt:19> index=3
+__p9k_build_right_prompt:4> element=background_jobs
+__p9k_build_right_prompt:6> [[ nd_jobs == _joined ]]
+__p9k_build_right_prompt:6> joined=false
+__p9k_build_right_prompt:9> element=background_jobs
+__p9k_build_right_prompt:13> [[ $element[0,7] -regex-match "custom_" ]]
+__p9k_build_right_prompt:16> prompt_background_jobs right 3 false
+prompt_background_jobs:1> local jobs_print=''
prompt_background_jobs:2: bad math expression: operand expected at end of string
+__p9k_prepare_prompts:40> __p9k_unsafe_RPROMPT=$'%f%b%k%F{000}%K{000}%F{002} %F{002}✔ %{\C-[[00m%}'
+__p9k_prepare_prompts:41> RPROMPT='$__p9k_unsafe_RPROMPT'
+__p9k_prepare_prompts:46> [[ '' == Yes ]]
+__p9k_prepare_prompts:49> local NEWLINE=$'\n'
+__p9k_prepare_prompts:52> [[ false == true ]]
+__p9k_prepare_prompts:63> PROMPT='$__p9k_unsafe_PROMPT'
+__p9k_ch_pwd:2> __p9k_preexec
+__p9k_preexec:6> _P9K_TIMER_START=1544831671.7253572941
+testFoo:3> set +vx

The problem seems to be the following (backwards):

prompt_background_jobs line 2: local total_jobs=$((  +  )) because $jobs_running and $jobs_suspended are not set
should be set in __p9k_background_jobs trigged by another precmd executed before (as far as I'm aware) __p9k_prepare_prompt
this function never runs and I don't understand why since add-zsh-hook -L outputs this:

 ❱ cat /tmp/hooks
typeset -g -a preexec_functions=( __p9k_preexec )
typeset -g -a chpwd_functions=( __p9k_ch_pwd )
typeset -g -a precmd_functions=( __p9k_background_jobs __p9k_prepare_prompts )

I have an idea for a workaround to change segments/background_jobs.p9k#L42 to:
  local total_jobs=$(( ${jobs_running:-0} + ${jobs_suspended:-0} ))

But it worked before, so maybe I'm missing something obvious why one precmd hook is not running.

What I use for debugging:
 ❱ cat foo.spec NC
setopt shwordsplit
SHUNIT_PARENT=$0
function setUp() {
  export TERM="xterm-256color"
  source powerlevel9k.zsh-theme
}
function testLeftMultilinePrompt() {
  local -a P9K_LEFT_PROMPT_ELEMENTS
  P9K_LEFT_PROMPT_ELEMENTS=(custom_world1)
  local P9K_CUSTOM_WORLD1='echo world1'
  local P9K_PROMPT_ON_NEWLINE=true
  __p9k_prepare_prompts
  local nl=$'\n'
  assertEquals "╭─%f%b%k%K{015} %F{000}world1 %k%F{015}%f ${nl}╰─ " "${(e)PROMPT}"
}
source shunit2/shunit2

On fix-ctrl-c:
 ❱ ./foo.spec
testLeftMultilinePrompt
prompt_background_jobs:2: bad math expression: operand expected at end of string

Ran 1 test.

OK

On next:
 ❱ ./foo.spec
testLeftMultilinePrompt

Ran 1 test.

OK

So, I think it comes down to __p9k_background_jobs and __p9k_prepare_prompts. On fix-ctrl-c:

If I remove __p9k_prepare_prompts the error is gone but the assetEqual fails
If I add __p9k_background_jobs before __p9k_prepare_prompts the test succeeds and the error is gone -> I'm not satisfied with this solution since the test has nothing to do with background_jobs and presumably comes from the default $RPROMPT. A potential solution could be to remove every irrelevant segments for each test  in setUp for all tests (or in oneTimeSetUp).
If I execute __p9k_background_jobs in a previous test (see below) even if it does nothing. The test succeeds and the error is gone. So there seems to be some kind of shared environment.

function testNothing() {
  __p9k_background_jobs
}

Funnily the test for the segment background_jobs does not depend on __p9k_background_jobs since it sets the variables manually. This basically means this function is never tested, doesn't it?

I don't understand why this is not happening on next but not on fix-ctrl-c. I'd appreciate any ideas and potential ways to fix this. What's even broken? The code or the tests?

OK, writing about this issue really helped. The problem is that on next with the simple __p9k_build_left_prompt the right prompt was simply ignored with a unevaluated $(__p9k_build_right_prompt). This changed when I moved the evaluation to __p9k_prepare_prompts.
My solution would be to include this in every setUp that does not rely on the right prompt:
local -a P9K_RIGHT_PROMPT_ELEMENTS
P9K_RIGHT_PROMPT_ELEMENTS=()

Can someone comment on if this is a good way to fix this or if it should be done differently?

On test is missing a fix:
ssertEquals '╭─ world   ╰─  rworld  ' "$(print -P ${PROMPT}${RPROMPT})"
What it should be (left):
00000000: e295 ade2 9480 1b5b 3339 6d1b 5b30 6d1b  .......[39m.[0m.
00000010: 5b34 396d 1b5b 3130 376d 201b 5b33 306d  [49m.[107m .[30m
00000020: 776f 726c 6420 1b5b 3439 6d1b 5b39 376d  world .[49m.[97m
00000030: ee82 b01b 5b33 396d 2020 e295 b0e2 9480  ....[39m  ......
00000040: 201b 5b31 411b 5b33 396d 1b5b 306d 1b5b   .[1A.[39m.[0m.[
00000050: 3439 6d1b 5b39 376d ee82 b21b 5b31 3037  49m.[97m....[107
00000060: 6d1b 5b33 306d 2072 776f 726c 6420 1b5b  m.[30m rworld .[
00000070: 3330 6d20 1b5b 3030 6d1b 5b31 420a       30m .[00m.[1B.

What it is (right):
00000000: e295 ade2 9480 1b5b 3339 6d1b 5b30 6d1b  .......[39m.[0m.
00000010: 5b34 396d 1b5b 3130 376d 201b 5b33 306d  [49m.[107m .[30m
00000020: 776f 726c 6420 1b5b 3439 6d1b 5b39 376d  world .[49m.[97m
00000030: ee82 b01b 5b33 396d 200a e295 b0e2 9480  ....[39m .......
00000040: 201b 5b31 411b 5b33 396d 1b5b 306d 1b5b   .[1A.[39m.[0m.[
00000050: 3439 6d1b 5b39 376d ee82 b21b 5b31 3037  49m.[97m....[107
00000060: 6d1b 5b33 306d 2072 776f 726c 6420 1b5b  m.[30m rworld .[
00000070: 3330 6d20 1b5b 3030 6d1b 5b31 420a       30m .[00m.[1B.

Not sure why a space is being turned into a linefeed...
diff 1 2
4c4
< 00000030: ee82 b01b 5b33 396d 2020 e295 b0e2 9480  ....[39m  ......
---
> 00000030: ee82 b01b 5b33 396d 200a e295 b0e2 9480  ....[39m .......

Yeah. That test is really fragile. But I wanted to have a kind of integration test. The trade-off is a lot of non-printable characters (like for cursor movement). 🙄
Not sure, why that character changed. I always suspect the editor I am using for changing that..

Sadly, it's not the editor. There is no diff to previous commits

Rebased to next and tried the debugging comments:

I guess this linebreak is the reason. But why is it not there in the "old" version. That is what confuses me. The linefeed was converted into a space. But there should have always been a linebreak, shouldn't it?

I modified the test (91e7b62691e75051991af44e0d131baf22f10ff9) to see if all checks go green after that. This is quite unsatisfying, though. Please give this a thorough look if this is a valid change, @dritter, @bhilburn!
Also, @dritter suggested to add autoload -Uz zsh/parameter in cdbd0621a02a3d2c7727e39f0962a4081855c675. I couldn't see any difference. Should I cherry-pick it anyway?


ISPN-12889 ClusteredLockTest.destroyLock random failures

Make sure to wait on unlocks to complete

https://issues.redhat.com/browse/ISPN-12889

It seems you need to add some BlockHound rules to allow the test to block

Thanks, I will add this in a bit here.
On Fri, Mar 26, 2021, 9:03 AM Gustavo @.***> wrote:

It seems you need to add some BlockHound rules to allow the test to block
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/infinispan/infinispan/pull/9188#issuecomment-808198042,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAE6H3YXFNPSKE4IC2BOMWTTFSAYXANCNFSM4ZZQOHTQ
.

Updated to add block hound exception.

Merged, thanks!


[SMALLFIX] Use static imports for standard test utilities in alluxio.master.file.PermissionCheckerTest

Use static imports for standard test utilities in alluxio.master.file.PermissionCheckerTest.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19131/
Test PASSed.

Merged build finished. Test PASSed.


Variable Modal

Is your feature request related to a problem? Please describe.
Clicking on a variable should open a type-specific editor (such as a calendar editor for a DataTime variable)
This is separate from the issue of creating a variable when editing or creating a template.
We need type specific editors for the following variables:

date
time
date & time (optional timezone offset)
address
monetary amount
boolean
list
string
number
enumerations
accordParty

Potential libraries:

Wojtekmaj
AirBnB

See this invision:
https://projects.invisionapp.com/d/main#/console/16741009/394829377/preview

@irmerk do you know of any frameworks/plugins/whatever you'd call them for type specific form fields like address, calendar, time, etc?

Before a user chooses a variable type
https://projects.invisionapp.com/d/main#/console/16741009/388109653/preview
After a user chooses a variable type
https://projects.invisionapp.com/d/main#/console/16741009/388109652/preview

There could be consideration for tabbing between variables, and on focus maybe the popup appears?


What should Dataset.count return for missing dims?

What is your issue?
When using a dataset with multiple variables and using Dataset.count("x") it will return ones for variables that are missing dimension "x", e.g.:
import xarray as xr
ds = xr.Dataset({"a": ("x", [1, 2, 3]), "b": ("y", [4, 5])})
ds.count("x")
# returns:
# <xarray.Dataset>
# Dimensions:  (y: 2)
# Dimensions without coordinates: y
# Data variables:
#     a        int32 3
#     b        (y) int32 1 1

I can understand why "1" can be a valid answer, but the result is probably a bit philisophical.
For my usecase I would like it to return an array of ds.sizes["x"] / 0. I think this is also a valid return value, considering the broadcasting rules, where the size of the missing dimension is actually known in the dataset.
Maybe one could make this behavior adjustable with a kwarg, e.g. "missing_dim_value: {int, "size"}, default 1.

This is quite confusing and I doubt it's intentional.
I would've expected b (y) int32 3 3 assuming that it would've been broadcast along the reduction dimension.
The final value is the result of
import numpy as np
from xarray.core.duck_array_ops import isnull

np.sum(np.logical_not(isnull(ds.b.data)), axis=())
# np.sum([True, True], axis=())

What happens when you call a ufunc with an empty axis tuple? I bet this is just casting bool to int.

What happens when you call a ufunc with an empty axis tuple?

This should also happen with all other ufuncs then?
I guess most of them just work, like mean, sum etc.

We discussed:

dropping variables without the dimension
Return ds.sizes["x"] by broadcasting b along x

For the other reductions
import numpy as np
import xarray as xr

from xarray.core.duck_array_ops import count

ds = xr.Dataset({"a": ("x", [1, 2, 3]), "b": ("y", [4, 5])})

for func in [np.nansum, np.nanprod, np.nanmean, np.nanvar, np.nanstd, count]:
    print(f"{func.__name__!s}({ds.b.data}, axis=()) = {func(ds.b.data, axis=())}")

gives
nansum([4 5], axis=()) = [4 5]
nanprod([4 5], axis=()) = [4 5]
nanmean([4 5], axis=()) = [4. 5.]
nanvar([4 5], axis=()) = [0. 0.]
nanstd([4 5], axis=()) = [0. 0.]
count([4 5], axis=()) = [1 1]

I guess the output for nansum, nanprod doesn't match what you would get by broadcasting along the absent dimension.

I think that changing the behavior of sum is quite a large breaking change.

Another option is to add an option: missing_dim: "raise", ignore" or "broadcast".
The default then would be ignore, which is the current implementation.
But for workflows of variables that are either DataArray or Dataset, this argument should be added to DataArray.sum/count/prod as well?


Update docs for feat: add catalog name resolution for postgres and http interface

A document change request is generated from
https://github.com/GreptimeTeam/greptimedb/pull/810

No longer needed because we don't expose catalog concept to end user


Issue with the detailed balance (Ben Li)

Eq. (2.62) - (2.64) in chapter "the advection-diffusion equation". The first equation in Eq. (2.62) seems wrong, since it doesn't satisfy detailed balance.

In (2.63), I don't understand why d(mu)=-d(delta mu), and I don't know how to derive (2.64) using (2.63).

Reply: I improved the derivation, see the new version. Indeed eq.2.62 was wrong. Still a passage in the derivation must be clarified (in red).


Dialogue should appear on item query

If item query is passed in, then we should make sure to bring up the clue window and then on close redirect to the page with a blank query.

resolved


Change "zipjs" -> "zip-js" to match name field

Fix module name in package.json: "zipjs" -> "zip-js"
This pull request is Finalized
Related issue(s) and/or pull request(s)
#165
Test cases, sample files
Additional information

Didn't put link to related issue in commit.  Will make a new PR.


Extension Tiled Assets

Related Issue(s): #
None
Proposed Changes:

Additional extension for tiled assets

PR Checklist:

[x] This PR has no breaking changes.
[ ] I have added my changes to the CHANGELOG or a CHANGELOG entry is not required.
[ ] API only: I have run npm run generate-all to update the generated OpenAPI files.

TODOS:

[ ] check/review alignment with other (especially eo) extensions
[x] add examples
[x] add schema
[x] align with OGC two dimensional tile matrix set specification

This work-in-progress but any comments are very welcome.
@Schpidi @ungarj

In general this seems like a good idea and thanks for pushing it forward. Personally, I would like to see this better aligned with STAC. At the moment it looks like it was simply converted from the OGC spec. For example, there is a title field, but title is already available in STAC and could probably be removed here. Similarly with abstract, keywords, identifier, type etc. Bounding Boxes could be aligned as well, it doesn't make much sense to introduce a new structure for bounding boxes if there's already one in STAC. Some fields probably need better descriptions. For example, what is a well-known scale set? Also, what fields are actually required for search and discovery and what fields are more for other use cases?

In general this seems like a good idea and thanks for pushing it forward. Personally, I would like to see this better aligned with STAC. At the moment it looks like it was simply converted from the OGC spec. For example, there is a title field, but title is already available in STAC and could probably be removed here. Similarly with abstract, keywords, identifier, type etc. Bounding Boxes could be aligned as well, it doesn't make much sense to introduce a new structure for bounding boxes if there's already one in STAC.

It is true, this extension directly uses the JSON schema defined for the TMS spec. I knew this was likely a point of discussion, but I initially decided to go this route first.
I think I could slim down the schema and to better align with the STAC way to do things.

Could the dimensions be aligned with the data cube extension?

Yes, I think this is feasible. But I think it would be bad to directly reference the datacube extension constructs.

Some fields probably need better descriptions. For example, what is a well-known scale set?

Yes, this could be extended. But I think I will wait until the structure has somewhat settled. But on the other hand I don't want to retell the TMS spec. Do you think it is okay to reference the TMS spec at some vital points?

I made a review with specific comments so it is easier to discuss specific points.

Yes, I think this is feasible. But I think it would be bad to directly reference the datacube extension constructs.

We may also align the data cube extension. In the end it would be great to just not reinvent the wheel multiple times.

Yes, this could be extended. But I think I will wait until the structure has somewhat settled. But on the other hand I don't want to retell the TMS spec. Do you think it is okay to reference the TMS spec at some vital points?

I like self-contained extensions, but if it is too complex to add here, links are fine, I think.

I am interested in implementing the proposal here and would like to help make changes so that it could make it in the specification as an extension as we are in the process of implementing tiled assets in some of our STAC services at Azavea.
What's the best way to help out with this proposal? Do you have plans to update this PR @constantinius or should I open a new one?

Thanks for helping out, @notthatbreezy, I am indeed in a bit of a struggle to find the time to further develop this spec extension.
What has become clear from the reviews by @m-mohr is that it is to complex and too much leaning on the TMS side instead of being more aligned with STAC. Also it seems to be too unclear what is actually necessary and what is not.
As a first step, I would slim down the spec and remove fields that are not useful (such as these maligned type fields, title, abstract, etc).
It was also requested to not duplicate the Dimension objects from the Datacube extension. So, imho it would be best to completely remove them from this spec and make a reference the dc spec objects instead.
One important thing last: the tile matrix set should probably not be included in the stac item itself, but be referenced from somewhere else. (Either a URL or from the collection/catalog). Then there is the thing about the wellKnownScaleSets which are basically predefined tile matrix sets in the TMS spec. This would be the third option. And then: in a tile set referencing a tile matrix, one can specify the actual limits within this tile matrix set. e.g: specifying the actual zoom levels and actual tile limits.
I think this is demonstrated best by an example: the Web Mercator Quad TileMatrixSet definition  defines a global tile grid with 24 tile matrices (zoom levels). If you have a dataset in only a part of the world, you can still align it to the global grid, but limit the zoom levels and then limit the tiles in that matrix of the ones actually intersecting with the data.
This is concept is not yet reflected in any way in the extension and would be rather important.
I think it is better to stay on this PR, but contributions are very welcome!
This turned out to be more  than expected. I hope we can collaborate on this (maybe on gitter?). Please let me know what you think.

@cholmes @m-mohr
I updated the schema and I tried to better explain the concepts behind the tiled assets. Please have a look at that and let me know if you think that there is something missing.

Just a minor thing and mostly a matter of taste, but why not use the prefix tiles or tile for the extension? tl seems not very intuitive.
Instead of for example tl:tile_matrix_sets it would simply be tiles:matrix_sets or tile:matrix_sets.

@m-mohr Thats a good point. I'll change it to tiles.
@notthatbreezy Have you had a chance to look at this PR? Does it also suit your requirements?

Done. The CI checks seem to be unrelated.

I see your point, tiled items are in of itself some kind of collection of tiles. The issue remains that we do not want an actual STAC Item representing a simple tile (due to the sheer number and repetitive nature of the metadata). When using asset_templates on Collection level this would mean that the collection would be devoid of Items, which is quite a different concept I think.  But I agree, that Items with tiled assets are somewhat between Items and Collections. But I would argue that they are closer to Items than Collections.

@m-mohr
I updated the header and schema to allow tiles:tile_matrix_set descriptions on Collections and Catalogs as suggested in the Readme. My json-schema skills are questionable at best, so please have a look at it and tell me if it is sound.
Thanks for the feedback, really appreciated!

This looks good to me - my own requirements have shifted slightly (we're only concerned with visual tiles at the moment so have shifted to the OGC API Tiles Spec) so this isn't an immediate need for us at the moment, but I think this would be useful for us when the need arises. I think the extension is thorough and will be useful.

So it has been merged and thus is scheduled to be released with the next version. :-)
Thanks @constantinius !

Thanks a lot! :-)


Updated Discover to have acceptCoBranded flag for the range "622126-622925"

Fixes #80

@aloisdg All the tests pass in my local. Can you help me here?

I will look into it later on. Have a nice evening :)


New Design for godocs

Is there any possibility for a new design (UI/UX)
and maybe capabilities for custom templates?
Something like: https://hexdocs.pm/ecto/Ecto.html

Is there any possibility for a new design (UI/UX)
Do you have UI/UX complaints, or just wanting a refresh? My guess is that unless something is broken the design will likely stay as is or track patterns from the golang.org.

and maybe capabilities for custom templates?
#400 discusses some templating using a custom assets folder and bespoke html templates.

See also https://blog.golang.org/go-brand

This is happening as part of https://github.com/golang/go/issues/33654. /cc @julieqiu


⚠ Bofrost.es has degraded performance

In da77f98, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1523 ms

Resolved: Bofrost.es performance has improved in 07482c6 after 17 minutes.


Add support for LinearOperator to cg and gmres in cupyx.scipy.sparse.linalg.

This PR adds support for LinearOperator to cupyx.scipy.sparse.linalg.cg and cupyx.scipy.sparse.linalg.gmres to improve compatibility with SciPy.
Thanks to @venkywonka for adding LinearOperator (https://github.com/cupy/cupy/pull/4258)

Jenkins, test this please.

Jenkins CI test (for commit 671c64623de8a3ec591096e5f2f73bed34156afe, target branch master) succeeded!

LGTM!


observe cancellation sends different events

when a particular block size is pre-selected, copper sends observation requests with an Observe (=0), an Uri-Path and a Block2 option, but the cancellation message only with Observe (=1) and Uri-Path.
this is in conflict with draft-ietf-core-observe-16 3.6, which says about eager cancellation of an observation: "which has the Token field set to the token of the observation to be cancelled and includes an Observe Option with the value set to 1 (deregister).  All other options MUST be identical to those in the registration request, except for the set of ETag Options". please consider saving the original request in order to satisfy the draft's requirements.
the issue can be worked around by setting "Block late negotiation".

the same issue also shows up when the debug option Uri-Host is set; in that case, i don't see any workaround.

Yes, these updates still need to be included. Feel free to provide pull-requests ;)
Another question is what actual problem you run into at the moment. Hosting environments and unforeseen options will be a problem, but otherwise I cannot see technical difficulties at the server side at the moment. Do you have input on that? (It would be good to identify the affected options.)

On Fri, Feb 13, 2015 at 06:50:39AM -0800, Matthias Kovatsch wrote:

Another question is what actual problem you run into at the moment.
Hosting environments and unforeseen options will be a problem, but
otherwise I cannot see technical difficulties at the server side at
the moment. Do you have input on that? (It would be good to identify
the affected options.)

i'm running into it as of developing the aiocoap library, which
ships a proxy (both forward and reverse) with observation support.
the issue is that i match the incoming requests by their cache-key
(only dropping the observe option). if a client now subscribed with a
block option but tries to unsubscribe without it, it appears to the
server like an unsubscription request that does not match any active
observation; it reacts (correctly in my opition) with a Bad Option
response -- not because it does not recognize the client's subscription
(which gets cancelled properly as it is matched by endpoint and token),
but because the proxy has no representation of the resource requested in
the unsubscription message.
(if the proxy were configured to deal with blockwise transfers, the
block option should be stripped out before calculating the cache-key,
but it seems practical to me to have the proxy running without blockwise
support by default).
does this make sense to you?

Yes, I can see the issues in proxies and they are also the rational behind the requirement in -observe. There is no question we need to fix this in Copper.
From your description, I guess it would be more robust to better separate token matching (used for the cancellation) from the cache lookups. For observe, you need to provide the full repesentation in your cache anyway (as a whole or all blocks), since the client is likely to fetch the other blocks after a notification (which is always only block no. 0). I do not think we actually have a consensus or best practices how to deal with blockwise transfers in caches. In Californium I would go (proxy development is currently on hold...) for always caching the whole response. This also allows also answering to larger block sizes.

in aiocoap, token matching and cache lookups are completely separated -- but the issue still surfaces, because the cancellation information affects the established observation (whose destruction is even withheld long enough so the original request could still be answered from it), but the cache-key that's looked up for request satisfaction was never queried.
as far as blockwise is concerned, the default aiocoap proxy does not do real caching, only for observations (where it has to cache lest to send "still interested" requests on every new incoming subscription).

I fixed it now in such a way that all debug options and the Block option are also added to the proactive cancellation.
Since it is (in my opinion) not a good practice to evaluate these option on the server side for cancellation, one can alter the debug options before cancellation to test server robustness.


Raw.save logs too many "Overwriting existing file." infos

MWE:
# %%
import os
from pathlib import Path
import mne
import numpy as np

sample_dir = Path(mne.datasets.sample.data_path())
sample_fname = sample_dir / 'MEG' / 'sample' / 'sample_audvis_raw.fif'
raw = mne.io.read_raw_fif(sample_fname, preload=True)

fname = Path(os.path.expanduser("~")) / "Desktop" / "try-raw.fif.gz"
raw.save(fname)
raw.save(fname, overwrite=True)

Output after second .save call:
Overwriting existing file.
Overwriting existing file.
Overwriting existing file.
Writing /home/stefanappelhoff/Desktop/try-raw.fif.gz
Closing /home/stefanappelhoff/Desktop/try-raw.fif.gz
[done]

These are two Overwriting existing file. too many, aren't they?

Comes from two calls in .save:
https://github.com/mne-tools/mne-python/blob/9dff61616bfc9ca411df578a4f3b963de8ac023c/mne/io/base.py#L1430
https://github.com/mne-tools/mne-python/blob/9dff61616bfc9ca411df578a4f3b963de8ac023c/mne/io/base.py#L1458
And then one additional one in _write_raw:
https://github.com/mne-tools/mne-python/blob/9dff61616bfc9ca411df578a4f3b963de8ac023c/mne/io/base.py#L2168
potentially an accident from #9613 @hoechenberger ?


Please add a license to this repo

Could you please add an explicit LICENSE file to the repo so that it's clear under what terms the content is provided, and under what terms user contributions are licensed?
Per GitHub docs on licensing:

[...] without a license, the default copyright laws apply, meaning that you retain all rights to your source code and no one may reproduce, distribute, or create derivative works from your work. If you're creating an open source project, we strongly encourage you to include an open source license.

Thanks!

Good idea. I'm much more of a mathematician than a programmer, so I didn't even think about the licensing. I chose the MIT license as it seemed to be the simplest, most permissive, and most aligned with how things work in mathematics.


Menu icons in collapsed menu start should have tooltip on hover

When you collapse menu, it's hard to understand what menuitem does each icon represent. We need to add some hover effect or just title.

MIssing thing is, how to load the login page.

Create the new module. app.page.login, file Login.module.js file,
create controller Login.controller.js file.
create services Login.Services.js file.
inculde this file in Index.html page.
it loads the Login page in UI-View under the content page,

ISSUE- Side bar and top navigation appreas, ideally login page should render fully across the page.
Solution.

Create another root page or main page.

if Index.html is root page then main section should be moved to main.html file

Create folder Main , create app.page.main mdoule in Main.module.js file, same way add controller and services file.

Inject the app.page.Main module in app.page module and

Include this new .js files in Index.html files.

include UI-VIEW in index.html file,(if it is not there un named ui-view). it will load the login page as full page.

on click of login Page user State.go('Main) state to go to main.html file which is having main section with Side,Page top, top content and another UI-VIEW. which will load the Side bar, page top, and content page, on click of menu, it should open the views in content page.

ISSUE- it opens the views with parent UI-VIEW(index.html) not in child view(Main.html).
There are two UI-Views (un named). one in Index.html page, another in Main.html page.
how to load the views on click of side bar in content page(main.html). Why it opens the views with Index.html page. Its really may be a bug or there way to load the content inside child view(main.html).
total there are 3 UI-VIEW, a) Index.html for login page to render, b) main.html to render the content,c) which loads submenu iteams( abstract true).
In Blur admin there are two UI-VIEW which works fine, why it doesn't works with 3 UI-VIEW.


Print HTTP routes on deploy

Fixes #566.
@michelleN I'm afraid I can't yet test this because I can't get the Hippo-Nomad stuff working correctly yet.  I will try to get that sorted in the morning.  If you have time do feel free to give this a quick test - I think it is right but am not sure in exactly what form Hippo returns the channel domain, so that at least is quite likely to be wrong!  Sorry!

Seems to work! Just verified against the local installer:
$ spin deploy
Successfully deployed application spin-hello-world version 1.0.0!
Application is running at spin-hello-world.spin-hello-world.local.fermyon.link
Available Routes:
  hello: http://spin-hello-world.spin-hello-world.local.fermyon.link/hello
    A simple component that returns hello.


Uncaught Error: dlopen(/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/...

[Enter steps to reproduce:]

...
...

Atom: 1.55.0 x64
Electron: 6.1.12
OS: macOS 11.2.3
Thrown From: Atom Core
Stack Trace
Uncaught Error: dlopen(/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keyboard-layout/build/Release/keyboard-layout-manager.node, 1): image not found
At electron/js2c/asar.js:138

Error: dlopen(/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keyboard-layout/build/Release/keyboard-layout-manager.node, 1): image not found
    at process.func (electron/js2c/asar.js:138:31)
    at process.func [as dlopen] (electron/js2c/asar.js:138:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:828:18)
    at Object.func (electron/js2c/asar.js:138:31)
    at Object.func [as .node] (electron/js2c/asar.js:147:18)
    at Module.load (internal/modules/cjs/loader.js:645:32)
    at Function.Module._load (internal/modules/cjs/loader.js:560:12)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:16:16)
    at customRequire (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:1:697035)
    at get_KeyboardLayoutManager (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:14:2798543)
    at get_manager (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:14:2798645)
    at Object.getCurrentKeyboardLayout (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:14:2798771)
    at e.keystrokeForKeyboardEvent (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:14:1058046)
    at KeymapManager.keystrokeForKeyboardEvent (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:11:1228079)
    at KeymapManager.handleKeyboardEvent (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:11:1226064)
    at WindowEventHandler.handleDocumentKeyEvent (/private/var/folders/jr/t06cf4v13xv19fv58ltdvcmm0000gn/T/AppTranslocation/FA41D628-F016-4168-82C8-EF69805F2300/d/Atom.app/Contents/Resources/app/static/<embedded>:11:284879)

Commands

Non-Core Packages

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/21285. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


Make client_id optional

PR for #51 - @witnessmenow this is still work in progress. I still need to remove code duplication in requestAccessTokensDeviceFlow for example
Are you opiniated about code styling? I noticed the prettier plugin formatted the code - if you have another formatting I should follow, please let me know.

The device flow method seems to be aspirational still @hansmbakker is it? Just judging by your link in #51
https://community.spotify.com/t5/Spotify-for-Developers/Device-Authorization-Grant-authentication-flow-for-custom/m-p/5485468#M7570
It seems like it's not something that is available yet, am I wrong?
It does seem like the PKCE method might be possible, and SHA encoding is probably requires device specific code, so maybe only available on ESP32 for example, but I'd imagine that satisfies the majority of users


Keeps on loading in the browser

I have added the Android Debug Database in my gradle file. The generated URL keeps on loading and doesn't show the content of the database.

I am using Splunk Mint bug tracking library (compile "com.splunk.mint:mint:4.4.0") to find the crashes in the released app builds. After disabling the library in Gradle, this library start working properly.
It will be great, if you could look into this issue and fix it.

Great, I will check it.


Remove Windows support

Context
The v8go library currently supports binaries for mac, linux, and windows. The last release of this library with an updated version of v8 was in May. We have been making efforts to automate the upgrade process here so that we can ship releases more regularly as patches are released/CVEs are found and fixed. However, we've been unable to merge any of those upgrade PRs opened by the automation @gustavocaso added because it fails to build the window binary on later versions of v8.
The windows binary is built using MinGW and relies on patches in a different repository to make the v8 binary compile. These patches have not been updated for later versions and so they fail to apply on upgrade attempts.
Additionally, there seems to be incompatibility with enabling Internationalization/ICU support with the existing windows build process because of what appears to be a reliance on something provided by VSCode.
As such:

We cannot ship releases of v8go because we cannot build the windows binary and do not want the versions of mac/linux to be out of step with windows. The last release with a v8 update is from May. We are 6 minors behind.
We cannot ship a release of v8go with ICU enabled in the v8 binary because we cannot build the windows binary with it using the current build process.

Proposal
This PR removes support for windows from v8go as a result of the above issues. If a user of the library can contribute a fix to the above issues now, great. We tried asking in the #v8go gophers channel a couple weeks ago but did not get much of a response. Until such time as a user is able to contribute the changes needed to make the windows binary build on recent versions of v8 & with intl support enabled, we can remove it as a blocker allowing us to cut releases of v8go with the latest v8 update for mac and linux, as well as with intl support.

Probably should also delete https://github.com/rogchap/v8go/tree/master/deps/windows_x86_64

Tests on 1.16.8 windows-latest Expected — Waiting for status to be reported (Required)
Tests on 1.17.1 windows-latest Expected — Waiting for status to be reported (Required)

@rogchap looks like these will need to be removed from the required checks before merging.


sanity check in insert

yeah, it shouldn't be called. Might as well panic in any case. This is just here in case we switch to an iterative approach, and the code will be incorrect in that case anyway. I'll mull it over.

Closing, this is mixing Insert and Update.


Log in with Google button is missing

Windows 10,
Log in with Google button is missing, can't log in.
Maps key is entered.

Read the patch notes. it was removed


Applespi: Package and Nixos flag

Motivation for this change
Applespi is required on newish macbooks for keyboard and trackpad to work. This adds a package for the driver and a flag to enable it.
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

I think it'd be interesting to have either this or #57419 merged by 19.09, so that newer apple laptops can be made more reasonably usable. Though, I think this PR is more appropriate, as it also adds an option to make it more usable.
Adding this one to the 19.09 milestone; the conflicts are likely trivial (module-list, all-packages), meaning that any nixpkgs contributor can fix it by that time. Hopefully one who can test the change.

apparently as of the upcoming v5.3 kernel release, the driver has been upstreamed and merged into the kernel? making this PR partially unnecessary as long as it is built into the kernel as a module.
https://github.com/torvalds/linux/commit/038b1a05eae6666b731920e46f47d8e2332e07ff#diff-21f587dcb9b25ed3dcff9dade6b0aa67

No worries, I was triaging random PRs.
At least the current status of this PR is known, anyone wanting to take it up now knows.

I think this can be closed as #71812 landed in master.

Yep, I agree with you. Applespi seems to have landed in the mainline kernel.


A little bit confusing instructions?

The PAC file uses hardcoded location (DE), if you want to use other location just download the file, change the hostname (available hostnames are {ca,de,us}.opera-proxy.net), change your browser configuration, and you should be ready to go.
^^^
Just want to clarify it: what does "change your browser configuration" mean?
If I edit pac.js and put US or CA instead of DE, what shall I do next?
Save the edited file locally and point Auto-detect to something like file://..../pac.js  ?

Yes.


implement decision step

Something like this:  source: https://drakonhub.com/how-to-flowchart

[x] decision rendered - single outcome, treat as a regular command step
[x] chain: command -> decision -> command, fixed layouting
[x] chain: command -> decision -> decision * N -> command, fixed layouting
[x] automatic layouting
[x] step dimensions expressed as cell proportions (already done with command steps)
[x] automatic custom connections
[x] decision outcome connections to next steps should be decorated with yes/no

where N is an arbitrary, finite, positive number, say in range 0-10.

basic rendering done:

current state:

looks like we're taking a step back, but in fact we are making a big leap forward:

no connections, true, but:
automatic layouting
steps no longer know their coordinates
steps named automatically based on their origin coordinates

current state:

current state:

Now connections require some corrections - it is possible for tree branches to overlap, e.g.:
steps :: Tree Step
steps =
    Node1
        (Main.Start)
        (Node2
            (Node1
                Main.Command
                (Node2
                    (Node1
                        Main.Command
                        (Leaf Main.End))
                    Main.Decision
                    (Node1
                        Main.Command
                        (Leaf Main.End))))
            Main.Decision
            (Node2
                (Node1
                    Main.Command
                    (Leaf Main.End))
                Main.Decision
                (Leaf Main.End)))

Implementing collision detection and coordinates positioning now.

Collision detection and avoidance works, now I need to resolve two things:

new data type for uniqueCoordinates - it feels very inefficient
new, more flexible mechanism of connecting steps adapting to shifting steps

Almost there, now just cosmetics left.

Done:


fix: support passing file:// URI when using custom-compile SQLite

I'm trying to use better-sqlite3 with an SQLite that is compiled with SQLITE_USE_URI=1:
const db = SQLite(`file:///foo/bar?vfs=myfs&mode=ro&immutable=1`);

This, however, doesn't work right now, since there's an erroneous assertion in the database creation.
With this patch, I can successfully connect to the database.
References https://github.com/WiseLibs/better-sqlite3/issues/483

It might solve your problem in a hacky way but it does not feel like the correct solution since these things are not mutually exclusive. Checking if the directory exists makes sense regardless if you're using file:// or not.
> require('url').fileURLToPath('file:///foo/bar?vfs=myfs&mode=ro&immutable=1')
'/foo/bar'

I don't know what the right solution is, but I feel like the current logic conflicts with the use of file://. E.g. it allows some of the options like readonly. So there needs to be a "URI" mode that forbids other options or does some more sanity checks.

I don't know what the right solution is, but I feel like the current logic conflicts with the use of file://. E.g. it allows some of the options like readonly. So there needs to be a "UR mode" that forbids other options or does some more sanity checks.

@Prinzhorn I agree that proper URI support would need to address these points, and this is discussed to some extend in https://github.com/WiseLibs/better-sqlite3/issues/483
However, some of the extensions actually rely on the "fake URI", like this one: https://github.com/mlin/sqlite_web_vfs:
const db = SQLite(`file:///__web__?vfs=web&mode=ro&immutable=1&web_uri=${encodeURIComponent(dbUrl)}`);

Do you think it'd be possible to have some escape hatch for these kind of use cases?
Thank you for your review.


Add render macro for rendering directly to Clerk notebook

(This macro was inspired by @mhuebert 's work on https://inside-out.matt.is/.)
This PR adds a macro that lets you write concise code in the case where you want to render some Reagent code directly in the notebook.
This comes up in cases where, for example, some other component is modifying a ratom and you want to render some view of it without any JVM input.
I'm not sure on the final name. The original macro from @mhuebert was called cljs.

Nice idea! This more concise version
(defmacro cljs [& body]
  `(clerk/with-viewer '(fn [_] (do ~@body)) nil))

works against latest main branch, where you no longer need v/html calls if you return a vector.

Excellent. Cc @mhuebert who had a different version against the latest master, with a few tweaks to fix some issue with interactive components. I need to upgrade too to give this simpler version a try.

This was the final @mhuebert edition from inside-out:
(defmacro cljs [& exprs]
  `(clerk/with-viewer
     {:render-fn '(fn [_# _#]
                    (nextjournal.clerk.render/render-reagent
                     [(fn []
                        (let [result# (do ~@exprs)]
                          (if (vector? result#)
                            result#
                            [nextjournal.clerk.render/inspect result#])))]))}
     {}
     nil))

With #357 we have eval-cljs based on sci now. For regular cljs, I'd envision being able to switch between sci, regular cljs (and potentially others) but without adding to the api so I think this is best kept in a library for now.


Password in clear text in 'Update configuration settings'

[Version]
V16.1 GA1
[Date]
2016/08/07
[Test category]
Creative
[Description]
The content of the parameter 'SSO_SIGNING_KEYSTORE_PASS' in written in clear text.
[Steps to reproduce]
Install CTMG in SAML mode.
Open Portal.
Login as Platform Admininstrator
Open 'Update configuration settings' section
Check the entry 'SSO_SIGNING_KEYSTORE_PASS'
[Expected result]
The content of the password field is displayed as '******'
[Observed result]
SSO_SIGNING_KEYSTORE_PASS "changeit"
[Details]
See above;
[Messages and logs]

This is a security issue....

The UI now uses a password input field for passwords. Readonly passwords are shown as five stars.


Replace community action with the GitHub API

This is needed as some repositories issue releases in a non-deterministic orders. However, so far all of the maintainers have stuck to using the latest tag correctly. The prior action did not take this into account.
Since this is a CI change, it is untested in a workflow but works when I run the curl request locally.

Much better! Thanks @lucabello


Update sbt-sonatype to 3.8

Updates org.xerial.sbt:sbt-sonatype from 3.7 to 3.8.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

Codecov Report

Merging #707 into develop will increase coverage by 0.05%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #707      +/-   ##
===========================================
+ Coverage    89.52%   89.57%   +0.05%
===========================================
  Files          112      112
  Lines         8943     8943
  Branches       329      329
===========================================
+ Hits          8006     8011       +5
+ Misses         937      932       -5

Impacted Files
Coverage Δ

...in/scala/com/twitter/algebird/scalacheck/Gen.scala
90.9% <0%> (-9.1%)
:arrow_down:

.../main/scala/com/twitter/algebird/Successible.scala
87.5% <0%> (-8.34%)
:arrow_down:

...main/scala/com/twitter/algebird/monad/Reader.scala
50% <0%> (-5.56%)
:arrow_down:

.../src/main/scala/com/twitter/algebird/ExpHist.scala
95.55% <0%> (-1.12%)
:arrow_down:

.../main/scala/com/twitter/algebird/BloomFilter.scala
94.69% <0%> (+0.44%)
:arrow_up:

.../main/scala/com/twitter/algebird/JavaMonoids.scala
79.03% <0%> (+1.61%)
:arrow_up:

...src/main/scala/com/twitter/algebird/Interval.scala
80% <0%> (+3.47%)
:arrow_up:

...ala/com/twitter/algebird/ApproximateProperty.scala
82% <0%> (+10%)
: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 bf03a28...c3f6f99. Read the comment docs.


Any plan to add routeSettings as a parameter?

Currently, there is no way to pass routeSetting which is causing screen name null in analytics tools, and it is the best practice to add route Settings to all kind of overlays

Hi @Praveena0989
Update 0.10.0 with routeSettings added to PullDownButton and showPullDownMenu.


fix: correct syncToInternal call in TZDateMini prototype

Sorry I missed your PR, but I fixed it in v1.0.1 while working on something else 😔 I appreciate your PR and mentioned you in the change log.


Unwinding assertion failures do not clearly indicate which loop is involved

Check 67: .unwind.0
         - Status: FAILURE
         - Description: "unwinding assertion loop 0"

SUMMARY:
 ** 1 of 67 failed (66 undetermined)
Failed Checks: unwinding assertion loop 0

What function? What loop?
I vaguely recall we used to see that information, so something changed.
Raw cbmc output:
** Results:
[.unwind.0] unwinding assertion loop 0: FAILURE

Probably a bug on our side, I did a quick test with cbmc:
** Results:
test.c function main
[main.unwind.0] line 3 unwinding assertion loop 0: FAILURE


Status option does not work as intended

Hi,
I think the status option does not work as you would expect for the last and error statuses.
When those statuses are displayed, they should be kept on the screen after the events have ended.
Currently, you wouldn't even see the last status after the last page has loaded because it is immediately hidden after the content is appended/loaded.
Maybe I'm wrong, but I just wanted to make sure :)
Thanks!

Same here too. it add display: none;

Thanks for reporting this issue. I'm tracking this bug in #706


Render Mermaid Charts in the Markdown Section of Jupyter notebook

It would be amazing if I could view mermaid charts in a markdown block when clicking the checkmark to view the rendered markdown. Currently, if I type in this mermaid block into a jupyter markdown block:
```mermaid
graph LR
A --> B
```
it just displays this:
graph LR
A --> B
instead of
graph LR
    A --> B

My workaround to see mermaid charts for now is:

Install the Markdown Preview Mermaid Support extension

Click Ctrl-Shift-P to open the Command Palette

Type in "Preview Mermaid" and press Enter to open a panel on the right

Click inside the mermaid block you want to visualize, and then the chart shows up in the Mermaid preview on the right

Would be cool if those steps could be replaced with a supported render of mermaid charts, possibly in combination with the above mentioned vscode extension!

We try to keep VS Code lean and we think the functionality you're asking for is great for a VS Code extension. Maybe you can already find one that suits you in the VS Code Marketplace. Just in case, in a few simple steps you can get started writing your own extension. See also our issue reporting guidelines.
Happy Coding!

Tracked by https://github.com/mjbvz/vscode-markdown-mermaid/issues/158


New Version Should Be Uploaded to NPM

Hi,
I see you have fixed the new Buffer issue 8 months ago. Unfortunately we could not use it because NPM package is still old. Can you create a new version with that change?

Sure. Thanks for noticing that.

Done

Thanks 👍


Implement A.4 Changes in Task 5 Artifact

Implement A.4 Changes in Task 5 Artifact Including Changes to Tables A-4-1 and A-4-2.

Document tickets were closed. Use this issue to track implementation.


You might find my bachelor's thesis interesting

Here's a link: http://taktoa.me/eqsat/Thesis.pdf
Unfortunately, my implementation didn't get very far. UIUC's senior thesis credit grades based on what you write, not what you implement.

Looks awesome at a first glance! It's on my todo list. I'll get back to you later with more details.

Hey Remy! I have not forgotten about this! I've only been able to take a skim so far, but it looks really cool.
I'm going to close this issue for now, but I've just opened the discussions feature on this repo if you'd like to start a thread on your work (even if just to show it off!). In particular, I'd love to hear if you think it would be worthwhile to bring any of the innovations from your work into egg

great! I actually think I might try using egg for something work-related in the nearish future (maybe a few months from now), so I'll possibly be getting more involved in the repo


Flaky test: Project .onDidChangeFiles() reports filesystem changes within project paths

This test sometimes fails:
Project
  .onDidChangeFiles()
    it Project .onDidChangeFiles() reports filesystem changes within project paths
      timeout: timed out after 60000 msec waiting for promise to be resolved or rejected

Rebuilding usually resolves the failure.

Example failing build: https://github.visualstudio.com/Atom/_build/results?buildId=42865&view=logs&jobId=4fbced83-508e-5fe0-c978-5c71ec0fc506&taskId=8ab1786f-98fc-5bc4-a8d0-c08e0675c13a&lineStart=2419&lineEnd=2422&colStart=1&colEnd=89

Another one: https://github.visualstudio.com/Atom/_build/results?buildId=42869&view=logs&jobId=4fbced83-508e-5fe0-c978-5c71ec0fc506&taskId=8ab1786f-98fc-5bc4-a8d0-c08e0675c13a&lineStart=2419&lineEnd=2420&colStart=1&colEnd=1

Another one: https://github.visualstudio.com/Atom/_build/results?buildId=42909&view=logs&jobId=4fbced83-508e-5fe0-c978-5c71ec0fc506&taskId=8ab1786f-98fc-5bc4-a8d0-c08e0675c13a&lineStart=2420&lineEnd=2424&colStart=1&colEnd=1

Another one:
https://github.visualstudio.com/Atom/_build/results?buildId=42918&view=logs&jobId=4fbced83-508e-5fe0-c978-5c71ec0fc506&taskId=8ab1786f-98fc-5bc4-a8d0-c08e0675c13a&lineStart=2419&lineEnd=2420&colStart=1&colEnd=1

We're one step closer to figuring out this flake. The test includes 3 promises, and at first, it wasn't clear which promise was failing to resolve. @as-cii and I pushed up a build with some debugging logic in it, and we have isolated the problem to this part of the test:
https://github.com/atom/atom/blob/c947c8005aff1de0586a757684158c1cef051efb/spec/project-spec.js#L1108


Add a wrapper for measures for use with missing values?

It seems to me this would be the simplest way to implement this kind of support, no?
Something like:
y = [1.0, 1.9, missing, 3.7]
yhat = [1.1, 1.8, 3.0, missing]
loss = SkipMissing(LPLoss(p=1))
julia> loss(yhat, y)
[0.1, missing, missing] # length preserved

and we could define all the aggregation methods (Sum, Mean, etc) to skip missing values.

Nah, on looking further into this, I think it is simpler to review the measures that don't automatically work for missing values already and fix these.


Add two more Slurm states which may be reported during normal operation

We have seen some intermittent failures in the Slurm tests, which appeared to be caused by the scheduler module encountering unexpected states when it looks at the job queue.
This change adds two more non-error states which are uncommon, but may be reported during normal Slurm runs.

Good point - I've updated the logic to track all possible Slurm job states, split into 'active' and 'inactive' categories.


Reimplement asynchronous software serial interface

Up until now, the revised Rust implementation of riftd has used an antiquated, poorly-documented
framework called mio for doing asynchronous networking on
serial devices. Since its beginning, I have been eyeballing a related framework called Tokio
as a better-supported successor to mio. As of this writing, Tokio seems poised to receive much more
love from core Rust (especially with futures-rs moving
forward so well).
Given how central the ability to do non-blocking handling of network I/O over this serial interface
is to the performance and scalability of this technology, I think we should reimplement this interface
using this newer standard for doing async I/O in Rust.
Overall the ideal async serial interface would take the given serial port (e.g. /dev/tty0) and
provide a full-duplex channel pair representing both incoming and outgoing bytes.

Pushing this back as the Rust community continues to be fickle around the matter :-1:


Make printing articles make sense

Reduce size of font to 14pt, make article images/screenshots full-width (since they will not be opening in a modal).

Looking better. I'm wondering if we can make the images print larger, maybe sit above or below the instructional text.

Alright didn't realize how easy it actually is to style 'printer' specific documents, but it's just a matter of creating a separate css file.

@billymeinke Articles are now printer friendly.

Looks much better. Thanks @enriquezm


Support for multiple columns?

Hi, I wonder if its possible to add more than one image data column? Right now we have image and caption. What should we do if we want to also add more columns (other than caption)?

check save_additional_columns parameter

indeed, you can use save_additional_columns


New schematron rule: origDate

Each origDate should always have one or more of the relevant dating attributes: @when, @notBefore, @notAfter, @from or @to. And a @calendar?
Probably a "warn" level rule.
Suggested in  #10

I don't think @calendar is needed if it is Gregorian. Actually there are probably some unresolved issues around Gregorian / Julian dating...
Get Outlook for Androidhttps://aka.ms/ghei36

From: andrew-morrison<EMAIL_ADDRESS>Sent: Tuesday, February 13, 2018 11:24:46 AM
To: bodleian/consolidated-tei-schema
Cc: Subscribed
Subject: [bodleian/consolidated-tei-schema] New schematron rule: origDate (#11)
Each origDate should always have one or more of the relevant dating attributes: @when, @notBefore, @notAfter, @from or @to. And a @calendar?
Probably a "warn" level rule.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/bodleian/consolidated-tei-schema/issues/11, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ATThVBAIypx4264smjgSOCK9LD_H3Mboks5tUXD9gaJpZM4SDmmN.

This was done in the previous release.


UndoMove does not place pieces back

Been doing some testing and noticed after a while that the bot was failing to see checkmate, despite having a clear if statement checking for it. After a bit I decied to output the FEN strings, and found this...
before any moves are made: rnbqkbnr/pppp1ppp/4p3/6N1/8/8/PPPPPPPP/RNBQKB1R b KQkq - 1 2
after making Move: 'e8e7' rnbq1bnr/ppppkppp/4p3/6N1/8/8/PPPPPPPP/RNBQKB1R w KQ - 2 3
after undoing Move: 'e8e7' rnbqkbnr/pppp1ppp/4p3/6N1/8/8/PPPPPPPP/RNBQKB1R b KQ - 3 3
after making Move: 'd8g5' rnb1kbnr/pppp1ppp/4p3/6q1/8/8/PPPPPPPP/RNBQKB1R w KQ - 4 4
after undoing Move: 'd8g5' rnbqkbnr/pppp1ppp/4p3/8/8/8/PPPPPPPP/RNBQKB1R b KQ - 5 4
after making Move: 'd8f6' rnb1kbnr/pppp1ppp/4pq2/8/8/8/PPPPPPPP/RNBQKB1R w KQ - 6 5
after undoing Move: 'd8f6' rnbqkbnr/pppp1ppp/4p3/8/8/8/PPPPPPPP/RNBQKB1R b KQ - 7 5

There is a white Knight on g5 that can be seen in the original FEN. When it is not taken, e8e7, the UndoMove function works fine. After it is taken, d8g5, the UndoMove function fails to place the Knight back, as the FEN string shows.
After some more testing, I found the culprit to be the board.TrySkipTurn() method. If placed twice in a row, effectively doing nothing, the UndoMove function fails to place a piece back,

If placed twice in a row, effectively doing nothing, the UndoMove function fails to place a piece back

If starting on white, wouldn't it simply skip whites turn then skip blacks turn?
I'm having, I think, similar issues as you. The AddPieceAtSquare method is throwing "Index was out of bounds of the array." exception when I call UndoMove after calling TrySkipTurn twice in a row. I'm calling
MakeMove -> TrySkipTurn -> MakeMove -> TrySkipTurn -> UndoSkipTurn -> UndoMove -> UndoSkipTurn -> UndoMove.
I'm not 100% sure if this is intended behavior and I'm just missing it or not.

For now my solution is
Board newboard = Board.CreateBoardFromFEN(board.GetFenString());
newboard.TrySkipTurn();

though I'm not sure how compliant that is with the rules.

For now my solution is
Board newboard = Board.CreateBoardFromFEN(board.GetFenString());
newboard.TrySkipTurn();

though I'm not sure how compliant that is with the rules.

Complies fine with the rules IMO, but will be slow, so it's not an ideal fix. Hopefully we'll get a real solution soon.


HCCDOC-2557: Adding a Doc tile to OCP Lib

HCCDOC-2557: Adding a Doc tile to OCP Lib from OS Learning Resources page

@Hyperkid123 hi, Martin! i hope you can review this for accuracy, etc. thanks!

/ok-to-test

Hi @jseseCCS there is no assiciated metadata file for the tile. It will not be "discovered" and seeded. The initial setup of a new tile is described here: https://github.com/RedHatInsights/quickstarts/blob/main/docs/quickstarts/README.md#create-a-working-branch-and-make-your-docs-updates

i knew i'd forgotten something, @Hyperkid123! thanks. adding now.

@Hyperkid123 here's an image grab of my folder structure with this yaml file. do I have to put it in its own folder and give it its own metadata file or can I keep it in the same folder (like you see in this image) and add two names to the metadata file?

@Hyperkid123 take another look? it seems like I made it worse but I can't imagine how!

@Hyperkid123 still getting all these errors. what am I doing wrong? and thank you so much for all your help!

@jseseCCS  The filename did not match. The security errors can be ignored. I've pushed the filename change. I'll merge the PR once I get thumbs up from you :slightly_smiling_face: .

@Hyperkid123 please go for it and thank you!!

/retest

tysm, @Hyperkid123!


Can't use IFRAME html in sites

Hi everyone,
I'm designning right now a new WordPress Theme for my self website and i'm tring to embed my Lychee site.
I'm using iFrame to do it and JS to take the height of the iframe. its all work but the content show up above all of my elements in the site.
Using Google Chrome and its all up on my locahost right now using WAMP.
Here the Code Preview:
`
		<style>
		iframe {
		width:100%;
		}
		</style>

	<iframe id="iframemanga" frameborder="0" scrolling="no" onload="setHeight('#iframemanga')" src="http://localhost/m/"> </iframe>
	`

I would recommend, NOT to use iframe, but create a sub-domain gallery.yourdomain for Lychee and link your MENU with website ... it will work a lot faster and less bugs... that what i did for my church website

I would like to do but i think its not very usefull cuz i wanna use my site template and style, if i will try to style Lychee itslef it will bugged for RTL Direction and self-style, i already tried it.

This issue will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community. Important issues have been migrated. Feel free to re-add your issue(s) when necessary.


Docker Jenkins Build Collector

I've set up the jenkins build collector with these ENV values:
  jenkins-build-collector:
    environment:
     - MONGODB_PORT=27017
     - MONGODB_HOST=db
     - MONGO_PORT=27017
     - MONGO_HOST=db
     - HYGIEIA_API_ENV_SPRING_DATA_MONGODB_DATABASE=dashboarddb
     - HYGIEIA_API_ENV_SPRING_DATA_MONGODB_USERNAME=dashboarduser
     - HYGIEIA_API_ENV_SPRING_DATA_MONGODB_PASSWORD=dbpassword

This fails at start up as the mongo can't connect:
2019-06-28 13:42:44,115 INFO  org.mongodb.driver.cluster - Exception in monitor thread while connecting to server localhost:27017
com.mongodb.MongoSocketOpenException: Exception opening socket
	at com.mongodb.connection.SocketStream.open(SocketStream.java:63) ~[mongo-java-driver-3.0.2.jar!/:na]
	at com.mongodb.connection.InternalStreamConnection.open(InternalStreamConnection.java:114) ~[mongo-java-driver-3.0.2.jar!/:na]
	at com.mongodb.connection.DefaultServerMonitor$ServerMonitorRun

As you can see, it is actually using localhost:27017 to connect - which to me makes no sense... I've even gone into the properties-builder.sh and set it there.... still no good...
For comparison, I've setup the github collector with the same ENV values and it works:
  github-collector:
    environment:
      - MONGODB_PORT=27017
      - MONGODB_HOST=db
      - MONGO_PORT=27017
      - MONGO_HOST=db
      - HYGIEIA_API_ENV_SPRING_DATA_MONGODB_DATABASE=dashboarddb
      - HYGIEIA_API_ENV_SPRING_DATA_MONGODB_USERNAME=dashboarduser
      - HYGIEIA_API_ENV_SPRING_DATA_MONGODB_PASSWORD=dbpassword

@no1melman let me try recreating it

@rvema I've forced the dbhost name in the hygieia core, then got a new error:
Caused by: java.lang.NullPointerException
	at org.springframework.scheduling.support.CronSequenceGenerator.parse(CronSequenceGenerator.java:265)
	at org.springframework.scheduling.support.CronSequenceGenerator.<init>(CronSequenceGenerator.java:95)
	at org.springframework.scheduling.support.CronSequenceGenerator.<init>(CronSequenceGenerator.java:82)
	at org.springframework.scheduling.support.CronTrigger.<init>(CronTrigger.java:44)
	at com.capitalone.dashboard.collector.CollectorTask.onStartup(CollectorTask.java:73)

From what I understand, the HudsonCollectorTask implements Collector. Collector has an abstract method getCron(). The point of this is to allow the HudsonCollectorTask to read its HudsonSettings to get the cron... So to me, it seems these settings aren't being read. I checked up the HudsonSettings and it doesn't have a attribute to read a particular property...
I'm don't know how spring boot gets its properties or where it is expecting to get the value from - but it appears to be null....
This is kind of suspicious seeing as the host variable was also ignored until set in the @Value attribute (I assume this is used to assign a default if can't be found).
So it appears that properties aren't being read properly? Where would these Settings classes be looking for their values?

Yeah so to get this to work, I had to specify the JENKINS_CRON env variable, but hardcode the host name in


Fix IIFE Bundling

Found a bug in ESMTK where IIFE bundling is falsly detecting a name collision that doesn't exist
When it's fixed, bump ESBuild in deps

Fixed in v0.5.1


Update documentation for .NET Isolated service bus bindings

[ ] Fill in the following template with all the information required to update the docs
[ ] Share with Matthew for public doc updates + PR

Template
See blob issue as an example of how to full this template in

Binding name
List of supported types
Link to code sample
Supported worker extension version
Required worker versions

Only close this issue when the public docs PR has been merged

Binding name
.NET Isolated ServiceBus Trigger Binding
List of supported types
Input

ServiceBusReceivedMessage
ServiceBusReceivedMessage[]
string & string[]
POCO & POCO[]

Link to code sample

ServiceBusTrigger sample

Supported worker extension version

Worker ServiceBus extension: 5.10.0-preview2

Required worker versions

Worker: 1.15.0-preview1
Worker Sdk: 1.11.0-preview1

Expected GA versions:

Worker: 1.17.0
Worker Sdk: 1.12.0


Notification box about upload file don't go away

Bug Report
System Information

Operating System (Including distribution name and version number)
Ubuntu 18.04.4 LTS

AMP version and build date (Always use the version number, 'latest' is not valid!)
<IP_ADDRESS>
--

Which AMP release stream you're using (Mainline, Nightly or FastTrack)
Mainline

I confirm:

[x] that I have searched for an existing bug report for this issue.
[x] that I am using the latest available version of AMP.
[x] that my operating system is up-to-date.

Symptoms

Enter Symptoms on this line.
Reproduction

Enter reproductions steps here.

upload any file via gui
upload complete, the notification box will stay here
Tested the API file transfer too, more slow (20 sec for 400 kb file) but notification will gone

Can confirm.

This should be fixed as of 1990+


Unwanted declaration causing compilation issues with gccgo

Signed-off-by: Srini Brahmaroutu<EMAIL_ADDRESS>

Can you please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "fix_gccgo_compilation" git@github.com:brahmaroutu/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

This will update the existing PR, so you do not need to open a new one.

LGTM (waiting on CI)

LGTM


There is a requirement that requires me to add a link into the message on error

There is a requirement that requires me to add a link into the message on error
When the user types data into the textbox input, base on a condition I need to show an error message that contains a link.
Solution
It would be greate if MessageOnError and CustomFunc support React Component

@kitokip Check v4.7.0 out, which provides shouldRenderMsgAsHtml in the validationOption to furfil your need.
Search asyncMsgObj.shouldRenderMsgAsHtml in the doc https://github.com/edwardfhsiao/react-inputs-validation/blob/master/docs/v4-doc.md#Textbox

Look like that it doesn't affect for customFunc function?

https://codesandbox.io/s/custom-functionwhen-providing-customfunc-forked-szsy4?file=/src/index.js
look at sencond textbox

@kitokip sorry, it's my mistake, it should be validationOption. shouldRenderMsgAsHtml

I am closing this for now, feel free to reopen if you have further issues.


The second issue

Just making some issues before setting up the integration.

Adding some comments

And more comments

One more


[Bug]: Example in the Semantic Retrieval notebook does not work

Bug Description
Hi,
I'm following https://github.com/run-llama/llama_index/blob/main/docs/examples/managed/GoogleDemo.ipynb
I configured service account auth, enabled the Google API, downloaded the paul graham doc and tried to index it, but got:
Newly created corpus ID is 2c40d915-0c18-4210-a412-a418ba6eac7f.
WARNING:llama_index.vector_stores.google.generativeai.genai_extension:Document e480262f-bae5-4f54-aecd-750c680e4ffe in corpus 2c40d915-0c18-4210-a412-a418ba6eac7f not found: 404 Documents does not exist
Response is Empty Response

The documents variable seems to be set correctly
Version
0.9.23
Steps to Reproduce
Run the example notebook
Relevant Logs/Tracbacks
No response

Eventually this worked - I think my service account was too new and not visible at first


A newly created bot will now fill out the endpoint name with its url.

Before, when creating a new bot, the endpoint created with it had a blank name which would show up as blank in the endpoint explorer. Now it uses the endpoint's url as the name.

Pull Request Test Coverage Report for Build 147

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 45.871%

Totals

Change from base Build 134:
0.0%

Covered Lines:
2861

Relevant Lines:
5203

💛  - Coveralls


Show separate color indicator bar in text & highlight buttons

Changes:

Add generic support for showing a color indicator in the base MenuButtonColorPicker component, and utilize that in MenuButtonTextColor and MenuButtonHighlightColor. Now, the current color will be shown with the "bar" that was present in the original icons. The bar will be omitted (effectively transparent) if no color/highlight value is active.
The MenuButton component now allows for passing children to override the default <IconComponent /> rendering
There are a few new associated icon components, the latter four of which work well as the IconComponent prop for MenuButtonColorPicker:

FormatColorBar (an icon with just the bottom horizontal bar, used as the color indicator in MenuButtonColorPicker)

BorderColorNoBar

FormatColorFillNoBar

FormatColorTextNoBar

FormatInkHighlighterNoBar

cc @ericdwang since you mentioned you were interested in this.

Looks great! :tada:
I just noticed one tiny inconsistency when selecting both colored and non-colored text/highlights: the text color button shows the color but the highlight color button doesn't. Like in this example "no" has no colors and "colors" has a text and background color:

@ericdwang Hm, yeah, so the way the Highlight button behaves is to check whether your selection is "active" for the highlight mark (see isActive, consistent with how Tiptap recommends showing button states in all of their examples, so the same as how we handle Bold, Italic, etc.). That means if you select something that has no highlighting on some portion of it, it shows as "not active" and does not show a current color.
By contrast, I don't have MenuButtonTextColor checking "is active", since it seemed moderately useful to show the "first" color of the selected text, but that's probably the wrong choice for the sake of consistency. I'll change it to show no color if there are multiple colors selected, the same as how Google Docs behaves (and other controls in mui-tiptap behave, like FontSize, etc.)

Ah I see, makes sense!


Pushy should provide a way of getting a notification that couldn't be queued

The documentation suggests the best practice is:

pushNotificationResponseHandler =
   new FutureListener<PushNotificationResponse<SimpleApnsPushNotification>>() {
      public void operationComplete() {
           try {
                responseFuture.get()
           } catch (ClientNotConnectedException e) {
                  apnsClient.getReconnectionFuture.await()
                  //Resend!?
           }
      }
   };

future = apnsClient.sendNotification(pushNotification);
future.addListener(pushNotificationResponseHandler);

But there's no way to get a reference in a FutureListener<PushNotificationResponse<SimpleApnsPushNotification>> to the push notification which has failed to send.
Handling this case involves one of:

Making assumptions about Pushy (i.e. is the queue processed in order - can I just maintain my own queue of unsent notifications and just assume the failed one is the head of queue?
Create a handler per notification.

Of these, 1 looks decidedly dodgy and 2 is unscalable.
What's the recommended way of resending PushNotifications that fail due to an ClientNotConnectedException.
It would be nice if either ApnsClient or the ClientNotConnectedException itself exposed the PushNotification.

…can I just maintain my own queue of unsent notifications and just assume the failed one is the head of queue?

We make no guarantees about order, so the queue approach won't work.

Create a handler per notification.

This has generally worked well so far, even for heavy use cases. I agree that there's an opportunity to decrease overall memory usage by adding a mechanism to get the original push notification when send futures fail (thereby paving the way for reuse of a single listener instance), but don't think I would describe the current situation as "unscalable;" either way, we're still talking about O(n) memory usage, where n is the number of notifications "in flight."

What's the recommended way of resending PushNotifications that fail due to an ClientNotConnectedException.

The most common approach is to add the listener as an anonymous implementation that refers to the sent notification (please excuse the lazy syntax here):
final Future<> sendFuture = client.sendNotification(exampleNotification);
sendFuture.addListener(new FutureListener<PushNotificationResponse<SimpleApnsPushNotification>>() {
      public void operationComplete() {
           try {
                responseFuture.get()
           } catch (ClientNotConnectedException e) {
                  apnsClient.getReconnectionFuture.await()

                  // exampleNotification is in scope
                  client.sendNotification(exampleNotification);
           }
      }
);

Alternatively, you could use a named listener that takes a push notification as a constructor:
final Future<> sendFuture = client.sendNotification(exampleNotification);
sendFuture.addListener(new ListenerThatKnowsAboutASpecificNotification(exampleNotification));

In short, this seems like a good thing to think about, but I think other things are likely to take priority.

We have a proposal for a potential solution in #535. What do you think?


triple+rest support custom uri like "proto annotations"

I use curl to send  rest request in triple port ，the command like
curl http:/xxxxxxx/org.apache.dubbo.samples.tri.grpc.Greeter/greet  -H "Content-Type: application/json" -X POST -d '{"name":"test"}'

But I can't change the uri, can dubbo support change the uri.May like this: https://cloud.google.com/endpoints/docs/grpc/transcoding,  use  "proto annotations" .
So the dubbo framework user  can custom the uri.

@icodening @EarthChen @suncairong163 PTAL
We can support this feature when merging rest protocol into triple protocol.

@xiaoyuuuuuupeng

ok will try it

No update for a longer time.

There are two things that need to be done: one is to add the Mapping annotation to the generated stub, and the other is to support filling path variable parameters into the pb object when parsing parameters.


Tree key

Adds key support to Tree control
WIP. Tree needs to scroll on key as well.

Codecov Report

Merging #68 (948a0dd) into main (e7039ee) will decrease coverage by 0.11%.
The diff coverage is 2.81%.

@@           Coverage Diff            @@
##            main     #68      +/-   ##
========================================
- Coverage   4.24%   4.13%   -0.12%
========================================
  Files         52      52
  Lines       3862    3967     +105
========================================
  Hits         164     164
- Misses      3698    3803     +105

Flag
Coverage Δ

unittests
4.13% <2.81%> (-0.12%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/textual/_ansi_sequences.py
0.00% <0.00%> (ø)

src/textual/_xterm_parser.py
0.00% <ø> (ø)

src/textual/events.py
0.00% <ø> (ø)

src/textual/keys.py
0.00% <0.00%> (ø)

src/textual/messages.py
0.00% <0.00%> (ø)

src/textual/reactive.py
0.00% <ø> (ø)

src/textual/widget.py
0.00% <0.00%> (ø)

src/textual/widgets/_directory_tree.py
0.00% <0.00%> (ø)

src/textual/widgets/_scroll_view.py
0.00% <0.00%> (ø)

src/textual/widgets/_tree_control.py
0.00% <0.00%> (ø)

... and 1 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 f1107a1...948a0dd. Read the comment docs.


[FLINK-13042][datastream] Make slot sharing configurable on StreamExecutionEnvironment

What is the purpose of the change

There is a requirement of Blink batch planner that providing a global setting that disabling slot sharing. To support that, will expose a PublicEvolving method on StreamExecutionEnvironment to globally disable slot sharing.

Note that, this method might be removed if there is a better approach to satisfy Blink batch planner in the future.

There is a discussion under https://github.com/apache/flink/pull/8931 at the moment

This PR should not go forward until there is an agreement reached

Brief change log

Add a PublicEvolving method on StreamExecutionEnvironment to enable/disable slot sharing

Verifying this change

Added integration tests to verify slot sharing behavior

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): yes
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes maybe
If yes, how is the feature documented? docs & JavaDocs

We don't need this at the moment. So close it.


Rendering Video Texture on 3D Plane

Hello,
I wanted to render video texture on 3d plane. I tried doing for 3D for the modelbatch using the 2D example provided in the library, however, I got the GL thread error.
I would be grateful if someone can help me with this.
Thanks :)

Please join the community for questions. This is an issue tracker and not a support forum.
A gl thread error indicates, that you are not drawing from the rendering thread.


fix changlog workflow trigger.

This should trigger on every new tag, not just master releases. Is that fine with you?

would you rather have us use this:
v[0-9]+.[0-9]+.[0-9]+


Random Travis SauceLabs automated tests failures => false positives (negatives?)

https://travis-ci.org/readium/readium-js-viewer/jobs/67224072#L2801
Error: [waitForElementByCss(".progress-bar",{},10000)] Element condition wasn't satisfied!

Why?
What can we do to prevent this from happening?

https://travis-ci.org/readium/readium-js-viewer/jobs/68155140#L2849
This one is different:
     Error: [sauceJobUpdate({"name":"307.2"})] Sauce job update failed with http status code:504
      at _stream_readable.js:908:16

Fixed via: https://github.com/readium/readium-js-viewer/commit/dfae67b1e3b40293f8639d3c34e9d87f27f4512b

Firefox fail:
https://travis-ci.org/readium/readium-js-viewer/jobs/70494247#L3521
Error: [waitForElementByCss("#introduction h2.title")] Element condition wasn't satisfied!
...yet Chrome passes:
https://travis-ci.org/readium/readium-js-viewer/builds/70494245
So it's not 100% bullet-proof yet, but perhaps there is nothing we can do about it. It's mostly green anyway: https://travis-ci.org/readium/readium-js-viewer/builds

Another SauceLabs / SauceConnect -related failure:
https://travis-ci.org/readium/readium-js-viewer/jobs/81764367
Starting Sauce Connect
$ travis_start_sauce_connect
Using temp dir /tmp/sc.JTEa
/tmp/sc.JTEa ~/build/readium/readium-js-viewer
Downloading Sauce Connect
--2015-09-23 12:31:04--  http://saucelabs.com/downloads/sc-4.3.11-linux.tar.gz
Resolving saucelabs.com (saucelabs.com)... <IP_ADDRESS>
Connecting to saucelabs.com (saucelabs.com)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 200 OK
Length: 3597352 (3.4M) [application/octet-stream]
Saving to: `sc-4.3.11-linux.tar.gz'
100%[======================================>] 3,597,352   2.18M/s   in 1.6s
2015-09-23 12:31:05 (2.18 MB/s) - `sc-4.3.11-linux.tar.gz' saved [3597352/3597352]
Extracting Sauce Connect
Waiting for Sauce Connect readyfile
23 Sep 12:31:06 - Sauce Connect 4.3.11, build 1757 2b421bb
23 Sep 12:31:06 - Using CA certificate bundle /etc/ssl/certs/ca-certificates.crt.
23 Sep 12:31:06 - Using CA certificate verify path /etc/ssl/certs.
23 Sep 12:31:06 - Starting up; pid 3722
23 Sep 12:31:06 - Command line arguments: sc-4.3.11-linux//bin/sc -i 405.2 -f sauce-connect-ready-6554 -l /home/travis/sauce-connect.log
23 Sep 12:31:06 - Using no proxy for connecting to Sauce Labs REST API.
23 Sep 12:31:06 - Resolving saucelabs.com to <IP_ADDRESS> took 1 ms.
23 Sep 12:31:06 - Started scproxy on port 44251.
23 Sep 12:31:06 - Please wait for 'you may start your tests' to start your tests.
23 Sep 12:31:06 - Starting secure remote tunnel VM...
23 Sep 12:31:09 - Secure remote tunnel VM provisioned.
23 Sep 12:31:09 - Tunnel ID: 13b58822c2c8489ba2207f7f32ae851b
23 Sep 12:31:09 - Secure remote tunnel VM is now: booting
23 Sep 12:32:35 - Error bringing up tunnel VM.
23 Sep 12:32:35 - Sauce Connect could not establish a connection.
23 Sep 12:32:35 - Please check your firewall and proxy settings.
23 Sep 12:32:35 - You can also use sc --doctor to launch Sauce Connect in diagnostic mode.
23 Sep 12:32:35 - Cleaning up.
23 Sep 12:32:35 - Shutting down named tunnel 405.2 (13b58822c2c8489ba2207f7f32ae851b).
23 Sep 12:32:39 - Goodbye.
No output has been received in the last 10 minutes, this potentially indicates a stalled build or something wrong with the build itself.
The build has been terminated


Make options like "save_flows" configurable for high level well packages

In GitLab by @JoerivanEngelen on Nov 17, 2023, 14:44
Created from: https://gitlab.com/deltares/imod/imod-python/-/work_items/623
Currently unable to use options like save_flows. Therefore, save_flows was hardcoded in the jinja template in the featurebranch, which is unwanted.
The user should be able to pass settings for high-level packages.

Update these settings are passed on to the Well package at the current state of the codebase, but not to the HorizontalFlowBarrier packages yet.


Add support for arguments and subcommands

Created support for arguments

/ops docker ps now accepts -a  flag similar to /ops gitlab version and /ops gitlab version check.
Introduced subcommands

Provider name removed from command definitions
ie. docker is not part of command definition, it is populated from provider command. It will be easy to rename provider names in future.

Default output is simplified by using only providers

@phoinixgrr dropped your review due to missing fun commands. Now every provider is correct.


Vane error during playback causes boot to abort, "sing: ovum failed"

See also this paste and this fora post.
If you're unlucky enough to have received #538, but haven't received the related and required #603 yet (which I wasn't smart enough to push onto the network simultaneously) then your hall or talk may cause gall to print an "invalid bone" error. It seems that if this happens during boot (more accurately, playback) then the boot aborts with sing: ovum failed! Userspace apps doing dumb things should probably not have that kind of show-stopping power.
Not sure if this happens exclusively for gall errors. I imagine an error in another vane can have the same result.

Closing as this no longer appears to be relevant.


chore: changing terraform to opentofu

nixos has no terraform package anymore. Changed it to opentofu.

I'm fine with changing it here for testing. We need to make this formally supported on collie-cli as well