ERROR: no curved landmarks constructed...

version: 2.2.2
data: sct_example_data
sct_register_to_template -i t2.nii.gz -s t2_seg.nii.gz -l labels.nii.gz -param step=1,type=seg,algo=slicereg,metric=MeanSquares:step=2,type=seg,algo=bsplinesyn,iter=3,shrink=1,metric=MeanSquares:step=3,type=im,algo=syn,metric=CC,iter=1 -r 0

Check template files...
  OK: /Users/julien/code/spinalcordtoolbox/data/template/MNI-Poly-AMU_T2.nii.gz
  OK: /Users/julien/code/spinalcordtoolbox/data/template/landmarks_center.nii.gz
  OK: /Users/julien/code/spinalcordtoolbox/data/template/MNI-Poly-AMU_cord.nii.gz

Check parameters:
.. Data:                 t2.nii.gz
.. Landmarks:            labels.nii.gz
.. Segmentation:         t2_seg.nii.gz
.. Path template:        /Users/julien/code/spinalcordtoolbox/data/template
.. Path output:
.. Output type:          1
.. Remove temp files:    0

Parameters for registration:
Step #1
.. Type #seg
.. Algorithm................ slicereg
.. Metric................... MeanSquares
.. Number of iterations..... 10
.. Shrink factor............ 1
.. Smoothing factor......... 0
.. Gradient step............ 0.5
.. Degree of polynomial..... 3
Step #2
.. Type #seg
.. Algorithm................ bsplinesyn
.. Metric................... MeanSquares
.. Number of iterations..... 3
.. Shrink factor............ 1
.. Smoothing factor......... 1
.. Gradient step............ 0.5
.. Degree of polynomial..... 3
Step #3
.. Type #im
.. Algorithm................ syn
.. Metric................... CC
.. Number of iterations..... 1
.. Shrink factor............ 1
.. Smoothing factor......... 0
.. Gradient step............ 0.5
.. Degree of polynomial..... 3

Check input labels...

 Get center of mass of the centerline/segmentation...
.. Smoothing algo = hanning
.. Windows length = 50

Get coordinates of landmarks along curved centerline...
Error during straightening on line 577
ERROR: no curved landmarks constructed...
/Users/julien/code/spinalcordtoolbox/scripts/sct_utils.py, line 98

strange, when running sct_straighten_spinalcord within the tmp folder, it works.
julien-macbook:~/sct_example_data/t2/tmp.160131174836_949683 $ sct_straighten_spinalcord -i seg_smooth_1mm_rpi_crop.nii.gz -s seg_smooth_1mm_rpi_crop.nii.gz -o seg_smooth_1mm_rpi_crop_straight.nii.gz -qc 0 -r 0 -v 1
/Users/julien/code/spinalcordtoolbox

Check input arguments:
  Input volume ...................... seg_smooth_1mm_rpi_crop.nii.gz
  Centerline ........................ seg_smooth_1mm_rpi_crop.nii.gz
  Final interpolation ............... spline
  Verbose ........................... 1

Create temporary folder...
mkdir tmp.160131175125_461092/

Copy files to tmp folder...
sct_convert -i seg_smooth_1mm_rpi_crop.nii.gz -o tmp.160131175125_461092/data.nii
sct_convert -i seg_smooth_1mm_rpi_crop.nii.gz -o tmp.160131175125_461092/centerline.nii.gz
cp data.nii data_1mm.nii
cp centerline.nii.gz centerline_1mm.nii.gz

Orient centerline to RPI orientation...
sct_image -i centerline_1mm.nii.gz -setorient RPI -o centerline_1mm_rpi.nii.gz

Get dimensions...
.. matrix size: 52 x 384 x 185
.. voxel size:  1.0mm x 1.0mm x 1.0mm

Smooth centerline/segmentation...
.. Get center of mass of the centerline/segmentation...
.. Smoothing algo = hanning
.. Windows length = 50

Get coordinates of landmarks along curved centerline...

Get coordinates of landmarks along straight centerline...

Pad input volume to account for landmarks that fall outside the FOV...
sct_image -i centerline_1mm_rpi.nii.gz -o tmp.centerline_pad.nii.gz -pad 68,30,30

Open padded centerline for reading...

Write NIFTI volumes...
.. File created: tmp.landmarks_curved.nii.gz
.. File created: tmp.landmarks_straight.nii.gz
cp tmp.landmarks_curved.nii.gz tmp.landmarks_curved_crop.nii.gz
cp tmp.landmarks_straight.nii.gz tmp.landmarks_straight_crop.nii.gz

Estimate b-spline transformation: curve --> straight...
isct_ANTSLandmarksBSplineTransform tmp.landmarks_straight.nii.gz tmp.landmarks_curved.nii.gz tmp.curve2straight_rigid.txt tmp.warp_curve2straight.nii.gz 5x5x10 3 LandmarksRealCurve.txt LandmarksRealStraight.txt 3 0
crop
isct_crop_image -i tmp.landmarks_straight.nii.gz -o tmp.landmarks_straight_crop.nii.gz -dim 0,1,2 -bmax
Cropping the following region:
Dimension 0: 63 125
Dimension 1: 191 253
Dimension 2: 23 221

Done! To view results, type:
fslview tmp.landmarks_straight_crop.nii.gz &

Concatenate rigid and non-linear transformations...
sct_concat_transfo -w tmp.curve2straight_rigid.txt,tmp.warp_curve2straight.nii.gz -d tmp.landmarks_straight_crop.nii.gz -o tmp.curve2straight.nii.gz

Estimate b-spline transformation: straight --> curve...
isct_ANTSLandmarksBSplineTransform tmp.landmarks_curved.nii.gz tmp.landmarks_straight.nii.gz tmp.straight2curve_rigid.txt tmp.warp_straight2curve.nii.gz 5x5x10 3 LandmarksRealStraight.txt LandmarksRealCurve.txt 3 0

Concatenate rigid and non-linear transformations...
sct_concat_transfo -w tmp.straight2curve_rigid.txt,tmp.warp_straight2curve.nii.gz -d data.nii -o tmp.straight2curve.nii.gz

Apply transformation to input image...
sct_apply_transfo -i data.nii -d tmp.landmarks_straight_crop.nii.gz -o tmp.anat_rigid_warp.nii.gz -w tmp.curve2straight.nii.gz -x spline

Apply transformation to centerline image...

Parse list of warping fields...
  Transfo #0: tmp.curve2straight.nii.gz

Check file existence...
   To view results, type:
fslview tmp.landmarks_straight_crop.nii.gz tmp.centerline_straight.nii.gz &

Generate output file (in current folder)...
  File created: warp_curve2straight.nii.gz
  File created: warp_straight2curve.nii.gz
  File created: seg_smooth_1mm_rpi_crop_straight.nii.gz

Done!

Maximum x-y error = 1.11 mm
Accuracy of straightening (MSE) = 0.74 mm

Finished! Elapsed time: 76s

To view results, type:
fslview seg_smooth_1mm_rpi_crop_straight.nii.gz &

I could not reproduce the issue. I've replicated what you did:

download sct_example_data with release 2.2.2 from release branch
run the first few lines from batch_processing
run the command you provided and the result is great.

Did you downloaded the sct_example_data again? Maybe the t2 image has been corrupted or changed.

I've downloaded sct_example_data again and the issue is gone. It works perfectly fine on 2.2_dev and 2.2.2. Indeed, maybe the T2 was corrupted.
Let's close it then :-)

Issue reappeared when running batch_processing from scratch. Might be related to files that were not closed at previous step.

The issue seems to happen only when fslview opens the results of the segmentation at the previous step. I've added a "-nodisplay" argument to batch_processing, which then runs without error: a7ce207657094da6fd5696d9c678ce6d900bdc64

I've re-run the batch_processing several times without the display, and unfortunately the bug is still there from time to time.
So it is likely caused by an instability in sct_straighten_spinalcord

Tried running the following command several times:
sct_register_to_template -i t2.nii.gz -s t2_seg.nii.gz -l labels.nii.gz -param step=1,type=seg,algo=slicereg,metric=MeanSquares:step=2,type=seg,algo=bsplinesyn,iter=3,shrink=1,metric=MeanSquares:step=3,type=im,algo=syn,metric=CC,iter=1

Pass/Fail: 2/0

I have also a set of data with which I got the same error. It failed 2 times and passed once when I changed -p for -param (deprecated argument).
I gathered the 3 command lines I tried and their outputs in a text file in folder_shared/sct_issues/20160202_722_no_curved_landmarks_constructed. I also added the data.
In case it could help you to find where the problem comes from...

Thank you very much Simon! it does help.

In the meantime, I found a bug in sct_register_to_template, that prevented to set cpu-nb flag. Fixed in: 375af639b06d228a2ffb1c1c9a07ad45577a40a9

