How can I dispatch an epic action to a substore?

This is a...

[ ] feature request
[ ] bug report
[x] usage question

What toolchain are you using for transpilation/bundling?

[x] @angular/cli
[ ] Custom @ngTools/webpack
[ ] Raw ngc
[ ] SystemJS
[ ] Rollup
[ ] Other

Environment
NodeJS Version: 8.11.2
Typescript Version: 2.7.2
Angular Version: 6.0.3
@angular-redux/store version: 9.0.0
@angular/cli version: (if applicable) 6.0.7
OS: win32 x64
Steps to reproduce:

Set up a child component using @WithSubStore
Set up an epic using createEpicMiddleware
Dispatch an action from the child component that triggers the epic.

Expected Behaviour:
The epic triggers and creates a new action. Both the root reducer and the substore reducer should receive the new action.
Actual Behaviour:
The root reducer receives the new action, but the substore reducer does not!
Additional Notes:
What is the correct way to use epics and fractal stores together?
I noticed that the action, when dispatched to the substore, has a property like this:
action['@angular-redux::fractalkey']
I found that I could set this property explicitly when creating the action to fix the issue.
This seems like it relies on some unpublished implementation detail of angular-redux, and I somehow feel like it defeats the purpose of epic/substore usage.
Any help would be appreciated!
Thanks

+1 I am also wondering how to add epic to fractal store

@rmorrise @mika1111  Since actions that are dispatched from component using WithSubStore decorator will append sub store path with '@angular-redux::fractalkey'. You can have other action to take the value of sub store path and epic could simply pass it along.
const ping = (...yourArgs, fractalKey?: string) => ({
  type: PING,
  ...yourArgs,
  '@angular-redux::fractalkey': fractalKey
})

const pong = (...yourArgs, fractalKey?: string) => ({
  type: PONG,
  ...yourArgs,
  '@angular-redux::fractalkey': fractalKey
})

const pingToDaPong = action$ => action$
  .ofType(PING)
  .map(action => {
    return pong(action.arg1, action.arg2, action['@angular-redux::fractalkey'])
  })


Move to GitHub?

Love the program. Would love to develop in Git, as I'm sure others would too.
GitHub makes it easy for others to pitch in. Is there any consideration of
moving superputty to Github?

Original issue reported on code.google.com by aavindraa on 16 Jul 2014 at 11:02

Done.


Add default text

💥 Proposal
The idea is to add a default text button in which when a user select a radio button then on pressing this button, it gives default text for that feature. Also a clear button to clear the text in text area. (Both for analysis and generate page)
Example :
User press the remove punctuation radio button
then default text button gives a random text to check this feature.
Like =text!!analysis
It will show this text on text area.
Have you read the Contributing Guidelines on Pull Requests ?
Yes

I want to solve this issue when you accept this issue

💥 Proposal
The idea is to add a default text button in which when a user select a radio button then on pressing this button, it gives default text for that feature. Also a clear button to clear the text in text area. (Both for analysis and generate page)
Example :
User press the remove punctuation radio button
then default text button gives a random text to check this feature.
Like =text!!analysis
It will show this text on text area.
Have you read the Contributing Guidelines on Pull Requests ?
Yes

Ok @Akash20x. I got it. You can work on it.
You are given 6 days to complete it.

Please add necessary labels to this issue

Will you add both clear and default text?

Will you add both clear and default text?

Yes

Ok.

Please check my pull request


Align Phases of Isomorphous Structures

Often times, I notice that SAD phasing solutions from isomorphous structures don't overlay in real space. This seems to have something to do with the choice of "phase origin". It'd be super useful if we had a function to make sure that two sets of structure factors have compatible origins. We should be able to implement something akin to @apeck12's method. After talking to @JBGreisman, we decided doing this in cases with good phases might boil down to solving a simple linear program. At any rate, this should absolutely be something we put in rs.algorithms.

@kmdalton Is the issue that both structures look correct but are rotated and translated with respect to each other? If that’s the case, then a full phase origin search isn’t needed (except in P1). There’s a limited number of equivalent phase origins for each space group — P212121, for example, has only 4 — so one could get away with just mapping the phases of the second structure onto each of those equivalent origins and checking for consistency with the first.

Yes, I think you've characterized the issue correctly, @apeck12. Do you know how we can generate the phase origins for a particular group programmatically? In cases like P21 or P61 without orthogonal symmetry elements, are there still finitely many phase origins, or are they restricted to a line?

I can confirm that

Searching a finite set of origins works to align phases in tetragonal lysozyme. I have started work on a new branch that implements the algorithm I tested.
Arbitrarily translating along the z-axis in spacegroup P63 leads to valid phase combinations and nice electron density.
Arbitrarily translating the phases along other axes does not result in interpretable density.

Based on this, I think we need a hybrid approach to cover all space groups. Space groups with orthogonal symmetry axes should be handled by the approach in 1. I'm afraid we will need to learn the translation vector by a grid search and/or optimization in cases like P63.
What edge cases am I missing?
How can we write effective tests for these new algorithms?

A few thoughts:

As a first pass to define the search space for a specific space group, I tried parsing the symmetry operations to detect polar axes and valid shifts along nonpolar axes. Polar axes are distinguished by having no symmetry operations that invert the axis (see “Polar Space Groups”). For nonpolar axes, my understanding is that permitted shifts are determined by the fractional translation associated with the symmetry operation that inverts the axis.

Here’s the code:
def find_equivalent_origins(sg, sampling_interval=0.1):
    """
    Determine fractional coordinates corresponding to equivalent phase
    origins for the space group of interest.
    Parameters
    ----------
    sg_object : cctbx object
        space group object
    sampling_interval : float (optional)
        search interval for polar axes as fractional cell length
    Returns
    -------
    eq_origins : array
        nx3 array of equivalent fractional origins
    """

    permitted = dict()
    for axis in ['x','y','z']:
        permitted[axis] = []

    # add shift for each symmetry element that contains an inversion
    sym_ops = list(sg_object.smx())
    for op in sym_ops:
        op_as_xyz = op.as_xyz()
        if "-" in op_as_xyz:
            for element in op_as_xyz.split(","):
                if "-" in element:
                    if any(i.isdigit() for i in element):
                        axis, frac = element.split("+")
                        permitted[axis[1]].append(float(frac[0]) / float(frac[2]))
                    else:
                        permitted[element.split("-")[1][0]].append(0)

    # eliminate redundant entries and expand nonpolar axes
    for axis in permitted.keys():
        permitted[axis] = np.unique(permitted[axis])
        if len(permitted[axis]) == 0:
            permitted[axis] = np.arange(0,1,sampling_interval)

    # get all combinations from each axis
    eq_origins = np.stack(np.meshgrid(permitted['x'],
                                      permitted['y'],
                                      permitted['z'])).reshape((3, -1)).T

    return eq_origins

Here are plots for valid origin shifts versus a random shift (second row, last column) for a P212121 crystal:

And for a P61 crystal:

I examined a few other space groups with similar results, but this should be tested more exhaustively.

I think for some non-primitive cells, each of the equivalent origins also needs to be checked when translated by (1/2,1/2,1/2).
For experimental data, would it make sense to weight each phase by the I/sig(I) of the associated reflection?
One test would be to check consistency between centric reflections and their expected values.

This looks really promising, and thank you for the source code. This should be very easy to translate to use gemmi space groups.

