Interpolation METHOD="spline" fails for TPXO8

Issue
I can easily model tides for TPXO8 using "bilinear" interpolation:
import numpy as np
import pandas as pd
from pyTMD import compute_tide_corrections

compute_tide_corrections(
    x=np.linspace(155, 160, 10),
    y=np.linspace(-30, -40, 10),
    delta_time=pd.date_range("2020-01", "2020-02", periods=10),
    DIRECTORY="/home/jovyan/tide_models_clipped",
    MODEL="TPXO8-atlas",
    EPSG=4326,
    TYPE="drift",
    TIME="datetime",
    METHOD="bilinear",
)

However, if I try the same code with "spline" interpolation (the default), this fails with ValueError: x dimension of z must have same number of elements as x:
compute_tide_corrections(
    x=np.linspace(155, 160, 10),
    y=np.linspace(-30, -40, 10),
    delta_time=pd.date_range("2020-01", "2020-02", periods=10),
    DIRECTORY="/home/jovyan/tide_models_clipped",
    MODEL="TPXO8-atlas",
    EPSG=4326,
    TYPE="drift",
    TIME="datetime",
    METHOD="spline",
)

Other models work fine with "spline" interpolation, e.g. FES2014:
compute_tide_corrections(
    x=np.linspace(155, 160, 10),
    y=np.linspace(-30, -40, 10),
    delta_time=pd.date_range("2020-01", "2020-02", periods=10),
    DIRECTORY="/home/jovyan/tide_models_clipped",
    MODEL="FES2014",
    EPSG=4326,
    TYPE="drift",
    TIME="datetime",
    METHOD="spline",
)

Expected outcome
All supported tide models can be run with the default "spline" interpolation method without raising errors.
Details
I'm using pyTMD=2.0.7 and scipy=1.10.1. Full error:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[23], line 1
----> 1 compute_tide_corrections(
      2     x=np.linspace(155, 160, 10),
      3     y=np.linspace(-30, -40, 10),
      4     delta_time=pd.date_range("2020-01", "2020-02", periods=10),
      5     DIRECTORY="/home/jovyan/tide_models_clipped",
      6     MODEL="TPXO8-atlas",
      7     EPSG=4326,
      8     TYPE="drift",
      9     TIME="datetime",
     10     METHOD="spline",
     11 )

File /env/lib/python3.8/site-packages/pyTMD/compute_tide_corrections.py:318, in compute_tide_corrections(x, y, delta_time, DIRECTORY, MODEL, ATLAS_FORMAT, GZIP, DEFINITION_FILE, EPSG, EPOCH, TYPE, TIME, METHOD, EXTRAPOLATE, CUTOFF, APPLY_FLEXURE, FILL_VALUE, **kwargs)
    316 # read tidal constants and interpolate to grid points
    317 if model.format in ('OTIS','ATLAS','TMD3'):
--> 318     amp,ph,D,c = pyTMD.io.OTIS.extract_constants(lon, lat, model.grid_file,
    319         model.model_file, model.projection, type=model.type,
    320         method=METHOD, extrapolate=EXTRAPOLATE, cutoff=CUTOFF,
    321         grid=model.format, apply_flexure=APPLY_FLEXURE)
    322     # use delta time at 2000.0 to match TMD outputs
    323     deltat = np.zeros((nt), dtype=np.float64)

File /env/lib/python3.8/site-packages/pyTMD/io/OTIS.py:410, in extract_constants(ilon, ilat, grid_file, model_file, EPSG, **kwargs)
    407     hci.data[hci.mask] = hci.fill_value
    408 elif (kwargs['method'] == 'spline'):
    409     # use scipy bivariate splines to interpolate values
--> 410     hci = pyTMD.interpolate.spline(xi, yi, hc, x, y,
    411         dtype=hc.dtype,
    412         reducer=np.ceil,
    413         kx=1, ky=1)
    414     # replace zero values with fill_value
    415     hci.mask |= D.mask

File /env/lib/python3.8/site-packages/pyTMD/interpolate.py:172, in spline(ilon, ilat, idata, lon, lat, fill_value, dtype, reducer, **kwargs)
    170 # construct splines for input data and mask
    171 if np.iscomplexobj(idata):
--> 172     s1 = scipy.interpolate.RectBivariateSpline(ilon, ilat,
    173         idata.data.real.T, **kwargs)
    174     s2 = scipy.interpolate.RectBivariateSpline(ilon, ilat,
    175         idata.data.imag.T, **kwargs)
    176     s3 = scipy.interpolate.RectBivariateSpline(ilon, ilat,
    177         idata.mask.T, **kwargs)

File /env/lib/python3.8/site-packages/scipy/interpolate/_fitpack2.py:1494, in RectBivariateSpline.__init__(self, x, y, z, bbox, kx, ky, s)
   1492     raise ValueError('y must be strictly increasing')
   1493 if not x.size == z.shape[0]:
-> 1494     raise ValueError('x dimension of z must have same number of '
   1495                      'elements as x')
   1496 if not y.size == z.shape[1]:
   1497     raise ValueError('y dimension of z must have same number of '
   1498                      'elements as y')

ValueError: x dimension of z must have same number of elements as x

"linear" also seems to fail with a different error:
compute_tide_corrections(
    x=np.linspace(155, 160, 10),
    y=np.linspace(-30, -40, 10),
    delta_time=pd.date_range("2020-01", "2020-02", periods=10),
    DIRECTORY="/home/jovyan/tide_models_clipped",
    MODEL="TPXO8-atlas",
    EPSG=4326,
    TYPE="drift",
    TIME="datetime",
    METHOD="linear",
)

---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[26], line 1
----> 1 compute_tide_corrections(
      2     x=np.linspace(155, 160, 10),
      3     y=np.linspace(-30, -40, 10),
      4     delta_time=pd.date_range("2020-01", "2020-02", periods=10),
      5     DIRECTORY="/home/jovyan/tide_models_clipped",
      6     MODEL="TPXO8-atlas",
      7     EPSG=4326,
      8     TYPE="drift",
      9     TIME="datetime",
     10     METHOD="linear",
     11 )

File /env/lib/python3.8/site-packages/pyTMD/compute_tide_corrections.py:318, in compute_tide_corrections(x, y, delta_time, DIRECTORY, MODEL, ATLAS_FORMAT, GZIP, DEFINITION_FILE, EPSG, EPOCH, TYPE, TIME, METHOD, EXTRAPOLATE, CUTOFF, APPLY_FLEXURE, FILL_VALUE, **kwargs)
    316 # read tidal constants and interpolate to grid points
    317 if model.format in ('OTIS','ATLAS','TMD3'):
--> 318     amp,ph,D,c = pyTMD.io.OTIS.extract_constants(lon, lat, model.grid_file,
    319         model.model_file, model.projection, type=model.type,
    320         method=METHOD, extrapolate=EXTRAPOLATE, cutoff=CUTOFF,
    321         grid=model.format, apply_flexure=APPLY_FLEXURE)
    322     # use delta time at 2000.0 to match TMD outputs
    323     deltat = np.zeros((nt), dtype=np.float64)

File /env/lib/python3.8/site-packages/pyTMD/io/OTIS.py:419, in extract_constants(ilon, ilat, grid_file, model_file, EPSG, **kwargs)
    416     hci.data[hci.mask] = hci.fill_value
    417 else:
    418     # use scipy regular grid to interpolate values
--> 419     hci = pyTMD.interpolate.regulargrid(xi, yi, hc, x, y,
    420         fill_value=hc.fill_value,
    421         dtype=hc.dtype,
    422         method=kwargs['method'],
    423         reducer=np.ceil,
    424         bounds_error=False)
    425     # replace invalid values with fill_value
    426     hci.mask = (hci.data == hci.fill_value) | D.mask

File /env/lib/python3.8/site-packages/pyTMD/interpolate.py:259, in regulargrid(ilon, ilat, idata, lon, lat, fill_value, dtype, reducer, **kwargs)
    257 data.mask = np.ones((npts), dtype=bool)
    258 # use scipy regular grid to interpolate values for a given method
--> 259 r1 = scipy.interpolate.RegularGridInterpolator((ilat, ilon),
    260     idata.data, fill_value=fill_value, **kwargs)
    261 r2 = scipy.interpolate.RegularGridInterpolator((ilat, ilon),
    262     idata.mask, fill_value=1, **kwargs)
    263 # evaluate the interpolator at input coordinates

File /env/lib/python3.8/site-packages/scipy/interpolate/_rgi.py:242, in RegularGridInterpolator.__init__(self, points, values, method, bounds_error, fill_value)
    240 self.grid, self._descending_dimensions = _check_points(points)
    241 self.values = self._check_values(values)
--> 242 self._check_dimensionality(self.grid, self.values)
    243 self.fill_value = self._check_fill_value(self.values, fill_value)
    244 if self._descending_dimensions:

File /env/lib/python3.8/site-packages/scipy/interpolate/_rgi.py:248, in RegularGridInterpolator._check_dimensionality(self, grid, values)
    247 def _check_dimensionality(self, grid, values):
--> 248     _check_dimensionality(grid, values)

File /env/lib/python3.8/site-packages/scipy/interpolate/_rgi.py:45, in _check_dimensionality(points, values)
     42     raise ValueError("The points in dimension %d must be "
     43                      "1-dimensional" % i)
     44 if not values.shape[i] == len(p):
---> 45     raise ValueError("There are %d points and %d values in "
     46                      "dimension %d" % (len(p), values.shape[i], i))

ValueError: There are 10800 points and 10802 values in dimension 1

That's not good! I'll work on this today.

Thanks @robbibt for finding this bug and pointing it out! Should be fixed in 2.0.8. Let me know if you have any more trouble!

Amazing, thanks so much @tsutterley (as always). I'll test it out and let you know.

It works nicely now on 2.0.8 - awesome! 🚀
A related TPXO8 question: for some of our other models, we've found we can get a really big performance boost by clipping our tide model NetCDF files to Australia before passing them to pyTMD. But for TPXO8, we're currently what I believe is the "compact" versions of the model, e.g.:

These aren't something I can easily clip, so I was wondering if pyTMD also supports the NetCDF versions of TPXO8's model files referenced on the OSU website here? https://www.tpxo.net/tpxo-products-and-registration

