Help text for plots and width of plots

Addition of help text for convergence plot and objective plot (@SQBL please review if the text is fitting). This is related to issue #9.
Furthermore the width of both plots have been set to 800 pixels to avoid cropping by the card. This is related to issue #70 (@j-or Please review if this is a suitable fix).

Looks good! I have changed the texts to use the Typography component, so we use the same sizes and styles everywhere. While I was at it, I removed the bullet points next to the plots (not something you did, I just forgot to remove them earlier 😄 ) Hope that's ok, let me know if something looks off.
For the image width I'm thinking that setting maxWidth instead of width might be good? That way, the image will be its actual size until it no longer fits the card and then scale to the width of the card. You can try it by adding a className to the two <img> and then adding the maxWidth: '100%' style in the plots.style.tsx file (exactly as I've done with the two <li>). Let me know if that doesn't make sense

Thanks, I agree that the formatting looks more consistent with the typography component (and nice that you removed the bullet points).
I also think your idea about setting maxWidth is more elegant. Please check if the latest commit corresponds to your idea (it seems to behave nicely).

Awesome, that's exactly what I was thinking. Hmm, it seems we now have a very large convergence plot though, because the backend spits out a big image? At least in my example. Unless we want a huge plot, maybe we should separately set the convergence img width=100% and maxWidth=some value (maybe the 800 you used earlier) so it's not quite as massive?

I made a small adjustment so the objective plot gets more space in case of many inputs. We can of course always adjust later. Nice work, I've approved so go ahead and merge if you want.


Memory issue merging NetCDF files using xarray.open_mfdataset and to_netcdf

What happened?
I have 5 NetCDF files (1 GiB each). They have 4 dimensions: time, depth, lat, lon. All the files have exactly the same depth, lat, lon. The time axis have the same interval and there are no gaps on this axis for all the 5 files (and there is continuity in the axis between files).
All I am doing is merging the files along the time-axis and saving it to a new NetCDF file.
Running the script, I allocated 185 GiB of memory (the maximum in my cluster).
The program runs until the to_netcdf() function is called. I get an error stating there is not enough memory.
What did you expect to happen?
As the 5 files are 1 GiB each, and I allocated 185 GiB (far more than 5² GiB), I expected the program to run and not require more than the allocated memory (after all, I gave 37 times the combined size of the files).
Minimal Complete Verifiable Example
path = './data/data_*.nc' # files are: data_1.nc data_2.nc data_3.nc data_4.nc data_5.nc
data = xr.open_mfdataset(path)

data = data.load() # uses 5 GiB - tested with a memory profiler