Bug seems to be related to an exception raised in:

    def worker_landmarks_curved(self, arguments_worker):
        """Define landmarks along the centerline. Here, landmarks are densely defined along the centerline, and every
        gapxy, a cross of 16 landmarks is created (small square and big square = 8+8).
        """
        try:
            iz = arguments_worker[0]
            iz_curved, x_centerline_deriv, y_centerline_deriv, z_centerline_deriv, x_centerline_fit, y_centerline_fit, \
                z_centerline = arguments_worker[1]

            temp_results = []

            if iz in iz_curved:
                # at a junction (gapxy): set coordinates for landmarks at the center of the cross
                coord = Coordinate([0, 0, 0, 0])
                coord.x, coord.y, coord.z = x_centerline_fit[iz], y_centerline_fit[iz], z_centerline[iz]
                deriv = Coordinate([0, 0, 0, 0])
                deriv.x, deriv.y, deriv.z = x_centerline_deriv[iz], y_centerline_deriv[iz], z_centerline_deriv[iz]
                temp_results.append(coord)

                cross_coordinates = compute_cross_centerline(coord, deriv, self.gapxy)
                print iz
                print cross_coordinates

                for coord in cross_coordinates:
                    # check if all coordinates are real (jcohenadad, see issue #584)
                    if not (is_number(coord.x) and is_number(coord.y) and is_number(coord.z)):
                        sct.printv('WARNING: Coordinates have complex values at iz='+str(iz)+': '+str(coord), 1, 'warning')
                        # set wrong coordinates that will be removed later on
                        coord.x, coord.y, coord.z = 99999, 99999, 99999
                    temp_results.append(coord)
            else:
                # not a junction: do not create the cross.
                if self.all_labels >= 1:
                    temp_results.append(Coordinate([x_centerline_fit[iz], y_centerline_fit[iz],
                                                    z_centerline[iz], 0], mode='continuous'))

            return iz, temp_results

        except KeyboardInterrupt:
            return

        except Exception as e:
            raise e

When outputing cross_coordinates (lines print cross_coordinates added for debugging), values seem OK:
...
115
[(55.7058397276517,<PHONE_NUMBER>72,<PHONE_NUMBER>91620, 1), (-4.25765272121928,<PHONE_NUMBER>72,<PHONE_NUMBER>08380, 2), (25.7240935032,<PHONE_NUMBER>34409,<PHONE_NUMBER>67119, 3), (25.7240935032,<PHONE_NUMBER>09923,<PHONE_NUMBER>32881, 4), (54.3366440732511,<PHONE_NUMBER>72,<PHONE_NUMBER>33407, 5), (-3.44752658129133,<PHONE_NUMBER>72,<PHONE_NUMBER>96958, 6), (54.8957135877237,<PHONE_NUMBER>72,<PHONE_NUMBER>03042, 7), (-2.88845706681867,<PHONE_NUMBER>72,<PHONE_NUMBER>66593, 8), (40.7149666154339,<PHONE_NUMBER>72,<PHONE_NUMBER>45810, 9), (10.7332203909985,<PHONE_NUMBER>72,<PHONE_NUMBER>54190, 10), (25.7240935032,<PHONE_NUMBER>03287,<PHONE_NUMBER>33559, 11), (25.7240935032,<PHONE_NUMBER>41045,<PHONE_NUMBER>66441, 12), (40.0303687882336,<PHONE_NUMBER>72,<PHONE_NUMBER>16704, 13), (11.1382834609624,<PHONE_NUMBER>72,<PHONE_NUMBER>48479, 14), (40.3099035454700,<PHONE_NUMBER>72,<PHONE_NUMBER>51521, 15), (11.4178182181988,<PHONE_NUMBER>72,<PHONE_NUMBER>83296, 16)]
130
[(56.0787954040747,<PHONE_NUMBER>18,<PHONE_NUMBER>90269, 1), (-3.91831812015352,<PHONE_NUMBER>18,<PHONE_NUMBER>09731, 2), (26.080238642,<PHONE_NUMBER>24035,<PHONE_NUMBER>40464, 3), (26.080238642,<PHONE_NUMBER>12195,<PHONE_NUMBER>59536, 4), (55.8552709891369,<PHONE_NUMBER>18,<PHONE_NUMBER>81650, 5), (-3.76100032700918,<PHONE_NUMBER>18,<PHONE_NUMBER>97174, 6), (55.9214776109304,<PHONE_NUMBER>18,<PHONE_NUMBER>02826, 7), (-3.69479370521566,<PHONE_NUMBER>18,<PHONE_NUMBER>18351, 8), (41.0795170230177,<PHONE_NUMBER>18,<PHONE_NUMBER>45134, 9), (11.0809602609035,<PHONE_NUMBER>18,<PHONE_NUMBER>54866, 10), (26.080238642,<PHONE_NUMBER>71075,<PHONE_NUMBER>20232, 11), (26.080238642,<PHONE_NUMBER>65155,<PHONE_NUMBER>79768, 12), (40.9677548155487,<PHONE_NUMBER>18,<PHONE_NUMBER>40825, 13), (11.1596191574757,<PHONE_NUMBER>18,<PHONE_NUMBER>48587, 14), (41.0008581264455,<PHONE_NUMBER>18,<PHONE_NUMBER>51413, 15), (11.1927224683725,<PHONE_NUMBER>18,<PHONE_NUMBER>59175, 16)]
145
[(56.0320903675827,<PHONE_NUMBER>47,<PHONE_NUMBER>63496, 1), (-3.96184433471667,<PHONE_NUMBER>47,<PHONE_NUMBER>36504, 2), (26.0351230164,<PHONE_NUMBER>00493,<PHONE_NUMBER>69502, 3), (26.0351230164,<PHONE_NUMBER>93643,<PHONE_NUMBER>30498, 4), (56.0246284132299,<PHONE_NUMBER>47,<PHONE_NUMBER>13115, 5), (-3.91969549305553,<PHONE_NUMBER>47,<PHONE_NUMBER>49636, 6), (55.9899415259215,<PHONE_NUMBER>47,<PHONE_NUMBER>50363, 7), (-3.95438238036386,<PHONE_NUMBER>47,<PHONE_NUMBER>86885, 8), (41.0336066920078,<PHONE_NUMBER>47,<PHONE_NUMBER>31748, 9), (11.0366393408582,<PHONE_NUMBER>47,<PHONE_NUMBER>68252, 10), (26.0351230164,<PHONE_NUMBER>73781,<PHONE_NUMBER>84751, 11), (26.0351230164,<PHONE_NUMBER>20355,<PHONE_NUMBER>15249, 12), (41.0298757148314,<PHONE_NUMBER>47,<PHONE_NUMBER>06557, 13), (11.0577137616887,<PHONE_NUMBER>47,<PHONE_NUMBER>74818, 14), (41.0125322711773,<PHONE_NUMBER>47,<PHONE_NUMBER>25182, 15), (11.0403703180346,<PHONE_NUMBER>47,<PHONE_NUMBER>93443, 16)]
WARNING: Exception during Straightening:
Error on line 590
need more than 0 values to unpack

Tried removing the "try/except" in the worker_landmarks_curved to always return an output:
Pass/Fail: 1/

So, this is NOT a problem related to multithreading. Maybe the "solve" of sympy.

Thoughts based on the error you provided just above (Error on line 588):
When Python says "need more than 0 values to unpack", generally it means that we try to iterate on a vector that is empty. It means that the following line (line 588 in sct_straighten_spinalcord.py) produces something that is empty, meaning that min(iz_curved) and max(iz_curved) are somewhat equal.
range(min(iz_curved), max(iz_curved) + 1, 1)

I couldn't reproduce the issue so far.
By the way, there is no way to interrupt the batch_processing script using Ctrl+C and that's the real pain...

N.B.: to interrupt batch_processing more easily now you can run: batch_processing.sh -nodisplay
there will no be fslview interruption, therefore easier to interrupt each bash processes.

FOUND THE BUG!
It is caused by the function solve, which fails and does not output a result. As a consequence an exception is raised because the two variables could not be attributed.
range :  [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25, 26, 27, 28, 29, 30, 31, 32, 33, 34, 35, 36, 37, 38, 39, 40, 41, 42, 43, 44, 45, 46, 47, 48, 49, 50, 51, 52, 53, 54, 55, 56, 57, 58, 59, 60, 61, 62, 63, 64, 65, 66, 67, 68, 69, 70, 71, 72, 73, 74, 75, 76, 77, 78, 79, 80, 81, 82, 83, 84, 85, 86, 87, 88, 89, 90, 91, 92, 93, 94, 95, 96, 97, 98, 99, 100, 101, 102, 103, 104, 105, 106, 107, 108, 109, 110, 111, 112, 113, 114, 115, 116, 117, 118, 119, 120, 121, 122, 123, 124, 125, 126, 127, 128, 129, 130, 131, 132, 133, 134, 135, 136, 137, 138, 139, 140, 141, 142, 143, 144, 145, 146, 147, 148, 149, 150, 151, 152, 153, 154, 155, 156, 157, 158, 159, 160, 161, 162, 163, 164, 165, 166, 167, 168, 169, 170, 171, 172, 173, 174, 175, 176, 177, 178, 179, 180, 181, 182, 183, 184]
[-11.6482616746123, 47.9986575027285]
[-10.1763542141227, 49.5885755343343]
[-9.00947672536885, 50.8159520240425]
[-7.85402216043869, 51.9333829844813]
[-6.79309979101478, 53.0624150511464]
[-5.97868812110498, 53.9380910214098]
[-5.33271162936171, 54.6144979171037]
[-4.80789467969575, 55.1530316835406]
[-4.25765272121928, 55.7058397276517]
[-3.91831812015352, 56.0787954040747]
[-3.96184433471667, 56.0320903675827]
[]
Error on line 242
need more than 0 values to unpack
Error on line 397
WARNING: Exception during Straightening:
Error on line 603
need more than 0 values to unpack