Excellent!
You're right. What's hard coded into the model class for TPXO8-atlas is the compact format. A model definition file should be able to be used with subsetted TPXO8-atlas netCDFs. I wrote the following model definition for someone using the (global) netCDF format. Check that the scaling is right for the units (they said it was decimeters).
format      netcdf
name        TPXO8-atlas-v1
model_file   hf.*_tpxo8_atlas_30c_v1.nc
grid_file   grid_tpxo8_atlas_30c_v1.nc
type        z
variable    tide_ocean
version     v1
scale     0.1
compressed  False
reference   https://www.tpxo.net/global/tpxo8-atlas

p.s. the above definition "file" uses the glob functionality to find files (so set --directory to point to where the files are located).

Neat! That's working nicely for me - I had to make two small changes:

Update grid_file to grid_tpxo8atlas_30_v1.nc (without an underscore and the "c" after 30); the grid file has a different name format to the other NetCDFs supplied by OSU.
Change scale to 0.001 to get outputs in metre units

Am now able to model tides for all pyTMD supported models in a fraction of the time - very exciting!

Thanks @robbibt! I'll make those change to my version of the definition file. I'm still thinking about the best way for users to enter these arguments, while still allowing for ease of use. The definition files are a bit of a kludgy solution.
Awesome! That's super exciting! :rocket:


I've built a snap - it's not working properly...

Given this application is fairly new, I thought I'd submit this PR to you and work on transferring ownership to you. It's fairly easy to do but takes a couple of days.
If you're interested, we can take care of this on forum.snapcraft.io
Just let me know if you're interested in pursuing this.
The snap has some issues but it's building - which took a lot of trial and error. That said, I'm thinking part of the issue is related to environment settings but I've kind of reached the end of my rope with this one. Sometimes snaps are easy and other times, well they're like this one :P

@kz6fittycent
I don't think it's worth getting a snap setup for this. The script needs a lot of rework, probably a rework completely using python to get it in to a good state. It has mostly served its intended purpose (provide data for my ISP for a networking issue I was having).
If I ever get back to this project I'll likely scrap everything and use the core from bpytop and implement the speedtest functionality on top of that.

If the public interest in this project increases before then I could possibly setup a docker image for this to decrease all the compatibility issues that currently plagues it.


Update README.md

The "signer" is a parameter in the "createData" function and must be initialized first.

My pull request is not passing the tests


Added a design doc for VRG reconciliation

Signed-off-by: Shyamsundar Ranganathan<EMAIL_ADDRESS>

Let's not merge this yet, I am adding modifications post implementation to this and will repost it for consideration.

Let's not merge this yet, I am adding modifications post implementation to this and will repost it for consideration.

Added status part of the CRD, and amended other state change as per implementation.


Too much text in kernel source picker

Testing #11963

We shouldn't have descriptions on items that just restate the titles. It's a lot of text and makes this picker a little overwhelming. Also the "kernel spec" item shouldn't be listed first when it's less likely than "python environments". And the items that lead to another selection screen should end in ...

Also the "kernel spec" item shouldn't be listed first when it's less likely than "python environments".

kai suggested the other way, that's why we changed it.
his argument was doing the lines  this is extension is supposed to work well for all jupyter kernels, not have too much special casing for python

We shouldn't have descriptions on items that just restate the titles

I like the richer quick pick a bit more. With that said, we don't have detail in a few other places, like the remote picker, so I'm fine with removing the detail and use title only.

@rebornix as this is a change in vscode, i'll leave this for you. I don't want to remove the items from Jupyter extension. i think its simpler to remove from vscode and have a consistent UX (for all controllers and sources from all extensions)

And the items that lead to another selection screen should end in ...

I can add this

This should now be fixed in the latest vscode and jupyter extension.


Highlight special characters in template string

This plugin highlights special characters such as \n, \x1f2e \u{101f2e} in '...' and "..." literals. Template string literals also can have special characters escaped with \ as well.
console.log(`nl\nbs\\bt\`f\u{1f2e}oo`)

This PR adds highlights for the special characters in template strings.
And I also find small mistake in highlight of \u{X...} character. Currently this plugin limits number of digits in \u{...} to 4 or 5, but actually it can be 1~6. Character with 6 digits is for UTF-32. (ref: MDN)
// OK
console.log('\u{1}');
console.log('\u{10FFFF}');

Sorry for the late reply! Thanks for the pull request!


Update tdi pipeline builder param name

Update tdi pipeline builder parameter name from bf_pipeline_config_binary_file to tdi_pipeline_config_binary_file

Ruh-roh. This change appears to have broken the unit test. If you have to make a change to the test (or the target-specific hal file), you'll probably have to do the same thing for all three TDI targets.

Made some minor edits and pipeline passes now (wasnt expecting -- so, original failure was perhaps due to randomness in the DpdkHalTest.SanityCheckFailureWhenExtURLsAreInvalid test case?)

BfPipelineConfig is the name of a Protobuf message. It has not been changed.

BfPipelineConfig is the name of a Protobuf message. It has not been changed.

Right. This looks like a major feature work -- creating new proto files (since original proto is hal/lib/barefoot/bf.proto)
It appears that we are only changing user-exposed names from bf_ to tdi_ or ipu_ as of now and more work is needed to clean the rest of the repository

BfPipelineConfig is the name of a Protobuf message. It has not been changed.

Right. This looks like a major feature work -- creating new proto files (since original proto is hal/lib/barefoot/bf.proto)
It appears that we are only changing user-exposed names from bf_ to tdi_ or ipu_ as of now and more work is needed to clean the rest of the repository

When the Barefoot platform code was copied and hacked to create the TDI platform code, they renamed BfDeviceConfig to TdiDeviceConfig, but they didn't rename BfPipelineConfig. This may have been an oversight (there were losts of those), or it may have been done intentionally, due, perhaps, to references in other software packages we don't know about.
My advice would be to leave it as-is for now, and to restore the TdiPipelineConfig changes you made to their original form.

Made some minor edits and pipeline passes now (wasnt expecting -- so, original failure was perhaps due to randomness in the DpdkHalTest.SanityCheckFailureWhenExtURLsAreInvalid test case?)

It's a really simple test. I don't see how anything shy of a memory corruption issue could have caused it to fail.

My advice would be to leave it as-is for now, and to restore the TdiPipelineConfig changes you made to their original form.

To clarify, you are suggesting that we do not change the parameter from -bf_pipeline_config_binary_file to -tdi_pipeline_config_binary_file?

No, I'm suggesting that you not change BfPipelineConfig to TdiPipelineConfig. because we haven't renamed it. I have no problem with your renaming the command-line flag.

ah ok. got it.
In that case, can I get an approval. I am sitll not sure if its a good idea to merge this late in the cycle -- will wait for a clearance from PM before merging...but lets have the approval completed.

ah ok. got it. In that case, can I get an approval. I am sitll not sure if its a good idea to merge this late in the cycle -- will wait for a clearance from PM before merging...but lets have the approval completed.

Okay. Will do.


Update: use orbclient 0.3.13 with drawing mode property

I published orbimage 0.1.16, so now you can use that


Plutarch 1.1

This is a chance to do breaking changes, but we should still keep it to a minimum.
Please post suggestions.
My suggestion:
type Term :: (a :: k -> Type) -> (s :: k) -> Type

to
data S
type Term :: (a :: S -> Type) -> (s :: S) -> Type

This would fix a ton of headaches wrt. type inference, functional dependencies, and probably many other things, while still keeping illegal terms illegal.

