NeuroPoly student feedback: Thomas Dagonneau

This issue is the global feedback for the mooc from Thomas Dagonneau.
Disclaimer : I'm not a very good english speaker so I might see some typo where there are not or miss some.

Feedback for the ReadMe page :

in 1.1 : maybe add (mOOC) after mini Open Online Course
feels confusing to speak from a book in 1.2 section when you didn't mention anything about a book previously
typo in 1.2 : Quantitative MRI (qMRI) aims to promises precise and reproducible measurements of tissue properties using MRI.
typo in 1.3 : Leveraging the NeuroLibre platform, readers can access fully reproduce the material in this book and allows them to engage with real qMRI data through their web browser.
typo in 1.3 space after the point : their web browser.This

Global feedback : Good first page : defines what is a mooc and the goal of the mooc

Feedback for Introduction to qMRI -> Basics of MRI and qMRI :

in the See also : maybe put the name of the article instead of the year ? At first I didn't get that it was a link to the article and I thought the only link was to Nishimura wikipedia page

Global feedback : Clear

Feedback for Introduction to qMRI -> The million dollar question:

not sure if that's the goal but in 1.1 the MR acronym has not the same feature as MRI to explain its signification
typo 1.1 : sug- gests (maybe it's normal I don't know much about written reports)
Fig1 : would be clearer and easier to read if the conventional MRI was on the right side of the figure since you speak about it before. Else you have to read the figure in the opposite direction to the one suggested by the text. (Else the figure's content is good)
typo in 1.1 : an- other

Global feedback : Loved the bean russian roulette example.  The historical aspect is very interesting !

Feedback for Introduction to qMRI -> A pictorial and historic journey into how MRI works

the figure numbers don't seem to be making reference to any figure in the section
typo in 1.1.1 : However, in reality, we don’t have access to observe NMR effects at such a fine-grained level
1.1.1 : quantum-jitters -> I don't know what it is so maybe defining it or putting a link to the wikipedia would be great (don't know if it was planned since there is no link in this page)
1.1.1 : Schrödinger’s cat -> put the wikipedia link ? Don't know how obvious this illustration is
FIg5 : maybe put it after the text citing it. When you read the page you don't really understand where it's coming from before the text about Bloch

Global feedback : Good ! In the version I reviewed most links where missing and also references but I think it's normal

Feedback for Introduction to qMRI -> Measuring and encoding the MRI signal

there is no under-title like in other sections
in 1.1 I feel like the reader could benefit from seeing the new equation after removing the components = 0 : In this case, the last two terms of the equation vanish, leaving the precessional component of the equation.
numbers of cited equations don't refer to the numbers of the equations
1.2 : The analogy with the defibrillator is not presented the same way as the analogy with beans and dating (in a square highlighted)
Fig4 : I know what you are talking about so I get it but maybe to emphasise what a gradient is you could put a graph under the MRI scanner showing how the value evolves according to z for example ?
typo in  1.4 : ob- servations

Global feedback : Good ! In the version I reviewed most links where missing and also references but I think it's normal

Feedback for Introduction to qMRI -> Two MRI sequences and two qMRI measurements

I feel like a bit of explanation for the difference between T2 and T2* could be appreciated. For now there is only : T2∗  – the effective T2
typo at the end : con- ventional

Global feedback : Good

Feedback for Introduction to qMRI -> Will qMRI take over the world?

the figure numbers don't seem to be making reference to any figure in the section
Fig1 : very nice Fig but I don't find it easy to understand
typo in 1.1 : Figure 2.23 shows that all

qMRI methods share a common methodolog

typo in 1.1 : chal- lenging
1.2 is just a title ?

Global feedback : Good, 1.2 is missing there is only the title.

Feedback for T1 Mapping -> Inversion recovery -> Introduction

typo space after the point : inversion recoverytechnique
the cited figures don't refer to existing figures
change in the way to count figure between the Introduction to qMRI chapter and this one (figure 1 vs figure 2.1)

Global feedback : Good

Feedback for T1 Mapping -> Inversion recovery -> Signal Modelling

typo : 5_T_1  (2 times)
Fig2.3 : maybe choose a default value of T1 so that when you load the page there is a difference between the two curves ?

Global feedback : Good

Feedback for T1 Mapping -> Inversion recovery -> Data Fitting

typo : 5_T_1, 1.5_T_1
only paper without a link :  Barral et al. 2010

Global feedback : Good

Feedback for T1 Mapping -> Inversion recovery -> Benefits and Pitfalls

References formating changed : green arrow

Global feedback : Good

Feedback for T1 Mapping -> Inversion recovery -> Other Saturation-Recovery T1 Mapping techniques
Global feedback : Good

Feedback for T1 Mapping -> Variable Flip Angle -> Introduction
Global feedback : Good

Feedback for T1 Mapping -> Variable Flip Angle -> Signal Modelling

typo : using Bloch simulations (orange) -> it's red
I think the text after Figure 2.9 should be in the legend of the figure ?

Global feedback : Good but not sure about the Fig 2.9 legend

@tomDag25 thanks for the feedback! I will reflect these when it nears minot tweaks phase.

Feedback for T1 Mapping -> Variable Flip Angle -> Data fitting

the link for linear least square doesn't work
no figure 2.12 ?
References with a green arrow
Problem with the order of elements at the end of the page

Global feedback : Good but no figure 2.12

Feedback for T1 Mapping -> Variable Flip Angle -> Benefits and Pitfalls
Global feedback : Good

Feedback for MP2RAGE -> Introduction

Figure 1 -> No link and also no figure 1 ? Probably figure 2.14

Global feedback : good

Feedback for T1 Mapping -> MP2RAGE -> Signal Modelling

