NREL mismatch loss addendum

Addendum to PR #2046 from a comparison with the authors implementation made publicly at https://github.com/NREL/bifacial_radiance/blob/144e6d00c2ddb391cf0904191d7c347be4436b1f/bifacial_radiance/mismatch.py#L166.

[x] Addresses comment https://github.com/pvlib/pvlib-python/pull/2046#issuecomment-2200703792
[x] I am familiar with the contributing guidelines
~[ ] Tests added~
~[ ] Updates entries in docs/sphinx/source/reference for API changes.~
[x] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[x] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[x] Pull request is nearly complete and ready for detailed review.
[ ] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

@cdeline asked for a comparison with an implementation of his model (Fit 3), and it has helped find one error and one issue:

Fixed error: During my investigations on the Internet, I confused Mean Absolute Difference with Mean Absolute Deviation, so that helper function in the example has been fixed.

Pending issue: The polynomial model in the authors implementation is evaluated over the unitless RMAD (Relative Mean Absolute Difference), whereas in the paper we can find the units of equation (4) being in percentage:
$$M[\%]_{Fit3} = 0.054 \Delta [\%]+ 0.068 \Delta [\%] ^2$$
Code in implementation:
mad = mad_fn(datac) /100  # (percentage)
mad2 = mad**2

fit3 = 0.054*mad + 0.068*mad2

First line converts mad_fn output from percentage to unitless ratio.
In the pvlib implementation, this has been understood as if the input to the polynomial is the RMAD in percentage, so we tuned the coefficients to match the units. Docs, see Parameters and Notes sections.

Down below is the code I used to compare the implementations.
Comparison code

Note this does not comply with the 2D broadcasting rules and datatypes IO in the authors implementation.
# %%
from pvlib.bifacial import power_mismatch_deline
import numpy as np

# %%
# bifacial_radiance implementation
def mismatch_fit3(data):
    '''
    Electrical mismatch calculation following Progress in PV paper
    Estimating and parameterizing mismatch power loss in bifacial photovoltaic systems
    Chris Deline, Silvana Ayala Pelaez,Sara MacAlpine,Carlos Olalla
    https://doi.org/10.1002/pip.3259

    Parameters
    ----------
    data : np.ndarray, pd.Series, pd.DataFrame
        Gtotal irradiance measurements. Each column is the irradiance for a module
        at a specific time.

    Returns
    -------
    fit3 :  Float or pd.Series
        Returns mismatch values for each module

    Equation: 1/(n^2*Gavg)*Sum Sum (abs(G_i - G_j))
    ## Note: starting with Pandas 1.0.0 this function will not work on Series objects.
    '''
    import numpy as np
    import pandas as pd

    if type(data) == np.ndarray:
        data = pd.DataFrame(data)

    datac = data[~np.isnan(data)]
    mad = mad_fn(datac) /100  # (percentage)
    print(f"{mad=}")
    mad2 = mad**2

    fit3 = 0.054*mad + 0.068*mad2

    if fit3.__len__() == 1:
        if isinstance(fit3, pd.Series):
            fit3 = float(fit3.iloc[0])
        else:
            fit3 = float(fit3)

    return fit3

def mad_fn(data, axis='index'):
    '''
    Mean average deviation calculation for mismatch purposes.

    Parameters
    ----------
    data : np.ndarray or pd.Series or pd.DataFrame
        Gtotal irradiance measurements. If data is a pandas.DataFrame, one
        MAD/Average is returned for each index, based on values across columns.

    axis : {0 or 'index', 1 or 'columns'}, default 'index'
        Calculate mean average deviation across rows (default) or columns for 2D data

        * 0, or 'index' : MAD calculated across rows.
        * 1, or 'columns' : MAD calculated across columns.
    Returns
    -------
    scalar or pd.Series:   return MAD / Average [%]. Scalar for a 1D array, Series for 2D.

    Equation: 1/(n^2*Gavg)*Sum Sum (abs(G_i - G_j)) * 100[%]

    '''
    import numpy as np
    import pandas as pd
    def _mad_1D(data):  #1D calculation of MAD
        return (np.abs(np.subtract.outer(data,data)).sum()/float(data.__len__())**2 / np.mean(data))*100
    if type(axis) == str:
        try:
            axis = {"index": 0, "rows": 0, 'columns':1}[axis]
        except KeyError:
            raise Exception('Incorrect index string in mad_fn. options: index, rows, columns.')

    ndim = data.ndim
    if ndim == 2 and axis==0:
        data = data.T
    # Pandas returns a notimplemented error if this is a DataFrame.
    if (type(data) == pd.Series):
        data = data.to_numpy()

    if type(data) == pd.DataFrame:
        temp = data.apply(pd.Series.to_numpy, axis=1)
        return(temp.apply(_mad_1D))
    elif ndim ==2: #2D array
        return [_mad_1D(i) for i in data]
    else:
        return _mad_1D(data)

# %%
# pvlib rmad function in example
def rmad(data):
    """
    Relative Mean Absolute Difference. Output is [Unitless]. Eq. (4) of [1]_.
    """
    mean = np.mean(data)
    mad = np.mean(np.absolute(np.subtract.outer(data, data)))
    return mad / mean

# %%
# test input data, one dimensional
G_global_dim1 = np.array([1059, 976, 967, 986, 1034, 1128])

# %%
# test input data, two dimensional
G_global_dim2 = np.repeat([1059, 976, 967, 986, 1034, 1128], 12).reshape(
    6, 12
)

# %%
# test implementations
for input_G_global in [G_global_dim1, G_global_dim2]:
    dim = input_G_global.ndim
    print(f"Testing for input data with {dim=}")
    # output from bifacial_radiance implementation
    result_bifacial_radiance = mismatch_fit3(input_G_global)
    # output from pvlib implementation
    rmad_G_global = rmad(input_G_global)
    result_pvlib = power_mismatch_deline(
        rmad=rmad_G_global, coefficients=(0, 0.054, 0.068)
    )
    print(f"{result_bifacial_radiance=}\n{result_pvlib=}\n-------------------")

@cdeline I suspect we should change the coefficients and the documentation to reflect that both the equation in the paper and in pvlib use the unitless RMAD, but before I proceed I'd like to have your confirmation.

Hi @echedey-ls.  Your implementation appears to be correct and matches the method as described in the paper.  I think that your use of Fit2 is the correct one based on the text of the paper, and the extra /100 in the bifacial_radiance mismatch_fit3 function is an error that we'll have to correct.  I like your unitless approach for everything better.

Thanks @cdeline for the insight.
This PR is then ready for review.

I think that we need to get a PR into bifacial_radiance to fix this bug and switch to the Fit2 which has more broad applicability, vs Fit3.  Here's your code, modified to show convergent behavior.  However, there is a problem if you try to compare against the Fit3 coefficients by passing them in to your power_mismatch_deline() function.
# %%
from pvlib.bifacial import power_mismatch_deline
import numpy as np

# %%
# bifacial_radiance implementation
def mismatch_fit2(data):
    '''
    Electrical mismatch calculation following Progress in PV paper
    Estimating and parameterizing mismatch power loss in bifacial photovoltaic systems
    Chris Deline, Silvana Ayala Pelaez,Sara MacAlpine,Carlos Olalla
    https://doi.org/10.1002/pip.3259

    Parameters
    ----------
    data : np.ndarray, pd.Series, pd.DataFrame
        Gtotal irradiance measurements. Each column is the irradiance for a module
        at a specific time.

    Returns
    -------
    fit3 :  Float or pd.Series
        Returns mismatch values for each module as a [%]

    Equation: 1/(n^2*Gavg)*Sum Sum (abs(G_i - G_j))
    ## Note: starting with Pandas 1.0.0 this function will not work on Series objects.
    '''
    import numpy as np
    import pandas as pd

    if type(data) == np.ndarray:
        data = pd.DataFrame(data)

    datac = data[~np.isnan(data)]
    mad = mad_fn(datac)   # (unitless)
    print(f"{mad=}")
    mad2 = mad**2

    fit2 = 0.142*mad + 0.032*mad2

    if fit2.__len__() == 1:
        if isinstance(fit2, pd.Series):
            fit2 = float(fit2.iloc[0])
        else:
            fit2 = float(fit2)

    return fit2

def mad_fn(data, axis='index'):
    '''
    Mean average deviation calculation for mismatch purposes.

    Parameters
    ----------
    data : np.ndarray or pd.Series or pd.DataFrame
        Gtotal irradiance measurements. If data is a pandas.DataFrame, one
        MAD/Average is returned for each index, based on values across columns.

    axis : {0 or 'index', 1 or 'columns'}, default 'index'
        Calculate mean average deviation across rows (default) or columns for 2D data

        * 0, or 'index' : MAD calculated across rows.
        * 1, or 'columns' : MAD calculated across columns.
    Returns
    -------
    scalar or pd.Series:   return MAD / Average [%]. Scalar for a 1D array, Series for 2D.

    Equation: 1/(n^2*Gavg)*Sum Sum (abs(G_i - G_j)) * 100[%]

    '''
    import numpy as np
    import pandas as pd
    def _mad_1D(data):  #1D calculation of MAD
        return (np.abs(np.subtract.outer(data,data)).sum()/float(data.__len__())**2 / np.mean(data))*100
    if type(axis) == str:
        try:
            axis = {"index": 0, "rows": 0, 'columns':1}[axis]
        except KeyError:
            raise Exception('Incorrect index string in mad_fn. options: index, rows, columns.')

    ndim = data.ndim
    if ndim == 2 and axis==0:
        data = data.T
    # Pandas returns a notimplemented error if this is a DataFrame.
    if (type(data) == pd.Series):
        data = data.to_numpy()

    if type(data) == pd.DataFrame:
        temp = data.apply(pd.Series.to_numpy, axis=1)
        return(temp.apply(_mad_1D))
    elif ndim ==2: #2D array
        return [_mad_1D(i) for i in data]
    else:
        return _mad_1D(data)

# %%
# pvlib rmad function in example
def rmad(data):
    """
    Relative Mean Absolute Difference. Output is [Unitless]. Eq. (4) of [1]_.
    """
    mean = np.mean(data)
    mad = np.mean(np.absolute(np.subtract.outer(data, data)))
    return mad / mean

# %%
# test input data, one dimensional
G_global_dim1 = np.array([1059, 976, 967, 986, 1034, 1128])

# %%
# test input data, two dimensional
G_global_dim2 = np.repeat([1059, 976, 967, 986, 1034, 1128], 12).reshape(
    6, 12
)

# %%
# test implementations
for input_G_global in [G_global_dim1, G_global_dim2]:
    dim = input_G_global.ndim
    print(f"Testing for input data with {dim=}")
    # output from bifacial_radiance implementation
    result_bifacial_radiance = mismatch_fit2(input_G_global)
    # output from pvlib implementation
    rmad_G_global = rmad(input_G_global)
    result_pvlib = power_mismatch_deline(
        rmad=rmad_G_global
    )
    print(f"{result_bifacial_radiance=}\n{result_pvlib=}\n-------------------")

@echedey-ls - I have a question on the 'coefficients' input into power_mismatch_deline - I do get good agreement with our Fit3 coefficients if I use a *100 in the second coefficient:
    result3_pvlib = power_mismatch_deline(
        rmad=rmad_G_global, coefficients=(0, .054, .068*100)
    )