data.to_netcdf('./output/combined.nc') #

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[ ] Complete example — the example is self-contained, including all data and the text of any traceback.
[ ] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
Traceback (most recent call last):
  File "/users/me/code/par2.py", line 78, in <module>
    preprocess_data(year, month)
  File "/users/me/code/par2.py", line 69, in preprocess_data
    data.to_netcdf(path=outpath)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/core/dataset.py", line 1882, in to_netcdf
    return to_netcdf(  # type: ignore  # mypy cannot resolve the overloads:(
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/backends/api.py", line 1210, in to_netcdf
    dump_to_store(
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/backends/api.py", line 1257, in dump_to_store
    store.store(variables, attrs, check_encoding, writer, unlimited_dims=unlimited_dims)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/backends/common.py", line 263, in store
    variables, attributes = self.encode(variables, attributes)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/backends/common.py", line 352, in encode
    variables, attributes = cf_encoder(variables, attributes)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/conventions.py", line 864, in cf_encoder
    new_vars = {k: encode_cf_variable(v, name=k) for k, v in variables.items()}
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/conventions.py", line 864, in <dictcomp>
    new_vars = {k: encode_cf_variable(v, name=k) for k, v in variables.items()}
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/conventions.py", line 273, in encode_cf_variable
    var = coder.encode(var, name=name)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/coding/variables.py", line 170, in encode
    data = duck_array_ops.fillna(data, fill_value)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/core/duck_array_ops.py", line 283, in fillna
    return where(notnull(data), data, other)
  File "/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/xarray/core/duck_array_ops.py", line 270, in where
    return _where(condition, *as_shared_dtype([x, y]))
  File "<__array_function__ internals>", line 180, in where
numpy.core._exceptions._ArrayMemoryError: Unable to allocate 43.6 GiB for an array with shape (280, 200, 277, 754) and data type float32

Anything else we need to know?
I allocated 185 GiB for this job, from my understanding, this means that merging 5 datasets with 1 GiB each requires more than 185 GiB memory. It sounds like a memory leak to me.
I am not the only one with this issue, cf: https://github.com/pydata/xarray/discussions/4890
Environment

/CSC_CONTAINER/miniconda/envs/env1/lib/python3.10/site-packages/_distutils_hack/init.py:33: UserWarning: Setuptools is replacing distutils.
warnings.warn("Setuptools is replacing distutils.")

Thanks for this bug report. FWIW I have also seen this bug recently when helping out a student.
The question here is whether this is an xarray, numpy, or a netcdf bug (or some combo). Can you reproduce the problem using to_zarr()? If so that would rule out netcdf as the culprit.

IIUC the amount of memory is quite what the dimensions suggest (assuming 4byte dtype):
(280 * 200 * 277 * 754 * 4 bytes) / 1024³ = 43.57 GB
I'm not that familiar with the data flow in to_netcdf but it's clear that the whole data is read into memory for some reason. The error happens at backend level, so assuming engine=netcdf4. You might try with engine="h5netcdf" or consider @TomNicholas suggestion of using to_zarr to possibly get the backends out of the equation.
Some questions @benoitespinola :
Can you show the repr's of the single file Dataset's and the repr of the combined?
Are your final data variables of that size (time: 280, depth: 200, lat: 277, lon: 754)?
Did you do some processing with the data, changing attributes/encoding etc?
Is it possible to create your source data files from scratch with random data? An MCVE showing that would help.
Further suggestions:
If you have multiple data variables, drop all but one prior to saving. Is the behaviour consistent for each of your variables?
Try to be explicit in the call to open_mfdataset (eg. adding keyword chunks etc.).
Try to open individual files and use xr.merge/xr.concat.

A single file (from ncdump -h):
dimensions:
	axis_nbounds = 2 ;
	x = 754 ;
	y = 277 ;
	deptht = 200 ;
	time_counter = UNLIMITED ; // (28 currently)
variables:
	float nav_lat(y, x) ;
		nav_lat:standard_name = "latitude" ;
		nav_lat:long_name = "Latitude" ;
		nav_lat:units = "degrees_north" ;
	float nav_lon(y, x) ;
		nav_lon:standard_name = "longitude" ;
		nav_lon:long_name = "Longitude" ;
		nav_lon:units = "degrees_east" ;
	float deptht(deptht) ;
		deptht:name = "deptht" ;
		deptht:long_name = "Vertical T levels" ;
		deptht:units = "m" ;
		deptht:positive = "down" ;
		deptht:bounds = "deptht_bounds" ;
	float deptht_bounds(deptht, axis_nbounds) ;
		deptht_bounds:units = "m" ;
	double time_centered(time_counter) ;
		time_centered:standard_name = "time" ;
		time_centered:long_name = "Time axis" ;
		time_centered:calendar = "gregorian" ;
		time_centered:units = "seconds since 1900-01-01 00:00:00" ;
		time_centered:time_origin = "1900-01-01 00:00:00" ;
		time_centered:bounds = "time_centered_bounds" ;
	double time_centered_bounds(time_counter, axis_nbounds) ;
	double time_counter(time_counter) ;
		time_counter:axis = "T" ;
		time_counter:standard_name = "time" ;
		time_counter:long_name = "Time axis" ;
		time_counter:calendar = "gregorian" ;
		time_counter:units = "seconds since 1900-01-01 00:00:00" ;
		time_counter:time_origin = "1900-01-01 00:00:00" ;
		time_counter:bounds = "time_counter_bounds" ;
	double time_counter_bounds(time_counter, axis_nbounds) ;
	float toce(time_counter, deptht, y, x) ;
		toce:standard_name = "sea_water_potential_temperature" ;
		toce:long_name = "temperature" ;
		toce:units = "degC" ;
		toce:online_operation = "average" ;
		toce:interval_operation = "60 s" ;
		toce:interval_write = "6 h" ;
		toce:cell_methods = "time: mean (interval: 60 s)" ;
		toce:_FillValue = 1.e+20f ;
		toce:missing_value = 1.e+20f ;
		toce:coordinates = "time_centered nav_lat nav_lon" ;
	float soce(time_counter, deptht, y, x) ;
		soce:standard_name = "sea_water_practical_salinity" ;
		soce:long_name = "salinity" ;
		soce:units = "1e-3" ;
		soce:online_operation = "average" ;
		soce:interval_operation = "60 s" ;
		soce:interval_write = "6 h" ;
		soce:cell_methods = "time: mean (interval: 60 s)" ;
		soce:_FillValue = 1.e+20f ;
		soce:missing_value = 1.e+20f ;
		soce:coordinates = "time_centered nav_lat nav_lon" ;
	float taum(time_counter, y, x) ;
		taum:standard_name = "magnitude_of_surface_downward_stress" ;
		taum:long_name = "wind stress module" ;
		taum:units = "N/m2" ;
		taum:online_operation = "average" ;
		taum:interval_operation = "120 s" ;
		taum:interval_write = "6 h" ;
		taum:cell_methods = "time: mean (interval: 120 s)" ;
		taum:_FillValue = 1.e+20f ;
		taum:missing_value = 1.e+20f ;
		taum:coordinates = "time_centered nav_lat nav_lon" ;
	float wspd(time_counter, y, x) ;
		wspd:standard_name = "wind_speed" ;
		wspd:long_name = "wind speed module" ;
		wspd:units = "m/s" ;
		wspd:online_operation = "average" ;
		wspd:interval_operation = "120 s" ;
		wspd:interval_write = "6 h" ;
		wspd:cell_methods = "time: mean (interval: 120 s)" ;
		wspd:_FillValue = 1.e+20f ;
		wspd:missing_value = 1.e+20f ;
		wspd:coordinates = "time_centered nav_lat nav_lon" ;

And after the merge, the only difference is in the time dimension that goes from 28 to 280 (or so)

Just tested with to_zarr and it goes through:
State: COMPLETED (exit code 0)
Nodes: 1
Cores per node: 2
CPU Utilized: 00:07:55
CPU Efficiency: 63.00% of 00:12:34 core-walltime
Job Wall-clock time: 00:06:17
Memory Utilized: 164.89 GB
Memory Efficiency: 44.56% of 370.00 GB

I did an extra run using a memory profiler as such:
import xarray as xr
import zarr
from memory_profiler import profile

@profile
def main():
    path = './data/data_*.nc' # files are: data_1.nc data_2.nc data_3.nc data_4.nc data_5.nc
    data = xr.open_mfdataset(path)

    data = data.load()
    data = data.compute()

    data.to_zarr()

if __name__=='__main__':
    main()

The profiled code was also completed with great success:
State: COMPLETED (exit code 0)
Nodes: 1
Cores per node: 2
CPU Utilized: 00:07:52
CPU Efficiency: 63.61% of 00:12:22 core-walltime
Job Wall-clock time: 00:06:11
Memory Utilized: 165.53 GB
Memory Efficiency: 44.74% of 370.00 GB

Here is the outcome for the memory profiling:
Line #    Mem usage    Increment  Occurrences   Line Contents
=============================================================
    5    156.9 MiB    156.9 MiB           1   @profile
    6                                         def main():
    7    156.9 MiB      0.0 MiB           1       path = './data/data_*.nc' # files are: data_1.nc data_2.nc data_3.nc data_4.nc data_5.nc

    8    209.3 MiB     52.4 MiB           1       data = xr.open_mfdataset(path)
    9
    10  82150.1 MiB  81940.8 MiB           1       data = data.load()
    11  82101.2 MiB    -49.0 MiB           1       data = data.compute()
    12
    13  90091.2 MiB   7990.0 MiB           1       data.to_zarr()

PS: in this test I just realized I loaded 8 files instead of 5.

Answering to the question 'Did you do some processing with the data, changing attributes/encoding etc?':
No processing. I do ask xarray to load the data (and I tried also loading + computing) and the final outcome is the same.
I try now to do an MCVE with dummy data.

By the way, prior to writing this ticket, I also did the following (which did not help):
Drop variables I do not care, keeping dimensions only and toce + soce ; I would expect to need less memory after that.

Because I want to have a worry-free holidays, I wrote a bit of code that basically creates a new NetCDF file from scratch. I load the data from Xarray, change the data to Numpy arrays and use the NetCDF4 library to write the files (does what I want).
In the process, I also slice the data and drop unwanted variables to keep just the bits I want (unlike my original post).
If I call .load() or .compute() on my xarray variable, the memory goes crazy (even if I am dropping unwanted variables - which I would expect to release memory). The same happens for slicing followed by .compute().
Unfortunately, the MCVE will have to wait until I am back from my holidays.

@benoitespinola Did you get along with this? If yes, the solution would be useful to other users. If not, an MCVE is always appreciated.


python310: 3.10.4 -> 3.10.5

https://docs.python.org/release/3.10.5/whatsnew/changelog.html#python-3-10-5-final
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 177643 run on aarch64-darwin 1

1 package built:

hello

But python310 and python39 tests are looking fine.

Well that nixpkgs-review was rather pointless apparently.

This triggered a mass rebuild so I wanted to get a package that would use Python several times in its dependency graph. It means also that python310 succeeded.


Monitoring of the OSIAM components

We should provide monitoring and metrics via the Spring Boot Actuator for all OSIAM backend components. For sure this can only be done, if the components are migrated to Spring Boot.

+1

Let's create the appropriate issues once the Boot'ification is complete.

We should also use the possibility to provide custom application information e.g. the OSIAM version which was already a feature request in auth-server and resource-server

Yes, sounds good. But how do we get the current version into the application.yaml file?

Yes, that's a good question. Maybe by maven within the build process?

Don't we put the version already in the manifest file? I thought it will be read from there by Spring Actuator.

Actuator can read from the pom

Alright, then we do it like this.

I don't think we need to create more issues for this topic. Let's just add actuator und configure everything appropriately: http://docs.spring.io/spring-boot/docs/current/reference/htmlsingle/#production-ready

I've put this into 3.1

Yes. But we should not forget to make the management port configurable (osiam.admin.port?) and the endpoints need to be secured. Maybe also a config to switch it on or off?

I thought we don't want to use an admin port, but secure the endpoints with, well, OSIAM :)

Maybe also a config to switch it on or off?

I don't think we should make this feature "toggleable". Who would switch monitoring off?

I thought we don't want to use an admin port, but secure the endpoints with, well, OSIAM :)

For sure with OSIAM :)

I don't think we should make this feature "toggleable". Who would switch monitoring off?

Nope.


Fix warning set-output command is deprecated

GitHub action updates from Node12 to Node 16 start giving the following warning/s-
The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Awesome, thanks for this PR! I'll make sure to take a look asap!


Add qchem-ops, IsingXX/YY/ZZ and MultiRZ

Before submitting
Please complete the following checklist when submitting a PR:

[X] All new features must include a unit test.
If you've fixed a bug or added code that should be tested, add a test to the
tests directory!

[X] All new functions and code must be clearly commented and documented.
If you do make documentation changes, make sure that the docs build and
render correctly by running make docs.

[ ] Ensure that the test suite passes, by running make test.

[ ] Add a new entry to the .github/CHANGELOG.md file, summarizing the
change, and including a link back to the PR.

[X] Ensure that code is properly formatted by running make format.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

Context:
This PR adds new gates in LightningGPU,

SingleExcitation
SingleExcitationMinus
SingleExcitationPlus
DoubleExcitation
DoubleExcitationMinus
DoubleExcitationPlus
OrbitalRotation
IsingXX/YY/ZZ
MultiRZ

Description of the Change:
The code is added in cuGates_param.hpp and StateVectorCudaBase..hpp where tests are in Test_StateVectorCudaManage.cpp and bindings take place in Bindings.cpp.
Benefits:
Create circuits with more gates.
Possible Drawbacks:
n/a
Related GitHub Issues:
n/a

I close this PR and will create a new one or two with all the changes.


Implicit TLS detection in CMakeLists.txt does not account for MSVC

__thread is not supported on MSVC.
We should add an elseif(MSVC) clause that checks for __declspec(thread) instead.
https://github.com/gperftools/gperftools/blob/fe85bbdf4cb891a67a8e2109c1c22a33aa958c7e/CMakeLists.txt#L357-L367

Sure. Go ahead and add, please.

https://github.com/gperftools/gperftools/pull/1435

Merged https://github.com/gperftools/gperftools/pull/1439


Cannot open Node-RED in vscode

I cannot oppen Node-RED with the shortcut or when I press F1 and go to open Node-RED. Does anyone know a solution for this? Already reinstalled Node-Red and vscode is up to the newest version.

I am experiencing same problem.

Same issue here, re-installed the extension several times, and disabled all other extensions.  Latest version, but portable.

Me too!

I am also experiencing same issue.Any solution?

Depends on VS Code supporing window.localStorage in WebView API: https://github.com/microsoft/vscode/issues/52246

Are there any workarounds, or does this mean that this issue is not going to be resolved?

I have this issue too. Afeter I installed installed in Power, the "node-red" command resulted in this error: Error loading settings file: \Users\FrancoisSt-Arnaud.node-red\settings.js
Now VS won't start the extension either...


added my repo File

@sanjay-kv  Kindly , review it

🚀 Congratulations for your submission
Your points will be reflected in the leaderboard till night.


Desired behavior
the ng g universal works when adding --clientproject
It would be good that when --clientproject not available schematics will pick de defaultproject in aabgular.json
Mention any other details that might be useful (optional)
Already Tested locally, only two small changes to do:
in /universal/schema.json delete  "required": [
"clientProject"
]
in /universal/index.ts within function getClientProject
replace
const  clientProject = workspace.projects[options.clientProject];
with
if(options.clientProject==undefined)
{
options.clientProject = workspace['defaultProject']
}
const  clientProject = workspace.projects[options.clientProject]
and that´s all!!

This issue was moved to angular/angular-cli#12102


Fix: optimize() with num_workers > 1 leads to deletion issues

Before submitting

[x] Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure to update the docs?
[ ] Did you write any new necessary tests?

What does this PR do?
Fixes #245 .
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in GitHub issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Even after increasing the number of workers to (1,2,4), no error occurred. I think this is just a non-deterministic behavior.
So, I'm thinking of modifying chunk name (while writing in cache) to be something like chunk-{rank}-{index}-{uuid}.bin. And when uploading the chunk, drop the UUID part.
# litdata/Streaming/writer.py file

def get_chunk_filename(self) -> str:
        unique_id = uuid.uuid4().hex
        if self._compression:
            return f"chunk-{self.rank}-{self._chunk_index}.{self._compression}-{unique_id}.bin"
        return f"chunk-{self.rank}-{self._chunk_index}-{unique_id}.bin"

# litdata/processing/data_processor.py file

def _upload_fn(self) -> str:
       # ...
      output_filepath = remove_uuid_from_bin_filename(output_filepath)

def remove_uuid_from_bin_filename(filepath: str) -> str:
    """Remove the unique id from the filepath. Expects the filepath to be in the format
    `chunk-<rank>-<counter>-<uuid>.bin` or `chunk-<rank>-<counter>-<compression>-<uuid>.bin`.

    e.g.: `chunk-0-0-9fe2c4e93f654fdbb24c02b15259716c.bin`
        -> `chunk-0-0.bin`

    """
    pattern = r".*-([0-9a-fA-F]{32})\.bin$"  # regex to check if it's in the expected format
    if not bool(re.match(pattern, filepath)):
        return filepath

    # uuid is of 32 characters, '.bin' is 4 characters and '-' is 1 character
    return filepath[:-37] + ".bin"

What do you think about this approach? I expect this to fix the race-condition issue (though I've not been able to reproduce the issue).
And, a failing test is due to some security warning in pytorch, unrelated to the change.
cc: @tchaton

let's merge this for now. If in future we get any race condition error, we'll get back to this.
What do you say @tchaton

This is fixed ;)


Add a header file path when used via Cocoapods

@blakewatters Travis is unhappy. Do you have to have a specific version of Ruby requested in the Gemfile ?


PyPi package and code fixes

This PR contains:

Project directory restructurisation and package setup.py fix for proper package installing via pip
Updated Monitoring class example
Object creation fix, now library methods returns object instead of classes

PyPi package and code fixes already done. And reauest have to much conflicts


Fix for error with funMangledMediaType and RequestBody ignoring mtype from headers

Changed RequestBody.process() to use Body.mangled_mtype() instead of self.funMangledMediaType. This brings it in line with ReposnseBody.
Changed RequestBody init call to parseContentType to use dict.get() instead of getattr to avoid passing None regardless of headers. The functionality remains the same, it will default to None if no content-type header is found.

Thanks for the patch!


docker system prune does not support "--filter until=..."

According to edge/engine/reference/commandline/system_prune.md, it should be possible to provide an 'until' filter to docker system prune. However, with Docker daemon 18.09.0 and fresh docker:latest I get:
$ docker system prune --force --filter "until=168h"
Error response from daemon: failed to prune build cache: Invalid filter 'until'
$ docker system prune --force --filter 168h
invalid argument "168h" for "--filter" flag: bad format of filter (expected name=value)

Thanks for sharing this issue. This particular content is maintained in an upstream repo. Since this issue is dated, I'm not sure if it is still applicable. If the issue is still applicable, please help create an issue or PR in the source repo at https://github.com/docker/cli/tree/master/docs/reference/commandline
Closing this as unable to fix in this repo.


Files open but are blank

This was working yesterday, so I'm not sure what is going on...but today after establishing my ssh tunnel with: ssh -R 52698:<IP_ADDRESS>:52698 root@ipaddress and the password, i am able to navigate and call the rmate index.php command, but the file that opens is BLANK. Happens with every file i try.
On the remote machine, when I pull up the files in a text editor, they have all of their content.
I tried resetting both machines on either side of the connection and uninstalling/reinstalling vscode.
This was such a great feature to have, please help me out!

ran this on my remote machine again:
sudo wget -O /usr/local/bin/rcode
https://raw.github.com/aurora/rmate/master/rmate
chmod a+x /usr/local/bin/rcode
now it's working.
How strange!


how to  solve

Issue Type: Feature Request
EROFS: read-only file system, open '/private/var/folders/jy/bqnppxld49sfd6tmnbjmwjvm0000gn/T/AppTranslocation/F4FFF088-86ED-4269-872C-0ABAB4FE1D01/d/Visual Studio Code.app/Contents/Resources/app/out/main.js'
VS Code version: Code 1.45.1 (5763d909d5f12fe19f215cbfdd29a91c0fa9208a, 2020-05-14T08:33:47.663Z)
OS version: Darwin x64 18.7.0

/needsMoreInfo

I want to install a plugin, but it's could not be installed, the editor prompts for this error.

I want to install a plugin, but it's could not be installed, the editor prompts for this error.

Could you be a little more specific? Which plugin are you trying to install that is giving you this problem.

@wang2498 pls provide more detailed steps as recommended by @mxrachel


Hyprland Crashes on Startup

Regression?
No
System Info and Version
 System/Version info
OS: EndeavourOS Linux x86_64
Host: 4284AN4 ThinkPad W520
Kernel: 6.10.1-arch1-1
Shell: bash 5.2.26
DE: GNOME 46.3.1
WM: Mutter
CPU: Intel i7-2760QM (8) @ 3.500GHz
GPU: NVIDIA Quadro 2000M
GPU: Intel 2nd Generation Core Processor
Memory: 8GB

Description
Installed EndeavourOS, and the end-4 dots for hyprland. Both installs were seamless, showed no issue. On attempting to launch hyprland, command failed to work immediately, outputting two "critical" errors, appears it cannot find my laptop GPU. And I was unable to find system info from hyprctl due to hyprland not launching, so I used neofetch.
How to reproduce
Attempt to launch Hyprland through console or login screen
Crash reports, logs, images, videos
hyprlandCrashReport2801.txt

hyprlandCrashReport2801.txt

--------------------------------------------
   Hyprland Crash Report
--------------------------------------------
Oops

Hyprland received signal 6(ABRT)
Version: fcff2dcac24ca497a39c1cb271d449ade037b7ad
Tag: v0.41.2-119-gfcff2dca
Date: Sun Jul 28 11:42:05 2024
Flags:

System info:
	System name: Linux
	Node name: astrallinux
	Release: 6.10.1-arch1-1
	Version: #1 SMP PREEMPT_DYNAMIC Wed, 24 Jul 2024 22:25:43 +0000

GPU:
	00:02.0 VGA compatible controller [0300]: Intel Corporation 2nd Generation Core Processor Family Integrated Graphics Controller [8086:0126] (rev 09) (prog-if 00 [VGA controller])
01:00.0 VGA compatible controller [0300]: NVIDIA Corporation GF106GLM [Quadro 2000M] [10de:0dda] (rev a1) (prog-if 00 [VGA controller])

os-release:
	NAME="EndeavourOS"
	PRETTY_NAME="EndeavourOS"
	ID="endeavouros"
	ID_LIKE="arch"
	BUILD_ID=rolling
	ANSI_COLOR="38;2;23;147;209"
	HOME_URL="https://endeavouros.com"
	DOCUMENTATION_URL="https://discovery.endeavouros.com"
	SUPPORT_URL="https://forum.endeavouros.com"
	BUG_REPORT_URL="https://forum.endeavouros.com/c/general-system/endeavouros-installation"
	PRIVACY_POLICY_URL="https://endeavouros.com/privacy-policy-2"
	LOGO="endeavouros"
Backtrace:
	# | Hyprland(_Z12getBacktracev+0x46) [0x5b3d280ffa96]
		getBacktrace()
		??:?
	#1 | Hyprland(_ZN13CrashReporter18createAndSaveCrashEi+0xabf) [0x5b3d2808968f]
		CrashReporter::createAndSaveCrash(int)
		??:?
	#2 | Hyprland(_Z25handleUnrecoverableSignali+0x5c) [0x5b3d2801e7ec]
		handleUnrecoverableSignal(int)
		??:?
	#3 | /usr/lib/libc.so.6(+0x3d1d0) [0x7884e395d1d0]
		??
		??:0
	#4 | /usr/lib/libc.so.6(+0x963f4) [0x7884e39b63f4]
		??
		??:0
	#5 | /usr/lib/libc.so.6(gsignal+0x20) [0x7884e395d120]
		??
		??:0
	#6 | /usr/lib/libc.so.6(abort+0xdf) [0x7884e39444c3]
		??
		??:0
	#7 | /usr/lib/libstdc++.so.6(+0x97b2c) [0x7884e3c97b2c]
		??
		??:0
	#8 | /usr/lib/libstdc++.so.6(+0xadffa) [0x7884e3cadffa]
		??
		??:0
	#9 | /usr/lib/libstdc++.so.6(_ZSt10unexpectedv+0) [0x7884e3c9752a]
		??
		??:0
	#1 | /usr/lib/libstdc++.so.6(+0xae2b6) [0x7884e3cae2b6]
		??
		??:0
	#11 | Hyprland(+0xb87df) [0x5b3d27f547df]
		??
		??:0
	#12 | Hyprland(_ZN11CCompositor10initServerENSt7__cxx1112basic_stringIcSt11char_traitsIcESaIcEEEi+0x7bd) [0x5b3d2801f8dd]
		CCompositor::initServer(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, int)
		??:?
	#13 | Hyprland(main+0xbca) [0x5b3d27fba01a]
		main
		??:?
	#14 | /usr/lib/libc.so.6(+0x25e08) [0x7884e3945e08]
		??
		??:0
	#15 | /usr/lib/libc.so.6(__libc_start_main+0x8c) [0x7884e3945ecc]
		??
		??:0
	#16 | Hyprland(_start+0x25) [0x5b3d27fe6ad5]
		_start
		??:?

Log tail:
[LOG] Creating the EventLoopManager!
[LOG] Creating the HookSystem!
[LOG] Creating the KeybindManager!
[LOG] Creating the AnimationManager!
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 42.08µs. Estimated average calc time: 0.10µs.
[LOG] Creating the ConfigManager!
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 23.06µs. Estimated average calc time: 0.04µs.
[LOG] Using config: /home/astral/.config/hypr/hyprland.conf
[INFO] !!!!HEY YOU, YES YOU!!!!: further logs to stdout / logfile are disabled by default. BEFORE SENDING THIS LOG, ENABLE THEM. Use debug:disable_logs = false to do so: https://wiki.hyprland.org/Configuring/Variables/#debug
[LOG] Creating the CHyprError!
[LOG] Creating the LayoutManager!
[LOG] Creating the TokenManager!
[LOG] [hookSystem] New hook event registered: preConfigReload
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 22.69µs. Estimated average calc time: 0.03µs.
[LOG] Using config: /home/astral/.config/hypr/hyprland.conf
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 23.13µs. Estimated average calc time: 0.05µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 22.43µs. Estimated average calc time: 0.03µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 22.64µs. Estimated average calc time: 0.04µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 22.65µs. Estimated average calc time: 0.04µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 45.14µs. Estimated average calc time: 0.04µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.83µs. Estimated average calc time: 0.08µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.77µs. Estimated average calc time: 0.04µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.72µs. Estimated average calc time: 0.05µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.73µs. Estimated average calc time: 0.09µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.77µs. Estimated average calc time: 0.09µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 45.10µs. Estimated average calc time: 0.07µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.81µs. Estimated average calc time: 0.07µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.91µs. Estimated average calc time: 0.07µs.
[LOG] Created a bezier curve, baked 255 points, mem usage: 4.08kB, time to bake: 44.69µs. Estimated average calc time: 0.09µs.
[LOG] Disabling stdout logs! Check the log for further logs.
[LOG] Creating the PointerManager!
[LOG] Creating the EventManager!
[LOG] [AQ] Creating an Aquamarine backend!
[LOG] [AQ] [libseat] [libseat/backend/seatd.c:64] Could not connect to socket /run/seatd.sock: No such file or directory
[LOG] [AQ] [libseat] [libseat/libseat.c:76] Backend 'seatd' failed to open seat, skipping
[ERR] [AQ] [libseat] [libseat/backend/logind.c:349] Could not take control of session: Device or resource busy
[LOG] [AQ] [libseat] [libseat/libseat.c:76] Backend 'logind' failed to open seat, skipping
[ERR] [AQ] [libseat] [libseat/libseat.c:79] No backend was able to open a seat
[ERR] [AQ] libseat: failed to open a seat
[ERR] [AQ] Failed to open a session
[ERR] [AQ] DRM Backend failed
[LOG] [AQ] Starting the Aquamarine backend!
[LOG] [AQ] Starting the Wayland backend!
[LOG] [AQ] Got registry at 0x5b3d51485100
[LOG] [AQ] Output WAYLAND-1: initialized
[CRITICAL] m_pAqBackend couldn't start! This usually means aquamarine could not find a GPU or enountered some issues. Make sure you're running either on a tty or on a Wayland session, NOT an X11 one.
[CRITICAL] Critical error thrown: CBackend::create() failed!

and the end-4 dots for hyprland.

Does Hyprland start without the dots?

and the end-4 dots for hyprland.

Does Hyprland start without the dots?

It does not. It does not start once it hits that error.

GPU: NVIDIA Quadro 2000M

Did you do this? https://wiki.hyprland.org/Nvidia/

UPD: What drivers are you using? The GPU is quite old, legacy proprietary drivers don't support wayland well or don't support, at all. In case of nouveau driver - there were some issues here.

UPD: What drivers are you using? The GPU is quite old, legacy proprietary drivers don't support wayland well or don't support, at all. In case of nouveau driver - there were some issues here, claiming hyprland doesn't work on them.

I hadn't done the NVidia thing, so I ran that quickly and no change. Checked my drivers and it shows kernel driver as nouveau for the NVIDIA Quadro 2000M.

can you retry this with HYPRLAND_TRACE=1 AQ_TRACE=1 Hyprland and give the hyprland.log
also the log seems like you're running it on top of another wayland compositor so try a different one or from tty as a sanity check

Probably duplicate of #7311

I also begin to have the same problem but I'm not using nix. I'm using AUR package.


Add WIP Nix command line support

dockter compile --nix Adds extra .default.nix and .nixDockerfile
dockter build $PWD --nix Creates project docker image, and nix-store docker volume (if needed)
dockter execute $PWD --nix Runs nix-shell inside a docker container from the project docker image with reusable /nix mounted from the nix-store docker volume

You can also try the demo-nix.sh with demo-magic.sh.

Thanks @bmpvieira, this looks great.
Out of interest. I did some comparison of the image sizes resulting from .Dockerfile and .nixDockerfile. I also created a Nix based Dockerfile which pre-built the Nix store by running nix-shell:
FROM nixos/nix
COPY .default.nix default.nix
RUN  nix-shell --pure

A summary of what I ran:
cd tests/fixtures/multi-lang-for-nix
dockter compile $PWD
docker build . --file=".Dockerfile" --tag=multi-lang
docker build . --file=".nixBuildDockerfile" --tag=multi-lang-nix-prebuilt
docker build . --file=".nixDockerfile" --tag=multi-lang-nix
docker images | head -n 4

REPOSITORY                                          TAG                 IMAGE ID            CREATED             SIZE
multi-lang                                          latest              ae78e517b3d0        18 minutes ago      1.01GB
multi-lang-nix                                      latest              53b6c33b44b6        26 minutes ago      154MB
multi-lang-nix-prebuilt                             latest              9f3a8b8f3122        27 minutes ago      1.98GB

Obviously the multi-lang-nix is tiny because it doesn't yet have packages installed. I was surprised how large the multi-lang-nix-prebuilt image is. It would be interesting to see how larger images from nixpkgs.dockerTools run on this .default.nix are.

Hey @nokome, as I explained in our call, multi-lang-nix-prebuilt is most likely larger because it includes all the dependencies and source code unlike multi-lang which for most cases probably only installs the binaries for Ubuntu. I've tried to use the Nix function dockerTools.buildImage() which should include fewer dependencies (mostly runtime I guess) but in my tests it was only 20 % smaller than multi-lang-nix-prebuilt and required a lot of workarounds since the Nix environment variables where not set, unlike when building a Docker image from nixos/nix image and using nix-shell.
Below are the two Nix files I used:
# Used in nixos/nix Docker image
with import <nixpkgs> {};

stdenv.mkDerivation rec {
  name = "multi-lang-for-nix";
  contents = [
    bashInteractive coreutils utillinux findutils gnugrep which
    openssl cacert glibcLocales
    nodejs
    nodePackages.dat
    python
    pythonPackages.numpy pythonPackages.pillow
    R
    rPackages.babynames
  ];
}

# Used in dockerTools.buildImage() Docker image
with import <nixpkgs> {};

pkgs.dockerTools.buildImage rec {
  name = "multi-lang-for-nix";
  # tag = "latest";
  # created = "now";
  contents = [
    bashInteractive coreutils utillinux findutils gnugrep which
    openssl cacert glibcLocales
    nodejs nodePackages.dat
    (python36.buildEnv.override {
      ignoreCollisions = true;
      extraLibs = with python36Packages; [
        numpy
      ];
    })
    (rWrapper.override {
      packages = with rPackages; [
        babynames
      ];
    })
  ];
  config = {
    Env = [ "NODE_PATH=/lib/node_modules" ];
  };
  runAsRoot = ''
    #!${stdenv.shell}
    mkdir -p /usr/bin
    # Fix env not in common location
    ln -s /bin/env /usr/bin/env
    # Fix Node CLI tools
    # /lib/node_modules/.bin/* gets copied to /bin/ however, links get broken
    # e.g., /bin/dat -> ../dat/bin/cli.js
    if [ -d /lib/node_modules/.bin ]; then
      for f in /lib/node_modules/.bin/*; do ln -sfn $(readlink -f $f) /bin/$(basename $f); done
    fi
  '';
}

However, the strength of Nix is that we reuse the /nix/store in a shared Docker data volume, so we save a lot of space in redundancy and the actual images end up tiny since they only contain the default.nix file. This way, it's also faster to spin them up (if packages already exist in the Nix store). We can also periodically garbage collect the /nix/store to remove unused packages.


Other fields in data generation

Hi Edward,
Thanks for the code, it really helps in understanding the paper better.
Currently your python code generates patient id and  ICD9 diagnosis codes.
I wanted to know what changes or modifications do I have to do in your Process_mimic and medGAN code if I need to  generate  synthetic data to incorporate fields such as Age,gender,Procedures etc.?
Do I have to include the desired fields in process_mimic file only or do I need to make changes to medGAN.py also?
Secondly, the data that would be generated (i.e. pid, icd9, age,gender, procedures) will take into consideration impact of  other variables or will it solely be dependent on the Patient id?
Thanks

Hi Ankit267,
In order to generate features (age, gender, procedure codes) other than Dx codes, you need training data that includes them in the first place.
So, yes, you need to modify process_mimic so that you can extract those features from MIMIC-III.
(Actually, you don't even need to bother with process_mimic. You can just write your own script to extract desired features from MIMIC-III)
As for medGAN.py:
If you are generating a mix of features such as age (non-negative integer) and presence of Dx codes (binary), you need to modify medGAN accordingly. For binary features, sigmoid activation function is used, but for non-negative integers, ReLU should be used.
So, for each output neuron of the generator, different activation functions should be used, and I'm not sure if that's doable in TensorFlow. (Maybe in PyTorch?)
There are other types of features, such as ethnicity, which is neither non-negative integer or binary.
Ethnicity is one-hot among multiple classes (e.g. you can't be both "Asian" and "Hispanic". There is "Other" instead.) For this feature type, you should use Softmax instead. But that's another technical issue I'm not sure if TensorFlow can handle.
So in summary, with my implementation of medGAN, you should stick to generating only one feature type.
If you want to generate non-negative integers and binary values at the same time, I think you can try with only using ReLU, but I'm not sure if that will work out great.
For the second question: medGAN will generate synthetic samples that closely follow the distribution of the real data samples. So the dependency among all features will come into play. Since Patient ID is not being generated (unless you include them in the training data) no features will depend on Patient ID.
Hope this helps.
Best,
Ed

Thanks Ed, that really helps.
Will get back to you in case of any concerns.

Dear Ed
As you mentioned above, Patient ID is not being generated (unless we include them in the training data). So, my question is  medGAN generate synthetic samples with same and in order Patient ID when we run it different times? Thank you

Hi Myshgithub,
The synthetic records generated by MedGAN will have no relationship whatsoever with the original Patient IDs. The generated records will be purely synthetic, and every time you generate a new batch, you get a fresh batch of synthetic records. Sometimes you might get duplicate records by chance, but that doesn't mean they are the same patient. MedGAN does not understand the concept of Patient ID, unless you modify it somehow.
Best,
Ed

Thank you so much Ed!
But, in the first comment here above: Ankit267 mentioned that
(Currently your python code generates patient id and ICD9 diagnosis codes.) and then you replied that: "Since Patient ID is not being generated (unless you include them in the training data) no features will depend on Patient ID."
So, my question is please what changes or modifications do I have to do you think in order to have ( Patient ID along with synthetic generated records)?
1- One way as discussed seems to be include the desired fields( Patient ID) in process_mimic file only. Is it correct?
2- Do you aware of any other way that we can  include them(Patient ID) in to the training data?
3-In (https://github.com/mp2893/medgan/issues/3) for interpreting the generated samples, is mentioned that: (“X is of shape (10000, 1071), and Each row corresponds to a single synthetic patient”). So, that is just Row-Number_ID? And they are ordered?
I would appreciate your responses...


Menu Bar Icon

Checklist

[x ] I have searched for existing issues

I would like to be able to remove the menu bar icon but command clicking does not allow me to drag it off the bar. I only use Monitor Control to allow F1 and F2 keys to control the brightness of an LG monitor connected to my Mac Mini. I rarely change contrast and try to keep as few Icons in the menu bar as possible.

Duplicate of #189


Latest Among Us update broke current method for scanning the game's memory

The capture program can't find the room code anymore. I assume it's because the game recently updated with all the beta branch features, which was known to create some issues with reading the game's memory.
I assume this also breaks the monitoring of character states? I can't tell since I can't use the bot fully right now. I assume the memory has been altered to make it harder for hacked clients, so maybe this repo has a forever game of cat/mouse ahead of us? :S

Huh? 2.4.2 is working perfectly on my end, and with our discord masses...
https://github.com/denverquane/amonguscapture/releases/tag/2.4.2

Wow, this one is on me. I totally spaced out and didn't even bother to check that my exe's were up-to-date. I'm sorry for the wasted cycles on this!


Create documentation

I would like to work on documentation . Kindly assign me this issue.

I would like to work on it

@JenilGajjar20
My account is facing some issues can you please assign it to @Gopal2021025

What kind of issues you are facing? @Snehil0603

Go for it @Gopal2021025


Function completion should only insert parameter labels

Given func foo(_ x: Int, y: Int) {}, if I type foo(...
Actual behavior
I am offered foo(x: Int, y: Int) in the completion box, and accepting it completes to foo(x: Int, y: Int).

There are a few problems with this:

The signature shown seems incorrect because the first parameter has no label. It should be foo(Int, y: Int)
Completion includes the first parameter label, which is incorrect because the function cannot be called that way.
Completion include type names, which is not useful and need to be manually deleted.

Expected behavior
I am offered foo(Int, y: Int) and accepting it completes to foo(|, y: ), where | represents the cursor position. Note that there is no x: label and no types have been inserted so I can immediately fill in with the values I need instead of needing to first erase the types.
Stretch feature: complete last parameters of closure types as trailing blocks.

Tracked in Apple’s issue tracker as rdar://112732434

Actually, this behaves as it is intended: The completion that is offered actually contains two snippets (in Xcode terminology placeholders) and the completion that is offered is the following (I’m using <> to indicate the snippets).
foo(<x: Int>, y: <Int>)

You can then use tab to jump from one snippet to the next.
https://github.com/apple/sourcekit-lsp/assets/4062178/781a2720-7a51-4781-854c-591676498830
Also, we provide the second parameter name (x) to indicate what the parameter without an external argument label is.
If you used an editor that doesn’t support LSP snippets (not sure if you could also turn it off in VS Code), the completion that you will get offered is what you suggested:
foo(, y: )

With that in mind, do you still think that this is a bug or would you say it behaves correctly?

Thanks for clarifying! I agree this is the intended behavior. I thought there were no snippets because they were highlighted so dimly that I didn't notice them.

While I'm here, does XCode use sourcekit-lsp, or sourcekitd directly?

Xcode uses sourcekitd directly.


Question / Help: Is there a way to dynamically add a route to a map?

I'm a bit confused on how to use addToSiteMap() function. Do you know of an easy way to create the object it's looking for?

Not ideal, but you'll have to access an HttpRequest's ._messageInfo attribute, as that is what addToSiteMap expects.


Rename endian.h to spirv_endian.h

If we later add a source/ as an -I include directory,
then avoid confusing other headers that want to include the
standard "endian.h" from /usr/include.

LGTM

+2

It seems endian.cpp is not reamed as spirv_endian.cpp?

New version also moves source/endian.cpp to source/spirv_endian.cpp.

LGTM


Need to add System Tray

Look at  crates mentioned in these Slint Issues:
https://github.com/slint-ui/slint/discussions/933
https://github.com/slint-ui/slint/discussions/1368

I never mentioned it or commented, but initial tray support was added 7 months ago in commit 41d8e81. I'm leaving this open as there are still some issues here that have not been resolved.


nixosModules.MooseFS: Format

Only format of MooseFS

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-reviewd workflow started

Extra args: -p nixosTests.moosefs
Free space: true
Who asked: lucasew

TARGETS

x86_64-linux: true
x86_64-darwin: false
aarch64-darwin: false

Status and logs: https://github.com/nixosbrasil/nixpkgs-reviewd/actions/runs/12454794428

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 367327

x86_64-linux

:white_check_mark: 1 test built:

nixosTests.moosefs

Hi,
Possible to backport this PR to 24.11 ?
Best Regards

Hi,
Possible to backport this PR to 24.11 ?
Best Regards

Just add the backport tag to the PR and the bot will create the PR automatically.


Move launder into 9.5 ifdef block

I didn't see this sorry.  You still want to move this into ifdef block.  Reason why I didn't was I was thinking for lower versions of PostgreSQL, we could in theory just have an sql function that similuates the same behavior as the Import Foreign schema does.  I assume it's legal in an fdw extension to package helper functions.  In that case, this function would be reused by that function.

For now it just causes warnings in 9.4 of unused function, #ifdef'ing it shuts it up. Since you didn't merge this into your branch before I merged into my master, this PR is orphaned, kill it.


feat(util): Don't use CGO for fqdn_windows

What does this PR do?
Removes CGO dependency on pkg/util/hostname for future usage in statically-linked datadog-installer
Motivation
Describe how to test/QA your changes
No functional change expected -- make sure the agent does report properly its hostname on Windows
Possible Drawbacks / Trade-offs
Additional Notes

/merge


Update wholesale package to 5.0.9

Description

References

#782

Codecov Report

Merging #1448 (5cc3492) into main (151afaa) will increase coverage by 18.92%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main    #1448       +/-   ##
===========================================
+ Coverage   50.10%   69.02%   +18.92%
===========================================
  Files          35       49       +14
  Lines         497      862      +365
  Branches       32        0       -32
===========================================
+ Hits          249      595      +346
- Misses        240      252       +12
- Partials        8       15        +7

Impacted Files
Coverage Δ

...ntegration/Controllers/WholesaleControllerTests.cs
100.00% <ø> (ø)

...Hub.WebApi/Controllers/WholesaleBatchController.cs
87.23% <ø> (ø)

...taHub.WebApi/Controllers/Wholesale/Dto/BatchDto.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

...trollers/MarketParticipant/Dto/FilteredActorDto.cs
16.66% <0.00%> (-83.34%)
:arrow_down:

...gistration/HealthEndpointRegistrationExtensions.cs
45.45% <0.00%> (-36.37%)
:arrow_down:

...WebApi/RequireNonNullablePropertiesSchemaFilter.cs
80.00% <0.00%> (-8.00%)
:arrow_down:

apps/dh/api-dh/source/DataHub.WebApi/Startup.cs
94.25% <0.00%> (-1.15%)
:arrow_down:

...ce/DataHub.WebApi.Tests/Fixtures/WebApiTestBase.cs
100.00% <0.00%> (ø)

...on/Controllers/MarketParticipantControllerTests.cs
100.00% <0.00%> (ø)

...ration/Controllers/MeteringPointControllerTests.cs
100.00% <0.00%> (ø)

... and 11 more

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


Identical Commit Check - Fixes

Identical Commit Check - Fixes
WIP

fixes: https://github.com/github/branch-deploy/issues/184

Could not fix :(


proposal network best practices

Describe the bug
This is more of a "best practices" question than an actual bug, but when using a proposal with nerfacto for example I often see two sorts of artifacts: (a) "waves" for lack of a better term:

and (b) grey/white floaters from time to time:

Do you have an intuition as to what might be causing this and are there any suggested best practices for alleviating these issues?
To Reproduce
Steps to reproduce the behavior:
Run nerfacto on an unbounded dataset. I can share my data if helpful (this is the boat dataset from ADOP)
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
Full screen shot below:

Additional context
Add any other context about the problem here.

could you try increase the log2_hashmap_size in nerfacto field? Can you share the sample datasets as pictured in the post?

I also got these artifacts.

Hi @hturki
Did you run the nerfacto model with default settings? Could you pls share your dataset?

Hey all
This should happen with the default nerfstudio parameters + a log2 hash map size of 20 + 4 features per level. The dataset in question is the Boat one in https://github.com/darglein/ADOP - I have it working with some custom data loading code that's probably messy to share but will try to make a more digestible repro at the end of the month

I got similar patterns. Any update on this?


Fix #63 - Deprecate 'MlflowDataSet' and rename it 'MlflowArtifactDataSet'

Apart from doc updates, I've added a DeprecationWarning and kept the old MlflowDataSet class.
We will removed it entirely in 0.4.0.

I just updated the changelog entry to "Deprecated" and add the issue number. I merge it.


update admin-ui to 1.1.2

Codecov Report

Merging #5118 into master will increase coverage by 0.15%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #5118      +/-   ##
============================================
+ Coverage     82.39%   82.54%   +0.15%
- Complexity    11001    11015      +14
============================================
  Files          1331     1331
  Lines         45047    45047
  Branches       5660     5660
============================================
+ Hits          37115    37184      +69
+ Misses         5280     5221      -59
+ Partials       2652     2642      -10

Impacted Files
Coverage Δ
Complexity Δ

...te/blob/pending_transfer/PutHeadChunkRunnable.java
76.71% <0%> (-4.11%)
12% <0%> (-2%)

...in/java/io/crate/breaker/RamAccountingContext.java
80% <0%> (-3.34%)
19% <0%> (-2%)

...ration/collect/collectors/LuceneBatchIterator.java
86.81% <0%> (-3.3%)
35% <0%> (-1%)

...ql/src/main/java/io/crate/metadata/TableIdent.java
89.28% <0%> (-1.79%)
28% <0%> (-1%)

...main/java/io/crate/metadata/doc/DocSchemaInfo.java
83.78% <0%> (-0.91%)
29% <0%> (-1%)

sql/src/main/java/io/crate/planner/Planner.java
88.04% <0%> (-0.8%)
36% <0%> (-1%)

.../elasticsearch/action/bulk/BulkShardProcessor.java
76.29% <0%> (+0.37%)
48% <0%> (+1%)
:white_check_mark:

...dices/create/TransportBulkCreateIndicesAction.java
66.96% <0%> (+0.45%)
33% <0%> (+1%)
:white_check_mark:

...er/src/main/java/io/crate/sql/tree/AstVisitor.java
32.2% <0%> (+0.84%)
38% <0%> (+1%)
:white_check_mark:

.../operation/collect/sources/ShardCollectSource.java
89.56% <0%> (+1.09%)
31% <0%> (+1%)
:white_check_mark:

... and 22 more

Continue to review full report at Codecov.

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


Empty file can not be parsed

I use csv.mmap(filename) to read csv file.
When the input file is empty (size 0), I expect a zero length output.
  if (csv.mmap(fileName)) {
    for (const auto row : csv)  // should not loop

but I get the following error
libc++abi.dylib: terminating with uncaught exception of type std::__1::system_error: Invalid argument fish: Job 1, '../bin/mp20_client.exe -in 0....' terminated by signal SIGABRT (Abort)

I am experiencing the same error.


Improve error message with title, description, and examples

JSON Schema allows for describing the rule with a title, description, and examples. However, currently only the automated error message is displayed.
This PR added the title to the CLI tool. This is identical to how the vs-code displays the error.
The PR also adds the hover content to the CI output. The hover content is displayed when a user hovers a property or content in an editor. This hover details includes the title, description, and examples in a formatted structure. Note, that the yaml service outputs Markdown. However, github-actions assumes ANSI outputs. Hence, a converter is used.
Colors are only displayed when the user adds:
env:
   FORCE_COLOR: true

to their action.yaml file. This logic is controlled by https://github.com/chalk/supports-color. Regardless, they only show up in the console, not the review comments.
Example showing the new output format:

Previousely only String does not match the pattern of "^https:\/\/youtu\.be\/[a-zA-Z0-9\-_]{11}$". was shown.

Very cool stuff, thanks again!
I'll merge and rebuild to ensure it's up to date.


[upstream-with-swift] Add workaround for MSVC bug

See https://github.com/apple/swift/pull/3759#event-734591467
The problem is that MSVC doesn't recognise the following syntax, reporting multiple errors here:
  size_t numTrailingObjects(
      typename TrailingObjects::template OverloadToken<SourceLoc>) const {
    return asDerived().hasArgumentLabelLocs()
             ? asDerived().getNumArguments()
             : 0;
  }

For some reason, MSVC ignores the fact that we gave friend access to llvm::TrailingObjects here.
Note that the Swift source code also has to be modified, as this is not enough. This is the new source code, that fully compiles after this change!
  // Work around MSVC bug: can't infer llvm::trailing_objects_internal, even though we granted friend access to it
	size_t numTrailingObjects(
		llvm::trailing_objects_internal::TrailingObjectsBase::OverloadToken<Identifier>) const {
		typename TrailingObjects::template OverloadToken<Identifier>) const {
    return asDerived().getNumArguments();
  }

Again, can we get this into real upstream LLVM instead?

Oh, so that's what you meant by upstream :(

For upstream, do you think we should remove the #ifdef and just change from private to protected

Ah, sorry for not being clear!
I'm not sure how I feel about that. protected everywhere isn't really harmful, but it does still inject extra members into the scope. Probably someone in the LLVM community will have a stronger opinion.

Ah, sorry for not being clear!

No problem, there are two upstreams :D
I've opened https://reviews.llvm.org/D29181 for discussion on this - first real submission to LLVM so I hope I've correctly done it :)

Looks like no one has looked at https://reviews.llvm.org/D29181, have I correctly published it?

That's correct enough. I'll poke the owner of TrailingObjects to hopefully get some movement.

Thanks Jordan, appreciate it!

Closed this - I'll also handle updating swift, when I get the time


Deprecation warning from Makie.jl for resolution keyword

When I try to generate a mesh using the great wavy walls example by @andrewwinters5000, which I adapted in https://github.com/trixi-framework/talk-2024-dlr-braunschweig-trends-dgsem/blob/f57927e5acce89c8c26ac0dd7444725e00e39e92/examples/generate_mesh_cylinder_with_sine_walls.jl, I get the following deprecation warning from Makie.jl:
┌ Warning: Found `resolution` in the theme when creating a `Scene`. The `resolution` keyword for `Scene`s and `Figure`s has been deprecated. Use `Figure(; size = ...` or `Scene(; size = ...)` instead, which better reflects that this is a unitless size and not a pixel resolution. The key could also come from `set_theme!` calls or related theming functions.
└ @ Makie ~/.julia/packages/Makie/VRavR/src/scenes.jl:220

Any idea what's wrong here and how to fix it?

I am not sure why it would still throw this warning because I removed all these deprecated keywords in c7f497fba157d35ab1d143d1a9df45306fdeae1d. Are you using the latest version of HOHQMesh.jl?

No, and I apologize for the overhead I caused 🤦. Thanks for the feedback!


Ceramic publish history

This PR mainly adds tracking of ceramic commitID's in NodeVersion, but due to some recent changes in the backend also patches a bunch of regressions.
Notable changes:

Bumped docker base image to same node version as nvmrc (+cascading fixes)
Add public IPFS node to compose cluster (can be used for testing publish)
We no longer depend on GRPC which speeds up builds quite a bit! :racehorse:
Upgrades to several IPFS related packages
TOGGLE_CERAMIC env no longer necessary due to (hopefully) idempotent stream tracking

(this may not build until this is merged and final codex package versions get published: https://github.com/desci-labs/desci-codex/pull/42)


Updating Packages

This pull request primarily updates package versions across multiple files. These updates include changes to the versions of actions/setup-dotnet, gittools/actions/gitversion/setup, gittools/actions/gitversion/execute, Azure.Identity, Selenium.WebDriver, Selenium.WebDriver.ChromeDriver, Microsoft.VisualStudio.Web.CodeGeneration.Design, and Microsoft.AspNetCore.TestHost.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.2%) to 77.797%

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
2626

Relevant Lines:
3321

💛  - Coveralls


add ticdat function for incorporating parameters_schema as a predicate rule

See what TicDatDeployer.__init__ does when passed a parameters_tdf. It tries to create a predicate for the parameters table in the input schema. Particularly if iris.py becomes one of our examples, then this functionality makes sense for handling offline parameters.

Addressed here


Segfault accessing null prim/attribute

If the prim or attribute below do not exist, this should throw an exception, it should not segfault:
stage.GetPrimAtPath(primPath).GetAttribute(attrName).Get(0);
The current crash will take down the Unity Editor.

Thread started with Swig user group to see if there is a way that Swig can validate UsdObjects on construction.

A new accessor has been added to USD.NET.Scene, which allows for safe prim access without crashes.


Update wechatwebdevtools to 0.17.171900

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as 69e443e86f62b7de514ce0e11a3227a036337381. Your contribution is still credited to you.


MNIST Example Error: 403 Forbidden

MNIST isn't MNIST-ing like it used to..
Encountering an issue while trying to access/download the MNIST dataset using the provided Python code. To reproduce the error, simply navigate to the mlx-examples/mnist directory and run the following command: python main.py
My Silly Workaround:
I was able to modify the mnist.py file to load the datasets from a local directory, which did end up solving this issue. Since this doesn't seem isolated to MLX specifically (I came across several workarounds for PyTorch and TensorFlow, for example), it may not be mendable by the MLX team.
Possible Long-term Solution:
Updating the base_url argument in the mnist.py file to a more reliable 3rd-party host could be worth it. I haven't tried this solution since I already had the dataset saved from another project, but I don't see why it wouldn't work. The only problem I can think of would be an ethical one, where Apple isn't directly sourcing the data from the official publisher.
Regardless, this isn't a major issue, and the impact is negligible. I just figured it was worth taking a look at for people simply exploring the provided examples or for anyone who might be trying to implement MLX into their own projects.

Yes this has been an issue lately. Changed the URL to something more stable in 749

@awni sorry for creating a new issue, I swear I looked through the repo for this but couldn't find anything. Thanks x10

Yes this has been an issue lately. Changed the URL to something more stable in 749

sorry for creating a new issue, I swear I looked through the repo for this but couldn't find anything. Thanks x10

I just made that PR right after you filed the issue ;)

this tomfoolery will not stand!


Bug related to baseurl setting

There is a small mistake in style.scss which results in not loading the banner if a baseurl was set.
Simply change
.header {   background-image: url("{{ site.banner }}");
to
.header {   background-image: url("{{ site.baseurl }}{{ site.banner }}");

Thanks for this!


IMN 573 - Improve consumer error handling

Closes IMN-573
This PR improves the consumer message error handling.
Currently, when an error occurs the consumer exits and doesn't consume any other message until the normal restart after 60 minutes to refresh the authentication.
With these changes, the consumer remains active and processes messages (for other stream IDs).

We need a smarter solution to improve the error handling. See the original issue description


[Feature][Connector-V2][Timeplus] New connector for Timeplus, a streaming-first analytics platform

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
Timeplus Proton, https://github.com/timeplus-io/proton, is an open-source streaming SQL engine, able to consume data from Kafka and write to Kafka or ClickHouse after streaming JOIN or ETL. It also supports REST API or SQL for data ingestion. JavaScript and remote User-Defined function are supported. Written in C++ and leveraging ClickHouse as a library, Timeplus Proton is simple, powerful and cost-efficient.
Timeplus Enterprise is the commercial offering, based on Proton, with extra cluster, user management, web console and other enterprise features.
We'd like to enable the integration for Seatunnel with Timeplus, so that data in MySQL/Postgres and many other systems can be loaded to Timeplus Proton or Timeplus Enterprise to enable real-time analytics with the unified SQL for batch and streaming.
Join https://timeplus.com/slack and visit https://timeplus.com to learn more.
Usage Scenario
Data in MySQL/Postgres and many other systems can be loaded to Timeplus Proton or Timeplus Enterprise to enable real-time analytics with the unified SQL for batch and streaming.
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

A PR turned from WIP to ready-to-review last week https://github.com/apache/seatunnel/pull/7385


Fix unused parameter warnings/errors with gcc 6.

Hi,
I have just hit a couple of "unused parameter" warnings that in my case are errors since I build with -Werror.
Ok to merge ?
Cheers,
Romain

Thanks, merged to branch PR/remove-gcc6-warnings.

In develop


Update Validation Serialized files

Files generated from the tests from #343

Hmm these changed recently from @iamralpht 's PR https://github.com/google/automotive-design-compose/pull/324. Not sure why they would be different again.

Did you find that this wasn't needed?

It was, but I merged it into #365


Is there any way to call vRP functions from C#?

Is there any way to call vRP functions from C#?
I'd like to have in my C# scripts something similar to the Proxy, so that I can then call vRP functions.
If this is not directly possible, is there a workaround?

If you follow the same protocol as for the vRP proxy system, then you probably can.
https://github.com/ImagicTheCat/vRP/blob/master/vrp/lib/Proxy.lua

Have you found a solution?

@chaze117 Unfortunately I ended up editing vRP and adding server events to do what I was trying to do.


Authentication fails at Micropub.rocks, but succeeds at IndieAuth.com

When I try to authenticate with my site at Micropub.rocks, I get an error saying that all 3 endpoints cannot be found. When I authenticate at IndieAuth.com, it works. All three endpoints are setup on my page, I have attached 3 screen shots, and the source for my homepage. I'm brand new to IndieWeb stuff, and for the life of me I cannot figure out why this is not working. I'm not sure if the problem is on my end, or if there is an issue. Thanks!

Updated with correct screenshots.


Added PEP8Bear and YAMLLintBear to .coafile

Fixes #191

By submitting this pull request I confirm I've read and complied with the below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the coding guidelines for this project.
[x] My code follows the skeleton code structure.
[x] This pull request has a descriptive title. For example, Added {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[x] This pull request will be closed if I fail to update it even once in a continuous timespan of 7 days.

Comment on d2dacaaa629415be8948e2179b0b0be1b994469b, file modular_exponential/modular_exponential.py, line 3.
The code does not comply to PEP8.
PEP8Bear, severity NORMAL, section pep8.
The issue can be fixed by applying the following patch:
--- a/modular_exponential/modular_exponential.py
+++ b/modular_exponential/modular_exponential.py
@@ -1,6 +1,7 @@
 """
 Modular Exponential
 """
+

 def mod_exponent(base, power, mod):
     """

Comment on d2dacaaa629415be8948e2179b0b0be1b994469b, file stack/stack.py, line 1.
The code does not comply to PEP8.
PEP8Bear, severity NORMAL, section pep8.
The issue can be fixed by applying the following patch:
--- a/stack/stack.py
+++ b/stack/stack.py
@@ -1,4 +1,5 @@
 class Stack(object):
+
     def __init__(self):
         self._list = []

@singhpratyush coala PEP8Bear has issues with some files in repo should I rectify them in my PR itself?

Yes, please do so.

Comment on d2dacaaa629415be8948e2179b0b0be1b994469b, file modular_exponential/modular_exponential.py, line 3.
The code does not comply to PEP8.
PEP8Bear, severity NORMAL, section pep8.
The issue can be fixed by applying the following patch:
--- a/modular_exponential/modular_exponential.py
+++ b/modular_exponential/modular_exponential.py
@@ -1,6 +1,7 @@
 """
 Modular Exponential
 """
+

 def mod_exponent(base, power, mod):
     """

Comment on d2dacaaa629415be8948e2179b0b0be1b994469b, file stack/stack.py, line 1.
The code does not comply to PEP8.
PEP8Bear, severity NORMAL, section pep8.
The issue can be fixed by applying the following patch:
--- a/stack/stack.py
+++ b/stack/stack.py
@@ -1,4 +1,5 @@
 class Stack(object):
+
     def __init__(self):
         self._list = []

@aviaryan sir even after making the changes the gitmate bot is raising the issue.

@Monal5031 Notice the first line in gitmate comment.
"Comment on d2dacaa, file stack/stack.py, line 1."
It is still analysing the first commit, this is a bug. For an update report, squash your commits.

Comment on abc37724bf5c1e088dd601310e19f7af986026a5, file modular_exponential/modular_exponential.py, line 3.
The code does not comply to PEP8.
PEP8Bear, severity NORMAL, section pep8.
The issue can be fixed by applying the following patch:
--- a/modular_exponential/modular_exponential.py
+++ b/modular_exponential/modular_exponential.py
@@ -1,6 +1,7 @@
 """
 Modular Exponential
 """
+

 def mod_exponent(base, power, mod):
     """

Comment on abc37724bf5c1e088dd601310e19f7af986026a5, file stack/stack.py, line 1.
The code does not comply to PEP8.
PEP8Bear, severity NORMAL, section pep8.
The issue can be fixed by applying the following patch:
--- a/stack/stack.py
+++ b/stack/stack.py
@@ -1,4 +1,5 @@
 class Stack(object):
+
     def __init__(self):
         self._list = []


update API reference doc a bit

I went to fix the problems reported in the "update API reference" task and did some fixing which is included here. I alphabetized unalphabetized stuff, fixed some broken links, updating some weird sentences, added a bit of missing info. Really there are more problems with this page though - there are a lot more missing parts, and this one doc is becoming somewhat of a grab bag where people add more crap to it.

Current coverage is 89.23% (diff: 100%)

Merging #2368 into new-docs will not change coverage

@@           new-docs      #2368   diff @@
==========================================
  Files            39         39
  Lines          1486       1486
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1326       1326
  Misses          160        160
  Partials          0          0

Powered by Codecov. Last update 73424a9...0c6b1ee

Updates to APIReference LGTM. Did you mean to also include that update to the RN tutorial?


How to use it?

Could you tell me how to use it? I don't know where is "Globals" object. Your example doesn't work for me as my Godot don't know what is Globals.
Also, I prefer using C# but it doesn't matter. I wanna this code start working any way.

Use "Engine" instead of "Globals" on 3.2.2.

Use "Engine" instead of "Globals" on 3.2.2.

Could you please tell me how to install it correctly? I did everything whatever in instruction but it doesn't work. Godot says "Failed to retrieve non-existent singleton 'LocalNotification'".
I suppose that the instruction from README is not correct as it says I should drop LocalNotification.gdap to android/plugins but I can see android-plugin directory in this repo. What should I do with that?

I should drop LocalNotification.gdap to android/plugins but I can see android-plugin directory in this repo

This should be a directory in your project root folder, not from this repo. I believe Godot creates it when you go to Project > Install Android Build Template. If not, just create it manually, then place the gdap file inside it.
After that, you should be able to see the plugin inside the Project > Export... > Android options, and the 'custom build' option as well (check both):

Another detail, the singleton will only load when running the game on an Android device (I test on my phone through usb, but an emulator should work too).

I should drop LocalNotification.gdap to android/plugins but I can see android-plugin directory in this repo

This should be a directory in your project root folder, not from this repo. I believe Godot creates it when you go to Project > Install Android Build Template. If not, just create it manually, then place the gdap file inside it.
After that, you should be able to see the plugin inside the Project > Export... > Android options, and the 'custom build' option as well (check both):

Another detail, the singleton will only load when running the game on an Android device (I test on my phone through usb, but an emulator should work too).

I did everything you said but I'm getting this erros right now

And this one when I'm trying to launch my game.

I'm getting this erros right now

I noticed I also get this error, even when running on my phone, but the plugin works anyway.

And this one when I'm trying to launch my game.

Are you launching on Android? The error will happen if you try to Engine.get_singleton("LocalNotification") when running on your computer.

I'm getting this erros right now

I noticed I also get this error, even when running on my phone, but the plugin works anyway.

And this one when I'm trying to launch my game.

Are you launching on Android? The error will happen if you try to Engine.get_singleton("LocalNotification") when running on your computer.

YEEES. THANK YOU! You are really good person. I didn't believe it can work but it works!
By the way, do you know how to change color of notification and an image of notification (right now I've got a grey square).

Anyway, I wanna say that the reason why it didn't work is that I made too small time interval. I mean don't do something like that ln.showLocalNotification("Message", "Title or application name", 0, 1).  Do something like that instead ln.showLocalNotification("Message", "Title or application name", 20, 1).

No problem. :)

By the way, do you know how to change color of notification and an image of notification (right now I've got a grey square).

This could definitely be coded somehow, but it's not available in this plugin yet. When I use Firebase + Cordova, it uses the icon from the app and properly show the transparent png on status bar .

the reason why it didn't work is that I made too small time interval.

Good to know, thanks for sharing!


Settings address

This PR adds in the Address tab to settings.
@morebrownies I found that overlaying every delete button on a delete / add address looked kind of weird. I also found that we want to prevent them from updating the form, since we clear it on success, if they started to update it, they would just be interrupted. So I just overlay the whole tab during saves:

Obviously, we could improve on that styling! Also, I only show the tab overlay once they move their mouse. This prevents it from flickering on / off very quickly on super fast saves (which most are. In fact I think you'll have a hard time even getting the overlay to show - I had to muck with the server to delay the response)

Cool thanks for the update. That seems like a good work around.  I'll check
it out once I get my server going.
On Aug 18, 2016 11:48 AM, "Robert Misiorowski"<EMAIL_ADDRESS>wrote:

This PR adds in the Address tab to settings.
@morebrownies https://github.com/morebrownies I found that overlaying
every delete button on a delete / add address looked kind of weird. I also
found that we want to prevent them from updating the form, since we clear
it on success, if they started to update it, they would just be
interrupted. So I just overlay the whole tab during saves:
[image: image]
https://cloud.githubusercontent.com/assets/794517/17786362/5cfb4a36-6539-11e6-9212-ab733d4811ac.png
Obviously, we could improve on that styling! Also, I only show the tab
overlay once they move their mouse. This prevents it from flickering on /
off very quickly on super fast saves (which most are. In fact I think
you'll have a hard time even getting the overlay to show - I had to muck
with the server to delay the response)
You can view, comment on, or merge this pull request online at:
https://github.com/OpenBazaar/openbazaar-desktop/pull/34
Commit Summary

Merge branch 'publish-status' into settings-address
tweaks to save contract between settings tab views and parent view
initial markup and css of the address tab
working through settings address tab
refactored communication between settings tabs and parent in regards
to savings
added comment to the Settings view
wired in save of new address
more tweaks to the settings address tab
added in test for the shipping address model
added in test for the splitIntoRows utiltiy function

File Changes

M .travis.yml
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-0
(3)
M js/languages/en-US.json
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-1
(17)
M js/models/ShippingAddress.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-2
(43)
M js/start.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-3
(14)
A js/templates/modals/settings/addresses.html
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-4
(7)
A js/templates/modals/settings/addressesForm.html
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-5
(83)
A js/templates/modals/settings/addressesList.html
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-6
(23)
R js/templates/modals/settings/general.html
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-7
(0)
R js/templates/modals/settings/page.html
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-8
(0)
M js/templates/modals/settings/settings.html
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-9
(7)
M js/utils/index.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-10
(31)
A js/views/modals/Settings/Addresses.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-11
(161)
A js/views/modals/Settings/AddressesForm.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-12
(32)
A js/views/modals/Settings/AddressesList.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-13
(40)
R js/views/modals/Settings/General.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-14
(23)
R js/views/modals/Settings/Page.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-15
(23)
M js/views/modals/Settings/Settings.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-16
(146)
M styles/_base.scss
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-17
(4)
M styles/_mixins.scss
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-18
(22)
M styles/components/_buttons.scss
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-19
(20)
M styles/components/_modal.scss
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-20
(2)
M styles/components/_statusBar.scss
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-21
(2)
M styles/modules/modals/_settings.scss
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-22
(37)
M test/_setup.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-23
(8)
A test/models/shippingAddress.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-24
(73)
A test/utils/index.js
https://github.com/OpenBazaar/openbazaar-desktop/pull/34/files#diff-25
(80)

Patch Links:

https://github.com/OpenBazaar/openbazaar-desktop/pull/34.patch
https://github.com/OpenBazaar/openbazaar-desktop/pull/34.diff

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenBazaar/openbazaar-desktop/pull/34, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AMdihif5fsYRD8n0r4cspfklklnvN5BBks5qhKjjgaJpZM4Jny7E
.

I'll catch up on the design questions this afternoon.

I'll merge this, another PR takes care of the style issues. We can change the zip/postal code text later if we want to.


Dask-friendly 1D interpolation

I'm writing an xarray+dask library to perform 1D interpolation (https://github.com/crusaderky/xarray_extras/blob/master/xarray_extras/interpolate.py).
The library takes a ND xarray with dask backend as y and one of its coords (a plain numpy array) as x.
Currently I'm wrapping around scipy.interpolate.make_interp_spline, and it's not great for a couple of reasons:

when working in dask, I might have multiple chunks on any axis of y except the interpolation axis; each chunk will translate to an equally-shaped array of spline coefficients. However, across all chunks there is only one array of knots, which (since my x is always a pure numpy array) is also a pure numpy array. With make_interp_spline, I am forced to needlessly re-compute the knots vector multiple times.
dask makes it super easy to wrap around functions that return numpy arrays. make_interp_spline however returns a BSpline object, from which I have to extract the t and c arrays.

My current code looks pretty awful:
spline = map_blocks(
    make_interp_spline,
    x, a.data, k=k, check_finite=False, dtype=float)
t_name = 'spline_t-' + tokenize(getattr, spline.name, 't')
spline_first_key = (spline.name, ) + (0, ) * spline.ndim
t = Array(
    {(t_name, 0): (getattr, spline_first_key, 't')},
    name=t_name, dtype=float,
    chunks=((x.size + k + 1, ), ))
    t.dask.update(spline.dask)
c = map_blocks(getattr, spline, 'c', dtype=float)

Besides looking ugly and needlessly recomputing t for each chunk, it also introduces a dependency on the first chunk from all other chunks in order to save memory (the spline evaluation of all chunks will read t from the first chunk). It would have been much nicer to just compute t in plain numpy.
Proposed design
Gut make_interp_spline and create two new functions out of it, make_interp_knots(x, k) and make_interp_coeffs(y, t, k), both of which return  a plain np.ndarray.
make_interp_spline would become a simple wrapper around those two. The only non-trivial piece of logic it would retain is the special short-circuit for k=0 and (k1=1 and t=None) (https://github.com/scipy/scipy/blob/v1.0.0/scipy/interpolate/_bsplines.py#L706).

Can you construct the knot vector once (copy-paste private finctions _not_a_knot/_augknt, both are trivial) and feed it to make_interp_spline(..., t=t)?

We had a discussion of whether make_interp_spline should return a tck-tuple or a BSpline object, in the PR which introduced it.

Am circling back to this now. What we can do is to refactor make_interp_spline into a wrapper over two leading-underscore functions, roughly _make_interp_knots(x, k, bc_type) and _make_interp_coefs(x, y, t, k, bc_type). These two functions will definitely not be public, no technically there will be no guarantees about them. However it's likely that they be quite stable on a timescale of several years at least. Then it's up to xarray or xarray_extras to decide if they want to reach into it and use these functions. Would that help @crusaderky ?

I haven't looked at this problem for 4 years, so my memory is failing me.
Private functions are better than having to reimplement everything from zero.
I think they're missing the axis and check_finite arguments (see my implementation above).

Indeed, these arguments are missing in my rough sketch above, should be added where needed in the real implementation. So if it's helpful for dask, PR is welcome (will review in less than four years, hopefully).

I'm afraid I stopped working on this a long time ago; xarray_extras as a whole is in long term maintenance only from my side.
Still I'll be happy to update xarray_extrras to use the scipy implementation if someone else codes it in.


Battery sensor no longer displayed

The battery sensor for the Sonic isn't displayed any more. The following error is showin the logs:
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 590, in state
    numerical_value = int(value)
                      ^^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'high'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 504, in async_add_entities
    await asyncio.gather(*tasks)
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 788, in _async_add_entity
    await entity.add_to_platform_finish()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1006, in add_to_platform_finish
    self.async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 742, in async_write_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 810, in _async_write_ha_state
    state = self._stringify_state(available)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 748, in _stringify_state
    if (state := self.state) is None:
                 ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/sensor/__init__.py", line 594, in state
    raise ValueError(
ValueError: Sensor sensor.battery has device class 'None', state class 'measurement' unit 'battery' and suggested precision 'None' thus indicating it has a numeric value; however, it has the non-numeric value: 'high' (<class 'str'>)

Thanks for letting me know. I'll get a fix for this pushed out in the next day or so

Can you update to latest release v0.1.12 and see if that has fixed it okay.

Yes that works now, thanks!


Error build with fresh tools: extension `zicsr' required

Hello!
Thank you for this great project with opensource license! I don't know Forth, but I'm interesting in pure Assembly code examples for RISC-V boards.
I tried to build the code with the latest Devuan testing, and it failed:
 % make build BOARD=longan-nano
/usr/bin/riscv64-unknown-elf-as -g -march=rv32imac -I src/boards/longan-nano -I src/mcus/gd32vf103 -I src -o fiveforths.o fiveforths.s
src/03-interrupts.s: Assembler messages:
src/03-interrupts.s:15: Error: unrecognized opcode `csrc mstatus,0x08', extension `zicsr' required
src/03-interrupts.s:18: Error: unrecognized opcode `csrs mie,zero', extension `zicsr' required
src/03-interrupts.s:22: Error: unrecognized opcode `csrw mtvec,t0', extension `zicsr' required
make: *** [Makefile:27: fiveforths.o] Error 1

Then I modified Makefile: ARCH ?= rv32imac_zicsr, and the code compiled successfully:
 % make build BOARD=longan-nano
/usr/bin/riscv64-unknown-elf-as -g -march=rv32imac_zicsr -I src/boards/longan-nano -I src/mcus/gd32vf103 -I src -o fiveforths.o fiveforths.s
/usr/bin/riscv64-unknown-elf-ld -m elf32lriscv -T src/boards/longan-nano/linker.ld -o fiveforths.elf fiveforths.o
/usr/bin/riscv64-unknown-elf-objcopy -O binary fiveforths.elf fiveforths.bin
/usr/bin/riscv64-unknown-elf-objcopy -O ihex fiveforths.elf fiveforths.hex
/usr/bin/riscv64-unknown-elf-objdump -D -S fiveforths.elf > fiveforths.dump

Software versions:
 % dpkg -l|grep riscv
ii  binutils-riscv64-unknown-elf          2.39-8+4                                     amd64        GNU assembler, linker and binary utilities for Risc-V processors
ii  gcc-riscv64-unknown-elf               12.1.0-7+11                                  amd64        GCC cross compiler for Risc-V processors

Just FYI.

Hi, thanks for the report.
I'll need to run some tests and figure out why this is happening. I'm not sure of the purpose of zicsr so let me research and I'll update the issue later.

Then I modified Makefile: ARCH ?= rv32imac_zicsr, and the code compiled successfully:

You can also just type make ARCH=rv32imac_zicsr. That's why I defined the ARCH variable with ?=.
It seems this is caused by a breaking change in gcc/binutils, where zicsr must now be explicitly provided.
I'll fix this and publish a new release after testing it.
Thanks @vazhnov for the detailed bug report and debugging.


Bug Fix - allow array of primitives in Web Api models

I've hit a bug in the Web Api implementation of FluentValidation. When I submit a model with a byte array in it FV tries to load a custom validator for each element of the array (in this case a byte).
The exception being thrown was:
"Additional information: No registration for type IValidator could be found. Note that there exists a registration for a different type FluentValidation.IValidator while the requested type is FluentValidation.IValidator<System.Byte>"
The model being submitted is:
    public class CreateDocumentCommand
    {
        public string FileName { get; set; }
        public byte[] Content { get; set; }
        public string Description { get; set; }
    }

The implementation is using the factory method (with Simple Injector).
public class ApplicationValidatorFactory : IValidatorFactory
 {
     private readonly Container container;
     public ApplicationValidatorFactory(Container container)
     {
         this.container = container;
     }

     /// Gets the validator for the specified type.
     public IValidator<T> GetValidator<T>()
     {
         return container.GetInstance<IValidator<T>>();
     }

     /// Gets the validator for the specified type.
     public IValidator GetValidator(Type type)
     {
         var validator = typeof(IValidator<>).MakeGenericType(type);
         var foo = (IValidator)container.GetInstance(validator);
         return foo;
     }
 }

        FluentValidationModelValidatorProvider.Configure(config, x =>
         {
             x.ValidatorFactory = new ApplicationValidatorFactory(container);
         });

The fix below extetends the IsSimpleType method to include arrays of primativetypes and resolves the issue.
Kind Regards
Steve

Thanks!

Thanks for a great library fella!


Missing "blocks" in S3 public access report

If I get the logic correctly, there is a "blocks" missing in the following line:
https://github.com/turbot/steampipe-mod-aws-compliance/blob/c7cea47662c03f4cc4a84a17e41872e8ace611dc/query/s3/s3_public_access_block_account.sql#L18
It should say 'Account level public access blocks not enabled for: '.

@fbausch, Appreciate using Steampipe; thank you!
You are right to catch the typo, we will fix the same, or you may feel free to raise PR. Thank you


As a DevOps Engineer I want to easily initialize Terraform's backend at AWS to store the state remotely

There're some approaches to this implementation:

Terraform only
Terraform and custom provider (Golang)
Terragrunt and Terraform
Ansible and Terraform
Ansible only
Custom application (Golang)

Terraform only
Complexity: low
Match Requirements: yes
Operational Scalability: high
Terraform and custom provider (Golang)
Complexity: high (need to publish and mantain a provider)
Match Requirements: yes
Operational Scalability: medium (need a team with Golang skills)
Terragrunt and Terraform
Complexity: medium (need to learn one more tool, not so popular)
Match Requirements: maybe (Terragrunt is not in our toolset)
Operational Scalability: high
Ansible and Terraform
Complexity: high (two completely different tools)
Match Requirements: maybe (we don't have Ansible in our toolset)
Operational Scalability: medium (although Ansible is a popular tool, this option requires a skill with it from our teams)
Ansible only
Complexity: low
Match Requirements: maybe (Ansible is not so good as Terraform to provision infrastructure)
Operational Scalability: high
Custom application (Golang)
Complexity: high (need to maintain an application)
Match Requirements: yes
Operational Scalability: low (need a highly skilled team)

I would like contribute with a little experience that i had on the first scenario using only terraform!I tried to implement this using the following logic:
To first run:

I use the data sources to verify if the module already exists to no try create. like this:

data "azurerm_resources" "exists" {
  resource_group_name = local.rg_name
  type                = "Microsoft.Storage/storageAccounts"
}

I create a bucket to save the state if the bucket no exists

resource "azurerm_resource_group" "main" {
  count    = length(data.azurerm_resources.exists.resources) == 0 ? 1 : 0
  name     = local.rg_name
  location = var.resource_group_location

  depends_on = [
    data.azurerm_resources.exists
  ]
}

resource "azurerm_storage_account" "main" {
  count = local.create ? 1 : 0

  name                = local.name_alphanum
  resource_group_name = azurerm_resource_group.main[0].name
  location            = azurerm_resource_group.main[0].location

  account_kind              = "BlobStorage"
  min_tls_version           = "TLS1_2"
  enable_https_traffic_only = true
  is_hns_enabled            = false
  nfsv3_enabled             = false
  shared_access_key_enabled = var.shared_access_key_enabled
  allow_blob_public_access  = false

  account_tier             = var.account_tier
  account_replication_type = var.account_replication_type

  depends_on = [
    data.azurerm_resources.exists
  ]
}

resource "azurerm_storage_container" "main" {
  count = local.create ? 1 : 0

  name                  = local.container_name
  storage_account_name  = azurerm_storage_account.main[0].name
  container_access_type = "private"

  depends_on = [
    data.azurerm_resources.exists
  ]
}

Then run my terraform to create infrastructure

⚠Problem
This has a big problem, terraform is declarative, that is he should decide when create or no create.
When I first tried this I had the following problem:

✅ In the first run it ok. Terraform create the bucket
❌ In the second run fails. Because terraform see that already exists this resource then no create the bucket but the problem is he created then he understands that should destroy since the resource should no longer exist.

♻ Solution
To solve this i use this lifecycle in all resources that has been created:
  lifecycle {
    ignore_changes = all
  }

So next time it won't destroy existing resources. like this:
resource "azurerm_resource_group" "main" {
  count    = length(data.azurerm_resources.exists.resources) == 0 ? 1 : 0
  name     = local.rg_name
  location = var.resource_group_location

  lifecycle {
    ignore_changes = all
  }

  depends_on = [
    data.azurerm_resources.exists
  ]
}

resource "azurerm_storage_account" "main" {
  count = local.create ? 1 : 0

  name                = local.name_alphanum
  resource_group_name = azurerm_resource_group.main[0].name
  location            = azurerm_resource_group.main[0].location

  account_kind              = "BlobStorage"
  min_tls_version           = "TLS1_2"
  enable_https_traffic_only = true
  is_hns_enabled            = false
  nfsv3_enabled             = false
  shared_access_key_enabled = var.shared_access_key_enabled
  allow_blob_public_access  = false

  account_tier             = var.account_tier
  account_replication_type = var.account_replication_type

  lifecycle {
    ignore_changes = all
  }

  depends_on = [
    data.azurerm_resources.exists
  ]
}

resource "azurerm_storage_container" "main" {
  count = local.create ? 1 : 0

  name                  = local.container_name
  storage_account_name  = azurerm_storage_account.main[0].name
  container_access_type = "private"

  lifecycle {
    ignore_changes = all
  }

  depends_on = [
    data.azurerm_resources.exists
  ]
}

💪 pros

No needs install other tools to work
Use the same language (HCL) for all

🥀 cons

Very complex to implement
Not all modules are searchable without returning an error.

Conclusion
This solution is so far from be the best or most appropriate for the problem. Because in addition to go against the principles of the standard terraform paradigm. It is impossible have a simple and standardized implementation.