in the following part I would keep using markdown for formatting, as a reader it feels strange to change in the middle of the text : no partial Fourier or parallel imaging acceleration), then these values are TA = TI1 - (n/2)TR, TB = TI2 - (TI1 + nTR), and TC = TRMP2RAGE - (TI1 + (n/2)TR), where n is the number of voxels acquired in the 3D phase encode direction varied within each GRE block. The value m{sub}`1z,ss is the steady-state longitudinal magnetization prior to the inversion pulse, and is given by:
typo : From Equations 2.13, 2.14, 2.15, and 2.13

Global feedback : good but there seem to be a problem with the markdown in this page

Feedback for T1 Mapping -> MP2RAGE -> Data Fitting

no link : (e.g. Levenberg-Marquardt)
the whole paragraph is allready in the Introduction : MP2RAGE is an extension of the conventional MPRAGE pulse sequence widely used in clinical studies Haase et al., 1989Mugler & Brookeman, 1990. A simplified version of the MP2RAGE pulse sequence is shown in Figure 2.14. MP2RAGE can be seen as a hybrid between the inversion recovery and VFA pulse sequences: a 180° inversion pulse is used to prepare the magnetization for T1 sensitivity at the beginning of each TRMP2RAGE, and then two images are acquired at different inversion times using gradient recalled echo (GRE) imaging blocks with low flip angles and short repetition times (TR). During a given GRE imaging block, each excitation pulse is followed by a constant in-plane (“y”) phase encode weighting (varied for each TRMP2RAGE), but with different 3D (“z”) phase encoding gradients (varied at each TR). The center of k-space for the 3D phase encoding direction is acquired at the TI for each GRE imaging block. The main motivation for developing the MP2RAGE pulse sequence was to provide a metric similar to MPRAGE, but with self-bias correction of the static (B0) and receive (B1-) magnetic fields, and a first order correction of the transmit magnetic field (B1+). However, because two images at different TIs are acquired (unlike MPRAGE, which only acquires data at a single TI), information about the T1 values can also be inferred, thus making it possible to generate quantitative T1 maps using this data.

Global feedback : good but repeating the paragraph highlighted above feels strange for the reader

Feedback for T1 Mapping -> MP2RAGE -> Benefits and Pitfalls
Global feedback : good

Feedback for T2 Mapping -> Introduction
Global feedback : good

Feedback for T2 Mapping -> T2 mapping vs T2-weighted imaging
Global feedback : good

Feedback for T2 Mapping -> Monoexponential T2 mapping -> Introduction
Global feedback : good

Feedback for T2 Mapping -> Monoexponential T2 mapping -> Signal modelling
Global feedback : good

Feedback for T2 Mapping -> Monoexponential T2 mapping -> Data fitting -> Data fitting
Global feedback : good

Feedback for T2 Mapping -> Monoexponential T2 mapping -> Data fitting -> T2*

eq 3.4 is the same as eq 3.2 and I'm not sure that it is normal
I'm not sure to understand this part : Until now, we have assumed that the transverse signal (Mxy) decays exponentially with the echo time divided by the T2 constant (see Eq. 3.3). However, in practice, other factors such as B0 inhomogeneities can cause a more rapid loss of the transverse signal; this results in a faster transverse decay, which is referred to as T2* relaxation (see Figure 3.1).

The relation between T2 and T2* is described as follows Brown et al., 2014:
1
T
2
∗
1
T
2
+
1
T
2
′
T
2
∗

1

T
2

1

+
T
2
′

1

Global feedback : I don't know much about the subject but this part wasn't so clear to me

Feedback for T2 Mapping -> Monoexponential T2 mapping -> Data fitting -> Noise
Global feedback : Good

Feedback for T2 Mapping -> Monoexponential T2 mapping -> Benefits and Pitfalls

MESE : only SE is highlighted

Global feedback : Good

Feedback for T2 Mapping -> Multiexponential T2 Mapping -> Introduction
Global feedback : Good

Feedback for T2 Mapping -> Multiexponential T2 Mapping -> Signal Modelling

There is a figure 3.7 legend that doesn't exist : same text as the legend from fig 3.4. There is allready a fig 3.4 in Noise for monoexponential T2 mapping

Global feedback : Good

Feedback for T2 Mapping -> Multiexponential T2 Mapping -> Data Fitting

According to the last feedback the number of the fig 3.5 is not the good one

Global feedback : Good

Feedback for T2 Mapping -> Multiexponential T2 Mapping -> Applications -> Myelin water fraction (MWF) imaging
Global feedback : Good

Feedback for T2 Mapping -> Multiexponential T2 Mapping -> Applications -> T2* and quantitative susceptibility mapping (QSM)
Global feedback : Good

Feedback for T2 Mapping -> Multiexponential T2 Mapping -> Applications -> Benefits and pitfalls of multi-exponential T2 mapping
Global feedback : Good but maybe I wouldn't put it in Applications so that it respects the organisation of the other sections

Thanks a lot for the feedback @tomDag25 ! Just wondering if you think you were done providing feedback, or if you think you'll continue at a certain point. No pressure either way!

I was just waiting because you told me that the remaining chapters were maybe not mature enough but yes I will @mathieuboudreau !

Feedback for B1 mapping -> Double Angle Methods->transmit and receive RF field amplitudes

"properties (electromagnetic permittivity ε and permeability μ) of the object being imaged" -> would put the parentheses at the end of the sentence

Global feedback : I didn't really get the explication of the distinction between B1+ and B1-, maybe a scheme could help

Feedback for B1 mapping -> Double Angle Methods -> Double Angle Methods

typo: technique swherein
typo: (Figure 4.1])

Global feedback: good

Feedback for B1 mapping -> Double Angle Methods -> Gradient echo and 180 degree spin echo method

Alternatively, a gradient echo could be used? -> really a question ? If so maybe write it like that: Alternatively, could a gradient echo be used?
typo: Assuming an an refocusing
no latex ? Knowing that alpha = B1 alpha_nominal, B1

Global feedback: Maybe a text to describe what the reader should understand from the figures because I don't find it so intuitive.  Else it's good !

Feedback for B1 mapping -> Double Angle Methods -> Appendix A
Feedback: nice proof but looks maybe a bit complicated for someone that doesn't do math... I don't know to what extent it's obvious that e^ix = cosx + isinx.
Other maybe easier ways to prove that is to compute the derivate of sin(2x)-2cos(x)sin(x), or basically use the identity: sin(a+b) = cos(a) sin(b) + sin(a) cos(b) ?
However that's just a remark because I don't know if someone that doesn't like math would bother to read the proof.

Feedback for B1 mapping -> Actual Flip Angle Imaging -> Introduction

change of formatting with: 1 Actual Flip Angle Imaging (AFI)
Feels strange to come back in the introduction on a subject we are already talking about since a few chapters

Global feedback: this part feels a bit less integrated to the course (come back on things already discussed) and feels more like a paper introduction instead of a course. Else the content on AFI is good !

Feedback for B1 mapping -> Actual Flip Angle Imaging -> Signal Modelling

Bloch equations -> maybe add a ref
I feel like fig4.5 would be better placed if it was after the paragraph where it's cited

Global feedback: good

Feedback for B1 mapping -> Actual Flip Angle Imaging -> Data fitting

Figure 4.10 doesn't display

Global feedback: good
Global feedback: good

Feedback for B1 mapping -> Actual Flip Angle Imaging -> Benefits and Pitfalls

add for the whole AFI section the signification of AFI and DAM abbreviations

Global feedback: good

Feedback for B1 mapping -> Filtering -> Filters and smoothing

in the next section -> i think you can remove it
eq 4.15: should be (fxg)(t) = ...
eq 4.16: maybe put g and not f because in the previous paragraph "f(t) is the signal of interest and g(t) is the kernel"
and is a measure of the width -> add sigma
Figure 4.16: if you want to put Gaussian width I would still put sigma for those more familiar with gaussian functions
add a link or an explanation for Fourier transform

Global feedback: good

Thanks for all this recent feedback - yes the B1 section needs more polish, it's on a to-do list; thanks for catching a lot of these minor issues

Feedback for B1 mapping -> Filtering -> Filters and smoothing

typo: help provide you with

Global feedback: good

Feedback for B1 mapping -> Filtering -> Recommendations, benefits, and pitfalls

typo: this it appears that

Global feedback: good

Feedback for B0 Mapping -> B0 inhomogeneities -> Introduction

typo: B{sub}0

Global feedback: good

Feedback for B0 Mapping -> B0 inhomogeneities -> Sources -> Magnetic Susceptibility

typo: susceptibility ()
no latex: Bz
probably due to typo but in eq 5.2 &chi is not defined

Global feedback: good

Feedback for B0 Mapping -> B0 inhomogeneities -> Effects on signal

not so clear to me what you mean by that: "carrier frequency"
fig 5.5 the animation doesn't work -> tried to launch after trying to connect to Jupyter but it doesn't seem to work cf. screenshot (5minutes after clicking the button )

To do so, a map of the B0 field can be acquired. -> Is it no mandatory ?

Global feedback: good but general warning the jupyter connection is not working on this page !

Feedback for B0 Mapping -> Dual Echo B0 Mapping -> Introduction

B0 mapping estimates the B0 field from the expected field for every voxel -> not clear
they are also vital to quantitative susceptibility mapping (QSM) -> vital for ? (not sure)

Global feedback: good

Feedback for B0 Mapping -> Dual Echo B0 Mapping -> Signal Theory

follows the following equation (not considering transient effects such as eddy currents) in the rotating frame of reference. -> would put the parentheses at the end of the sentence
typo missing gamma: e, is the gyromagnetic

Global feedback: good
Global feedback: good

Feedback for B0 Mapping -> Dual Echo B0 Mapping -> Single frequency population
Global feedback: good

Feedback for B0 Mapping -> Dual Echo B0 Mapping -> Multi frequency population
Global feedback: good

Feedback for B0 Mapping -> Dual Echo B0 Mapping -> Benefits and Pitfalls

advanced B0 mapping section -> put the advanced in the link

Global feedback: good

Feedback for B0 Mapping -> Phase unwrapping -> Introduction

typo: (offset less than )

Global feedback: good, maybe it would be good to help the reader understand to put a little animation what phase unwrapping means: for example a point going around the circle and a curve that is drawn base on the coordinate of the points with a jump when passing from +pi to -pi ?

Feedback for B0 Mapping -> Phase unwrapping -> Temporal unwrapping

(>=2) -> put in markdown
Eq. 1 does not exist
Not sure that's a problem but Fig 5.10 is too wide

Global feedback: good

Feedback for B0 Mapping -> Phase unwrapping -> Spatial unwrapping

Fig 5.12 not displayed

Global feedback: good

Feedback for B0 Mapping -> Phase unwrapping -> Phase unwrapping ambiguity
Global feedback: good

Feedback for B0 Mapping -> Phase unwrapping -> Problematic phase map properties
Global feedback: good

Feedback for B0 Mapping -> Phase unwrapping -> Software

last paragraph: make it as a list since you are listing it will be easier to read

Global feedback: good

Feedback for B0 Mapping -> Advanced B0 Mapping Methods -> Introduction
Global feedback: good

Feedback for B0 Mapping -> Advanced B0 Mapping Methods -> Multi-echo B0 Mapping

typo missing word ?: the phase changes per unit time
typo: δTE should be less than but greater than zero

Global feedback: good

Feedback for B0 Mapping -> Advanced B0 Mapping Methods -> Realtime B0 mapping

EPI abreviation without legend

Global feedback: good

Feedback for B0 Mapping -> Conclusion
Global feedback: good but there is no Conclusion in other parts, I find it nice but maybe try to have the same structure in every chapter

Feedback for Magnetization Transfer Imaging -> Quantitative Magnetization Transfer -> Introduction

typo: of the the

Global feedback: Maybe I didn't pay attention in other pages but I saw in this one that you don't use the same formating when declaring a variable and using it in an eq. Maybe do all in latex. Else good

Feedback for Magnetization Transfer Imaging -> Quantitative Magnetization Transfer -> Introduction

typo: of the the

Global feedback: Maybe I didn't pay attention in other pages but I saw in this one that you don't use the same formating when declaring a variable and using it in an eq. Maybe do all in latex. Else good

Feedback for Magnetization Transfer Imaging -> Quantitative Magnetization Transfer -> Data fitting
Global feedback: Good.

Feedback for Magnetization Transfer Imaging -> Quantitative Magnetization Transfer -> Summary
Global feedback: Good.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Ratio -> Introduction

I find this sentence really long and I misread 70s and 80s at first thinking you were speaking of seconds so maybe not use that: Although the images acquired by clinical MRI machines can only be generated from signal from mobile hydrogen, these do interact with nearby molecules and atoms via the electromagnetic fields they mutually generate, and in the 70s and 80s a cross-relaxation mechanism was discovered that sensitizes mobile protons to nearby targeted semi-solid molecules, such as myelin Edzes & Samulski, 1977Edzes & Samulski, 1978Wolff & Balaban, 1989.
add definition: MS

Global feedback: Good.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Ratio -> Introduction

maybe put a link to this equation: Δx⋅Δp ≥ constant , this whole part is pretty dense...

Global feedback: good.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Ratio -> MTR in practice

two table 2
typo: substantial Figure 6.11 illustrates how

Global feedback: good.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Ratio -> Parting Thoughts

typo: segment -> section

Global feedback: good.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Saturation -> Introduction

has been shown to be: -> eq missing

Global feedback: good.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Saturation -> Theory

typo: into [4]
typo: equation [1].

Global feedback: I feel like it's a bit dense, at least for me that doesn't do much MRI physics, maybe split it in multiple chapters or make a short version with key take aways for people that are not necessary in the field.

Feedback for Magnetization Transfer Imaging -> Magnetization Transfer Saturation -> Simulations
Global feedback:

Global feedback: I found the course really cool ! I learned a lot and I feel like most of the content is not to hard to get. The global improvements I would suggest is standardizing the organisation between different parts: as a reader it can be surprising if not.
Else I think it's really good ! Don't hesitate if you want to rereview some parts or explain more my feedback !

@tomDag25

typo : using Bloch simulations (orange) -> it's red

I dunno, to me it's orange.

Though it is right on the border of red and orange on the colorwheel, when a large section is "painted" with it I clearly would say orange more than red,

Maybe one of us has a slight color blindness? haha

@mathieuboudreau just checked it on my screen and I think it comes from the fact I didn’t zoom in. I agree when zooming on the fig that it’s orange but when looking at it without zooming I thought it was red I don’t know why... Maybe I’m in fact a bit color blind haha


Is there any function from which I can listen the address change in xverse wallet?

Is there any function from which I can listen the address change in xverse wallet?

Hey @mohitgarg05
This is not currently available but it's being worked on and will be released in an upcoming version.

Hey vector,
Thankx for your reply , i want to ask one more thing that is there any way
to impose gas fee on my btcTransfer?
Thank you
Mohit garh
On Thu, 21 Mar 2024 at 12:12 PM, Victor Kirov @.***>
wrote:

Hey @mohitgarg05 https://github.com/mohitgarg05
This is not currently available but it's being worked on and will be
released in an upcoming version.
—
Reply to this email directly, view it on GitHub
https://github.com/secretkeylabs/sats-connect/issues/84#issuecomment-2011320678,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT4BU2PUSMYD7CC5GA4JXTDYZJ6LTAVCNFSM6AAAAABFAWSXN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJRGMZDANRXHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hey Mohit
Not at the moment. We're going to be adding that in as well but it won't be a hard setting, it will be an initial setting and the user can change the fees they want to use.

duplicated with: https://github.com/secretkeylabs/sats-connect/issues/117


fix(hapi): enable getLog()

hapi v15 disabled it by default but i think good uses it so its causing errors (as seen in new relic).

this change was deployed on the 6th and not only did it fix the error rate, it also looks like it reduced memory consumption:
prod:

Date
10-03
10-04
10-05
10-06
10-07
10-08
10-09

Error rate
0.0407%
0.0449%
0.0397%
0.0398%
0
0%
0%

Memory
886 MB
912 MB
902 MB
790 MB
156 MB
222 MB
288 MB

staging:

Date
10-03
10-04
10-05
10-06
10-07
10-08
10-09

Error rate
0%
0%
0.0342%
0%
0%
0%
0%

Memory
802 MB
829 MB
202 MB
322 MB
135 MB
240 MB
346 MB


add sample app url to repo

Added Sample app to the README.

I signed it!

Thanks, can you change the commit message to follow our convention please?

@Dignifiedquire  Changed commit message.

Thanks :octocat:


GLSL matrix row/column major-ness issue

Problem description
Following four CTS tests are failing as a regression,
dEQP-VK.glsl.matrix.div.uniform.mediump_mat2x3_float_vertex
dEQP-VK.glsl.matrix.div.uniform.mediump_mat2x3_float_fragment
dEQP-VK.glsl.matrix.div.uniform.highp_mat2x3_float_vertex
dEQP-VK.glsl.matrix.div.uniform.highp_mat2x3_float_fragment

This regression is from #4069, and it appears to be related to the matrix row/column major-ness.
Note that there are other tests that are similar but passing,
dEQP-VK.glsl.matrix.div.const.mediump_mat2x3_float_vertex // PASSING
dEQP-VK.glsl.matrix.div.const.mediump_mat2x3_float_fragment // PASSING
dEQP-VK.glsl.matrix.div.const.highp_mat2x3_float_vertex // PASSING
dEQP-VK.glsl.matrix.div.const.highp_mat2x3_float_fragment // PASSING

The issue seems to be limited to the case where the matrix data is fed via uniform.
Goal
It appears to be an issue with matrix-major-ness.
We may want to use RowMajor decorator on matrices, but it is unclear why the test is failing.
Repro step
The following shaders are used for the CTS test,
// Vertex shader
#version 310 es
layout(location = 0) in highp vec4 a_position;
layout(location = 0) out mediump vec4 v_color;
layout(std140, set = 0, binding = 0) uniform buffer0 { mediump mat2x3 u_in0; };
layout(std140, set = 0, binding = 1) uniform buffer1 { mediump float u_in1; };

void main (void)
{
        gl_Position = a_position;
        mediump mat2x3 res = u_in0 / u_in1;
        v_color = vec4(res[0] + res[1], 1.0);
}
// Fragment shader
#version 310 es
layout(location = 0) out mediump vec4 dEQP_FragColor;
layout(location = 0) in mediump vec4 v_color;

void main (void)
{
        dEQP_FragColor = v_color;
}

The test expect a full white image, but we generate a yellow image.
I cannot figure out what values are used for u_in0 and u_in1.
But it is probably using the same value as the test below.
As a comparison, the passing test, "dEQP-VK.glsl.matrix.div.const.mediump_mat2x3_float_vertex", uses the following vertex shader.
#version 310 es
layout(location = 0) in highp vec4 a_position;
layout(location = 0) out mediump vec4 v_color;
const mediump mat2x3 in0 = mat2x3(-0.6, -0.7, -0.2, -0.1, -1.2, 0.0);
layout(std140, set = 0, binding = 0) uniform buffer1 { mediump float u_in1; };

void main (void)
{
        gl_Position = a_position;
        mediump mat2x3 res = in0 / u_in1;
        v_color = vec4(res[0] + res[1], 1.0);
}

I think this is just because the CTS code isn't properly setting matrix majorness when triggering a slang compile. We should specify column major in the slang api when compiling these shaders.

I see. That sounds simple.

I had a chance to try out the column-major setting on VK-CTS.
But the tests were still failing.
diff --git a/external/vulkancts/framework/vulkan/vkShaderToSpirV.cpp b/external/vulkancts/framework/vulkan/vkShaderToSpirV.cpp
index 07ad78809..96d4bbf1d 100644
--- a/external/vulkancts/framework/vulkan/vkShaderToSpirV.cpp
+++ b/external/vulkancts/framework/vulkan/vkShaderToSpirV.cpp
@@ -1302,6 +1302,7 @@ namespace vk

                        sessionDesc.targets = &targetDesc;
                        sessionDesc.targetCount = 1;
+                       sessionDesc.defaultMatrixLayoutMode = SLANG_MATRIX_LAYOUT_COLUMN_MAJOR;

                        for (int shaderType = 0; shaderType < glu::SHADERTYPE_LAST; shaderType++)
                        {

There might be something else going on.


MBAM 2.5 SQL Server SP Supported Levels

Hi
Please could you confirm whether SQL Server 2014 SP3 is supported to host MBAM 2.5  SP1 databases.
The date stamp on the supported configurations page indicates that it was last updated prior to the release of SQL Server 2014 SP3 hence asking for clarification:
https://docs.microsoft.com/en-us/microsoft-desktop-optimization-pack/mbam-v25/mbam-25-supported-configurations
Thanks
Rich

@officedocsbot assign @jvsam

Hi @richhudson, thank you for taking the time to open this issue. The content of this documentation remains up-to-date, hence, only Microsoft SQL Server 2014 SP1 & SP2 are supported for the MBAM Server features. If you are using Windows 10, we would definitely love to hear your feedback so I encourage you to submit your feature suggestions and ideas via User Voice hub. Your feedback goes directly to the Microsoft engineering teams responsible for building features and products for Windows 10.
Also, if you have issues with MBAM, I would recommend asking question on the TechNet Forum.
Thank you for engaging with the community!

Hi again @richhudson, we haven't heard or received any updates from you. We are hoping that the information we shared helped. We will now close this issue, however, feel free to re-open if you have suggestions or ideas to improve the quality of this documentation.
Thanks for being part of the Microsoft Docs community!

@officedocsbot close


Tell Composer the JSON extension is required

As per #1490, this asks Composer to check whether the JSON extension is used. It seems to be used in the Phinx core to read its own composer.json file.
Not all PHP installations - particularly Alpine - can guarantee that this extension is loaded.

Shall we proceed with this? @lorenzo

Thanks @MasterOdin, good work :+1:


Adding disease_types to data_sources.json

In this patch I have added a new key "disease_types" (type string[]) to all data sources. Currently, only data sources related to .ddls = "Precision Medicine and Diagnostics" have values, all others are added as empty arrays.
These were added as a custom tag that we want to use for filters on the Precision Medicine Portal, with @natashiab (Natashia Benzian Olsson) selecting the tags.
I have tested the new JSON in our application and it seems to be working correctly as far as I can tell.

Precision medicine disease_type tags added. Not used on main Platform site for filtering, not affecting utility of entries on platform


Different Ansys version are not opening

Information

Qiskit Metal version:
Python version:
Operating system:

What is the current behavior?
When using more recent versions of Ansys, Metal doesn't connect to it.
Steps to reproduce the problem
What is the expected behavior?
Suggested solutions
Workaround is to start Ansys before trying to connect to it.
Since the present version of Metal uses pyepr to connect to Ansys, the fix to this would require working with pyepr also.

Conversation in slack
https://qiskit.slack.com/archives/C01R8KP5WP7/p1645803131847369

Some notes to help person helping to fix this.
connect_ansys() is used in initate_renderer
_initiate_renderer is called by start() in renderer_base.py
Seems like the new scenario does not allow user to select the Ansys version.

Notes for earlier version of code to connect to ansys.  These features probably bneed to be edited in the more recent paradigm.
path_var (str): Name of the OS environment variable that contains the path to the
Ansys executable. Only used when path=None.
Defaults to 'ANSYSEM_ROOT202' (Ansys ver. 2020 R2)

https://github.com/Qiskit/qiskit-metal/blob/main/tutorials/4 Analysis/B. Advanced - Direct use of the renderers/4.11 Analyze and tune a transmon.ipynb
https://files.slack.com/files-pri/T7RSPHKK2-F034N1WL1T7/image.png

We plan to replace the Ansys renderer using Windows Comm, with pyaedt Ansys.  This will become a sunset issue.


Yale Connect WiFi Bridge compatible?

Describe Your Problem:
Hi, I’m looking to purchase the Yale Connect WiFi bridge as it is currently on a 50% discount and hence get my Conexis L1 lock into HomeKit via homebridge. I was suggested to look at your plug-in as it may be compatible given the similarities between August and Yale. Do you know / think it would work?
Thank you very much!

I got it connected to Yale Doorman V2N + Access module + Wifi-bridge. Sadly it didn't work reliably. Got some errors after couple unlocks and plugin crashed.

I ended up doing this all via Home Assistant and the August plugin over there. Worked well for me. Just in case anyone else wants to do it.

I ended up doing this all via Home Assistant and the August plugin over there. Worked well for me. Just in case anyone else wants to do it.

You got the lock in Apple Home app? How did you do it exactly?

I ended up doing this all via Home Assistant and the August plugin over there. Worked well for me. Just in case anyone else wants to do it.

You got the lock in Apple Home app? How did you do it exactly?

Yeah. Just added it to HA via the August app, and then added a HomeKit  service in HA in Accessory mode.

I ended up doing this all via Home Assistant and the August plugin over there. Worked well for me. Just in case anyone else wants to do it.

You got the lock in Apple Home app? How did you do it exactly?

Yeah. Just added it to HA via the August app, and then added a HomeKit service in HA in Accessory mode.

Ok, thanks for the info. Have to consider installing HA just for that August plugin.


Is this package dead?

No, this package is working fine.

Not mean to be rude, but that doesn't really answer my question. We're facing some hard choices in our project and we were considering using this package. However, lack of any activity whatsoever is very repulsant. I think you could agree it wouldn't be wise to depend on a package which is abandonded — in case any bugs or vulnerabilities appear.
There are 4 open issues and 6 pull requests, some of them for a few years now. Is this package under any development at all? Are the bugs being fixed? Is the FSA compliance going to be merged any time soon?
Thank you.

Wonderful occasion to step-in and contribute to the (open-source) project, since you’re using it!

@albinotonnina I'm literally not using it yet. And I would love to contribute, however at the moment there are 6 open PRs ignored for years so chances are mine would be ignored just as well :) Cheer 👍


Compile failure on Manjaro Linux

Template

Version of vulkano: 0.19
OS: Manjaro Linux
GPU (the selected PhysicalDevice): NVIDIA Geforce GTX 1650 Super
GPU Driver: NVIDIA 460.27.04
Upload of a reasonably minimal complete main.rs file that demonstrates the issue: Not relavent

Issue
Whenever I try and use vulkano I just get these error from cargo:
   Compiling vulkano v0.19.0
error[E0609]: no field `shaderf3264` on type `PhysicalDeviceFeatures`
   --> /home/protobit/.cargo/registry/src/github.com-1ecc6299db9ec823/vulkano-0.19.0/src/features.rs:459:28
    |
459 |     core { shader_f3264 => shaderf3264 },
    |                            ^^^^^^^^^^^ help: a field with a similar name exists: `shaderInt64`

error[E0308]: mismatched types
    --> /home/protobit/.cargo/registry/src/github.com-1ecc6299db9ec823/vulkano-0.19.0/src/command_buffer/sys.rs:1455:38
     |
1455 |         vk.CmdSetBlendConstants(cmd, constants); // TODO: correct to pass array?
     |                                      ^^^^^^^^^
     |                                      |
     |                                      expected `&[f32; 4]`, found array `[f32; 4]`
     |                                      help: consider borrowing here: `&constants`

error: aborting due to 2 previous errors

I'm unsure if it something wrong with my setup but I followed the setup in the README so idk what's wrong

Hi, I encountered the same issue and solved it by upgrading the vulkan version to 0.20.0 and downgrading the Vulkan SDK from 1.2 to 1.1

Hi, I encountered the same issue and solved it by upgrading the vulkan version to 0.20.0 and downgrading the Vulkan SDK from 1.2 to 1.1

@ProtoByter, @caandewiel Probably related to f3264 naming issue fixed in version 0.20.0: https://github.com/vulkano-rs/vulkano/commit/620b829bdb782bc15db7daf1b8c537ebcd1311c5#diff-d2ec10f741a852314cc59ac3d609fdfdf58f63e57b20885b32c7e83579dc92e4R1172 So, probably you need just to upgrade to the latest Vulkano.
LMK if it works. I would also like to know if it works with SDK 1.2. We don't yet fully support version 1.2, but I suppose it shouldn't lead to failures like this at least.

@ProtoByter, @caandewiel Probably related to f3264 naming issue fixed in version 0.20.0: https://github.com/vulkano-rs/vulkano/commit/620b829bdb782bc15db7daf1b8c537ebcd1311c5#diff-d2ec10f741a852314cc59ac3d609fdfdf58f63e57b20885b32c7e83579dc92e4R1172 So, probably you need just to upgrade to the latest Vulkano.
LMK if it works. I would also like to know if it works with SDK 1.2. We don't yet fully support version 1.2, but I suppose it shouldn't lead to failures like this at least.

Just tested it, it does compile when I set the dependency of vulkano to:
vulkano = {git = "https://github.com/vulkano-rs/vulkano.git"}

Just tested it, it does compile when I set the dependency of vulkano to:
vulkano = {git = "https://github.com/vulkano-rs/vulkano.git"}

It seems that vk-sys 0.5.3 contains a breaking change by renaming shaderf3264 to shaderFloat64. This breaks semver. I believe it should've been released as 0.6 instead. Should vk-sys 0.5.3 be yanked?

It seems that vk-sys 0.5.3 contains a breaking change by renaming shaderf3264 to shaderFloat64. This breaks semver. I believe it should've been released as 0.6 instead. Should vk-sys 0.5.3 be yanked?

@alvinhochun Good point. Then we will have to yank other subproject too :/ ok, I will work on it, sorry for the issue.

@alvinhochun Good point. Then we will have to yank other subproject too :/ ok, I will work on it, sorry for the issue.


HTTP: Fix mp4/flv file size exceed 2GB download error

背景：#2780 #2781 ，http服务分发超过2GB大文件时(如flv、mp4)，会出现连接超时或者overflow日志
原因：超过2GB后，int类型无法完整表示文件大小

Codecov Report

Merging #2809 (b9757c4) into 4.0release (3188c77) will increase coverage by 0.93%.
The diff coverage is 65.38%.

@@              Coverage Diff               @@
##           4.0release    #2809      +/-   ##
==============================================
+ Coverage       60.31%   61.25%   +0.93%
==============================================
  Files             121      121
  Lines           50977    53047    +2070
==============================================
+ Hits            30746    32493    +1747
- Misses          20231    20554     +323

Impacted Files
Coverage Δ

trunk/src/protocol/srs_http_stack.hpp
100.00% <ø> (ø)

trunk/src/protocol/srs_service_http_conn.cpp
94.37% <0.00%> (ø)

trunk/src/app/srs_app_http_static.cpp
45.79% <55.55%> (ø)

trunk/src/protocol/srs_http_stack.cpp
92.83% <75.00%> (ø)

trunk/src/kernel/srs_kernel_mp4.hpp
100.00% <0.00%> (ø)

trunk/src/app/srs_app_config.cpp
68.04% <0.00%> (+1.95%)
:arrow_up:

trunk/src/kernel/srs_kernel_mp4.cpp
90.62% <0.00%> (+2.00%)
:arrow_up:

Continue to review full report at Codecov.

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

NICE 👍


New .csproj cannot reference GAC assemblies

You can no longer reference some GAC assemblies without specifying a HintPath, at which point there isn't much point in using the GAC at all.
Class1.cs:
using System;
using Microsoft.SolverFoundation.Services;

public class Class1
{
    public static readonly Type X = typeof(ISolver);
}

Old .csproj, working:
<Project ToolsVersion="15.0">
  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" />
  <PropertyGroup>
    <OutputType>Library</OutputType>
    <TargetFrameworkVersion>v4.6.2</TargetFrameworkVersion>
    <OutputPath>bin\$(Configuration)\</OutputPath>
  </PropertyGroup>
  <ItemGroup>
    <Reference Include="Microsoft.Solver.Foundation" />
  </ItemGroup>
  <ItemGroup>
    <Compile Include="*.cs" />
  </ItemGroup>
  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
</Project>

New .csproj fails to find the reference:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net462</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <Reference Include="Microsoft.Solver.Foundation" />
  </ItemGroup>

</Project>

This also does not work: <Reference Include="Microsoft.Solver.Foundation, Version=<IP_ADDRESS>89, Culture=neutral, PublicKeyToken=31bf3856ad364e35" />.
Why is this broken?

I'm also witnessing a weird phenomenon where some GAC assemblies can be referenced and the build succeeds, but Visual Studio shows build warnings and reference node warning icons saying the reference could not be resolved and if you open any of the .cs files, the editor creates a whole bunch of errors as though the reference was not referenced at all. The weird part is, it builds fine. For example, <Reference Include="DevExpress.Data.v16.2, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b88d1754d700e49a, processorArchitecture=MSIL">.
But the moment I add a <SpecificVersion>false</SpecificVersion> or a true` subelement, it goes back to not building at all.
I would really like to understand this phenomenon.
I don't want to use HintPaths because of the chance something will be different across developer machines and the build server, and why should I have to add ugly GAC paths?

