draggable is not working properly with complex components

Hi
First of all, thank you for the hard work in this component. It's the closest and best-maintained solution I've found so far in terms of combining a virtual list with draggable items.
I'm doing a proof-of-concept to see if it works well but I'm running into issues with the draggable part.
Code snippet for reference:
<virtual-drag-list
      :data-key="'id'"
      :data-source="queue"
      :draggable="editing"
      @top="scrolledToTop"
      @bottom="scrolledToBottom"
      @ondragend="sortQueue"
    >
      <template slot="item" slot-scope="{ record }">

          <list-item :item="record" />

            sort_ios

      </template>
    </virtual-drag-list>

The items should only be draggable after the user clicks a button that sets editing to true.
When I put draggable="true" on the entire div class="sortable-item"> it does allow dragging (sometimes) but it's very picky on where I select the item. It can only be selected in areas that are not taken by any children elements (e.g. whitespace areas).
Preferably, I'd like only the
 which contains an icon to allow dragging. I tried 2 things:
Putting draggable="true" on the
 and on the . Both options do not allow me to drag, although it looks like I can drag the icon itself while I hold down the mouse button. Every time I try to drag I see the following error in the logs:
index.js?ef2b:223 Uncaught TypeError: Cannot read properties of undefined (reading 'left') at Draggable._handleMousedown (index.js?ef2b:223:1)
Any ideas on how to get this to work properly?
Thank you!

Sorry to reply so late, I will use your method to test once, if it is solved will notify you in time

@mrooding Sorry for the late update, please update to the latest version to confirm if your issue is resolved, I look forward to hearing from you

Hi @mfuu
Thanks for the update. I'll retest this as soon as possible and get back to you!

Hey @mfuu
I was able to test it a bit and it seems to behave a lot better. There are a few different cases with different behavior.
case 1: setting the actual icon to be draggable
In this scenario, I'm putting draggable="true" on the  that's actually rendering the drag icon. In this case, it works really well.
case 2: setting the wrapping div to be draggable
In this scenario I'm putting draggable="true" on the
 surrounding the . In this case, the default browser behavior to drag the actual icon seems to win over the draggable behavior. I'm unable to drag based on this setup.
case 3: setting the
 to be draggable
This seems to behave exactly the same as I originally mentioned: sometimes it works and sometimes it doesn't. After further testing, it seems to have to do with the content of the actual list item. If there's text-overflow it doesn't allow dragging. If the title is very short, it always allows dragging by clicking on the empty space in one of the list items.

In all cases, I never saw the javascript error that I first reported which is very good!

@mrooding
Thank you for your test, I'll check it

Has it been repaired @mfuu

Hi @mfuu
2.6.0 works fine for the setup that I'm interested in now. Thank you very much!
Something unrelated (let me know if you want me to open a separate issue for it):
I'm using the @top and @bottom to fetch more items depending on where the user is within the virtual list data. When triggering @top an HTTP call is done to fetch additional data and the list is updated. However, the list jumps to the first item in the updated result instead of staying at the beginning of the list before fetching additional items.
An example:
I'm displaying items 400 - 600 and the user scrolls to the top. Item 200 to 400 are being fetched. The top of the list is shown and item 200 is the first to be shown.
It would be nice if the list stays around item 400 and the user is able to scroll up towards 200.
Is this something that can be provided as a feature?
As a workaround, I'm now storing the ID of the item in the list before fetching additional data and then using a watch to scroll back to the new index in the list based on the ID.
Thanks again for the effort!

Something else that I just noticed, when using @ondragend it may be useful to indicate the parameter types of the callback in the documentation.
Beyond that, new does not contain the item and index when I drag an item from index 0 to index 1:
{node: div, rect: {…}, offset: {…}, item: null, index: null}

Also, since new is a reserved keyword, it may be better to give that parameter a different name in the documentation.

Hi @mfuu
2.6.0 works fine for the setup that I'm interested in now. Thank you very much!
Something unrelated (let me know if you want me to open a separate issue for it):
I'm using the @top and @bottom to fetch more items depending on where the user is within the virtual list data. When triggering @top an HTTP call is done to fetch additional data and the list is updated. However, the list jumps to the first item in the updated result instead of staying at the beginning of the list before fetching additional items.
An example:
I'm displaying items 400 - 600 and the user scrolls to the top. Item 200 to 400 are being fetched. The top of the list is shown and item 200 is the first to be shown.
It would be nice if the list stays around item 400 and the user is able to scroll up towards 200.
Is this something that can be provided as a feature?
As a workaround, I'm now storing the ID of the item in the list before fetching additional data and then using a watch to scroll back to the new index in the list based on the ID.
Thanks again for the effort!

@mrooding
First of all thank you very much for your advice.
I will consider this issue, and if feasible, will probably pass it as an optional props, because this may create other problems in data judgment.

Something else that I just noticed, when using @ondragend it may be useful to indicate the parameter types of the callback in the documentation.
Beyond that, new does not contain the item and index when I drag an item from index 0 to index 1:
{node: div, rect: {…}, offset: {…}, item: null, index: null}

Also, since new is a reserved keyword, it may be better to give that parameter a different name in the documentation.
old does contain the original item + index but without new I'm unable to detect what sorting change actually occurred.
If I drag another item that's not at index 0 it sometimes triggers @ondragend with both old and new with an index being 0.
How do I properly detect what change occurred?

@mrooding
I will check this question carefully.
At present, you can use the arr and changed parameters to view the changes of the array. If the data you are currently getting with these two parameters is as expected?

@mrooding
Sorry for the late reply.
The data error problem of the ondragend function has been fixed in version 2.6.2