Now working on a solution to replace solve with an analytical solution.

Fixed in branch jca_issue722:
Comparative results:
# ====================================================================================
# 2016-02-03
# commit: abfb85cc68e0cb0a90bb787abbfe9af54e4628e9
isct_test_function -f sct_straighten_spinalcord -d /Volumes/data_raid/data_shared/sct_testing_small/ -p "-i t2/t2.nii.gz -s t2/t2_seg_manual.nii.gz"
# ====================================================================================
Results for "sct_straighten_spinalcord -i t2/t2.nii.gz -s t2/t2_seg_manual.nii.gz":
Dataset: /Volumes/data_raid/data_shared/sct_testing_small/
             subject      dice  dist_max    duration       mse     status
0         100_004_s1  0.976176  1.410000  127.860137  0.720000   0.000000
1         200_005_s2  0.969072  1.530000  200.115950  0.770000   0.000000
2    20150910_joshua  0.975742  0.840000   83.738272  0.470000   0.000000
3      20151025_emil  0.976413  0.360000  135.089394  0.220000   0.000000
4       20151025_max  0.980482  1.260000  135.530449  0.740000   0.000000
5             e25428  0.970905  2.410000  118.113098  0.650000   0.000000
6           errsm_40  0.974875  1.500000  201.529225  0.860000   0.000000
7           errsm_42  0.976250  1.770000  196.346473  0.770000   0.000000
8            sct_002  0.964649  1.400000  157.393522  0.770000   0.000000
9   sct_example_data  0.772341  1.170000  101.124114  0.630000  99.000000
10     vanderbilt_05  0.969028  1.580000   84.948703  0.720000   0.000000
11              Mean  0.955085  1.384545  140.162667  0.665455   9.000000
12               STD  0.060777  0.517829   43.880300  0.178962  29.849623
Passed: 10/11

# ====================================================================================
# 2016-02-03
# commit: cd5a3c7c3fcacfd3ac3c0ddb4f78e3fb591dd3ad
isct_test_function -f sct_straighten_spinalcord -d /Volumes/data_raid/data_shared/sct_testing_small/ -p "-i t2/t2.nii.gz -s t2/t2_seg_manual.nii.gz"
# ====================================================================================

Results for "sct_straighten_spinalcord -i t2/t2.nii.gz -s t2/t2_seg_manual.nii.gz":
Dataset: /Volumes/data_raid/data_shared/sct_testing_small/
             subject      dice  dist_max    duration       mse     status
0         100_004_s1  0.956846  1.120000  163.191365  0.700000   0.000000
1         200_005_s2  0.970180  1.600000  246.217484  0.780000   0.000000
2    20150910_joshua  0.945589  0.900000  117.843979  0.510000  99.000000
3      20151025_emil  0.858958  1.170000  172.469985  0.380000  99.000000
4       20151025_max  0.935496  1.220000  179.034716  0.740000  99.000000
5             e25428  0.929668  2.900000  153.112982  0.670000  99.000000
6           errsm_40  0.957569  1.330000  247.201897  0.840000   0.000000
7           errsm_42  0.974663  1.850000  245.169056  0.810000   0.000000
8            sct_002  0.963804  1.340000  169.929566  0.760000   0.000000
9   sct_example_data  0.892490  1.140000  116.666484  0.720000  99.000000
10     vanderbilt_05  0.965213  1.300000   90.459352  0.640000   0.000000
11              Mean  0.940952  1.442727  172.845170  0.686364  45.000000
12               STD  0.035889  0.545181   54.443796  0.135888  51.701064
Passed: 6/11

There is another issue raised by the new implementation, when checking label_vertebrae. The inside of the cord does not seem to be properly registered. Below switching between previous and new implementation:

OK, there is a labeling problem. See below:

checked labels-- they seem fine

OK, the problem arises when estimating rigid transformation: for some reasons, this step rotates the volume by 90deg.

straight2curve_affine --> OK

curve2straight_affine --> NOT OK!!!

I already had this issue. It was an issue in the numbering of labels. In my case, that was the ordering of labelling when creating the labels in straight and curved space that was different.

that's what i though, but the generated labels (in nifti format) seem to be ok. See below: curved, curved_affine,  straight.
The rigid and straight seem fine, but the curved_affine is rotated. So it seems like the affine transformation introduces a problem.
This observation is supported by the fact that the straight2curve transformation is correct (see result above). I guess if the labels were wrong, then we should observe a problem in both cases.

When running sct_straighten_spinalcord on sct_testing_data, the result is correct, again suggesting that labels are fine:

Mmm I see. That could be related to the affine transformation that is computed before the bspline transform, probably due to some instability. But that is weird because normally there are no instability with this pattern of labels.

ok, some additional elements: I increased the gapxy from 30 to 40, and results is now correct:

So it's pretty clear that this is an instability of the affine transfo. I should mention that I've simplified the cross: it is no more a cross+square, but only a cross (so only 8 labels instead of 16). I will add the square like you did, maybe this will increase the stability.

by the way: now the routine to generate crosses is the same for the curved and the straight. I think you will like the new implementation ;-)

Yes the square is quite important, because the bspline transform we compute is only interpolation. So, with only a cross, it does not know what to do at the edges of the images. Providing a square, and even better, a double square (16 points), increases the stability a lot!
Is the implementation in the branch jca_issue722. I already like what I see there.

yes it,s there. I'm currently adding the square.

OK it solved the problem!!

Great!

fixed in https://github.com/neuropoly/spinalcordtoolbox/pull/726


Feature/Add Verbose to Chains

add verbose (console callback manager) to chains
updated conversation chain to have ability to use chatprompt template

Is the document loader input in the additional parameters?


[BUG] Mismatch between overall manufacturer numbers and individual manufacturer donation numbers

Summary
The manufacturer donations report individual values are not being constrained by the timeframe selected.  They should be.
Details
This report is found under "Reports | Donations-Manufacturer" in the left hand menu.
The manufacturer donations report  is showing the total manufacture donations according to the selected timeframe,  but the top manufacturers are not showing according to the selected timeframe.
Example, from January 9, 2024

Some things to consider:
1/  Are the top manufacturers being selected and ranked according to the timeframe?
2/ The values shown for each of those manufacturers should be constrained by the timeframe.
Criteria for completion

[ ] The top manufacturers are selected and ranked according to the activity during the timeframe
[ ] The values shown for each manufacturer in that list are the donations during the selected timeframe
[ ] Tests to demonstrate the above.

@cielf  I can take on this issue

You've got it!


Add radd, rsub and rmul

This is a supplement PR to https://github.com/wadetb/tinynumpy/pull/29.
This PR adds the r versions of __add__, __sub__ and __mul__ so that pre-(add/sub/mul) by a scalar works.

Thanks for the PR- in the future please try to add tests, to confirm (at least limited) numpy compatibility.

Sounds good! This was just something really quick that I needed for my project. Also I found that the add/mul/div implementations don't work well in the current master. Something about _data not being the right size (forgot the error message and I don't have too much time to reproduce this). I switched it to your implementations from https://github.com/wadetb/tinynumpy/pull/25 and they were a lot better.


The stargazer channel page has some problems as seen here. There is also a lot of blue which does not match the white background.

Yes, these channel pages still need some work. But I didn't think the blue was an issue. Do you know what the default channel colors used to be? Looking at archives all the channels are different colors, so I based it off of Jawed's channel, which used this same blue color.

i think the default was just a normal white or grey back then.

I think it'll be cool if for the customtube layouts from 2008-2012 (before channel banners layouts) to change channel pages's colors to whatever we want.

I think it'll be cool if for the customtube layouts from 2008-2012 (before channel banners layouts) to change channel pages's colors to whatever we want.

maybe the most common color of the banner could be the color (though that COULD be hard.)


Refactor, Documentation & Feature Request

module refactor
utils/scraping
jsdoc
update license, package.json & README/md

also pls update the CONTRIBUTING.md before npm version

thanks, :thumbsup:


Error during sigdial (Misplaced alignment tab character & )

While running this command
./bin/generate examples/sigdial --proceedings --handbook
Misplaced alignment tab character &.
 &
l.275    Ruihong Huang, Texas A\&
M University, United States\

Did you change the input from the sigdial example?
In this case, remember that input text should be escaped to be later compiled in latex

I did not change it.
Anyway it worked when I use the below commands. Thanks a million.
./bin/generate examples/sigdial --proceedings --overwite.

Perfect


Parse Cargo dependencies referencing git repos

Please use Conventional Commits. The type in the title could be either fix or feat depending on your view; I'd probably say it's a fix.

Please use Conventional Commits. The type in the title could be either fix or feat depending on your view; I'd probably say it's a fix.

Also, the commit message body should contain Fixes #7372. in order to auto-close the issue upon merge.


Add keyboard accessibility to move crop area with arrow keys

This PR adds keyboard support for moving the crop area in react-easy-crop using the arrow keys, addressing issue #569.

Users can now move the crop area using the arrow keys.
The movement is based on a customizable step size (controlled via the keyboardStep prop).

Thanks for this PR, it looks good to me. Could you please reference the new option on the README?

Thanks for this PR, it looks good to me. Could you please reference the new option on the README?

Done! 👍