As it turns out, HintPath with SpecificVersion false is not a useful workaround because the HintPath does not exist on different machines.

Adding this to the bottom of the new csproj and reopening the solution fixes all problems:
  <!-- Workaround for https://github.com/dotnet/roslyn-project-system/issues/1739 -->
  <PropertyGroup>
    <AssemblySearchPaths>$(AssemblySearchPaths);{GAC}</AssemblySearchPaths>
  </PropertyGroup>

I can repro this.  I'll need to look into what has changed regarding the GAC.
This should work in the mean time.
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net462</TargetFramework>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.Solver.Foundation" Version="3.1.0" />
  </ItemGroup>

</Project>

This issue was moved to dotnet/sdk#987


Move byte-checks of Keccak to Syscall side and include lookups for length bytes

Cherry-picked from https://github.com/o1-labs/proof-systems/pull/2276/

Closing to keep https://github.com/o1-labs/proof-systems/pull/2276. Was simpler to use git merge than cherry-picking.


Please add model support for gpt-4o-mini

GPT-4o-mini is now available via API and is by far the most cost-efficient model. All that's needed is to add a new enum key linked to the string "gpt-4o-mini" for the dynamically retargeting model, and another for "gpt-4o-mini-2024-07-18" for the version that will not dynamically update as new models are released.
For App authors, this model is extremely desirable to start working with ASAP.
Here's a link to the OpenAI reference document section:
https://platform.openai.com/docs/models/gpt-4o-mini
And the relevant announcement:
https://openai.com/index/gpt-4o-mini-advancing-cost-efficient-intelligence/
Thanks in advance.