🐮，thanks
---- 回复的原邮件 ----
| 发件人 | @.> |
| 日期 | 2022年05月20日 20:34 |
| 收件人 | @.> |
| 抄送至 |<EMAIL_ADDRESS>|
| 主题 | Re: [mfuu/vue-virtual-drag-list] draggable is not working properly with complex components (Issue #4) |
@mrooding
Sorry for the late reply.
The data error problem of the ondragend function has been fixed in version 2.6.2
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@mfuu thanks for the fixes. I've tested 2.6.3 and the @ondragend issue. Here's what I can see:
Dragging index 0 to index 1
the list property passed to the @ondragend callback is the original list and not the updated list. The changed property is simply a boolean that indicates that something changed. With just these 2 properties I cannot determine what exactly changed.
The from property contains the right index + item of the object I moved. I'm just not sure why the key is an empty string:
index: 0
item: Object
key: ""

The to is however not what I would expect:
index: 0
item: null
key: "1"

The to contains index 0 again, whereas I would expect this to be index 1.
Based on all @ondragend properties I'm still not sure how to detect what actually changed. Is there something that I'm doing wrong?
Dragging index 2 to index 3
This is where it goes quite wrong. The list property contains the list but index 0 to 2 are null values:
(201) [null, null, null, {…}, {…}, {…}..........]
from:
index: -1
item: null
key: "2"

to:
index: -1
item: null
key: "3"

The nullable items inside the list also cause duplicate key errors:
[Vue warn]: Duplicate keys detected: ''. This may cause an update error.
This behavior is reproducible with any drag action that's not from index 0 to index 1.
Thanks again for the support!

@mfuu thanks for the fixes. I've tested 2.6.3 and the @ondragend issue. Here's what I can see:
Dragging index 0 to index 1
the list property passed to the @ondragend callback is the original list and not the updated list. The changed property is simply a boolean that indicates that something changed. With just these 2 properties I cannot determine what exactly changed.
The from property contains the right index + item of the object I moved. I'm just not sure why the key is an empty string:
index: 0
item: Object
key: ""

The to is however not what I would expect:
index: 0
item: null
key: "1"

The to contains index 0 again, whereas I would expect this to be index 1.
Based on all @ondragend properties I'm still not sure how to detect what actually changed. Is there something that I'm doing wrong?
Dragging index 2 to index 3
This is where it goes quite wrong. The list property contains the list but index 0 to 2 are null values:
(201) [null, null, null, {…}, {…}, {…}..........]
from:
index: -1
item: null
key: "2"

to:
index: -1
item: null
key: "3"

The nullable items inside the list also cause duplicate key errors:
[Vue warn]: Duplicate keys detected: ''. This may cause an update error.
This behavior is reproducible with any drag action that's not from index 0 to index 1.
Thanks again for the support!

This may be caused by the comparison of the key value of the number type and the string type，I will fix it in 2.6.4

I just tested 2.6.6 and it seems to work fine now with moving from index 1 to 2 and from 0 to 1.
Both the list and from and to have the right data.
I'll keep experimenting and get back to you when I find something else.
Thanks so far!

Hey @mfuu
I found some weird behavior in terms of the scrollToIndex when calling it the first time after the list initially gets populated. Let's say that I load an initial list of 400 items. After the list propagates from the store to the component, I call scrollToIndex(200). You can see the vertical scrollbar jump to the middle but the list shows the first ~20 items of the list.
I can provide you with a video if it helps but I'd prefer to share it in private.

Thanks so far!

Thank you for your issue, i will check it
May be you can raise this proposal as a new issue

Hey @mfuu
I found some weird behavior in terms of the scrollToIndex when calling it the first time after the list initially gets populated. Let's say that I load an initial list of 400 items. After the list propagates from the store to the component, I call scrollToIndex(200). You can see the vertical scrollbar jump to the middle but the list shows the first ~20 items of the list.
I can provide you with a video if it helps but I'd prefer to share it in private.

@mrooding
Thank you for your issue, i will check it
May be you can raise this proposal as a new issue if possible


Update the version on crates.io

Now that breaking changes are mostly gone (I don't think you're using std::io, std::os, etc.), would it be possible to update the version of nalgebra on crates.io?

I second this request.

Done!

Thank you!


docs: Mention step to setup nodes on OpenStack

The --setup step was missing which lead to cryptic error messages when
running the following step (calling the dummy test suite).

Can one of the admins verify this patch?

ok to test


Service Template Inheritance

Start and end date: 2018-02-19 to 2018-02-19
Contributor: Kálmán Képes, @nyuuyn

Adds feature to set inheritance on Service Templates

Any idea why this branch merges over 200 commits ? Did I really forget to pull ?

You did not follow http://eclipse.github.io/winery/dev/ToolChain#steps-for-working-on-a-topic.
You branched from origin/master, not from upstream/master. I changed the base to ustutt.
Since yesterday, we changed the branches so that origin/master is upstream/master and if someone does not follow the ToolChain, not so much goes wrong.

Here, it doesn't have 200 commits. Only 4. You can merge the commits into one. Hints are given at http://eclipse.github.io/winery/dev/ToolChain#steps-for-working-on-a-topic. Then, you can cherry-pick this commits for eclipse/master.

Is https://github.com/OpenTOSCA/winery/pull/69 the successor of this PR?

To be honest, I'm not sure how the other PR got there, maybe I messed up moving the commits to a PR for Eclipse Winery?

I think, this has the wrong target branch and #69 got it right.


no way to retrieve peripherals

I don't see this exposed:
https://developer.apple.com/documentation/corebluetooth/cbcentralmanager/1519127-retrieveperipherals

Humm :S
https://github.com/jordanebelanger/SwiftyBluetooth/blob/e38dfdfb7686e99d983675988bc6228705919464/SwiftyBluetooth/Source/SwiftyBluetooth.swift#L71
https://github.com/jordanebelanger/SwiftyBluetooth/blob/e38dfdfb7686e99d983675988bc6228705919464/SwiftyBluetooth/Source/Central.swift#L168

Hm. Seems like it's just that the docs are out of date then:
http://cocoadocs.org/docsets/SwiftyBluetooth/0.2.0/

Yea looks like cocoadoc is broken and won't update for the latest version, prolly because I changed my account name and something broke.
Looks like we're done providing documentation outside of code :) removed the link from the readme

@jordanebelanger have you seen this? https://github.com/realm/jazzy/ 😊


[4.4.dev7] The Default Environment setting is reset after the "cold" project loading (res:// | uid:// issue)

Tested versions

reproducible in 4.4.dev7 (maybe 4.4.x)
not reproducible: 4.3.stable

System information
Godot v4.4.dev7 - Windows 11 (build 26100) - Multi-window, 1 monitor - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Ti (NVIDIA; <IP_ADDRESS>36) - 12th Gen Intel(R) Core(TM) i5-12400 (12 threads)
Issue description
The default environment setting (rendering/environment/defaults/default_environment) is reset after the "cold" project loading (deleting the .godot folder and loading project for the first time). However, it stays on place if it was saved as res://... path, not as uid://... value. Seems like uid related bug.
Steps to reproduce
Steps for 4.4.dev7:

create new project;
create an Environment asset;
set it as default in Project Settings;
close the project;
delete the .godot folder;
open the project.

Not reproduced with 4.3 -> 4.4 migration because its value will still be saved as res://, not uid://.
Minimal reproduction project (MRP)
test.zip

Seems like the Main Scene (run/main_scene) setting has a similar issue: Godot will not open the Main Scene after the "cold" loading if it is saved as uid:// value, but open perfectly fine if it is saved as res://. Shoud I create a separate issue for that? Pretty sure it is the same, but in theory can require a fix in several places.

This maybe due to running scan of filesystem after loading environment and main scene, or something like what. Needs investigating


Potential performance improvement? Something similar to Julia's @simd?

Hi Serge,
i hope you are fine at home.
I investigated about the bad result for Pythran mentioned in  https://github.com/Thierry-Dumont/BenchmarksPythonJuliaAndCo/wiki/5-The-FeStiff-benchmark (see the resulting PR https://github.com/Thierry-Dumont/BenchmarksPythonJuliaAndCo/pull/12).
I isolated two issues. The first one is about the lack of native struct (so a method uses Python getattr and 2 function calls instead of 1). Using __slots__ and PyPy removes most of the overhead!
Then, even just for the numerical kernel (the code is here https://github.com/paugier/bench_integrate_callback/tree/master/struct_simd/only_simd), Pythran is slower than Julia on this example (ratio Julia/Pythran ~ 2.4).
The implementation in Julia contains the same information than the one in Python, except for a macro @simd (https://docs.julialang.org/en/v1/base/base/#Base.SimdLoop.@simd, which is used to "annotate a for loop to allow the compiler to take extra liberties to allow loop re-ordering").
Even without @simd Julia is faster than Pythran, but it seems that it helps Julia to be faster (here, Pythran runs in 0.30 µs):
Julia without simd 0.157 µs
Julia with simd    0.125 µs
ratio (without simd)/(with simd): 1.26

There may be a (small) performance issue with this code and Pythran
For me, the macro @simd is somehow similar to OpenMP comments, so i was wondering if we could also add such information in a comment (something like # simd for) just before the for loop. Could Pythran do something useful with that?

The correct way to achieve that goal the pythran way would be through openmp simd directives. i'll have a look!

@paugier : using the appropriate vector form:
#pythran export compute(float[3], float[3], float[6,3,2], float[21])
import numpy as np

def compute(x , y , grads, out):
    det = -(x[1] - x[2]) * y[0] + (x[0] - x[2]) * y[1] - (x[0] - x[1]) * y[2]
    dv = 1.0 / (6.0 * det)
    ii = 0
    fgrads = grads.flatten()
    for i in range(6):
        for j in range(i + 1):
            out[ii] = dv * np.sum(fgrads[i * 6] * fgrads[j * 6])
            ii += 1

This got vecotrized with -DUSE_XSIMD just fine. Can you confirm?

Note:
def compute(x , y , grads, out):
    det = -(x[1] - x[2]) * y[0] + (x[0] - x[2]) * y[1] - (x[0] - x[1]) * y[2]
    dv = 1.0 / (6.0 * det)
    ii = 0
    for i in range(6):
        for j in range(i + 1):
            out[ii] = dv * np.sum(grads[i] * grads[j])
            ii += 1

works just fine too.

And I particularly like this form, even if it's arguably less intuitive:
#pythran export compute(float[3], float[3], float[6,3,2], float[21])
import numpy as np

def compute(x , y , grads, out):
    det = -(x[1] - x[2]) * y[0] + (x[0] - x[2]) * y[1] - (x[0] - x[1]) * y[2]
    dv = 1.0 / (6.0 * det)
    for i in range(6):
        for j in range(i + 1):
            out[(i * (i +1)) // 2 + j] = dv * np.sum(grads[i] * grads[j])

Nice new versions! Unfortunately, I don't get any speedup compared to the original code. Worse, -DUSE_XSIMD doesn't accelerate these functions, and even makes your first version slower than without -DUSE_XSIMD!
Without -DUSE_XSIMD:
compute :  0.28 µs
compute1:  0.29 µs
compute2:  0.28 µs
compute3:  0.28 µs
Julia:   0.12 µs
ratio Pythran/Julia: 2.30

And with  -DUSE_XSIMD:
compute :  0.28 µs
compute1:  0.38 µs
compute2:  0.28 µs
compute3:  0.28 µs
Julia:   0.12 µs
ratio Pythran/Julia: 2.32

Are you using the fixed-size version in the pythran annotation?

The previous results were with the fixed-size version in the pythran annotation.
With the standard annotations, it gives:
Without -DUSE_XSIMD:
compute :  0.30 µs
compute1:  0.30 µs
compute2:  0.45 µs
Julia:   0.13 µs
ratio Pythran/Julia: 2.36

With -DUSE_XSIMD:
compute :  0.30 µs
compute1:  0.38 µs
compute2:  0.48 µs
Julia:   0.13 µs
ratio Pythran/Julia: 2.37

With -DUSE_XSIMD -march=native:
compute :  0.30 µs
compute1:  0.39 µs
compute2:  0.57 µs
Julia:   0.13 µs
ratio Pythran/Julia: 2.36

-DUSE_XSIMD and -march=native slow down the computation!

Hi,
I am mostly completely out of concern, but I've looked at the old Just In Time compiler used by the no more maintained HOPE library.
It is doing JIT using only pure python code to achieve this. I found it quite amazing because the code is really simple.
Unfortunately only a few subset of python is supported and the library is not able to compile the famous mandelbrot example below:
@hope.jit
def mandel(x, y, max_iters):
    """
    Given the real and imaginary parts of a complex number,
    determine if it is a candidate for membership in the Mandelbrot
    set given a fixed number of iterations.
    """
    c = complex(x, y)
    z = 0.0j
    for i in range(max_iters):
        z = z*z + c
        if (z.real*z.real + z.imag*z.imag) >= 4:
          return i

    return max_iters

@hope.jit
def create_fractal(min_x, max_x, min_y, max_y, image, iters, w, h):
    height = h
    width = w

    pixel_size_x = (max_x - min_x) / width
    pixel_size_y = (max_y - min_y) / height

    for x in range(width):
        real = min_x + x * pixel_size_x
        for y in range(height):
          imag = min_y + y * pixel_size_y
          color = mandel(real, imag, iters)
          image[y, x] = color

image = np.zeros((1024, 1536), dtype = np.uint8)
start = timer()
create_fractal(-2.0, 1.0, -1.0, 1.0, image, 50)
dt = timer() - start

print("Mandelbrot created in %f s" % dt)
imshow(image)
show()

But using its own examples, it appears to be near 2 times faster than numba default jit
from scipy.integrate import odeint
import hope
import numba

P, d, B, G, A = 0, 0.0001, 0.0095, 0.0001, 0.0001
N = 10**3
def f_nat(y, t):
    dy = np.empty(3)
    dy[0] = P - B*y[0]*y[1] - d*y[0]
    dy[1] = B*y[0]*y[1] + G*y[2] - A*y[0]*y[1]
    dy[2] = d*y[0] + A*y[0]*y[1] - G*y[2]
    return dy

y0, t = np.array([500., 0., 5.]), np.linspace(0, 5., N)

@hope.jit
def f_hope(y, t, P, d, B, G, A):
    dy = np.empty(3)
    dy[0] = P - B*y[0]*y[1] - d*y[0]
    dy[1] = B*y[0]*y[1] + G*y[2] - A*y[0]*y[1]
    dy[2] = d*y[0] + A*y[0]*y[1] - G*y[2]
    return dy

@hope.jit
def f_opt(y, t, dy, P, d, B, G, A):
    dy[0] = P - B*y[0]*y[1] - d*y[0]
    dy[1] = B*y[0]*y[1] + G*y[2] - A*y[0]*y[1]
    dy[2] = d*y[0] + A*y[0]*y[1] - G*y[2]
    return dy

import numba

@numba.jit
def f_numb(y, t, P, d, B, G, A):
    dy = np.empty(3)
    dy[0] = P - B*y[0]*y[1] - d*y[0]
    dy[1] = B*y[0]*y[1] + G*y[2] - A*y[0]*y[1]
    dy[2] = d*y[0] + A*y[0]*y[1] - G*y[2]
    return dy

@numba.jit
def f_numb_opt(y, t, dy, P, d, B, G, A):
    dy[0] = P - B*y[0]*y[1] - d*y[0]
    dy[1] = B*y[0]*y[1] + G*y[2] - A*y[0]*y[1]
    dy[2] = d*y[0] + A*y[0]*y[1] - G*y[2]
    return dy

dy = np.empty(3)
print ("native python")
%timeit odeint(f_nat, y0, t)
print ("hope")
%timeit odeint(f_hope, y0, t, args=(P, d, B, G, A))
print ("hope without allocation")
%timeit odeint(f_opt, y0, t, args=(dy, P, d, B, G, A))
print ("numba")
%timeit odeint(f_numb, y0, t, args=(P, d, B, G, A))
print ("numba without allocation")
%timeit odeint(f_numb_opt, y0, t, args=(dy, P, d, B, G, A))

native python
952 µs ± 5.73 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)
hope
187 µs ± 126 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)
hope without allocation
170 µs ± 929 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)
numba
320 µs ± 1.1 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)
numba without allocation
289 µs ± 746 ns per loop (mean ± std. dev. of 7 runs, 1000 loops each)

Using a biggest value for N this disappear on my laptop showing no diff between native python, numba or hope.
But there may be a nice optimization to find in hope JIT.
Honestly I have not investigate more how the library is working.
Just wanted to share this measurement. I have not tested it with pythran too :(
Shame on me. Will do it as soon as possible.
Hope it can help anyway.


associate routes with tabs of address details

for those three tabs we need three different routes

assigned to beiciye


Option to remove guildbook calendar button on minimap

Some people don't want their minimap to have any buttons, there needs to be an option to remove it.

It looks like this is no longer needed


PLUTO-709: Making TCKSimpleTestDriver extendable

This PR was tested on master with PR 28 and PR 31 already applied.


refactor: templates pane redesign

Changes

Redesign template information
Add new default icon
Style search input
Remove delete icon

Screenshots

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Scrollbar isn't showing/working for me and there is an overflow at the bottom:

@olensmar - I fixed the overflow problem and added scrollbar.


Azure Search Improvements for Java (January-June 2021)

This tracks the work for Azure Search Improvements for Java.

Closing as June release has passed


Fix sphinx version to 1.7.9

It looks the newer version of sphinx is causing test errors in Travis tests for MacOS.

Jenkins, test this please.

Jenkins CI test (for commit 956a99ec4113f71d3c0214958539271f9679d176, target branch master) succeeded!

LGTM!

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

Could not backport before release so I removed the label.

It seems sphinx-doc/sphinx#5510 fixed the issue (sphinx>=1.8.2, which hasn't been released).


dev-master/1.3.x-dev Version Causes Issues with Default Laravel Installation

In the default Laravel Installation $ composer create-project laravel/laravel :
    - laravel/framework[v11.9.0, ..., v11.33.2] require fruitcake/php-cors ^1.3 -> found fruitcake/php-cors[dev-master, 1.3.x-dev (alias of dev-master)] but it does not match your minimum-stability.

While it is possible to work around this by adjusting minimum-stability or using specific package configurations, this issue impacts a smooth out-of-the-box experience for Laravel users, might also confuse users who aren’t familiar with Composer's stability settings.
Thank you for your attention to this matter! Please let me know if there’s anything I can do to assist further.

Are you using the default packagist repo? 1.3.0 is stable and tagger.

It turns out the issue was related to the China mirror I am using (https://mirrors.aliyun.com/composer) after switching back to (https://repo.packagist.org), the problem was resolved, and the fruitcake/php-cors package version v1.3.0 was installed without any issues.
Thanks again.


Documentation: ".values" instead of ".columns"

Hi
In https://cap.cloud.sap/docs/node.js/api#INSERT --> section ".columns(...)": The text states "subsequently specified .rows or .columns.". Should be "subsequently specified .rows or .values. "
Regards

Thank you. We will have a look at it and get to you back here asap.
Best regards,
Iwona

It was fixed in the documentation https://cap.cloud.sap/docs/node.js/api#INSERT.


operator: Optimize and use Kubernetes approach for GVK validation in isTektonInstalled func

This is a bit optimized approach used in Kubernetes to validate are CRD's installed or not. The benefits are:

Not require to get a list of all CRD's with Spec (can be very large) as it's very resource consumption and heavy operation for API server
Validates by full GVK value, not just CRD name, which can be not formal and sometimes never changing from one version to another (for v1alpha1 -> v1beta1 - v1 CRD names will be the same)

Thank you @dmvolod and welcome to the community!


Overwrite glyphs contour count if it's less than -1.

The test pdf has a contour count of -70, but OTS doesn't
like values less than -1.
Fixes issue #7562.

/botio test

Off by one!
/botio test

/botio test

/botio makeref


Readline() not work well (sdb plugin)

Hello! I'm callmekohei!

Problesm
Readline() not work well.
like this
(sdb) mycmd foo.exe
....
(sdb)                  // <--- I can not watch input strings (^_^;;;

repro

wirte code

open Mono.Debugger.Client
open Mono.Debugging.Client

open System.IO

[<Sealed; Command>]
type MyCommand() =
    inherit Command()
    override __.Names   = [|"mycmd"|]
    override __.Summary = "aaa bbb ccc"
    override __.Syntax  = "ddd eee fff"
    override __.Help    = "Help Help Help"
    override __.Process(args) =

        let file = new FileInfo(args)
        Debugger.Run(file)

        let width = System.Console.WindowWidth
        Log.Info(String.replicate width "─" )

02 compile and move dll
$ fsharpc -a -r:$(dirname $(which sdb))/../lib/sdb/sdb.exe test.fsx
$ mv test.dll ../.sdb/

03 launch sdb and run
$ sdb
(sdb) mycmd foo.exe
...
(sdb)                  // <--- I can not watch input strings (^_^;;;

work around
add following
let ps = new System.Diagnostics.Process()
ps.StartInfo.FileName <- "/bin/stty"
ps.StartInfo.Arguments <- "echo"
ps.Start() |> ignore
Log.Info(string(ps.Id))
()

Hello! I'm callmekohei!
Solved! (^_^)b
It's need to sleep time ( 5ms ).
open Mono.Debugger.Client
open Mono.Debugging.Client

open System.IO

[<Sealed; Command>]
type MyCommand() =
    inherit Command()
    override __.Names   = [|"mycmd"|]
    override __.Summary = "aaa bbb ccc"
    override __.Syntax  = "ddd eee fff"
    override __.Help    = "Help Help Help"
    override __.Process(args) =

        let file = new FileInfo(args)
        Debugger.Run(file)

        let width = System.Console.WindowWidth
        Log.Info(String.replicate width "─" )

        Process.Start("stty","echo")
        System.Threading.Thread.Sleep 5
        ()

Thanks (^_^)/


Make previous submission for a PII challenge load straight into the answer box

Resolves #1463
This method mimics the way we do it in CodeWOF. It avoids the JS step by just putting it in the answer box to begin with

Will reformat as a hotfix PR


NPM Audit Fix Run

A run on npm audit to remove/reduce security vulnerabilities.

always like a good npm audit 👍


Questions about Fig.5.

Hello, you've did a great work!
I've read your paper but take a lot of time to think about fig.5, So I wonder if you can help me.
There is my question: In the bottom figure, it seems like the height of patches are different in the left one and right one, so I guess it means the area of segment mask for specific label in images. But it becomes almost the same in the middle, why? Did I made a mistake?

Thanks for reading.
The bottom of Fig.5 shows the transformation from left to right. Actually, after transforming the living room to the bedroom, we mainly focus on what objects the transformed would become next.
That is why it seems the same in the middle.


ENH: static typing for qhull

Reference issue

https://github.com/scipy/scipy/issues/13299: static typing for qhull.pyx
What does this implement/fix?

Add qhull.pyi stub file to declare types for classes/functions in qhull.pyx
imports numpy.typing if possible, otherwise numpy.typing.ArrayLike falls back to Any

Additional information

Unable to import __future__ annotations in Cython, so we need stub files
I tried to type all functions that were possible, the only thing I think I missed was the qhull_options argument of the constructor for _QhullUsers because it has dynamic type during runtime (str and bytes)
mypy appears to parse the file:

import numpy as np
from scipy.spatial import ConvexHull
ConvexHull([]).add_points(np.random.rand(30, 2), 'not a bool')

# mypy output: test.py:3: error: Argument 2 to "add_points" of "ConvexHull" has incompatible type "str"; expected "bool"  [arg-type]

Interesting, thanks @mckib2

I'm not sure if these annotations make it to IDEs or not

Trying with mypy on some example code would be the easiest - if mypy sees the annotations, then IDEs should work too. Anyway, mypy is the most important consumer, if it doesn't see these annotations inside .pyx files, then they won't be very useful and we need stub files instead.

Looking at this a little bit more, it looks like mypy has no way of getting any info from Cython sources, so stubs it is.  Even though it looked like Cython was using some of the extra typing information, I'll go ahead and leave qhull.pyx unchanged.
My remaining concern is about how to keep the stubs and sources synced.  Is there a way we could test to make sure that the stubs were current?  If this isn't easy, it doesn't seem like that's in the scope of this PR, just a thought

My remaining concern is about how to keep the stubs and sources synced. Is there a way we could test to make sure that the stubs were current? If this isn't easy, it doesn't seem like that's in the scope of this PR, just a thought

Should be relatively straightforward, I hope. First step is to remove
[mypy-scipy.spatial.qhull]
ignore_missing_imports = True

from mypy.ini, then run the tests with python runtests.py --mypy (see gh-11706).
You may see other failures though, because I think we haven't added this in CI - if so and it's more than a few, let's open a separate issue for that and fix it up. Now that numpy 1.20 has decent typing, we can get back to adding annotations in scipy much more easily.

My remaining concern is about how to keep the stubs and sources synced. Is there a way we could test to make sure that the stubs were current? If this isn't easy, it doesn't seem like that's in the scope of this PR, just a thought

Should be relatively straightforward, I hope. First step is to remove
[mypy-scipy.spatial.qhull]
ignore_missing_imports = True

from mypy.ini, then run the tests with python runtests.py --mypy (see gh-11706).
You may see other failures though, because I think we haven't added this in CI - if so and it's more than a few, let's open a separate issue for that and fix it up. Now that numpy 1.20 has decent typing, we can get back to adding annotations in scipy much more easily.

Thanks for pointing out mypy tests!
There are more than a few errors when I run (against numpy==1.20.0rc2):
Found 97 errors in 22 files (checked 647 source files)

I'll open another PR to see what we can do about those error.

Thanks for pointing out mypy tests!
There are more than a few errors when I run (against numpy==1.20.0rc2):
Found 97 errors in 22 files (checked 647 source files)

I'll open another PR to see what we can do about those error.


Possible To Add Support for AutoGluon Models?

This may exist today, but I cannot figure out how to do it.
It appears that lime wants a 1d array as the data_row parameter in the .explain_instance method. However AutoGluon Predictor Fns appear to only support their type of TabularDataset or that of a pd.DataFrame.
Both AutoGluon & Lime are great packages and it would be awesome if these two projects played nice and were able to be used together. Let me know if I am missing something or if this is supported today and I just am unable to figure it out. Thanks!

Was Able to solve this with a snippit of Autogluon code that wraps their predictor ... See below:
class AutogluonWrapper:
    def __init__(self, predictor, feature_names, target_class=None):
        self.ag_model = predictor
        self.feature_names = feature_names
        self.target_class = target_class

    def predict_proba(self, X):
        if isinstance(X, pd.Series):
            X = X.values.reshape(1,-1)
        if not isinstance(X, pd.DataFrame):
            X = pd.DataFrame(X, columns=self.feature_names)
        preds = self.ag_model.predict_proba(X)
        if predictor.problem_type == "regression" or self.target_class is None:
            return preds
        else:
            return preds.to_numpy()


[Bug]: testcontainers-postgresql wait strategy invalid when persistent volume is set

Module
PostgreSQL
Testcontainers version
1.19.3
Using the latest Testcontainers version?
Yes
Host OS
Windows
Host Arch
x86
Docker version
Client:
 Version:           24.0.2-rd
 API version:       1.42 (downgraded from 1.43)
 Go version:        go1.20.4
 Git commit:        e63f5fa
 Built:             Fri May 26 16:43:15 2023
 OS/Arch:           windows/amd64
 Context:           default

Server:
 Engine:
  Version:          23.0.6
  API version:      1.42 (minimum version 1.12)
  Go version:       go1.20.4
  Git commit:       9dbdbd4b6d7681bd18c897a6ba0376073c2a72ff
  Built:            Fri May 12 13:54:36 2023
  OS/Arch:          linux/amd64
  Experimental:     false
 containerd:
  Version:          v1.7.2
  GitCommit:        0cae528dd6cb557f7201036e9f43420650207b58
 runc:
  Version:          1.1.7
  GitCommit:        860f061b76bb4fc671f0f9e900f7d80ff93d4eb7
 docker-init:
  Version:          0.19.0
  GitCommit:

What happened?
When I mount a persistent volume to the PostgreSQL container at /var/lib/postgresql/data, all subsequent startups will fail with error
Caused by: org.testcontainers.containers.ContainerLaunchException: Timed out waiting for log output matching '.*database system is ready to accept connections.*\s'

When observing docker logs, the following is shown for fresh container
The files belonging to this database system will be owned by user "postgres".
This user must also own the server process.

The database cluster will be initialized with locale "en_US.utf8".
The default database encoding has accordingly been set to "UTF8".
The default text search configuration will be set to "english".

Data page checksums are disabled.

fixing permissions on existing directory /var/lib/postgresql/data ... ok
creating subdirectories ... ok
selecting dynamic shared memory implementation ... posix
selecting default max_connections ... 100
selecting default shared_buffers ... 128MB
selecting default time zone ... UTC
creating configuration files ... ok
running bootstrap script ... ok
sh: locale: not found
2023-12-20 14:25:43.715 UTC [30] WARNING:  no usable system locales were found
performing post-bootstrap initialization ... ok
syncing data to disk ... ok

Success. You can now start the database server using:

    pg_ctl -D /var/lib/postgresql/data -l logfile start

initdb: warning: enabling "trust" authentication for local connections
initdb: hint: You can change this by editing pg_hba.conf or using the option -A, or --auth-local and --auth-host, the next time you run initdb.
waiting for server to start....2023-12-20 14:25:44.080 UTC [36] LOG:  starting PostgreSQL 16.1 on x86_64-pc-linux-musl, compiled by gcc (Alpine 13.2.1_git20231014) 13.2.1 20231014, 64-bit
2023-12-20 14:25:44.081 UTC [36] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2023-12-20 14:25:44.082 UTC [39] LOG:  database system was shut down at 2023-12-20 14:25:43 UTC
2023-12-20 14:25:44.084 UTC [36] LOG:  database system is ready to accept connections
 done
server started
CREATE DATABASE

/usr/local/bin/docker-entrypoint.sh: ignoring /docker-entrypoint-initdb.d/*

waiting for server to shut down....2023-12-20 14:25:44.201 UTC [36] LOG:  received fast shutdown request
2023-12-20 14:25:44.201 UTC [36] LOG:  aborting any active transactions
2023-12-20 14:25:44.202 UTC [36] LOG:  background worker "logical replication launcher" (PID 42) exited with exit code 1
2023-12-20 14:25:44.202 UTC [37] LOG:  shutting down
2023-12-20 14:25:44.202 UTC [37] LOG:  checkpoint starting: shutdown immediate
2023-12-20 14:25:44.206 UTC [37] LOG:  checkpoint complete: wrote 925 buffers (5.6%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.005 s, sync=0.001 s, total=0.005 s; sync files=0, longest=0.000 s, average=0.000 s; distance=4269 kB, estimate=4269 kB; lsn=0/191BA30, redo lsn=0/191BA30
2023-12-20 14:25:44.209 UTC [36] LOG:  database system is shut down
 done
server stopped

PostgreSQL init process complete; ready for start up.

2023-12-20 14:25:44.313 UTC [1] LOG:  starting PostgreSQL 16.1 on x86_64-pc-linux-musl, compiled by gcc (Alpine 13.2.1_git20231014) 13.2.1 20231014, 64-bit
2023-12-20 14:25:44.314 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
2023-12-20 14:25:44.314 UTC [1] LOG:  listening on IPv6 address "::", port 5432
2023-12-20 14:25:44.314 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2023-12-20 14:25:44.315 UTC [52] LOG:  database system was shut down at 2023-12-20 14:25:44 UTC
2023-12-20 14:25:44.316 UTC [1] LOG:  database system is ready to accept connections

When observing docker logs for a container that starts with pre-existing database
PostgreSQL Database directory appears to contain a database; Skipping initialization

2023-12-20 14:27:21.144 UTC [1] LOG:  starting PostgreSQL 16.1 on x86_64-pc-linux-musl, compiled by gcc (Alpine 13.2.1_git20231014) 13.2.1 20231014, 64-bit
2023-12-20 14:27:21.144 UTC [1] LOG:  listening on IPv4 address "<IP_ADDRESS>", port 5432
2023-12-20 14:27:21.144 UTC [1] LOG:  listening on IPv6 address "::", port 5432
2023-12-20 14:27:21.144 UTC [1] LOG:  listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432"
2023-12-20 14:27:21.145 UTC [24] LOG:  database system was interrupted; last known up at 2023-12-20 14:25:44 UTC
2023-12-20 14:27:21.145 UTC [24] LOG:  database system was not properly shut down; automatic recovery in progress
2023-12-20 14:27:21.146 UTC [24] LOG:  redo starts at 0/191BAA8
2023-12-20 14:27:21.148 UTC [24] LOG:  invalid record length at 0/19A0218: expected at least 24, got 0
2023-12-20 14:27:21.148 UTC [24] LOG:  redo done at 0/19A01E0 system usage: CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.00 s
2023-12-20 14:27:21.148 UTC [22] LOG:  checkpoint starting: end-of-recovery immediate wait
2023-12-20 14:27:21.150 UTC [22] LOG:  checkpoint complete: wrote 122 buffers (0.7%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.002 s, sync=0.001 s, total=0.002 s; sync files=0, longest=0.000 s, average=0.000 s; distance=529 kB, estimate=529 kB; lsn=0/19A0218, redo lsn=0/19A0218
2023-12-20 14:27:21.152 UTC [1] LOG:  database system is ready to accept connections

If I test the regex shown above on https://www.freeformatter.com/java-regex-tester.html for first one I get

However, if I test the same regex against second output

Note that in second log output, there is no line after database system is ready to accept connections string. In the first example, it seems to match the middle result
2023-12-20 14:25:44.084 UTC [36] LOG:  database system is ready to accept connections
 done

If I remove final \\s from regex, then it works for already started container.
Edit: It appears that times(2) is problematic, and not the regex itself.
Relevant log output
No response
Additional Information
No response

Fixed with override
public class PSQLContainer extends PostgreSQLContainer<PSQLContainer> {
    public PSQLContainer(DockerImageName dockerImageName, boolean persistenceEnabled) {
        super(dockerImageName);

        this.waitStrategy =
                new LogMessageWaitStrategy()
                        .withRegEx(".*database system is ready to accept connections.*\\s")
                        .withTimes(persistenceEnabled ? 1 : 2)
                        .withStartupTimeout(Duration.of(60, ChronoUnit.SECONDS));
    }
}

If persistence is not supposed to work, simply close the bug.

Alternative wait strategy that mostly seems to work just fine most of the time, just a bit more noisy on the container logs
public class PSQLContainer extends PostgreSQLContainer<PSQLContainer> {
    public PSQLContainer(DockerImageName dockerImageName) {
        super(dockerImageName);

        this.waitStrategy =
                new ShellStrategy()
                        .withCommand(
                                "psql -q -o /dev/null -c \"SELECT 1\" -d $POSTGRES_DB -U $POSTGRES_USER")
                        .withStartupTimeout(Duration.of(60, ChronoUnit.SECONDS));
    }
}

In case somebody needs it.


Added format option to allow color picker widget to return alternate color formats

Added format option to allow color picker widget to return alternate color formats.
Furthermore, now it is also possible to enable the slider for the alpha channel for transparency.
https://github.com/wintercms/winter/issues/94

Replaced by #33, thanks for your help @webbati!


key UTXOs by (txid, vout)

instead of address, which shouldn't have ever been done in the first place.
Fixes #30. Thanks to @billygarrison for finding this.

Tested on testnet.

Tested on Windows (testnet)


Drone cache is stored in /tmp/drone by default

This should respect the system $TMPDIR rather than being hard coded to /tmp/drone.
We are currently using the undocumented $DRONE_TMP variable to override this, but it feels like it should default to $TMPDIR.
ioutil.TempDir should return the proper temporary directory.

i got the problem too


[SPARK-6733][ Scheduler]Added scala.language.existentials

Can one of the admins verify this patch?

Jenkins, test this please.

(note -- for future references for a pr like this maybe you don't need a jira)

Test build #29782 has started for   PR 5384 at commit 82a3a1f.

I've not raise in JIRA, Reporter is Raymond Tay

Test build #29782 has finished for   PR 5384 at commit 82a3a1f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.
This patch does not change any dependencies.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/29782/
Test PASSed.

I agree that both of these fix warnings. LGTM.

I'm merging this. Thanks.


AKS nodepools

What problem are you facing?
There is no option to add node pools to the AKS cluster.
How could Crossplane help solve your problem?
Add it to the API.

Thanks for bringing this up @vfarcic, there's been more interest in this recently :)
Does #262 cover this request already for you?

My bad. I should have found that issue before creating a new one.


Build and runtime error on MacOS Catalina

When trying to build MaterialXView, I encounter following error with Xcode 12.4 on Catalina 10.15.7 (sorry for the outdated setup!):
[100%] Linking CXX executable ../../bin/MaterialXView
Undefined symbols for architecture x86_64:
  "_OBJC_CLASS_$_MTLTileRenderPipelineDescriptor", referenced from:
      objc-class-ref in libMaterialXRenderMsl.a(MetalState.mm.o)
ld: symbol(s) not found for architecture x86_64
clang-14: error: linker command failed with exit code 1 (use -v to see invocation)
make[3]: *** [bin/MaterialXView] Error 1
make[2]: *** [source/MaterialXView/CMakeFiles/MaterialXView.dir/all] Error 2
make[1]: *** [source/MaterialXView/CMakeFiles/MaterialXView.dir/rule] Error 2
make: *** [MaterialXView] Error 2

The issue can be resolved as follows
--- a/source/MaterialXGenMsl/CMakeLists.txt
+++ b/source/MaterialXGenMsl/CMakeLists.txt
@@ -21,15 +21,18 @@ set_target_properties(
     OUTPUT_NAME ${MATERIALX_MODULE_NAME}${MATERIALX_LIBNAME_SUFFIX}
     COMPILE_FLAGS "${EXTERNAL_COMPILE_FLAGS}"
     LINK_FLAGS "${EXTERNAL_LINK_FLAGS}"
+    INTERFACE_LINK_LIBRARIES "-undefined dynamic_lookup"
     INSTALL_RPATH "${MATERIALX_SAME_DIR_RPATH}"
     VERSION "${MATERIALX_LIBRARY_VERSION}"
     SOVERSION "${MATERIALX_MAJOR_VERSION}")

which, however, leads to a number of linker warnings.
Starting MaterialXView causes a sanitizer error
/Users/pablode/tmp/MaterialX/source/MaterialXRenderMsl/MetalState.mm:60:29: runtime error: load of value 190, which is not a valid value for type 'bool'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /Users/pablode/tmp/MaterialX/source/MaterialXRenderMsl/MetalState.mm:60:29 in
[1]    48884 abort      ./bin/MaterialXView

which can be fixed using following patch:
--- a/source/MaterialXRenderMsl/MetalState.h
+++ b/source/MaterialXRenderMsl/MetalState.h
@@ -56,7 +56,7 @@ struct MetalState
     id<MTLRenderCommandEncoder>         renderCmdEncoder = nil;
     std::stack<MaterialX::MetalFramebufferPtr> framebufferStack;

-    bool supportsTiledPipeline;
+    bool supportsTiledPipeline = false;

     id<MTLDepthStencilState> opaqueDepthStencilState = nil;
     id<MTLDepthStencilState> transparentDepthStencilState = nil;

Good catches, @pablode, and these would be good to address in the upcoming 1.38.8 release.  Would you have the bandwidth to propose a pull request for these changes, or should we aim to address them on our side?

Unfortunately I don't have time to investigate this further right now.
I submitted this as an issue because I have concerns about the -undefined dynamic_lookup approach, which apparently could be replaced by a symbol-specific flag (ld -U, which I wasn't able to get to work, possibly due to Obj-C interop). Furthermore, a dozen warnings are thrown and possibly more on newer macOS versions.

Completely understood, @pablode, and given that additional context, we'll likely want to continue the discussion about this proposed fix beyond the timeline of 1.38.8.


Why is config.validate() called inside UnroutedHandler? Should we let user decide for themselves?

config.validate() should not be called in UnroutedHandler as it is hydrating the config with defaults, it is not just validating it. It is expected that NewHandler comes with all the bells and whistles via the validate call but UnroutedHandler should be slightly more primitive. User can always copy paste the code.
Alternatively, we can make Validate public and let user call it themselves.
config.isAbs should not be private if we wish for user to construct their own validate() function.

I don't understand your point, sorry. Yes, validate might be ill-named as it also sets the defaults, but why would you want to disable this behavior? The handler depends on all config fields to be set, so it is important that default values are set when no value is provided by the caller. In the end, this also benefits the user as it reduces the possibility of errors.

UnroutedHandler should be slightly more primitive. User can always copy paste the code.

I disagree with both. UnroutedHandler should be the same as the routed Handler, except for the routing. Everything else should be the same. Also, requiring the user to duplicate internal code is not the preferred approach.


flutter_inappwebview-Swift.h not found

Plugin works fine in Android. iOS won't build due to
'flutter_inappwebview/flutter_inappwebview-Swift.h' file not found
Im not very experienced with Xcode, but it is Xcode 11.3
I have tried deleting
pubspec.lock
Podfile.lock
/ios pod install
flutter clean
with no impact.
in the Changelog it makes mention of --ios-language swift, but i don't know what this really means or how I would verify this setting.
Thoughts?
#UPDATE
Attempting a build using use_frameworks! in gradle seems to have an impact but is not consistent.
So if I,
--Build with use_frameworks! - Build fails due to Facebook Login plugin
--Build without use_frameworks! - Build succeeds and video plays well
--Build again without use_frameworks! - Build fails with original error.
Process can be repeated. I don't really understand what use_frameworks! does or why attempting a build with it makes an impact.

Issue has been resolved as of this moment.
I added use_frameworks! to my Podfile, which caused other packages to break (looking at you Facebook Login), after which I removed that line, reran my build for giggles and it now is working.
I'm not sure what difference that failing build made.

Issue persists after second build. So if I,
--Build with use_frameworks! - Build fails due to Facebook Login plugin
--Build without use_frameworks! - Build succeeds and video plays well
--Build again without use_frameworks! - Build fails with original error.

I am facing the same issue with flutter v1.22.6 and youtube_player_flutter v7.0.0+7.


Replace C arrays with std::array

Description
SFML uses C style arrays quite a lot so I wanted to start chipping away at replacing them with std::array. I chose to start with instances where switching to std::array was particularly straightforward. Eventually it would be nice to enable the clang-tidy check that forbids C arrays.
For more reading on why C style arrays are not ideal, see the C++ Core Guidelines.
https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#slcon1-prefer-using-stl-array-or-vector-instead-of-a-c-array

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

24 of 46 (52.17%)  changed or added relevant lines in 10 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.04%) to 53.001%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/SFML/Graphics/Texture.cpp
0
2
0.0%

src/SFML/Network/TcpSocket.cpp
0
3
0.0%

src/SFML/Audio/AudioDevice.cpp
0
4
0.0%

src/SFML/Audio/SoundFileWriterWav.cpp
0
13
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.04%

Covered Lines:
10345

Relevant Lines:
18507

💛  - Coveralls

Rebased on master and fixed merge conflicts


use port 8443 to not need root or additional caps

This is relevant on Kubernetes that has additional security restrictions in place.

Thank you!


Support Ubuntu 20.04

with SQL Server 2019.
Also changes testing to allow testing in branches and pull requests.

Hey @krlmlr, thanks for the PR! I'd like to wait until a 20.04 repository is available.

These are the official instructions to install on 20.04 -- sure we may need to change at some point.
https://docs.microsoft.com/en-us/sql/connect/odbc/linux-mac/installing-the-microsoft-odbc-driver-for-sql-server?view=sql-server-ver15

I'm mistaken -- it works well enough on 20.04 (tests pass for dm), agree to wait until it's officially supported.

Per https://docs.microsoft.com/en-us/sql/linux/quickstart-install-connect-ubuntu?view=sql-server-ver15:

Ubuntu 20.04 is supported starting with SQL Server 2019 CU10.

Awesome! Can you back out the changes to index.js?


Update ciris, ciris-refined to 2.1.0

Updates

is.cir:ciris
is.cir:ciris-refined

from 2.0.1 to 2.1.0.
GitHub Release Notes - Version Diff
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.
Configure Scala Steward for your repository with a .scala-steward.conf file.
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 = "is.cir" } ]

labels: library-update, semver-minor

Codecov Report

Merging #165 (2ba4ca4) into develop (ad60b43) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #165   +/-   ##
========================================
  Coverage    50.00%   50.00%
========================================
  Files            5        5
  Lines           84       84
========================================
  Hits            42       42
  Misses          42       42

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 ad60b43...2ba4ca4. Read the comment docs.


Add automerge bot

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/enlighten-feedstock#6.


nixos/mail/dovecot2: re-introduce extra settings and rename sieveScripts

Description of changes
https://github.com/NixOS/nixpkgs/pull/275031 introduced structured configuration for the dovecot2 sieve plugin, by doing so, it broke SNM configuration doing Sieve configurations.
I also made imapsieve.mailbox.*.causes a list instead of an element.
I introduced a warning mechanism to give help to users who may have used extraConfig and will have it explode on them.
This attempts to fix up the public API to make it possible for SNM to pick up the pieces.
Here's patch for SNM, GitLab is too broken for me to open a PR over there:
From a15c4eef88d126ea9d02efbfbfdfd0fcbf9a9976 Mon Sep 17 00:00:00 2001
From: Raito Bezarius<EMAIL_ADDRESS>Date: Sun, 14 Jan 2024 20:55:20 +0100
Subject: [PATCH] dovecot: support new `sieve` API in nixpkgs

Since https://github.com/NixOS/nixpkgs/pull/275031 things have became more structured
when it comes to the sieve plugin.

Relies on https://github.com/NixOS/nixpkgs/pull/281001 for full
features.
---
 mail-server/dovecot.nix | 50 +++++++++++++++++++++--------------------
 1 file changed, 26 insertions(+), 24 deletions(-)

diff --git a/mail-server/dovecot.nix b/mail-server/dovecot.nix
index 7d73ee2..3e5f9fb 100644
--- a/mail-server/dovecot.nix
+++ b/mail-server/dovecot.nix
@@ -175,8 +175,14 @@ in
       mailPlugins.globally.enable = lib.optionals cfg.fullTextSearch.enable [ "fts" "fts_xapian" ];
       protocols = lib.optional cfg.enableManageSieve "sieve";

-      sieveScripts = {
-        after = builtins.toFile "spam.sieve" ''
+      pluginSettings = {
+        sieve = "file:${cfg.sieveDirectory}/%u/scripts;active=${cfg.sieveDirectory}/%u/active.sieve";
+        sieve_default = "file:${cfg.sieveDirectory}/%u/default.sieve";
+        sieve_default_name = "default";
+      };
+
+      sieve = {
+        scripts.after = builtins.toFile "spam.sieve" ''
           require "fileinto";

           if header :is "X-Spam" "Yes" {
@@ -184,8 +190,26 @@ in
               stop;
           }
         '';
+
+        pipeBins = [
+          pipeBin
+        ];
       };

+      imapsieve.mailbox = [
+        {
+          name = junkMailboxName;
+          causes = [ "COPY" "APPEND" ];
+          before = "${stateDir}/imap_sieve/report-spam.sieve";
+        }
+        {
+          name = "*";
+          from = junkMailboxName;
+          causes = [ "COPY" ];
+          before = "${stateDir}/imap_sieve/report-ham.sieve";
+        }
+      ];
+
       mailboxes = cfg.mailboxes;

       extraConfig = ''
@@ -307,28 +331,6 @@ in
           inbox = yes
         }

-        plugin {
-          sieve_plugins = sieve_imapsieve sieve_extprograms
-          sieve = file:${cfg.sieveDirectory}/%u/scripts;active=${cfg.sieveDirectory}/%u/active.sieve
-          sieve_default = file:${cfg.sieveDirectory}/%u/default.sieve
-          sieve_default_name = default
-
-          # From elsewhere to Spam folder
-          imapsieve_mailbox1_name = ${junkMailboxName}
-          imapsieve_mailbox1_causes = COPY,APPEND
-          imapsieve_mailbox1_before = file:${stateDir}/imap_sieve/report-spam.sieve
-
-          # From Spam folder to elsewhere
-          imapsieve_mailbox2_name = *
-          imapsieve_mailbox2_from = ${junkMailboxName}
-          imapsieve_mailbox2_causes = COPY
-          imapsieve_mailbox2_before = file:${stateDir}/imap_sieve/report-ham.sieve
-
-          sieve_pipe_bin_dir = ${pipeBin}/pipe/bin
-
-          sieve_global_extensions = +vnd.dovecot.pipe +vnd.dovecot.environment
-        }
-
         ${lib.optionalString cfg.fullTextSearch.enable ''
         plugin {
           plugin = fts fts_xapian
--
2.43.0

I also needed to add services.dovecot2.sieve.extensions = [ "fileinto" ];, I'm not sure why.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

cc @nlewo

Here's patch for SNM, GitLab is too broken for me to open a PR over there:

Filed as https://gitlab.com/simple-nixos-mailserver/nixos-mailserver/-/merge_requests/316.

IF everything can be configured via structured settings, why do we not remove the extraConfig (but with a proper mkRemovedOptionModule error)?
Then nixos-simple-mailserver can just adjust to use the structured settings.

@2xsaiko In the future, it's my personal opinion, but I would prefer to avoid dealing with formatting nitpicks as those are pointless (in the presence of the formatting RFC which will trigger a treewide anyway) work that interrupts moving forward things, the other fix was… well.

Alright!

IF everything can be configured via structured settings, why do we not remove the extraConfig (but with a proper mkRemovedOptionModule error)? Then nixos-simple-mailserver can just adjust to use the structured settings.

The structured settings right now, at least in this PR, are only for the plugin block, I'm going to have to look into the rest of the configuration syntax to see if it's possible to make it structured. From what I've seen it should be possible though.

Nope, works fine here now.

There still seem to be some problems, the generated dovecot conf contains these lines:
plugin {
    imapsieve_mailbox1_before = file:/var/lib/dovecot/imapsieve/before/report-spam.sieve
  imapsieve_mailbox1_causes = COPY,APPEND
  imapsieve_mailbox1_name = Junk
  imapsieve_mailbox2_before = file:/var/lib/dovecot/imapsieve/before/report-ham.sieve
  imapsieve_mailbox2_causes = COPY
  imapsieve_mailbox2_from = Junk
  imapsieve_mailbox2_name = *
  sieve = file:/var/sieve/%u/scripts;active=/var/sieve/%u/active.sieve
  sieve_after = /var/lib/dovecot/sieve//nix/store/gca1qh0ipsqf7c9h3z7jcagygcpkadbi-spam.sieve
  sieve_after2 = /var/lib/dovecot/sieve//nix/store/5126h7gj5jnmppj2ynxacw0yqibx2qd7-custom-spam.sieve
  sieve_default = file:/var/sieve/%u/default.sieve
  sieve_default_name = default
  sieve_extensions = +editheader +fileinto
  sieve_global_extensions = +vnd.dovecot.pipe
  sieve_pipe_bin_dir = /nix/store/bflj5daxrrfmlj64anfq5dydwd5xrfjb-sieve-pipe-bins
  sieve_plugins = sieve_imapsieve sieve_extprograms
}

The location for sieve_after and sieve_after2 is not correct.
This is with this PR and the SNM PR applied.

@chvp could you try with #284749?

Good catch, the from/to natural variables there are confusing…


Throttled daemon fails to start on boot

Describe the bug
Throttled works if started manually, but the daemon is failing to start on boot (NixOS unstable).
Steps To Reproduce
Steps to reproduce the behavior:

Add to your configuration.nix:

services.throttled.enable = true;

Change to unstable channel and run nixos-rebuild switch
Reboot
Throttled's daemon should fail to initialize
Run systemctl status throttled.service and check the error info

Expected behavior
Throttled's daemon should fail to initialize on boot with this error:
[space@CirnOS:~]$ systemctl status throttled.service
× throttled.service - Stop Intel throttling
     Loaded: loaded (/etc/systemd/system/throttled.service; enabled; preset: enabled)
    Drop-In: /nix/store/mpld67zwhiq4i6gdim71zjb7qwhdc64a-system-units/throttled.service.d
             └─overrides.conf
     Active: failed (Result: exit-code) since Mon 2023-09-11 10:16:06 WEST; 20s ago
   Duration: 241ms
    Process: 692 ExecStart=/nix/store/6y21vx7nskvzdfw3x3bkqwd33r1pf10j-throttled-0.10.0/bin/throttled.py>
   Main PID: 692 (code=exited, status=1/FAILURE)
         IP: 0B in, 0B out
        CPU: 85ms

set 11 10:16:06 CirnOS throttled.py[692]:     retcode = call(*popenargs, **kwargs)
set 11 10:16:06 CirnOS throttled.py[692]:   File "/nix/store/vagb0sjv83ybi435i6iiv10hjrdghph9-python3-3.>
set 11 10:16:06 CirnOS throttled.py[692]:     with Popen(*popenargs, **kwargs) as p:
set 11 10:16:06 CirnOS throttled.py[692]:   File "/nix/store/vagb0sjv83ybi435i6iiv10hjrdghph9-python3-3.>
set 11 10:16:06 CirnOS throttled.py[692]:     self._execute_child(args, executable, preexec_fn, close_fd>
set 11 10:16:06 CirnOS throttled.py[692]:   File "/nix/store/vagb0sjv83ybi435i6iiv10hjrdghph9-python3-3.>
set 11 10:16:06 CirnOS throttled.py[692]:     raise child_exception_type(errno_num, err_msg, err_filenam>
set 11 10:16:06 CirnOS throttled.py[692]: FileNotFoundError: [Errno 2] No such file or directory: 'modpr>
set 11 10:16:06 CirnOS systemd[1]: throttled.service: Main process exited, code=exited, status=1/FAILURE
set 11 10:16:06 CirnOS systemd[1]: throttled.service: Failed with result 'exit-code'.

Notify maintainers
@michaelpj

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[space@CirnOS:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.5.1, NixOS, 23.11 (Tapir), 23.11pre523790.db9208ab987c`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.17.0`
 - channels(space): `"nixos"`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/throttled-failed-to-run/34742/2

PR to fix: #264582


⚡Host-Specific Collection Creation in Burp Upload

💭 Introduction:
Akto offers multiple channels for generating an API inventory, including AWS, GCP, and Burp. APIs are currently organised as a part of a collection, with all APIs uploaded via Burp automatically assigned to the Burp collection. However, we aim to enhance this system by creating host-specific collections to improve utilisation. This will allow us to better organise APIs according to host, enabling more efficient management and use of our API inventory.
🎯 Requirements:

Experience with Java
Basic understanding of Burp

✅ Task summary:

[ ] Ask to be assigned to the issue.
[ ] Wait to be assigned. We will try to assign in less than 2 hours.
[ ] Currently, we use same apiCollectionId for all API uploads via Burp. Check here.
[ ] We will have to do a lot of 🪄 here. Firstly, extract the "host" header from the request headers. If null, use original apiCollectionId.
[ ] Secondly, use a new API collection id based on this "host" header. A simple strategy is to do hostHeader.hashCode(). Create this collection in a similar manner as this one here.
[ ] Use the new API Collection id as collection_id instead of result.put("akto_vxlan_id", collection_id+"");
[ ] Submit a pull request here

🙋🏼♂ Questions:
If you have questions, need any help, or just want to hang out, make sure to join us on our Discord server.

Hi if this issue is still open, I would love to be assigned on it. Currently using SpringBoot for few of my personal projects and I think I can resolve the issue and submit the pull request.

I've assigned it to you, @atharvamalji . Happy hacking! Feel free to join our Discord if you need assistance.
PS: This is a challenging issue.. so I will suggest you discuss the approach with our team before you start the implementation.


Feat(plugins): Add strict mode and ignore_case flags to natural_sort filter

Change Summary
This PR adds two new flags to the natural_sort filter

a strict flag to be able to raise when the sort_key is missing in one item in the iterable dict or Namespace (default is False for backward compatibility).
a ignore_case flag to be able to not ignore the case. Today natural_sort calls lower by default on some strings and this leads to AVD generating config not in the same order as EOS which does not use lower.

Related Issue(s)
Fixes #4242
Component(s) name
plugins
Proposed changes
Adding two flags to the function, with backward compatible default values to avoid impacting the rest of AVD
The goal is then to review the config ordering we have today against EOS and to use the flags where appropriate.
How to test
pytests have been added
Checklist
Note to reviewers

[x] ignore_case: False for a list of string effectively reimplements builtin sort from Jinja -> is it worth it?

Repository Checklist

[x] My code has been rebased from devel before I start
[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation and documentation have been updated accordingly.
[x] I have updated molecule CI testing accordingly. (check the box if not applicable)

The special catch in the template of hardware_counters should be removed in 5.0.0. The negative test of that is causing the tests here to fail.

Sorry but I feel like there is a better way to name this. "ignore_case" flag would naturally imply that we're ignoring case, which would further imply the default behavior is being case-sensitive. The fact that "ignore_case" must be set so that we would NOT ignore_case is odd behavior in my opinion.


[R-T4.3-14] Publish/load RADON Models to/from RADON Template Publishing Service

ID
R-T4.3-14

Section
WP4: Modeling Environment

Type
FUNCTIONAL_SUITABILITY

User Story
As a Software Developer, I want to manage my RADON Models using the RADON Template Publishing Service.

Requirement
Push and load RADON Models to or from the RADON Template Publishing Service once users decide to finalize a model or want to modify an existing one.

Extended Description
Recently, the RADON Template Publishing Service has been introduced (cf. D5.3 Technology Library) as an additional layer on top of VCS to manage RADON Models. This service offers enterprise-ready features such as comprehensive user management and the option to publish RADON Models in a certain version such that the content can no longer be changed, which forces users to organize each modification in a new version, inspired by software package management tools. Essentially, this service is an instantiation of what we call “Template Library” in RADON, on top of the possibility to use Git to version your models. Therefore, the GMT must be able to push RADON Models to the RADON Template Publishing Service once users decide to finalize a model and put it into production. Similarly, the GMT needs to be able to pull a model from the service to modify it and push it as a new version back.

Affected Tools
GRAPHMODEL_TOOL

Means of Verification
Manual testing based on case studies.

Dependency
N/A

@gcasale please approve new requirement.

Approved.
Note: the description is not meant to discuss the TPS but the requirement, the part that should be kept is primarily "Therefore, ...".

@gcasale please review and approve.
XLB and UST agreed to integrate the Template Library Publishing Service (TLS) with an IDE plugin to query and publish TOSCA types and models. We noticed lately that it makes more sense to use Winery (GMT) only as an editor and the IDE to commit, publish, and update your models with the TLS. This will be then a similar workflow as in traditional software engineering where you use an IDE to write your code and then Git to commit and push. Therefore, we fulfill this requirement by implementing an Eclipse Che plugin to establish the connection between the TLS and IDE.

As this is not critical for exploitation we can go for this change.
Approved.
On Tue, 8 Sep 2020, 13:43 Michael Wurster<EMAIL_ADDRESS>wrote:

@gcasale https://github.com/gcasale please review and approve.
XLB and UST agreed to integrate the Template Library Publishing Service
(TLS) with an IDE plugin to query and publish TOSCA types and models. We
noticed lately that it makes more sense to use Winery (GMT) only as an
editor and the IDE to commit, publish, and update your models with the TLS.
This will be then a similar workflow as in traditional software engineering
where you use an IDE to write your code and then Git to commit and push.
Therefore, we fulfill this requirement by implementing an Eclipse Che
plugin to establish the connection between the TLS and IDE.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/radon-h2020/radon-gmt/issues/30#issuecomment-688840975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDBKEG4WLDE3XB2MEZOFIDSEYRHNANCNFSM4NYOV54A
.


Keyboard shortcuts to move to collaborators location

It would vastly improve accessibility if we could navigate to another collaborators location via keyboard shortcuts. I work with a visually impaired programmer that works so zoomed into VS17 that he effectively has to scroll two screens up and over to click on a collaborator to jump to their location so keyboard accessibility would save so much time on his end for sure. I certainly would love to be able to jump between people without having to leave the keyboard myself.
Something similar to the Windows shortcut, Win key + [num], which launches the app on the taskbar corresponding to the number key pressed would work well for this. Not sure what keybindings might be available for this sort of thing in VS/VS Code, but maybe something like Ctrl + Shift + 2  jumping you to the second collaborator on the GUI element.

@RyanWilsonBIBend We've actually been discussing the need for keyboard shortcuts for some of the key Live Share actions, so this is really timely feedback! 😄Besides jumping to another participant's file location, are there other actions that stand out as benefiting from keybindings? We could potentially use this request to track improving keyboard navigation in general.

@lostintangent The things I can think of that I'd use if there were keyboard shortcuts are

Jump to collaborator
Jump and pin to collaborator
Start VS Live Share Collaboration session
if/when #73 is implemented, keyboard shortcut to "summon" people to your location

@RyanWilsonBIBend Sounds good! Those actions are definitely on our radar for keyboard bindings (along with "summon").

We’re not able to prioritize this issue over the other higher-impact issues we receive every week, based on the votes and comments from others in the community and our understanding of the issue. However, rest assured that we love your input. If you feel it deserves to stay open, then clarify your use case and contact us to let us know how severe it’s for you.


HugoHelperDateIsNow breaks YAML and JSON front matter

Hugo defines 3 front matter formats: TOML, YAML, and JSON.
The HugoHelperDateIsNow command inserts date =, which works for TOML but breaks YAML and JSON front matter.
I haven't checked any of the other helpers. I suggest this either gets fixed or you put some sort of "TOML front matter only" caveat in the README.

I don't use Hugo anymore, so don't know if this is even still a problem anymore.


[KOGITO-4373] Remove kogito-internal and post kie-api unfork cleanup

https://issues.redhat.com/browse/KOGITO-4373
Summary
After kie-api unfork there are some additional cleanup that can be performed to isolate usages of old code. The goal of this PR is to remove kogito-internal module, remove kie7 dependencies from kogito-bom (so that users will not find them in the dependencyManagement and then update part of the codebase to use the Kogito* specific classes (KogitoProcessInstance etc)
Details

Removed kogito-internal module because it is empty now
Removed kie7 dependencies from kogito-bom, in this way user will not find them in the dependencyManagement and we don't want to use them directly (now where are used version is explicit)
Removed all kie-api and kie-internal exclusions
KogitoEventListener now is no more extending EventListener so it is decoupled from 7
KogitoWorkItemHandler now is no more extending WorkItemHandler so it is decoupled from 7
KogitoWorkItemManager now is no more extending WorkItemManager so it is decoupled from 7
Renamed org.kie.kogito.process.workitems.KogitoWorkItem as org.kie.kogito.process.workitems.InternalKogitoWorkItem to avoid name clash
Renamed org.kie.kogito.process.workitems.KogitoWorkItemManager as org.kie.kogito.process.workitems.InternalKogitoWorkItemManager to avoid name clash
Updated most of the usages of org.kie.api.runtime.process.ProcessInstance to org.kie.kogito.internal.process.runtime.KogitoProcessInstance (not updated where I was not sure)
Same for WorkItemManager -> KogitoWorkItemManager, KogitoWorkItem etc for all the new Kogito* classes

PR lists

https://github.com/kiegroup/kogito-runtimes/pull/1042
https://github.com/kiegroup/kogito-apps/pull/662

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

jenkins retest this please

jenkins retest this please

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

Manually killed, try again...
jenkins retest this please

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

The Linux check has failed. Please check the logs.

jenkins retest this please

The Linux check is successful.


EF 1.1.1 update EF 1.1.2 Throw Exception

EF 1.1.1 update EF 1.1.2 Throw Exception:
using (var db = _dbContext.Database.BeginTransaction())
{
//to do post put
db.Commit();
}
Task.Run(()=>{
//to do query
})
{
"name": "Microsoft.ApplicationInsights.Dev.Message",
"time": "2017-05-21T02:47:53.1603026Z",
"tags": {
"ai.internal.sdkVersion": "aspnet5c:2.1.0-beta1",
"ai.location.ip": "<IP_ADDRESS>",
"ai.operation.name": "GET MaintenanceTeams/GetAssignList",
"ai.cloud.roleInstance": "DESKTOP-8NCRL9O",
"ai.internal.nodeName": "DESKTOP-8NCRL9O",
"ai.application.ver": "<IP_ADDRESS>",
"ai.operation.id": "0HL4VUEUSTP6U",
"ai.operation.parentId": "0HL4VUEUSTP6U"
},
"data": {
"baseType": "MessageData",
"baseData": {
"ver": 2,
"message": "An exception occurred in the database while iterating the results of a query.\r\nSystem.InvalidOperationException: Invalid attempt to call Read when reader is closed.\r\n   at System.Data.SqlClient.SqlDataReader.TryReadInternal(Boolean setTimeout, Boolean& more)\r\n   at System.Data.SqlClient.SqlDataReader.Read()\r\n   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable.Enumerator.BufferlessMoveNext(Boolean buffer)\r\n   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](Func2 operation, Func2 verifySucceeded, TState state)\r\n   at Microsoft.EntityFrameworkCore.Query.QueryMethodProvider.<_ShapedQuery>d__31.MoveNext()\r\n   at System.Linq.Enumerable.SelectEnumerableIterator2.MoveNext()\r\n   at Microsoft.EntityFrameworkCore.Query.Internal.LinqOperatorProvider.ExceptionInterceptor1.EnumeratorExceptionInterceptor.MoveNext()",       "severityLevel": "Error",       "properties": {         "Exception": "System.InvalidOperationException: Invalid attempt to call Read when reader is closed.\r\n   at System.Data.SqlClient.SqlDataReader.TryReadInternal(Boolean setTimeout, Boolean& more)\r\n   at System.Data.SqlClient.SqlDataReader.Read()\r\n   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable.Enumerator.BufferlessMoveNext(Boolean buffer)\r\n   at Microsoft.EntityFrameworkCore.Storage.Internal.SqlServerExecutionStrategy.Execute[TState,TResult](Func2 operation, Func2 verifySucceeded, TState state)\r\n   at Microsoft.EntityFrameworkCore.Query.QueryMethodProvider.<_ShapedQuery>d__31.MoveNext()\r\n   at System.Linq.Enumerable.SelectEnumerableIterator2.MoveNext()\r\n   at Microsoft.EntityFrameworkCore.Query.Internal.LinqOperatorProvider.ExceptionInterceptor1.EnumeratorExceptionInterceptor.MoveNext()",
"AspNetCoreEnvironment": "Development",
"CategoryName": "Microsoft.EntityFrameworkCore.Query.Internal.SqlServerQueryCompilationContextFactory",
"DeveloperMode": "true"
}
}
}
}

@a764578566 does this reproduce if you await on the Task returned by Task.Run()? Otherwise, can you please provide complete repto code?

@divega  don't await on the Task returned by Task.Run()
Task Run(()=>{
//to do query and push message to app
//not affect return whether the operation is a success
});

@a764578566 if the delegate passed to Task.Run() contained any calls to the DbContext, including EF queries, that would likely be the cause of the error. That query will execute in a different thread and in the meanwhile the transaction will be disposed (I am basing this only on the code you posted, but likely the connection will be closed as well when the DbContext gets disposed).
In summary, if you are going to use the context in a separate thread, you need to await to make sure nothing else does anything on the context in originated thread until the Task is completed.

@divega Thanks
I wonder why it's wrong in EF 1.1.2

Closing this as by-design, per @divega's comments above. @a764578566 If this has been working for you in the past, it's only because you have been getting lucky with the timing of the threads. It would never have been expected to work reliably with any version of EF.


Connection to IBM i keeps disconnecting. Error in dev tools console.

After successfully connecting to the IBM i, I get a pop-up saying "Connection lost" with the following error in the console. Not sure if the error is related, though.
mainThreadExtensionService.ts:82 Error: No such file
	at 101 (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:232673)
	at SFTP.push (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:249360)
	at CHANNEL_DATA (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:167608)
	at 94 (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:332785)
	at e.exports.U (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:193609)
	at F.decrypt (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:289945)
	at e.exports.$ [as _parse] (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:193133)
	at e.exports.parse (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:197520)
	at Socket.<anonymous> (c:\Users\VRPG\.vscode\extensions\halcyontechltd.code-for-ibmi-2.10.1\dist\extension.js:2:169786)
	at Socket.emit (node:events:517:28)
	at addChunk (node:internal/streams/readable:335:12)
	at readableAddChunk (node:internal/streams/readable:308:9)
	at Readable.push (node:internal/streams/readable:245:10)
	at TCP.onStreamRead (node:internal/stream_base_commons:190:23)

Expected behavior
I expected the connection to maintain.
Screenshots

Environment (please complete the following information):

Extension version 2.10.1
IBM i OS version 7.4

Additional context
When connecting via SSH in the Windows terminal or git bash, the connection is stable.
I just realized that regular SSH connections also drop. This is a foreign IBM i to me and I'm not sure what the problem is. Will investigate. Leaving this up, just in case those issues are actually not related.
Is there a way for me to show more information?

Hi @KerimG
The SSH client can take additional parameters, including '-vvv' for verbose debug info.
PuTTY also has a debug setting in the connection.
HTH.

Hey @chrjorgensen ,
thanks for the info. I forgot to close the ticket. Turns out the sysadmin of the system set the SSH server's ClientAliveTimeout to 10 for no reason, because they thought it's more secure that way...


rotate bundle

Description
Rotate encrypted test bundle
Review Instructions
Circle ci builds should work
Issue
n/a
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[x] Check that you pass the basic style checks and unit tests by running mvn clean install
[x] Ensure that the PR targets the correct branch. Check the milestone or fix version of the ticket.
[x] Follow the existing JPA patterns for queries, using named parameters, to avoid SQL injection
[x] If you are changing dependencies, check the Snyk status check or the dashboard to ensure you are not introducing new high/critical vulnerabilities
[x] Assume that inputs to the API can be malicious, and sanitize and/or check for Denial of Service type values, e.g., massive sizes
[x] Do not serve user-uploaded binary images through the Dockstore API
[x] Ensure that endpoints that only allow privileged access enforce that with the @RolesAllowed annotation
[x] Do not create cookies, although this may change in the future
[x] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

The skip ci is the right thing to do, but it's probably going to break our nightly deploys when merged, alas. Unless we fixed that in the deployer.

Tried to edit the message on the squash commit to force a buiild


ListItem button onPress slow performance

I am having performance issue with List and onPress. onPress is delayed too much. I do not know if there is problem on iOS , but there is on Android.

Interesting => this issue go away if I use ListView from React instead of List from NativeBase

Ok I found out why this is happening.
If you wrap  in , then onEndReached in List will keep firing to infinity......

Can you elaborate by what you mean 'wrap list in content'?

Do not use List inside Content

So, one of the reasons to use native base is the nice setup of    where content scrolls and header and footer are pinned to the top and botttom, respecitively.. When I remove  from wrapping a list that breaks that layout. How should one address that pattern using a list, without using the  wrapper?

@srhise did you solve this without using .. ?
I'm facing a similar situation and would like to get some feedback on possible solutions before I open another issue.

@cyclops24 Solution provided in above comment dd'nt work?

I also find this: facebook/react-native#11809
Maybe it's related to React-Native version. Now NativeBase is using version 0.43.0 but it seems performance issue for list and button fixed in React-Native version 0.44.0.

@mnlbox Check http://docs.nativebase.io/docs/Compatibility.html
And also It is not recommended to use Content inside List.

Wait, is it list inside content or content inside list??
Can you explain why that is not recommended, ie. what is the impact?

@srhise I think you right.
This issues is about List inside Content.
/CC: @shivrajkumar

Guys I have same issue. Please help me to fix this issue. 😢
/CC: @SupriyaKalghatgi , @obykoo

NativeBase List will be deprecated, very soon this will be documented
So please use React Native's FlatList in place of it
You can use ListItem for the necessary styling
#759

@SupriyaKalghatgi Any update on NativeBase List? Will it be replaced by something else or should we stick to FlatList? Thanks!

@SupriyaKalghatgi Didn't help performance for me. It's like the TouchableOpacity blocks screen rendering.

update: I think the problem is FlatList from React Native, not the Button by NativeBase
I replace FlatList with LargeList: https://github.com/bolan9999/react-native-largelist
is work for me, performance get a lot better,
no idea why a third party lib have better performance then FlatList, but it work for me.

@Laurensdc Are you using FlatList or NativeBase List?

@SupriyaKalghatgi I used both and had the same problems. I did keep the  component from NativeBase in both scenarios, but I tried wrapping it in FlatList as suggested. This did not help performance unfortunately.
What was interesting is that the actual dispatch of the onPress() comes late. It's like the screen goes into 'pressed' state for a whole second, and only then starts to dispatch the function. The Redux actions for example are only dispatched after that second of delay.


Update GIT tag to <IP_ADDRESS> for Aerospike server

Tag 3.7.5 no longer exists under https://github.com/aerospike/aerospike-server. It is now on tag <IP_ADDRESS> I have been able to perform the tests on Azure changing this tag without issue.

I have signed it

Thanks for making this fix. Looks good.


Can i add contact manager code in java.

This is a simple Contact Management System in Java that allows users to add, display, search, and delete contacts. Each contact contains a name and a phone number. The system ensures that phone numbers are exactly 10 digits. It provides a user-friendly menu for performing operations like adding a contact, displaying all contacts, searching for a specific contact by phone number, and deleting a contact. The program runs in a loop until the user chooses to exit.

Sure you can add


Feature flag CodeLens integration

CodeLens integration can take sometime to scan and may not be preferable in some environments even where the C# Omnisharp extension is available.
Feature flag the integration as a setting

Fixed by #16


Fixed: Outdated py-evm version in Guide Doc - #1983

OS: macOS 11.2.2
Python Version (python --version): Python 3.6.13

What is wrong?

Guide Doc building_an_app_that_uses_pyevm is outdated, py-evm old version 0.2.0a40 causes script below to fail.

chain.get_vm().state.get_balance(MOCK_ADDRESS)

"Edit on GitHub" link redirects to 404 page.

https://py-evm.readthedocs.io/en/latest/guides/building_an_app_that_uses_pyevm.html

How can it be fixed
Update py-evm in doc to the latest version "py-evm==0.3.0a20"
https://github.com/ethereum/py-evm/blob/master/docs/guides/building_an_app_that_uses_pyevm.rst

Thanks for that!


Create 5exceptions.txt

@rohit-5ex First stage of verification passed, official domain is in.linkedin.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision. Proof of domain ownership: https://in.linkedin.com/in/amit-kumar-vyas

@rohit-5ex Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

@rohit-5ex You might miss my previous comment. Please react.

@rohit-5ex I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


add message for empty tracts for report 2

Closes #72
@keileyo and @colleenms - let me know if this message is ok.

Looks good to me


Added .gitkeep for ftp/sftp upload

1. Why is this change necessary?
When creating a theme and simple uploading it in phpstorm for example, the IDE will not upload the empty folder. This will cause a fatal error, which is not displayed in the administration but only on the cli.
2. What does this change do, exactly?
It adds a .gitkeep file to the assets folder
3. Describe each step to reproduce the issue or behaviour.
Use php bin/console theme:create and upload the created theme directly in phpstorm to your server. When choosing the theme in the admin it will show the following error:

in the CLI it will throw a error like this

4. Please link to the relevant issues (if any).

5. Checklist

[x] I have squashed any insignificant commits
[x] I have read the contribution requirements and fulfil them.

Hi @Isengo1989,
thank you for your contribution. 👍 I have imported your PR and it should be merged soon.
Best regards
Tobias


Gradle dependency contains invalid character

It is preventing copy-paste in an existing Gradle project to work out of the box

I signed it!

Good catch! Thanks!


fix(trace): fixed wrong type in externalSubjectId examples

To be honest, I was not part of the meeting, nor can I find any description of this in AAS part 1 or 2 V3. The whole externalSubjectId is way to vague to me in the AAS v3 specs. Someone else must approve who is more involved.

I forward you the email in which the DTR team confirmed or better showed mit this solution.


doors can be placed without being connected to a wall

doors can be placed without being connected to a wall which results in the door creating its own wall when used
To Reproduce
Steps to reproduce the behavior:

run ./cub3D maps/42.cub
walk through the door in the middle of the room
watch the door creating walls to its left and right

Expected behavior
should be an error while parsing

fixed by @vytkuklys


Measurement period config

update exm125 and exm130 to include measurement year guards. The other measures work without it.

Just need to bring up to date


package metadata incompatible with current version of maturin

Problem description
python bindings cannot be built due to the following error
$ maturin build --release --no-sdist --strip --manylinux off
💥 maturin failed
  Caused by: Failed to parse Cargo.toml at Cargo.toml
  Caused by: invalid type: sequence, expected a map for key `package.metadata.maturin.project-url` at line 32 column 1

Problem location
In python_binding/Cargo.toml the project-url in [package.metadata.maturin] uses an older style format
project-url = ["Homepage, https://pravega.github.io/pravega-client-rust/"]

Suggestions for an improvement
per https://github.com/PyO3/maturin/issues/386
changing to the following resolves the issue with maturin version 0.10.6
project-url = { homepage = "https://pravega.github.io/pravega-client-rust/" }

Additional context:
The automated maturin build script uses maturin version v0.8.3
To upgrade to the latest maturin we would also need to ensure the RUST library is manylinux complaint.


ofdm equalizer

hi,
Do you have the code for ofdm equalizer?
Thanks,
Nikil

Hi Nikil,
Thanks for your attention to this work.
The source code is now published.
Be safe,
Zhongyuan


Add stylesheet for folder html

Previously, an error existed where icons were not being displayed
for folders. This stylesheet addition allows the icons to be seen
again.

 S3Commander icons are not displaying

Steve, I wouldn't add a stylesheet into a component template. That belongs in the index.html file that's including S3Commander. It's not up to the individual components to make decisions about which CDNs to use or which stylesheets to include. That would make it very difficult to override or customize the styling of the entire application.


improve example test by sorting output

What changed?
Improving the example test by sorting the activities returned by Stats()
The example implementation range over a map and return the result in a slice. Because of this, the assertion against expected result would not always pass since the order is random. This made the test flaky

Why?

How did you test it?
unit tests

Potential risks

From the docs on strings.Compare it looks like we can/probably should just use < here... but they appear to be semantically identical so I'll stamp now and land whichever you prefer :) thankfully it's not a locale-sensitive comparison at least.

I saw the last comment after merging. Followed up in #1367


Ruby Sass End-of-Life: 26 March 2019

https://github.com/rails/rails/issues/32896
https://github.com/rails/sass-rails/issues/420
It sounds like rails intends to make this the canonical replacement for sass-rails and the sass ruby gem.
This gem depends on the deprecated sass gem for integration with Sass Value types and some utility code.
If you'd like to extract the shared code to a separate gem that can be used by both this library and ruby-sass and be made a maintainer for the shared library, please let me know and I can assist you with creating the repos and any permissions.

There are now only nine months remaining to migrate this package away from importing Ruby Sass. Is any work being done on that? @bolandrm

I'm more or less using webpack/webpacker for everything now, so I don't currently have any plans to take this on.  It shouldn't be too complicated though, so if someone from the community would like to open a PR i'll be sure to review and merge.

Closing this - work in progress here:  https://github.com/sass/sassc-ruby/pull/85


[QUESTION] How to ensure CASE filters use comparisons

About You:
Name: Kyle Porter
Company: Dremio
Your question:
Repro Steps:

Install Tableau Desktop (eval) version 2020.4.2 (20204.21.0217.1203) 64-bit
Install Dremio ODBC 32-bit driver version <IP_ADDRESS>0. Use the instructions found here: https://docs.dremio.com/drivers/dremio-connector

Configure an ODBC DSN.

Upload the following CSV file to your Dremio home space:

cat < EOF > cato_txn_oneview.csv
rowid,mifid_ii_priips_bmk_id,mifid_ii_priips_bmk_source,asset_class
1,1001,AA,F
2,1002,BB,G
3,1003,CC,H
4,1004,DD,I
5,1005,EE,J
6,1006,FF,K
7,1007,GG,L
8,1008,HH,L
EOF

Go into the “cato_txn_oneview” VDS and click on the “tableau” tool bar button to generate a Tableau .tds file.
Click on the .tds file and sign into the Dremio server via the Tableau workbook
a. Create a worksheet that uses the Tableau live connection with an exclude filter:
b. Drag the “Asset Class” field to the “Filters” box.
c. Select the “F” value and press OK.
d. Drag the “mifid_ii_priips_bmk_source” field to the “Rows” section.
e. See the rows that have an “Asset Class” of “F”.
f. Edit the “Asset Class” filter.
g. Select the “Exclude” option and press OK.
h. See the Tableau error message.
i. See the Dremio Jobs page and see the SQL generated by Tableau is not valid Dremio syntax:

WHERE clause must be a condition
SELECT "cato_txn_oneview"."mifid_ii_priips_bmk_source" AS "mifid_ii_priips_bmk_source" FROM "home.Preparation_Tier"."cato_txn_oneview" "cato_txn_oneview" WHERE (CASE WHEN ("cato_txn_oneview"."asset_class" = 'F') THEN 0 ELSE 1 END) GROUP BY "cato_txn_oneview"."mifid_ii_priips_bmk_source"
When similar operation performed with Table -> Oracle below is where condition
Tableau → Oracle
WHERE ((CASE WHEN ("EMPLOYEES"."JOB" = 'Analyst') THEN 0 ELSE 1 END) <> 0)
but when dremio is involved Tableau → Dremio → Oracle
WHERE ((CASE WHEN ("EMPLOYEES"."JOB" = 'Analyst') THEN 0 ELSE 1 END))
Is there a capability that can be added to the TDC file which will ensure that the CASE has the <> 0 added to it as Dremio cannot handle filters that are essentially WHERE 0 or WHERE 1.

Other ODBC isn't a part of the Connector SDK. Documentation for Other ODBC can be found here: https://help.tableau.com/current/pro/desktop/en-us/examples_otherdatabases.htm