Thank you for this contribution!

@allcontributors add @JGuardiola95 for code


Integration tests fail

When running cargo test on a brand new cw-template I get:
Finished test [unoptimized + debuginfo] target(s) in 0.02s
     Running unittests (target/debug/deps/ramon-a83cd65c19a8fc85)

running 4 tests
test contract::tests::proper_initialization ... ok
test contract::tests::increment ... ok
test contract::tests::reset ... ok
test integration_tests::tests::count::count ... FAILED

failures:

---- integration_tests::tests::count::count stdout ----
thread 'integration_tests::tests::count::count' panicked at 'called `Result::unwrap()` on an `Err` value: error executing WasmMsg:
sender: USER
Execute { contract_addr: "Contract #0", msg: Binary(7b22696e6372656d656e74223a7b7d7d), funds: [] }

Caused by:
    Generic error: Invalid input: address not normalized', src/integration_tests.rs:68:60
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Am I missing something?
Dependencies versions:

rustc version: rustc 1.60.0
cargo-generate v0.13.0
cargo-run-script v0.1.0

Issue resolved.


click on overlaying objects misfire sometimes

pixi.js version 3.0.9
There are two Containers named A and B, A sit on top of B, and A is smaller than B.
By my experiment, click on A will trigger click event both on A and B, in that order.
But sometimes, click on A only triggers click event on B, is there any know issues related to this? I haven't try to extract a reproduce example, I'll do it if you need, but first, I want to know if this ring some bells to someone?
I noticed that there is change related to 3.0.11: Fixed interaction issue with overlaying objects, I'm wondering what's that about?

Have you tried to reproduce in the latest version of 3.x?

I am also having this issue, in 3.x
I made the root stage clickable and objects in front of it also clickable, both actions fire. I'm guessing this is because the event-emitter fires on both. I've tried setting zIndex, which may have been deprecated anyways, but also made sure the draw order is correct. How can I make sure only the first hit succeeds in a mousedown event?

How can I make sure only the first hit succeeds in a mousedown event?

Your issue is not exactly same with mine, but FYI, I attach properties to mouseData in the top object's event handler, then in the bottom object's event handler I can tell if it's triggered from another object. I think the mouseData is reused across event handlers. But I'm not sure if this is the right thing to do. @englercj