https://github.com/jamesrochabrun/SwiftOpenAI/releases/tag/v3.5
Done!

Thank you! I was going to submit a PR but you beat me to it!


Allow custom SageMaker Estimator arguments

The current implementation for SageMaker only allows a limited set of arguments, however the Estimator and HuggingFace Estimator support many additional arguments. With the current config wizard, this would be very hard to implement, hence I added a new field to SageMakerConfig to allow overrides to be set.
This e.g. allows the usage of Spot Instances.

Hello @dennisbappert, how will user specify additional_args as I don't see it being part of the accelerate config questionarrie ?

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

I purposefully kept it out of the questionnaire, as it might be hard to enter in an interactive console session. It is mentioned in the documentation and for more advanced use-cases. I can add a hint in the questionnaire to guide the user towards the documentation.
Do you think it will be valuable as an interactive input?

Yes it has to be added directly. I modified the docs to include an example for Spot Instances, let me make that a bit more clear in the docs.

cc @philschmid

see ^^^

I'm working on some more configuration right now: adding custom docker container support (Dockerfile in the repository), enhancing the documentation to cover local mode (which works fine, would just be nice as part of the documentation).

You can merge this one, I will create additional PRs for the other changes.


Add Romanian blockpage

DNS for gambling sites points to a page on <IP_ADDRESS> (which also sends a self-signed cert if you hit it with https).

https://github.com/TheTorProject/ooni-pipeline/pull/67 should close this issue.


Switching from Angular 4 to 5 causing "Uncaught Error: No NgModule metadata found"

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @TipeIO

I'm submitting a ...
[X] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Bug: I am porting my Angular 4 application to Angular 5 and although the application successfully builds the application does not load.

What is the current behavior?
I am getting the following error when I run my application: Uncaught Error: No NgModule metadata found for 'AppModule'.

Full Stack Trace:

What is the expected behavior?

Application should run. I have tried researching for a possible solution - and I have found plenty, but none have helped. I have tried:

Deleting the node_modules folder and reinstalling everything
Some suggested that simply making a change in the AppModule and saving solves the problem. However after the update whenever I save a file I'm always getting this error: ERROR in Debug Failure. False expression: Host should not return a redirect source file from getSourceFile
I've went through all my project dependencies and they are up to date

I was thinking that there's some error when bootstrapping the application but I don't know what it could be.

 Please for questions like this use https://stackoverflow.com.

Okay understood, I've created this post  on stackoverflow - maybe you can help. I've been trying to fix this for the past 3 days and I'm going crazy.


Unable to build ios app with Apple watch extension : infer

I am trying to build and check possible crashes and errors for iOS app with Apple watch extension. The command I am using is like below,
infer -- xcodebuild -target TestApp -configuration Debug -sdk iphonesimulator
This is not getting build with the following error,
Check dependencies
target specifies product type 'com.apple.product-type.watchkit2-extension', but there's no such product type for the 'iphonesimulator' platform
Please help me in resolving this issue.
Thanks in advance!

Hello @sulthana,
The support for the analysis of the Watch extension is not yet implemented, we are working on it.
Thanks.

Any progress on that?

It seems that the version of clang used in Infer since 0.8.1 supports the flags used to compile Watch apps. Please try the latest version of Infer on your project and re-open this issue if it still doesn't work. Thanks!


Translate Newsletter#69 into Japanese

@Ichiro0219 can you allow edits from maintainers? I think I would be able to push commits (if needed) if you set that.
In this case there is an extra space which is causing the build to fail.
See screenshot here for example:
https://user-images.githubusercontent.com/6248932/61183973-6989b780-a648-11e9-86b3-b17089acd1e4.png

@Ichiro0219 can you allow edits from maintainers? I think I would be able to push commits (if needed) if you set that.
In this case there is an extra space which is causing the build to fail.
See screenshot here for example:
https://user-images.githubusercontent.com/6248932/61183973-6989b780-a648-11e9-86b3-b17089acd1e4.png
@bitschmidty
I checked the box "Allow edits from maintainers."
Can you edit it?