I guess that's the desired behavior since it includes a conversion from [%] to [unitless] on the input and output...

I think that we need to get a PR into bifacial_radiance

Gotcha, I'd like to do that.

I do get good agreement with our Fit3 coefficients if I use a *100 in the second coefficient

Yeah, that's the desired behaviour. My original comparison isn't exactly right now that you confirm what issues the bifacial_radiance implementation has:

The model coefficients are for % input
However, the input is unitless
The output is supposed to be in % per your comparison code for fit 2 docstring:Returns
  -------
  fit3 :  Float or pd.Series
      Returns mismatch values for each module as a [%]

So a fair comparison would be what is currently documented in pvlib:

Previous issues addressed
Second-order coeff in pvlib.bifacial.power_mismatch_deline multiplied by 100

Code below
Details

# %%
from pvlib.bifacial import power_mismatch_deline
import numpy as np

# %%
# bifacial_radiance implementation
def mismatch_fit3(data):
    '''
    Electrical mismatch calculation following Progress in PV paper
    Estimating and parameterizing mismatch power loss in bifacial photovoltaic systems
    Chris Deline, Silvana Ayala Pelaez,Sara MacAlpine,Carlos Olalla
    https://doi.org/10.1002/pip.3259

    Parameters
    ----------
    data : np.ndarray, pd.Series, pd.DataFrame
        Gtotal irradiance measurements. Each column is the irradiance for a module
        at a specific time.

    Returns
    -------
    fit3 :  Float or pd.Series
        Returns mismatch values for each module

    Equation: 1/(n^2*Gavg)*Sum Sum (abs(G_i - G_j))
    ## Note: starting with Pandas 1.0.0 this function will not work on Series objects.
    '''
    import numpy as np
    import pandas as pd

    if type(data) == np.ndarray:
        data = pd.DataFrame(data)

    datac = data[~np.isnan(data)]
    mad = mad_fn(datac)  # (percentage)
    print(f"{mad=}")
    mad2 = mad**2

    fit3 = 0.054*mad + 0.068*mad2

    if fit3.__len__() == 1:
        if isinstance(fit3, pd.Series):
            fit3 = float(fit3.iloc[0])
        else:
            fit3 = float(fit3)

    return fit3

def mad_fn(data, axis='index'):
    '''
    Mean average deviation calculation for mismatch purposes.

    Parameters
    ----------
    data : np.ndarray or pd.Series or pd.DataFrame
        Gtotal irradiance measurements. If data is a pandas.DataFrame, one
        MAD/Average is returned for each index, based on values across columns.

    axis : {0 or 'index', 1 or 'columns'}, default 'index'
        Calculate mean average deviation across rows (default) or columns for 2D data

        * 0, or 'index' : MAD calculated across rows.
        * 1, or 'columns' : MAD calculated across columns.
    Returns
    -------
    scalar or pd.Series:   return MAD / Average [%]. Scalar for a 1D array, Series for 2D.

    Equation: 1/(n^2*Gavg)*Sum Sum (abs(G_i - G_j)) * 100[%]

    '''
    import numpy as np
    import pandas as pd
    def _mad_1D(data):  #1D calculation of MAD
        return (np.abs(np.subtract.outer(data,data)).sum()/float(data.__len__())**2 / np.mean(data))*100
    if type(axis) == str:
        try:
            axis = {"index": 0, "rows": 0, 'columns':1}[axis]
        except KeyError:
            raise Exception('Incorrect index string in mad_fn. options: index, rows, columns.')

    ndim = data.ndim
    if ndim == 2 and axis==0:
        data = data.T
    # Pandas returns a notimplemented error if this is a DataFrame.
    if (type(data) == pd.Series):
        data = data.to_numpy()

    if type(data) == pd.DataFrame:
        temp = data.apply(pd.Series.to_numpy, axis=1)
        return(temp.apply(_mad_1D))
    elif ndim ==2: #2D array
        return [_mad_1D(i) for i in data]
    else:
        return _mad_1D(data)

# %%
# pvlib rmad function in example
def rmad(data):
    """
    Relative Mean Absolute Difference. Output is [Unitless]. Eq. (4) of [1]_.
    """
    mean = np.mean(data)
    mad = np.mean(np.absolute(np.subtract.outer(data, data)))
    return mad / mean

# %%
# test input data, one dimensional
G_global_dim1 = np.array([1059, 976, 967, 986, 1034, 1128])

# %%
# test input data, two dimensional
G_global_dim2 = np.repeat([1059, 976, 967, 986, 1034, 1128], 12).reshape(
    6, 12
)

# %%
# test implementations
for input_G_global in [G_global_dim1, G_global_dim2]:
    dim = input_G_global.ndim
    print(f"Testing for input data with {dim=}")
    # output from bifacial_radiance implementation
    result_bifacial_radiance = mismatch_fit3(input_G_global)
    # output from pvlib implementation
    rmad_G_global = rmad(input_G_global)
    result_pvlib = power_mismatch_deline(
        rmad=rmad_G_global, coefficients=(0, 0.054, 0.068*100)
    )
    print(f"{result_bifacial_radiance=}\n{result_pvlib=}\n-------------------")

And now bifacial_radiance output is x100 the one in pvlib, which is expected since the former works with percentages and the latter with a [0,1] ratios:
Testing for input data with dim=1
mad=0    5.9729
dtype: float64
result_bifacial_radiance=2.748472705106455
result_pvlib=0.02748472705106455
-------------------
Testing for input data with dim=2
mad=0     5.9729
1     5.9729
2     5.9729
3     5.9729
4     5.9729
5     5.9729
6     5.9729
7     5.9729
8     5.9729
9     5.9729
10    5.9729
11    5.9729
dtype: float64
result_bifacial_radiance=0     2.748473
1     2.748473
2     2.748473
3     2.748473
4     2.748473
...
11    2.748473
dtype: float64
result_pvlib=0.02748472705106455
-------------------

@kandersolar @cwhanse this PR is ready for review😄 👀


API Key / Username / Password for DoT

EPIC

EPIC

BLOCKED waiting for Username and Password from SLB for DoT

Thanks a ton, Chris! We're implementing the DoT auth now using your server!

@randypitcherii @cherrera2001   Is this one still blocked ?