@inetfuture yes thank you. I realize what I was doing wrong-- this other issue tipped me off (https://github.com/pixijs/pixi.js/issues/2362)
Good luck, and I think your issue may be related to: https://github.com/pixijs/pixi.js/issues/2364


Issues in Governance Connectors UI in Console

Describe the issue:
Governance connectors UI in the Console is implemented to be data driven. But currently, Governance API doesn't support any metadata regarding the fields hence some front end level decisions have been made in order to figure out the property field type etc.
The following logic determines if a toggle or input has to be rendered based on the property value.
const getFieldComponent = (property: ConnectorPropertyInterface) => {
    if (property.value === "true" || property.value === "false") {
        return RenderToggle;
    } else {
        return RenderInput;
    }
};

The problem with this is, if you enter true or false to any of the input fields, it'll convert to a toggle element and this is irreversible unless you change it to a value other than true or false using the API.
How to reproduce:

Go to Console -> Manage -> User Onboarding.
In Lite User Registration, Enter true for Manage Lite-Sign-Up purposes input field.
Observe that the UI element being transformed to a toggle.

Expected behavior:
It should remain an input & API should provide necessary metadata regarding the type of the input field.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.11.0-RC1
OS: Mac
Database: H2
Userstore: LDAP

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Check for latest dat CLI

It'd be helpful to check to make sure you have the latest CLI installed (or at least one that is compatible), similar to PR #2.
I was having issues with the keys because the latest dat CLI changed what is returned on dat keys.

Good point! Adding that.


Maximum call stack size exceeded

Hello,
I am trying to create a hub with 2.500 videos. When i click create button, below error occours after every 5 minutes.
I am using VHA version 3.0.0.

Thank you for the report! I'll see if I can replicate and figure out what's happening.
Best guess is something is going wrong with chokidar -- but it would be weird for it to choke 😉 on such a small collection 😅
If you have other clues do share 🤝

Hey @ridvanaltun -- I've fixed up some things with the extraction process (fewer stat calls for example) …
If you have the time, could you please re-try this procedure again with the same settings that caused the problem before -- to see if the problem persists?
I'm hoping to release 3.0.0 with as few bugs as possible. At the moment I'm unable to replicate the problem 😅

I built the app from master branch.
Unfortunately the error keeps coming up. The first error took 5 minutes to arrive, and then 10 minutes later the second error came.

I just say an estimate, but I think there might be an error with the heavy CPU usage of my computer because when I start to create a hub, the computer's fan noise and fan usage increase.

As you see, the CPU is being used completely. The computer may be using CPU instead of GPU to create clips.

Thank you @ridvanaltun for checking again 🙇
I will check the code and run the app to see if I can fix 🤝

I tried today and couldn't replicate on my end. I wonder what on your end is different 🤔
In my case, all I have is a folder with many videos (and virtually no other files besides that). Could you, if not too private, share a bit about what you have?

the source / input folder that you choose -- if you right-click (in Windows) and click Properties -- how many files does it say it has?
do you run the app via npm start? If so, you'll have a log in your terminal of what happens. Is there anything there that seems like an error message?

I decided to log how long it takes for chokidar to run through my directory of videos - and it took mere seconds 🚀 ... it's unclear why chokidar would be running after even 1 minute of extraction (as your error shows).
Is it possible that you have a symlink in your folder that points to a parent folder? That is a circular refence? Maybe chokidar is stuck re-scanning the same path non-stop?
Thank you for any information that you could share that you think is relevant 🙇

I would like to help,
There is 7949 files, 646 folders and 254 GB data. I keep all this data in a HDD.
I count files with powershell using below command:
Get-ChildItem -Recurse | Group-Object Extension -NoElement

Output:
 Count Name
----- ----
 2106 .mp4
    7 .pdf
 5253 .jpg
  147 .png
  154 .jpeg
    5 .zip
   61 .ts
   60 .txt
    2 .docx
   15 .url
    3 .html
    1 .gif
   15 .nfo
    9 .exe

I am sure there is no symlink in my folders.
The errors occurs in preventing sleep stage after i think 25-30 seconds with no any console log.
When I click ok in error dialog the CPU usage changes like below:

By the way the VHA holds progress percent %14 for 10 minutes. After 10 minutes the VHA increase the percent very slowly (like 3 minutes for %1). Is it normal?

Thank you for additional details.
VHA progress is coded a bit weird at the moment -- it tries to estimate time remaining by multiplying the number of items remaining-to-extract by the average time it takes to extract (based on files already extracted this run). But the total keeps increasing because there are more files being added to the queue.
I'm planning on reworking the computation so that it always knows the total number of files to be extracted.
The preventing sleep message is just a reminder that I have turned on the powerSaveBlocker so that users can run their app overnight if they want to. That specific event should run just as the first video gets added to the app gallery 👌
VHA should just ignore all other non-video files (explicitly within chokidar file-found event), so I'm still unsure what is causing the error.
Have you tried running the app against a folder with just a few video files (and no other files inside) just to check? I suspect that will work without a problem.
Just to confirm: in your original description of an error after 5 minutes -- by then -- the app was already showing videos and some screenshots, right?
So the overall picture is the app seems to work for a bit, but then seems to be stuck doing nothing, and finally gives an error (OS popup), and clicking the error makes the error go away, the app continues to be stuck, and then another error appears?
It's so bizzarre -- the chokidar (that is throwing the error it seems) should only run for under 10 seconds -- by which it should have scanned through all the files on the hard drive and shut itself down.
ps -- just to be clear -- this is happening when creating a new hub, not trying to re-scan an older one, right?

I found the error reason.
Related:
paulmillr/chokidar#841
nodejs/node#27534
In short, chokidar freeze when trying to handle a file if has UTF character (like a Turkish or German character) in file or folder name.

I take it there is an İ somewhere in your path? 😅
So it seems the problem isn't due to chokidar but due to node 😓
Weird that the result isn't that chokidar ignores a path and everything works otherwise, but instead some crash 😢
I suspect this isn't up to my app to fix 😕  -- but I don't know how best to address this issue. Any thoughts?

I take it there is an İ somewhere in your path? 😅

Unfortunately 😅

I added the stalled label while we wait for this PR to get merged and for the code to finally make it to Electron 🤞
https://github.com/nodejs/node/pull/27662
Not closing the issue until we have the fix in Node 🤞

@ridvanaltun -- I think I fixed the problem with #602 -- which is in now in the main branch 🎉
Please try out the latest main and see if your problem persists. I am 99% confident your problem has been solved 😁
Please update us so I can close this issue 🤝

@ridvanaltun -- I think I fixed the problem with #602 -- which is in now in the main branch 🎉
Please try out the latest main and see if your problem persists. I am 99% confident your problem has been solved 😁
Please update us so I can close this issue 🤝

Yes, problem solved, thanks 👍

Thanks for getting back so quickly 🤝 I'm glad it worked out 😁


Ping Pong Ball Score-Keeper

Ping Pong Ball Score-Keeper

Project Overview

In this project user can keep score of various 2 players games. The maximum score can be 11 and score is reset after every game.The buttons turn red and green for winner and loser respectively. After the winning the game the score can not be increased.

Use Of the Project

User can keep score of their games.

Languages Used

HTML
CSS
Javascript

How to Use

Using the project is very easy.
First select the maximum Score/Wining Score.
To increase the score of player 1 by pressing player-1 button and increase the score of player 2 by pressing player-2 button.
After reaching the maximum score winner would be declared by an alert message and green color of button and score.
To play another game press reset button and choose your winning score.

Sample Pictures

Opening Page

counting page

player1 wins

game is reset

Assigned @MonishRaman


Identifiable!, BelongsTo!, HasMany! are brocken when used with custom_derive

Using one of those 3 macros with custom_derive will result in the following error:
error: expected ident, found ,
  --> <diesel macros>:21:25
   |
21 | Identifiable ! { $ args , fields = [ $ ( $ fields ) * ] , } } ; (
   |                         ^

Example Code:
custom_derive! {
     #[derive(Identifiable)]
     #[table_name(users)]
     struct User {
         id: i32,
         name: String,
     }
}

This is most likely caused by custom_derive not passing custom attributes down to the macro. The documentation mentions clearly:
"Secondly, the macro is passed the entire item, sans attributes. It is the derivation macro's job to parse the item correctly."
We should change the syntax here to something like:
custom_derive! {
     #[derive(Identifiable(table_name=users))]
     struct User {
         id: i32,
         name: String,
     }
}

https://github.com/diesel-rs/diesel/issues/438


As a moderator I cannot see for what sponsorhip the company signed up for anymore

Just saw that intermax applied for sponsorship but I have no idea which one... So don't know what exactly I am approving to now.. Used to be able to see that so somewhere in the reengineering something was left out? I'll leave the approval for now untill I know what they selected. can you tell from the admin side somehow?

Closed in #300


upstream: code clean up of upstream cluster creation

Commit Message: upstream: code clean up of upstream cluster creation
Additional Description:
In the previous impl, we will create a stats Scope instance and a TransportSocketFactoryContext instance for every cluster in the ClusterFactoryImplBase::create call.
Both objects will be forwarded to createClusterImpl call, and then be forwarded to createClusterWithConfig call, and then be forwarded to construction method of cluster, and finally be forwarded to construction method of ClusterImplBase.
Both objects finally will be managed by the new created cluster instance.
However, this long forwarding path make it's hard to track the lifetime of these objects or to understand these code.
This patch make a change to create both objects in the construction method of ClusterImplBase directly.
Risk Level: Mid.
Testing: n/a.
Docs Changes: n/a.
Release Notes: n/a.
Platform Specific Features: n/a.

This PR introduced lots of code change but has no any new function. But I still think it make sense in the long term to improve the code readability/maintainability.

/retest


fflogs: 8.5.16 -> 8.5.25

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/RPGLogs/Uploaders-fflogs/releases.
meta.description for fflogs is: Application for uploading Final Fantasy XIV combat logs to fflogs.com
meta.homepage for fflogs is: https://www.fflogs.com/client/download
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.5.19 with grep in /nix/store/x9dbcfgnic6r47zc3kav4wpad58nshq8-fflogs-8.5.19
found 8.5.19 in filename of file in /nix/store/x9dbcfgnic6r47zc3kav4wpad58nshq8-fflogs-8.5.19

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

fflogs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/x9dbcfgnic6r47zc3kav4wpad58nshq8-fflogs-8.5.19 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fflogs https://github.com/r-ryantm/nixpkgs/archive/bcf9e3c3ba9dac4c9a2515cf95414d4d99f538de.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bcf9e3c3ba9dac4c9a2515cf95414d4d99f538de#fflogs

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/x9dbcfgnic6r47zc3kav4wpad58nshq8-fflogs-8.5.19
ls -la /nix/store/x9dbcfgnic6r47zc3kav4wpad58nshq8-fflogs-8.5.19/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

1 package built:

fflogs

Maintainer pings
cc @sersorrel for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/RPGLogs/Uploaders-fflogs/releases.
meta.description for fflogs is: Application for uploading Final Fantasy XIV combat logs to fflogs.com
meta.homepage for fflogs is: https://www.fflogs.com/client/download
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 8.5.25 with grep in /nix/store/jlhg11cxry8ymsyr9sb9r2ig9yaxnl1s-fflogs-8.5.25
found 8.5.25 in filename of file in /nix/store/jlhg11cxry8ymsyr9sb9r2ig9yaxnl1s-fflogs-8.5.25

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

fflogs

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jlhg11cxry8ymsyr9sb9r2ig9yaxnl1s-fflogs-8.5.25 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A fflogs https://github.com/r-ryantm/nixpkgs/archive/9c04703dccf38a16ad4dcc458690f397ed7b4ef8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/9c04703dccf38a16ad4dcc458690f397ed7b4ef8#fflogs

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jlhg11cxry8ymsyr9sb9r2ig9yaxnl1s-fflogs-8.5.25
ls -la /nix/store/jlhg11cxry8ymsyr9sb9r2ig9yaxnl1s-fflogs-8.5.25/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

fflogs

Maintainer pings
cc @sersorrel for testing.

@NixOS/nixpkgs-merge-bot merge


Repo structure, and code cleanup

@all-collaborators please add @just1and0 for code

@all-contributors please add @just1and0 for code


Interstitials not working

Hello, I am trying to get Interstitial ads working in my .NET Maui application. It is currently failing on both iOS and Android.
In my view model, I am retrieving the singleton IInterstitialAdService from Service provider and running:
_interstitialAdService.PrepareAd("{MY_AD_UNIT_ID}");
Then, on button click I am running:
_interstitialAdService.ShowAd();
On that method call, my app crashes with an error:
System.ArgumentNullException: Value cannot be null. (Parameter 'adUnitId')
at ObjCRuntime.ThrowHelper.ThrowArgumentNullException(String argumentName) in /Users/builder/azdo/_work/1/s/xamarin-macios/src/ObjCRuntime/ThrowHelper.cs:line 28
at Google.MobileAds.InterstitialAd.Load(String adUnitId, Request request, InterstitialAdLoadCompletionHandler completionHandler)
at Plugin.AdMob.Services.InterstitialAdService.PrepareAd(String adUnitId)
at Plugin.AdMob.Services.InterstitialAdService.ShowAd()
When I inspect _insterstitialAdService, I can see the private property "_ad" is indeed populated and it's AdUnitId property is populated with my ad unit id.
I've also tested using the test unit id's here: https://developers.google.com/admob/ios/test-ads#demo_ad_units with the same result.
Am I doing something wrong or is this a bug?
Additionally, it doesn't seem like AddTestDevice is working. When I run the application on my simulator, I am able to see a Test Ad banner. However, when I try running on my device, I am not able to see the banner.
In my MauiProgram.cs, I am executing:
AdConfig.AddTestDevice("{MY_TEST_DEVICE_AD_ID}");
This device has been added to AdMob as a test device with a matching Ad ID.
In the console, I see a message:
 To get test ads on this device, set:
Objective-C
GADMobileAds.sharedInstance.requestConfiguration.testDeviceIdentifiers = @[ @"{string of characters}" ];
Swift
GADMobileAds.sharedInstance().requestConfiguration.testDeviceIdentifiers = [ @"{string of characters}" ]
I've tried updating my call to AddTestDevice with the supplied characters from the message, but there was no change.

Hi @alexberardi, it's a bug, thank you for reporting it.
You can work around it by setting AdConfig.DefaultInterstitialAdUnitId to match your ad unit id.

I would need to look into it, unfortunately I don't have an iOS device to test with, only the simulator.
From what I figure, the test device Ids are set correctly - https://github.com/marius-bughiu/Plugin.AdMob/blob/887bfeecc801b77bccd93c2cc085bf2defa4201e/src/Plugin.AdMob/Platforms/iOS/InterstitialAdService.cs#L26
A wild guess woould be that maybe that test devide IDs need to be set before Request.GetDefaultRequest()? I'll try to have a look at it this weekend and see if I can figure it out, but I can't make any promises.
In the meantime you could try:

calling PrepareAd twice, the second call would be guaranteed to have the test device IDs set if that's the issue
in case there's something wrong with the AdConfig - you could try setting MobileAds.SharedInstance.RequestConfiguration.TestDeviceIdentifiers from your code (this is platform-specific code, so it would need to go into your Platforms/iOS folder.

alexberardi, have you managed to get this working? If not, I'm going to mark this with help-needed until I get my hands on an iOS device so I can troubleshoot; maybe someone else finds some spare time to do it.

I have not been able to get it working on a physical device, but I'm not sure if that's because my app is not published yet or if there's an underlying issue.

@marius-bughiu My app is now published and I can confirm the ads are not working on iOS. I'm able to see the requests come through on AdMob but nothing shows on the device.

@alexberardi, I've got my hands on an iPhone XR running iOS 17.4.1 and at least the test ads are showing - these are photos running the sample app in the repo:

1
2
3

Could it be something related to the iOS version of the device or some privacy settings?

@marius-bughiu - finally was able to get ads working on iOS. The issue is that you need to set the DefaultBannerAdUnitId for production ads to work. I set the value and now I'm getting ads. Thank you! Leaving open because I'm not sure if you'd like to fix that but feel free to close if you are too busy.


Deprecate old geoutils organisation (github.com/GeoUtils)

To do:

archive geoutils organisation and do necessary clearup, flag existence of this new project.
consider what to do with PyPI and conda versions of old geoutils org stuff
shift geoutils.readthedocs.io registration to GlacioHack/GeoUtils
GitHub actions for automatic documentation building/hook to readthedocs

2020-11-18:
geoutils.readthedocs.io registration shift completed
webhooks for readthedocs in place

Repositories georaster and geoutils in organisation github.com/GeoUtils now marked as archived, with deprecation notices included in readmes

georaster.readthedocs.io deleted


ListShowList view doesn't respect column gutter size of show view

Describe the bug
ListShowList views don't respect the column gutter size of the show view.
To Reproduce
Steps to reproduce the behavior:

Create an edit view with a main column containing a card on each side
Adjust the horizontal gutter size of said column to 0
Create a ListShowList view on the same table, using the edit view as the show view
Embed ListShowList view in a page and open it
See that the columns don't touch

Additional context
Theme: tabler

this was a general problem with columns-with-cards not implmenting the gutters. Fixed in #2752


SE(3) / Sim(3) vs separate rotation, translation and scaling

Hi Hannes
I want ask you about your thoughts on treating SE(3) / Sim(3) as a single object vs treating them as separate parts.
Libraries like sophus and manif do provide support for SE3, and LieTorch even provides support for Sim3. But the expressions involved in their jacobian become increasingly complex and sometimes do not have closed forms.
I was looking online for a discussion on the relative merit of these two approaches, and found your paper, Trajectory Representation and Landmark Projection for Continous Time Structure from Motion (https://arxiv.org/abs/1805.02543). Here you have a comparison in the case of spline interpolation. I want to ask whether you think the same would hold true in a pose-graph optimization setup, or some SLAM setup.

Hi,
The paper you cite only deals with spline interpolation in different spaces. In that case there is a difference between separate interpolation of R3 and SO(3) vs SE(3) which is both mathematical and practical.
For a pose graph where you have discrete parameters to optimize I don't think you will have much of a difference. It is possible that there are differences in how fast an optimization process converges, or that one representation allows for a slightly faster implementation. But this is not something I have investigated.

Thanks a lot for the quick response!


Updating instance consul_z1 fails while bosh deploy

I'm trying to install cloud foundry on openstack and ran into a problem.

I've increased the timers in the Update Block of the deployment manifest and recreated the vm serveral times without success.
While checking the logs of the consul vm, i've noticed that the files
/var/vcap/jobs/consul_agent/config/certs/agent.crt
/var/vcap/jobs/consul_agent/config/certs/agent.key
are empty. The other ones seem to be correct. Both key and certificate are included in the stub-file.
Here are the logs:
consul_agent.stderr.log
consul_agent.stdout.log
has anyone an idea what seems to be the issue here?

Hi @fluppirt
thanks for reporting this issue. It seems like this isn't actually OpenStack related, but rather an issue with how you deploy Cloud Foundry itself. Please re-open this issue on either https://github.com/cloudfoundry/cf-release or https://github.com/cloudfoundry/cf-deployment, depending on what you're using.
I'm closing this issue here for now.


Nothing happens

This is my Code, but when i Push the button, nothing happens.
{ ������������"accessory": "GPIODevice", ������������"type": "MotionSensor", ������������"name": "Hall Motion", ������������"pin": 27 ��������}
I use a contact button on gpio 27 Pin 13 with doorpi and homebridge-contactsensor and it works with an raspi3. But i need a Motion sensor.
Can you help me?

Hi,
For physical pin 13 (GPIO 2, BCM 27), you have to use the pin number "2" in your config file (see "Pin configuration" in the README).
Let me know if it solve.

thanks a lot.. now it works.. but only one time... i push the button and it hangs on triggered... i use the occupancy option and it does not go back to untriggered.

You have to push then release the button. Motion sensor will hang on triggered then hang off when button is released. Occupancy sensor will hang on triggered then hang off after specified "timeout" (60 sec if not specified).

thank you.. it works, great work!


My IDEA can't import the jar.what is your Maven's mirror?

Dependency 'com.binance.api:binance-api-client:1.0.0' not found less... (Ctrl+F1)
Inspects a Maven model for resolution problems.
My IDEA can't import the jar.what is your Maven's mirror?

You can just download binance api and install it to your local maven repo

Read the README of https://github.com/binance-exchange/binance-java-api, as aaronpon said, just clone the binance-java-api repo, and do mvn install to install the lib in your local mvn repository.


Improve the README.md file

Make the documentation more informative and descriptive.

I am interested with the issue.

I would like to work on this issue
please assign it to me for gssoc20.

I would also like to work on it

I would like to work on this...

Its will be an honour to work on that issue

@vipuldcoder you can take this up. Others can have a look at other  beginner friendly issues. I will open more easy issues soon.

@sharmaaditya570191 I would like to do the documentation part in a detailed manner, pls assign me this issue

@sharmaaditya570191  Can i work on this issue
is anybody assigned?

@vipuldcoder had been assigned this issue earlier.

In Progress.

@sharmaaditya570191 Can I work on this ?


The transparency log component listens for (new) blocks and indexes transactions

Using the add listener API from the blockchain component, the transparency log component listens for blocks and indexes all transaction payloads in its transaction payload index component (#680)

can you check on the progress of this one as well @tiainen @johanvos ?

I believe this is pending the work that is done in #893. @fishseabowl can probably confirm?

Transparency log service can register related APIs with the blockchain service through the blockchain::add_payload_listener method in #893.


Classic distribution of fastutil.jar is missing some classes

All classes in package it.unimi.dsi.fastutil (like it.unimi.dsi.fastutil.Hash) are missing in the classic distribution.
At least for version 8.5.5

I pushed yesterday 8.5.6 which was supposed to fix that. I don't know why, it's still not out. Checking...

I don't know what to say—that never happened to me before. It seems to be an internal sonatype issue. Please try to look again for 8.5.6 tomorrow.

It is still not released :(

Sonatype has something really broken. I have uploaded 8.5.6 twice, yesterday and today, without any error. And it is not published yet.

The situation is made worse by the fact that there is no clear endpoint for reporting bugs. JIRA? Support? If you have suggestions about how to contact the correct Sonatype guys, please let me know...


Update NewApp.WPF.fsproj

Use .Net 4.7.2 as default target Framework.

I am a bit baffled about the error from Travis:
  Unable to find version '<IP_ADDRESS>' of package 'Xamarin.Android.Support.CustomTabs'.
    /Users/travis/.nuget/packages/: Package 'Xamarin.Android.Support.CustomTabs.<IP_ADDRESS>' is not found on source '/Users/travis/.nuget/packages/'.
    /Users/travis/build/fsprojects/Fabulous/build_output: Package 'Xamarin.Android.Support.CustomTabs.<IP_ADDRESS>' is not found on source '/Users/travis/build/fsprojects/Fabulous/build_output'.
    https://api.nuget.org/v3/index.json: Error downloading 'Xamarin.Android.Support.CustomTabs.<IP_ADDRESS>' from 'https://api.nuget.org/v3-flatcontainer/xamarin.android.support.customtabs/<IP_ADDRESS>/xamarin.android.support.customtabs.<IP_ADDRESS>.nupkg'.
    crypt32.dll

Any suggestions why upgrading the WPF template would lead to this? I would understand if Mono would have issues with .Net 4.7.2 as it is not yet supported. but this build seams to break due to a failure during NuGet restore.

@mallibone I don't think it's related to your upgrade, I've already seen this error.
Not sure why but I think it's due to the old version of nuget.exe embedded in the templates.
Will re-trigger the CI build to check.


Updated content

Updated code sample indentation.

Docs Build status updates of commit 11ab39a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/extensions-chromium/developer-guide/alternate-distribution-options.md
:white_check_mark:Succeeded

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

#sign-off and merge.


Fix typo

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/microservices/index.md
:white_check_mark:Succeeded
View

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

#sign-off

good catch, thanks!


Failed to grant permissions

Expected behaviour
I expect to not have to include undesired permissions in order for Shot to record screenshots.
Actual behaviour
I'm using Showkase to automatically create my snapshot tests with Shot on Windows. This is working as intended. A sample setup is located here: https://github.com/davidvavra/showkase-screenshot-tests/tree/shot
@ShowkaseScreenshot(rootShowkaseClass = ShowkaseModule::class)
abstract class ComposeTests : ShowkaseScreenshotTest, ScreenshotTest {

    override fun onScreenshot(
        id: String,
        name: String,
        group: String,
        styleName: String?,
        screenshotType: ShowkaseScreenshotType,
        screenshotBitmap: Bitmap
    ) {
        compareScreenshot(bitmap = screenshotBitmap, name = "$group - $name")
    }
}

@ShowkaseRoot
class ShowkaseModule : ShowkaseRootModule

When I attempt to record my snapshots, I get the following error message:
com.beamimpact.beam.ComposeTests_ShowkaseCodegen > color_screenshot_test_14[Pixel_6_Pro_API_32(AVD) - 12] FAILED
        junit.framework.AssertionFailedError: Failed to grant permissions, see logcat for details
        at junit.framework.Assert.fail(Assert.java:50)

I'm able to fix this by adding the following permissions to my app/src/main/AndroidManifest.xml file:
    <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
    <uses-permission android:name="android.permission.WRITE_EXTERNAL_STORAGE" />

However, I don't want these permissions in my actual app. I want them to only be active for the instrumented test.
I tried adding those permissions to an app/src/androidTest/AndroidManifest.xml but it did not fix the problems. It only seems fixed when I include it in the app's AndroidManifest.xml.
Steps to reproduce

Include and configure showkase
Include and configure shot
Add ComposeTests class to androidTest
Run .\gradlew.bat debugExecuteScreenshotTests -Precord

Version of the library

Windows 10
Pixel 6 Pro API 32
Shot 5.14.1
showkase 1.0.0-beta14
showkase-processor 1.0.0-beta14
showkase-screenshot-testing 1.0.0-beta14

I also tried adding in the sharedUserId to the manifest per the README instructions, but then I get the error below. I'm not sure if that is related or not.
> Task :app:connectedDebugAndroidTest FAILED
Exception thrown during onBeforeAll invocation of plugin com.google.testing.platform.plugin.android.AndroidDevicePlugin.
Failed to install APK(s): app\build\outputs\apk\androidTest\debug\app-debug-androidTest.apk
INSTALL_FAILED_SHARED_USER_INCOMPATIBLE: Package <my-package> tried to change user null
com.android.ddmlib.InstallException: INSTALL_FAILED_SHARED_USER_INCOMPATIBLE: Package <my-package> tried to change user null
        at com.android.ddmlib.internal.DeviceImpl.installRemotePackage(DeviceImpl.java:1315)
        at com.android.ddmlib.internal.DeviceImpl.installPackage(DeviceImpl.java:1141)
        at com.android.tools.utp.plugins.deviceprovider.ddmlib.DdmlibAndroidDevice.installPackage(DdmlibAndroidDevice.kt)
        at com.android.tools.utp.plugins.deviceprovider.ddmlib.DdmlibAndroidDeviceController$executeAsync$deferred$1.invokeSuspend(DdmlibAndroidDeviceController.kt:173)
        at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
        at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106)
        at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678)
        at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665)

I was able to fix this by adding the AndroidManfiest to the debug folder instead


Fix typos

Apparently renaming a branch closes the PR. Thought Github would be smarter...

  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/main*:refs/remotes/origin/main* +refs/tags/main*:refs/tags/main*
  remote: Repository not found.
  Error: fatal: repository 'https://github.com/laminlabs/lndocs/' not found
  The process '/usr/bin/git' failed with exit code 128
  Waiting 19 seconds before trying again
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/main*:refs/remotes/origin/main* +refs/tags/main*:refs/tags/main*
  remote: Repository not found.
  Error: fatal: repository 'https://github.com/laminlabs/lndocs/' not found
  The process '/usr/bin/git' failed with exit code 128
  Waiting 17 seconds before trying again
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --progress --no-recurse-submodules --depth=1 origin +refs/heads/main*:refs/remotes/origin/main* +refs/tags/main*:refs/tags/main*
  remote: Repository not found.
  Error: fatal: repository 'https://github.com/laminlabs/lndocs/' not found
  Error: The process '/usr/bin/git' failed with exit code 128

Same issue as with my other fork PRs. Guess I can't clone it because the secret is not available for forks?

Thank you, @Zethson! Re: CI, see the discussion in the Bionty PR.


CFE_ES_RestartApp() WriteToSysLog Typo

The first WriteToSysLog() occurrence indicates the message is from CFE_ES_DeleteApp instead of CFE_ES_RestartApp:
CFE_ES_WriteToSysLog ("CFE_ES_DeleteApp: Cannot Restart a CORE Application: %s.\n",

Imported from trac issue 278.  Created by dcmccom3 on 2019-04-15T07:13:22, last modified: 2019-05-23T16:43:54

Trac comment by dcmccom3 on 2019-04-15 07:16:00:
Found the same typo in CFE_ES_ReloadApp()
The file is cfe_es_api.c


MAISTRA-1902: Fix disablePolicyChecks logic in yaml template

Aren't we already setting a default value in global.yaml? We can just put in the value as it is

@dgn The issue is when we set that value to false. Currently, because of  | default true, it is always set to true.
See for instance: https://github.com/helm/helm/issues/3308

@dgn The issue is when we set that value to false. Currently, because of  | default true, it is always set to true.
See for instance: helm/helm#3308

Ah, I see! What I had in mind was just doing:
  disablePolicyChecks: {{ .Values.global.disablePolicyChecks }}

-- because we know the value will be set (it's defined in global.yaml, which is always applied first in the operator). But I'm fine with the current approach, too.


Adds roles for varied access

Description of changes

Creates a Role table giving certain users the ability to have varied access.  This is accomplished in conjunction with Pundit.
Todo

[x] update user serializer to include a user's role

Issue Resolved

Fixes #289

Could this be a good way to add a role for the slack bot?


See for example the differences on rbk-dev and rbk-www for these two cases:

czi file, RID = 16-2GM0
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
jpg file, RID = 16-2GYG
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG

Do we know why?
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute Viterbi School of Engineering Professor,
Preventive Medicine Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA  90292-6695
Phone:  +1<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On Feb 29, 2020, 1:40 AM +0100, svoinea<EMAIL_ADDRESS>wrote:
I have tested random some images for the current master version on rbk-dev and some images are black.
See for example the differences on rbk-dev and rbk-www for these two cases:

czi file, RID = 16-2GM0
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
jpg file, RID = 16-2GYG
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=AA3OGXTWBNGJVZTTE7ZNOP3RFGVIBA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRIR26Q, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA3OGXX3MM3Q5T5ZONKKGMLRFGVIBANCNFSM4K6EV24A.

Yes, I did find the reason for the jpg file. We were not handling the case
for simple files in the new code.
I am still working on the czi file issue.
On Sat, Feb 29, 2020 at 4:50 AM Carl Kesselman<EMAIL_ADDRESS>wrote:

Do we know why?
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute Viterbi School of Engineering
Professor,
Preventive Medicine Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA 90292-6695
Phone: +1<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On Feb 29, 2020, 1:40 AM +0100, svoinea<EMAIL_ADDRESS>wrote:
I have tested random some images for the current master version on rbk-dev
and some images are black.
See for example the differences on rbk-dev and rbk-www for these two
cases:

czi file, RID = 16-2GM0
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
jpg file, RID = 16-2GYG
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=AA3OGXTWBNGJVZTTE7ZNOP3RFGVIBA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRIR26Q>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AA3OGXX3MM3Q5T5ZONKKGMLRFGVIBANCNFSM4K6EV24A>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=ACBTW6P336UNI2SJSRTMCT3RFECDHA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENLZJ4Y#issuecomment-592942323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBTW6KRBCYMAYB5WY7ZP43RFECDHANCNFSM4K6EV24A
.

This is a czi file converted to pyramid tiff  and running out of the image server?
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute Viterbi School of Engineering Professor,
Preventive Medicine Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA  90292-6695
Phone:  +1<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On Feb 29, 2020, 3:13 PM -0800, Vipul Singh<EMAIL_ADDRESS>wrote:
Yes, I did find the reason for the jpg file. We were not handling the case
for simple files in the new code.
I am still working on the czi file issue.
On Sat, Feb 29, 2020 at 4:50 AM Carl Kesselman<EMAIL_ADDRESS>wrote:

Do we know why?
Carl

Dr. Carl Kesselman
Dean’s Professor, Epstein Department of Industrial and Systems Engineering
Fellow, Information Sciences Institute Viterbi School of Engineering
Professor,
Preventive Medicine Keck School of Medicine
University of Southern California
4676 Admiralty Way, Suite 1001, Marina del Rey, CA 90292-6695
Phone: +1<PHONE_NUMBER>
Email<EMAIL_ADDRESS>Web: http://www.isi.edu/~carl
On Feb 29, 2020, 1:40 AM +0100, svoinea<EMAIL_ADDRESS>wrote:
I have tested random some images for the current master version on rbk-dev
and some images are black.
See for example the differences on rbk-dev and rbk-www for these two
cases:

czi file, RID = 16-2GM0
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GM0
jpg file, RID = 16-2GYG
https://dev.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG
https://www.gudmap.org/chaise/record/#2/Gene_Expression:Image/RID=16-2GYG

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub<
https://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=AA3OGXTWBNGJVZTTE7ZNOP3RFGVIBA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4IRIR26Q>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/AA3OGXX3MM3Q5T5ZONKKGMLRFGVIBANCNFSM4K6EV24A>.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=ACBTW6P336UNI2SJSRTMCT3RFECDHA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENLZJ4Y#issuecomment-592942323,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBTW6KRBCYMAYB5WY7ZP43RFECDHANCNFSM4K6EV24A
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=AA3OGXUAV4GMYTXCGCMTQPDRFGLCZA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENMIPTY#issuecomment-593004495, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA3OGXR2VR6CMHQVSIJDK73RFGLCZANCNFSM4K6EV24A.

I need to check that.

Please do.  I don’t think we should spend any time debugging the old pipeline.
Carl
Sent from my iPhone
On Feb 29, 2020, at 3:23 PM, Vipul Singh<EMAIL_ADDRESS>wrote:

I need to check that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/informatics-isi-edu/openseadragon-viewer/issues/11?email_source=notifications&email_token=AA3OGXW3H5QKW7RFZ5JOP3DRFGMHNA5CNFSM4K6EV24KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENMIUZI#issuecomment-593005157, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AA3OGXXM6DXF7VH4HOGFGU3RFGMHNANCNFSM4K6EV24A.

@svoinea Is this still an issue?


exclude search from archived repos

Hi @jpoehnelt 👋
So this I was playing around with regex and I noticed, that the secret actually syncs archived repositories.

I'm not sure if it's a bug in Github, but archived repositories are supposed to be read-only. I think they just haven't added a guard for that yet.
So I'm not really sure if this is supposed to be another CLI option where you remove archived repositories in the search list via archived:false or should it be excluded by default.
Thoughts? Not sure if this is a bugfix or a feature?
PS. I accidentally hit submit, I'm not finished writing. 🤣

I just test via settings, even though it's archived, I can freely add a secret in their GUI.

I vote for excluding by default. This will need to be another filter after the list call. Should be fairly trivial.

hey @jpoehnelt! thanks for the last update. Just reporting, I've tested 1.4.1 and it seems to ignore archived repositories now 🙌
No secrets synced: 👍


Quiz results are not saved

Describe the bug
We conducted a quiz, but after the quiz, the results were not loaded, and it shows:
The quiz settings are the following:

To Reproduce

Go to https://artemis.ase.in.tum.de/course-management/335/quiz-exercises/13733
Click on Statistics
Scroll down to Grading
See error No results are present

Expected behavior
See statistics, students get results
Screenshots
Student point of view:

Missed due date

Instructor point of view:

Here, you will see no statistics are loaded:

Here is the Participation

Although overall statistics view show that the students have participated:

Even more weird is that after Release of practice
The practice participations seem to be counted (not shown in the Participations view), but statistics get updated:

Which version of Artemis are you seeing the problem on?
7.0.4
What browsers are you seeing the problem on?
Chrome, Safari, Microsoft Edge, Firefox, Other (specify in "Additional context")
Additional context
Artemis application
Relevant log output
No response

I saw that even though I did not participate in the official conduction time (I did it as practice), I can see the "View result" button. Which I should not
After clicking, I see an empty page:

We are working on a quick fix for the issue. We restarted the production system and it seems that for now the quizzes work again. We will investigate why they did not work and update the code to make it more reliable

Is there a chance to recover failed participation from the data base? E.g. for this exercise where no participations were registered for the 9th and 10th of May.
If there is no way to recover the participations, we will find another solution for our course.

Was this issue related to/fixed by #8703?

this was recently fixed


grpc for pod create and fix bug in the commit before

Signed-off-by: Yao Zengzeng<EMAIL_ADDRESS>

lgtm @YaoZengzeng  This is such a easy patch, but waste a lot of your time. Read through the process whenever you start writing code :/


Add onSwipeMove prop

Hey,
React native PanResponder has another property called onPanResponderMove, I had to animate a component relative to the amount of movement of a swipe. So I implemented it.
I think it won't be creating any conflict with the current usage case. I also updated the readme & example.
Thank you.

Hi,
I'm glad you are using this module, and thank you for pull request.
Since I'm trying to keep API as simple as possible it is for the best not to add additional methods.
However feel free to use code in this repository to publish more complex gesture recognizer with larger API.
Regards


Fix estimating fee on prepared calls

Closes #556
Introduced changes

Instead of changing the documentation as suggested in the issue, removed the check for max_fee.

ValueError is not raised when max_fee is given
Raising the error there doesn't make much sense since it's a fee estimation of an isolated prepared function call object (so the max_fee in the call is not tied to the method in any way)

[ ] This PR contains breaking changes

There is no technical reason, just UX. In my opinion, it is not OK to receive an EstimatedFee.overall_fee higher than the max_fee set during preparation.

Ok then. I do not have a strong opinion on that. We can leave it as it is.


Inconsistency of parameter ordering between GetOrCreateTable and GetOrCreateDataset

In BigQueryClient, we have the following methods (amongst many other overloads):
public virtual BigQueryDataset GetOrCreateDataset(DatasetReference datasetReference, GetDatasetOptions getOptions = null, Dataset resource = null, CreateDatasetOptions createOptions = null)
public virtual BigQueryTable GetOrCreateTable(TableReference tableReference, Table resource, GetTableOptions getOptions = null, CreateTableOptions createOptions = null)
public virtual BigQueryTable GetOrCreateTable(TableReference tableReference, TableSchema schema, GetTableOptions getOptions = null, CreateTableOptions createOptions = null)

For GetOrCreateDataset, the resource comes after the GetDatasetOptions; for both GetOrCreateTable overloads shown here, the resource comes before the options. I think we should make these consistent.
(We need to do this before next GA.)

Yep, nice catch, I will do this now. I will have to put the resource before the options in all cases, because for tables the resource is not optional.


about i18n

Principle
The current situation is that the player chooses the interface language and the system character set on his own, because for sa:mp windows the character set is ansi, depending on the different regions.
The server-side texts are all utf8, and when transmitting to the player, the utf8 is converted into an array of decimal bytes for transmission with the character set chosen by the player.
When the player transmits text data to the server, custom events are also defined by polyfill (see gamemode for details) and passed to the server as decimal byte arrays, which are decoded to utf8 by the server according to the character set chosen by the player.
By now a two-way closed-loop internationalized data transfer has been achieved.
Nickname
See gamemode and #6 for support for multi-byte nicknames.
Note
It is worth noting that:

if the player prefers not to select the correct character set, both the server and the player will receive the text as the wrong character set.
even if the text entered by a player is received by the server as normal in utf8, it will still be garbled if it is sent to another player in a character set that does not support that character. For example, if a player with gbk charset sends to a player with only English charset, the other player will still see the Chinese text is still garbled like '?'.

Currently thinking about how to dynamically add languages to Locales for better scalability, such as content stored in databases or other modules.


Modbus rtu conflict [BUG]

Describe the bug
Integration works with my WM good, but other light modbus device will be damaged. I have delay 40-120 seconds
Without smartThing integration my  modbus rtu work:
If I switched on in lovelace, lamp is turn on after 0.5 seconds and lovelace switch is turn on after 1 seconds. All is good.
With smartThing integration my modbus rtu is delayed:
If I switched on in lovelace, lamp is stay turn off and lovelace switch is turn on after 1 seconds but retutn in turn off staying cause receive lamp modbus status.
After 40-120 seconds my lamp is switch on.
Screenshots
Environment details:

Environment HASSIO docker on Intel NUC 8i3
Home Assistant version installed: 2021.6.4
Component version installed: 0.8.13
Last know working version: in October 2020 hassio + modbus rtu + LG works fine. But it was other version hassio too.
LG device type and model with issue:  F4V5VS0W
LG devices connected (list): wm only

Output of HA logs
No errors in log!

I don't have enough element to analyze this problem. At least you should find a specific version of the integration that work without issue in current HA version, so that I can analyze the difference that could cause this conflict.

Could it be related to the call to time.sleep()? I see the standard integrations are using "await asyncio.sleep" instead.

I don't have enough element to analyze this problem. At least you should find a specific version of the integration that work without issue in current HA version, so that I can analyze the difference that could cause this conflict.

How can I install not last version ha-smartthinq-sensors?

I can't install old version ha-smartthinq-sensors with hassio 2021.6.4
Log Error:
The custom integration 'smartthinq_sensors' does not have a valid version key (None) in the manifest file and was blocked from loading. See https://developers.home-assistant.io/blog/2021/01/29/custom-integration-changes#versions for more details

copy smartthinq_sensors 0.6.1 to
└── custom_components
└── smartthinq_sensors
reboot hassio
log:
2021-06-14 13:56:49 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration hacs which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
hassio - SmartThinQ LGE Sensors - NO component(

Just replace the file manifest. json in integration folder with the one avaliable in last release and reboot. It should work.

My integration does not show up more!

Just try to clean browser cache

Install integration succsessful in local network. Testing different versions.

Could it be related to the call to time.sleep()? I see the standard integrations are using "await asyncio.sleep" instead.

No, it couldn't. We cannot use asyncio.sleep because ThinQ library is not async (it use requests). But this is not a problem, and also exist many standard integration not based on async library.

Its ONVIF bug,  not smartThing. Closed!


Add icon "merge branch"

+1

I could add a toolbar button for merge revision, but then there will be no icon for that.
See issue https://github.com/markiewb/nb-git-toolbar/issues/2

Hi
Text button will do the work :)


Fix bug: HookedEncoder not being moved to GPU

Description
Changes

Move self.to(device) later in the file, after parameters have been initialised
Move majority of tests to the CPU, and add a specific test for the GPU

Fixes #306
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have not rewritten tests relating to key interfaces which would affect backward compatibility

Thanks Rusheb.


improve doc for manual override

/azp run

Looks like the CI has some problem failed to get the finished signal.

Looks like the pipeline was renamed so the "check" policy is waiting for a check with a name that is never going to arrive.

/azp run


Add command-line options for ignoring local deletions

This adds a command-line option "-ignoreD" which will treat any Ds in the CIGAR string as being equivalent to M/X/=. In other words, coverage is calculated ignoring any local deletions.
The functionality to do this was already present in the code in BlockedIntervals.cpp; this patch simply exposes that functionality to the command line.
For reference (including example SAM files), see issue #867.

Awesome, thanks for this contribution!