@Ichiro0219 that did not work for me.
I have instead submitted a PR to your branch here:
https://github.com/cryptogarageinc/bitcoinops.github.io/pull/1

Can you squash the commits down into a single commit before merge please?

Can you squash the commits down into a single commit before merge please?
I ’m sorry to trouble you.I squashed the commits.
Could you check it?

Can you squash the commits down into a single commit before merge please?
I ’m sorry to trouble you.I squashed the commits.
Could you check it?

LGTM @Ichiro0219
tACK fec12cc4d35816486c1384eb643c40c367f4671f


Improve lexing by storing all tokens in a flat fashion

Currently, in hash-parser, the parser stores trees of tokens (by nested braces/parens) as separate vectors. This is wasteful in terms of allocation. It would be better if they were all stored in the same vector, and each opening brace token had a position offset stored within it to the closing brace. This way, the parsing can still be parallelised while not having to resort to multiple vectors.

Am I correct in assuming that the initial changes are on the ast-tests branch?

After some investigation, it's noted that full flattening yields no performance gain to rather using partial token trees. Therefore, keeping with the same token tree API is more ergonomic in the codebase and therefore further efforts on flattening won't be carried out.

After some investigation, it's noted that full flattening yields no performance gain to rather using partial token trees. Therefore, keeping with the same token tree API is more ergonomic in the codebase and therefore further efforts on flattening won't be carried out.

This issue was resolved by https://github.com/hash-org/lang/pull/93

This issue was resolved by https://github.com/hash-org/lang/pull/93


[Feature]: Improve Contact Form Validation and UX Enhancements

Is your feature request related to a problem? Please describe.
The current contact form lacks robust validation, clear error messaging, and feedback while the form is being submitted. This leads to confusion for users, especially when fields are left blank or filled out incorrectly. Additionally, there is no loading indicator to show that the form is being processed, which may cause users to assume the form submission has failed.
Describe the solution you'd like
Add specific error messages for missing fields (name, email, message) to provide clear feedback to the user.
Implement email validation to ensure proper email formatting.
Introduce a loading state while the form is being submitted to improve user experience.
Refactor modal usage to avoid duplicate rendering and ensure it closes automatically after a timeout.
Enhance responsiveness and accessibility of the form (e.g., aria-labels for screen readers).
Describe alternatives you've considered
Keeping the form as-is with basic validation, but this would not improve the user experience and might lead to more form submission errors or confusion.
Using a different library for handling form validation (e.g., Formik), but this would increase dependencies unnecessarily for a simple form.
Additional context
These changes will ensure a more user-friendly experience when filling out and submitting the contact form.

@pradeeptosarkar Please check it out i made a PR for this.

closing as resolved


Object field access is reported as unused (ast)

When using dependency_tracking_method = "ast" object field access does not mark the dependency as used.
rules_scala version: 97997c4d7335d655c0737364a1648a03a083811f
Example repo:
https://github.com/arjantop/rules_scala_bug
Output:
INFO: Analyzed target //:test (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
ERROR: /Users/arjan/code/rules_scala_bug/BUILD:22:1: scala //:test failed (Exit 1)
error: Target '@maven//:com_twitter_finagle_base_http_2_11' is specified as a dependency to //:test but isn't used, please remove it from the deps.
You can use the following buildozer command:
buildozer 'remove deps @maven//:com_twitter_finagle_base_http_2_11' //:test
one error found
one error found
java.lang.RuntimeException: Build failed
	at io.bazel.rulesscala.scalac.ScalacProcessor.compileScalaSources(ScalacProcessor.java:256)
	at io.bazel.rulesscala.scalac.ScalacProcessor.processRequest(ScalacProcessor.java:76)
	at io.bazel.rulesscala.worker.GenericWorker.runPersistentWorker(GenericWorker.java:45)
	at io.bazel.rulesscala.worker.GenericWorker.run(GenericWorker.java:111)
	at io.bazel.rulesscala.scalac.ScalaCInvoker.main(ScalaCInvoker.java:41)
Target //:test failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 0.548s, Critical Path: 0.18s
INFO: 0 processes.

If so can you try with version > 2.12.8?

@ittaiz Yes, it works with 2.12.10. Its a constant.
https://github.com/twitter/finagle/blob/develop/finagle-base-http/src/main/scala/com/twitter/finagle/http/Fields.scala

I’ll wait for @Jamie5 before I close but I think we just won’t support it

Yeah this would be at least quite difficult to support pre 2.12.4. Because constants are inlined in the AST and only in 2.12.4 was the original expression made available.
So unused_dependency_checker_ignored_targets is probably your friend until you can move off pre-2.12.4.

Thanks, that makes sense. Either way we are close to migrating to 2.12.x. Will try with the scala 2.12.x before reporting a bug next time.


Creative Commons License for binary-search dependency

Hi.
One of the dependencies for angular-gettext-tools is binary-search Which has a CC0-1.0. Given  the license restrictions for that dependency, could there be any legal implication if I use this library ?

@crisefd Do you have any background on those restrictions?

I'm no lawyer, but if anything I'd think that a public domain license imposes even less restrictions than MIT.

Hi there! I'm not a lawyer either, but my understanding is backed by this website:
https://tldrlegal.com/license/creative-commons-cc0-1.0-universal
Seems fine to use it as a dependency.


Fixes error in cirs_to_itrs

Fixes #15

Yes, there is definitely a bug, but the fix is not the right one, and also does not address the same bug in related other calls. I have committed and pushed a different fix to this issue to the v1.0.1 branch.


You guys did very good job...i want to execute this project but i got error while executing..please can you help me? contact me on<EMAIL_ADDRESS>

Hello,
Can you please mention the exact issue?

I have installed all libraries.but following error occurrs
On Sat, 28 Oct, 2023, 3:50 am Atharva Jadhav, @.***>
wrote:

Hello,
Can you please mention the exact issue?
—
Reply to this email directly, view it on GitHub
https://github.com/online5/FinalYearBEProject/issues/2#issuecomment-1783570661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRY76WOY5LAWOGM25UIDZDYBQXR7AVCNFSM6AAAAAA6J62S6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGU3TANRWGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hii...I want your help to execute this project...I am ready to pay...please
contact me :<PHONE_NUMBER>
On Sat, 28 Oct, 2023, 8:27 am Pramod Said, @.***> wrote:

I have installed all libraries.but following error occurrs
On Sat, 28 Oct, 2023, 3:50 am Atharva Jadhav, @.***>
wrote:

Hello,
Can you please mention the exact issue?
—
Reply to this email directly, view it on GitHub
https://github.com/online5/FinalYearBEProject/issues/2#issuecomment-1783570661,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRY76WOY5LAWOGM25UIDZDYBQXR7AVCNFSM6AAAAAA6J62S6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOBTGU3TANRWGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi,
As you can already notice, the project is MIT licensed.
For your error, kindly post the error messages that you get. It is only possible to help you if you share the error with us!

This is my issue..please check attached photo
On Mon, 6 Nov, 2023, 5:09 pm Atharva Jadhav, @.***>
wrote:

Hi,
As you can already notice, the project is MIT licensed.
For your error, kindly post the error messages that you get. It is only
possible to help you if you share the error with us!
—
Reply to this email directly, view it on GitHub
https://github.com/online5/FinalYearBEProject/issues/2#issuecomment-1794628965,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMRY76T6UFRQFEJYMBZJ3ODYDDD6DAVCNFSM6AAAAAA6J62S6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTOOJUGYZDQOJWGU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Logged elapsed time is wrong

This is what I get when I log the time for a request, and then Hosting is also logging the same time:
verb: Orchard.Hosting.OrchardContainerMiddleware[0]
      Request took 42ms
info: Microsoft.AspNet.Hosting.Internal.HostingEngine[2]
      Request finished in 0.0047ms 200 text/html; charset=utf-8

Something in how this TimeSpan is constructed must be wrong.

@sebastienros I took a look at our logging and I don't see an obvious error. Do you have a project available for me to repro this and take a look at why there is a discrepancy between the logged times?

The issue comes from the fact that the tick counts are base on Environment.TickCount but displayed using  Timespan instance. The tick counts don't represent the same thing for both, with a difference of about 10000. The solution is to use DateTime.Now.Ticks instead on Environment.Ticks as you can't assume that the system timer will always return milliseconds (which seems to be the case on my system).
Here is some code showing the difference:
using System;

public class Program
{
	public static void Main()
	{
		var startEnvironmentTicks = Environment.TickCount;
		var startDateTimeTicks = DateTime.Now.Ticks;

		System.Threading.Thread.Sleep(50);

		Console.WriteLine("Environment: {0}", Environment.TickCount - startEnvironmentTicks);
		Console.WriteLine("DateTime: {0}", DateTime.Now.Ticks - startDateTimeTicks);
	}
}

Which results in:
Environment: 79
DateTime: 781292

It's also mentioned in this help page under the remarks section: https://msdn.microsoft.com/en-us/library/system.environment.tickcount(v=vs.110).aspx

We will need to find a substitute for timing requests. DateTime.Now is too slow since it's a system call.

This gist shows the perf differences between different techniques:
https://gist.github.com/sebastienros/40981268e057eb9235f3
For 1 million calls
Environment.TickCount: 15ms
DateTime.UtcNow.Ticks: 41ms
DateTime.Now.Ticks: 135ms
Not sure the way I profiled is right but I assume it should help deciding.

The constructor used here: accepts 100-ns ticks:
https://github.com/aspnet/Hosting/blob/f3384d97cb7bba9b9f2d33268bad78e54714395a/src/Microsoft.AspNet.Hosting/Internal/HostingLoggerExtensions.cs#L35
While the value from Environment.TickCount are millisecond ticks.

Completed in #485, Thanks @tmds!


Module crons may overlap if cron_max_run_time is set too low

There are a lot of ways crons could behave strangely if the last longer than cron_max_run_time.  Perhaps we should set our own cron_max_run_time value in the framework, instead of letting people specify them in config.json.  Also, refactoring REDCap's cron runner to handle start date setting and checking differently could go a long way.  Perhaps we should get the next module cron to run AFTER each module cron finishes, instead of getting them all beforehand.  On the every minute cron, this could prevent any more crons from running after the minute mark has been passed.

If one cron runs too long, it's run time is rolled into the calculation used to check cron_max_run_time for all other crons that run in the same process.  This could cause all kinds of weird behaviors with lower cron_max_run_time values...


fix(SearchSelect): prevent losing focus when clicking clear/selecting option

I'm merging this, hope that's ok!


Improving the form tab

This PR adds functionality to the Form tab, including:

Designed to look more like a table with shading and full-size text boxes
A "New table" button that allows you to drag out the size of your table
Toggle delete buttons
Rows and columns are highlighted when deleting
Form (and all of table-magic) is now responsive. Cells are resized and scrollbars added when things start getting tiny.

This also removes the duplicate row/column button. I can't say I ever used it. Let me know if this is missed and I can add it back.
:cat:

@francisfuzz
Duplicate of #


TransferManager. uploadDirectory not uploading empty directories.