How about we rename some types, or at the very least, relocate them? PMaybe and similar comes to mind - not clear whether its a builtin data or scott encoded type. Related: #56
Some more convenient exports right in Plutarch.Prelude would be nice. I feel like we have enough things now such that users won't need the unsafe functions either. If they do, it'll be quite rare. So #28 is something to think about.
I assume you're already thinking of the PLift and DefaultUni issues raised, so there's that.
Maybe controversial opinion, but I feel like some of the types that can be trivially created using pconstant (in all scenarios, i.e they don't contain Terms) shouldn't have a PlutusType instance. Or at least, not a PCon instance. Especially PUnit.....

We can try to rename, and ideally reduce the number of, various types and constructors. I for one have serious trouble assigning meanings and distinguishing between PData, PAsData, PIsData, PIsDataRepr, and PIsDataReprRepr.

Unsafe functions are already not in the prelude, not sure what you mean by that.

pcon is pretty fundamental, especially once do-notation is added to the mix.

Unsafe functions are already not in the prelude, not sure what you mean by that.

I meant the unsafe function exports from the Plutarch module. It is pretty essential to import that module as it stands; I'm just slightly concerned about it also exporting unsafe functions

Some pre-emptive module reorganization could be worth it, the codebase is starting to get big.
Some suggestions:

Organize plutus builtin types separately (Builtin as submodule/s?)

Plutarch.Builtin
Plutarch.Builtin.etc

Section some unsafe_ functions to Plutarch.Unsafe
Types (or better name) submodules for (scott or not) encoded types and their utils

Plutarch.Types.List  (ListLike moved to it's own module probably too)
Plutarch.Types.etc

Putarch.Api.V1 is pretty huge, might be worth splitting and re-exporting. We'll start to want util modules for API types anyway.

some naming suggestions for @blamario's concerns -

PIsDataRepr -> PConstrRepr (since it only really encodes constr)

taking a cue from generics-sop:

PIsDataReprRepr -> PSOP, or PCode... or to be more verbose, PConstrEncoding

Two other things:

I really want do-notation. We can do something with QualifiedDo.
We probably want to rename PDataList to e.g. PTuple, PDataHList, PHListData, or something else.

Definitely agree. @kozross

It would be nice if bumping plutus lower-bound requires a minor version bump for plutarch. Bumping plutus frequently comes with incompatibilities with other libraries (e.g. plutus-extra, which upon bumping plutus bumps a major version) and needs to be really necessary, e.g. for GHC bumping reasons. Compatibility with plutus-extra is quite desirable I find for plutarch, since they should theoretically play really nice together, so it's something to keep in mind. @L-as

How about we rename some types, or at the very least, relocate them? PMaybe and similar comes to mind - not clear whether its a builtin data or scott encoded type. Related: #56

I think this should also apply to Plutarch.Rec because IIUC that defines Scott-encoded records. Should be more like Plutarch.ScottEncoded.Rec or Plutarch.Types.ScottEncoded.Rec for avoiding misunderstanding.

.Rec is for mutually recursive values. Data-encoded recursion makes no sense.

I named it .Rec because it can ambiguously refer both to mutually-recursive values and to records in general. It already contains functionality for both, and there should be more to come. I'm open to suggestions if anybody finds it confusing.

#99 is essentially 1.1. We probably want to add release notes, etc. before merging.

OTOH many people are waiting for staging to land, so perhaps we should do the release notes later.

Thanks everyone for helping on this!


ERROR: Failed to load font

Non-fatal error related to loading fonts such as:
Reproducible with or without PDF combined option.


Fillcolor

Not sure if you want oFillColor and oStrokeColor in the project, but here is a pull request for them if you want otherwise feel free to close. Fairly inexperienced at Haskell, so I could have made some mistakes.
Includes:

An oFillColor and oStrokeColor method.
Example that draws a read (1,2) to the screen.

Setting stroke color and fill color to black would overwrite any defaults made by the user. We could get around this by wrapping the colors in a Maybe but I think it would be better to just add a DrawAttributes to the Object structure. I'll close this PR.


[Enhancement] Panel swipe on touch/swipe event listener..

[ ] bug

[x] enhancement

[ ] feature-discussion (RFC)

Framework7 Version: 1.5.2

Platform and Target: Crosswalk cordova Android

Live Link or JSFiddle/Codepen: N/A

What you did
Enabled swipePanel:'left' on the app.
On dragging finger across the screen the panel can be seen to move - as an enhancement can there be another event created that can be used ie
$$('.panel-left').on('panel:swiping', function () {
    //custom code - ie hide DOM
});

As the user puts their finger on the screen and moves  across the screen the event above is initialised - as panel:close'/'open' is only called when finger is raised off the screen.
Expected Behavior
'panel:swiping' to be triggered if swpePanel is enabled to allow developers to customise content panel to help improve performance.
Actual Behavior
There is no event to allow developers to trigger the above behavior.
I will experiment with the touchmove event on the window

http://framework7.io/docs/side-panels.html:

Panel Events
Panel events could be very useful to detect how user interact with your panels, or to do something in JavaScript when Panel opened/closed

Event
Target
Description

panel:open
Panel Element

Event will be triggered when Panel starts its opening animation

panel:opened
Panel Element

Event will be triggered after Panel completes its opening animation

panel:close
Panel Element

Event will be triggered when Panel starts its closing animation

panel:closed
Panel Element

Event will be triggered after Panel completes its closing animation

Would panel:open not suffice?

Unfortunately panel:open only works  when I swipe and move finger off the screen..
I'm using this googlemaps plugin for cordova - https://github.com/mapsplugin/cordova-plugin-googlemaps
The map appears under the browser in a dimension defined by a container div - so you can't layer it over element and so make all layers transparent.
I want to set a background color on page as the user is swiping the panel open or close.
But using the above means as I swipe panel:open is not initialised until I release my finger from the screen and I need the background colour as I'm doing a touchmove with the panel.
I tried $$(document).on('touchmove' but that doesn't initialise if I try the swipe close the panel. Works great swiping the panel open.

Quick example you can see as I swipe the page panel is transparent to show google maps behind the browser so I need to set a BG colour as a default Google map grey and hide the Google map during the transition - as the background is transparent on panel swipe looks wrong.


module 'nemo' has no attribute 'core'

Hi everybody,
I am trying to run a transcribe model on google colab, as:
from deeppavlov import build_model, configs

model = build_model(configs.nemo.asr)
text_batch = model("my_path")

I had an initial problem with model=build_model(configs.nemo.asr)  because model nemo was not found, even if it was installed. Thus I solved this issue installing nemo from Nvidia repository as:
!pip install git+https://github.com/NVIDIA/NeMo.git
However, I am facing this new issue:
2020-05-11 10:47:37.536 ERROR in 'deeppavlov.core.common.params'['params'] at line 112: Exception in <class 'deeppavlov.models.nemo.asr.NeMoASR'>
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/deeppavlov/core/common/params.py", line 106, in from_params
    component = obj(**dict(config_params, **kwargs))
  File "/usr/local/lib/python3.6/dist-packages/deeppavlov/models/nemo/asr.py", line 163, in __init__
    **self.nemo_params['AudioToMelSpectrogramPreprocessor']
  File "/usr/local/lib/python3.6/dist-packages/nemo/collections/asr/audio_preprocessing.py", line 362, in __init__
    super().__init__(n_window_size, n_window_stride)
  File "/usr/local/lib/python3.6/dist-packages/nemo/collections/asr/audio_preprocessing.py", line 74, in __init__
    super().__init__()
  File "/usr/local/lib/python3.6/dist-packages/nemo/backends/pytorch/nm.py", line 136, in __init__
    NeuralModule.__init__(self, name)  # For NeuralModule API
  File "/usr/local/lib/python3.6/dist-packages/nemo/core/neural_modules.py", line 70, in __init__
    super().__init__()
  File "/usr/local/lib/python3.6/dist-packages/nemo/core/neural_interface.py", line 41, in __init__
    self._app_state = nemo.utils.app_state.AppState()
  File "/usr/local/lib/python3.6/dist-packages/nemo/utils/metaclasses.py", line 36, in __call__
    cls.__instances[cls] = super(Singleton, cls).__call__(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/nemo/utils/app_state.py", line 43, in __init__
    self._device = nemo.core.DeviceType.GPU
AttributeError: module 'nemo' has no attribute 'core'

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-54-5e369d1d15ef> in <module>()
----> 1 model = build_model(configs.nemo.asr)

9 frames
/usr/local/lib/python3.6/dist-packages/nemo/utils/app_state.py in __init__(self, device)
     41         # Had to set it to None in argument to avoid circular import at the class initialization phase.
     42         if device is None:
---> 43             self._device = nemo.core.DeviceType.GPU
     44         else:
     45             self._device = device

AttributeError: module 'nemo' has no attribute 'core'

Should I install nemo in some other way or is there a solution for this?
Thanks very much
Stefano

Update:
I restart the runtime and I got this issue right now:
[nltk_data] Downloading package punkt to /root/nltk_data...
[nltk_data]   Package punkt is already up-to-date!
[nltk_data] Downloading package stopwords to /root/nltk_data...
[nltk_data]   Package stopwords is already up-to-date!
[nltk_data] Downloading package perluniprops to /root/nltk_data...
[nltk_data]   Package perluniprops is already up-to-date!
[nltk_data] Downloading package nonbreaking_prefixes to
[nltk_data]     /root/nltk_data...
[nltk_data]   Package nonbreaking_prefixes is already up-to-date!
################################################################################
### WARNING, path does not exist: KALDI_ROOT=/mnt/matylda5/iveselyk/Tools/kaldi-trunk
###          (please add 'export KALDI_ROOT=<your_path>' in your $HOME/.profile)
###          (or run as: KALDI_ROOT=<your_path> python <your_script>.py)
################################################################################

/usr/local/lib/python3.6/dist-packages/nemo/collections/asr/audio_preprocessing.py:57: UserWarning: Could not import torchaudio. Some features might not work.
  warnings.warn('Could not import torchaudio. Some features might not work.')
/usr/local/lib/python3.6/dist-packages/nemo/collections/asr/audio_preprocessing.py:61: UserWarning: Unable to import APEX. Mixed precision and distributed training will not work.
  warnings.warn("Unable to import APEX. Mixed precision and distributed training will not work.")
[NeMo I 2020-05-11 10:59:34 features:144] PADDING: 16
[NeMo I 2020-05-11 10:59:34 features:152] STFT using conv
2020-05-11 10:59:34.545 ERROR in 'deeppavlov.core.common.params'['params'] at line 112: Exception in <class 'deeppavlov.models.nemo.asr.NeMoASR'>
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/deeppavlov/core/common/params.py", line 106, in from_params
    component = obj(**dict(config_params, **kwargs))
  File "/usr/local/lib/python3.6/dist-packages/deeppavlov/models/nemo/asr.py", line 165, in __init__
    self.jasper_encoder = JasperEncoder(**self.nemo_params['JasperEncoder'])
TypeError: __init__() missing 1 required positional argument: 'feat_in'
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-13-5e369d1d15ef> in <module>()
----> 1 model = build_model(configs.nemo.asr)

2 frames
/usr/local/lib/python3.6/dist-packages/deeppavlov/models/nemo/asr.py in __init__(self, load_path, nemo_params_path, **kwargs)
    163             **self.nemo_params['AudioToMelSpectrogramPreprocessor']
    164         )
--> 165         self.jasper_encoder = JasperEncoder(**self.nemo_params['JasperEncoder'])
    166         self.jasper_decoder = JasperDecoderForCTC(num_classes=len(self.labels), **self.nemo_params['JasperDecoder'])
    167         self.greedy_decoder = GreedyCTCDecoder()

TypeError: __init__() missing 1 required positional argument: 'feat_in'

Hi, @Steboss
Did you run ! python -m deeppavlov install asr_tts before running model = build_model(configs.nemo.asr)? Could you write all executed commands before AttributeError: module 'nemo' has no attribute 'core' appeared?
Also, please send all code that led to the TypeError: __init__() missing 1 required positional argument: 'feat_in'. Did you modify either DeepPavlov asr config or NeMo modules configuration file?

Hi @IgnatovFedor
Thanks for your reply!
I am trying now to run  python -m deeppavlov install asr_tts before all so I'll let you know asap.
These are all the commands I run:
!pip install deeppavlov
!pip install git+https://github.com/NVIDIA/NeMo.git
!pip install frozendict
!pip install kaldi_io
!pip install unidecode
!pip install torch_stft
!pip install soundfile
!pip install braceexpand
!pip install webdataset

I am sorry for the so many pip install, but everytime I was running the build_model command I got a missing module

Hi @IgnatovFedor
After running ! python -m deeppavlov install asr_tts :
!pip install deeppavlov
!pip install git+https://github.com/NVIDIA/NeMo.git
!pip install frozendict
!pip install kaldi_io
!pip install unidecode
!pip install torch_stft
!pip install soundfile
!pip install braceexpand
!pip install webdataset
from deeppavlov import build_model, configs
! python -m deeppavlov install asr_tts

model = build_model(configs.nemo.asr)

I have this output:
[nltk_data] Downloading package punkt to /root/nltk_data...
[nltk_data]   Package punkt is already up-to-date!
[nltk_data] Downloading package stopwords to /root/nltk_data...
[nltk_data]   Package stopwords is already up-to-date!
[nltk_data] Downloading package perluniprops to /root/nltk_data...
[nltk_data]   Package perluniprops is already up-to-date!
[nltk_data] Downloading package nonbreaking_prefixes to
[nltk_data]     /root/nltk_data...
[nltk_data]   Package nonbreaking_prefixes is already up-to-date!
################################################################################
### WARNING, path does not exist: KALDI_ROOT=/mnt/matylda5/iveselyk/Tools/kaldi-trunk
###          (please add 'export KALDI_ROOT=<your_path>' in your $HOME/.profile)
###          (or run as: KALDI_ROOT=<your_path> python <your_script>.py)
################################################################################

/usr/local/lib/python3.6/dist-packages/nemo/collections/asr/audio_preprocessing.py:57: UserWarning: Could not import torchaudio. Some features might not work.
  warnings.warn('Could not import torchaudio. Some features might not work.')
/usr/local/lib/python3.6/dist-packages/nemo/collections/asr/audio_preprocessing.py:61: UserWarning: Unable to import APEX. Mixed precision and distributed training will not work.
  warnings.warn("Unable to import APEX. Mixed precision and distributed training will not work.")
[NeMo I 2020-05-11 10:59:34 features:144] PADDING: 16
[NeMo I 2020-05-11 10:59:34 features:152] STFT using conv
2020-05-11 10:59:34.545 ERROR in 'deeppavlov.core.common.params'['params'] at line 112: Exception in <class 'deeppavlov.models.nemo.asr.NeMoASR'>
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/deeppavlov/core/common/params.py", line 106, in from_params
    component = obj(**dict(config_params, **kwargs))
  File "/usr/local/lib/python3.6/dist-packages/deeppavlov/models/nemo/asr.py", line 165, in __init__
    self.jasper_encoder = JasperEncoder(**self.nemo_params['JasperEncoder'])
TypeError: __init__() missing 1 required positional argument: 'feat_in'
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-13-5e369d1d15ef> in <module>()
----> 1 model = build_model(configs.nemo.asr)

2 frames
/usr/local/lib/python3.6/dist-packages/deeppavlov/models/nemo/asr.py in __init__(self, load_path, nemo_params_path, **kwargs)
    163             **self.nemo_params['AudioToMelSpectrogramPreprocessor']
    164         )
--> 165         self.jasper_encoder = JasperEncoder(**self.nemo_params['JasperEncoder'])
    166         self.jasper_decoder = JasperDecoderForCTC(num_classes=len(self.labels), **self.nemo_params['JasperDecoder'])
    167         self.greedy_decoder = GreedyCTCDecoder()

TypeError: __init__() missing 1 required positional argument: 'feat_in'

@Steboss, you don't have to run all this pip install commands. After running python -m deeppavlov install asr_tts all necessary dependencies will be installed. Please, try following code:
! pip install deeppavlov
! python -m deeppavlov install asr
! python -m deeppavlov download asr
from deeppavlov import build_mode, configs
model = build_model(configs.nemo.asr)

The issue was in absence of necessary model files that downloaded with python -m deeppavlov download asr. Alternative way of downloading files is using download=True additional kwarg in build_model.

Thanks @IgnatovFedor now everything's working fine :)
I am trying to convert a wav file into text running  model([ "path_to_my_wav.wav"]) but I have this error:
[NeMo I 2020-05-11 16:24:12 actions:728] Evaluating batch 0 out of 1
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-3-4e492d812e9b> in <module>()
----> 1 text_batch = model(["linee_test.wav"])

17 frames
/usr/local/lib/python3.6/dist-packages/torch/nn/functional.py in _pad(input, pad, mode, value)
   3406             assert len(pad) == 4, '4D tensors expect 4 values for padding'
   3407             if mode == 'reflect':
-> 3408                 return torch._C._nn.reflection_pad2d(input, pad)
   3409             elif mode == 'replicate':
   3410                 return torch._C._nn.replication_pad2d(input, pad)

RuntimeError: Argument #4: Padding size should be less than the corresponding input dimension, but got: padding (256, 256) at dimension 3 of input 4

my wav file is about 150 MB and 15 minutes long
Thanks a million
Stefano

As mentioned in documentation, models work with single channel wav files. This error is most likely occurs due to your file is multi-channel.
If linee_test.wav is multi channel, you could create single channel version using following code:
from scipy.io.wavfile import read, write
sr, data = read('linee_test.wav')
with open('linee_test_mono.wav', 'wb') as fout:
    write(fout, sr, data[:, 0])
'''

Ouch
that's great. Now it works fine
I just changed the title of the issue, so it's better and more related to what discussed.
Thanks very much @IgnatovFedor
I am closing the issue


Add mediaSession API

Currently only supported in chrome v 57 for android. #3171
https://developers.google.com/web/updates/2017/02/media-session
Edge is under construction.
https://developer.microsoft.com/en-us/microsoft-edge/platform/status/mediasession/?q=media session

Also I created a sample with wavesurfer.js library and the API here:
https://github.com/entonbiba/Media-Session-API-wavesurfer-js

Available via MDN: https://caniuse.com/?search=mediaSession


[FEATURE] Add hover effect to the navbar elements and add redirect link to logo too

Description
Currently the navbar elements do not have any hover effect and that looks a bit pale. Would like to add some subtle hover effect to that in all the pages
Also, would like to add homepage redirect link to the logo in all pages
Screenshots

@Arnabdaz Please assign this to me

@PoulavBhowmick03 go ahead

@Arnabdaz  is @PoulavBhowmick03  working on this issue?


Creates sdcore-smf charm.

Creates sdcore-smf charm.
Apologies for the diff, I initially pushed a first draft to main, wanted to delete the repo and start over from a clean history, but I don't have permission for deleting repos. 💡 For the charm.py you might want to view the file instead of the diff, it's pretty much new.
⚠ Don't merge before the fiveg_nrf interface is merged.

There are a bunch of conflicts and as you mentioned the diff makes it hard to review. Do you mind finding a way to create a proper diff to make the review easier?

There are a bunch of conflicts and as you mentioned the diff makes it hard to review. Do you mind finding a way to create a proper diff to make the review easier?

I've created this PR in my personal account, first commit wasn't empty. so I didn't find any way to fix the diff here. I can remove the repo and create another one, but that would take time.

There are a bunch of conflicts and as you mentioned the diff makes it hard to review. Do you mind finding a way to create a proper diff to make the review easier?

I've created this PR in my personal account, first commit wasn't empty. so I didn't find any way to fix the diff here. I can remove the repo and create another one, but that would take time.

No need to delete the repo. Can't you squash the commits or simply create a new branch with all the files in a single new commit?

There are a bunch of conflicts and as you mentioned the diff makes it hard to review. Do you mind finding a way to create a proper diff to make the review easier?

I've created this PR in my personal account, first commit wasn't empty. so I didn't find any way to fix the diff here. I can remove the repo and create another one, but that would take time.

No need to delete the repo. Can't you squash the commits or simply create a new branch with all the files in a single new commit?

Done. Now It should show properly, let me know if that works.

Please add the following branch protection rule:
Require conversation resolution before merging


API url regexps

Make regexps for API more flexible, to allow e.g. teamId contains dashes. Solves #394

:+1:

Coverage remained the same at 23.766% when pulling ca9ca72b835d8d4b5add968937da12501f23da9b on feature/api-url-patterns into 1b4181a7249d246ab558237be4f1ccdd76b453f5 on master.


[Backport] [1.x] ignore_malformed parameter on ip_range data_type throws mapper_parsing_exception (#2429)

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>Description
Backport of https://github.com/opensearch-project/OpenSearch/pull/2429 to 1.x
Issues Resolved
Closes https://github.com/opensearch-project/OpenSearch/issues/1873
Check List

[x] New functionality includes testing.

[x] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I think it needs a rebase
* What went wrong:
Execution failed for task ':distribution:bwc:minor:buildBwcLinuxTar'.
> Building 1.3.0 didn't generate expected file /var/CITOOL/workflow/OpenSearch_CI/PR_Checks/Gradle_Check/search/distribution/bwc/minor/build/bwc/checkout-1.3/distribution/archives/linux-tar/build/distributions/opensearch-min-1.3.0-SNAPSHOT-linux-x64.tar.gz


Provide details for when your extension has suspicious content

I wanted to publish a new version of my extension, and I got the following message:
Your extension has suspicious content. Please fix your extension metadata, or reach contact support if you need assistance.

There hasn't been a change to the extension metadata since the last release. Not sure why it is happening. Also, it would be great to point out the actual issue.

Just verified the documentation again and did a couple of tests, the same happens on the Manage Publishers & Extensions page.

At least, there is a pointer to the docs which point out that the following constraints are checked:

The icon provided in package.json may not be an SVG.
The badges provided in the package.json may not be SVGs unless they are from trusted badge providers.
Image URLs in README.md and CHANGELOG.md need to resolve to https URLs.
Images in README.md and CHANGELOG.md may not be SVGs unless they are from trusted badge providers.

I validated this myself, and all should be valid. Just to make sure I did the following:

Cleared out the readme and only kept the title in it.
Verified that no images are used in the changelog
Removed badges

Packaged and uploaded the vsix file again, but still, validation fails.

@estruyf I apologies you have to experience this.
Could you share your extension details? We will take a look

Here is my compressed vsix file. Hope this helps.
vscode-front-matter-beta-7.0.1956421.vsix.zip

Appreciate your patience while we investigate the issue.
I will see if we can provide you a workaround in the meantime

We're running into the same error message when trying to update our extension. Do you know what could cause this error message?

I encountered the same problem trying to update the Time Travel Debug for C/C++ extension. Here's a screenshot of the error message:

I was unable to contact marketplace support due to issue #347. I've attached the offending .vsix in case that will help diagnose the problem: udb-1.2.0.vsix.zip

We have noticed surge of spam extensions in the marketplace this week. We dialed up the spam detection code, and this is side effect, unfortunately.
Please note that the team is working tirelessly to restore the service, and we apricate your patience

Hi @estruyf @tronical @gareth-rees
We have shipped the fix and you should be able to update your extension as desired. Feel free to retry and confirm here. Thank you for your patience.

Thanks @sathaker, I can confirm that the APIs are working again.

@sathaker @prashantvc I was able to update the Time Travel Debug for C/C++ extension. Thank you for fixing the problem.

We're running into the same issue,  when trying to update the Tabnine extension

Hi @prashantvc
I'm having the same issue, unable to upload vscode theme like before.(extension has suspicious content...)

Even I am facing the same error
Your extension has suspicious content. Please fix your extension metadata, or reach contact support if you need assistance.
githooks-0.1.3.zip
please help me out

@Lakshmikanth2001 As @prashantvc mentioned we enabled the spam detection code, and this is side effect. If you are still facing the problem, please let us know we will investigate further.

no the issue is resolved thank you

Thanks @Lakshmikanth2001  for the update closing the issue.


Feature Request: abort request with AbortController signal

Most probably not a small set of developers had run into wanting to abort an ongoing fetch call. For instance an autocomplete input that makes requests as key events are happening, even with a debouncer.
Now in most browsers AbortController API is available in order to fulfill this task.
This issue is an open question for the community if there's interest in including this feature and what we could achieve. I think an extension of RSAA is needed at least.
What do you guys think? I could definitely set up some code proposals or tests in the next days if this idea gains traction 😃

Love it!
My first question is whats the API? You invoke a fetch call by dispatching a RSAA, how would cancelling that fetch later happen?

Bug question... have not thought about it yet 😅

I see it this way,
const actionCreator = () => ({
    [RSAA]: {
        // ...
        signals: [signal1, signal2, /* ... */]
        // ...
    }
})

when REQUEST was started, abort all signals

And maybe some global signal can be useful

I like this idea!  But would we really need a new Action Type?  According to MSN docs, an aborted fetch should reject the promise, meaning if it could produce a normal Failure action, with a specific error type?
If that's the case, the RSAA would only need to be given the custom signal option?  That way, the feature dispatching the RSAA world need to maintain a reference to the controller in order to abort the request, which seems more a responsibility of an Action Creator util method, and not redux-apu-middleware itself.

And actually, thinking on this more, could we use the extent options RSAA property to supply the signal already?  Would only work if browser supports it, or we supply our own polyfill though... afk ATM but I might try this out a bit on my end too :)

I think we should not polyfill, just like fetch.

So I tested this out with native fetch, and I think this kinda works already 🎉
I still need to play with this more and test... but I was able to "abort" an RSAA generated fetch request, by utilizing the existing options hash supported by the RSAA spec (in current 2.3.0, but likely would work in 3.0.0 as well?).
Here's what I did to "abort" a network request that was initiated by redux-api-middleware:
// example case for basic action creator
const myRsaaCreator = (rsaaProps) => {
  const controller = new window.AbortController()
  const { signal } = controller
  const rsaa = {
    [RSAA]: {
      // ...
      options: { signal }
    }
  }
  return [ rsaa, controller ]
}

// then later dispatch...
const [ rsaa, controller ] = myRsaaCreator()
dispatch(rsaa)

// then later abort...
controller.abort()

tricky part is figuring out where you need access to this controller, so unlike my above example, may not always make sense to tightly couple 1 signal to 1 rsaa.  A more realistic example is that you may have some type of "context change" that you want to issue an "abort" signal to, so that any and/or all pending network requests can be closed to free up per-domain channels :)
@nason I think maybe this just needs to be tested a bit more and documented in v 3.0.0?  I can help out with this is you feel there needs to be additional changes, since it may help to better handle the "error" message and document behavior... here's the error action payload I received when I "aborted" an rsaa fetch:
{
    name: 'RequestError',
    message: 'The user aborted a request.'
}

In v 3.0.0, this would be a Failure action, but might help if we can distinguish an "abort" failure from other types of failures?

@darthrellimnad I think we started using that approach in our codebase and I just recently figured out that on abort SUCCESS action is being called with payload = error InternalError: The user aborted a request. Do you see any workaround to have FAILURE called?


Unable to set color by using hex

If I send the following payload to a light node, it always changes to a specific blue (004bff) instead of the provided color.
Example:
{ payload: { hex: "55fc65" } }

Change-event for the new color is always:
msg.payload {
  on: true
  brightness: 100
  rgb: [0, 75, 255]
  hex: "004bff"
  updated: "2018-02-11T15:57:47+00:00"
}

What is going on here? Using NodeRed 0.17.5 and node v6.11.3

Duplicate of #30

I don't think it's a duplicate of #30 - I am not able to change the color of the Hue bulbs to anything. It will ALWAYS result in blue (004bff). Changing it in the app or the api directly is working fine. There must be an issue in the hex conversion.

I have used the same payload with the same hex value and can not reproduce the error. Can you try this flow with the latest version of HueMagic?
[{"id":"788bf116.22eaf","type":"inject","z":"98ee540.5ceec3","name":"","topic":"","payload":"{\"hex\":\"55fc65\"}","payloadType":"json","repeat":"","crontab":"","once":false,"onceDelay":"","x":340,"y":180,"wires":[["a14298db.f50c08"]]},{"id":"a14298db.f50c08","type":"hue-light","z":"98ee540.5ceec3","name":"Filmlicht","bridge":"94d83a9d.7d7c9","lightid":"7","x":540,"y":180,"wires":[["1c8f713.fefca0f"]]},{"id":"94d83a9d.7d7c9","type":"hue-bridge","z":"","name":"Foddy Hue Bridge","bridge":"<IP_ADDRESS>","key":"F42iT5PO5NtrlZ56AkD8RnmQhsor-4c77n18-0PJ","interval":"5000"}]

I have the exact same issue - no matter what hex value I specify I get the same color: 004bff.
Using NodeRed 0.17.5, HueMagic 1.4.1

Just for confirmation, I do have the same odd "004bff" behaviour as klein0r.

I also can confirm this issue. I have
node v6.11.4
node-red v0.18.4
node-red-contrib-huemagic 1.4.1
HUE Bridge BSB002 |<PHONE_NUMBER>
HUE bloom LLC011 | <IP_ADDRESS>78
Have this problem for a while now

Same here


feat: adds readOnly

Description

Adds readonly
replaces ts-jest with swc/jest

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.13.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add test case showing failure in issue #846

Baseline generated from before 400612015beae1b0f228484f405d88c25ca3a498
Refs #846

#846 is closed as of https://github.com/microsoft/TypeScript-TmLanguage/commit/8975bb1b2e440e860a47f9a5a70621a43d3b89be so I'm going to close this as a cleanup.


Option to support VB for auto-generation

Is there an option to auto generate these same files in VB?

If you compile the files into their own DLL then they will appear to have VB interfaces if you call them from VB.


import logi

HI,
Could someone tell me how to install logi ? search the key word in the internet ,not find such packages.
$python3 quack.py --server
Traceback (most recent call last):
File "quack.py", line 23, in
import onewaymeasurement
File "/home/lichaowen/ultra_ping/onewaymeasurement.py", line 8, in
import logi_pi_timer
File "/home/lichaowen/ultra_ping/logi_pi_timer.py", line 6, in
import logi
$sudo pip install logi
ERROR: Could not find a version that satisfies the requirement logi (from versions: none)
ERROR: No matching distribution found for logi

For one-way latency measurement, you'll need both hosts to have access to a synchronised time source. We used a LOGI Pi FPGA, which we talked to using https://github.com/fpga-logi/logi-tools. Check the README section 'Latency Measurement' for more details.


New version: Unity.Unity.2022 version 2022.3.42f1

Created by 🥟 Dumplings in workflow run #7330.

Log
Updated: 2022.3.41f1 → 2022.3.42f1
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

/azp run

This package installs and launches normally on a Windows 10 VM.


Feature request/question.

I am playing on PO 1.5.0 on Fver 2.0.23
I have set the option when pole is overloaded to "nothing". So I can integrate the mod into my server. Is there a way to view which poles are overloaded? My suggestion is a mode that adds sparks or an icon to overloaded poles to easily identify them.

Thanks for the suggestion. Not a bad idea but I'm afraid I'm not planning to add more features to Power Overload at this point, especially not for 'temporary' modes like this. (There will be an update at some point, containing these commits, but 24c6497939e0c254c7823520f422f2aeed5e9ef0 is completely broken and will need some time to get working, so I'm not sure when it will be done ready to release).

Would you accept a pull request if I implemented it to your satisfaction?

Sorry for the late response - yes I'd accept it if it is well segregated so it doesn't make lots of the existing code too much more complex to read and maintain.


Evaluate default procs with the interaction's binding

Fixes #217.
This is the latest in a long line of attempts.

@AaronLasseigne can you review this? It's not ready to go yet. I still need to do documentation. But this is working and it was less work than I expected. The only commit worth looking at is 87a43e7e1add621a1357693dfb0b761b74506f85.

The general approach is good. I played with it a bit and it seems to work well. I know you said it's not entirely done but I have a few nitpicks that I'll add.

BTW, I think this is a feature enhancement so a minor release seems right to me.

I renamed interaction to context. I didn't choose binding because it's a method that's available on basically everything and I didn't want to shadow it.

:shipit:

I added some documentation.

Looks good.


indent(1) is empty

Issue description
Steps to reproduce
nix-shell -p indent --run 'man indent'

This looks like upstream issue.
$ du -sh */man/indent.1
0	indent-2.2.10/man/indent.1
60K	indent-2.2.12/man/indent.1
48K	indent-2.2.9/man/indent.1

This issue has been fixed by upgrading the package.

Thanks!


Perl 6 support

Hey
First, great tool.
Secondly, unfortunately not much time atm for digging out how to implement a new language.
But if someone reads this and has some free time, a new style for the newly and almost released Perl 6 would be just awesome.
Also, the current Perl syntax could get a face lift for recent Perl news, like function signatures.
Best!

It would've worked if there was a non-zero amount of people watching highlight.js issues list for an opportunity to get themselves busy with implementing a language definition someone's asked about. This is not the reality, however. If someone wants to implement a highlighting definition for a language they don't need a request for that.


Driver interviews USB controller during first run

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
Driver interviews the USB controller, starting with an attempt to query for VersionCCCommandClassGet. I don't think it's valid to do so?
What do you observe?
When starting node-zwave-js for the first time, it attempts to interview the controller, starting with an attempt to send a VersionCCCommandClassGet. That fails after 3 tries. The controller is marked dead, then the interview completes and it is marked alive again.
What did you expect to happen?
I don't expect a VersionCCCommandClassGet to work on a USB serial controller. It also doesn't make much sense for the controller to go dead and then alive?
There don't seem to be any problems caused by these events, it just seems like a weird thing to do.
Steps to reproduce the behavior:

Start node-zwave-js. Wipe the cache files if necessary.
Driver queries controller information.
Driver tries to send VersionCCCommandClassGet command to controller.
Restart node-zwave-js, the interview is not attempted again

Device information
Manufacturer: Zooz
Model name: ZST10 700
Node ID in your network: 1
How are you using node-zwave-js?

[X] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version:
node-zwave-js branch: 8.8.3
zwavejs2mqtt branch: 6.0.2
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
Don't know, this is a new device
If yes, where did it work?
No response
Attach Driver Logfile
zwavejs_2021-12-05.log

Can you easily test if the same happens on 8.7.x?

I double checked what happens with my stick:
14:59:36.036 CNTRLR « [Node 001] received response for protocol info:
                      basic device class:    Static Controller
                      generic device class:  Static Controller
                      specific device class: PC Controller
                      node type:             Controller

and here's yours:
2021-12-06T01:50:38.478Z CNTRLR « [Node 001] received response for protocol info:
                                  basic device class:    Static Controller
                                  generic device class:  Static Controller
                                  specific device class: Gateway
                                  node type:             Controller

The difference is the specific device class Gateway, which must support Manufacturer Specific and Version CCs while the PC Controller has no such requirement.
In any case, no CCs should be queried on the controller.


jQuery 404's in demo site

While testing out the demo, I noticed that I couldn't select an image from the media library. It was a JavaScript issue and was caused by http://mezzanine.jupo.org/static/mezzanine/js/jquery-1.8.3.min.js 404-ing.

It's fixed.


Fade out support for CQ 5.3 - CQ 5.5

Currently, Neba still supports AEM versions that are no longer supported by Adobe. This support can be faded out in a sense that newer releases no longer support these versions. However, bugfixes should still be backported for users on these deprecated versions.

Note: transitioning to a java version incompatible with AEM < 5.6 will effectively end support forn these product versions.

Done. NEBA 4.0.0 will be available for AEM 6.1+, previous AEM versions will still be supported by NEBA 3.x


List of users should include additional information to make it easier to read for admins

When displaying the list of users in an Organization, it should require each employee/user’s name next to the login credentials. Otherwise, no way to differentiate them on the list.

[] Add name field to user
[] Display name field in users#index


Efficiency and typing improvements for webhooks and use native implementations

[x] Use built-in crypto implementations and remove crypto-js and buffer-equal-constant-time dependencies
[x] Eliminate un-necessary conversions back and forth between buffers and strings (signature is parsed as hex)
[x] Eliminate un-necessary array copy (.map(eventJson => eventJson))
[x] Improve typings by specifying the return type as Event[]
[x] Improve typings by allowing the body to be passed in as crypto.BinaryLike which can eliminate un-necessary conversions between buffers and strings
[x] Export verifySignature()

May supercede #167 (sorry 🙏🏼 )

Hi @brettwillis
Do you know when this will be released, please?
We have a ticket for this : https://gocardless.atlassian.net/browse/PDFR-32681

I don't have access to that ticket, and I cannot merge this PR. Need a contributor perhaps @prolific117 or @barrucadu ?


Project configuration updates for sdk 31

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: AdamKobus:x: gegitorYou have signed the CLA already but the status is still pending? Let us recheck it.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: AdamKobus:x: gegitorYou have signed the CLA already but the status is still pending? Let us recheck it.

@mciechanowicz any update when this pull request is gonna be merged?

any update on this? would need it urgently

Hey all! What's the status on this? :)

Hi, how is this PR going? It would be very valuable to get this merged to allow SDK 31.


Updated HUAWEI VRP configuration prompt pattern.

Description
On some versions of HUAWEI VRP, the output of the command display current-configuration includes the current OS version in the following format:
<HOSTNAME>display current-configuration
[V200R009C00SPC500]
#
sysname HOSTNAME
...

Since this version string matches the pattern used to detect the prompt in configuration mode, scrapli stops reading the output from this specific command, since it assumes that a valid prompt has been found.
The updated pattern tries to prevent this from happening by using a regex negative lookahead to exclude the version string from the allowed prompt patterns.
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Tested this change on the devices where scrapli didn't read the entire output from the display current-configuration command, and confirmed that the output is now read until the end.
Also added some new test cases to check that the pattern excludes such versions strings.
Checklist:

[X] My code follows the style guidelines of this project (no GitHub actions compalints! run make lint before
committing!)
[X] I have commented my code, pydocstyle and darglint are happy, docstrings are in google docstring format, and all
docstrings include a summary, args, returns and raises fields (even if N/A)
[X] I have added tests that prove my fix is effective or that my feature works, if adding "functional" tests please
note if there are any considerations for the vrnetlab setup
[X] New and existing unit tests pass locally with my changes

Awesome! Thanks for putting the comments in for that... we'll thank ourselves in the future when we try to remember what the hell we did :D


Check whether a number is a perfect number or not in JavaScript

💥 Proposal
To check whether it is a perfect number or not.
Perfect number is a positive integer which is equal to the sum of its proper positive divisors.
For example: 6 is the first perfect number
Proper divisors of 6 are 1, 2, 3
Sum of its proper divisors = 1 + 2 + 3 = 6.
Hence 6 is a perfect number.
Have you read the Contributing Guidelines on Pull Requests?
Yes

\assign

/assign


Fix LegacyCard design according to the design doc

Design Doc
In LegacyCard.ts, the design isn't in accordance with the design doc. You have to fi the typography of this particular component ACC. to the doc. The styles for the LegacyCard can be found here

I would like to work on this.


[Patching] Fix CVE-2021-3749

Description
axios before v0.21.2 is vulnerable to Inefficient Regular Expression Complexity. Denial of Service (DoS) describes a family of attacks, all aimed at making a system inaccessible to its original and legitimate users. There are many types of DoS attacks, ranging from trying to clog the network pipes to the system by generating a large volume of traffic from many machines (a Distributed Denial of Service - DDoS - attack) to sending crafted requests that cause a system to crash or take a disproportional amount of time to process.
The Regular expression Denial of Service (ReDoS) is a type of Denial of Service attack. Regular expressions are incredibly powerful, but they aren't very intuitive and can ultimately end up making it easy for attackers to take your project down.
CVE-2021-3749
CWE-400
CWE-1333
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:N/A:H

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (see CONTRIBUTING.md)
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Code follows the module structure standards.
[x] Wrote unit and integration tests
[x] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[x] Added relevant godoc comments.
[x] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer
[x] Review Codecov Report in the comment section below once CI passes

The dependency bumps are outside of the scope of this project and should be fixed in the cosmos vuepress theme repository. We have decided not to change in any dependency versions in the short term as we might migrate away from vuepress in the future


could this project be simplified/refactored with FastHTML?

https://github.com/AnswerDotAI/fasthtml
Maybe all the complications with NPM, React, bundling that are not familiar to python users like myself could be removed?

I don't understand the point in fasthtml when we've had dominate for years.
I have a much better idea for rendering the frontend server side, hopefully I can work on it later this year or next.

See #48 which is related.

@samuelcolvin I don't understand the "we've had dominate for years". what and how has fastui dominated?

same, I'm curious. I know pydantic is very popular, but I haven't seen a solution that is widely popular for converting pydantic models into form UIs.
I read #48 I think the value of FastHTML is that instead of writing templates in something few have heard about (Jinja language) you can write the templates in Python that maps cleanly to HTML

@samuelcolvin I don't understand the "we've had dominate for years". what and how has fastui dominated?

Ha, no I meant https://github.com/Knio/dominate

same, I'm curious. I know pydantic is very popular, but I haven't seen a solution that is widely popular for converting pydantic models into form UIs.
I read #48 I think the value of FastHTML is that instead of writing templates in something few have heard about (Jinja language) you can write the templates in Python that maps cleanly to HTML

Ye I agree that's the way to go. I've no idea whether fasthtml or dominate are the best way to go, I harboured some ideas to build something similar in rust, but haven't had the time to put into it.
In the end, fastui has worked well for our use case, but we haven't had the time to put into extending it further yet.
If we do, I guess this is the way we'll go.


Clarify license

Just realized there's no mention of a license type anywhere. Can you please clarify, and might I suggest the MIT license: http://opensource.org/licenses/mit-license.php?

MIT license and please link as mentioned in the license. Thanks.


Azure.Identity initial prototype

These changes were originally in Initial Azure.Core.Identity prototype implementation #5984 which was split into multiple PRs.

Did you think about how we are going to add more diagnosability to the auth process? I think in the current implementation authentication exception lists sources that were tried and why they didn't work.

I'm slightly concerned with having to have a client per AzureCredentials and having to expose client options from every applicable credentials type. For example, how are we going to flow options into EnvironmentCredentialProvider?
I wonder if we can change AzureCredential objects only to store data required for auth and use a single client (maybe in TokenCredentialProvider) to send requests to identity server.

Closing and reopening to unstick CI


video-recorder: certificate verify failed: unable to get local issuer certificate

Hi together,
We've deployed moon2 in version 2.5 to k8s but we're having issues with uploading the video recording to our internal s3, the upload fails with "[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:1002)".
Opening the URL of our internal S3 within a manually started Chrome pod and and uploading a file there succeeds without any certificate warning.
The internal s3 is signed by SwissSign, which is available in the browsers cert store.
Browser: chrome, version 112
Our setup is described in https://github.com/aerokube/moon/issues/377.
Thanks!

@toby181 you then have to add your TLS certificate provide CA to additional trusted CAs: https://aerokube.com/moon/latest/#ca-certs

@vania-pooh
I already did that. I've exported the certificate chain (in total 4 certs) base64 encoded and added it to the additionalTrustedCAs in both the order it was exported and in reversed order.
I've already tried the CA only.
apiVersion: helm.toolkit.fluxcd.io/v2beta1
kind: HelmRelease
metadata:
  name: &name moon2
  namespace: moon1
spec:
  releaseName: *name
  interval: 5m
  timeout: 15m
  chart:
    spec:
      chart: moon2
      version: 2.5.0
      sourceRef:
        kind: HelmRepository
        name: ...
        namespace: ...
  values:
    quota:
      moon:
        namespace: ...
    ingress:
      enabled: true
      ...
    configs:
      default:
        sessionTimeout: 10m
        additionalTrustedCAs: |
          -----BEGIN CERTIFICATE-----

In the video-recorder pod I'm testing the connection with curl and got the following results.
$ curl https://minio.example.com/
curl: (77) error setting certificate verify locations:  CAfile: /etc/pki/tls/certs/ca-bundle.crt CApath: none
$ export CURL_CA_BUNDLE=/etc/ssl/certs/ca-certificates.crt
$ curl https://sre-mr-minio.st2.gfk.com:9000/
-> "AccessDenied" response, which is good

@aandryashin any ideas?

Will check, looks like the issue.
вт, 6 июн. 2023 г., 12:16 Ivan Krutov @.***>:

@aandryashin https://github.com/aandryashin any ideas?
—
Reply to this email directly, view it on GitHub
https://github.com/aerokube/moon/issues/384#issuecomment-1578267289, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23M5WHMVN7LTTUBQEYDXJ3YONANCNFSM6AAAAAAY37E6FA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Fixed in https://github.com/aerokube/moon2/pull/212. Will be released as Moon 2.5.2.


Blog posts in XTable website compliance with ASF policies

The new website brings a few blog posts from OneTable: https://xtable.apache.org/blog/
We should make sure we remove any that is not compliant with ASF policies.

@hiteshs, @zabetak , @jbonofre , I'd appreciate a bit of help from you all to double check this. Thanks


refreshToken and subscriptions

My app uses topic subscriptions to send targeted messages to the apps. In our code I assume that when onRefreshToken is called the user has either reinstalled their app or lost their token somehow and that they get a completely fresh token without subscriptions. Given that assumption I re-subscribe to the topic of interest in onRefreshToken. This seems to work fine but I run into some corner cases where some users end up with multiple subscriptions and all I can see when I check the logs on our app server for one of those tokens is that refreshToken has been called, and not much more.
Can subscriptions persist even if the token is refreshed?

that is a firebase team question. However I don't think you need to resubscribe as it is not mentioned in their docs


F# RFC FS-1032 - Support for F# in the dotnet sdk

A proposal is being put forward by Microsoft to include F# functionality directly into the dotnet SDK (also known as "Microsoft.NET.Sdk").   Although it is not a language change, this is being handled via the F# RFC mechanism as F# RFC 1032 - add F# support to the dotnet sdk, which is the primary next-generation cross-platform SDK for .NET Framework and .NET Core programming. This work can be seen as the next logical step in the evolution of the FSharp.NET.Sdk which is published from this repo.
This proposal directly impacts the work of this project.  Please discuss the proposal and give feedback on it, either below, or on the RFC thread, or on one of the linked PR discussion threads. This proposal has not yet been fully implemented or committed, so it is very important that we discuss it as soon as possible and get eyes on the implementation. Some of the work has started, so we are playing catch-up with regard to transparency here.
For practical purposes the proposal would change very little about .NET Core programming with F#. At the most, if the work is completed, it will simply allow the removal of the reference to FSharp.NET.Sdk from the first line of project files, and allow the optional removal of a reference to the F# compiler package. Benefits such as an NGEN'd compiler are explained in the RFC, as are some of the drawbacks raised by community members.

Closing.
Applied with 2.0.0-preview2, see 2.0.0-preview2 wiki page for more info


a fix for #1377

Pull request checklist

[x] All tests pass. Demo project builds and runs.
[x] I have resolved any merge conflicts.
[x] I have followed the coding style, and reviewed the contributing guidelines. Confirmation: @Lucashuang0802

This fixes issue #1377.
What's in this pull request?

Each time you access self.messageBubbleContainerView.subviews.count, it's a different value when removal happens. So, we just need to avoid using subscript.

[Current coverage][cc-pull] is 60.97%

Merging [#1741][cc-pull] into [develop][cc-base-branch] will not change coverage

@@            develop      #1741   diff @@
==========================================
  Files            62         62
  Lines          2224       2224
  Methods         612        612
  Messages          0          0
  Branches        134        135     +1
==========================================
  Hits           1356       1356
  Misses          793        793
  Partials         75         75

[][cc-pull]

Powered by Codecov. Last updated by [d9dd5df...7787bc1][cc-compare]
[cc-base-branch]: https://codecov.io/gh/jessesquires/JSQMessagesViewController/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/jessesquires/JSQMessagesViewController/compare/d9dd5dffc4c7fee53083024d34c64f61dfe98178...7787bc12518d3a953879d8d045ae8b70ad8f8931?src=pr
[cc-pull]: https://codecov.io/gh/jessesquires/JSQMessagesViewController/pull/1741?src=pr

lol. duh. of course. can't believe this has been there for so long. 😢
this still modifies the collection while iterating, we need to iterate over a copy
NSArray *copy = [self.messageBubbleContainerView.subviews copy];
for (NSUInteger i = 0; i < copy.count; i++) {
    UIView *subview = self.messageBubbleContainerView.subviews[i];
    if (subview != _mediaView) {
         [subview removeFromSuperview];
     }
}

thanks for looking into this @Lucashuang0802 ! 😄

Hmm, good to know: http://stackoverflow.com/questions/5163169/fast-enumeration-vs-nsenumerator-in-objective-c/5163334#5163334. I will recreate the fix with your change. @jessesquires thx

actually, i think this is going to have the same problem

Oh yeah, cuz removeFromSuperview will remove the UIView and subviews will have a chance to be out of bounds.  Submit another solution.

lol. this will still have the same issue.
we need to use NSEnumerator and remove subviews while (next != nil)

Hmm, I think internally enumerateObjectsWithOptions:usingBlock: like how you describes? I actually try this code and it's working. Any thoughts on that?
NSMutableArray *mutableArray = [NSMutableArray arrayWithObjects:@2, @3, @3, @2, @7, @3, @1, nil];

    // solution that crashes
    NSArray *copy = [mutableArray copy];
    for (NSUInteger i = 0; i < copy.count; i++) {
        NSNumber *subview = mutableArray[i];
        if ([subview isEqualToNumber:@3]) {
            [mutableArray removeObject:subview];
        }
    }

    // solution that does not crash
    [mutableArray enumerateObjectsWithOptions:NSEnumerationReverse usingBlock:^(NSNumber *number, NSUInteger index, BOOL *stop) {

        if ([number isEqualToNumber:@3]) {
            [mutableArray removeObject:number];
        }
    }];

Ah, of course 👍
One last comment -- there's no need to use NSEnumerationReverse, we can just pass 0 for options

Can't believe it's there for a long time as well. So sorry for not paying too much attention on it. lol. Thx for your patience.

Thanks so much @Lucashuang0802 !

thx for ur patience.


Dashboard can be served on a non-root path

Wait... actually it should be minor bump, not bugfix :thinking:


Component library support

This is very early support for component libraries. I am interested in feedback as this is the first time I have worked on static file hosting and file manipulation.

CompLibTestChannel.zip
This is a sample channel you can use to test this. It is currently set up so that you open VS Code with the compWorkspace.code-workspace file to open it as a work space of two nested work spaces but you also could just have the launch config one level higher in the folder structure.
I have it done this way to simulate my work environment where we have a demo channel and the comp lib checked out in parallel with there own .vscode fold with launch configs and such.

I tested this PR out and it works!!!! Nice Job @chrisdp. Breakpoints in component library NICE.

In addressing feedback I discovered a problem with breakpoints that I will need to address. It seems like the breakpoints are no longer being added into the source channel. This is related to the logic in the deleteBreakpointsForClientPath function. I will look into this after addressing the more simple feedback.

I managed to fix the breakpoint issue and other feedback. We are good for another round of review 😃

@chrisdp This looks great! Fantastic work. I just have the one comment, (adding json schema info for the componentLibraryFiles" section, and then we're good to go.

@chrisdp Oh, another thing. Can you please update the readme with your new debug config settings, as well as a section explaining how this stuff works? If the user doesn't know about it...then it doesn't exist. :D

@TwitchBronBron I will update the README on usage first thing tomorrow. I have already pushed up the schema you requested so feel free to take a look at that.


Add a new sets option

Refs #15

Coverage remained the same at 100.0% when pulling d4ec5ad42b5eccc5711e852598abc52b00ea0cc0 on feature/sets-options into 9da1e04122292aeca72069a4475521dbf7282c73 on master.


Is pipeline parallelism supported?

Feature request
pipeline parallelism
Motivation
To support running model on multiple nodes.
Your contribution
No.

No and it's probably never going to land. Multi node inference increases the latency by too much.
Quantization should be the type of methods to reach for if a given model exceeds the hardware you have.

inference

Thank you!


matmulnbits zezo_point fix

Currently the matmulnbits parsing introduces a fixed type uint8_type for zero_point, and misses int8_type.

https://github.com/microsoft/onnxruntime/blob/main/docs/ContribOperators.md#com.microsoft.MatMulNBits according to the spec zero point can only be uint8/int32/float16/float?

https://github.com/microsoft/onnxruntime/blob/main/docs/ContribOperators.md#com.microsoft.MatMulNBits according to the spec zero point can only be uint8/int32/float16/float?

And if the zero point isn't specified, it can be inferred to be uint8 or int32, would be my guess. I should fix my test case, and do it for int32 instead of int8.
@kahmed10, btw, I don't see any type checking in this parser code -- unless it is done somewhere else.

I don't think we can assume it is handled. If things are type constrained you'll need to add that for those inputs

Is this still needed? Too many failures to approve

These failures have likely nothing to do with this change, @causten, because I added a fix for an int8 type, which isn't exercised anywhere in the codebase. But this PR is still important to debug our current fusion issues, as I am debugging one right now. It is kept as a back-burner item. Thanks.

Closing this. This test-model was used to verify a simple int8 test-case; The type checking can be added later, because int8 case should have been flagged by the parser.


Update Set-CsMeetingConfiguration.md

I have added the following to the description for CustomFooterText: Multiple lines are supported using the Enter key. This request came via Content Idea 92143. Thanks!

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsMeetingConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

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

@Cloud-Writer Thanks Meera!

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsMeetingConfiguration.md
:white_check_mark:Succeeded
View (skype-ps)

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


[Expo Config Plugin] Fixed select identifiers and added system codes support

Follow up. Fixed select-identifiers being added in the wrong place. Also added support for system codes.
Test Plan
Added to a project and used prebuild previews to see if code was being added to the correct location (Info.plist instead of entitlements).

@EvanBacon it's awesome, thanks!


[Issue 5387] Update Netty version for improving memory usage

Motivation
Want to improve memory usage to be able to create many PulsarClients and Consumers.
(Fix https://github.com/apache/pulsar/issues/5387)
Modifications
Update Netty version to 4.1.34.Final and netty-tcnative-boringssl-static version to 2.0.22.Final.
In Netty community, memory increase issue(https://github.com/netty/netty/issues/8814) was reported (at 4.1.32.Final). It was fixed by https://github.com/netty/netty/pull/8825 (at 4.1.34.Final).
When I apply 4.1.34.Final then memory usage was decreased correctly.

https://mvnrepository.com/artifact/io.netty/netty-all/4.1.42.Final is the latest release why not just update to that.

@aahmed-se
Thank you for your review.
In https://github.com/netty/netty/issues/8814 issue, problem was occurred patch version update. So, update version chose just fixed one. Not for critical reason such as build failure.
As follow suggestion, I'll try to update to https://mvnrepository.com/artifact/io.netty/netty-all/4.1.42.Final.

Please take another look.

@equanz Great catch on correlating #5387 with the netty memory usage issue.
The netty upgrade is already part of #3613 which is also breaking down netty-all into individual modules.

The netty version is already upgraded as part of #3613


New language has no flag displayed

When I add a new language, I got no flag displayed for it, even the flag files are exist and the database contains similar entries for old and new languages.

I found that, two files contains path to the flags, but only for the default languages, seems never updated:

public/js/enso.js
public/js/enso.js.map

Did I miss something?

This issue belongs to laravel-enso/localisation repo.

Did you checked the readme?

Did you checked the previous issues on this subject?

even the flag files are exist

Where did you placed the new flag?

You can also contribute with Magyar localisation for the global project.
https://github.com/laravel-enso/Localisation/pulls?q=is%3Apr+is%3Aclosed

Thank for pointing to the readme of localisation, I missed it.
Otherwise, after I added a new language, I was able to translate using the localization manager and I got several messages displayed in this new language.
Regarding flags, I did nothing, there are flags under public/images/vendor/flag-icon-css/flags/1x1 and 4x3.
Using Enso 2.0.35 it just works, I got the flag immediately, without any installation.

Ah ok. I was talking about v2.1.24+
changelog


Reset repo visibility to public, update Apache license

Update Apache license
Reset repo visibility to public

Keeping original Apache license


[RFC] timing only mode

I want to use this module to calculate the response times, but don't want to send back a header. Instead, I want to provide a callback that will be called when the response time is calculated so I can then store this in a stats db/tracker.
Thoughts?

Ok, so I'm thinking of just making it accept a function over the options object all together. Basically what it does is makes the middleware constructor even more polymorphic. The first argument can be a function and it'll be invoked with the response time.
app.use(responseTime(function (req, res, time) {
  // do something
}))

:+1:

Sorry for the time this spent in my backlog :) Published in 2.3.0.


Integration between Home Site & SharePoint Start Page

Hi,
Is the integration between the home site and the SharePoint Start Page being cancelled or postponed?
This was finally a good way to not completely separate the SharePoint Start Page from the rest of the Intranet. Users get confused when they suddenly end up on a completely different page when they click on "SharePoint". Admin cannot control this page as well. What's the point of a root home page..
Thanks for the update..

@BlazeOfFeathers thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Fixed typo in README

isArray example was incorrectly written as isString

Thanks

Duplicate #80. Tests fails.

This was fixed in #81 already, but thanks for contributing!


Custom Import Form:  Bug when form fields are also in Resource.import_id_field

Description
When using a custom import form AND any of those custom form fields are part of the import_id_fields (because those fields are a compound key) in the corresponding custom resource in admin.py, all rows in the csv will be falsely classified as new instances instead of existing instances, thus resulting in a database error due to key violation.
Theory:

django-import-export/import_export/resources.py line 777:

The argument 'row' being passed to self.get_or_init_instance(instance_loader, row) is not including the form fields/values in the custom import form, thus not representing truly what is to be inserted since the custom form field data will be inserted also for each row.
If any of those fields from the custom import form happen to also be in the import_id_fields then when Resource.get_instance() is called, it will not detect an existing instance (resources.py line 390) and will return 'None', and so in turn, get_or_init_instance() is treating that row as a new instance instead of existing.
So the fix would be to make sure that the form fields are added to the 'row', so that get_instance returns correctly

To Reproduce
Steps to reproduce the behavior:

Implement a custom import form as per documentation
Make sure at least one of the fields in the custom import form is part of each row's primary key and thus would also exist in the resources.ModelResource.Meta.import_id_fields
Submit a .csv with records that already exist in the application
Receive a response that is passed back from the database regarding a duplicate key error.

Versions (please complete the following information):

Django Import Export: 3.3.6
Python 3.9.1
Django 4.2.9

Expected behavior
User should be presented with an expected update view which would show which fields will change on the existing records.
Additional context
I discovered this because I have implemented a custom import form that contains two fields, both of which are also in the import_id_fields list in the custom resource (there are 4 fields total in import_id_fields, the other two fields are in the csv).

Thanks for submitting.  It sounds likely that the get_or_init_instance is not detecting your form values, because it would have no way of knowing that these are part of import_id_fields.  Since you are extending logic using forms, I would suggest this would require additional logic on your part, rather than this being in a bug in the core library, although I haven't looked into this in detail yet.  I think the fix would be to add the fields to the row as you suggest.
I would suggest switching to v4 beta if you can because all future dev will be in that release from this year.
It would be ideal if you could

Test you still get the issue when using v4 beta (it's likely that you will but is perhaps worth testing)
A reproducible example using the example app would be ideal.

Thanks @matthewhegarty.  I will look into setting up an environment so I may test the v4 beta and see if I can make a test example of this use case.
Re: is django-import-export is seeing the form fields in my own implementation, I believe yes because  1) When I submit a CSV with new records (i.e. new instances), the form field values are viewable in the Confirm Import screen appended to the CSV values as part of the overall record, and everything inserts correctly.  Also, 2) I see the form values in the debugger as I step through.


Crashing at start

Version
3.0.8 (a1e8985)
Seed
R710-0000-ZTHEA0CF9Y020N2TEL69ZZZZZZZZZZZZZZZ
Scenario
RE 1 (Jill)
Player Character
Claire
What happened?
I just booted Bio 1, went into a new game with Jill's scenario and the black screen with "they have escaped into the mansion..." appeared before crashing. Didn't even get to see the first cutscene.

Some playable skins might crashed the game.  : claire , leon , leon.re2r , regina , rebecca , enrico , wesker , richard.
I wonder why Rebecca skin also crashed too.
I copied  jpn\enemy\CHAR13.EMD to replace biorand\data\re1\pld1\rebecca\CHAR11.EMD , then Rebecca skin work fine.

Nice!! it worked, thank you so much :)

Here's the problem.

The problem is caused by face.png being copied to char10. It has been since fixed in dev. builds.


About GD updates

Your Question
Will geode be down like it has been for 2.206 during every update, or is it only taking long due to the switch to 64bit?

only because of 64 bit, it'll be faster for others

only because of 64 bit, it'll be faster for others

2.2 to 2.204 actually didn't need any update for me.


Simplify CLI args validation and ensure CLI args take precedence over config file.

What does this PR do?

We simplify the processsing of config file sub-sections (e.g. fsdp, megatron, ipex, deepspeed, etc). Previously, any time a dict was encountered in a config, all dicts in the config were re-processed.
CLI options such as --megatron_some_option will no longer be overridden by config options (there was previously no if getattr(args, name, None) check before overwriting the arg.name).

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[x] Did you write any new necessary tests?

Issues
Somewhat related to #2731.
Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.
@muellerzr

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

FYI this broke FSDP as defaults for that are not always None. Making a follow-up

Let me know if I need to resubmit this PR. If you could suggest a test case to catch the problem you found, that would be great.


Uncaught Error: dlopen(~/Downloads/atom-mac/Atom.app/Contents/Resources/app.asar.unpacked/node_mo...

[Enter steps to reproduce:]

...
...

Atom: 1.58.0 x64
Electron: 9.4.4
OS: macOS 11.5.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: dlopen(/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keyboard-layout/build/Release/keyboard-layout-manager.node, 1): image not found
At electron/js2c/asar.js:140

Error: dlopen(/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app.asar.unpacked/node_modules/keyboard-layout/build/Release/keyboard-layout-manager.node, 1): image not found
    at process.func [as dlopen] (electron/js2c/asar.js:140:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:1034:18)
    at Object.func [as .node] (electron/js2c/asar.js:149:18)
    at Module.load (internal/modules/cjs/loader.js:815:32)
    at Module._load (internal/modules/cjs/loader.js:727:14)
    at Function.Module._load (electron/js2c/asar.js:769:28)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:74:18)
    at customRequire (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:1:756278)
    at get_KeyboardLayoutManager (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:14:2894476)
    at get_manager (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:14:2894578)
    at Object.getCurrentKeyboardLayout (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:14:2894704)
    at e.keystrokeForKeyboardEvent (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:14:1119649)
    at KeymapManager.keystrokeForKeyboardEvent (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:11:1224825)
    at KeymapManager.handleKeyboardEvent (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:11:1222810)
    at WindowEventHandler.handleDocumentKeyEvent (/Users/soomin/Downloads/atom-mac/Atom.app/Contents/Resources/app/static/<embedded>:11:286651)

Commands

Non-Core Packages
teletype 0.13.4
termination 0.7.6

Seems like a bad install, have you tried re-installing?

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/21285. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


HMR failed, handshake response code: 200

WebSocket connection to 'ws://jot.dev/_/bundle_hmr' failed: Error during WebSocket handshake: Unexpected response code: 200
When I run an app for the first time, this error shows in the console and HMR doesn't work. Let me know if I can be more specific!

Is jot.dev the correct address and are you using HTTPs by any chance?

jot.dev is correct and I am not using https

We havent seen this in quite a while, think it's ok to close.


[BUG] Docker builds might be broken?

Context

❯ docker inspect elasticdump/elasticsearch-dump:v6.58.0 | jq .[0].RepoDigests[0]
"elasticdump/elasticsearch-dump@sha256:82b05a2b93749b19d91f5dffd26f5d5124862e328c19643473c8b6e5c5fbe6cb"

❯ docker inspect elasticdump/elasticsearch-dump:latest | jq .[0].RepoDigests[0]
"elasticdump/elasticsearch-dump@sha256:aa9ddc3e40b839ed116e417f19970343c2af4fd1298e10902d96c73b2c5ca1de"

❯ docker run --rm -it elasticdump/elasticsearch-dump:v6.58.0 --version
6.54.0

❯ docker run --rm -it elasticdump/elasticsearch-dump:latest --version
6.47.0

Describe the bug
Docker builds seem to be packaging the wrong versions of elasticsearch-dump ?
Current behavior
I would have expected both images to have v6.58.0 in it, but both seems to contain older, and surprisingly different versions.
It seems to me like things started going sideways around the v6.48.0 release, as it points to the v6.49.0 release:
❯ docker run --rm -it elasticdump/elasticsearch-dump:v6.48.0 --version
Unable to find image 'elasticdump/elasticsearch-dump:v6.48.0' locally
v6.48.0: Pulling from elasticdump/elasticsearch-dump
Digest: sha256:67e21b8921288f95e3a7d15df63457cb8403a1d8976b79f7eaf4e4a993c485cb
Status: Downloaded newer image for elasticdump/elasticsearch-dump:v6.48.0
6.49.0

Expected behavior
Docker image version v6.58.0 contains elasticdump v6.58.0

Not exactly, you have to remember the push the app to npm before building the docker. Sometimes the package takes some time to be reflected in npm and the docker build is very fast. You can use the script with buildargs to build a docker image with a specific version if you like.

Nvm it was a invalid caching issue on DockerHub part. I'll rebuild those images manually and re-push thanks!

Rebuilt all the images.