API Endpoints for source rankings

From JIRA CI-7:
An API endpoint that exposes source metadata (including priority rankings) would be a useful addition.

similar to #141


TLS with self-signed certificates by default

Requires knowing the host name and domains (local and external where applicable) and setting up nginx site configs accordingly.
When should the certificate expire? What should happen when it has expired? Could do like CeroWRT are doing and generate a certificate which is valid for a year and auto-regenerate a new certificate before the current one expires.

https://wiki.mozilla.org/Security/Server_Side_TLS#Nginx

https://cipherli.st/

Added to the 1.0.0 milestone. Might postpone.


Pullsync ignores stamp timestamp changes

Context
Bee 2.1.0 (and earlier)
Summary
Related to #4703
Pullsync only pulls chunks that are not already in the reserve based solely on address and batch.  If content was re-uploaded, the chunk/batch/stamp timestamp is updated and the newly stamped chunk is pushed.   Even after #4703 is fixed, pullsync will still not propagate the current timestamp for the chunk/stamp.
Expected behavior
Stamp timestamps should match across all nodes and reserves.
Actual behavior
Given that pullsync only requests chunks that are not already in the reserve based on address and batch, updated timestamps will not be pulled and therefore will not be updated throughout the storing neighborhood.   Only those nodes that were directly pushed (presuming #4703 is fixed) will have the current stamp timestamp.
https://github.com/ethersphere/bee/blob/8c61408db7cd63227cfe161967e3b70f90b9fde3/pkg/pullsync/pullsync.go#L185-L192
Steps to reproduce
I'll leave this to your imagination.
Possible solution
The pullsync protocol needs to be updated to communicate not only the batch, but the entire stamp so that the pulling node can determine if the local stamp timestamp is current and/or update to the latest timestamp.

It's worse than just the timestamp.   Pullsync will also not pull a re-stamped chunk that happens to have been re-stamped into a different INDEX of the batch's bucket.   This is a critical flaw in mutable stamps.
And worse, since many chunks exist in the swarm whose stamps were forgotten in the great batch purge, those chunks CANNOT be reliably re-stamped with the original stamp because they may end up at different indices which will not be properly propagated to nodes still holding the original stamped chunks.

@ldeffenb you are spot on, i indicated this issue in this writeup https://hackmd.io/@zelig/SJVY-ACKa
it also explain how it can lead to reserve mismatches to this day since the 'great batch purge'.
instead of the batchID indeed the pullsync offers should use the hash of the serialised stamp.
Nodes should store this hash inlocalstore alongside the stamp and can be used to detect  restampings on differring index.

https://github.com/ethersphere/bee/pull/4717


travelandgambling.blog

wpdesktop-2024-04-10T<IP_ADDRESS>.zip
wpdesktop-2024-04-10T<IP_ADDRESS>.zip

Header
Header
Header
Header
Header

Cell
Cell
Cell
Cell
Cell

Cell
Cell
Cell
Cell
Cell

Cell
Cell
Cell
Cell
Cell

Cell
Cell
Cell
Cell
Cell

Cell
Cell
Cell
Cell
Cell

@myfirststaticwebbsite,
We see that the issue template has not been filled, could you please do so as it helps us analyze the issue [SDK version, steps followed before you ran into this error or stand alone code/colab gist to reproduce the issue faced and reopen this issue. Thanks!


Quadform p-value=2

Sometimes when running quadform.qf(), it outputs
2.0, 4, array([      0.,       0.,       0.,       0.,       0.,       0.,     1000001.])
when I input, for example,
chi2val=359.4848154869432
[36.94163236 24.76110077 24.74140768 18.61546473 18.61546473 18.61546473  18.61546473 18.61546473 18.58195836 12.4401378  12.4401378  12.4401378  12.4401378  12.4401378  12.4401378  12.4401378  12.4401378  12.4401378  12.4401378  12.4125513 ]
Should it just be interpreted as p-value essentially = 1?

YYang,
Thanks for your question and thanks for using FastLmmCLib! I should be able to look into this tomorrow.

Carl

Carl Kadie, Ph.D.
FaST-LMM & PySnpTools Teamhttps://fastlmm.github.io/
(Microsoft Research, retired)
https://www.linkedin.com/in/carlk/
Join the FaST-LMM user discussion and announcement list via @.***?subject=Subscribe> (or use web sign uphttps://mail.python.org/mailman3/lists/fastlmm-user.python.org)
From: YYang @.>
Sent: Thursday, September 14, 2023 1:41 PM
To: fastlmm/fastlmmclib @.>
Cc: Subscribed @.***>
Subject: [fastlmm/fastlmmclib] Quadform p-value=2 (Issue #1)
Importance: High
Sometimes when running quadform.qf(), it outputs
2.0, 4, array([ 0., 0., 0., 0., 0., 0., 1000001.])
when I input, for example,
chi2val=359.4848154869432
[36.94163236 24.76110077 24.74140768 18.61546473 18.61546473 18.61546473 18.61546473 18.61546473 18.58195836 12.4401378 12.4401378 12.4401378 12.4401378 12.4401378 12.4401378 12.4401378 12.4401378 12.4401378 12.4401378 12.4125513 ]
Should it just be interpreted as p-value essentially = 1?

Reply to this email directly, view it on GitHubhttps://github.com/fastlmm/fastlmmclib/issues/1, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABR65P47KPTEKIUHGQVYWJLX2NTWNANCNFSM6AAAAAA4YZXIO4.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>

I'm trying to reproduce the issue, but haven't been able to. Here is what I tried:
def test_sep23_qf():
    eigvals = np.array(
        [
            36.94163236,
            24.76110077,
            24.74140768,
            18.61546473,
            18.61546473,
            18.61546473,
            18.61546473,
            18.61546473,
            18.58195836,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4401378,
            12.4125513,
        ]
    )
    result = qf(359.4848154869432, eigvals)
    print(result)

And I saw this:
(0.36692794169308307, 0, array([2.70007200e+00, 4.10000000e+01, 1.00000000e+00, 3.79025439e-03,
       1.53418915e-01, 0.00000000e+00, 2.20000000e+01]))

Can you help me reproduce the issue?
Thanks, Carl

Manually running qf() on chi2vals and eigvals seem to have fixed this. I'm not sure what the cause of it is.
I was calling qf() from python script running in a slurm array on an HPC, and around 6 out of 10 times my script would stop at the qf() step and cannot proceed further. Within the 40% of the times it succeeded in running it would produce the 2.0, 4, array([      0.,       0.,       0.,       0.,       0.,       0.,     1000001.]) result.
Just collecting chi2vals and eigvals, then running the qf separately fixed the issue.

Excellent!
From: YYang @.>
Sent: Saturday, September 16, 2023 11:44 AM
To: fastlmm/fastlmmclib @.>
Cc: Carl Kadie @.>; Comment @.>
Subject: Re: [fastlmm/fastlmmclib] Quadform p-value=2 (Issue #1)
Importance: High
Manually running qf() on chi2vals and eigvals seem to have fixed this. I'm not sure what the cause of it is.
I was calling qf() from python script running in a slurm array on an HPC, and around 6 out of 10 times my script would stop at the qf() step and cannot proceed further. Within the 40% of the times it succeeded in running it would produce the 2.0, 4, array([ 0., 0., 0., 0., 0., 0., 1000001.]) result.
Just collecting chi2vals and eigvals, then running the qf separately fixed the issue.

Reply to this email directly, view it on GitHubhttps://github.com/fastlmm/fastlmmclib/issues/1#issuecomment-1722291302, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABR65PY5JE5PQDG2IGYPGB3X2XXOJANCNFSM6AAAAAA4YZXIO4.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>

Hi Carl,
I'm encountering a very strange issue with quadform.
I'm doing several matrix operations etc to end up with a chi2val and a
numpy array of eigvals.
If I save the eigvals (np.save) and load it, checking that all elements in
the array is equal to the generated array, the resulting p-values and trace
from qf.qf() is different when using the generated eigvals vs the
save-and-loaded eigvals.
Specifically in a test case:
chi2val = 2998.1754859648086
eigvals_ = [601.283561554898 ,<PHONE_NUMBER>469181 ,<PHONE_NUMBER>2616828 ,
91.17672134789915 , 49.23921245678543 , 37.04139197122487 ,
30.87515072058668 , 30.83070804169951 , 27.387885652473727 ,
24.78420217167126 , 24.735588068850603 ,
18.639481731694534 , 18.606664684701848 , 18.598222876777783 ,
18.5877915113293 , 18.532732895132938 , 18.434416620497895 ,
12.45605352114341 , 12.454616462471623 , 12.453538158305353 ,
12.452084879831665 , 12.4512097640746 ,
12.449710630488513 , 12.443930072105454 , 12.440381732335313 ,
12.432859778920395 , 12.419744093636476 , 12.40376996354169 ,
12.389340897058112 , 12.366556596613494 , 9.867422198584155]
qf.qf(chi2val, eigvals) = (0.025548898985580548, 0, array([1.19763908e+02,
7.61530000e+04, 2.00000000e+00, 2.57624534e-04, 2.08522844e+01,
2.43653829e-01, 9.50000000e+01])) the original eigvals generated via
sp.sparse.linalg.eigsh()
qf.qf(chi2val, eigvals_) = (0.08532721621254413, 0, array([3.36862960e+00,
2.42000000e+02, 1.00000000e+00, 2.46419103e-04, 5.93668452e-02,
0.00000000e+00, 2.20000000e+01])) saved and loaded
eigvals == eigvals_ gives 'True' for all elements.
Do you have any ideas for what could be the issue?
Thanks,
Yang
On Sat, Sep 16, 2023 at 11:56 AM Carl Kadie @.***>
wrote:

Excellent!
From: YYang @.>
Sent: Saturday, September 16, 2023 11:44 AM
To: fastlmm/fastlmmclib @.>
Cc: Carl Kadie @.>; Comment @.>
Subject: Re: [fastlmm/fastlmmclib] Quadform p-value=2 (Issue #1)
Importance: High
Manually running qf() on chi2vals and eigvals seem to have fixed this. I'm
not sure what the cause of it is.
I was calling qf() from python script running in a slurm array on an HPC,
and around 6 out of 10 times my script would stop at the qf() step and
cannot proceed further. Within the 40% of the times it succeeded in running
it would produce the 2.0, 4, array([ 0., 0., 0., 0., 0., 0., 1000001.])
result.
Just collecting chi2vals and eigvals, then running the qf separately fixed
the issue.

Reply to this email directly, view it on GitHub<
https://github.com/fastlmm/fastlmmclib/issues/1#issuecomment-1722291302>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ABR65PY5JE5PQDG2IGYPGB3X2XXOJANCNFSM6AAAAAA4YZXIO4

.
You are receiving this because you commented.Message ID:
<EMAIL_ADDRESS>
—
Reply to this email directly, view it on GitHub
https://github.com/fastlmm/fastlmmclib/issues/1#issuecomment-1722293624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABF7K4TEHTVK6PAV7ULAD3TX2XY4FANCNFSM6AAAAAA4YZXIO4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Yang,
I’m happy to try to help. I’m trying to create an IPython notebook to show the problem. However, I’m having trouble creating the test case.
In your example, I see eigvals_, but not eigvals. Also eigvals_ is a 1-D list, so it can’t be given give it to qf.qf.
Yours,

Carl

From: YYang @.>
Sent: Thursday, December 28, 2023 1:08 PM
To: fastlmm/fastlmmclib @.>
Cc: Carl Kadie @.>; Comment @.>
Subject: Re: [fastlmm/fastlmmclib] Quadform p-value=2 (Issue #1)
Importance: High
Hi Carl,
I'm encountering a very strange issue with quadform.
I'm doing several matrix operations etc to end up with a chi2val and a
numpy array of eigvals.
If I save the eigvals (np.save) and load it, checking that all elements in
the array is equal to the generated array, the resulting p-values and trace
from qf.qf() is different when using the generated eigvals vs the
save-and-loaded eigvals.
Specifically in a test case:
chi2val = 2998.1754859648086
eigvals_ = [601.283561554898 ,<PHONE_NUMBER>469181 ,<PHONE_NUMBER>2616828 ,
91.17672134789915 , 49.23921245678543 , 37.04139197122487 ,
30.87515072058668 , 30.83070804169951 , 27.387885652473727 ,
24.78420217167126 , 24.735588068850603 ,
18.639481731694534 , 18.606664684701848 , 18.598222876777783 ,
18.5877915113293 , 18.532732895132938 , 18.434416620497895 ,
12.45605352114341 , 12.454616462471623 , 12.453538158305353 ,
12.452084879831665 , 12.4512097640746 ,
12.449710630488513 , 12.443930072105454 , 12.440381732335313 ,
12.432859778920395 , 12.419744093636476 , 12.40376996354169 ,
12.389340897058112 , 12.366556596613494 , 9.867422198584155]
qf.qf(chi2val, eigvals) = (0.025548898985580548, 0, array([1.19763908e+02,
7.61530000e+04, 2.00000000e+00, 2.57624534e-04, 2.08522844e+01,
2.43653829e-01, 9.50000000e+01])) the original eigvals generated via
sp.sparse.linalg.eigsh()
qf.qf(chi2val, eigvals_) = (0.08532721621254413, 0, array([3.36862960e+00,
2.42000000e+02, 1.00000000e+00, 2.46419103e-04, 5.93668452e-02,
0.00000000e+00, 2.20000000e+01])) saved and loaded
eigvals == eigvals_ gives 'True' for all elements.
Do you have any ideas for what could be the issue?
Thanks,
Yang
On Sat, Sep 16, 2023 at 11:56 AM Carl Kadie<EMAIL_ADDRESS>wrote:

Excellent!
From: YYang<EMAIL_ADDRESS>Sent: Saturday, September 16, 2023 11:44 AM
To: fastlmm/fastlmmclib<EMAIL_ADDRESS>Cc: Carl Kadie<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [fastlmm/fastlmmclib] Quadform p-value=2 (Issue #1)
Importance: High
Manually running qf() on chi2vals and eigvals seem to have fixed this. I'm
not sure what the cause of it is.
I was calling qf() from python script running in a slurm array on an HPC,
and around 6 out of 10 times my script would stop at the qf() step and
cannot proceed further. Within the 40% of the times it succeeded in running
it would produce the 2.0, 4, array([ 0., 0., 0., 0., 0., 0., 1000001.])
result.
Just collecting chi2vals and eigvals, then running the qf separately fixed
the issue.

Reply to this email directly, view it on GitHub<
https://github.com/fastlmm/fastlmmclib/issues/1#issuecomment-1722291302>,
or unsubscribe<
https://github.com/notifications/unsubscribe-auth/ABR65PY5JE5PQDG2IGYPGB3X2XXOJANCNFSM6AAAAAA4YZXIO4

.
You are receiving this because you commented.Message ID:
@.@.mailto:***<EMAIL_ADDRESS>
—
Reply to this email directly, view it on GitHub
https://github.com/fastlmm/fastlmmclib/issues/1#issuecomment-1722293624,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABF7K4TEHTVK6PAV7ULAD3TX2XY4FANCNFSM6AAAAAA4YZXIO4
.
You are receiving this because you modified the open/close state.Message
ID<EMAIL_ADDRESS>
—
Reply to this email directly, view it on GitHubhttps://github.com/fastlmm/fastlmmclib/issues/1#issuecomment-1871490179, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABR65P3PSERPKLFXCZXW77LYLXNTTAVCNFSM6AAAAAA4YZXIO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZRGQ4TAMJXHE.
You are receiving this because you commented.Message ID<EMAIL_ADDRESS>

SKS_matrix.zip
Hi Carl,
I've uploaded a test data file. It's a 286 x 286 hermitian matrix stored as a numpy pickle.
SKS = np.load('SKS_matrix.npy', allow_pickle=True)
eigvals = sp.sparse.linalg.eigsh(SKS, k=40, return_eigenvectors=False)
eigvals = np.sort(phis1[np.where(eigvals > 1e-8)])[::-1]
Q =<PHONE_NUMBER>020846
qf.qf(Q, eigvals, acc=1e-7)

(1.5605687704312032e-06,
0, array([1.38445730e+02, 2.88600000e+03, 1.00000000e+00, 1.07098983e-02, 3.09010677e+01, 1.56866434e-01, 7.00000000e+01]))

However, when I do
qf.qf(Q, eigvals.astype(np.float64), acc=1e-7)
qf.qf(Q, eigvals.astype(float), acc=1e-7)

(0.07390013408306717,
0, array([3.73084527e+00, 1.60000000e+01, 1.00000000e+00, 6.28393232e-03, 9.16086013e-02, 0.00000000e+00, 1.80000000e+01]))

I suspect this has something to do with how sp.sparse.linalg.eigsh returns eigenvalues at machine precision, and some interaction between numpy's float64, python's float64, and C's float64_t.
for i in eigvals: print(i)

.359430815339177
24.26909591539037
22.16434645647498
20.224204984397332
20.172814719026565
19.399440077993294
17.795433563879307
17.508064982626617
16.361888020192687
14.465250735324332
14.41711515040611
14.386829364762525
14.215687314249916
13.495097410402899
13.08161070596028
10.55561990110087
10.552489150903842
10.548784209417585
10.5454783523345
10.542843626630576
10.536236974566016
10.515292888092002
10.504225556873383
10.450495164026837
10.402338819902715
10.35366667058023
9.781555167439034
8.238279080876648
8.043976264341268
2.0183786464449573
2.142120841448611e-05

for i in eigvals: print("{:.60f}".format(i))

25.359430815339177200939957401715219020843505859375000000000000
24.269095915390369810893389512784779071807861328125000000000000
22.164346456474980584516742965206503868103027343750000000000000
20.224204984397331941181619185954332351684570312500000000000000
20.172814719026565200010736589320003986358642578125000000000000
19.399440077993293840563637786544859409332275390625000000000000
17.795433563879306859689677366986870765686035156250000000000000
17.508064982626617478445041342638432979583740234375000000000000
16.361888020192687065446079941466450691223144531250000000000000
14.465250735324332254094770178198814392089843750000000000000000
14.417115150406109691516576276626437902450561523437500000000000
14.386829364762524718912573007401078939437866210937500000000000
14.215687314249915829122983268462121486663818359375000000000000
13.495097410402898674419702729210257530212402343750000000000000
13.081610705960279972259741043671965599060058593750000000000000
10.555619901100870450250113208312541246414184570312500000000000
10.552489150903841874651334364898502826690673828125000000000000
10.548784209417584989409988338593393564224243164062500000000000
10.545478352334500371512149285990744829177856445312500000000000
10.542843626630576281399953586515039205551147460937500000000000
10.536236974566016044718708144500851631164550781250000000000000
10.515292888092002243638489744625985622406005859375000000000000
10.504225556873382885214596171863377094268798828125000000000000
10.450495164026836647508389432914555072784423828125000000000000
10.402338819902714917020603024866431951522827148437500000000000
10.353666670580230046994074655231088399887084960937500000000000
9.781555167439034192966573755256831645965576171875000000000000
8.238279080876647952891289605759084224700927734375000000000000
8.043976264341267778945621103048324584960937500000000000000000
2.018378646444957347227955324342474341392517089843750000000000
0.000021421208414486109049669829151874012040934758260846138000

But, even if I create a new np.array with the above values, or save-and-load eigvals to disk, I still get the same 2nd qf() results.

Do we expect SKS to contain imginary numbers?
import numpy as np
SKS = np.load('SKS_matrix.npy', allow_pickle=True)
SKS

array([[ 0.04429796-1.27082473e-09j, -0.04714658-1.32167941e-10j,
         0.01433359-3.37525097e-10j, ...,  0.02179421-4.00270757e-09j,
        -0.02024132-1.62844172e-09j, -0.03052728-7.82222069e-10j],
       [-0.04714658-3.93228427e-11j,  5.00593631+7.15638676e-10j,
      #...

Also, what is phis1?

Also, what is phis1?

Edited. Sorry.
SKS should be complex since S is the sq rt of another matrix

OK. I can reproduce the problem and think I have a workaround.
I think the issue is that qf.qf expects a contiguous array, but
eigvals = np.sort(eigvals[np.where(eigvals > 1e-8)])[::-1] is noncontiguous.
The fix is to call np.ascontiguousarray (or .copy(), or as_type, etc)
qf.qf(Q, np.ascontiguousarray(eigvals), acc=1e-7), eigvals, eigvals.dtype
It then always produces the 0.07390013408306784, ... answer.
If this seems right, let me know and I'll eventually update the qf.qf code to always call np.ascontiguousarray.

Carl
p.s. The way I could see that the original eigvals was not contiguous was to print eigvals.flags. It said:

C_CONTIGUOUS : False
  F_CONTIGUOUS : False
  OWNDATA : False
  WRITEABLE : True
  ALIGNED : True
  WRITEBACKIFCOPY : False

I see. So any modification to the generated eigvals turns it back to a contiguous array.
I tested a few other eigenvalue computation functions and it seems like they all generate non-contiguous arrays.
Thank you so much, Carl. Have a great New Year's.

Hi Carl,
I discovered what was causing the original issue of this thread.
When chi2val is too small or too large, the p-value can become 2.0 or a negative tiny float before rounding to 1.0 or 0.0, respectively. And where these happens depend on the acc option.
Probably float under/overflows?
qf.qf(0.068, np.array([12.4]))
>>>(0.9409680834684881, 0, array([4.51497377e+00, 9.88515000e+05, 7.00000000e+00, 9.30972623e-03, 7.39576675e+05, 5.52618098e-06, 1.83000000e+02]))

qf.qf(0.067, np.array([12.4]))
>>>(2.0, 1, array([0.00000000e+00, 7.10867000e+05, 6.00000000e+00, 9.30971243e-03, 8.06810919e+05, 5.44491361e-06, 1.83000000e+02]))

qf.qf(25428, np.array([620.5, 312.0, 204.8, 137.4]))
>>>(2.4491519923230953e-13, 0, array([2.23133202e+01, 5.68300000e+03, 1.00000000e+00, 2.47007504e-04, 1.40350064e+00, 2.14053048e+00, 3.60000000e+01]))

qf.qf(25429, np.array([620.5, 312.0, 204.8, 137.4]))
>>>(-1.1501910535116622e-13, 0, array([2.23141681e+01, 5.68300000e+03, 1.00000000e+00, 2.46997786e-04, 1.40350064e+00, 2.14061910e+00, 3.60000000e+01]))

qf.qf(22961, np.array([620.5, 312.0, 204.8, 137.4]), acc=1e-7)
>>>(3.0772051573535464e-12, 0, array([2.00929994e+01, 1.77000000e+03, 1.00000000e+00, 2.73181747e-04, 4.83299118e-01, 6.07757605e+00, 3.40000000e+01]))

qf.qf(22962, np.array([620.5, 312.0, 204.8, 137.4]), acc=1e-7)
>>>(-4.4364512064021255e-13, 0, array([2.00938365e+01, 1.77000000e+03, 1.00000000e+00, 2.73169826e-04, 4.83299118e-01, 6.07785630e+00, 3.40000000e+01]))

Thanks.


Give appropriate URL to evaluated JS files.

This will enable remote debugger to correctly load and display available runtime scripts.
Modules will be sourced from JAVASCRIPT_SOURCE_PLATFORM_URL template.
Program scripts which includes main script and any script loaded using phantom.injectJs() will be sourced from JAVASCRIPT_SOURCE_CODE_URL template.
Issue #12864

@iradul I think you messed up the pull request a bit :-) I see wrong commits here.

Oops I have no idea what that was ... Looks like it's fixed after rebase

Landed in master! Thank you!


buffer is truncated when resizing terminal rather than wrapping

I'm not sure if this is an intended behaviour, but when a terminal is resized, characters which exceed the terminal width are truncated and lost rather than wrapping.
Details

Browser and browser version: Electron
OS version: All
xterm.js version: 2.3.1

Steps to reproduce

Create a terminal
Write characters to fit current column width
Resize terminal to be less than sentence character width, characters not wrapped, instead truncated
Resize terminal back to previous wider column width, characters lost not re-appearing

Duplicate https://github.com/sourcelair/xterm.js/issues/325


Platform efm32gg

Add support for the efm32 giant gecko family

I'm confused, I think you reviewed old files, couldn't find any of those leftovers, was it in a specific commit?

Yes, I did the reviews per commit but Github UI is awful showing that :/

Should be clean now

Should be all good now


[ADAM-563] Add warning message when saving Parquet files with incorrect extension

Resolves #563. Prints warning if file does not have "*.adam" extension.

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

Rebased on ToT.

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


Add Gravity

I have update the risk view descriptions to be in line with #794. Please check if the changes are correct before proceeding

I decided not to include PSTAKE because it is currently 635th by mc on coingecko, so it fells below our top-300 coingecko criteria


Unknown prop animate on  tag with React 15.2.1 and AutoComplete

Problem description
Since I updated React (15.2.1) and material-ui (0.15.2), I get the error below from React:
warning.js:44 Warning: Unknown prop animate on <input> tag. Remove this prop from the element.
For details, see https://fb.me/react-unknown-prop
in input (created by TextField)
in div (created by TextField)
in TextField (created by AutoComplete)
in div (created by AutoComplete)
in AutoComplete (created by SuggestInput)

Steps to reproduce
This is my component SuggestInput :
    <AutoComplete
    className={this.props.className}
    animate={true}
    fullWidth={true}
    hintText={this.props.hintText}
    dataSource={this.state.data}
    onUpdateInput={this.handleUpdateInput}
    onNewRequest={this.handleNewRequest}
    filter={this.filter}/>

Versions

Material-UI: 0.15.02
React: 15.2.1
Browser: Chrome latest

There is not animate property on the documentation for this component.
As far as I know, that isn't supposed to do anything.


fix grammer

Coverage remained the same at 97.673% when pulling 66494c869737cc6f5080c4b5c4d7f71bc866e69a on ijin:fix_grammer into 02ac9390445b98ee5f0c80006556a151ec0f07ae on marcy-terui:master.

Thanks!! 💦  💦  💦


Improve outdated examples of package sources in NuGet.Configs

Several examples here are outdated, using terminology that is inconsistent with our other verbiage in docs/messages/blogs, and may be confusing
eg, "NuGet official package source"

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

ID: 370d9d91-9f35-79aa-3ace-7b77436e988f
Version Independent ID: 3da4591a-3d3d-c07a-6672-97964de13b14
Content: Common NuGet configurations
Content Source: docs/consume-packages/configuring-nuget-behavior.md
Service: nuget
GitHub Login: @JonDouglas
Microsoft Alias: jodou

@donnie-msft - I am not sure why I assigned this issue to myself. I have now removed my assignment. Sorry for the confusion. Please feel free to assign this issue to yourself if you are still interested.

We apologize for the inconvenience. We are phasing out GitHub issues as the feedback mechanism on NuGet documentation content and replacing it with a new feedback system using the Feedback link within the articles. If you believe your feedback needs to be preserved and addressed, please submit a PR or open a new issue at https://github.com/NuGet/Home/issues


Hive Partition pruning cases

Khurram,
I added some hive dynamic partition pruning tests. The functional run passed. Can you kindly review?

Rahul

Verified that commit ID is same in functional run.
Test failures are not related to any of your changes.
Tests look good. LGTM.


[bug] :windows 10 : vulkan: release build: dynamic heap exhausted

😂 I'm so sorry, found another bug.
Steps to re-produce

In code: Make use of SHADER_RESOURCE_VARIABLE_TYPE_DYNAMIC
Run the program in Release build on Windows with Vulkan backend.
Click the taskbar app icon, to minimize the rendering window.

Problem
Diligent Engine: Warning: Space in dynamic heap is almost exhausted. Allocation forced wait time of 2.0 ms. Increase the size of the heap by setting EngineVkCreateInfo::DynamicHeapSize to a greater value or optimize dynamic resource usage

I can also re-produce this issue with builtin sample: `GLTFViewer` and `Tutorial27_PostProcessing`.

My guess is vulkan swapchain's present somehow not happen with minimized window.
Assosiated dynamic heap recycle code, not running.

What is you Windows version? GPU? Driver version?

Win10 Professional x64 - 22H2 19045.4651
GPU: GTX1070 v556.12
Vulkan: v<IP_ADDRESS>

When play with and tracking down this bug with sample Tutorial27_PostProcessing,
I may found another small bug in that sample.
around Tutorial27_PostProcessing.cpp, line 303:
CurrCamAttribs.f4ViewportSize = float4{Resolution.x, Resolution.y, 1.f / Resolution.x, 1.f / Resolution.y};

After window resize, it is using previous window size.
Should update viewport size first:
Resolution.x = m_PostFXFrameDesc.Width;
Resolution.y = m_PostFXFrameDesc.Height;
CurrCamAttribs.f4ViewportSize = float4{Resolution.x, Resolution.y, 1.f / Resolution.x, 1.f / Resolution.y};

When play with and tracking down this bug with sample Tutorial27_PostProcessing, I may found another small bug in that sample.
around Tutorial27_PostProcessing.cpp, line 303:
CurrCamAttribs.f4ViewportSize = float4{Resolution.x, Resolution.y, 1.f / Resolution.x, 1.f / Resolution.y};

After window resize, it is using previous window size. Should update viewport size first:
Resolution.x = m_PostFXFrameDesc.Width;
Resolution.y = m_PostFXFrameDesc.Height;
CurrCamAttribs.f4ViewportSize = float4{Resolution.x, Resolution.y, 1.f / Resolution.x, 1.f / Resolution.y};

Everything is correct here; we have Source Size and Output Size. The first value determines the size of the render targets, and the second one determines the window size. They should differ because we are using FSR in this tutorial

Got it, I understand what you mean. We just need to move SetupUpsamplingSettings before setting the resolution.

https://github.com/DiligentGraphics/DiligentSamples/pull/188

Does this reproduce in Debug build?

Only reproduce in Release build.
And must click the taskbar icon to minimize.
I found that click window's top right minimize button, no warning, strange.


VT mouse mode causes clicks to display read-only pane popup

Windows Terminal version (if applicable): 1.7.391.0

The new read-only pane mode is great until mouse mode is enabled (check out vim, htop, mc in WSL)
Mouse events go straight into TrySendInput (or similar) which pops up a dialog. Every time you click. :smile:

/cc @don-vito fyi for read-only stuff

@DHowett - I kinda did this intentionally, as mouse click might modify the page.

That's a fair choice. I'm worried that it'll catch people off-guard, because clicking in a pane will obscure the content & they may not be expected to know that the app asked for mouse mode.
The hover thing worries me a bit more. Hover tracking will make it impossible to mouse over the terminal without getting a dialog :smile:

That's a fair choice. I'm worried that it'll catch people off-guard, because clicking in a pane will obscure the content & they may not be expected to know that the app asked for mouse mode.
The hover thing worries me a bit more. Hover tracking will make it impossible to mouse over the terminal without getting a dialog 😄

So just to make sure we are aligned on the principles:

No matter what we do not pass input to the connection, neither key, click nor hover (hover theoretically can also modify the content)
Said that, we need to make the notification less invasive

If we agree on the above, the next question how to make the notification less invasive. We can:

Show nothing (aka remove the dialog box completely)
Show some kind of info bar
Show the dialog as today on all occasions (key input / click) but not on mouse hover

WDYT?

Show the dialog as today on all occasions (key input / click) but not on mouse hover

That actually seems like it makes sense to me. I'm thinking of the case where I've got some tmux panes open, but the tab is read only. I try to click on one to focus it and OH NO IT DOESN'T FOCUS! THE TERMINAL IS BROKEN! - No, you just enabled read-only mode. That's why the mouse click didn't work.
Idk, that makes sense to me.

Show the dialog as today on all occasions (key input / click) but not on mouse hover

That actually seems like it makes sense to me. I'm thinking of the case where I've got some tmux panes open, but the tab is read only. I try to click on one to focus it and OH NO IT DOESN'T FOCUS! THE TERMINAL IS BROKEN! - No, you just enabled read-only mode. That's why the mouse click didn't work.
Idk, that makes sense to me.

Yeah. Makes sense to me as well. @DHowett  - any other thoughts? :blush:

Sure!

@zadjii-msft - I am noob with VT. Can you guide me what is the simplest way to enable mouse position reporting?

Silenced the dialog on mouse move and wheel. Hope it is better now 🙈

In https://github.com/microsoft/terminal/issues/6981, I hoped that the read-only mode would just suppress mouse reporting:

Do not forward any mouse events to processes using the pseudoconsole. Instead, let mouse events scroll the scrollback and select text. This should override the ENABLE_MOUSE_INPUT mode and perhaps also ENABLE_QUICK_EDIT_MODE.

ENABLE_QUICK_EDIT_MODE

@KalleOlaviNiemitalo - you are right as usual. This would be a better experience, but it requires a change in the approach. I would suggest to advance with what we have and  to open this as a follow up. The question is if what we have now is usable: the only mouse interaction you are allowed is scrolling and hovering.


Feature/node https agent

Flow now supports https agent extra params.
See https://nodejs.org/dist/latest-v7.x/docs/api/https.html#https_https_request_options_callback

Sorry this never got looked at. We've since added the https.Agent typedef, though this PR was more thorough about the options. Since part of this has already been landed separately, I'm going to close, but feel free to reopen if you would like to get the options in.


Airflow AMI

I'm in the process of setting up a EC2 instance for our airflow environment and it occurred to me how useful it would be if an AMI already existed.
Do you guys run airflow on a dedicated box in-house and is it an EC2? Would you ever consider making a standard AMI?
This could definitely alleviate some of hassle of setting up the backend components (Mysql/Celery executor components..)

We have specific requirements at Airbnb to glue all these systems together.
I have no experience setting up AMIs but I'm sure the community would appreciate if someone was to set this up...

I'm doing something a little parallel to this. I'm in the process of Dockerizing airflow so that you'll be able to deploy to Heroku with a couple of commands.

@neovintage did you end up making progress on this? Would be curious to see it. I am attempting something similar atm.

Yes I did and it works. :-)
https://Github.com/heroku/heroku-airflow
There's zero documentation on how to release it to heroku and is all in my head unfortunately. I'll try to write that up soon
Sent from my iPhone

On Oct 2, 2015, at 1:27 PM, Jasmine Tsai<EMAIL_ADDRESS>wrote:
@neovintage did you end up making progress on this? Would be curious to see it. I am attempting something similar atm.
—
Reply to this email directly or view it on GitHub.

Nice, are you running the scheduler + webserver in the same instance or
separately ?
On Fri, Oct 2, 2015 at 2:17 PM, Rimas Silkaitis<EMAIL_ADDRESS>wrote:

Yes I did and it works. :-)
https://Github.com/heroku/heroku-airflow
There's zero documentation on how to release it to heroku and is all in my
head unfortunately. I'll try to write that up soon
Sent from my iPhone

On Oct 2, 2015, at 1:27 PM, Jasmine Tsai<EMAIL_ADDRESS>wrote:
@neovintage did you end up making progress on this? Would be curious to
see it. I am attempting something similar atm.
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/airbnb/airflow/issues/280#issuecomment-145157140.

--
Jasmine Y. Tsai
+1<PHONE_NUMBER>

Scheduler runs in a separate process. Although, it's not documented in the current Procfile on the project.

Closing. If this is still a question please contact the apache mailing list.


GLSL grammar update.

Add names to operands.
Fix capability requirements for UnpackDouble2x32.
Fix operands list error for NClamp.

+2


Matrix

I have made things!
Checklist

[x] I'm sure that I did not unrelated changes in this pull request
[x] I have created at least one test case for the changes I have made
[x] I have updated the documentation for the changes I have made
[x] I have added my changes to the CHANGELOG.rst

Related issues
Format is:

Closes #727
Refs #727

I tried to follow the symmetry with the integers() function of the same provider, that's why I use the two parameters start and end. The implementation is quite straightforward.

@ceccoemi I'm closing this because of conflicts. Can you please recreate it?

@lk-geimfari I will recreate it as soon as I can.


non win-1252 characters in SVG

Asciidoctor input:
[blockdiag, format="svg"]
.Пример диаграммы
----
blockdiag {
  A[label="изображение"];
  B[label="color filtering"];
  A -> B;
}
----

Theme file:
font:
  catalog:
    PTAstraSerif:
      normal: PTAstraSerif-Regular.ttf
      bold: PTAstraSerif-Bold.ttf
      italic: PTAstraSerif-Italic.ttf
      bold_italic: PTAstraSerif-BoldItalic.ttf
  fallbacks:
    - PTAstraSerif
base:
  font_family: PTAstraSerif

Generated svg:
...
  <text fill="rgb(0,0,0)" font-family="sans-serif" font-size="11" font-style="normal" font-weight="normal" text-anchor="middle" textLength="64" x="320" y="67">color filtering</text>
...

Asciidoctor execution:
$ asciidoctor-pdf \
    -a pdf-style=$HOME/.asciidoctor/themes/basic-theme.yml \
    -a pdf-fontsdir=$HOME/.asciidoctor/fonts \
    -r asciidoctor-diagram \
    test.adoc
The following text could not be fully converted to the Windows-1252 character set:
| изображение

There is no way to specify SVG font for 'blockdiag', it will always use "sans-serif".
Afaik, there is no way to replace default "sans-serif" in asciidoctor.
If I manually edit generated svg and change "sans-serif" with "PTAstraSerif", everything work without errors.

I'd argue this bug lies with Prawn SVG. Asciidoctor does allow you to map generic font names like sans-serif in your font catalog.
font:
  catalog:
    sans-serif:
      normal: PTAstraSerif-Regular.ttf
      bold: PTAstraSerif-Bold.ttf
      italic: PTAstraSerif-Italic.ttf
      bold_italic: PTAstraSerif-BoldItalic.ttf

However, Prawn SVG is not respecting these mappings. If it sees a generic font name (e.g., sans-serif), it maps it to a built-in AFM font (e.g., Helvetica) without first consulting the font registry. You can see where this happens here:
https://github.com/mogest/prawn-svg/blob/5d40c5fab3eabc34650269c663931f98c3b4430e/lib/prawn/svg/font.rb#L24
If Prawn SVG where to consult the font registry first, everything would work.
There is a workaround for this problem. Create a file named prawn-svg-patch.rb with the following content:
require 'prawn-svg'

Prawn::SVG::Font::GENERIC_CSS_FONT_MAPPING['sans-serif'] = 'PTAstraSerif'

Then load it when you are calling Asciidoctor PDF:
$ asciidoctor-pdf -r ./prawn-svg-patch.rb -r asciidoctor-diagram diagram.adoc

Would you be willing to file the bug/feature request in Prawn SVG?

I also spent time trying to figure out how to get blockdiag to use a font other than sans-serif. Even though I was able to configure a custom fontmap and define new fonts, I could never figure out where it decides to use sans-serif in the SVG. It almost appears as though the font configuration is for raster output only. blockdiag is extremely limited in this regard and the documentation lacking. That's too bad. If there was a clear way to set the font in blockdiag, I'm sure we could add it to Asciidoctor Diagram.

See https://github.com/mogest/prawn-svg/issues/93.

PR sent.

Thank you for workaround. I'm not familiar enough with Prawn to file bugreport.
I reported font selection problem to blockdiag upstream.
https://bitbucket.org/blockdiag/blockdiag/issues/82/add-ability-to-change-font-family-for-svg
But, they think that this can be achieved with 'fontmap' and don't explain how.
P.S. Sorry, for delay in replying.

Thank you for workaround. I'm not familiar enough with Prawn to file bugreport.

I've already filed a report and submitted a patch. See https://github.com/mogest/prawn-svg/pull/94

they think that this can be achieved with 'fontmap' and don't explain how.

Perhaps I'm just too stupid, but I could not figure out how to use fontmap. The configuration is extremely obtuse. But we already have the mapping we need if we can just get prawn-svg to respect it.

This is now resolved upstream. You'll be able to take advantage of it in alpha.16.

Great. Thank you.


adding link to the first paragraph copy on the /projects page

Fixes #86

Added a link to the copy in the first paragraph
Please feel free to reject this PR. I'm not sure why the _site/feed.xml is in this PR, especially since y'all have _site in the .gitignore

thoughts and feedback welcome.

:jack_o_lantern:


ShowNoResults not working

As per [http://semantic-ui.com/modules/search.html#/settings] ShowNoResults is false by default. I've enabled it by setting it true. But, still it is not showing "no results" message.
Below is coffee script code snippet,
$('#topNav .ui.search, .heroImage .ui.search').search   type: 'category'   minCharacters: 2   cache: false   searchDelay: 300   showNoResults: true   fields:     description: 'body'   apiSettings:     url: 'http://path/to/url?query={query}'     onResponse: (response) ->       response       onSelect: (result, response) ->     window.location.assign result.url

Hi @hemantthorat,
Thank you for posting, but although it’s a valid usage question, we’ve limited GitHub Issues to bug reports and feature requests, keeping the board more manageable for maintainers; see the contributing guidelines for more information on what kind of posts should find themselves into the GitHub Issues board.
To get answers or feedback that might allow you to repost this issue, please use one of our other support resources:

Gitter chatroom
Official forums
StackOverflow


Option to change the language without using the URL

Hi there,
How we can change user language without using the url and headers detection! ..
The issue with the URL is that the user when get back to the website, the language will be returned to the default language (e.g. en) not the language that the he have selected last time he visited the website (e.g fr).
So I want to store the language at localStorage on my app's settings page .. then Next.js app get the language from the localStorage.
How we can achieve that?
Thanks

Please check the NextJs documentation. This functionality is not handled by next-i18next.

@isaachinman do you have an idea about how to achieve this? example? I have searched a lot about this issue but I got nothing to help!!

Read the main documentation:
https://nextjs.org/docs/advanced-features/i18n-routing#automatic-locale-detection
https://nextjs.org/docs/advanced-features/i18n-routing#leveraging-the-next_locale-cookie
If you have any questions please use StackOverflow.

@munafio did you find a suitable solution to do this? If so, would you mind sharing?

@isaachinman I'm not sure this statement is correct:

This functionality is not handled by next-i18next.

next.js has support for internationalised routing.
So next.js does by default detect the locale (language-region-script) and automatically re-route the user based on locale. You can also change the locale yourself by simply adding a button and an onClick handler something like the following:
  const onToggleLanguageClick = (newLocale) => {
    const { pathname, asPath, query } = router;
    router.push({ pathname, query }, router.asPath, { locale: newLocale });
  };

But this re-route's the user. You cannot just change / select a language, it is tied to locale and routing.
As a user of next-i18next I want to be able to select the set of translations to use independent of the user's locale.
Tying language to locale makes an assumption about the user that they speak the language in the region they are browsing from. If I use myself as an example, I am an English speaker, but live in Norway. So I want to select language, regardless of my region.
With react-i18next I can do this with the changeLanguage function. It seems there is no equivalent offered from next-i18next. If next-i18next is responsible for deciding which set of translations to use then I believe you should be able to provide a changeLanguage function to do this.
Does this make sense / seem reasonable @isaachinman or have I misunderstood something?

https://github.com/i18next/next-i18next/blob/master/examples/simple/pages/index.js#L61

Thanks @adrai but this still changes the locale. The language is dependant upon the locale (i.e. region). Is it possible for these to be independent?

Like already mentioned before... the language detection is handled by next.js
https://nextjs.org/docs/advanced-features/i18n-routing#automatic-locale-detection
https://nextjs.org/docs/advanced-features/i18n-routing#leveraging-the-next_locale-cookie
fyi: your example English in Norway in theory you would need this accept-language header: en-NO... but I don't know if this will ever work, because it is not a "standard".
So it would be simpler if your accept-language header was en only...
How is your accept-language header currently set? (basically what is: navigator.languages in your browser?)

Thank you, I have read the next documentation and understand that next does by default, detects the locale (language-region-script) and automatically re-route the user based on locale (as mentioned in my comment above).
You may also note that you can disable this behaviour if you choose: https://nextjs.org/docs/advanced-features/i18n-routing#disabling-automatic-locale-detection
Not to repeat myself but I want to be able to select the set of translations to use independent of the user's locale.
What you seem to be saying is that next-i18next decides which set of translations to use based on the locale information from next.js. What I am suggesting is that the user of next-i18next (i.e. me as a developer) should be able to manually change the set of translations used.
Your response indicates that the set of translations used is dependant on the locale information detected by next.js and I am arguing that, while that is a good default, it should be possible to choose independently.

fyi: in theory you could do something like this:
export const getServerSideProps = async ({ locale, req }) => {
  // inspect header accept-language header and set locale to your desired value
  if (req.headers['accept-language'].indexOf('en-NB') > -1) locale = 'en'
  return {
    props: {
      ...await serverSideTranslations(locale, ['common', 'footer']),
    },
  }
}

this would set the locale language to the actual language you want to show without changing the route...

@adrai What @ChristopherHButler was saying was that he wants the ability to change the language that the webpage is displayed in without any change in the URL that is displayed to the client browser. Like in regular React: click button -> content changes language, URL remains the same.

The solution is to use a different i18next package: medium article

@ChristopherHButler would https://nextjs.org/docs/advanced-features/i18n-routing#leveraging-the-next_locale-cookie solve your issue?

@shettayyy NEXT_LOCALE only works with internationalized routing. It just helps to redirect the user to, let's say, example.com/en/page when you go to example.com/page.

So actually I found a solution which can be implemented using this library, but it requires server-side rendering. Also, you must use the pages/ folder with getServerSideProps(). What you do is disable Next.js automatic locale detection and redirection completely and manually store a locale cookie on client side. When client makes the request to the Next.js backend, the server reads the cookie and renders the page in the language that is specified in the cookie and then returns that rendered page to the client. You might think that "Damn, that must be resource heavy always rendering those pages", but no! Actually, what you do in production is that you put a reverse-proxy like NGINX, Apache or Traefik between the client and your Next.js server (Express on Node), which caches the requests based on the request cookie contents. This way, the reverse proxy will serve the client a pre-cached page of the website and your backend is spared from having to re-render the same page over and over again. You can set up re-caching rules so that when the data changes on the source page, the reverse proxy will make a new copy of the page. Eventually, the reverse-proxy cache will contain copies of all the locale versions of your website and serve them to the clients. This works also for user account pages, which content depends on the user account. You can just create the page with placeholders, so that the page is rendered with loader spinners and then the data itself is fetched on client-side. Easy peasy. 😄

fyi: in theory you could do something like this:
export const getServerSideProps = async ({ locale, req }) => {
  // inspect header accept-language header and set locale to your desired value
  if (req.headers['accept-language'].indexOf('en-NB') > -1) locale = 'en'
  return {
    props: {
      ...await serverSideTranslations(locale, ['common', 'footer']),
    },
  }
}

this would set the locale language to the actual language you want to show without changing the route...

So yeah, that's the gist of the solution, but you also must disable the automatic language detection and redirection by Next.js. Also, there is a problem with correctly setting the <html lang=''> property, because Next.js changes it automatically only when its language autodetection is used, meaning that you will need to set it yourself manually. Also, it's a brainfuck to do it manually, because Next.js does not provide a simple and elegant solution to change the lang property yourself.

The solution is to use a different i18next package: medium article

It works, thank you.
But it's hard to believe that one needs to get another package to make such a basic functionality work!

The solution is to use a different i18next package: medium article

Hello
I am  using next js 14 how can I implement this package in it

@nikita-fxbytes
I use next-international for my projects, works very well.
This is the content of the middleware.ts file to rewrite requests from a locale subpath to root path:
import { createI18nMiddleware } from 'next-international/middleware'
import { NextRequest } from 'next/server'

const I18nMiddleware = createI18nMiddleware({
  locales: ['et', 'en'],
  defaultLocale: 'et',
	urlMappingStrategy: 'rewrite'
})

export function middleware(request: NextRequest) {
  return I18nMiddleware(request)
}

export const config = {
  matcher: ['/((?!api|static|.*\\..*|_next|favicon.ico|robots.txt).*)']
}

Thank you, @aabmets, next-international works well.
I put a lot of thought into choosing the i18n library, and thanks to that, I found a way to make it work without changing the route.
document pages also good.

It works for me. To make changeLanguage work, you need to add this to _app.js That will initialize the i18n
import { appWithTranslation } from 'next-i18next';
import nextI18NextConfig from '../next-i18next.config'; // Import the next-i18next config
...

function App({ Component, router, pageProps }) {
  const { query } = router;

  return (
...
}

export default appWithTranslation(App, nextI18NextConfig);


Upgrade to cuda 12.1.1.

For the moment, seeing what the CI says about this. May make a more general way to switch based on findings.

Should we close this in favor of https://github.com/openxla/iree/pull/13875 ? (and delete the cuda_12_1_1 branch - trying to keep the number of branches in the shared repo small if possible)


Permessage deflate

This PR implements permessage deflate.
This PR introduces an API change because outgoing messages must be handled in order to apply compression.
This PR only works with PHP 7.0.18+ and 7.1.4+.
Closes #18

@cboden Could we get this merged and released? We're using it in production without any issues.

@cboden yesterday's IRC ping was about moving this forward.

@mbonneau Can you resolve the conflicts please?

I'm really really interested by this PR for my project (Movim) :)

@mbonneau thanks a lot for the long and hard work on that pull request!
I'm really excited to have this feature included in my project https://github.com/movim/movim for the upcoming release


CDK Pipeline assets build fails

Describe the bug
Changed the code in "backend/dataall/cdkproxy/stacks/dataset.py" to change the bucket versioning behaviour to be "false" by default (line 244) , see screenshot

Updated code was then pushed back to repo and the code build started , it then failed

Tried reversing code change and rebuilding but still fails with the same issue , here's an excerpt from the build log:
/usr/local/bin/cdk-assets -> /usr/local/lib/node_modules/cdk-assets/bin/cdk-assets
/usr/local/bin/docker-credential-cdk-assets -> /usr/local/lib/node_modules/cdk-assets/bin/docker-credential-cdk-assets

<EMAIL_ADDRESS>added 129 packages from 105 contributors in 6.589s

[Container] 2022/11/09 14:32:48 Phase complete: INSTALL State: SUCCEEDED
[Container] 2022/11/09 14:32:48 Phase context status code:  Message:
[Container] 2022/11/09 14:32:48 Entering phase PRE_BUILD
[Container] 2022/11/09 14:32:48 Phase complete: PRE_BUILD State: SUCCEEDED
[Container] 2022/11/09 14:32:48 Phase context status code:  Message:
[Container] 2022/11/09 14:32:48 Entering phase BUILD
[Container] 2022/11/09 14:32:48 Running command cdk-assets --path "assembly-dataall-dev-cicd-stack-dataall-dev-backend-stage/dataalldevcicdstackdataalldevbackendstagebackendstackC936AB86.assets.json" --verbose publish "bdb08b9984816ecf43301a0c6d8913e270ee2448940b6b95e430dd222a4d90be:123456789012-eu-west-1"
verbose: Loaded manifest from assembly-dataall-dev-cicd-stack-dataall-dev-backend-stage/dataalldevcicdstackdataalldevbackendstagebackendstackC936AB86.assets.json: 23 assets found
verbose: Applied selection: 1 assets selected.
info   : [0%] start: Building and publishing bdb08b9984816ecf43301a0c6d8913e270ee2448940b6b95e430dd222a4d90be:123456789012-eu-west-1
verbose: [0%] check: Check s3://cdk-hnb659fds-assets-123456789012-eu-west-1/bdb08b9984816ecf43301a0c6d8913e270ee2448940b6b95e430dd222a4d90be.zip
error  : [100%] fail: Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1
Failure: CredentialsError: Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1
at Request.extractError (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/protocol/query.js:50:29)
at Request.callListeners (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
at Request.emit (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
at Request.emit (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/request.js:686:14)
at Request.transition (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/request.js:22:10)
at AcceptorStateMachine.runTo (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/state_machine.js:14:12)
at /usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/state_machine.js:26:10
at Request. (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/request.js:38:9)
at Request. (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/request.js:688:12)
at Request.callListeners (/usr/local/lib/node_modules/cdk-assets/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
[Container] 2022/11/09 14:32:48 Command did not exit successfully cdk-assets --path "assembly-dataall-dev-cicd-stack-dataall-dev-backend-stage/dataalldevcicdstackdataalldevbackendstagebackendstackC936AB86.assets.json" --verbose publish "bdb08b9984816ecf43301a0c6d8913e270ee2448940b6b95e430dd222a4d90be:123456789012-eu-west-1" exit status 1
[Container] 2022/11/09 14:32:48 Phase complete: BUILD State: FAILED
[Container] 2022/11/09 14:32:48 Phase context status code: COMMAND_EXECUTION_ERROR Message: Error while executing command: cdk-assets --path "assembly-dataall-dev-cicd-stack-dataall-dev-backend-stage/dataalldevcicdstackdataalldevbackendstagebackendstackC936AB86.assets.json" --verbose publish "bdb08b9984816ecf43301a0c6d8913e270ee2448940b6b95e430dd222a4d90be:123456789012-eu-west-1". Reason: exit status 1
[Container] 2022/11/09 14:32:48 Entering phase POST_BUILD
[Container] 2022/11/09 14:32:48 Phase complete: POST_BUILD State: SUCCEEDED
[Container] 2022/11/09 14:32:48 Phase context status code:  Message:
I am not sure if it really is missing credentials ? those weren't changed - only the code was updated. This happened about 6 months ago when we were first implementing this version and @louishourcade helped resolve it while he was working on the project.
How to Reproduce
Changed code, committed and pushed changes - code build fails
Expected behavior
codebuild project completed successfully
Your project
n/a
Screenshots
No response
OS
n/a
Python version
3.7
AWS data.all version
1.0
Additional context
No response

Hello @gavclark, does this happen with any change that you apply on the pipeline? Also, if you just release a change with no modifications of the code, does the error persist?

Hi @gavclark this happens on a cross-account deployment when codebuild (in the source account) tries to upload assets into the cdk assets bucket in another account.
To fix this issue, the target account should be properly bootstrapped with the command:
cdk bootstrap --trust  -c @aws-cdk/core:newStyleStackSynthesis=true --cloudformation-execution-policies arn:aws:iam::aws:policy/AdministratorAccess aws:// /

Thank you @xavierxc14!


using boilerplate

Please someone right a full guide on how to use it or a newly updated biolierplate, i'm new to node js and can't figure out things since weeks now!

Hire someone who know Node.js and learn along the way.

I will try to update the boilerplate as soon as I will be able to. In meantime, as @collindavis try to find someone who knows js, since as its mentioned in the readme, you need to have some basic knowledge of js and json to be able to use nexrender, at least at that point in time. :)


Using Fixed Version runtime distribution mode

Hi,
Is it possible to init WebView from a folder path, ie fixed version runtime mode?
More details here:
https://learn.microsoft.com/en-us/microsoft-edge/webview2/concepts/distribution
Sorry if I missed something obvious, but I didn't find a way with Photino :)
Thanks!

Scratch that, we have already implemented a method that lets you set the runtime path.
See Photino.NET/PhotinoWindow.NET.cs:
/// <summary> Set runtime path for WebView2 so that developers can use Photino on Windows using the "Fixed Version" deployment module of the WebView2 runtime. See https://docs.microsoft.com/en-us/microsoft-edge/webview2/concepts/distribution </summary>
        public PhotinoWindow Win32SetWebView2Path (string data) {
            if (IsWindowsPlatform)
                Invoke(() => Photino_setWebView2RuntimePath_win32(_nativeType, data));
            else
                Log("Win32SetWebView2Path is only supported on the Windows platform");

            return this;
        }

You will still have to contend with any differences in the API that is used by the native code in Windows, and potentially missing features in macOS and Linux.

Thank you, I'll try what you suggested :)


West flash error: FATAL ERROR: command exited with status 1: /opt/SEGGER/JLink_V770e/JLinkExe -nogui 1 -if swd -speed 4000 -device nrf52 -CommanderScript /tmp/tmpk3q73nykjlink/runner.jlink -nogui 1

Hi,
I am having troubles when I try to flash my microcontroller nrf52840 with the command "west flash" and the error I get is the next:
FATAL ERROR: command exited with status 1: /opt/SEGGER/JLink_V770e/JLinkExe -nogui 1 -if swd -speed 4000 -device nrf52 -CommanderScript /tmp/tmpk3q73nykjlink/runner.jlink -nogui 1
-In the past I was working with zephyr vanila and everything was working fine but now that I work with nrf 2.2.0 I have troubles.

-I only get to flash with the next comand correctly nrfjprog -f nrf52 --debugreset && nrfjprog -f nrf52 --program build/zephyr/zephyr.hex --sectorerase --verify && nrfjprog -f nrf52 --debugreset.

-This error just happens when my memory flash used is larger than 50% so seems maybe is something related to partitions but I do not understand because I have other project that I can flash correctly so the problems seems is just related to this project.
¿maybe the bootloader is not working correctly?
I can send you all information you need, thank you in advance.
Best regards

Use the nrfjprog flash runner, not jlink, e.g. west flash -r nrfjrpog

What you say kind of work I will keep checking, but what could be the reason this command works in one project and not in another, the error message I get does not specify if my problem is about the memory or another reason, I would like to make it work with west flash because I always pass a preconfiguration of the flash before flashing the program.bin and I have some troubles when I use west flash -r nrfjprog after pass the preconfiguration.

@rubenwat this is not a Zephyr bug, but rather an NCS one if at all. Please post this question in the Nordic DevZone instead.


"Drănic","Booveni","Foișor","Padea" are listed in the drop-down menu

Description:
When user writes "Drănic","Booveni","Foișor","Padea" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law
Precondition:
The website is up an running.

Step 1
Write in the search bar "Drănic"

Expected results
The location appears in the drop-down menu as "Sat Drănic, DOLJ, (Drănic)"

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Booveni"

Expected results
The location appears in the drop-down menu as "Sat Booveni, DOLJ, (Drănic)"

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Foișor"

Expected results
The location appears in the drop-down menu as "Sat Foișor, DOLJ, (Drănic)"

Step 7
Write in the search bar "Padea"

Expected results
The location appears in the drop-down menu as "Sat Padea, DOLJ, (Drănic)"


[SPARK-27031][SQL] Avoid double formatting in timestampToString

What changes were proposed in this pull request?
Removed unnecessary conversion of microseconds in DateTimeUtils.timestampToString to java.sql.Timestamp which aims to output fraction of seconds by casting it to string. This was replaced by special TimestampFormatter which appends the fraction formatter to DateTimeFormatterBuilder: appendFraction(ChronoField.NANO_OF_SECOND, 0, 9, true). The former one means trailing zeros in second's fraction should be truncated while formatting.
How was this patch tested?
By existing test suites like CastSuite, DateTimeUtilsSuite, JDBCSuite, and by new test in TimestampFormatterSuite.

Can one of the admins verify this patch?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8407/
Test PASSed.

Test build #102938 has started for PR 23936 at commit 09a9f54.

Test build #102938 has finished for PR 23936 at commit 09a9f54.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Is there any behavior change?

I believe there shouldn't be.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8409/
Test PASSed.

Test build #102940 has started for PR 23936 at commit ff68c90.

Test build #102940 has finished for PR 23936 at commit ff68c90.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Test build #4592 has started for PR 23936 at commit 23d4eac.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8466/
Test PASSed.

Test build #103000 has started for PR 23936 at commit 23d4eac.

Test build #103000 has finished for PR 23936 at commit 23d4eac.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class FractionTimestampFormatter(timeZone: TimeZone)

Test build #4592 has finished for PR 23936 at commit 23d4eac.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class FractionTimestampFormatter(timeZone: TimeZone)

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/8514/
Test PASSed.

Test build #103052 has started for PR 23936 at commit d2b2b74.

Test build #103052 has finished for PR 23936 at commit d2b2b74.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

Merged to master


[OB3] ELK configurations to integrate ELK with the OB Accelerator for Analytics

Purpose

This PR includes the ELK configurations required to add ELK support to the OB Accelerator in order to perform OB Analytics based on the respective specification requirements.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Fix entry point for TypeScript transpile

Though this is not this library's problem, here is a tweak fix necessary for TypeScript project.
Documents
"Untyped imports" Section:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-1.html
// Succeeds if `node_modules/asdf/index.js` exists
import { x } from "asdf";

FYI

Common PR ( resolved ) in other library: https://github.com/reactjs/react-modal/pull/294

This change should absolutely not be necessary, since it's how node's module resolution algorithm has always worked and will always work.
The best practice in JS - which TypeScript deviates from at its peril - is to always omit index filenames, as well as extensions.

Okay, thanks for yoru quick feedback.
For those who may have the same problem, here is a temporar workaround:
import { shallow } from 'enzyme/build/index.js';

@ljharb
here is a related issue, and its milestone says this bug will be fixed in TypeScript 2.2
https://github.com/Microsoft/TypeScript/issues/12839


Cannot delete/upgrade Rancher catalog apps when their creator user is deleted

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Create a catalog app deployment, then delete the original user as someone else. Now, with the other user, try to delete or upgrade the app installation.
Result:
App cannot be upgraded nor deleted (clone works OK) with following message:
Users.management.cattle.io "user-cpf9t" not found; waiting on helm-controller_local

Rancher pod logs show:
2018/10/24 13:42:17 [ERROR] AppController p-tmfn2/tpsk [helm-controller] failed with : users.management.cattle.io "user-cpf9t" not found

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.1.1
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Custom
Machine type (cloud/VM/metal) and specifications (CPU/memory): VMs, 4 node setup
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.1", GitCommit:"4ed3216f3ec431b140b1d899130a69fc671678f4", GitTreeState:"clean", BuildDate:"2018-10-05T16:46:06Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.1", GitCommit:"b1b29978270dc22fecc592ac55d903350454310a", GitTreeState:"clean", BuildDate:"2018-07-17T18:43:26Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

For the record, it's possible to work around this issue by manually creating the user using kubectl and setting the fixed ID that is missing:
# get some existing user
kubectl get users.management.cattle.io u-6v928 -o yaml > tmpuser.yaml

Edit the yaml file, replace the ID with the deleted user's one. Then, create the user again running:
kubectl create -f tmpuser.yaml

Tried your workaround but got error on first step
$ kubectl get users.management.cattle.io u-123abc -o yaml > tmpuser.yaml
error: the server doesn't have a resource type "users"

Tried the workaround with rancher v2.1.1 and got the following error message:
$ kubectl get users.management.cattle.io u-some-user -o yaml > tmpuser.yaml
error: the server doesn't have a resource type "users"

There's another way to fix this now?

Delete app issue will be fixed in 2.2.
Created a new ticket for the upgrade issue https://github.com/rancher/rancher/issues/17265

The bug is reproduced on Rancher:v2.1.1 as steps in the original ticket.
Bug fix is validated on Rancher: master 1/3 as steps in the original ticket.
Result: the catalog is removed successfully.


Migration: Component task-ilo-alignment-modal.coffee Review Document

Description
As the first step of Migration: Component task-ilo-alignment-modal.coffee, comepleted the component review and created the markdown document.
Fixes # (issue)
No issues
Type of change
went through the component and reviewed how function works.
created the markdown document.
The previous PR:
https://github.com/thoth-tech/documentation/pull/472

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as
expected)
[ ] Documentation (update or new)

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have requested a review from ... on the Pull Request

@WilliamALowe will be reviewing this.

Reviewed 14/12/2023 - COPIED FROM PREVIOUS REPO
##Purpose

Not used to maintain alignment, but is used to set a unit task's
alignment rating to a given learning outcome from an admin/convenor account,
and apply justification as to why it relates to that learning outcoming.

##Outcomes

Again, does not maintain alignment.
Acts a popup window for Admin/Convenor accounts.
Does not include the ability to to enter learning outcome of assignments,
only includes the adding a  rating + justification. (adding the actual learning
outcomes occurs on a different component).
Consider what data points are in use and need to be maintained
(refer to factory and controller methods).

##Proposal

Consider changes that can affect how the data can be input, while
still maintaining their typing and formats, rather than purely visual changes.
Consider 2 other (3 Total) different implementations, each with prototype
and/or test data.
Consider advantages + disadvantages for each.
Ensure that passed data, and output data are maintained.

##Checklist

Consider more functional tests, e.g: test that data can be validated/checked
within other components where the output data is used - student task
submission.
Consider a test to ensure that data is maintained in correct format and type.

##Other notes

Minor grammatical errors.
Make PR on thoth-tech website repo here : https://github.com/thoth-tech/ThothTech-Documentation-Website/tree/main/src/content/docs/products/ontrack/documentation/front-end-migration

Re-reviewed 15/12/2023

Please see contributing document and ensure that documentation passes all tests. Merging will not occur otherwise.


Version 1.1.0 seems to be missing from the maven-verse..

I see 1.2.0-SNAPSHOT and 1.0.0.
https://oss.sonatype.org/#nexus-search;quick~vertx-lang-typescript
It does seem however that version 1.0.0 is working just fine (just an older typescript I suppose).

Interesting. I'm pretty sure I uploaded the files. Nevertheless, I uploaded them again. Thanks for letting me know!
Cheers,
Michel


Add user.js

For lazy people it's enough to copy this into ones ~/.mozilla/firefox/*.default/ directory and customize it and for sys admins it's a bit easier to distribute it to user.js

Nice idea. I will duplicate the text version in a user.js after it is more or less finalized.


Avoid BytesWarning on logging responses

type(response['body']) is bytes. Formatting it as %s results in a BytesWarning on Python 3.

Codecov Report
Base: 93.54% // Head: 93.49% // Decreases project coverage by -0.05% :warning:

Coverage data is based on head (0152e0d) compared to base (47def35).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2847      +/-   ##
===========================================
- Coverage    93.54%   93.49%   -0.06%
===========================================
  Files           63       63
  Lines        13398    13396       -2
===========================================
- Hits         12533    12524       -9
- Misses         865      872       +7

Impacted Files
Coverage Δ

botocore/parsers.py
99.47% <100.00%> (-0.01%)
:arrow_down:

botocore/compat.py
91.33% <0.00%> (-1.34%)
:arrow_down:

botocore/hooks.py
95.27% <0.00%> (-0.40%)
:arrow_down:

botocore/httpsession.py
91.40% <0.00%> (-0.40%)
:arrow_down:

botocore/utils.py
79.01% <0.00%> (-0.20%)
:arrow_down:

botocore/auth.py
97.84% <0.00%> (-0.01%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Bump clojure version to 1.10.1

The latest stable version is 1.10.1 so it would be nice to use that instead of 1.10.0 :).

Hi, sorry for taking so long to get to this.  I'm gonna merge your changes, but with the test modifications removed.  Those should be kept as-is.

Merged.  Thanks for your contribution.


Monitor and restart training if stuck in endless evaluation

Sometimes training stops because it gets stuck in an endless cycle of evaluation, even when all the containers are still running.  It would be ideal to monitor for this error (e.g. no additional training iterations created over the last hour) condition and restart training by: -
1 - Notifying via SNS that the error condition has occurred
2 - Restart training

Fixed via https://github.com/aws-deepracer-community/deepracer-on-the-spot/pull/128
Opted to only notify rather than restart, to allow for tourbleshooting.


Display the repository configuration

Add a screen that shows the content of the repository configuration section of the ORT result file.

Hi,
I am very interested into this tool, is there a way to implement this workbench into environment for multiple users?

@mariannoem-eltras It sounds like your request is not directly related to this ticket, could you create a new and describe more detailed what you mean by "environment for multiple users"?


[GSSOC'23] Image finder

I want to add image finder project. Please assign me this issue under GSSOC'23.

Could you please assign me this as a GSSOC Contributor.

@Aditi323 i am working on it so please look another issue.

please assign me this issue


Simplify the handling of scalar parameters

Passing a scalar to some top-level functions can be quite bothersome. For example, the functions  LinOp::apply(alpha, b, beta, x), Dense::scale(alpha), Dense::add_scaled(alpha, b) only accept a LinOp* as scalar. Also, passing a double would result in a compile-time error. Since this is quite unintuitive, a way to pass a single scalar into these functions should be introduced.
Current ideas:

add a new mixin EnableScalarApplyLinOp which implements overloads for LinOp::apply + add overloads to Dense

Pro:

simple to implement

Con:

need to add mixin to all matrix formats, solvers, preconditioners, etc
re-implementation of apply, which is already implemented twice
need to add overloads for new functions/mixin to new classes

replace LinOp* parameter type by a variant-like type, e.g std::variant<LinOp*, double>

Pro:

no additions to exisiting code
only requires one definition for new functions
clearer distinction that the alpha parameter should not be any LinOp

Con:

requires implementation of std::variant (perhaps not all of that)
changes API, although user code should not break

introduce a function LinOp* gko::scalar(T) which creates the corresponding LinOp in a global cache

Pro:

no overloads/additions to exisiting code

Con:

handling global cache

I'm mostly in favor of 2., due to the noted pros. The second-best choice would be 1., and 3. is just for reference.

I think #179 and #37 are somewhat related to this issue.

This is some example code for the variant-like type in 2: https://godbolt.org/z/djqqjhesb

I prefer 1., since it produces the clearest interface without any indirection, since double and const LinOp* are not really alternative types I would expect inside a Sum Type. Since the implementation of the overloads can defer directly to the other overloads, this should not need a full re-implementation, but instead only act as a wrapper to the other EnableLinOp::* overloads. With this kind of thin and cheap-to-implement wrapper, I think the balance between development effort for us vs. usability improvement for applications heavily tilts in the applications' favor.

As Tobias says, 1 should be relatively cheap to implement and manage, very much like some other mixins that we have. It makes code a bit less easy to understand, but considering the amount of mixins we have already, I don't think that is a main issue here.

I also prefer 1, and 3 without global cache.
Can 1 make EnableScalarApply inherited from EnableLinOp? it can take apply from EnableLinOp.
(Is it what @upsj means for wrapper?)
for 3, I would prefer std::shared_ptr scalar(value, exec) the specific short initialize for scalar.
the usage is apply(scalar(1.0, exec).get(), ...)

For 1. if EnableScalarApply is implemented by inheriting from EnableLinOp, should that be a replacement of EnableLinOp in concrete classes or an addition. If it is an addition, would any problems appear due to diamond inheritance?

Yes, it should be a replacement (as a feature-extended specialization), otherwise we get diamond inheritance.

If it should be a replacement, are there any downsides to directly add the overloads to EnableLinOp? Or the other way around: are there any instances where the feature-extension is not wanted?
Of course, extending EnableLinOp directly would be interface-breaking, due to the required template parameter.

Yes, anything interface-breaking would only be in the scope of 2.0, and I think we can do this without obvious downsides without the need to change EnableLinOp directly. Some types like Permutation matrices don't have a ValueType.

if just wrapping the another apply in EnableLinOp, it should be the same. and all needs to contain value_type in the class

Instead of introducing a new mixin, we could also add a specialization of EnableLinOp for the case if the ConcreteType contains the value_type member. We wouldn't need to find all the places where we need to replace the mixin, but we need to copy most of the EnableLinOp implementation (or introduce a base class?). Any thoughts on that?

@MarcelKoch I like the idea! I'm just not sure whether it is better to be explicit with a different type, or do this implicitly based on value_type, which might be hidden a bit too much, in case users want this functionality as well?


fix fade animation for toast

Fixes: #27779

I think it is nice to have animation when showing too.

I'm a bit mix matched about that 🤔 IMO a toast should be visible ASAP so the animation will slower the display

Yes, let's allow the same approach as modal. https://getbootstrap.com/docs/4.1/components/modal/#remove-animation

it work as the same of our Modal plugin, if you remove the fade class or set animation to false you won't have any animation.

@Johann-S, can you also remove the toasts from the DOM like we do with the alerts (https://deploy-preview-27792--twbs-bootstrap4.netlify.com/docs/4.1/components/alerts/#dismissing)?
The problem with not removing them is that there's a margin above the second toast if we dismiss the first: https://deploy-preview-27792--twbs-bootstrap4.netlify.com/docs/4.1/components/toasts/#stacking

The major OS's notification UI components have animation when display.

iOS / macOS: Notification
Android: Snackbar
Windows: Balloon

It is OK that there is no animation at default, but it should be allowed customization IMO.

Ow, I like that customizable idea!

Maybe we should merge this and make it customizable in a another PR?

@MartijnCuppens Not a blocker to me, at least.

The customization of the show animation can be tackled in an other PR, so we should merge this one, except if you have any other concerns ?

If every UI item has fade-in animation, but toast doesn't, you'll get the clunky feeling.
On the other side, 300ms animation is hardly a delay that would break the world, especially in the web app that already deals with latency because it may be waiting for API response, pooling the backend for events, using features like refetchInterval={} that separate user seconds away from learning something.

@Johann-S

I don't understand your second point 😟 The animation delay can be changed but I don't think it's what you mean 🤔

My point is: web applications are rarely realtime, most events are signalized to user with delay, so thinking that component animation is a problem is misguided. ;)

Oh ok now I understand 👍
That's true, but folks can enable/disable that animation


python311Packages.dvc-objects: 0.21.2 -> 0.22.0 , python311Packages.dvc-data: 0.48.1 -> 0.49.1

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.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.

Result of nixpkgs-review pr 231626 run on x86_64-linux 1

2 packages failed to build:

python311Packages.dvclive
python311Packages.dvclive.dist

22 packages built:

dvc
dvc-with-remotes
dvc-with-remotes.dist
dvc.dist
python310Packages.dvc-data
python310Packages.dvc-data.dist
python310Packages.dvc-gs
python310Packages.dvc-gs.dist
python310Packages.dvc-http
python310Packages.dvc-http.dist
python310Packages.dvc-objects
python310Packages.dvc-objects.dist
python310Packages.dvclive
python310Packages.dvclive.dist
python311Packages.dvc-data
python311Packages.dvc-data.dist
python311Packages.dvc-gs
python311Packages.dvc-gs.dist
python311Packages.dvc-http
python311Packages.dvc-http.dist
python311Packages.dvc-objects
python311Packages.dvc-objects.dist


[build.ps1] build swift-lmdb using cmake

This is tracked by rdar://139431575 as a part of https://github.com/swiftlang/vscode-swift/issues/562
Build swift-lmdb on Windows using newly added cmake files in the repository. This change makes it so that IndexStoreDB uses the same version of LMDB as swift-docc and enables SourceKit-LSP to use swift-docc as a dependency to fulfill documentation requests on behalf of the VS Code Swift extension.

https://github.com/swiftlang/swift-lmdb/pull/7
https://github.com/swiftlang/indexstore-db/pull/228
@swift-ci build toolchain

https://github.com/swiftlang/swift-lmdb/pull/7
https://github.com/swiftlang/indexstore-db/pull/228
@swift-ci build toolchain

https://github.com/swiftlang/swift-lmdb/pull/7
https://github.com/swiftlang/indexstore-db/pull/228
@swift-ci build toolchain

https://github.com/swiftlang/swift-lmdb/pull/7
https://github.com/swiftlang/indexstore-db/pull/228
@swift-ci build toolchain

https://github.com/swiftlang/swift-lmdb/pull/6
https://github.com/swiftlang/indexstore-db/pull/228
@swift-ci build toolchain


use of commands within EXTRA - GO block

that's probably minor thing - but I always trying to use forbidden commands inside EXTRA - like !let etc
it'd be nice to check if there are lines starting with "!" between EXTRA and GO and highlight it as an error

Should be possible. Only hitch is that !INCLUDE is valid in EXTRA. I will test some options.


Default Jersey thread pool should resize based on heuristics (e.g. queue depth)

The ThreadPoolExecutor used by default does not resize except under extreme conditions, and the algorithm cannot easily be changed. See #689 for context.
An algorithm that can start off with relatively few threads and adapt to load should be investigated.

Please do not do this.
The number of threads can be specified using configuration.
For production environments, the initial thread count should be the final thread count.
For dev environment you can live with a small number.
Variable number of threads is a nightmare to tune and troubleshoot. In most production environments you end up with a fixed thread pool anyway to avoid cold starts (experience from tuning WLS environments).

You misunderstand: configuration will continue to work as is. Users would still be completely free to specify max == min to achieve fixed size behavior. When the configuration does allow for growth, however, the hard-wired algorithm in ThreadPoolExecutor can only be affected by controlling the queue capacity, requiring a very small queue for growth to ever occur under reasonable conditions.
I was unaware of the specifics of the hard-wired growth algorithm and I strongly suspect others are as well; this can easily lead to very surprising behavior where growth (above min) 'never' occurs (exactly as happened with our previous default configuration). I want to eliminate this surprise by choosing a different growth strategy by default, while adding the ability to select a strategy via configuration.
The goals here are to:

Provide more control over when the pool should grow above the configured min ("core") size, replacing the hard-wired algorithm with a simple growth strategy interface (effectively a named predicate) that can be selected via configuration.
Provide 3 simple growth strategy implementations, e.g. whenQueueIsFull (prefer queueing over growth, same as hard-wired strategy), whenPoolHasRoom (prefer growth over queuing) and one based on some simple heuristic like queue size or growth rate.
Provide reasonable default behavior with sizing based on cores, a large queue size ( maybe based on memory) and the heuristic growth strategy.

Note that if a fixed pool is configured, the growth strategy will never be used.
We can debate whether the default pool size should be fixed or allow growth. My strong preference is to set a reasonable minimum but allow for growth to defer the need for tuning.

We do allow setting the queue capacity via config; however, trying to control pool growth solely via this parameter is very challenging...
First, you have to set it small enough that you will fill the queue or else growth will not occur; choosing such a value is non-trivial as it is highly dependent on specific loads. Once you do fill the queue, new threads will start spinning up, but... the moment you hit the max pool size, subsequent requests will be rejected until threads free up. Under continuous load, when you reach this point the queue is no longer helping to buffer load.
If this is the only knob we expose to manage growth then I strongly suspect most users will choose to use a fixed pool to avoid unnecessary request rejections (I certainly would); if so, the knob is useless for this purpose.
I'm reasonably confident that we can provide a better out-of-the box experience and am exploring options.

I agree that the default "growth" is useless. Nevertheless I am not for giving too many options.
I still think that dynamic number of threads is a very bad idea and would vouch for no-growth under all conditions.
I also agree that using queue for growth is not good, as it slows down the scale-up.
Nevertheless trying it on our own is a dangerous path. In WLS, there is also a "clever" thread pool, that tries to dynamically modify thread number.
Nevertheless when you start "cold" WLS in production under very heavy load, it will not manage to start the threads in time and will just fail.
So if you want to come up with some clever solution, it must be really clever, as otherwise we bring more problems.

If you really believe that growth should not be supported at all, then we should eliminate the current core and max pool size configuration elements and replace them with a single value. It makes no sense at all as things stand currently to have both.
However, I don't think this is the right approach: users should be able to control whether or not to use a fixed size.
The point of this work is not to try to satisfy demanding production cases! For those cases, users will clearly need to do their own tuning and will most likely choose a fixed pool.
For development, testing and moderate load cases having a default that supports growth avoids violating the principle of least surprise. Adding a simple configuration to control growth rate, again with a reasonable default, is consistent with such a model.

Let's take up the discussion on the PR. If we all conclude that the actual implementation is not worth it, so be it, but I'd like to have all the facts in front of us.


Adding smoke tests to Jenkins

Description
Added infrastructure to perform smoke tests in every Jenkins build. The idea is to have a minimal set of tests to validate each new build, they should not be an extensive test suite as they're executed each Jenkins build.
I added a few basic tests, but more are necessary, now testing

version match
reloading current map
loading every map
some blueprints validity
synchronous mode

Some may be more necessary than other, we can evaluate after a while, but some more tests seem to me necessary to add asap (not adding them due to lack of time, but other people can start adding their own).
Possible Drawbacks
Build time gets slightly longer.

This change is

Not adding it to the CHANGELOG as it does not affect user functionality.


fix useSyncExternalStoreWithSelector missing getServerSnapshot argument

When working with next.js ssr, useSyncExternalStoreWithSelector requires the getServerSnapshot argument, otherwise an error will be thrown.

Thanks :+1:


VoxelWorld

VoxelObject based on VoxelEmbeddedMultiMesh with the addtion data streaming.

I'm starting to think that this shouldn't be implemented directly into Voxel-Core, but instead be it's own independent thing

I think this is cool

The rework has changed the way VoxelObjects work, so what I'm thinking about is making VoxelWorld separate from VoxelObjects, but it'll still be a object that displays voxel content. However the way it does so is by using using other VoxelObjects to split the content voxel (i.e. chunking), with optional multiple threading.

The rework has changed the way VoxelObjects work, so what I'm thinking about is making VoxelWorld separate from VoxelObjects, but it'll still be a object that displays voxel content. However the way it does so is by using using other VoxelObjects to split the content voxel (i.e. chunking), with optional multiple threading.


Update find commands to ignore flags

Codecov Report

Merging #184 (802b388) into master (b9fa09b) will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #184      +/-   ##
============================================
+ Coverage     73.26%   73.34%   +0.07%
- Complexity      584      588       +4
============================================
  Files            94       94
  Lines          1803     1808       +5
  Branches        232      232
============================================
+ Hits           1321     1326       +5
  Misses          403      403
  Partials         79       79

Impacted Files
Coverage Δ

...java/seedu/address/logic/commands/FindCommand.java
100.00% <ø> (ø)

...in/java/seedu/address/commons/util/StringUtil.java
96.66% <100.00%> (+0.66%)
:arrow_up:

.../seedu/address/logic/parser/FindCommandParser.java
100.00% <100.00%> (ø)

...address/logic/parser/FindFoldersCommandParser.java
100.00% <100.00%> (ø)

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 b9fa09b...802b388. Read the comment docs.


Xcharge reset in custom ROM when lock device

Set option

Check

When lock -> unlock device

Same issue here and often even after few minutes when still unlocked


[release-v1.1] Use Vert.x 4.2.5

Replace the use of the vertx-stack-depchain with individual
dependencies and bump Vert.x from 4.2.4 to 4.2.5.
vertx-stack-depchain is just a bom, to avoid declaring
every single dependency, however, it's not productized at 4.2.5
https://maven.repository.redhat.com/ga/io/vertx/vertx-stack-depchain/4.2.5.redhat-00001/
Signed-off-by: Pierangelo Di Pilato<EMAIL_ADDRESS>

/lgtm

/cherry-pick release-v1.2

/cherry-pick release-v1.3


Displaying Contribution Data in GitHub contribution graph

Description
The current GitHub contribution graph/chart lack specific data insights within the graph.

I propose including additional data (as shown in the attached image), such as:

Total Contributions: A numerical display summarizing contributions over the year.

This Week’s Contributions: Contributions within the current week.

Best Contribution Day: The day with the highest number of contributions.

Today: No. of contribution made Today

Average Contributions Per Day.

Streak Details: Longest streak and the current streak.

This enhancement would provide users with data insights.

Example URLs
https://github.com/{username}
for example:
https://github.com/prem-k-r

This could be its own extension or possibly part of GitHub, I don’t think it's data that's easily available.

GitHub's GraphQL API provides detailed information about user contributions. It can be used.

This is not something Refined GitHub is for. The extension provides small enhancements, not entire new views. Every feature longer than 200 lines is painful to maintain (when compounded by 200 features, 7 years so far, frequent GitHub changes)

Sorry, I may have made you misunderstand with the earlier image.

I understand this may be beyond the scope of this project.
Thank you for your time.


Message signing

Hiro wants to support message signing and verification in its wallets, similar to the user flow described the MyCrypto wallet. Ledger device support is needed to offer parity between our software and hardware wallets.
The feature has been discussed in context of Hiro's wallet https://github.com/blockstack/stacks-wallet-web/issues/1051, and the Stacks blockchain https://github.com/blockstack/stacks-blockchain/issues/2693
Ultimately, we'd like support for structured data signing, like Ethereum's EIP-721, though this task is scoped to prefixed bytestring message signing.
Spec

The device should support prefixed bytestring message signing

Errors should be thrown for invalid content, or messages that would exceed the device's memory limits

An ascii-encoded preview of the message should be printed on the device

Users can be warned or prohibited from passing too length content by wallet clients

The readable prefix should be "Stacks Signed Message", with a similar formatting specification to EIP-712
A new method, sign, should be added to the @zondax/ledger-blockstack package

These are relatively high-level requirements. If more details needed, let us know. We'll follow up with input from the blockchain team.

@kyranjamie can we close this issue?

@jleni The only question I have pertains to the js lib. There haven't been any changes to it. How do we interface with/consume the new text signing method? Do we use the existing sign function exposed? This only accepts type Buffer currently, so, would we first need to convert strings?

@jleni The only question I have pertains to the js lib. There haven't been any changes to it. How do we interface with/consume the new text signing method? Do we use the existing sign function exposed? This only accepts type Buffer currently, so, would we first need to convert string

Yes, You should use the sign function, there are no changes in the current interface as the parser internally will recognize the data as either a transaction or message. of course, the result of the parsing is shown on the screen for validation from the user before signing it the same as it is done for transactions.

closed via #82


[PE-D][Tester B] Delete command does not update the summary panel

Expected output: Deleting a candidate from the list should update the summary panel
Actual output: Deleting a candidate from the list does not update the summary panel
Before deleting:

After deleting:

(Total number of applicants and number reflected under a certain status is not updated)
Steps to replicate:

run the executable jar file

Do not click on any candidate on the Applicants panel yet

run delete INDEX with any valid INDEX

Pay attention to the Overview panel

Additional details:
The summary panel will show the correct numbers again once you run summary command again.
Suggestion: Create an observer (use an observer pattern) and attach it to the overview panel that is activated when a delete command is called

Labels: severity.Medium type.FunctionalityBug
original: PeterHW963/ped#2

duplicated