I'm trying to upload a list of directories from my local to my COS bucket using TransferManager.
I noticed that empty directories are not getting uploaded, just directories with files inside are being uploaded to my COS bucket.
I'm using the below method with includeSubdirectories = true.
uploadDirectory(String bucketName, String virtualDirectoryKeyPrefix, File directory, boolean includeSubdirectories)
How to get all directories uploaded, including the empty ones?
Thanks.

Buckets and objects are the primary resources, and objects are stored in buckets. COS has a flat structure instead of a hierarchy like you would see in a file system. The folder concept you see in the console is a means of grouping objects. The console does this by using a shared name prefix for the grouped objects. In other words, the grouped objects have names that begin with a common string. This common string, or shared prefix, is the folder name.
Coming to TransferManager, it checks for the files in a directory and if there are none, it can't be uploaded. Non empty directory can be uploaded


feat: support sharding distSQL parse to AST

Type of change:

[ ] Bugfix
[x] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

reference issue: #270
Pre-submission checklist:

[ ] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?

Codecov Report

Merging #392 (d4a192a) into main (e52f093) will decrease coverage by 6.79%.
The diff coverage is 3.99%.

:exclamation: Current head d4a192a differs from pull request most recent head c98d05a. Consider uploading reports for the commit c98d05a to get more accurate results

@@            Coverage Diff             @@
##             main     #392      +/-   ##
==========================================
- Coverage   47.34%   40.55%   -6.79%
==========================================
  Files          48       51       +3
  Lines        5264     6157     +893
==========================================
+ Hits         2492     2497       +5
- Misses       2547     3435     +888
  Partials      225      225

Impacted Files
Coverage Δ

pitr/cli/internal/cmd/backup.go
73.47% <0.00%> (ø)

...tor/cmd/shardingsphere-operator/manager/manager.go
0.00% <ø> (ø)

...perator/pkg/controllers/compute_node_controller.go
0.00% <0.00%> (ø)

...phere-operator/pkg/controllers/proxy_controller.go
0.00% <ø> (ø)

...operator/pkg/controllers/proxyconfig_controller.go
0.00% <ø> (ø)

...pkg/controllers/shardingsphere_chaos_controller.go
46.46% <ø> (ø)

...sphere-operator/pkg/distsql/visitor/encrypt_rdl.go
48.76% <ø> (ø)

...ingsphere-operator/pkg/distsql/visitor/mask_rdl.go
0.00% <ø> (ø)

...or/pkg/distsql/visitor/read_write_splitting_rdl.go
0.00% <0.00%> (ø)

...gsphere-operator/pkg/distsql/visitor/shadow_rdl.go
0.00% <0.00%> (ø)

... and 7 more

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


CI canary for binary size

CI job should build an example application against the latest SDK and measure size on linux & mac os.

Could be integrated into  #956

We should also decide what we want to do when necessary changes would increase the binary size beyond what we consider acceptable.


Child deletion issue in many-to-one form

Hi,
I am experiencing a very problematic issue on a form submission involving a many-to-one relationship:
Here is what I want to achieve:
I have a company with users attached to it. I want to be able to add / update / delete users in the company form.
Schema:
<database name="default" namespace="Acme\Bundle\BlogBundle\Model">

        <column name="id" type="integer" required="true" primaryKey="true" autoIncrement="true" />
        <column name="name" type="varchar" required="false" primaryString="true" />

        <column name="id" type="integer" required="true" primaryKey="true" autoIncrement="true" />
        <column name="company_id" type="integer" required="true" />
        <column name="firstname" type="varchar" required="false" primaryString="true" />

        <foreign-key foreignTable="company" onDelete="CASCADE" onUpdate="CASCADE">
            <reference local="company_id" foreign="id"/>
        </foreign-key>

</database>

Controller:
...
public function companySaveAction(Request $request)
{
    $company = CompanyQuery::create()->findOne();
    $formObj = $this->createForm('company_type', $company);
    $formObj->handleRequest($request);

    if ($formObj->isValid()) {
        $company->save();

        return $this->redirect($this->generateUrl('acme_blog_company'));
    }

    return array('form' => $formObj->createView());
}
...

Form type:
...
public function buildForm(FormBuilderInterface $builder, array $options)
{
    $builder->add('name');
    $builder->add('users', 'collection', array(
        'type' => 'user_type',
        'allow_add' => true,
        'allow_delete' => true,
        'by_reference' => false
    ));
    $builder->add('save', 'submit');
}
...

My issue is that on first form submission, everything works fine: The users are created properly and the company name is updated as it should.
But if I submit my form again (without deleting the users, just changing their name), they get deleted.
After that, I can add new users, they get created and if I submit the form again, they disappear, etc...
Here is a sample of the request parameters sent to the controller (we are here only working in a company edition scenario to avoid confusion):
First request:
company_type[name]:Main company
company_type[users][0][firstname]:user 1
company_type[users][1][firstname]:user 2
company_type[save]:
company_type[_token]:...

=> All the users are created just fine
Second request:
company_type[name]:Main company
company_type[users][0][firstname]:user 1 edited
company_type[users][1][firstname]:user 2 edited
company_type[save]:
company_type[_token]:...

=> All the users are gone. Note: I have the same result, even if I don't change the names (I have the issue even if I simply resend the same form).
This issue seems to be linked to the fact that in the second request, the company.removeUser method is called => the users are removed instead of being updated.

After some deeper inspection, I think that I have found the origin of the problem:
This row https://github.com/symfony/symfony/blob/2.8/src/Symfony/Component/PropertyAccess/PropertyAccessor.php#L467 checks if the child object(s) already linked to the parent is equal to the one we are storing instead. If there is a match, then the object is skipped (not added nor removed).
When using doctrine, the two objects are equal:
object(Acme\Bundle\BlogBundle\Entity\User)[475]
  private 'id' => int 3
  private 'firstname' => string 'name edited' (length=11)
  ...

object(Acme\Bundle\BlogBundle\Entity\User)[475]
  private 'id' => int 3
  private 'firstname' => string 'name edited' (length=11)
  ...

Whereas when using Propel, the two objects differ:
object(Acme\Bundle\BlogBundle\Model\User)[464]
  ...
  protected 'id' => int 54
  protected 'firstname' => string 'name' (length=4)
  ...

object(Acme\Bundle\BlogBundle\Model\User)[444]
  ...
  protected 'id' => int 54
  protected 'firstname' => string 'name edited' (length=11)
  ...

And even when there are no modifications, the handle is still different and the condition is still not matched:
object(Acme\Bundle\BlogBundle\Model\User)[464]
  ...
  protected 'id' => int 57
  protected 'firstname' => string 'name edited' (length=4)
  ...

object(Acme\Bundle\BlogBundle\Model\User)[444]
  ...
  protected 'id' => int 57
  protected 'firstname' => string 'name edited' (length=11)
  ...

We have encountered a similar problem (collection emptied on update) and provided a minimal Symfony2 project showing this problem: https://github.com/spyrit/MinimalS2P2
There is also a Propel1 branch with (almost) the same code without this problem.
Regards.

Same issue here..
And the child elements get deleted regardless of the allow_delete option of the collection type.

Hmm, it seems that removing the by_reference option (or setting it to true) resolves the issue!

No, then it is not possible to delete a child.

To sum up the situation, for each collection the Symfony PropertyAccessor test the inside objects equality to know if it has to keep it or remove it from the collection.
    // Symfony/Component/PropertyAccess/PropertyAccessor.php
    /**
     * Adjusts a collection-valued property by calling add*() and remove*()
     * methods.
     *
     * @param object             $object       The object to write to
     * @param string             $property     The property to write
     * @param array|\Traversable $collection   The collection to write
     * @param string             $addMethod    The add*() method
     * @param string             $removeMethod The remove*() method
     */
    private function writeCollection($object, $property, $collection, $addMethod, $removeMethod)
    {
        // At this point the add and remove methods have been found
        // Use iterator_to_array() instead of clone in order to prevent side effects
        // see https://github.com/symfony/symfony/issues/4670
        $itemsToAdd = is_object($collection) ? iterator_to_array($collection) : $collection;
        $itemToRemove = array();
        $propertyValue = &$this->readProperty($object, $property);
        $previousValue = $propertyValue[self::VALUE];
        // remove reference to avoid modifications
        unset($propertyValue);

        if (is_array($previousValue) || $previousValue instanceof \Traversable) {
            foreach ($previousValue as $previousItem) {
                foreach ($collection as $key => $item) {
                    if ($item === $previousItem) {
                        // Item found, don't add
                        unset($itemsToAdd[$key]);

                        // Next $previousItem
                        continue 2;
                    }
                }

                // Item not found, add to remove list
                $itemToRemove[] = $previousItem;
            }
        }

        foreach ($itemToRemove as $item) {
            $object->{$removeMethod}($item);
        }

        foreach ($itemsToAdd as $item) {
            $object->{$addMethod}($item);
        }
    }

If the object is in the collection (of the persisted objects) keep it (continue 2), but if it's not, remove it $itemToRemove[] = $previousItem;. This assertion is correct and it's working for Doctrine and Propel1.
In fact, the strict equality operator === test if both objects make reference to the same memory space. However, this is not the case as we have to use the by_reference option (to make propel works) that clone the original object.
// Symfony/Component/Form/Form.php
public function setData($modelData)
{
     //...
     if (is_object($modelData) && !$this->config->getByReference()) {
         $modelData = clone $modelData;
     }
}

A simple equality comparison == make it works but we cannot ask to Symfony to change this part of the code which otherwise is correct. So, we have to find what is the difference between Propel1 and 2 and fix it. Easy to say...

Well, after a deep look into propel1, it seems that the behavior was to let Symfony call the removeUser for all the entities already in the DB and the addUser for all the submitted entities. So exactly what we encounter on propel2.
The code and the behavior of the generated addUser and removeUser is pretty much the same on Propel1 and 2, so I guess that the key of the solution must be in the object's save procedure (so on the propel side)

After a comparison of generated classes between propel2 and propel1, I have found the workaround that was used in propel1: in my Company.addUser generated class, there is a condition that unschedule the user from deletion:
if (!in_array($l, $this->collUsers->getArrayCopy(), true)) { // only add it if the **same** object is not      already associated
	$this->doAddUser($l);

	if ($this->usersScheduledForDeletion and $this->usersScheduledForDeletion->contains($l)) {
		$this->usersScheduledForDeletion->remove($this->usersScheduledForDeletion->search($l));
	}
}

Here is the corresponding code in propel2:
    public function addUser(ChildUser $l)
    {
        if ($this->collUsers === null) {
            $this->initUsers();
            $this->collUsersPartial = true;
        }

        if (!$this->collUsers->contains($l)) {
            $this->doAddUser($l);
        }

        return $this;
    }

And simply by replacing propel2's method content by propel1's, it solves the problem. I am working on a PR... :smile:

Very good job, you get it !

So this is fixed and can be closed, right?

IMHO yes

yup


in_header false values in the yaml files