What I had done in my branch was just to search the union of all origins available to all nonpolar groups. I have this implemented as
 v = np.array([0, 1/6, 1/4, 1/3, 1/2, 2/3, 3/4, 5/6])
 v = np.stack(np.meshgrid(v,v,v)).reshape((3, -1))
 ```
 This is a tad inefficient, but prevents having to write anything space group specific. I think my little meshgrid should include all possible translations across all groups. When I tested that grid with my experimental maps, there was a clear winner. To be concrete, I had experimental SAD phases for tetragonal lysozyme (space group 96) that I wanted to align to an F-Model output from an isomorphous PDB.
 ![image](https://user-images.githubusercontent.com/2790777/102902308-d3f97680-443c-11eb-8f6d-3157d86712d7.png)

 Here are the residuals I was evaluating over my meshgrid:
 ![image](https://user-images.githubusercontent.com/2790777/102902633-48341a00-443d-11eb-9d77-e26857bb8a08.png)

 Here are the phases after alignment:
 ![image](https://user-images.githubusercontent.com/2790777/102902546-23d83d80-443d-11eb-847b-04787074b84a.png)
 ^^ There might still be something confusing going on with centrics. I am not sure. I may need to look into it...

 It is a little silly, but I think this could be a decent fallback implementation. I'll have a go at translating your code to `rs`/`gemmi` when i have a spare moment.

2. Easy enough to just always check that vector. It is not a very expensive calculation mercifully.
3. This is a really good idea. When we implement this in `rs.algorithms` we should have a `weights=` keyword.
4. You are thinking about something like equation 6 in your manuscript?

I looked into the centrics and scatter plot was just misleading. Turns out most of them are correct.

Here’s the Gemmi version of the find_equivalent_origins function that takes as input the space group number rather than a cctbx object:
def find_equivalent_origins(sg_no, sampling_interval=0.1):
    """
    Determine fractional coordinates corresponding to equivalent phase
    origins for the space group of interest.
    Parameters
    ----------
    sg_no : int
        space group number
    sampling_interval : float (optional)
        search interval for polar axes as fractional cell length
    Returns
    -------
    eq_origins : array
        nx3 array of equivalent fractional origins
    """

    permitted = dict()
    for axis in ['x','y','z']:
        permitted[axis] = []

    # add shift for each symmetry element that contains an inversion
    sym_ops = gemmi.find_spacegroup_by_number(sg_no).operations()
    sym_ops = [op.triplet() for op in sym_ops]
    for op in sym_ops:
        if "-" in op:
            for element in op.split(","):
                if "-" in element:
                    if any(i.isdigit() for i in element):
                        axis, frac = element.split("+")
                        permitted[axis[1]].append(float(frac[0]) / float(frac[2]))
                    else:
                        permitted[element.split("-")[1][0]].append(0)

    # eliminate redundant entries and expand polar axes
    for axis in permitted.keys():
        permitted[axis] = np.unique(permitted[axis])
        if len(permitted[axis]) == 0:
            permitted[axis] = np.arange(0,1,sampling_interval)

    # get all combinations from each axis
    eq_origins = np.stack(np.meshgrid(permitted['x'],
                                      permitted['y'],
                                      permitted['z'])).reshape((3, -1)).T

    return eq_origins

It seems like the main motivation for restricting the search is to avoid invalid origins since align_phases is very efficient. Though perhaps experimental data are always of high enough quality that an invalid origin will never be mistaken for a correct one based on the residuals.
Eq. 6 is what I had in mind for the centric vs. expected values residual. I can try to write something for this.

Updated version of the above code, which accounts for centering:
def find_equivalent_origins(sg_no, sampling_interval=0.1):
    """
    Determine fractional coordinates corresponding to equivalent phase
    origins for the space group of interest.
    Parameters
    ----------
    sg_no : int
        space group number
    sampling_interval : float (optional)
        search interval for polar axes as fractional cell length
    Returns
    -------
    eq_origins : array
        nx3 array of equivalent fractional origins
    """

    permitted = dict()
    for axis in ['x','y','z']:
        permitted[axis] = []

    # add shift for each symmetry element that contains an inversion
    sym_ops = gemmi.find_spacegroup_by_number(sg_no).operations()
    ops = [op.triplet() for op in sym_ops]
    for op in ops:
        if "-" in op:
            for element in op.split(","):
                if "-" in element:
                    if any(i.isdigit() for i in element):
                        axis, frac = element.split("+")
                        permitted[axis[1]].append(float(frac[0]) / float(frac[2]))
                    else:
                        permitted[element.split("-")[1][0]].append(0)

    # eliminate redundant entries and expand polar axes
    for axis in permitted.keys():
        permitted[axis] = np.unique(permitted[axis])
        if len(permitted[axis]) == 0:
            permitted[axis] = np.arange(0,1,sampling_interval)

    # get all combinations from each axis
    eq_origins = np.stack(np.meshgrid(permitted['x'],
                                      permitted['y'],
                                      permitted['z'])).reshape((3, -1)).T

    # add centering operations
    cen_ops = np.array(list(sym_ops.cen_ops)) / 24.0
    eq_origins = eq_origins + cen_ops[:,np.newaxis]
    eq_origins = eq_origins.reshape(-1, eq_origins.shape[-1])
    eq_origins = np.delete(eq_origins, np.where(eq_origins==1)[0], axis=0)

    return np.unique(eq_origins, axis=0)

I can confirm that this code recovers [0.5, 0.5, 0.5] which is the correct translation vector for my experimental data in space group 96. Here's the full list:
[[0.   0.   0.  ]
 [0.   0.   0.25]
 [0.   0.   0.5 ]
 [0.   0.   0.75]
 [0.   0.5  0.  ]
 [0.   0.5  0.25]
 [0.   0.5  0.5 ]
 [0.   0.5  0.75]
 [0.5  0.   0.  ]
 [0.5  0.   0.25]
 [0.5  0.   0.5 ]
 [0.5  0.   0.75]
 [0.5  0.5  0.  ]
 [0.5  0.5  0.25]
 [0.5  0.5  0.5 ]
 [0.5  0.5  0.75]]

This is very cool, @apeck12. I will modify this slightly and add it to the branch.
I think I might add a little optimization routine to refine the translation vector after the grid search. I think we will probably need that to get precise translation vectors for these groups like P61.

Also, @apeck12, if you want to polish this up and submit a PR, I'd obviously be totally cool with that.

On second thought -- my experiments lead me to believe this function is probably too rough to optimize easily.

Apparently the global minimum is hilariously narrow. I wonder if the problem better posed in real space (credit to @JBGreisman for suggesting)?

Curious, thanks for sharing this. I meant to look into doing optimization rather than a grid search some time ago but never got around to it, so this is interesting to see. We didn’t have better luck finding a common or crystallographic phase origin in real rather than reciprocal space, but our data might have been sufficiently different (very incomplete and not reduced) to be irrelevant to this case.
And thanks for adding the find_equivalent_origins function to phase.py. I just added another function for calculating the centric phase residuals in case that’s useful (and adjusted the test case in that script accordingly), but please feel free to revise or omit it as you see fit.

in the real space - would it be similar to Phenix find_alt_orig_sym_mate and CCP4 csymmatch?

@wojdyr, the goal is to align two electron density maps directly without a coordinate file. I cannot find a function that does this anywhere. If you know of one, please let us know. Thanks!

Come to think, I guess this problem is pretty much molecular replacement without the rotation search component.

I have a naive question: the equivalent origins here are not the same thing as equivalent origins mentioned in
http://legacy.ccp4.ac.uk/html/alternate_origins.html ?

Yes, those are exactly what we're trying to account for. The problem is that many groups, take for instance P21 have infinitely many alternate origins corresponding to translations along a particular axis. What @apeck12 's snippet does is infer the translation vectors mapping to possible alternate origins. If there are infinitely many possibilities, the function will construct a grid over them with some sampling rate. Does that answer your question?
Also, thanks for that webpage. I had been struggling to find something like that.

I am afraid the residuals don't look much better in real space.

These are real data from a pair of density modified experimental SAD maps from AutoSol. They are off by a [0.5, 0.5, 0.5] translation. The maps are from the same underlying data set merged with drastically different software. I line-scanned each of the axes keeping the other two fixed at their optimal values.
This is just to say that real space doesn't seem any more promising unfortunately.

I'm not quite sure I understand the distinction between red/black entries in the CCP4 table linked by @wojdyr. @apeck12's function for generating possible alternate origins seems to agree nicely with the table for sg19:
from reciprocalspaceship.algorithms.phase import find_equivalent_origins # phalign branch
import gemmi

find_equivalent_origins(gemmi.SpaceGroup(19))

Output:
array([[0. , 0. , 0. ],
       [0. , 0. , 0.5],
       [0. , 0.5, 0. ],
       [0. , 0.5, 0.5],
       [0.5, 0. , 0. ],
       [0.5, 0. , 0.5],
       [0.5, 0.5, 0. ],
       [0.5, 0.5, 0.5]])

However, for sg4,  it seems to only produce the first of the red entries:
find_equivalent_origins(gemmi.SpaceGroup(4), sampling_interval=0.25)

Output:
array([[0.  , 0.  , 0.  ],
       [0.  , 0.25, 0.  ],
       [0.  , 0.5 , 0.  ],
       [0.  , 0.75, 0.  ]])

Does the function miss some alternate origins? Or am I misunderstanding the distinction between red/black entries in the CCP4 table?

@JBGreisman, thanks for catching this discrepancy. Examining space group 4 specifically, the space group diagram for the primitive cell indicates only a polar y axis and no alternative origins:
http://img.chem.ucl.ac.uk/sgp/large/004ay1.htm
For the unconventional B-type lattice, there’s an additional translation that corresponds to one of the alternative origins listed in the CCP4 table:
http://img.chem.ucl.ac.uk/sgp/large/004dy1.htm
I haven’t encountered this before, but according to the fourth slide here, monoclinic B is a transformation of the primitive monoclinic lattice with twice the unit cell volume.
I’m not sure how to retrieve the full set of alternative origins listed in the CCP4 table from the space group’s symmetry operations alone. Perhaps the best approach would be to search the union of all valid nonpolar origins, amended for any polar axes, as @kmdalton described earlier in this issue. However, if some of the origins listed in that table correspond to unconventional lattices, would reindexing potentially be required to find the origin that aligns the phases?

This has been dormant for a while, but I just wanted to summarize where I think things stand:

Non-polar spacegroups: we can use the function above written by @apeck12 to generate the possible alternate origins (or we can just brute-force the superset of all possible ones), and the minimal residual between phases (or a similar metric) should give us the appropriate shift in origin.
Polar spacegroups: these are a bit trickier, because of the continuous axis (or axes). As shown by @kmdalton, the residuals are non-trivial to use for optimization, but the appropriate solution does show very distinct minimum.

We could start out by restricting this function to non-polar spacegroups. I know how to classify spacegroups as polar or not in sgtbx (sgtbx.space_group_info(i).number_of_continuous_allowed_origin_shifts()), but is there any way to do this in gemmi?

For reference, here is an updated link with alternate origins for each spacegroup: https://www.ccp4.ac.uk/html/alternate_origins_allgroups.html

Here's a short snippet that uses gemmi to classify spacegroups as polar, and checks it against the 530 spacegroup settings in sgtbx -- mostly adding this here in case it proves useful:
import gemmi
from cctbx import sgtbx

def is_polar(sg):
    """Returns whether spacegroup is polar"""
    sym_ops = sg.operations().sym_ops
    a  = np.array([ op.rot for op in sym_ops ])
    return ~(a < 0).any(axis=2).any(axis=0).all()

# Check against sgtbx
for sg in sgtbx.space_group_symbol_iterator():
    sgtbx_polar = sgtbx.space_group(sg).info().number_of_continuous_allowed_origin_shifts() > 0
    if sgtbx_polar != is_polar(gemmi.SpaceGroup(sg.universal_hermann_mauguin())):
        # Will only print spacegroup if there's a disagreement
        print(sg.universal_hermann_mauguin())

I realized that I had solved the 3D translation search problem with a simple iterative algorithm and forgot to post it publicly. I'm just going to put this code up here so it is public. @JBGreisman is working on putting this into rs.algorithms.
import numpy as np
import reciprocalspaceship as rs
import gemmi

reference_mtz_file = "RTSAD_HEWL_refine_25.mtz"
mtz_file = "overall_best_denmod_map_coeffs.mtz"

ds = rs.read_mtz(reference_mtz_file).stack_anomalous().join(rs.read_mtz(mtz_file)).dropna()

dmin = -1.

ds = ds[ds.compute_dHKL().dHKL >= dmin]

H = ds.get_hkls().astype('float32')
phi_ref = ds['PHIF-model'].to_numpy()
phi = ds['PHWT'].to_numpy()

class TranslationSolver:
    def __init__(self, H, phi_ref, phi):
        """
        H : millers as an N x 3 numpy array
        phi_ref : reference phases in degrees as a numpy array length N
        phi : other phases in degrees as a numpy array length N
        """
        self.H = H
        self.phi_ref = phi_ref
        self.phi = phi
        self.t = None
        self.D = (np.deg2rad(phi) - np.deg2rad(phi_ref)) / (2. * np.pi)

        # Current voxel size in fractional coordinates
        self.grid_size = 0.5

        # Current bounding box which contains the solution
        # 3 ranges in fractional coordinates
        self.a_range = [0., 1.]
        self.b_range = [0., 1.]
        self.c_range = [0., 1.]

    def _fit(self):
        T = np.mgrid[
            self.a_range[0] : self.a_range[1] : self.grid_size,
            self.b_range[0] : self.b_range[1] : self.grid_size,
            self.c_range[0] : self.c_range[1] : self.grid_size,
        ].reshape((3, -1)).T
        loss = np.sum(np.square((self.D[:,None] -<EMAIL_ADDRESS>- np.round((self.D[:,None] - H@T.T))), axis=0)
        tbest = T[np.argmin(loss)]

        # Update grid size for next iteration
        self.grid_size = self.grid_size / 2.

        # Update bounding box around solution
        a,b,c = tbest
        self.a_range = [max(a-self.grid_size, 0.), min(a+self.grid_size, 1.)]
        self.b_range = [max(b-self.grid_size, 0.), min(b+self.grid_size, 1.)]
        self.c_range = [max(c-self.grid_size, 0.), min(c+self.grid_size, 1.)]
        self.t = tbest

        return tbest

    def fit(self, maxiter=10):
        for i in range(maxiter):
            t = self._fit()
        return t

ts = TranslationSolver(H, phi_ref, phi)
tbest = ts.fit()

out = rs.read_mtz(mtz_file)
H = out.get_hkls()
out['PHWT'] = rs.utils.canonicalize_phases(out['PHWT'] + np.rad2deg(2*np.pi*H@tbest))
out.write_mtz("translated.mtz")

Just to close up this discussion -- as per the above, a good way to think of this is as an image registration problem. Conveniently, scikit-image supports this and the function, skimage.registration. phase_cross_correlation() works well for this task. It also works in reciprocal space using a 3D complex grid of the data if given space="fourier".
Though I have the outline of a solution here (#174), this seems better suited for rs-booster -- that's the library where we will be putting more applications-based tasks to separate them from the core library.


[Hotfix][hudi-sync] Refactor method up to parent-class

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
(For example: This pull request adds quick-start document.)
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
(Please pick either of the following options)
This pull request is a trivial rework / code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end.
Added HoodieClientWriteTest to verify the change.
Manually verified the change by running a job locally.

Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #2822 (5116b0c) into master (ab4a7b0) will decrease coverage by 9.33%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master    #2822      +/-   ##
============================================
- Coverage     61.96%   52.62%   -9.34%
- Complexity      335     3709    +3374
============================================
  Files            54      485     +431
  Lines          1993    23210   +21217
  Branches        235     2463    +2228
============================================
+ Hits           1235    12215   +10980
- Misses          638     9918    +9280
- Partials        120     1077     +957

Flag
Coverage Δ
Complexity Δ

hudicli
40.29% <ø> (?)
215.00 <ø> (?)

hudiclient
∅ <ø> (?)
0.00 <ø> (?)

hudicommon
50.70% <ø> (?)
1977.00 <ø> (?)

hudiflink
56.54% <ø> (?)
516.00 <ø> (?)

hudihadoopmr
33.33% <ø> (?)
198.00 <ø> (?)

hudisparkdatasource
72.06% <ø> (?)
237.00 <ø> (?)

hudisync
46.00% <50.00%> (?)
131.00 <0.00> (?)

huditimelineservice
64.36% <ø> (?)
62.00 <ø> (?)

hudiutilities
69.79% <ø> (+7.82%)
373.00 <ø> (+38.00)

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

The diff coverage is 50.00%.

@@             Coverage Diff              @@
##             master    #2822      +/-   ##
============================================
- Coverage     61.96%   52.70%   -9.26%
- Complexity      335     3729    +3394
============================================
  Files            54      485     +431
  Lines          1993    23292   +21299
  Branches        235     2491    +2256
============================================
+ Hits           1235    12277   +11042
- Misses          638     9924    +9286
- Partials        120     1091     +971

Flag
Coverage Δ
Complexity Δ

hudicli
40.29% <ø> (?)
215.00 <ø> (?)

hudiclient
∅ <ø> (?)
0.00 <ø> (?)

hudicommon
50.74% <ø> (?)
1985.00 <ø> (?)

hudiflink
56.84% <ø> (?)
527.00 <ø> (?)

hudihadoopmr
33.33% <ø> (?)
198.00 <ø> (?)

hudisparkdatasource
72.06% <ø> (?)
237.00 <ø> (?)

hudisync
46.00% <50.00%> (?)
131.00 <0.00> (?)

huditimelineservice
64.36% <ø> (?)
62.00 <ø> (?)

hudiutilities
69.84% <ø> (+7.87%)
374.00 <ø> (+39.00)

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


[INTEL MKL] Batch Matmul enhancements.

This PR removes CBLAS support for double and complex type. We still leave the support for float to enable broadcasting. For threadpool case we fall back to eigen when broadcasting is detected.

@penpornk I have made the changes requested. Please let me know.

@Srini511 Can you please resolve conflicts? Thanks!

@gbaned fixed it.

@penpornk Quick question regarding build configuration. I recently observed that matmul decompositions for native tensorflow + mkldnn contraction kernels vary greatly depending on whether what --march option is specified. Is this meant to happen this way? Do you have  recommendation on what to use for performance comparison purposes?


CKingX.haveibeenpwned version 0.5.1

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Pretty sure the reason it is taking this long on WinGetSvc-Validation (Installer Validation) is because of smartscreen in the way

@wingetbot waivers Add Validation-Executable-Error

Pretty sure the reason it is taking this long on WinGetSvc-Validation (Installer Validation) is because of smartscreen in the way

The validation failed because this is a CLI application. The service checks that running the executable after installation launches an app that doesn't immediately fail. For CLI applications we have to manually verify that an error is the expected result when running the app with no arguments.


fix: resize control plane alternative service log post suggestion

This command is wrong, it needs -p before each parameter.

/lgtm


Fix/docs styling

Resolves #199, resolves #200
Table of Contents
Uses the same line-height value as the docs navigation.

Before
After

Overlay
Add a sticky gradient overlay to the bottom of the navigation container.

Before
After

@tylandavis might want to add bottom margin to your sidebar content that is equal to or greater than the height of your gradient so that the user can scroll past it safely. The gradient also needs to be twice as large so that it never falls in between list items and goes unseen.

@jacobsfletch I increased the height on the gradient. As far as bottom margin goes, since its position: sticky the overlay's height is still there, and doesn't block the nav items when fully scrolled.

@tylandavis 👏👏👏


Feature Request: Integrate option to enable/disable display of yahoo messenger emoticons

Some of us still use the emoticons from yahoo messenger and type them directly in the message. I offer to contribute to this feature request, if you'd like. There is a chrome plugin that enables this in the facebook chat from facebook.com.
Please take the time to consider my request and please respond asap. I would like to develop it asap.
Thanks,
Daniel.

Actually the dev isnt maintaining this software. He is in the process of creating a new version with another framework (NWJS isnt very well documented nor very flexible). But during that time I decided to maintain my own fork. If you want to develop that option, please make a pull request to my fork so I can add it.

Is it still something needed.
I would be glad to have you on board to make that feature appear in the software.
Please see https://github.com/Aluxian/Facebook-Messenger-Desktop/issues/458

Hi @Sytten ,
Glad to be hearing from you. What are the terms for an official maintainer?
And besides the question, I would like to know more about the feature request implementation.
Looking forward to hearing from you asap
Warm Regards,
Daniel M. Colceag.

Not sure to understand what you mean by terms for official maintainer.
I develop the software until @Aluxian release the v2.
You can contribute if you want by doing some PR.

Sent from my iPhone

On 28 Jun 2016, at 03:51, Émile Fugulin<EMAIL_ADDRESS>wrote:
Not sure to understand what you mean by terms for official maintainer.
I develop the software until @Aluxian release the v2.
You can contribute if you want by doing some PR.

Reply to this email directly, view it on GitHub, or mute the thread.

Mostly to do some bug fixing and releasing the builds.
You have full control over the code.
Yes create the story. I don't have the time to implement it myself, but someone might do it (or you if you feel like it)

the story was to be reviewed by both and then discuss the details of implementation, ux/ui details, coding style and so on and so forth.
what do you say?
and after we finish discussing the details and I implement it, we review it and see if you or aluxian  like the imlementation, I could take some bug fixing on the list, if I have the time
Sent from my iPhone

On 28 Jun 2016, at 04:35, Émile Fugulin<EMAIL_ADDRESS>wrote:
Mostly to do some bug fixing and releasing the builds.
You have full control over the code.
Yes create the story. I don't have the time to implement it myself, but someone might do it (or you if you feel like it)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I like it!
The code is a bit messy right now, so a discussion about the implementation would be more than welcomed. I know the v2.0 will be an entirely new code, but I don't like the idea of increasing the mess for no reason.
You can join me on gitter if you want to chat, I'm always connected (Time zone East Canada).
I have a busy week, but maybe this week-end would be a good time for that.
Thanks for your help!


detect operator in strings

Describe the bug
Normatrix detect operator like '/' in strings declaration.
To Reproduce
static const char str_help[] = "USAGE\n./asm file_name[.s]\n

get :
sources/main.c:26: bad space for operator: /
 -> nope: sources/main.c (2)

Expected behavior
should detect nothing
Desktop:

OS: Linux Mint 20.3 Cinnamon
Version 5.2.7

a way arout could be to split your string like this:
static const char str_help[] = "USAGE\n./asm file_name[.s]\n"
"rest of the string";


TF Lite C++ standalone Interpreter

Feature request
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): Source
TensorFlow version (use command below): 1.4
Python version: 2.7.12
Bazel version (if compiling from source): 0.7.0
GCC/Compiler version (if compiling from source): 5.4.0
CUDA/cuDNN version: None
GPU model and memory: None
Exact command to reproduce: None

I deploy TensorFlow models to environments with very limited memory, and was very excited to see how tiny the TF Lite kernel is (several hundred kilobytes). I know the project is young, but I was wondering if there are any plans to support a standalone C++ build for the Lite architecture, for deployment to (Linux) environments that do not have a TensorFlow runtime.
For my purposes, this package should support the following:

Loading a model configuration
Loading input tensors
Invoking the model with input tensors
Storing the result in output tensors
Some serialization method for the input- and output tensors

I would like to know:

Is this a possible feature for TF Lite?
What would the scope be of implementing this?
Are there any plans to do so?

I'm not exactly sure what you mean by "standalone c++" build, but I assume independent of TensorFlow tooling is wha tyou mean.
There is already a makefile that builds basically just the interpreter that @petewarden made. You may need to modify the ops to be driven by the "reference ops" rather than the "optimized ops" to avoid using neon. But this should be definitely possible, and something we hope to support better in the future.

My goal is to use NDK to compile C++ static libraries that can be called by Android. I don't quite understand what you're saying "modify the ops to be driven by the "reference ops" rather than the "optimized ops" to avoid using neon. "  PS: I refer to /tensorflow/contrib/lite/makefile

hi, asslle:
My goal is to use NDK to compile C++ static libraries that can be called by Android. I don't quite understand what you're saying "modify the ops to be driven by the "reference ops" rather than the "optimized ops" to avoid using neon. " PS: I refer to /tensorflow/contrib/lite/makefile

aselle's point is that you can change the makefile to build a static library for Android, or any other system. However, a lot of the TF Lite operators depend on neon, which is available on Android but not on every other system. If you don't have neon (or a suitable replacement) you will have to manually edit the source code in kernels/*.cc. (On linux you may sometimes rely on sse as a neon replacement)

where can i find the makefile made by @petewarden ?

@WellDone2094 https://github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/lite/Makefile

i don't know how i missed that

@JulianNeeleman, is your question sufficiently answered?

@aselle Yes, thank you.

The link to the TF lite makefile posted by @JulianNeeleman is no longer available. Can someone update it with the new link? It is probably shifted here - https://github.com/tensorflow/tensorflow/tree/master/tensorflow/contrib/lite/tools/make

you can find solution in my git

@wantt Thanks! Will take a look.

Where can I find a version of the TensorFlow Lite makefile?

@wantt Thanks for git repo. On linux, I would able to compile it with -no-pie option. Could you please tell me how did you build libtensorflow-lite.a library? With objdump i can see that its for arch=x86_64 did you compile it on linux or macOS?
Has anyone tried on macOS?

I could able to run it on macOS, my working repo is here: https://github.com/milinddeore/TfLite-Standalone-build-Linux-MacOS


Update config.cpp

Added versions of the Contact DLC radio backpacks that have increased carry capacity to fit the AN/PRC-117F radio from the ACRE mod.

Looks good to me.


up & down arrow keys scroll window as well as navigating the datepicker

preventing window scrolling while open would be great.

Thanks @tm-fari for reporting this. This is not expected, and would be fixed in the next release.
Keep in touch! :)

Closed by #91.


wondering when will the new version be released for influxdb v1.8.x?

I am currently using influxdb v1.8.10 and there is no new 1.x version has been released for about 2 years. I read I am still interested in using 1.x version of influxdb. Therefore I am wondering: 1. when will the bug fix / security version of influxdb be released for 1.x version? 2. Is the update path of updating influxdb 1.x version to 2.x version clear? Is there a way that we can update influxdb 1.x to 2.x easily without data migration?

See https://github.com/influxdata/influxdb/issues/24426#issuecomment-1810390991


CleanWebpackPlugin 这个插件报错 npm直接报错 is not a constructor

Bug report

What is the current behavior?
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?

Other relevant information:
webpack version:
Node.js version:
Operating System:
Additional tools:

It looks like you just deleted/ignored our lovely crafted issue template. It was there for good reasons. Please help us solving your issue by answering the questions asked in this template. I'm closing this. Please open a new issue with filled issue template. Also make sure your issue is not a question. Questions should be posted on Stack Overflow.


Modify .gitignore file to ignore .idea folder and Android platform detection code

Modify .gitignore file to ignore .idea foler and Android platform detection code refine.

gimme a separate pull request for "modify .gitignore to ignore .idea folder".

ok, I'll create a separate branch and pull to your branch other than master, right?

you can create a separate branch from the master, and plz send another pull request to my master.
Thanks


Drug exposure Checks seem to be incompatible with some RDBM systems (Synapse Analytics dedicated instance)

I am trying to run CdmOnboarding::cdmOnboarding against a Synapse Analytics dedicated instance database.
During the DrugExposureDiagnostics I get this error:
Execution of DrugExposureDiagnostics failed: Error in rJava::.jcall(resultSet, "Z", "next"): com.microsoft.sqlserver.jdbc.SQLServerException: USE statement is not supported to switch between databases. Use a new connection to connect to a different Database.
This seems to be caused by
cdm <- CDMConnector::cdm_from_con(connection, cdm_schema = cdmDatabaseSchema)
at the R/DedChecks.R.
If I am correct, this database, although it resembles sqlserver in many ways, does not support USE statement.
We need to find a way to execute checks without requiring USE statement.

Synapse Analytics is currently not supported by CdmConnector, which is needed for the DrugExposureDiagnostics results


Using VIEW_STYLE_GRID

Is there documentation on how to use or customized the VIEW_STYLE_GRID?

I was able to display an image on the Grid view when I set the field "name" into "image" but when I change it to something else it doesn't work. Does it only work on name: image ?
It is working when I upload the image in here

{label: "Main Image",
name: 'image',
widget: 'image',
required: false,
tagname: ''}

But when I try it here

It does not work.
What I tried to do is changing the EntryListing.js into these 3 different approaches. But nothing works.
const imageField = selectInferedField(collection, 'more_assets.desktop_image');
const imageField = selectInferedField(collection, 'more_assets[desktop_image]');
const imageField = selectInferedField(collection, 'more_assets[0]desktop_image');

Very good question @henval2020.
We're inferring the image field so any one of the values here:
https://github.com/netlify/netlify-cms/blob/460d1e6835af122f86278b4b2f9c2e3b7bc91488/packages/netlify-cms-core/src/constants/fieldInference.tsx#L63
should work.
I think a good approach will be to implement https://github.com/netlify/netlify-cms/issues/2868.
Another thing we should do is to make inference configurable (related to https://github.com/netlify/netlify-cms/issues/445).
Closing, please comment if more information is needed

Thank you so much @erezrokah. It works now, I was able to retrieve the newly added fields like this.

By updating this part

Last question for this if I may,
I tried to fill out the mobile image instead of desktop but it seems like the imageField returns desktop_image value on EntryListing.js

Then I tried this (setting a hardcoded mobile_field)
const imageField = selectInferedField(collection, 'image'); into const imageField = 'mobile_image'
Then mobile image was retrieved.
I also noticed that when I switch the position of the fields on my config.yml, without changing the const imageField = selectInferedField(collection, 'image'); Mobile images were retrieved.

The main question is, is there a way that I can set the desktop_image as the fallback of an image when mobile_image is not available (no image were uploaded on mobile_image field)
I also tried setting the field fallbackToFirstField to true but it doesn't do the trick. Am I missing something here

The inference code only looks at field names in the configuration and not their values, so the solution has to be somewhere else.
You could create a new function called selectInferedFields to return all possible fields, then use it here:
https://github.com/netlify/netlify-cms/blob/04ccb56e668ce24372675b8e937c773215a6d2d6/packages/netlify-cms-core/src/components/Collection/Entries/EntryListing.js#L43
Then implement the fallback here:
https://github.com/netlify/netlify-cms/blob/04ccb56e668ce24372675b8e937c773215a6d2d6/packages/netlify-cms-core/src/components/Collection/Entries/EntryCard.js#L127
Doing https://github.com/netlify/netlify-cms/issues/2868 however can provide more flexibility in the future

Awesome! I missed checking the EntryCard.js. Thank you so much for your help @erezrokah . I was able to create a fallback with this.

It's messy right now, but I'll try to refactor this. cheers!


Running locally in Windows

Hello,
I'm trying to run this locally in windows.
I have Kubernetes v1.16.5 running using Docker Desktop.
After cloning the repo, setting up username, password in bbb-basic-auth-secret.yaml and jitsi credentials in jitsi-secret.yaml, I have been getting this error

So, what can I do regarding this error?
If anyone could help me out with this, I'd highly appreciate it.
Another question that I have is, how can I access it after successfully running it?
What I mean is, on which port I can access the jitsi web?
e.g. localhost:80 ? or localhost:8443?
Thank you
Ayush.

Hi Ayush,
Let me clarify a few things:

We tested our setup against Kubernetes v1.17.2 and v1.17.6, so maybe you will run into issues regarding your v.1.16.5 cluster
All files named bbb-* are not necessary for this setup as they are just there to set up monitoring of an affiliated project (https://github.com/schul-cloud/bbb-deployment). You can delete them and remove them from your kustomization.yaml files. I have also added this explanation to the readme now (on the dev branch).
I highly doubt that it is easily possible to run this setup on a local machine. For example we use services of type LoadBalancer which are typically handled by your Cloud provider. Another problem you will run into is lacking resource requirements. If you take a look at the minimal resources required for this setup (checkout the Deployments, StatefulSets and DaemonSets) you will realize that this setup is too heavy for running even on a high end PC. One more problem you will face is the fact that our setup uses a public domain. You will have to adjust the ingresses to serve from local.

Nevertheless, regarding your error: Where do you see this error? When you apply the manifests? Or in the logs of a certain container? It seems like your container cannot resolve the cert-manager webhook.
If you really manage to run this setup locally I'd appreciate any valuable input about how to get it up and running.
Jitsi will be available through the HAProxy Ingress (HTTPS/443).


[Hub Generated] Review request for Microsoft.Maps to add Creator resource under version preview/2020-02-01-preview

This is a PR generated at OpenAPI Hub. You can view your work branch via this link.
Contribution checklist:

[x] I have reviewed the documentation for AME onboarding.
[x] Validation tools have been run on swagger spec(s) updated and errors have all been fixed with this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.

adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Can one of the admins verify this patch?

Private Atlas resource will be renamed to Creator. This change adds Creator operations to replace existing Private Atlas operations. We will deprecate Private Atlas later. Both resource types exist in the same "preview" API version.

Hi, Pls fix linter error.

Hi @ruowan, linter errors suggest that new APIs should use the standard response message. I have referenced types from common-types/resource-management/v1/types.json#/definitions/ErrorResponse which have the same structure as required in https://github.com/Azure/azure-resource-manager-rpc/blob/master/v1.0/common-api-details.md#error-response-content.
So, expected response schema is compliant with guidelines. Is that a tool issue? Or am I missing something?
Could you please advise?

You could reference v2 types, v2 has fixed this issue. Here is a link https://github.com/Azure/azure-rest-api-specs/blob/master/specification/common-types/resource-management/v2/types.json.

/azp run automation - sdk

You could reference v2 types, v2 has fixed this issue. Here is a link https://github.com/Azure/azure-rest-api-specs/blob/master/specification/common-types/resource-management/v2/types.json.

Hi @ruowan, after referencing v2 type I've got PropertyRemoved failures. Is linter getting diff to the last commit or to master?

/azp run automation - sdk

hi @ruowan ,
I confirmed the false positive using diff tool
https://portal.azure-devex-tools.com/tools/diff/eyJzb3VyY2VVcmwiOiJodHRwczovL2dpdGh1Yi5jb20vQXp1cmUvYXp1cmUtcmVzdC1hcGktc3BlY3MvYmxvYi9tYXN0ZXIvc3BlY2lmaWNhdGlvbi9tYXBzL3Jlc291cmNlLW1hbmFnZXIvTWljcm9zb2Z0Lk1hcHMvcHJldmlldy8yMDIwLTAyLTAxLXByZXZpZXcvbWFwcy1tYW5hZ2VtZW50Lmpzb24iLCJ0YXJnZXRVcmwiOiJodHRwczovL2dpdGh1Yi5jb20vQXp1cmUvYXp1cmUtcmVzdC1hcGktc3BlY3MvYmxvYi82M2M0MDAxMWExMDdkMTFhM2NiYmM3NWYzMGE2NmNhMjkwMDM2MTg1L3NwZWNpZmljYXRpb24vbWFwcy9yZXNvdXJjZS1tYW5hZ2VyL01pY3Jvc29mdC5NYXBzL3ByZXZpZXcvMjAyMC0wMi0wMS1wcmV2aWV3L21hcHMtbWFuYWdlbWVudC5qc29uIn0=
I'm using v1/types.json on existing APIs, with no changes to them. I'm using v2/types.json only on added APis.
Could you please take a look today?

Thanks for your feedback. It's actually a false positive. We will fix our tools later.
Could you pls change all v1 to v2?  The only difference between v1 and v2 is /definitions/ErrorResponse.  Your service is a preview version. If no customer impact, better to migrate it to v2.

Thanks for your feedback. It's actually a false positive. We will fix our tools later.
Could you pls change all v1 to v2? The only difference between v1 and v2 is /definitions/ErrorResponse. Your service is a preview version. If no customer impact, better to migrate it to v2. We allow this kind of breaking change.

Agree that it is necessary to migrate v1 to v2, but we should make a new API version. Current preview version is used by customers and there would be impact if I change the rest of APIs from v1 to v2.

/azp run automation - sdk

Thanks for your feedback. It's actually a false positive. We will fix our tools later.
Could you pls change all v1 to v2? The only difference between v1 and v2 is /definitions/ErrorResponse. Your service is a preview version. If no customer impact, better to migrate it to v2. We allow this kind of breaking change.

Confirmed that v2 matches what our RP actually returns. Updated all references to v2 types.
@ruowan , could you please complete this PR?


SwiftGif: This image named "xxx" does not exist

i just cant load anything  ,,
xcode :Version 9.0 (9A235)

Solved taking the gif out of the assets folder

What do you mean by "taking the gif out of the assets folder" ???


Updates via the Microsoft Store break Visual Studio 2022 due to font sharing

Windows Terminal version
1.14.1962.0
Windows build number
10.0.19044.0
Other Software
Visual Studio 2022 17.2.6
Steps to reproduce
Whenever Windows Terminal updates via the MS Store it modifies the installed Cascadia fonts on the machine. As Visual Studio 2022 also uses these fonts, this breaks VS (all code windows render as white with no text) and VS can only be fixed by restarting the machine. This is an extremely poor developer/productivity experience.
I am not filing this as a VS bug because it is Terminal that is pulling the rug out from under VS and causing the latter to fall over. VS may need to be fixed to be more tolerant, but that is the Terminal team's conversation to have with the VS team, not mine.
Possibly related to/dupe of #10482, which was closed because... the team doesn't care about developer experience? I dunno.
Expected Behavior
VS2022 works.
Actual Behavior
VS2022 does not work.

closed because... the team doesn't care about developer experience?

We've pursued this issue to the ends of the earth--seriously, the people who own packaged fonts are tired of hearing from us--but as it stands today we're just in a bind.
Terminal ships with Windows 11, and the inclusion of Cascadia in the package has made Cascadia a de-facto system font. If we remove the font from the package, we're risking breaking any application that has taken a dependency on it as a system font (VS, Teams comes to mind, Stack Overflow and a few other sites have also chosen to depend on it, but that's neither here nor there as they're a website.) If we don't remove the font from the package, older versions of the OS have an issue with the font being locked during upgrade.
If we remove it from the package and add it as a dependency package (there's no such thing as an optional dependency, for what it's worth), we complicate distribution and OS ingestion. We've gotten enough bug reports from people who cannot find the VCLibs package that we ship as part of our own releases to know that installing dependency packages is nearly a no-go for some folks. The downstream effect is that everybody gets a deployment error because they haven't installed the font first.
I guess we could just remove it, get it into an OS update (Windows 12?), and take the heat.
Of note is that it's fixed in Windows 11, and reports like yours help us work up the organizational will to get that fix backported to Windows 10.


Test fails on Unix with undefined symbol: GSS_C_NT_HOSTBASED_SERVICE

Sorry I am new to node/npm/jspm, .. and need some kerberos/gss functionality.  I installed this package, but when I try to run the test I get:
...../node_modules/kerberos/build/Release/kerberos.node: undefined symbol: GSS_C_NT_HOSTBASED_SERVICE
at Error (native)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)
at Module.require (module.js:365:17)
at require (module.js:384:17)
at Object. (/home/markus/sources/github-master/node_modules/kerberos/lib/kerberos.js:1:78)
at Module._compile (module.js:460:26)
at Object.Module._extensions..js (module.js:478:10)
at Module.load (module.js:355:32)
at Function.Module._load (module.js:310:12)

this module is specifically made for mongodb kerberos support. no tests will work when not having the correct environment. kerberos testing is non-trivial unfortunately so it's a manual affair for now.


Backend interface should not expose internal storage types

The current Backend API is exposing the way that data happens to be stored in our current backends. I would expect privateKeys to return something like Array<{privateKey: string, signingAddress: string}>, like an ORM would do.
Originally posted by @lalexgap in https://github.com/statechannels/monorepo/pull/1523

Curious if we feel there has been an improvement here / if this still warrants an issue

Curious if we feel there has been an improvement here / if this still warrants an issue

IMO: no and yes


how to handle errors?

Hello,
Does this package support error handling to play nicely w/ember/json-api? I am noticing that my django rest backend is returning errors in 2 formats depending if it's an ember-data request or an ember-ajax request and I'm having to hack together error objects in the client...
With ember-data I am getting an error response which is an Array of objects that look like (this is what it looks like in chrome inspector):
Object {source: Object, detail: "Please enter a url.", title: "Invalid Attribute"}
detail: "Please enter a url."
source: Object
    pointer: "/data/attributes/source_url"
title: "Invalid Attribute"

However in this case my only reference to the field is under pointer which makes it a hack to grab that for my use case.
The actual network response is showing:
{title: ["Please enter a title."], source_url: ["Please enter a url."]}
source_url: ["Please enter a url."]
    0: "Please enter a url."
title: ["Please enter a title."]
    0: "Please enter a title."

So in this case how can I improve this? ideally id like to keep ember-specific formatting (e.g. json-api) in the client so my api can remain client-agnostic... how are you guys handling this? how can I improve? thanks!

The object your getting in ember-data is JSON API format.  (the adapter does this intentionally)
See docs for JSON API error objects
ember-data wants the errors formatted this way.  The errors should propagate to the object's errors attribute automatically for you.  e.g.:
newUser.save().then(user => {
  console.log('success!');
}).catch(response => {
  console.log(newUser.get('errors'));
});

@dustinfarris it does populate the errors property but it is in the format as above which makes it hard/ugly to strip out the field name...
      onError = (response)=>
        # Delete the record from the client since it failed.
        content.deleteRecord()
        # Cycle through errors and set the property.
        errors = []
        for error in response.errors
          # For now we must strip the field name from the pointer.
          _f = error.source.pointer.split('/')
          field = _f[_f.length - 1]
          # Push the error key and value into the array.
          errors.push key: field, value: error.detail
        @set 'errors', errors

ideally I wouldn't need that _f = ...

@erichonkanen you're reading errors from the response object, which gives you the raw JSON API object.  The idea is that ember-data will interpret that and propagate it to your ember data model that you were trying to save.  See above I am getting errors from the model, not the response.  Try that and see if you get a better result.

ok let me check..

@dustinfarris if I do content.errors how do I know which fields have errors?

@erichonkanen i'm not sure what the best javascript is to inspect the errors.  i usually just refer to them in the templates, e.g. (excerpt of a signup form—email field):

  {{#each model.errors.email as |error|}}

{{error.message}}

  {{/each}}
  {{input id="id_email" value=model.email type="email" class="email form-control" viewName="email"}}

@dustinfarris ok closing this for now... for my use case I need to grab the data inside an action so ideally there'd be a cleaner way to get the field name but its not mission critical! thanks

hello, I'm back.. so I'm running into this again and trying to wrap my head around error handling. We are running into an issue where we need to also include the http status_code and an application-level code key in the json-api error. How should we handle this? Do I need to modify django-rest-framework to include that information and if so in what format?


Add missing words in CONTRIBUTING

^^

@MikeMcC399 Feel free to leave this open, we might merge it.

@Ein-Tim
Sorry, I forgot about this one when I deleted my fork.
I'm really out of the game now, so it is over to you guys.

@MikeMcC399 Okay, no worries, I'll cherry pick your commit later and provide a new PR with it.


Revised GF tracer transport, wet removal and evaporation

This is zero diff in the default configuration (USE_TRACER_TRANSPORT = 0).

*********** Important ***********

GOCART default choice : aerosols are NOT friendly to MOIST
This updated revision is no longer ZERO DIFF with JASON-3_2, since radiatively active gases are being transported differently.

Changes:
1. GEOS_MoistGridComp.F90

   * GF and UW_sh schemes change _all_ tracers friendly to MOIST

@srfreita  Shouldn't  1. be corrected to: GOCART tracers are by default friendly to MOIST, except aerosols and their gas precursors.

@srfreita  you can edit the original comment and perhaps we can then delete the ones where we pointed out that it needs a correction.

@srfreita you can edit the original comment and perhaps we can then delete the ones where we pointed out that it needs a correction.

I changed the original comment.

I am closing this pull request since it is outdated


changeState documentation

Hi
Sorry for the not well formatted issue.
In the docs the example code has the wrong event name:
[switch-handle-width]="handleWidth"
          [switch-base-class]="'bootstrap-switch'"
          (onChangeState)="onChange($event)">
  </bSwitch>

I know the readme says at two other places that its actually changeState, but it still took me 1 hour to fix this issue, since I was only looking at the example. :/
I would encourage using the new notation in the example too.
Thx!
Best,
Patrik

Sorry @bpatrik
I totally forgot to do it.


ci: let make all command test all the quickstart docs

What would you like to be added:

Let make all command test all the quickstart docs.
make all don't have to build layotto, since we already build it when running the quickstart tests.

Why is this needed:
Contributors can make all and find the CI issues locally before pushing to remote repo

subtask of https://github.com/mosn/layotto/issues/532


GEMINI data not updated?

It looks like the GEMINI data were not updated:
IOError: Gemini cannot open this annotation file: /opt/bcbio/gemini_data/dbsnp.151.20180423.tidy.vcf.gz.

This makes it non-functional, because the program was updated, but the data files weren't. upgrade --data --datatarget gemini will not download the new data, suggesting it needs an update.

Luca;
Thanks for the report. The latest bcbio versions moved to using the vcfanno/vcf2db based approach for gemini annotations to synchronize across build 37 and 38 with annotations supported. As a result it no longer automatically runs the gemini update for the old approach. You can run:
gemini update --dataonly

to re-sync those if you're using the older approach for running, but we don't do it by default to prevent all the duplicated larger files. Hope that fixes it for you.

I did exactly that, mainly because I've had a few issues with vcf2db in the past. This works for now, thanks.


Control generated class access modifier for C#

Add --class-access-modifier flag.
Fixes #4788.

@microsoft-github-policy-service agree

Enum support is still missing,
Please review if the current solution makes sense.


Update docs build script [skip ci]

This PR removes a variable that is no longer necessary for docs builds after the calver transition.

@gpucibot merge


[macos] Crash when requesting speech permission when running from VS Code

I had a crash issue when run on mac app. This issue happended 100% on debug & release mode
This is the minimal test project
I just create a project with flutter create speech_test and setup platform channel.
The issue happens in mac os platform channel when I asking speech permission on this line:
SFSpeechRecognizer.requestAuthorization()

Logs
flutter run --v
{"CFBundleName":"speech_test","DTXcode":"1210","DTSDKName":"macosx10.15","NSHumanReadableCopyright":"Copyright © 2020
com.example. All rights
reserved.","DTSDKBuild":"19G68","CFBundleDevelopmentRegion":"en","CFBundleVersion":"1","BuildMachineOSBuild":"19H2","NS
PrincipalClass":"NSApplication","CFBundleIconName":"AppIcon","CFBundlePackageType":"APPL","CFBundleIconFile":"AppIcon",
"CFBundleSupportedPlatforms":["MacOSX"],"DTPlatformName":"macosx","CFBundleShortVersionString":"1.0.0","NSMainNibFile":
"MainMenu","CFBundleInfoDictionaryVersion":"6.0","CFBundleExecutable":"speech_test","DTCompiler":"com.apple.compilers.l
lvm.clang.1_0","CFBundleIdentifier":"com.example.speechTest","DTPlatformVersion":"10.15.6","DTXcodeBuild":"12A7403","LS
MinimumSystemVersion":"10.15","DTPlatformBuild":"12A7403","NSSpeechRecognitionUsageDescription":"App needs to access
speech"}
[+3280 ms] Observatory URL on device: http://<IP_ADDRESS>:52507/akuZ6vCfY4w=/
[   +9 ms] Caching compiled dill
[ +155 ms] Connecting to service protocol: http://<IP_ADDRESS>:52507/akuZ6vCfY4w=/
[   +2 ms] Launching a Dart Developer Service (DDS) instance at http://<IP_ADDRESS>:0, connecting to VM service at
http://<IP_ADDRESS>:52507/akuZ6vCfY4w=/.
[ +300 ms] DDS is listening at http://<IP_ADDRESS>:52509/29R3AcOpmrs=/.
[ +110 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:52507/akuZ6vCfY4w=/
[        ] Waiting for macOS to report its views...
[   +8 ms] Waiting for macOS to report its views... (completed in 7ms)
[  +14 ms] DevFS: Creating new filesystem on the device (null)
[  +21 ms] DevFS: Created new filesystem on the device
(file:///var/folders/11/wq1vn8yd2739xsmmzbq3m06w0000gn/T/com.example.speechTest/speech_testyBCWdV/speech_test/)
[   +2 ms] Updating assets
[ +115 ms] Syncing files to device macOS...
[   +2 ms] <- reset
[        ] Compiling dart to kernel with 0 updated files
[   +3 ms] <- recompile package:speech_test/main.dart 74867277-0531-4bf0-9ce9-1b8ac3ec4b54
[        ] <- 74867277-0531-4bf0-9ce9-1b8ac3ec4b54
[ +137 ms] Updating files.
[        ] DevFS: Sync finished
[        ] Syncing files to device macOS... (completed in 144ms)
[        ] Synced 0.0MB.
[   +1 ms] <- accept
[   +7 ms] Connected to _flutterView/0x7f9ec107d820.
[   +2 ms] Flutter run key commands.
[   +2 ms] r Hot reload. 🔥🔥🔥
[   +1 ms] R Hot restart.
[        ] h Repeat this help message.
[        ] d Detach (terminate "flutter run" but leave application running).
[        ] c Clear the screen
[        ] q Quit (terminate the application on the device).
[        ] An Observatory debugger and profiler on macOS is available at: http://<IP_ADDRESS>:52509/29R3AcOpmrs=/
[+8221 ms] Service protocol connection closed.
[        ] Lost connection to device.
[   +3 ms] DevFS: Deleting filesystem on the device
(file:///var/folders/11/wq1vn8yd2739xsmmzbq3m06w0000gn/T/com.example.speechTest/speech_testyBCWdV/speech_test/)
[ +259 ms] Ignored error while cleaning up DevFS: TimeoutException after 0:00:00.250000: Future not completed
[   +3 ms] "flutter run" took 50,296ms.
[ +236 ms] ensureAnalyticsSent: 226ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hook priority 4
[   +6 ms] Shutdown hooks complete
[        ] exiting with code 0

flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.194, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-US)
    • Flutter version 1.24.0-8.0.pre.194 at /Users/ductran/Documents/Soft/flutter
    • Framework revision 018467cdb1 (24 hours ago), 2020-11-11 02:04:03 -0500
    • Engine revision 81f219c59c
    • Dart version 2.12.0 (build 2.12.0-31.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/ductran/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/ductran/Library/Android/sdk
    • ANDROID_SDK_ROOT = /Users/ductran/Library/Android/sdk
    • Java binary at: /Applications/Android Studio 3.5 Preview.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio 3.5 Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • macOS (desktop)  • macos      • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 86.0.4240.193

• No issues found!

@ductranit
Tried your sample project on latest Master in debug and release mode and didn't see the crash. The app ran properly without any issues. Tapped on FAB which asks for permission. Granted it and was able to tap on FAB that results in flutter:true printed in console as below:

flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.208, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.24.0-8.0.pre.208 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 15d2d8a875 (11 hours ago), 2020-11-11 20:04:03 -0500
    • Engine revision d97a81c889
    • Dart version 2.12.0 (build 2.12.0-31.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.15.1

[✓] Connected device (4 available)
    • iPhone SE (2nd generation) (mobile) • 6C85835D-FBFD-4AB3-8DE8-B4FAD35E5367
      • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)
    • macOS (desktop)                     • macos
      • darwin-x64     • Mac OS X 10.15.4 19E2269 darwin-x64
    • Web Server (web)                    • web-server
      • web-javascript • Flutter Tools
    • Chrome (web)                        • chrome
      • web-javascript • Google Chrome 86.0.4240.193

• No issues found!

Can you provide more details around the issue like a video ? What happens after you grant permission ?

@darshankawar weird that it doesn't happen on yours. My friend can preproduce it.
It doesn't show any permission modal, just crash
Here is the video

@ductranit
I got the permission for the first time when I installed the app and didn't see the issue. Can you check if there are any settings disabled in your machine ?

@darshankawar well, if I know what it is, I already fixed it. Do you have any clues about this issue?
The weird thing is that I can run the project Runner.xcworkspace in xcode without any issue (it can show the permission & I can accept it)
So I think it's something relate to flutter build tool (had the same issue with flutter run )

Hi @darshankawar. I get the crash on my system.
My flutter doctor

[✓] Flutter (Channel master, 1.24.0-8.0.pre.60, on Mac OS X 10.15.7 19H2 darwin-x64, locale
en-AU)
• Flutter version 1.24.0-8.0.pre.60 at /Users/adrian/Projects/flutter
• Framework revision 91b82347af (9 days ago), 2020-11-03 12:48:25 -0800
• Engine revision<PHONE_NUMBER>
• Dart version 2.12.0 (build 2.12.0-3.0.dev)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
• Android SDK at /Users/adrian/Library/Android/sdk
• Platform android-29, build-tools 29.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Xcode 12.2, Build version 12B45b
• CocoaPods version 1.8.1
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 3.5)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin version 41.0.2
• Dart plugin version 191.8593
• Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
[✓] VS Code (version 1.51.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.16.0
[✓] Connected device (3 available)
• macOS (desktop)  • macos      • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
• Web Server (web) • web-server • web-javascript • Flutter Tools
• Chrome (web)     • chrome     • web-javascript • Google Chrome 86.0.4240.198
! Error: iPhone is not connected. Xcode will continue when iPhone is connected. (code -13)
! Error: iPad (4) is not connected. Xcode will continue when iPad (4) is connected. (code
-13)
• No issues found!

Hi @adriandewitts @ductranit
I can reproduce this issue the crash on the latest master when running from Flutter on VS Code only
VS Code

Xcode
Meanwhile, when running from Xcode, it asks for permission, and the app runs after granting it

Terminal works fine too

Code Sample
https://github.com/flutter/flutter/issues/70374#issue-741536937

flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.268, on macOS 11.0.1 20B29 darwin-x64,
    locale en-GB)
    • Flutter version 1.24.0-8.0.pre.268 at /Users/tahatesser/Code/flutter_master
    • Framework revision e803b13f01 (6 hours ago), 2020-11-16 22:55:59 -0800
    • Engine revision f0e80100f7
    • Dart version 2.12.0 (build 2.12.0-49.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/tahatesser/Code/sdk
    • Platform android-30, build-tools 30.0.2
    • ANDROID_HOME = /Users/tahatesser/Code/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.10.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (4 available)
    • RMX2001 (mobile) • EUYTFEUSQSRGDA6D • android-arm64  • Android 10 (API 29)
    • macOS (desktop)  • macos            • darwin-x64     • macOS 11.0.1 20B29
      darwin-x64
    • Web Server (web) • web-server       • web-javascript • Flutter Tools
    • Chrome (web)     • chrome           • web-javascript • Google Chrome
      86.0.4240.198

• No issues found!

@TahaTesser yes, it's same for me. It only happens on vs code. Xcode works well but I can't debug dart code on it

@TahaTesser Yes, same for me.

The minimal test project link doesn't seem to work anymore. @ductranit could you provide the code you used to reproduce this issue, or provide the steps taken to do the same?

@exaby73 wow, incredible that you come back to me on this bug after 2 years.
I don't have that sample now, but I can remember the steps to reproduce

Create new flutter project (with macOS platform)
Create macOS platform channel
In macos channel add a method to ask for speech permission like this

import Speech

public func authorize() {
          SFSpeechRecognizer.requestAuthorization { status in
            }
       }

Then call this method channel in flutter
Make sure to add NSSpeechRecognitionUsageDescription in Info.plist
Run the app on vscode and you will see the crash (it works when run on xcode)

Can you provide a public Github repo that includes the code needed to verify the bug against the latest versions?

@exaby73 here you are https://github.com/ductranit/flutter_speech_crash
I can still reproduce this issue on flutter stable 3.3.9

I can reproduce it as well :) Thank you for the sample. I was having some trouble trying to come up with one. I'll update the labels to reflect the same. To note: I can also reproduce this on the master channel 3.7.0-9.0.pre.11 as well as 3.3.10

For posterity, here's the reproducible sample code:
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;
  static const platform = MethodChannel('test_channel');
  Future<void> _incrementCounter() async {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });

    await platform.invokeMethod('askPermission');
  }

  @override
  Widget build(BuildContext context) {
    // This method is rerun every time setState is called, for instance as done
    // by the _incrementCounter method above.
    //
    // The Flutter framework has been optimized to make rerunning build methods
    // fast, so that you can just rebuild anything that needs updating rather
    // than having to individually change instances of widgets.
    return Scaffold(
      appBar: AppBar(
        // Here we take the value from the MyHomePage object that was created by
        // the App.build method, and use it to set our appbar title.
        title: Text(widget.title),
      ),
      body: Center(
        // Center is a layout widget. It takes a single child and positions it
        // in the middle of the parent.
        child: Column(
          // Column is also a layout widget. It takes a list of children and
          // arranges them vertically. By default, it sizes itself to fit its
          // children horizontally, and tries to be as tall as its parent.
          //
          // Invoke "debug painting" (press "p" in the console, choose the
          // "Toggle Debug Paint" action from the Flutter Inspector in Android
          // Studio, or the "Toggle Debug Paint" command in Visual Studio Code)
          // to see the wireframe for each widget.
          //
          // Column has various properties to control how it sizes itself and
          // how it positions its children. Here we use mainAxisAlignment to
          // center the children vertically; the main axis here is the vertical
          // axis because Columns are vertical (the cross axis would be
          // horizontal).
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times:',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ), // This trailing comma makes auto-formatting nicer for build methods.
    );
  }
}

@DanTup Could this be an issue for the VSCode plugin?  (Note that this issue is really old, so we're not sure if it still occurs.)

I can reproduce this with the repo above. I don't think there's anything in the Dart-Code plugin itself that would contribute to this, but I have have weird issues with MacOS permissions in the past (for example VS Code randomly popping up asking for permission to something, I say no, and then months later something tried to use it).
In this instance, I don't have any apps shown in the Speech Recognition permissions section though. VS Code was listed (as blocked) in Microphone access, but that also made no difference.
There's nothing in the Flutter output when this happens, except the VM Service closing. I presume the native app is crashing - though I don't know if there's a way to tell why?

Hello everyone. A while ago I submitted an #134191 that may be relevant to this. That one has the smallest reproducible code sample, a crash log, and a screen record video. Please check it out if you are looking for additional information regarding this issue.

I think a big clue is in this related issue / comment:
https://github.com/flutter/flutter/issues/134191#issuecomment-1740117608


Added missing issuers data to dashboard

What does this PR do?
Related ticket
Fixes #
Type of change

[ ] Chore
[ ] Bug fix
[ ] New feature
[ ] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (add2afa) 93.63% compared to head (17af717) 93.63%.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #524   +/-   ##
=======================================
  Coverage   93.63%   93.63%
=======================================
  Files          20       20
  Lines         330      330
  Branches       76       76
=======================================
  Hits          309      309
  Misses         10       10
  Partials       11       11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Data storage converison calculator

Issue Id you have worked upon -
#226
Briefly explain your program logic -
Converts any type of data to the required data type, including conversions between bits, bytes, megabytes, etc.
Screenshots(Attach 2 screenshots of your own input and output) -

By raising this PR I affirm that -

[x] My code follows the guidelines of this project.

[x] I have performed a self-review of my own code.

[x] I have commented my code.

[x] My code gives the correct output.

[x] I affirm that I strictly follow contributing guidelines and code of conduct.

Congratulations 🎉 @shrinidhihegde3 on your contribution.
Thanks for adding value to this repository. 😊
Let's celebrate this fest 😃 by providing more developers 👨💻👩💻 an opportunity to contribute. Help us make this repository reach far and more by sharing this to your friends and colleagues.
Your efforts can make difference.

@shrinidhihegde3,
Good work. I am really impressed by the continuous updates you have made based as per my comments.
Your dedication is truly appreciated. 👏
Keep Contributing!! 🎉

Hey 👋, may you also encourage us by clicking the star icon on the repository main page. ⭐


Add in Snowflake's new query_parameterized_hash column

Add in Snowflake's new query_parameterized_hash column
Docs: https://docs.snowflake.com/en/user-guide/query-hash

Hey @sidhreddy, thanks for opening the issue!
Agree this would be great to add in. Are you willing to make a contribution? This should be a quick change, would just need to add it to stg_query_history and query_history_enriched

@ian-whitestone I just tried to push a change to address this and am getting 403. I am guessing the repo settings are not allowing folks outside your team to contribute.

@ian-whitestone Can you look into this PR.


fix(@h6s/website): revert bump react18

docusaurus does not support react18 yet
https://github.com/facebook/docusaurus/issues/7264
@JaeYeopHan i missed #223 branch deploy of "h6s.dev" was canceled. sorry 🙏

check example/react result..
next-dev.js?95dc:20 Warning: ReactDOM.render is no longer supported in React 18. Use createRoot instead. Until you switch to the new API, your app will behave as if it's running React 17

i'll reopen pr after resolve above issue


different date format in Recent Logins but the Dashboard shows the same

I run four Webmin and three of them show one date format (yyyy-mm-dd hh:mm) and one of them show another date format in the Recent Login. I have the exact same date format set in Authentic Theme but still it will show different date format. The rest of the Dashboard shows the same date format on all four Webmin's.
Is this something i can change in Authentic Theme or what?

Yes, this info is coming from WebminCore, while theme has it's own API for doing it.
I will talk to Jamie if we just could provide backward compatibility to the theme.

You are welcome to continue there a bit latter.
https://github.com/webmin/webmin/issues/820

Follow this up. I'm adding full locale support.
https://github.com/qooob/authentic-theme/issues/1035


JPEG minithumbnail error

[ 1][t24][1609642107.754322767][Photo.cpp:398][#6][!Td]	Receive unexpected JPEG minithumbnail

https://github.com/tdlight-team/tdlight/commit/c66972354af5c89867e67c2440572b7811c5b1f1

https://github.com/tdlight-team/tdlight/commit/c66972354af5c89867e67c2440572b7811c5b1f1

After the patch: [ 1][t24][1609677600.002752542][Photo.cpp:398][#6][!Td]	Receive unexpected JPEG minithumbnail

After the patch: [ 1][t24][1609677600.002752542][Photo.cpp:398][#6][!Td]	Receive unexpected JPEG minithumbnail

not a bug, this is a side-effect of "disable_minithumbnails"
https://github.com/tdlight-team/tdlight/blob/49c2bbd9b52693889c59ec129ada3e2aba6a29a6/td/telegram/Photo.cpp#L397
since we deactivate the saving of previews, tdlib doesn't know if it is there or not, so if it receives it and thinks it shouldn't be there, it doesn't know what to do, it would be enough to remove the log, it is good as it is.
https://github.com/tdlight-team/tdlight/blob/49c2bbd9b52693889c59ec129ada3e2aba6a29a6/td/telegram/Photo.cpp#L398

not a bug, this is a side-effect of "disable_minithumbnails"
https://github.com/tdlight-team/tdlight/blob/49c2bbd9b52693889c59ec129ada3e2aba6a29a6/td/telegram/Photo.cpp#L397
since we deactivate the saving of previews, tdlib doesn't know if it is there or not, so if it receives it and thinks it shouldn't be there, it doesn't know what to do, it would be enough to remove the log, it is good as it is.
https://github.com/tdlight-team/tdlight/blob/49c2bbd9b52693889c59ec129ada3e2aba6a29a6/td/telegram/Photo.cpp#L398


[firebase-database] ServerValue.timestamp()

Hello. I am trying to add server timestamp to realtime database:
        database.ref('/table').set({
            date: ServerValue.timeStamp()
        });

But, this doesn't create anything. ServerValue.timeStamp() returns empty object. I am not sure, is this a bug or is there another way to add timestamp

fixed: https://github.com/NativeScript/firebase/commit/05ec18d09442183e0f3ea75d80ff74d029488202


QWindowsContext: OleInitialize() failed:  "COM error ...

Hello,
Every time I run the example project, or any project, I get a console window appear displaying the following error:
QWindowsContext: OleInitialize() failed:  "COM error 0xffffffff80010106 RPC_E_CHANGED_MODE (Unknown error 0x080010106)"
The program continues and seems to work fine... but any idea why is this happening?
Thanks!

I had the same problem, and solved it by added [STAThread] to Main(string[] args)
Here is some example code:
  [STAThread]
  public static int Main(string[] args)
  {
    RuntimeManager.DiscoverOrDownloadSuitableQtRuntime();

    using (var app = new QGuiApplication())
    using (var engine = new QQmlApplicationEngine()) {
      var results = QmlRegistry.RegisterType<SomeType>();

      engine.Load("main.qml");
      return app.Exec();
    }
  }

While not a direct answer to this problem, these stackoverflow answers helped me solve this:

QCoreApplication QApplication with WMI
Do i need to call CoInitialize before interacting with COM in .NET?


Update to latest miniconda and test in OSX

Is there a reason why we are using relatively old software versions for e.g. Ubuntu, Miniconda2 instead of 3, ... ?
https://github.com/biocore/empress/blob/7cd33d29331d188a784158ad7729e448cb443531/.travis.yml#L4
Shouldn't we add testing on OSX?

Now that we've transitioned to using GitHub Actions (main CI, standalone CI), it looks like we are using the latest Ubuntu (and I think conda) versions.
So I think all that is left for this issue is adding testing on macOS systems -- it looks like this is very doable with GitHub Actions (there is an example of this about midway down that page).


Windows CMake corrections

Motivation
This change is to fix building with CMake on Windows platforms.
Modifications
The primary change is to provide more library names for zlib, protobuf, and curl that will match the windows versions of these libraries. It also includes small changes to find clang-format and clang-tidy, and also to install the dll in the bin folder instead of the lib folder.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): no
The public API: no
The schema: no
The default values of configurations: no
The wire protocol: no
The rest endpoints: no
The admin cli options: no
Anything that affects deployment: no

Documentation

Does this pull request introduce a new feature? no

@heronr Add a gitlab workflow to check compilation on windows.

@heronr can you add a github action to verify it can build on windows?

Apologies, how do I do that?

@heronr
check the workflows  under
.github/workflows/
add a new one and target the windows os , you can use vcpkg to install dependencies and cache them. After that just make sure that it can be compiled under windows using cmake.

I think it the windows github action would be good to go in a different PR anyway.

Add label release-2.5.1, due to #6458 dependency


azcopy login fails on Azure ML Compute Instance

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
10.6.1
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
azcopy login
What problem was encountered?
Got this error msg:

Failed to perform login command:
failed to get keyring during saving token, key has been revoked
NOTE: If your credential was created in the last 5 minutes, please wait a few minutes and try again.

How can we reproduce the problem in the simplest way?
Repro + mitigation:

Connect to Azure ML Compute Instance using SSH (Jupyter Terminal results in the same issue)
Run azcopy login command
a.	Observe that after successful device login using browser, error msg is shown in shell

Failed to perform login command:
failed to get keyring during saving token, key has been revoked
NOTE: If your credential was created in the last 5 minutes, please wait a few minutes and try again.

Run keyctl new_session command. This is based on different posts such as this one - https://github.com/Azure/azure-storage-azcopy/issues/1021
Run azcopy login command again
a.	Observe that after successful device login using browser, login is now successful

INFO: Login succeeded.

Have you found a mitigation/solution?
Yes, see above.

Hi @sjuratov ,
Thanks for reaching out to us!
This is a known issue in CentOS, RHEL and some flavors of Linux. What flavor of Linux are you using?
FYI: We will have a work around very soon with v10.7.0 release of AzCopy where we will start supporting device login.
Please read this to know more about keyctl in AzCopy. Reach out to us if you need more clarity on this.

Hi @mohsha-msft ,
Here is a version of Linux VM

No LSB modules are available.
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.7 LTS
Release:        16.04
Codename:       xenial

Can you confirm that v10.7.0 would fix issue for Ubuntu as well? What is ETA for v10.7.0 release?

@sjuratov 10.7.0 is released.
To be precise, 10.7.0 does not fix the error with linux keyring expiring. It provides a comfortable wokround.
Set the enviroment variable AZCOPY_AUTO_LOGIN_TYPE=DEVICE, and the copy command will ask you to login to Azure Servce.
[nakulkar@nakulkar-dev:~]$ azcopy copy file512k "https://.blob.core.windows.net/container1/"
INFO: Scanning...
To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code  to authenticate.

AZCOPY_AUTO_LOGIN_TYPE=DEVICE
This asks me to login for every single CLI command. Obviously that does not work.