nope
From: niaalex<EMAIL_ADDRESS>Sent: Thursday, November 29, 2018 3:40 PM
To: hashmapinc/Drillflow<EMAIL_ADDRESS>Cc: Chris Herrera<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [hashmapinc/Drillflow] API Key / Username / Password for DoT (#104)
@randypitcherii https://github.com/randypitcherii  @cherrera2001 https://github.com/cherrera2001  Is this one still blocked ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/hashmapinc/Drillflow/issues/104#issuecomment-443003241 , or mute the thread https://github.com/notifications/unsubscribe-auth/AFphD3bGrrNu6uAACoTfapP3L9-r0_aDks5u0FRKgaJpZM4Y1sRO .  https://github.com/notifications/beacon/AFphD0lr-jB7-f4Ty3KQ4aNwjhocPM9mks5u0FRKgaJpZM4Y1sRO.gif

@randypitcherii @cherrera2001  disregard


feat(multilist): add new multi selection list widget

NOTE: Please also see the following for another alternative option:

https://github.com/tui-rs-revival/ratatui/pull/19

Upstream: #531

Added a new widget like the List widget but it allows for multiple selections rather than just one.

In order to complete this PR, it needs:

[ ] Unit tests
[ ] Docs
[ ] Clean up of code that is very similar to the code in List

This issue was closed due to it becoming stale. Feel free to reopen and work on this.

This issue was closed due to it becoming stale. Feel free to reopen and work on this.


Create CODEOWNERS

Dev Docs related to this Pull Request:
Link to Dev Doc or PR: https://github.com/instructlab/dev-docs/pull/165
Checklist:

[X] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[X] Changelog updated with breaking and/or notable changes for the next minor release.
[X] Documentation has been updated, if necessary.
[X] Unit tests have been added, if necessary.
[X] Functional tests have been added, if necessary.
[X] E2E Workflow tests have been added, if necessary.

After discussing with @mairin, it seems best to not introduce a new permissions management system on top of the existing MAINTAINERS.md mechanism, so will pursue that instead.


Automatically detect & flag new actions and conditions

Since Actions/Conditions require explicit code gen definitions, I should make sure that new actions / conditions are captured.

Done w/ analysis in 2.0.


OpenSsl.USE_KEYMANAGER_FACTORY incorrectly set to false with BoringSSL

Motivation
SSL unit tests started failing for me (RHEL 7.6) after #9162. It looks like the intention was to prevent use of the io.netty.handler.ssl.openssl.useKeyManagerFactory property when using BoringSSL, but it now gets set to false in that case rather than the prior/non-BoringSSL default of true.
Modification
Set useKeyManagerFactory to true rather than false in BoringSSL case during static init of OpenSsl class.
Result
Tests pass again.

@normanmaurer sorry if this is a false alarm, I may have misinterpreted the original intention.

@njhill doh! ... yes you are right.


add tls.ClientHelloInfo.Conn for recursive GetConfigForClient calls

See https://github.com/caddyserver/caddy/issues/5680#issuecomment-1665042702.

I'm sorry for the trouble this is causing. But you're right, this is quite complex.
The race condition should be fixed now.

We have another confirmation of the fix. :+1:

Thanks for the review @mholt and thank you @francislavoie and everyone else involved in testing!


Load models from s3 test

I'll remove the INFO logging, but we should still set env var TF_CPP_MIN_LOG_LEVEL=1 if we want to reduce logging


In FreeRADIUS After Changing User Password, Apply Must Be Clicked Before Change Becomes Effective

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I have searched the existing issues and I am convinced that mine is new.

Describe the bug
After changing a user's password in FreeRADIUS and clicking save, Apply on the Users screen must be clicked else the password change will not take effect.
To Reproduce
Steps to reproduce the behavior:

Set up FreeRADIUS with a user.
Using System: Access: Tester confirm access.
In FreeRADIUS, under Users, edit the user to change their password and click Save to return to the list of users.
Test the new password with System: Access: Tester and observe access denied.
Test the old password with System: Access: Tester and observe access granted.
Click Save on the Users page.
Test the new password with System: Access: Tester and observe access granted.

Expected behavior
I would have expected the user's password change would take effect once I clicked save on the user's account page. As with other parts of OPNsense I would expect that if an additional Save is required for changes to take effect, a banner notification about applying settings (or something similar) would be displayed.
Additional context
Note: Stopping and starting FreeRADIUS does not make the password change take effect. If Save is clicked after stopping/starting the service the password change will take effect.
Environment
OPNsense 20.7.7_1-amd64
FreeBSD 12.1-RELEASE-p11-HBSD
OpenSSL 1.1.1i 8 Dec 2020
os-freeradius 1.9.9

I think that for most grid views you have to hit apply/save a second time, just like with Firewall Alias or similar

I think that for most grid views you have to hit apply/save a second time, just like with Firewall Alias or similar

I think that for most grid views you have to hit apply/save a second time, just like with Firewall Alias or similar

I can see that. If a firewall rule gets changed and apply needs to be clicked, the UI tells you. That's my issue here; it appears to be done and not need any more clicking, yet isn't actually applied.
This is what appears for a firewall rule; I'd think the same could happen here.

I think that for most grid views you have to hit apply/save a second time, just like with Firewall Alias or similar

I can see that. If a firewall rule gets changed and apply needs to be clicked, the UI tells you. That's my issue here; it appears to be done and not need any more clicking, yet isn't actually applied.
This is what appears for a firewall rule; I'd think the same could happen here.

This is from legacy code where you have to apply on the upper right, like with ntp or similar.
The new code usually needs a hit on apply after adding something to the grid, like FreeRadius, WireGuard, Postfix and so on.

This is from legacy code where you have to apply on the upper right, like with ntp or similar.
The new code usually needs a hit on apply after adding something to the grid, like FreeRadius, WireGuard, Postfix and so on.


Find a global team list

Can you find and hook a global teams UtlVector (g_Teams)? I found that class in client.dll
https://github.com/ValveSoftware/source-sdk-2013/blob/master/mp/src/game/client/c_team.h

Loop through entity list and find CTeam entities.

I have exception here. What could it be?

Uh i forgot check for null 🤦♂

I checked entity list, there no CTeam entities


Break up and pivot large table

Fixes #16723, #9196

Break up into one table per .NET Framework version
Pivot rows and columns
Replace checkmark and plus sign characters with emojis (see #16721)
Add VS 2017 and 2019 info
Update new features links
Acrolinx-type improvements

Love it! I'm wondering if it would be nice to promote the headers for the frameworks to H2? this way they show on the quick nav on the right.

I tried this but because there are 16 of them, the quick-nav became too long. Instead I hand-created a table of links to the versions.
Thanks for reviewing!


Check Python version

Exit with an error status if the Python version is not 2.7.

Gary reviewed this code in a previous pull request.  Merging.


ibmmq and prometheus exporter

Description
I'm trying to run ibmmq-nri and am getting a message saying
Jun 14 23:51:33 sufra04selwcllog1001 newrelic-infra-service: time="2022-06-14T23:51:33Z" level=debug msg="Integration stderr (not parsed)." cfg_protocol_name=cfg-nri-ibmmq component=integrations.runner.Runner integration_name=nri-prometheus line="time="2022-06-14T23:51:33Z" level=warning msg="error while scraping target" component=Fetcher error="Get \"http://localhost:9157/metrics\\\": dial tcp [::1]:9157: connect: connection refused"" parent_integration_name=nri-ibmmq runner_uid=99287eb533
Following steps in NR docs it is not saying i need to install anything beyond the ibmmq-nri, but nothing is listening on port I'm exposing.
Is there a step I'm missing?  Or are there more logs for ibmmq-nri?
THanks!
Mark
Steps to Reproduce
Expected Behavior
Relevant Logs / Console output
Your Environment

ex: Browser name and version:
ex: Operating System and version:  Centos 7

Additional context

Hi ,
The next best step is to visit New Relic Support, where you can engage the New Relic Support Community or open a support ticket depending on your support level. The support team is best positioned to assist with your specific needs.
Please provide a link to this GitHub issue when submitting your community post or support ticket.
Thanks!


central.spacestation14.io times out

Description
central times out so have to go outside!!!!
Reproduction
curl -v https://central.spacestation.14.io
Screenshots

Additional context
happens for me and another person

we're aware, this shouldn't go on github


Implement withoutActuallyEscaping verification

Check that an withoutActuallyEscaping(noescape_closure) { // scope} closure
has not escaped in the scope using the is_escaping_closure %closure
instruction.
rdar://35525730

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - 89e972f5a864cf75c0ff6703f275f39a378a41dc

Build failed
Swift Test OS X Platform
Git Sha - 89e972f5a864cf75c0ff6703f275f39a378a41dc

@swift-ci Please smoke test

@swift-ci Please test source compatibility

NSDictionary.enumerateKeysAndObjects' withoutActuallyEscaping is escaping a noescape closure. https://github.com/apple/swift-corelibs-foundation/blob/master/Foundation/NSDictionary.swift#L486
The code can be made correct by moving the code that uses the returned closure into withoutActuallyEscaping.

Foundation/NSIndexSet.swift has the same issue.

👍🏻👍🏻

Please test with following PR:
https://github.com/apple/swift-corelibs-foundation/pull/1463
@swift-ci Please test linux

Build failed
Swift Test Linux Platform
Git Sha - 89e972f5a864cf75c0ff6703f275f39a378a41dc

The successful source compatibility suite run is here: https://ci.swift.org/view/Pull Request/job/swift-PR-source-compat-suite/837/

Build failed
Swift Test Linux Platform
Git Sha - b9602754cd97633486dd7f04abb41f813ba06981

Please test with following PR:
https://github.com/apple/swift-corelibs-foundation/pull/1463
@swift-ci Please test linux

apple/swift-corelibs-foundation#1463 has to be merged first to prevent breakage

@swift-ci Please test mac os x

@swift-ci Please test macos

Build failed
Swift Test OS X Platform
Git Sha - 89e972f5a864cf75c0ff6703f275f39a378a41dc


add some caching support by disabling random data generated by jQuery

perhaps it should be configurable? altho my use cases, i need it to be enabled, but library maintainer may have better acceptance if this is configurable. but then again the library is not maintained :D

also, the commits are inconsistent:

first two modify src/jquery.embedly.js
third also modifies jquery.embedly.js

modify of jquery.embedly.js should be omitted, as it's part of build process

also: datatype change is only ok if the source allows cross-origin requests. so should be configurable as well.


Cannot access the Wiki

Hello,
I tried to review the Wiki but this redirects back to the LibreTasks repository.
I'm eager to give LibreTasks a try!

Click on view more branches, go to wiki branch.
Standard structure of many github projects.

Thank you for the education.
Does this type of wiki structure limit searching the wiki? (I would guess to the selected branch.)
If so that would appear to be not helpful - but I'm not yet familiar with this structure (schema, perhaps?).

Had the same problem.
Fixing this would help not-so-savvy users, who might be confused and fail to give the app a try.

@biotinker, perhaps there is a procedure that would migrate the wiki data from your code to the actual repo wiki.
I think that @pataquets and I are really expressing what many other users/potential users of the app are experiencing.
That is, I believe many users would be helped if the wiki could be accessed in the standard manner.
Please give this some consideration.
TIA

Agreed, is confusing. That said, as a developer I knew to check the Issues tab when I hit a problem, so managed to find this, but many users wouldn't think (or care enough) to do this.


duped records in AKFIN_TAXONOMIC_GROUPS

Issue

Duplicate records for the following species codes (see SQL below): 82691, 82741, 82740, 82690
select species_code, count() from gap_products.akfin_taxonomic_groups group by species_code
having count()>1;

Hi @jeanlee-akfin
Thanks for the catch! This was a mistake on my part when creating the table and I should have looked for duplicates. This has been remedied in gap_products.akfin_taxonomic_groups. Do we need to do a transfer of that table to the AKFIN side or are you pulling that table from the AFSC database?

If you can do a transfer to AKFIN that would be great - I'm developing some mat views on our side that will be using that table.

Cool, I sent it up the chain.

Hey @jeanlee-akfin, was the transfer of that table successful?

Yes - we got the updated version of the table loaded and I confirmed it has no dupes

yay


Campaign | Posters: Mark as damaged toggle

As an app user I want to be able to edit the state of the poster to damaged.
Acceptance criteria:

[ ] when clicking on an existing poster #197, the user can toggle the state of the poster to the state damaged.
[ ] on the map the icon color changes to red in this state.

Prototype: https://www.figma.com/proto/7wE0Iveas3GGb8u4oL0714/App-Design-Mitglieder-App-(1)-(Copy)?node-id=2670-9551&t=b8swD5IOAFfZ0OSo-1
API endpoint reference: https://github.com/verdigado/gruene-api/issues/60

Issue is reviewed.
Note: the information about a damaged poster needs to be saved in the database (Grüne Atlas).
@nscharioth @leagerndt

Issue is reviewed.
Note: the information about a damaged poster needs to be saved in the database (Grüne Atlas).
@nscharioth @leagerndt

Yes, that information will not be lost. It is saved via the API and can accessed for such functions in the cockpit.


VSTS build solution error for renewing certificate (.pfx) file

Environment
VSTS details
AccountName:
Team project name:
Build Definition Name:
Build Number:
Private Agent: Windows Server 2012 R2
Issue Description
Build Solution task in VSTS
C:\Program Files (x86)\Microsoft Visual Studio\2017\Professional\MSBuild\Microsoft\VisualStudio\v15.0\AppxPackage\Microsoft.AppXPackage.Targets(4216,5): error APPX0108: The certificate specified has expired. For more information about renewing certificates, see http://go.microsoft.com/fwlink/?LinkID=241478
Note: I did create a new certificate using visual studio 2017 and pushed new certificate to the remote repo and that helped me in getting rid of the same error I was getting in a powershell task to import certificate. The error is coming from UWP project.
Error logs
logs_487.zip](https://github.com/Microsoft/vsts-tasks/files/2288060/logs_487.zip)

This is an msbuild warning, the build app behaved properly by reporting it.  Is this issue only happening in AzureDevOps or does it also happen when you build locally?  If it is specific to pipelines please reopen.


check in fail in v2.1

check in fail in v2.1
util/cron.go:34 check in fail, error: Get "https://v18.teachermate.cn/wechat/wechat/guide/signin?openid=xxxx": EOF
util/cron.go:34 check in fail, error: request wrong

The API is no longer available. I don't have time to maintain this project.
Additionally, I have not been able to find a suitable alternative.

fixed


Your password must contain shaanxi noodles (𰻝)
Capital letters can't be anywhere near each other
Your password must not contain more than one lowercase letter

Your emojis aren't diverce enough

I dunno how you'd code that, but that's one of the most hilariously inane password criteria I've ever seen 🤣

I dunno how you'd code that

If you examine the site page through web inspector, you can find a script with a set of rules and their regular expressions.


How to find intersection point of screen location and object?

[X] Review the documentation: https://docs.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/viro
[x] Use the latest ViroReact release: https://docs.viromedia.com/docs/releases

Environment
Please provide the following information about your environment:

Development OS: Mac
Device OS & Version: Android 8
Version: react-viro v2.14.0
Device(s): Nokia 6v2

Description
I am trying to place objects on a horizontal plane at the position on the screen that the user clicks. The problem is planes are often small and the click falls outside of the plane - eg if the phone is pointed into the distance.
performARHitTestWithPoint(evt.nativeEvent.locationX * PixelRatio.get(), evt.nativeEvent.locationX * PixelRatio.get())
Works fine when you click on a plane or a feature point but what I was hoping to do was to extend a Quad (or the plane itself) out into the distance beyond the plane's extent and get the hit points with that but object hits are never returned.
Is there an alternate way to do this or is this a bug? I only have access to Androids currently so I'm not sure if this is an Android only issue or not.
I can see the documentation doesn't make mention of objects but is there another way I could use a point on the screen to cast a ray and see if it intersects with a particular object (or preferably an infinite plane). Can a plane's extent be modified as they are created?
The use case in I am not approaching this the right way at all is to be able to find the lowest possible plane (I can do this already and it works well) to estimate where the floor is and place objects anywhere on the floor not just on the planes detected.
Reproducible Demo
Let us know how to reproduce the issue. Include a code sample, screen capture, video recording. The more information you provide, the better we can support you.

Turns out after playing around that onClick serves my purpose (but doesn't resolve the exact question) if anyone has the same question. It returns the world position of the click which when you have a floor plane is pretty much spot on.
It's not likely to work properly for an arbitrary click above the floor plane though.


Bring our fork of congomongo up to date with what we're running in production

We're using a jar built from https://github.com/brandonbloom/congomongo/commit/58d755baa9dc5daeab67516058ccd2a590766a2f as a mongofinil dependency in production.
This PR pulls in those commits so we can rebuild if necessary

Sorry, targeted the wrong upstream fork


Strange propagation of VC when using quantifiers

stumbled on that surprising example with @jakesilverman
module Test

open FStar.Seq

let _in (s:seq 'a) = k:nat{k < length s}
let works (s:seq 'a) =
  match length s with
  | 0 -> ()
  | _ -> let s' = tail s in assert (forall (i:_in s'). True) ; ()

let fails (s:seq 'a) =
  match length s with
  | 0 -> ()
  | _ -> assert (forall (i:_in (tail s)). True) ; ()

The second fails with Failed to verify implicit argument: Subtyping check failed; expected type (s:FStar.Seq.Base.seq 'a{ Prims.b2t (FStar.Seq.Base.length s > 0) }); got type FStar.Seq.Base.seq 'a at the level of the argument of the function (where it does not hold) instead of keeping it in the branch of the match

This doesn't have much to do with quantifiers.
I see it as a duplicate of #580 , since what's happening is that the implicit argument of forall is not being type-checked in a context that includes the control-flow assumption of length s <> 0.
The plan to fix this is @aseemr-msr 's two-phase type-checker. .Which is close to completion. Also see #1307


windows

Hi, can we create windows .exe file, or is there any code made for windows??

The code is C90 and should compile fine on Windows

is there windows version?

I can compile it with VS2015 through CMake, but I don't know why it crashed at pitch_downsample, the calling stack is: denoise_proc => rnnoise_process_frame => compute_frame_features => pitch_downsample, what should I do to config this file.

Has anyone had success with running this on Windows 7 or 10 x64?
It works great on Linux, Mac and MSYS2 in Windows when the compiled dll has a dependency on MSYS-2.0.dll.
However, I am unable to get it to run on Windows with no dependencies.  On two separate Windows 7 64bit computers, it compiles in Visual Studio 2015 and MSYS2 MingW64 environments and then the example program runs but cuts out before processing the entire file.  Upon further inspection with custom code, the vad probability after each call to rnnoise_process_frame is always 0.  The same exact code works on Mac, Linux, and MSYS2 when it is dependent on MSYS-2.0.dll.
Please comment on why this might only work on linux variants or confirm/negate/augment my findings.
I noticed this project popped up a couple months ago but it does not appear to be complete. https://github.com/jagger2048/rnnoise-windows , I suspect the project owner was unsuccessful and abandoned the project.
Thanks,

Has anyone had success with running this on Windows 7 or 10 x64?
It works great on Linux, Mac and MSYS2 in Windows when the compiled dll has a dependency on MSYS-2.0.dll.
However, I am unable to get it to run on Windows with no dependencies. On two separate Windows 7 64bit computers, it compiles in Visual Studio 2015 and MSYS2 MingW64 environments and then the example program runs but cuts out before processing the entire file. Upon further inspection with custom code, the vad probability after each call to rnnoise_process_frame is always 0. The same exact code works on Mac, Linux, and MSYS2 when it is dependent on MSYS-2.0.dll.
Please comment on why this might only work on linux variants or confirm/negate/augment my findings.
I noticed this project popped up a couple months ago but it does not appear to be complete. https://github.com/jagger2048/rnnoise-windows , I suspect the project owner was unsuccessful and abandoned the project.
Thanks,

Hi, I recently update the vs2017 project so it can run on win10 x64 platform successfully.
Meanwhile, the rnnoise_process_frame() return 0 maybe is that your input data is not correct. The parameters of the function are S16 format PCM and treat them as float data, not truly F32 format PCM data.

windows file io has the binary mode and text mode.
binary mode is not specified in rnnoise_demo.c.

I compile on Windows and Linux. However run with same PCM files, get diffierent F32 files

I compile on Windows and Linux. However run with same PCM files, get diffierent F32 files

two F32 have the same size, data Not

Meanwhile, the rnnoise_process_frame() return 0 maybe is that your input data is not correct. The parameters of the function are S16 format PCM and treat them as float data, not truly F32 format PCM data.

Thanks. This is exactly what I was doing wrong.  If anyone else runs into this issue.  The floats submitted to rnnoise_process_frame() should be values between -32,768 and 32,767 rather than -1 and 1.

Meanwhile, the rnnoise_process_frame() return 0 maybe is that your input data is not correct. The parameters of the function are S16 format PCM and treat them as float data, not truly F32 format PCM data.

Thanks. This is exactly what I was doing wrong. If anyone else runs into this issue. The floats submitted to rnnoise_process_frame() should be values between -32,768 and 32,767 rather than -1 and 1.

I am trying to run this on real time microphone stream in a plugin. Do we have to convert the samples(float32 by default) to short int and then load it to float type array? Direct conversions are giving distorted output. Any leads will be great in this regard.
Thanks

Meanwhile, the rnnoise_process_frame() return 0 maybe is that your input data is not correct. The parameters of the function are S16 format PCM and treat them as float data, not truly F32 format PCM data.

Thanks. This is exactly what I was doing wrong. If anyone else runs into this issue. The floats submitted to rnnoise_process_frame() should be values between -32,768 and 32,767 rather than -1 and 1.

I am trying to run this on real-time microphone stream in a plugin. Do we have to convert the samples(float32 by default) to short int and then load it to float type array? Direct conversions are giving distorted output. Any leads will be great in this regard.
Thanks

No, you can simply convert the f32 PCM sample to target RNNoise data  format，i.e. data_to_proc = f32 * 32768.0f, not the short int.
For more details, you can reference the following link
B&R
Jagger

Meanwhile, the rnnoise_process_frame() return 0 maybe is that your input data is not correct. The parameters of the function are S16 format PCM and treat them as float data, not truly F32 format PCM data.

Thanks. This is exactly what I was doing wrong. If anyone else runs into this issue. The floats submitted to rnnoise_process_frame() should be values between -32,768 and 32,767 rather than -1 and 1.

I am trying to run this on real-time microphone stream in a plugin. Do we have to convert the samples(float32 by default) to short int and then load it to float type array? Direct conversions are giving distorted output. Any leads will be great in this regard.
Thanks

No, you can simply convert the f32 PCM sample to target RNNoise data format，i.e. data_to_proc = f32 * 32768.0f, not the short int.
For more details, you can reference the following link
B&R
Jagger

Thanks for the suggestion. This seems to be not resolving issue. The input audio samples are converted to [-32768,32767] from [-1,1] but the output samples are having too small in magnitude. (FYI: I am using JUCE framework for this plugin)

Meanwhile, the rnnoise_process_frame() return 0 maybe is that your input data is not correct. The parameters of the function are S16 format PCM and treat them as float data, not truly F32 format PCM data.

Thanks. This is exactly what I was doing wrong. If anyone else runs into this issue. The floats submitted to rnnoise_process_frame() should be values between -32,768 and 32,767 rather than -1 and 1.

I am trying to run this on real-time microphone stream in a plugin. Do we have to convert the samples(float32 by default) to short int and then load it to float type array? Direct conversions are giving distorted output. Any leads will be great in this regard.
Thanks

No, you can simply convert the f32 PCM sample to target RNNoise data format，i.e. data_to_proc = f32 * 32768.0f, not the short int.
For more details, you can reference the following link
B&R
Jagger

Thanks for the suggestion. This seems to be not resolving issue. The input audio samples are converted to [-32768,32767] from [-1,1] but the output samples are having too small in magnitude. (FYI: I am using JUCE framework for this plugin)

the output samples are having too small in magnitude
In rnnoise, its output range [ -32768, 32767], you can just convert to [-1, 1], the PCM format. For the magnitude, maybe you need to add an AGC model after applying noise suppression.

windows file io has the binary mode and text mode.
binary mode is not specified in rnnoise_demo.c.
(i have not tested if specifying binary mode fix the bug)

Solved in #130


The visual results from validation_step are wrong.

Maybe the checkpoint is not updated correctly?

Could you please provide me with the paper version of the hamer_vit_transformer.yaml file?
I am not sure that the batch_size is 8?

The batchsize is the main reason because the loss is a sum loss.

Hello, I have a similar problem: I have trained the hamer on 8*A800 GPU as the original configuration, but the epoch is still 0 until I get to 1,000,000 steps and after I run the demo, there is a mismatch in the visualisation results, what can I do to fix it? Looking forward to your reply.

The batchsize is the main reason because the loss is a sum loss.
Hello, I have a similar problem: I have trained the hamer on 8*A800 GPU as the original configuration, but the epoch is still 0 until I get to 1,000,000 steps and after I run the demo, there is a mismatch in the visualisation results, what can I do to fix it? Looking forward to your reply.


Any explanation of how "model" and "state" are used in the APIs

It seems that "model" and "state" could be used interchangeably with some the naming of the API methods.

@justin808 a very, very good point. we'll make sure not to do this while completing the docs for the next version. thank you!


Add watchdog and kill switch for es-loader #44

Issue #, if available: #44
Description of changes: To avoid unnecessary invocation of es-loader, I added watchdog and es-loader stopper triggered when the SIEM OpenSearch cluster is running out of storage space and FreeStorageSpace remains at 0 for 30 minutes.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

ありがとうございます！


Optimize per-executor neotest coverage collection

Is your feature request related to a problem? Please describe.
#3462 enables neotest coverage, but it may be optimized. Currently covered OPs are stored in a global mutex-protected package-level variable and every Executor modifies it on every VM instruction handling. It slows our tests down if tests are running in parallel.
Describe the solution you'd like
Implement https://github.com/nspcc-dev/neo-go/pull/3462#discussion_r1718516293.
Describe alternatives you've considered
No alternatives, the proposed solution fits perfectly.

Makes sense.


[Windows] Improve the pack script

Support "apk-common-opts" config to pack release app
Support "apk-ext-opt" config to pack extension app

BUG=https://crosswalk-project.org/jira/browse/XWALK-6452

LGTM


feat: add package_ignore_dependencies option

Overall I think this is fine. I don't love the proliferation of package_xyz_dependencies attributes we've got going on, but that's the model for now. I don't know of a better user experience; maybe something like package "annotations".
Another thought I had about this particular problem was declaring "package groups" or something - basically one package would contain multiple wheels. But that's a larger change with presumably hairy edge cases.

yes. i think we can do some rework here.
another issue we could address: when package_build_dependencies are defined, but the package goes away (eg because of renovate updating) the whole thing will break.


Add Log analytics Resource AAD Token

This PR fixes #

please add some context in the PR description and screenshots if available, it is not very clear what are you trying to achieve with this PR

Pull Request Test Coverage Report for Build 742029977

6 of 6 (100.0%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 43.549%

Totals

Change from base Build 739683685:
-0.002%

Covered Lines:
26183

Relevant Lines:
54939

💛  - Coveralls

This PR fixes #

Can you fill out the PR description?


Allow paths in errors from validator methods

Yeah, I'll add info about it to the documentation. I only left it here as a draft for now until I have time to finish it.


Angular routing going wrong

** Please describe which feature you have a question about? **
Actually I have an Angular app with a working routing :
myapp.com/ is home
myapp.com/auth/signin is signin page
When I go on /auth/signin through Angular routing, no problem. But when I go directly to /auth/signin or if I'm already on /auth/signin and refresh page, I got this :
<Error>
AccessDenied
<Message>Access Denied</Message>
<RequestId>6F7B0AEF5C90BB18</RequestId>
<HostId>
s/9h0NkmYgltzWSYiN9UgwQ+8B3Pn2LZJbD0s5/hbRrkrLo6WjKTzEEdxGdCtBpqIpbtfGS6mm8=
</HostId>
</Error>

So I've added redirection through Amplify console and managed to get rid of error with something like that : https://nimb.ws/1tak03
But if I choose 200 rewrite it doesn't work... I can only have my Angular app properly working on /my/path with a redirect from <*> to /index.html with 404 rewrite...
Can someone explain me ? Am I missing something ?
** Provide additional details**
Actually as a workaround I've enabled hosting on my Amplify app and I manage redirect through S3/Cloudfront rules... But it's not what I want, because now I must build myself and publish myself my app because Amplify Console use a managed S3 (if I understand well) and auto build push to this managed S3, not to the "hosting" S3.

Ay,
To solve this problem, I've added this redirection in my Amplify Console :
</^[^.]+$|\.(?!(css|gif|ico|jpg|js|png|txt|svg|woff|ttf|map|json)$)([^.]+$)/>

@KrustyHack Can you reupload the CF File please?

Ay,
https://nimb.ws/BiXlUk.

Hey everyone! I'm currently working on an Angular project and the redirects is life saving, but with this angular project, when I add the amplify routing redirect, it completely changes the fonts across the app. Has anyone experienced this?
As soon as I remove the redirect, the font styles return but then routing is lost... :(

Ay @BrianHHough,
What is the extension of your fonts files ? Check if it exists in the snippet I put above : https://github.com/aws-amplify/amplify-console/issues/489#issuecomment-596192908


FINERACT-1391:AccountNumberPreferencesPrefixEight

Description
Describe the changes made and why they were made.
Ignore if these details are present on the associated Apache Fineract JIRA ticket.
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!

[ ] Write the commit message as per https://github.com/apache/fineract/#pull-requests

[ ] Acknowledge that we will not review PRs that are not passing the build ("green") - it is your responsibility to get a proposed PR to pass the build, not primarily the project's maintainers.

[ ] Create/update unit or integration tests for verifying the changes made.

[ ] Follow coding conventions at https://cwiki.apache.org/confluence/display/FINERACT/Coding+Conventions.

[ ] Add required Swagger annotation and update API documentation at fineract-provider/src/main/resources/static/api-docs/apiLive.htm with details of any API changes

[ ] Submission is not a "code dump".  (Large changes can be made "in repository" via a branch.  Ask on the developer mailing list for guidance, if required.)

FYI our guidelines for code reviews are at https://cwiki.apache.org/confluence/display/FINERACT/Code+Review+Guide.

@rrpawar96  thank you for sending this fix, could you please check and fix the travis issue?
@ptuomola  @vidakovic @thesmallstar  It would be really helpful if any one of you do the code review.

btw: I have a fix for the Travis issue... will submit later

Yes, it seems the Travis build is been fixed by this PR #1981, let me also try again by rebasing.

Hi @ptuomola, The comments has been addressed, please re-review the PR.
Two param digits and prefixCharacter, 'prefixCharacter' will be taken to assign the prefix which will be respective length which will stored in the 'digits' param

@ptuomola @rrpawar96  & @vidakovic
I would like to do a functional test on this please advise

@ptuomola I see the issue has now been fixed.

@rrpawar96  @bharathcgowda  @ptuomola
I pulled this your branch and did a functional test - It works well for me

@ptuomola I see the issue has now been fixed.

@ikimbra  did you do a test please advise

Thanks, @ptuomola for reviewing, I have addressed all comments, Requesting for re-review.
Removed the use of Digits and integer, used the length of Prefix and constant name instead, created new generic logic for creating Prefix and tested.

@ptuomola Comment has been addressed, Removed unused properties.


[Bugfix] Drilldown - Cleaning attached events correctly on destroy() …

(Reopen of https://github.com/zurb/foundation-sites/pull/8550 with a clean history, as asked)
Having a responsive drilldown/dropdown clickable only menu configured like this :
data-responsive-menu="drilldown medium-dropdown" data-click-open="true" data-disable-hover="true"
If you load the page (dropdown mode), then resize to small to get your drilldown, then go back to medium/large to get back your dropdown the event handler for drilldown is still registered in handlerQueue.handlers[0] while the dropdown one is in handlerQueue.handlers[1], thus the drilldown handler catch the request (first array item...) and the dropdown handler never got it...
In this specific case you can't click your dropdown menu anymore after resizing (click event is still registered to the destroyed drilldown menu...). This bug didn't appeared in dropdown hover mode because drilldown is click only mode, and then events does not "collide".
This patch try to unregister every event handlers attached to submenuAnchors on Drilldown, the same way as it is constructed in _prepareMenu in place of the
.off('.zf.drilldown').end().off('zf.drilldown')
which seems to be inconsistent.

This is a nice one, much thanks!

Great, thanks


Flutter doctor with android studio 4.0

I have all the plugins installed, but it's saying I am missing them. Does anyone have this problem?
This only happens after I upgrade to 4.0

Its not compatible https://github.com/flutter/flutter-intellij/issues/4581

Hi @Nothing-Works
I see there's an open issue addressing the case you described (thank you @devmgs)
Please follow up on that issue, I'm closing the current one as duplicate.
If you disagree, please write in the comments and I will reopen it.
Thank you


Node red contrib enocean

This pull request introduces 2 alerts when merging d9e4b19f018158c6fa6deb4ebf7acb7a0a231e26 into 110fdff09bc4cea89fccbdf7d171c50c00f464cc - view on LGTM.com
new alerts:

2 for Unused variable, import, function or class

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging d231a1b4a9204937dd77c290c802e2180dbefb49 into 110fdff09bc4cea89fccbdf7d171c50c00f464cc - view on LGTM.com
new alerts:

2 for Unused variable, import, function or class

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging f602b7845d1da0c0c8fed6b349f646a29c893876 into 110fdff09bc4cea89fccbdf7d171c50c00f464cc - view on LGTM.com
new alerts:

2 for Unused variable, import, function or class

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging 58b5aced9bcf818df384a981ba77bc4576d30d62 into 110fdff09bc4cea89fccbdf7d171c50c00f464cc - view on LGTM.com
new alerts:

2 for Unused variable, import, function or class

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging 9bd98f8e7bc7a104b1a2101ed1305e2853a22211 into 110fdff09bc4cea89fccbdf7d171c50c00f464cc - view on LGTM.com
new alerts:

2 for Unused variable, import, function or class

Comment posted by LGTM.com

This pull request introduces 2 alerts when merging e392830c2f557424d41e07c423527a8ea6ead145 into 110fdff09bc4cea89fccbdf7d171c50c00f464cc - view on LGTM.com
new alerts:

2 for Unused variable, import, function or class

Comment posted by LGTM.com


Update the dm-or_ephmeral or get_tweet button design

Stuff like this or

Look a bit ugly(the key is having menu running in menu with interaction or something, so this could be updated

Some of them also need a timeout to remove the buttons and say, option not chosen.

get_tweet also needs the new button which will lead to a media paginator(will be disabled if media is None)
Thanks to @Soheab for help with earlier things

I need to fix this, before I finish, maybe hitting normal can just edit the message into using the paginator hmm

Better Design is now used in one.

https://github.com/JDsProjects/JDBot/commit/f0ba9cf2be92b359db014756c92f7103a077c90b
Looks a little cleaner.

Will need to look at things again when updated again after this


Tables with defaulable record fields are not working properly

Description:
It seems record default-able fields are not properly working with in-memory tables
import ballerina/io;

type Person record {
    int id;
    int age = -1;
    float salary;
    string name;
    boolean married;
    float amount = 55.0;
};

public function main() {
    table<Person> personTable = table{
        { key id, age, salary, name, married, amount },
        [ { 1, 30,  300.5, "Mary", true },
          { 2, 20,  300.5, "John", true }
        ]
    };

    io:println(personTable);
}

Current Output: Empty
Expected Output:
id=1 age=30 salary=300.5 name=Mary married=true amount=55.0 id=2 age=20 salary=300.5 name=John married=true amount=55.0

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/19440


pythran optimises variable assignement in loop away

Took me a while to figure out what was going on, but it seems if we create a new variable inside an if statement in a loop it gets optimized away.
The following code:
#pythran export amintest1(float, float[])
def amintest1(x, y):
    d0 = 1000.
    for i in range(y.shape[0]):
        d = abs(x-y[i])
        if d < d0:
            d0 = d
            j = i
    return j, d0

gets optimized to (output of pythran -P test.py)
def amintest1(x, y):
    d0 = 1000.0
    for i in __builtin__.range(__builtin__.getattr(y, 'shape')[0]):
        d = __builtin__.abs((x - y[i]))
        if (d < d0):
            d0 = d
            pass
    return (i, d0)

note the assignment to j has been optimized away thus will give wrong results. If we declare j before the loop works correctly.
def amintest2(x, y):
    d0 = 1000.
    j = 0
    for i in range(y.shape[0]):
        d = abs(x-y[i])
        if d < d0:
            d0 = d
            j = i
    return j, d0

becomes:
def amintest2(x, y):
    d0 = 1000.0
    j = 0
    for i in __builtin__.range(__builtin__.getattr(y, 'shape')[0]):
        d = __builtin__.abs((x - y[i]))
        if (d < d0):
            d0 = d
            j = i
    return (j, d0)

Also if I compile with pythran -03 -march=native -ffast-math -fopenmp -DUSE_XSIMD the time difference is surprisingly large (can assigning one variable more make such a difference?)
In [1]: x=np.random.randn(10**6)

In [2]: import test

In [3]: timeit(test.amintest1(x[100000], x))
485 µs ± 36.4 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

In [4]: timeit(test.amintest2(x[100000], x))
3.24 ms ± 25.8 µs per loop (mean ± std. dev. of 7 runs, 100 loops each)

Thanks for analysing this bug down to a very simple test case. I'll have a look at it.

I can reproduce the issue. Note however that the compiler has no way to guess that the condition will be true at least once, so you may end up with j being undefined. I'd rather have
#pythran export amintest1(float, float[])
def amintest1(x, y):
    j, d0 = 0, y[0]
    for i in range(1, y.shape[0]):
        d = abs(x-y[i])
        if d < d0:
            d0 = d
            j = i
    return j, d0

be invalid and have pythran raise an error about it.
#pythran export amintest1(float, float[])
def amintest1(x, y):
    d0 = 1000.
    for i in range(y.shape[0]):
        d = abs(x-y[i])
        if d < d0:
            d0 = d
            j = i
    return j, d0

Not sure I understood correctly, but you are saying that the second case should raise an error?
I don't have an issue initialising all variables before if statements, however I think this should be well documented and a clear error raised in the cases when they have not been initialised, because this is a bitch to debug (and somewhat unexpected) if it happens somewhere inside a large function.

BTW is the large timing difference expected?


server side rendering with new angular cli fails

Hi Everyone,
I'm submitting an issue for server side rendering with new angular cli
based on this link "https://github.com/swimlane/ngx-datatable/blob/master/docs/universal/server-side-rendering.md" we should introduce two injectable service ( ServerDimensionsHelper, ServerScrollBarHelper ) for SSR support. but there is  problem with "ServerDimensionsHelper"
[x ] bug report => search github for a similar issue or PR before submitting
[] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior

when we run npm run ng build we get below error.
ate: 2019-09-02T07:44:12.427Z - Hash: ef8354dd966e154629bf - Time: 31826ms

WARNING in ./node_modules/express/lib/view.js 81:13-25
Critical dependency: the request of a dependency is an expression

ERROR in ./node_modules/cookie-signature/index.js
Module not found: Error: Can't resolve 'crypto' in '/Users/ehsan/Sites/lingemy-ui/node_modules/cookie-signature'
ERROR in ./node_modules/etag/index.js
Module not found: Error: Can't resolve 'crypto' in '/Users/ehsan/Sites/lingemy-ui/node_modules/etag'
ERROR in ./node_modules/destroy/index.js
Module not found: Error: Can't resolve 'fs' in '/Users/ehsan/Sites/lingemy-ui/node_modules/destroy'
ERROR in ./node_modules/etag/index.js
Module not found: Error: Can't resolve 'fs' in '/Users/ehsan/Sites/lingemy-ui/node_modules/etag'
ERROR in ./node_modules/express/lib/view.js
Module not found: Error: Can't resolve 'fs' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/mime/mime.js
Module not found: Error: Can't resolve 'fs' in '/Users/ehsan/Sites/lingemy-ui/node_modules/mime'
ERROR in ./node_modules/send/index.js
Module not found: Error: Can't resolve 'fs' in '/Users/ehsan/Sites/lingemy-ui/node_modules/send'
ERROR in ./node_modules/express/lib/application.js
Module not found: Error: Can't resolve 'http' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/express/lib/response.js
Module not found: Error: Can't resolve 'http' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/express/lib/request.js
Module not found: Error: Can't resolve 'http' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/express/lib/request.js
Module not found: Error: Can't resolve 'net' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/content-disposition/index.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/content-disposition'
ERROR in ./node_modules/express/lib/application.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/express/lib/view.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/express/lib/response.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/express/lib'
ERROR in ./node_modules/mime/mime.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/mime'
ERROR in ./node_modules/mime-types/index.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/mime-types'
ERROR in ./node_modules/send/index.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/send'
ERROR in ./node_modules/serve-static/index.js
Module not found: Error: Can't resolve 'path' in '/Users/ehsan/Sites/lingemy-ui/node_modules/serve-static'
ERROR in ./node_modules/destroy/index.js
Module not found: Error: Can't resolve 'stream' in '/Users/ehsan/Sites/lingemy-ui/node_modules/destroy'
ERROR in ./node_modules/send/index.js
Module not found: Error: Can't resolve 'stream' in '/Users/ehsan/Sites/lingemy-ui/node_modules/send'
ERROR in ./node_modules/body-parser/lib/read.js
Module not found: Error: Can't resolve 'zlib' in '/Users/ehsan/Sites/lingemy-ui/node_modules/body-parser/lib'
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>ng: `ng "build"`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>ng script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/ehsan/.npm/_logs/2019-09-02T07_44_13_438Z-debug.log

Expected behavior

Build has to be success, but it fails.
Reproduction of the problem

since it's big and time taking, take clone of angular universal starter project, add ngx-datatable to it, update your angular CLI to latest one, and follow shared link above, try ng build 100% you get failure.
What is the motivation / use case for changing the behavior?

As per angular Developers, in the new CLI it checks that we should not import any server side modules like express or other node modules which are not supported in browser level into the application.
** In ServerDimensionsHelper we are importing import { Request } from 'express';  which is wrong, we should not use express in browser level source.**
Here is the source
import { Injectable, Inject } from '@angular/core';
import { Request } from 'express';
import { REQUEST } from '@nguniversal/express-engine/tokens';
import { DimensionsHelper } from '@swimlane/ngx-datatable';

@Injectable()
export class ServerDimensionsHelper extends DimensionsHelper {

  constructor(@Inject(REQUEST) private request: Request) {
    super();
  }

  getDimensions(element: Element): ClientRect {
    const width = parseInt(this.request.cookies['CH-DW'], 10) || 1000;
    const height = parseInt(this.request.cookies['CH-DH'], 10) || 800;

    const adjustedWidth = width;
    const adjustedHeight = height;

    return {
      height: adjustedHeight,
      bottom: 0,
      top: 0,
      width: adjustedWidth,
      left: 0,
      right: 0
    };
  }
}

Please tell us about your environment:

I'm using MAC OS Mojave and nod 10.16.3 , below is more data.
0 info it worked if it ends with ok
1 verbose cli [ '/Users/ehsan/.nvm/versions/node/v10.16.3/bin/node',
1 verbose cli   '/Users/ehsan/.nvm/versions/node/v10.16.3/bin/npm',
1 verbose cli   'run',
1 verbose cli   'ng',
1 verbose cli   'build' ]
2 info using<EMAIL_ADDRESS>3 info using<EMAIL_ADDRESS>4 verbose run-script [ 'preng', 'ng', 'postng' ]
5 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>6 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>7 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /Users/ehsan/.nvm/versions/node/v10.16.3/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/ehsan/Sites/lingemy-ui/node_modules/.bin:/Users/ehsan/.rvm/gems/ruby-2.4.0/bin:/Users/ehsan/.rvm/gems/ruby-2.4.0@global/bin:/Users/ehsan/.rvm/rubies/ruby-2.4.0/bin:/Users/ehsan/.nvm/versions/node/v10.16.3/bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/go/bin:/Applications/Wireshark.app/Contents/MacOS:/Users/ehsan/.rvm/gems/ruby-2.4.0/bin:/Users/ehsan/.rvm/gems/ruby-2.4.0@global/bin:/Users/ehsan/.rvm/rubies/ruby-2.4.0/bin:/usr/local/sbin:/Users/ehsan/.rvm/bin
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /Users/ehsan/Sites/lingemy-ui
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'ng "build"' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec ng script
13 verbose stack Error<EMAIL_ADDRESS>ng: `ng "build"`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/Users/ehsan/.nvm/versions/node/v10.16.3/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:301:16)
13 verbose stack     at EventEmitter.emit (events.js:198:13)
13 verbose stack     at ChildProcess.<anonymous> (/Users/ehsan/.nvm/versions/node/v10.16.3/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:55:14)
13 verbose stack     at ChildProcess.emit (events.js:198:13)
13 verbose stack     at maybeClose (internal/child_process.js:982:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:259:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /Users/ehsan/Sites/lingemy-ui
16 verbose Darwin 18.7.0
17 verbose argv "/Users/ehsan/.nvm/versions/node/v10.16.3/bin/node" "/Users/ehsan/.nvm/versions/node/v10.16.3/bin/npm" "run" "ng" "build"
18 verbose node v10.16.3
19 verbose npm  v6.9.0
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>ng: `ng "build"`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>ng script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

Table version:

Since ngx-databale doesn't support SSR and we should d tricky actions, YES, it's related t all version I think.

Angular version:

8.2.4

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

it's in build stage, not related to any browser.

Current workaround  is that we can change the ServerDimensionsHelper  to below.
import { Injectable, Inject } from '@angular/core';
import { DimensionsHelper } from '@swimlane/ngx-datatable';

@Injectable()
export class ServerDimensionsHelper extends DimensionsHelper {

  constructor() {
    super();
  }

  getDimensions(element: Element): ClientRect {
    const width = 1000;
    const height = 800;

    const adjustedWidth = width;
    const adjustedHeight = height;

    return {
      height: adjustedHeight,
      bottom: 0,
      top: 0,
      width: adjustedWidth,
      left: 0,
      right: 0
    };
  }
}


Typescript definitions available?

I wanted to use this package in a TypeScript project and couldn't find any type definitions.
I was wondering if there will be any effort made to support TypeScript definitions in the future, either in this repo or in the DefinitelyTyped repo?

I just submitted a PR with definitions to DefinitelyTyped here. They can also be merged directly into this project if @jaredreich would like.


feat(tld): added localhost to accepted TLDs

This allows the common TLD for <IP_ADDRESS> to be used with mehserve as
well (see https://en.wikipedia.org/wiki/.localhost).
I'm still working on this but I wanted to propose the idea for discussion.

Sounds good to me. What OS are you on?

I'm on Ubuntu LTS, but I'll be testing this with workmates on OSX.

Fantastic. I was wondering if it'll need a resolver installed for OSX or not (like we have for the other two domains).

Can you clarify what you mean by resolver?
I'm working on troubleshooting getting this change working on OSX

Just realized what you meant. sudo cp /etc/resolver/meh /etc/resolver/localhost got it working as expected!

2.1.0 released :tada:


Incremental data loading

Investigate keeping cursor open - streaming.

In postgres, the bulk of the time is spent before ANY data is returned by the cursor.  For example, in a request that takes 15 second, the first data is returned around 14.5 seconds after the request is started, and then I can fetch 300,000 records in less than 0.5 seconds (including converting them to a desired format).


add link to ccsbase

Adds a link to ccsbase https://ccsbase.net/ in the ccs database section. (paper and website)
Having the website there does not fit with the pattern of the other databases, but i thought it's nice to have it here since it is the easiest to use among the databases, because you dont have to sign up.

Adds a link to ccsbase https://ccsbase.net/ in the ccs database section. (paper and website)
Having the website there does not fit with the pattern of the other databases, but i thought it's nice to have it here since it is the easiest to use among the databases, because you dont have to sign up.

Thanks, @SteffenHeu!


Error scraping apache: strconf.ParseFloat

Contstantly getting the error:
INFO[0020] Error scraping apache: strconv.ParseFloat: parsing "": invalid syntax  file=apache_exporter.go line=156
Startup command is:
./apache_exporter -log.level debug -scrape_uri "https://my.domain.name/server-status?auto"
The output of the server-status is:

my.domain.name
ServerVersion: Apache/2.4.23 (Unix) OpenSSL/1.0.2h
ServerMPM: worker
Server Built: Aug 31 2016 10:54:08
CurrentTime: Thursday, 08-Sep-2016 15:09:32 CEST
RestartTime: Thursday, 08-Sep-2016 15:08:07 CEST
ParentServerConfigGeneration: 1
ParentServerMPMGeneration: 0
ServerUptimeSeconds: 85
ServerUptime: 1 minute 25 seconds
Load1: 0.00
Load5: 0.01
Load15: 0.05
Total Accesses: 10
Total kBytes: 38
CPUUser: .05
CPUSystem: 0
CPUChildrenUser: 0
CPUChildrenSystem: 0
CPULoad: .0588235
Uptime: 85
ReqPerSec: .117647
BytesPerSec: 457.788
BytesPerReq: 3891.2
BusyWorkers: 2
IdleWorkers: 48
Scoreboard: R_______________________K_______________....................................................................................................
TLSSessionCacheStatus
CacheType: SHMCB
CacheSharedMemory: 512000
CacheCurrentEntries: 0
CacheSubcaches: 32
CacheIndexesPerSubcaches: 88
CacheIndexUsage: 0%
CacheUsage: 0%
CacheStoreCount: 0
CacheReplaceCount: 0
CacheExpireCount: 0
CacheDiscardCount: 0
CacheRetrieveHitCount: 0
CacheRetrieveMissCount: 1
CacheRemoveHitCount: 0
CacheRemoveMissCount: 0

Hi! Thank you for the report. I think it must be resolved now. Can you test it with the HEAD?

I tested it and it works! Thanks!


filename is a zend_string in 8.1

Only fix the build (test suite not ready)

Thanks. I'll make a release too. Don't worry about the test suite, as it never passed anyway.


[breaking] move hydrate and router options to handle

It doesn't really make sense to put these options in the .svelte files because then they get shipped to the client where they're not needed (especially in the case of hydrate), so they make more sense as resolve options
On the client-side, you can implement router: false on a specific link using rel="external". I'm not sure I've ever seen a use case for doing it to all links on a page besides just saving the weight of the JS router, which you can still do with the resolve option. It's possible we could add a method alongside cancel if there were such a use case
For now this makes hydrate and router booleans, but we could easily allow people to continue specifying these options in the page if we made them functions which took the page as a parameter. I think this could be a good idea because prerender really doesn't need to be shipped to the client either, since it only matters during build-time. I'd like to make it a resolve option as well, but fear that may be more disruptive if backwards compatibility is not kept. Sadly it's not tree shaken as unused today
The one possible alternative to this PR for getting stuff out of the client would be to implement something like <script context="module:ssr"> as suggested in https://github.com/sveltejs/rfcs/pull/27
hydrate remains as an app-level option because it is passed to the compiler

Yes, now you would just set hydrate = false, so it should be easier to do

The demo app will need to be updated too:
https://github.com/sveltejs/kit/blob/86f8f8a52c50e3854aa29ab33d9b03b217178070/packages/create-svelte/templates/default/src/routes/about.svelte#L4-L10

The reason we removed export const ssr was that in the cases where you wanted to make it false, you couldn't load the page in order to know the value of the property. Moving it to resolve was a marked reduction in usability, but the trade-off made sense because otherwise it basically didn't work at all.
Here, the trade-off is a bit different — we're sacrificing ergonomics to save a very small number of bytes. If we added the function form, we'd be adding the bytes right back. To me it really doesn't seem worth it.

I'm not sure I've ever seen a use case for doing it to all links on a page besides just saving the weight of the JS router

I have — this covid tracker embed on the front page of https://www.nytimes.com is rendered with SvelteKit — it's part of the same app that powers the tracker itself. Without disabling the router, clicking on a link will cause bad things to happen, since we're on a page that Kit doesn't 'own':

It's not practical to put rel="external" on all the links in the embed, because some of the components might be reused by other parts of the app. This is admittedly a niche scenario, but it demonstrates by controlling router independently of hydrate can be useful.
If we were really motivated to remove those bytes, one option could be to add __config.json files alongside pages, or something like that. It does add a bit of noise and complexity though.

I don't feel very strongly about this, so won't object to closing it. Alternatively, maybe we can just remove the page-level router flag add a method alongside cancel that would let you decide to do the navigation as a server-side navigation instead of a client-side navigation? I think it'd be more powerful and would let us remove all the router enable/disable stuff

a very small number of bytes

It's not just the bytes for the flag, but also for the ability to enable and disable the router, which I'm not sure is strictly necessary and adds a bit more complication

I have — this covid tracker embed on the front page of https://www.nytimes.com

I'm assuming it would disable the router app-wide though in that case and not just on the one page? Because you could still do that here

Alternatively, maybe we can just remove the page-level router flag add a method alongside cancel

That would only work if hydrate was true

It's not just the bytes for the flag, but also for the ability to enable and disable the router

I could probably be persuaded that disabling the router when you navigate to a router = false page is unnecessary. It's really not a lot of code there — three this.enabled references in if conditions and two methods that do nothing beyond toggle the boolean

I'm assuming it would disable the router app-wide though in that case and not just on the one page?

The router is enabled on the app as a whole, but the embeds have router = false


LR and RF notebooks

Notebooks ran successfully on Colab and peer reviewed.

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

@floriankrb over to you.

Dependencies scikit-learn and matplotlib should be added in the notebook...


Added a simpler way to generate bindings if needed.

I replaced  bindgen.sh with a more cross-platform way to generate bindings.
This adds an optional dependency on bindgen.
It can be used like this:
cargo build --features=generate_bindings

Or in the .toml of someone who has this as a dependency by doing something like:
[dependencies.spirv-reflect]
version = "0.1"
features=["generate_bindings"]

There is an issue with rustfmt_bindings(true) not actually formatting the constants at the top of the generated file, I have opened a discussion no the wg-bindged about this since it looks like a bug.

Very nice! Thank you


Guidance for returning result from read

Hi,
I need a little bit of guidance:
wire.read(length, function(err, res) {   // result contains a buffer of bytes });
taking in consideration this function, how can I return result value to be used in the code outside of this function?
something like: var returnedValue = wire.read(length, function(err, res){ return res})
Thank you!

I'd recommend using Promises if you can't use the callback block.


iproute: attaching xdp program not working if built without libbpf

Describe the bug
iproute can not attach xdp programs without libbpf.
Steps To Reproduce
I writen a tiny xdp program (gist) to connect the cloudflare warp vpn with wireguard kernel module. When I tried to attach this xdp program , ip link reported an error.
reproduce (copy wgcf_bpf_helper.c from gist):
~/working/nixos-xdp/ok ❯ ls
shell.nix  wgcf_bpf_helper.c

~/working/nixos-xdp/err ❯ cat shell.nix
with import <nixpkgs> {};

mkShell {
  packages = [
    clang llvm llvmPackages.stdenv glibc_multi libbpf
    iproute
  ];
}

~/working/nixos-xdp/err ❯ clang -O2 -target bpf -c wgcf_bpf_helper.c -o wgcf_bpf
clang-11: warning: argument unused during compilation: '--gcc-toolchain=/nix/store/9pgq84sf921xh97gjj2wh7a7clrcrh4m-gcc-12.2.0' [-Wunused-command-line-argument]

~/working/nixos-xdp/err ❯ sudo ip link set dev wlo1 xdp object wgcf_bpf section wg-cf-xdp-ingress
ELF contains non-{map,call} related relo data in entry 0 pointing to section 8! Compiler bug?!
Error fetching program/map!

Expected behavior
If rebuild iproute with libbpf, it works:
~/working/nixos-xdp/ok ❯ cat shell.nix
with import <nixpkgs> {};

mkShell {
  packages = [
    clang llvm llvmPackages.stdenv glibc_multi libbpf
    (iproute.overrideDerivation (oldAttrs: {
      nativeBuildInputs = [ bison flex pkg-config libbpf ];
    }))
  ];
}

~/working/nixos-xdp/ok ❯ clang -O2 -target bpf -c wgcf_bpf_helper.c -o wgcf_bpf
clang-11: warning: argument unused during compilation: '--gcc-toolchain=/nix/store/9pgq84sf921xh97gjj2wh7a7clrcrh4m-gcc-12.2.0' [-Wunused-command-line-argument]

~/working/nixos-xdp/ok ❯ sudo ip link set dev wlo1 xdp object wgcf_bpf section wg-cf-xdp-ingress

Additional context
I am not sure if this is an upstream bug.   The upstream code looks like it should works with or without libbpf.
FYI,  archlinux and debian does build iproute with libbpf.
Notify maintainers
@primeos
Metadata
~ ❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.12-zen1, NixOS, 23.05 (Stoat), 23.05.20230218.5f4e07d`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.13.2`
 - channels(kasei): `"nixpkgs"`
 - nixpkgs: `/nix/store/z6hqic45nv80ymdbf6hx4xlh3s4s193i-source`

Closed by #321962


[WIP] Title Respawn

This pull request will add a new death and respawn system from the Overcast Network. The documentation of this module is available on the docs page. This pull will close issue #699 too.
I have a few questions;

how should we send titles?
how should we prevert dead people to interact with the gameplay (exactly as the observers)?
how blackout should work?

EDIT I known about getOrDefault, I've fixed it.

Nice. You can send titles, I'll PR your repo later. Make sure you prevent the user from dismounting from the horse. In Title Respawn, will you remove the comments if not, there are spelling mistakes on line 147 (broke should be break), line 119 (reseted should be reset).

For titles, they can be sent using SportBukkit API. The relevant patch is [here] (https://github.com/OvercastNetwork/SportBukkit/blob/master/Bukkit/0034-Title-API.patch).
For dead players, in the observers module there could be a check for if the player is dead, that seems fine to me.
Blackout is just a blindness potion effect, I'm not too sure what you are asking.

Thanks for the answers!

Make sure you prevent the user from dismounting from the horse.

Which event (?) I should use?

@TheMolkaPL VehicleExitEvent

@TheMolkaPL Yeah, I would have default values and check for NPEs

docs page:
https://docs.oc.tc/modules/spawns
the delay defaults to 2 seconds, and the minimum is 1 second
they should not need an update, they would use default values

@Pablete1234 2 seconds is the default value.
StringUtils.timeStringToSeconds(head.getAttributeValue("delay", "2s"));

@twizmwazin @dentmaged what do you think about two new events:

CardinalRespawnEvent - called when player is able to respawn (after the scheduling)
CardinalRespawnClickEvent - (cancellable) called when player left-clicks and tries to respawn.

May be useful for the CTF mode or other modules.

@TheMolkaPL Cardinal already has it's own [CardinalSpawnEvent](https://github.com/twizmwazin/CardinalPGM/blob/master/src/main/java/in/twizmwaz/cardinal/event/CardinalSpawnEvent.java) and `[CardinalDeathEvent]{https://github.com/twizmwazin/CardinalPGM/blob/master/src/main/java/in/twizmwaz/cardinal/event/CardinalDeathEvent.java).

Cardinal already has a CardinalSpawnEvent and CardinalDeathEvent

But CardinalSpawnEvent is called when player respawns. I will call CardinalRespawnEvent when player is able to respawn. In other words; when scheduling to respawn "Left click to respawn in Xs" are "Left click to respawn". The subtitle of the respawn screen may be change to for example "You will respawn when the flag is dropped" istead of "Left click to respawn".

Wouldn't the module's titles be controlled from within the module? Is there any use in Cardinal for this event? I'm not against adding it, although it would be more appropriate to name it CardinalPlayerAbleToRespawnEvent

@twizmwazin I use Cardinal in other plugins so I can keep updating Cardinal and still have my features. I think providing an API for developers is a good idea.

@TiTo-418 What? It has a label, it doesn't need [WIP] in the title anymore.

@TheMolkaPL do not make the player ride a horse, make it ride an armor stand, you can give it invisible:1b, and marker:1b, marker tag makes no collision box (you can place blocks inside it), and with invisible no one can see it

@TheMolkaPL Are you going to finish this?

@dentmaged I don't have access to my computer right now.

@TheMolkaPL I see. Sorry.

@TheMolkaPL Why are you closing this? Will this be rebased or are you unable to continue it?

I don't have time to continue this feature. I can push my local changes if you want.

@TheMolkaPL push your local changes, they are always going to be userfull for whoever keeps implementing this

https://github.com/TheMolkaPL/CardinalPGM/commit/d68abeb11182a22b948a39093f162df4fb5b1243


Patch 1

fixed two typos and added a small GETTING STARTED section

@MarcinKonowalczyk Thank you very much for the improvements!


Gardenlet to project a service account token into the Seed for IaaS authentication purposes

How to categorize this issue?

/area security
/kind enhancement
What would you like to be added:
Gardenlet to project a service account token or centrally managed SPN secret into the Seed for IaaS authentication purposes
Why is this needed:
In order Gardener to start supporting these two scenarios:

Enable alternative authentication method for "GCP|AWS secret configuration" - support default OIDC provider (Garden cluster K8S API as an OIDC provider) and use OIDC for authentication when managing consumer IaaS resources as described in the issue https://github.com/gardener/gardener-extension-provider-gcp/issues/343
Enable alternative authentication method for "Azure secret configuration" by using a default (centrally managed) SPN per Azure Directory (Tenant) ID for managing consumer IaaS resources as described in the issue https://github.com/gardener/gardener-extension-provider-azure/issues/383

gardenlet has to be extended to support the following two scenarios:

gardenlet to request a service account token relevant for the authentication to GCP|AWS|other OIDC client(IaaS) (based on the Seed purpose) from the Gardener OIDC provider and to project it into the Seed, so that the Seed extension provider can find it and use it for authentication
gardenlet to check for refreshed centrally managed Azure SPNs and to update it in the Seed cluster so that the Azure extension provider can find it and use it for authentication to the Azure APIs

Note: For OIDC scenarios, the name of the secret resource defined by the consumer in the project should be used as a token attribute if it is necessary to offer the possibility to configure two or more separate secrets for one and the same GCP proejct | AWS account.

For security reasons, it is very important to make sure that the token audience can be uniquely defined on the Gardener side. This is why the best will be to consider as audience the combination between the name of the K8S namespace (of the project) and the name of the secret, for example <k8s_namespace_name>-<secret_name>.
This is because the names of the secrets are unique only in the context of a Gardener project (K8S namespace).

@donistz No, I don't understand that. If the end user gives us an audience, we can use it. Also, what secret? There is none necessary anymore.

Can we clarify this purpose of the ticket? Otherwise, I assume we have to close it.

@rfranzke  I changed the issue, so that only the Azure SPN is concerned because after discussions we came to the conclusion that it is better for Gardener to integrate with an external OIDC Provider for OIDC based authentication to IaaS APIs instead to play the OIDC Provider role.
@dkistner Please, provide more details what improvements you think are necessary for the gardenlet in order to start supporting the concept for a central Azure SPN per Azure tenant ID. Thanks in advance!

OK, I will move this issue to the Azure extension repo since it is cloud provider specific - gardenlet/gardener itself is agnostic and will not implement such specifics.


Installation not working on elementaryos 5.0 (Debian)

After a (apparently) successful installation, running 'howdy' in the terminal shows this error message

sudo: howdy: command not found

I've searched for similar issues already, and my issue has not been reported yet.
Linux distribution (if applicable):
elementary OS 5.0
Howdy version: 2.5

on ElementaryOS, I needed to do sudo apt-get install software-properties-common before sudo add-apt-repository ppa:boltgolt/howdy

Thanks for figuring it out!
@rileyball would installing software-properties-commonalso enable PPAs for vanilla debian?

Same issue on Pop!_OS 19.04. Command not found.
sudo apt-get install software-properties-common solved nothing for me.

@belosthomas @boltgolt You shouldn't enable PPAs on Debian. Doing so is typically called making a "franken-debian" and does more harm than good to your system.


Revert the lea-based lowering for iadd in the x64 backend

The lea-based lowering for iadd on x64 (#5986) caused a compilation performance regression on some inputs, so we're going to revert for now and investigate other paths for implementing this optimization.

I've done some further investigation of this with @elliottt. We've got a clif function which before this commit took 15 seconds to codegen and after this commit took ~3 seconds. Profiling showed that virtually all time was spent in this loop. We worked together to develop a better means of merging sets there which reduced the compilation time of the test case in question, both before and after this commit, to ~0.2s. It turns out though that this new means isn't actually necessary since the code has already changed on main! This means that relanding this should be a "simple" matter of updating regalloc2 first and then this can re-land.
This bodes well in my mind in that we were pretty surprised that this was the root cause of the regression. Tickling regalloc2 slightly different though makes a lot of sense and it made what was arguably already a pathological case even worse. Fixing the underlying pathology should provide some nice benefits!


tests/lib/spsc_pbuf running on nucleo_l152 and stm32f3_disco

When running the tests/lib/spsc_pbuf/ on nucleo_l152re or stm32f3_disco
The flash prefetch is disabled to PASS the test
With prefetch enable the[test_spsc_pbuf.test_stress_0cpy fails:
$ west build -p auto -b nucleo_l152re tests/lib/spsc_pbuf/ ; west flash
START - test_stress_0cpy
41% remaining:0 ms mss
Ztress execution report:
         context 0:
                 - executed:23059, preempted:0
                 - ticks initial:4, optimized:4
         context 1:
                 - executed:22250, preempted:416
                 - ticks initial:4, optimized:4
        Average CPU load:47%, measurements:150
2% remaining:14000 ms
    Assertion failed at WEST_TOPDIR/zephyr/tests/lib/spsc_pbuf/src/main.c:502: stress_claim_free: check_buffer(buf, len, ctx->read_cnt) is non-zero

 FAIL - test_stress_0cpy in 16.549 seconds

no error reported  (v3.5.0-rc2)


Use gRPC local channel credentials

Add a new field local_credentail into GoogleGrpc which supports Envoy to use gRPC local channel credentials.
Signed-off-by: JimmyCYJ<EMAIL_ADDRESS>Risk Level: Low
Testing: Unit test

cc @lizan @qiwzhang

cc @htuch


Installer fails on Fedora 39

After running the installer, nothing gets written into the home directory.
After running the installer (daedalus-5.4.0-67194-mainnet-8dc751094-x86_64-linux.bin)
cardano2@zen3:~$ ls -la ~/.local
ls: cannot access '/home2/cardano2/.local': No such file or directory

cardano2@zen3:~$ ls -la /tmp/nixuJgF8k/home2/cardano2/
total 20
drwxr-xr-x. 5 cardano2 cardano2 4096 Jul 12 00:56 .
drwxr-xr-x. 3 cardano2 cardano2 4096 Jul 12 00:55 ..
drwxr-xr-x. 3 cardano2 cardano2 4096 Jul 12 00:56 .daedalus
drwxr-xr-x. 4 cardano2 cardano2 4096 Jul 12 00:56 .local
drwxr-xr-x. 2 cardano2 cardano2 4096 Jul 12 00:56 .nix-defexpr

Thanks! You can see in the log that the installer is creating this directory:
mkdir: created directory '/home2/cardano2/.local/share/Daedalus/mainnet'

What is /home2/? It feels a bit non-standard, are you sure it's correct?
Then I see you're trying to list directory .local in your current directory which is ~ according to your shell or $HOME, or your home directory.
The only reason for this that I see is your $HOME directory not being /home2/cardano2?
Can you show me the output of:
❯ echo $HOME

And of pwd (the current directory) right before you try to ls .local:
❯ pwd

And also let's check if it's in the location that the installer deems your home:
❯ ls /home2/cardano2/.local/share/Daedalus/mainnet

Uh-oh, wait, I re-read your original issue, so you said that everything ends up in /tmp/nixuJgF8k/home2/cardano2/, hmmmm.
Let me look at the code of the installer ⏳

Okay, I get it. It's because of this line:
    nixUserChrootFlags = "-c -m /home:/home -p HOME";

The installer is being run from a chroot, where only /home is mounted as /home. We don’t know about /home2. This is rather silly, we should've mounted $HOME instead, sorry.
There have never been any reports about this, I'm assuming because almost no users have their $HOME under somewhere else than /home. But it's a good test case, thanks for finding it. :)
Solution 1 – wait for 5.5.0
We're currently getting rid of the chroot, as it's problematic on the newest Ubuntu. The next release will be a simple installer which unpacks a completely portable directory which you can move to– and run from anywhere.
Solution 2 – try the new installer today
You can download the installer from PR #3207 – from the Artifacts tab in the Linux section: https://buildkite.com/input-output-hk/daedalus/builds/23160
Solution 3 – rename /home2 back to /home
Or maybe symlink? I don’t know your exact setup.

Happy to hear this :) Thanks for helping us test!


Mouse support for Talons

Shoutout to TinkerOS for making the Castle Frankenstein changes.

Oh right. That's what I get for trying to edit the file with github.dev so I can commit without bothering with the terminal

Talons is the only game which I would personally not add mouse support to. I am personally against this change, maybe add a toggle for this.


Cannot  set the focus to the input box for the 'Python Interactive' window

Bug: Notebook Editor, Interactive Window, Editor cells

Steps to cause the bug to occur
In the latest stable release switching focus to the interactive window with ctrl/cmd+1/2 doesn't put the focus in the input box of the Python interactive window (i.e. where it says "Type code here and press shift-enter to run")
It looks like it was first noted and ultimately fixed in https://github.com/Microsoft/vscode-python/issues/4733, but then appeared again in https://github.com/microsoft/vscode-python/issues/7597 and fixed again in https://github.com/microsoft/vscode-python/pull/7632.
While shifting focus to the interactive window works, moving focus to the input box within the window automatically doesn't.
Actual behavior
ctrl/cmd+1/2 focuses the Python interactive window
Expected behavior
Focus should be placed within the input box of the interactive window

Your Jupyter and/or Python environment
Please provide as much info as you readily know

Jupyter server running:                       Local
Extension version:                               2020.1.58038
VS Code version:                                1.41.1
Setting python.jediEnabled:               true
Python and/or Anaconda version:     3.7.3
OS:                                                     Mac
Virtual environment:                          conda

Developer Tools Console Output

Microsoft Data Science for VS Code Engineering Team: @rchiodo, @IanMatthewHuff, @DavidKutu, @DonJayamanne, @greazer

Validate.


Maya API server pod keeps crashing

Description

While installing openebs, maya api server pod keeps crashing
Openshift version 3.10
k8s version 1.10

Expected Behavior

Maya API server has to be run successfully

Current Behavior
[root@gitlab-master ~]# kubectl get pods -n openebs
NAME                                         READY     STATUS             RESTARTS   AGE
maya-apiserver-5f4bc4cd47-hqwpj              0/1       CrashLoopBackOff   5          6m
maya-apiserver-77dfdbc4b7-nzw56              0/1       CrashLoopBackOff   5          6m
openebs-admission-server-ff685dfd9-q8lpc     1/1       Running            0          6m
openebs-ndm-8vwn7                            1/1       Running            0          6m
openebs-ndm-d2pb7                            1/1       Running            0          6m
openebs-ndm-k88sz                            1/1       Running            0          4m
openebs-ndm-rmg9c                            1/1       Running            0          6m
openebs-ndm-slrdb                            1/1       Running            0          6m
openebs-provisioner-85cdcd8469-767b5         1/1       Running            0          6m
openebs-snapshot-operator-7bdc94f4cd-6csq2   2/2       Running            0          6m

maya api server log
maya.log

@nsathyaseelan Can you please attach the logs of maya-apiserver pods.

This was caused to an intermittent checkin in the ci build, which was fixed.


Building Wheels for dlib

Installations:

What to do ?
Specs:
RTX 4090, CUDA version 12.2

any update?

pip install cmake