I'm just looking through the source for WebFundamentals and there are a few cases where the yaml files are declaring in_header: false.
_betterbook-showcase.yaml is a good example.
	toc:
	- title: "Web Showcase"
	  path: /showcase/
	  home: true
	- title: "Case Studies"
	  path: /showcase/case-study/
	- title: "Spotlights"
	  path: /showcase/spotlight/
	- title: Web Fundamentals
	  path: /fundamentals/
	  in_header: false
	- title: Web Starter Kit
	  path: /starter-kit/
	  in_header: false

Is there any reason behind this? Looks like it's not needed.

You'll have to ask @pbakaus as that is his file, I can't find any references to it.

Acutally, I typed it wrong.

It's not just that file, it's in betterbook-wsk and betterbook

Right, so it is used to determine if it should be placed as a header item in the main bar.  navigation.liquid uses it.
This should allow us to deploy sub-sections of the site and have them live, but not, importantly have them linked from anywhere.

Right - so its useful adding the links you think you'll want when a site goes live, but otherwise there is no reason.

It controls whether it will be in the header or not, the build normally resolves everything automatically

so no intention to add that link to header === useless having it in the betterbook.

I believe we might use it to stop pages appearing in other categories too.

I wish the links in the footer would be automatically put in, but currently that's the way of defining them.


Ability to apply @param.depends  to all (nested) Parameters of a class

I am developing a framework using param that aids engineers (who are not so savvy in Python) in defining 'constants' and associated metadata related to an engineering design. The engineer user would a simple python Parameterized class to organize these constants, and the framework goes through the class, and generates files for another tool that needs to consume these 'constants'.
The hierarchy is something like this:
# part of the framework
class Constant(param.Parametrized):
    val = param.Integer()
    metadata = param.String()
    ....

# written by engineer A
class ModuleConstants(param.Parametrized):
    constX = param.ClassSelector(default=Constant(val = 10, metadata = "xxx"), class_=param.Parameterized)
    constY = param.ClassSelector(default=Constant(val = 20), class_=param.Parameterized)

    @param.depends('constX.val', watch=True)
    def update_derived(self):
        self.constY.val = self.constX.val * 2

# written by engineer B
class TopConstants(param.Parametrized):
    const1 = param.ClassSelector(default=Constant(val = 2), class_=param.Parameterized)
    const2  = param.ClassSelector(default=Constant(val = 4), class_=param.Parameterized)
    child = param.ClassSelector(default=ModuleConstants(), class_=param.Parameterized)

    @param.depends('const1.val', watch=True)
    def update_derived(self):
        self.child.constX.val = self.const1.val + 10

t = TopConstants()
t.const1 = 100
print(t.child.constY) # we expect (100 + 10) * 2 = 220

So there could be an unlimited number of nested Constant parameters here, and there are dependencies between the val parameter of the Constant parameters at different levels of hierarchy.
The problem we have been running into is that because we could have tens of these Constant parameters defined, it's very easy to forget for the engineers to include the list of dependencies in the @param.depends decorator. What I want is for update_derived() to be called if any of the Constant.val integers change, regardless of how many levels deep the Constant is nested. Performance is really not an issue here so I don't mind all the redundant operations this would cause. There are no circular dependencies to worry about.
I am only slightly better at python than these forgetful engineers myself, and I have been unsuccessful in working around this issue. I have tried:

.param.watch() (lower level API): doesn't support nested parameters?
Dynamic parameters and the 'pull' model: doesn't supported nested parameters
Programmatically decorating update_derived() with something like this in __init__() of TopConstants, where list_of_all_params is obtained by manipulation of the .param.values() dict:

def __init__(self, **kwargs):
    super().__init__(**kwargs)
    self.update_derived = param.depends(*list_of_all_params, watch=True)(self.update_derived)

This compiles but doesn't seem to register a watcher.

Reactive Expressions: the documentation shows how to create an expression from two parameters, but not how to assign that back to another parameter

I am wondering if there is a solution here or if this (admittedly weird) use case is unsupported.

any pointers here folks?


--no-site-packages argument is in help but not recognized

Be sure to check the existing issues (both open and closed!), and make sure you are running the latest version of Pipenv.
Check the diagnose documentation for common issues before posting! We may close your issue if it is very similar to one of them. Please be considerate, or be on your way.
Make sure to mention your debugging experience if the documented solution failed.
Issue description
Describe the issue briefly here.
Expected result
Describe what you expected.
Actual result
When possible, provide the verbose output (--verbose), especially for locking and dependencies resolving issues.
Steps to replicate
Provide the steps to replicate (which usually at least includes the commands and the Pipfile).

Please run $ pipenv --support, and paste the results here. Don't put backticks (`) around it! The output already contains Markdown formatting.
If you're on macOS, run the following:
$ pipenv --support | pbcopy

If you're on Windows, run the following:
> pipenv --support | clip

If you're on Linux, run the following:
$ pipenv --support | xclip

@NFSpeedy Did you mean to close this?

@matteius Yes, I figured it out. All arguments are accepted after the main command except some that have to be given before the command.
As in my case:
pipenv --no-site-packages sync --dev -v
It confused me, because -v, for example, is a global arg, and --no-site-packages should be the same in my mind.


Use global variable for accorion-tabs

This allows the use of Bitters variables if they are available.
If not, there is a fallback value for the variable.
The tabbing in the variables might not be according to our style guides but it makes it easier to read.
What do you think @kylefiedler, @tysongach, @kaishin?
Is this a good middle ground between Bitters and Refills?
Sidenote why Refills should not depend on Bitters:
Consider this scenario. It has happened a few times in the past:

Refills depends on Bitters v1
User gets Bitters v1
User changes Bitters v1 locally
Bitter gets an update. Turns into Bitters v2 remotely
Refills updates to follow Bitters v2 instead of v1
User copies a Refills component
Component doesn’t work locally
It requires Bitters v2 and not Bitters v1
User can’t switch to Bitters v2 because the local files are all customized and changed
Refills should not be dependent on Bitters

I agree with not having Refills depend on Bitters, but I think this approach is more logical.

I like that we are brining down the complexity of the component but I agree with @tysongach that I prefer if we could get to a point where there aren't any variables.


Adding History of Changes of HTML / CSS / SCSS / JS Files

team,
I would like to suggest a feature, that brackets needs to maintain a log history of changes based on every timestamps. where users can able to compare or revert back the code changes from the History backlog tool and update the same. something like code recovery from brackets time to time.
we can have panel like "validation problems" at footer of the console and they can view the changes respective to corresponding pages and they select and compare with working copy and modify those changes if required.
thanks
Manoj

This is exactly what a version control system is for and Brackets has a very functional and well-maintained git extension. I would suggest that recreating git in Brackets is not advised and git should fill the requirements just fine.

There is a very powerful brackets git extension for exactly the same purpose you suggested. Please check it out. There is a steep learning curve for getting used to Git though. Closing.

thanks fro reply, i was not aware of extension.


Add custom classname to Container react component

How can I add a className to Container component? Using className prop doesn't work.

Did you try like this ?
count= {this.state.count}

Container accepts style property for now. But you can put it inside another div and set it's className to have desired look.

@kutlugsahin It might be even more useful to allow overriding the div (think a headless component). Then it's easier to compose to fit an existing solution.

Actually there is an undocumented render prop for that, see following code
<Container render={(setRef) => (
  <div ref={setRef}>
   {...} // render draggables here
  </div>
 )} />

render prop is documented now


Working branch 1

Please approve my code?

@kshirasaagar Welcome to GitHub! You've got the workflow down for pull requests. Going to decline this one because the only difference are the 2 rows of comments added, but nice work on learning the ropes.

@kshirasaagar Welcome to GitHub! You've got the workflow down for pull requests. Going to decline this one because the only difference are the 2 rows of comments added, but nice work on learning the ropes.


Duplicate field name: no error raised

If tableschema contains the same field name more than once, the data are valid and no error is raised when reading the data.
I get an error when I try to create a NetCDF file from the schema (duplicate variables name not allowed)
See 10.5281_zenodo.3634411; the field "Ni_flags" is defined twice in tableschema and also in the original csv file.

As a curiosity: the package "goodtables" raises an error in this case (unless goodtables --skip-checks check_name_i_cant_remember). When I implemented the validator I didn't know that goodtables could be used to validate full data packages but I plan to add extra validations using goodtables as well.
The frictionless data specification allow to have duplicated column names (I guess that it's good to be able to describe not optimal tabular data!). We can discuss what to do in this package tomorrow (@jen-thomas might have some ideas :-) )

Related to this: https://github.com/Swiss-Polar-Institute/frictionless-data-packages/issues/3

For this particular dataset I've notified the scientist so they can correct the dataset in Zenodo.
Related to this: we should perhaps discuss how to deal with changes / updates to datasets.
eg. correction -> new version -> new DOI


Add stub for Testing module

Now that swift-testing is present in the toolchain, SwiftPM builds of the test target in this package will automatically import Testing, causing our test target to link the Testing library in the toolchain. This is problematic, because the Testing module links Foundation (from the toolchain) which results in a variety of issues and crashes when running our tests caused by the presence of two Foundations (classes of the same name are indistinguishable and one implementation is nondeterministically picked leading to failures when bridging). To resolve this, we will use the same solution we found for XCTest - to vendor a small copy of the library within our project that will be picked up instead of the toolchain version. In our case, we only need a small stub because we don't actually use the Testing module yet.
In the future, now that SwiftPM supports cycles in package dependencies as long as there is no target-level cycle, we could update the package build to use the package build of swift-testing (which we could have rely on the package build of Foundation thus ensuring only one Foundation is present) but for now this simple solution gets us building and testing again.

@swift-ci please test

Confirmed that the tests still pass with this change, and they also pass with the linked change so this should resolve the issue that the linked PR was originally hitting when re-ordering the swift-testing build

@swift-ci please test

@swift-ci please test Windows platform


No sound from nDax

Ran nDax as follows:
scap@Computer:~$ ./nDAX-linux-amd64 -station Maestro -daxch 2
2020/06/23 04:50:07 Waiting for station: Maestro
2020/06/23 04:50:07 Found client ID 0x7518157A UUID AC586976-05EE-4252-9B9F-692189960DFA
2020/06/23 04:50:07 Looking for slice: A
2020/06/23 04:50:07 Found slice 0
2020/06/23 04:50:07 enabled RX DAX stream 4000009
2020/06/23 04:50:07 enabled TX DAX stream 84000000
excessSamples: 0

I was able to set up wsjt-x with the flexdax.rx.monitor and flexdax.tx
TX seems to work and I even got rig control working with nCat. I was spotted as well so I know I got out. RX I have no audio though.
I don't see any audio being sent from nDax to the RX loopback.

Does it work if you don't use -daxch 2 ?

no I did that to try something different. The slice was also set to channel 2. I have tried one as well.

Okay, I'll look into it.
On Tue, Jun 23, 2020 at 3:35 PM scap1784<EMAIL_ADDRESS>wrote:

no I did that to try something different. The slice was also set to
channel 2. I have tried one as well.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kc2g-flex-tools/nDAX/issues/7#issuecomment-648375029,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABR3BEP5YXWDU2JFBD4M3RYD7YXANCNFSM4OFP2RRA
.

I am using a 6600 I think from your QRZ page your using a 6400... Don't know why that would matter as the protocols are the same. I am writing some software for this as well and I have used your code to understand various parts of the protocol. I am not as far as you. I am not very familar with Go but I will take a look this evening at the code and see if I can't figure it out... Its almost like the program just isn't sending the audio to pulse.
Oh and as for the computer. I am running Cinnamon (4.4.8) on Linux Mint (19.3).

Try using tcpdump or ngrep to see whether you're getting a stream of
UDP packets from the radio. Also try running nDAX with -debug-timing
which will enable lots of additional debug messages. Let it run for a few
seconds, ^C it, and then provide the complete output.
On Tue, Jun 23, 2020 at 3:41 PM scap1784<EMAIL_ADDRESS>wrote:

I am using a 6600 I think from your QRZ page your using a 6400... Don't
know why that would matter as the protocols are the same. I am writing some
software for this as well and I have used your code to understand various
parts of the protocol. I am not as far as you. I am not very familar with
Go but I will take a look this evening at the code and see if I can't
figure it out... Its almost like the program just isn't sending the audio
to pulse.
Oh and as for the computer. I am running Cinnamon (4.4.8) on Linux Mint
(19.3).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kc2g-flex-tools/nDAX/issues/7#issuecomment-648377769,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAABR3DBZZZ7JCQH2EPCNV3RYEAOBANCNFSM4OFP2RRA
.

Lots of UDP traffic.
Not really any debug print out
scap@Jupiter:~$ ./nDAX-linux-amd64 -daxch 2 -station N5SDR-Maestro -debug-timing2020/06/23 14:48:22 Waiting for station: N5SDR-Maestro
2020/06/23 14:48:22 Found client ID 0x7518157A UUID AC586976-05EE-4252-9B9F-692189960DFA
2020/06/23 14:48:23 Looking for slice: A
2020/06/23 14:48:23 Found slice 0
2020/06/23 14:48:23 enabled RX DAX stream 4000009
2020/06/23 14:48:23 enabled TX DAX stream 84000000
excessSamples: 0

Ok so note Linux Mint has a pretty restrictive firewall... Open it up and stuff works. Thanks for the responses and writing some useful software! :)

This is awesome... I don't need windows anymore :)

Thanks for the info — seems like I have something to add to the documentation :)


I met this error when run main.py

Hi, @errollw
I met this error when run main.py
kraken-2:EyeTab_Python dti$ python3 main.py
Traceback (most recent call last):
File "main.py", line 5, in
import gaze_system as gaze_system
File "/Users/dti/EyeTab/EyeTab_Python/gaze_system.py", line 4, in
import draw_utils
File "/Users/dti/EyeTab/EyeTab_Python/draw_utils.py", line 5
def draw_cross(bgr_img, (x, y), color=(255, 255, 255), width=2, thickness=1):
^
SyntaxError: invalid syntax
What's wrong with me?
Thanks in advance~

我也遇到同样的问题，楼主是否已经解决了？

Hello,
I met the same error.

I think that you need to replace def draw_cross(bgr_img, (x, y), color=(255, 255, 255), width=2, thickness=1):
with
def draw_cross(bgr_img, x, y, color=(255, 255, 255), width=2, thickness=1):


Websocket setup to resolve #57

@itsniezwe could you review my pull request and merge if it looks ok for you?

also resolves #2


Can't install on local host

Hi There is any chance to update this to the new version of all dependencies
When i try to execue yarn start command i get an issue like below

Hello,
It seems the version of the node.js is the issue here, but I don't have the time to dig deeper. You can still run it locally if you switch to node.js 16.
Here's how to do it with nvm (https://github.com/nvm-sh/nvm):

Delete the node_modules folder if present
If you don't have node.js 16 install run nvm install 16
Switch to node.js 16: nvm use 16
Run: yarn install
Run :yarn start

Thanks! It works :)


Added functional tests and first FT to check homepage

This change creates a base.py which contains the Functional Test class. Currently, the class subclasses the LiveServerTestCase class and has setUp and tearDown methods. test_finding_contributr_projects.py is not completely finished, and stops at an intentional failure with a reminder to finish the test. Currently, it only checks for Contributr in the browser title and on the home page. I'll be pushing for more test cases to get ahead of some of the view development and to see if we can create user stories for testing overall functionality during development and for regression testing after new features are introduced.

@talaniz So why a PR when the functionality isn't finished or when the tests are failing?

2 reasons:

The test is technically in a passing state, the fail is expected &
initiated as a reminder to continue working on the test.
The format is established in case anyone wants to contribute to it. FT's
will usually be added to when new functionality is introduced.

On Sep 27, 2015, at 12:17 AM, Harshvardhan Pandit<EMAIL_ADDRESS>wrote:
@talaniz https://github.com/talaniz So why a PR when the functionality
isn't finished or when the tests are failing?
—
Reply to this email directly or view it on GitHub
https://github.com/Djenesis/contributr/pull/81#issuecomment-143526354.

@talaniz why do you want an intentionally failing test to be included that will break the CI builds every time? If the functionality isn't finished, you should create an issue on it, or a branch, if more people are working on it.

K, thx for the input
On Oct 1, 2015, at 2:49 PM, Harshvardhan Pandit<EMAIL_ADDRESS>wrote:
@talaniz https://github.com/talaniz why do you want an intentionally
failing test to be included that will break the CI builds every time? If
the functionality isn't finished, you should create an issue on it, or a
branch, if more people are working on it.
—
Reply to this email directly or view it on GitHub
https://github.com/Djenesis/contributr/pull/81#issuecomment-144859352.

The reason there is an intentional fail is a reminder to complete the functional test. These tests are not meant to be covered by Travis CI and at the moment, they should not be covered in Pytest either. This is for testing that the code meets the expected functionality and that new features don't break any existing features. I've added the @pytest.mark.xfail and I'm checking to make sure this won't break Travis CI before pushing again.

Closing this request. Too many issues with functional test against Travis CI. Will send another later.


example isn't working in Azure Devops Server 2019

Hi -
I'm trying to get the example working on our installation of Azure DevOps.
Working on our Bug types:
Created new picklist (string) field named "MajorRelease" (no spaces) with values "Release Blue" and "Release Red"

Created another picklist (string) field named "MinorRelease" (no spaces) with values "Blue.1", "Blue.2", "Blue.3", "Red.A", "Red.B" and "Red.C".

Pasted the sample JSON into the Cascading Lists Config for our project settings, and saved the configuration.
Opened a new window, created a new bug, and i can see the two new fields.    Under Major Release, I select "Release Blue", and then under Minor Release, i expect to only see the Blue items listed, but instead I see all 6 items (red and blue).

Can you please validate to make sure the field name is in fact custom.majorrelease? You may need to use the REST API's to do so. Do you get any kind of error in the dev console?

Did this issue get resolved. I am still facing the issue


Enabling Search input field.

Description
Enabling Search input field to filter the courses being searched.
Wanna Work upon[y/n] ??
yes

Can I work on it? Please assign it to me.


Azure cli version 2.67.0 no longer supporting --password for client certificate as secret. And azure/login@v2 is started failing.

Hello,
We work with client Certificate as Secret when AZ login with SPN. We haven't faced any issue till last week when this action was using Azcli version < 2.67.0.
Due to the latest release of Az cli, It started failing with below error:
Attempting Azure CLI login by using service principal with secret...
Error: AADSTS7000215: Invalid client secret provided. Ensure the secret being sent in the request is the client secret value, not the client secret ID, for a secret added to app '***'.
Error: The error may be caused by passing a service principal certificate with --password. Please note that --password no longer accepts a service principal certificate. To pass a service principal certificate, use --certificate instead.
Can we expect a new release from Azure/login Action to support this recent change? Or can we have an option to use a custom Azure Cli version which we can set to < 2.67.0??
Right now, as a workaround I am downgrading the Az cli version and using the same action. I have to use this azure/login action because it's the mandatory pre-requisite step before using azure/aks-set-context@v3.

- name: Downgrade Az cli version
      shell: bash
      run: |
            AZ_DIST=$(lsb_release -cs)
            AZ_VER=2.65.0
            sudo apt-get install azure-cli=${AZ_VER}-1~${AZ_DIST} --allow-downgrades
            az version

 - uses: azure/login@v1
      with:
        creds: "{
          \"clientId\": \"${{ env.ARM_CLIENT_ID }}\",
          \"clientSecret\": \"key.pem\",
          \"subscriptionId\": \"${{ env.ARM_SUBSCRIPTION_ID }}\",
          \"tenantId\": \"${{ env.ARM_TENANT_ID }}\"
          }"

Duplicate of https://github.com/Azure/login/issues/496. Let's track the issue there.


🚧 Explicit errors for common pitfalls or mismatched expectations.

Whenever you're learning a new language, unless it's identical to whatever you're familiar with already, it's likely you'll run into a few hiccups along the way when things don't work as you expect.
Ren's design aims to be accessible to JavaScript developers while also taking inspiration from ML and leaning into functional programming a lot more. Inevitably there are going to be things people try to do in Ren that won't work, and while some of those things can be solved at the language level (eg 9281c142c02d0cffbeb4bce2a8c4ed930d3875df), others can be addressed with a friendly error message specific to the situation.
I'd like this issue to act as a tracker for any of these common pitfalls or mismatched expectations, so we can make sure our error messages leave users feeling confident not confused!

[ ] Potential confusion between fun declarations and expressions.

In the programming languages discord, one user pointed out that typically everything that follows the = would be assumed to be an expression. In Ren, this isn't the case:
fun <name> = <...args> =>
    <expr>

The arguments list follows the equals, with the body following =>. This was designed to align visually with javascript but there is a semantic difference between the two (x => x + 2 is a valid expression in js, but the equivalent in Ren would be fun x => x + 2)
Because the current expression language doesn't actually include a = operator, we could look for expressions that take the form fun x = y and emit a helpful error showing the expression lambda syntax and explaining the difference.
Example error message:
todo

[ ] Omitting the else clause in an if expression.

Ren is an expression-oriented language. There are no statements, and that includes if statements. In Ren, an if expression evaluates to a value and so needs an accompanying else clause to evaluate when the condition is false.
Example error message:
todo

[ ] Attempting to early-return with the ret keyword.

As mentioned above, Ren does not have statements. A function or let binding body consists of exactly one expression. There is no concept of early returning, and the ret keyword exists purely as a visual marker for that expression.
Example error message:
todo

[ ] Inserting expressions between local bindings in a function.

Example error message:
todo

Add forgetting the then keyword, and optionally parentheses around the condition (valid as an expression, but not required).

[ ] Attempting c-style if/else statements by mistakenly using blocks.

In c-style languages, if statements typically look like:
if (some condition) {
  ...
} else {
  ...
}

This is almost valid ren syntax because of block expressions, but it will fail when the parser looks for the then keyword can't find it! (some condition) { ... } would be treated as function application and not a separate of the condition and then block, so we should have a specific error message here when there is a missing `then keyword.
The correct ren, for reference, might look like:
if some condition then {
  ret ...
} else {
  ret ...
}

Example error message:
todo

Fun declarations no longer exist, so that error can now be ruled out. (Everything after = is now an expression)