Monte Carlo estimation of KL divergence differs a lot from the analytical result

Hello everybody,
I'm currently trying to implement a VAE with several layers of stochastic variables. As the KL divergence is not tractable, I'm trying to approximate it with Monte Carlo method. As it didn't work at all, I tried to compute it with only one layer, and it doesn't work : I can verify it by comparing the Monte Carlo estimation with the analytical value (see for example Tutorial on VAE)
So i defined two functions, one computing the analytical value and the other one a Monte Carlo estimate.
The base code is the tutorial from Keras, the lines 22 to 49 being modified
n_sample = 500
# utils
def log_stdnormal(x):
    """log density d'une loi normale centree"""
    c = - 0.5 * math.log(2*math.pi)
    result = c - K.square(x) / 2
    return result

def log_normal2(x, mean, log_var):
    """log density d'une loi normale de moyenne mean et de variance exp(log_var)"""
    c = - 0.5 * math.log(2*math.pi)
    result = c - log_var/2 - K.square(x - mean) / (2 * K.exp(log_var) + 1e-8)
    return result
# input
vae_input = Input(shape=(original_dim,nchar), batch_shape=(batch_size,original_dim,nchar))

z_mean_0, z_log_var_0 = encoder(vae_input)
eps0 = Lambda(sample_eps, name='sample_eps0')([z_mean_0, z_log_var_0])
print(eps0)
z0 = Lambda(sample_z, name='sample_z0')([z_mean_0, z_log_var_0, eps0])

x_decoded_mean = generator(z0)

def kl_cost(args):
    z_mean, z_log_var = args
    return - 0.5 * K.sum(1 + z_log_var - K.square(z_mean) - K.exp(z_log_var))
kl_ = Lambda(kl_cost)([z_mean_0, z_log_var_0])
kl_model = Model(vae_input, kl_)

def monte_carlo_kl_div(args):
    mean, log_var = args
    for k in range(n_sample):
        epsilon = K.random_normal(shape=(batch_size, latent_dim), mean=0.,
                              std=epsilon_std)
        z = mean + K.exp(log_var / 2) * epsilon
        try:
            loss += K.sum(log_normal2(z, mean, log_var) - log_stdnormal(z) , -1)
        except NameError:
            loss = K.sum(log_normal2(z, mean, log_var) - log_stdnormal(z) , -1)
    return loss / n_sample

carlo = Lambda(monte_carlo_kl_div)([z_mean_0, z_log_var_0])
carlo_model = Model(vae_input, carlo)

vae = Model(vae_input, x_decoded_mean)

When evaluating carlo_model on a test sample, I get negative values. Why ? It should be approximately the same as the output of kl_model

Have you tried using the KL implementation provided in losses/objectives.py? Some missing details such as clipping might have an impact.

How many Monte Carlo samples are you taking? MC estimates of the KL are quite high variance, you might need 100 samples or so to get a reasonable approximation. It's not too difficult though, because once you have a sample you just have the calculate the log-density ratio which is not very computationally taxing.

Hi, thanks for your answers
I finally made it work, and the problem seemed to be purely numerical, as @talpay pointed out
Indeed, I was generating logvar in a very narrow range (about [-1e-3, 1e3]), and when I switched to generating std instead (by using a softplus activation layer), I got values of about log(2), which made it work.
I'm not sure this was the reason but I can't think of anything else
Using MC estimation with 50 samples instead of analytical formula caused the model to have the same train/test performance, with a 28% training speed increase


Cannot change thickness of drawing/spapes on iPad.

Currently supported by touch events have been supported, I hope the official support.

@Edric-Li
Thank you for your suggestion.
I will register it as a good first issue.

@lja1018 Thanks.

Hello, do you have a good solution to the problem of changing the size of drawing slider on ipad? Thank you


MINOR: [Java] Fix maven-checkstyle-plugin configuration

Rationale for this change
maven-checkstyle-plugin configuration refers to several unrecognized properties, causing build output to print several messages like:

[WARNING] Parameter 'format' is unknown for plugin 'maven-checkstyle-plugin:3.1.0:check (validate)'

What changes are included in this PR?
Fix checkstyle configuration and use the correct outputFileFormat and inputEncoding properties in place of the unrecognized format and encoding ones.
Are these changes tested?
As this is a build change with no code change, only via a local build + visual inspection of the build output
Are there any user-facing changes?
No

@github-actions crossbow submit -g java

Thanks @laurentgo for creating this PR, I kicked the crossbows to validate the change.

@lidavidm seems like the crossbows are passing, there is a failing Windows CI and it seems to be not related. Although we could re-run and verify. The change looks fine to me.


Add product lifecycle page and regions page

Description & motivation
Added a new page for product lifecycles and regions
Closes #2135 and #2134
To-do before merge

[x] Update any content from google docs
[x] Link to new region page in the following docs:

https://docs.getdbt.com/docs/dbt-cloud/cloud-configuring-dbt-cloud/connecting-your-database#ip-restrictions
https://docs.getdbt.com/docs/dbt-cloud/dbt-cloud-enterprise/setting-up-enterprise-snowflake-oauth#server-error-500

Prerelease docs
If this change is related to functionality in a prerelease version of dbt (delete if not applicable):

[ ] I've added versioning components, as described in "Versioning Docs"
[ ] I've added a note to the prerelease version's Migration Guide

Checklist
If you added new pages (delete if not applicable):

[x ] The page has been added to website/sidebars.js
[ ] The new page has a unique filename

If you removed existing pages (delete if not applicable):

[ ] The page has been removed from website/sidebars.js
[ ] An entry has been added to _redirects
[ ] Run link testing to update the links that point to the deleted page

hi @azzam34, made some tweaks to the language so it has some active voice changes (rather than passive). i can tell some of the product lifecycle stages are in passive voice for a reason so i changed some content to active if it didn't have anything to do with the lifecycle state.
i also added plan availability to the regions content. the reason why is - if we're linking to the regions page on other pages like the [IP restrictions section](https://docs.getdbt.com/docs/dbt-cloud/cloud-configuring-dbt-cloud/connecting-your-database#ip-restrictions, then i think we also need to treat the page as a 'regions and IP' page, which is an important reference to dev, team and enterprise users. (there are a lot of users that reach out to support asking for IP addresses)
please let me know what you think and provide any feedback, thank you!


Silence! 🤫 😄

Wonderful plugin, thanks for making it!
It's a bit noisy with auto-sync: I constantly get walls of "Syncing", "Working branch clean", "Successfully set remote":

Could you imagine a change where for example notifications were put into something like warning and info functions, and an option for the user to switch to "warnings only" in which case the plugin would run silently for all standard operations?

I made a quick prototype-branch just as illustration, to show the kind of direction I mean.


New UI for task accordion

Developer Name: Joy Gupta
PR Number(s): 606
UI Fixes/Improvements:

[x] Replace assigned to with Created By
[x] Add Priority field to the right
[x] Add Type field to the right
[x] Implement infinite scroll and pagination in the backend, instead of handling it in the frontend
[x] Use the following UI as it also serves the same purpose in my-site

Issue Ticket Number:

https://github.com/Real-Dev-Squad/website-dashboard/issues/601

Backend Changes

[ ] Yes
[x] No

Frontend Changes

[x] Yes
[ ] No

Is Under Feature Flag

[x] Yes
[ ] No

Database changes:

[ ]  Yes
[x] No

Breaking changes (If your feature is breaking/missinsomething please mention pending tickets):

[ ] Yes
[x] No

Deployment notes:
NA
Tested in local

[x] Yes
[ ] No

QA Instructions, Screenshots, Recordings
Run yarn test and there you can see test case running.

Working Proof
Functionality
https://github.com/Real-Dev-Squad/website-dashboard/assets/56365512/d62efba3-d9ac-4809-8659-5257da34a9ac
Responsiveness
https://github.com/Real-Dev-Squad/website-dashboard/assets/56365512/e429191e-29f5-4f21-851d-fcffa1b0a155

Some of the cards may appear red even though the progress is 100%. This is because I've implemented the same logic that is currently used on the status side. Additionally, the progress bar will persist no matter what the status is. This is because this page does not allow anyone to change it; it is only in read-only mode. Also, hiding it wouldn't make sense because the Super User wouldn't be able to find out the task completion percentage.

Some of the cards may appear red even though the progress is 100%. This is because I've implemented the same logic that is currently used on the status side.

Status site task card colours are broken. Please don't implement same here.

Additionally, the progress bar will persist no matter what the status is. This is because this page does not allow anyone to change it; it is only in read-only mode. Also, hiding it wouldn't make sense because the Super User wouldn't be able to find out the task completion percentage.

We cannot update the status to DONE or VERIFIED unless the percentage is 100%. So even if you display that percentage it will be always 100% providing no extra value. That is the reason why it's proposed to remove the progress bar for other statuses in my-site and status site.

Also please add more description. Currently it's not really clear what this PR does.

Implement infinite scroll and pagination in the backend, instead of handling it in the frontend

Hi @ardourApeX, before I review your pull request, I'd like to clarify a few things to avoid any confusion:
Where is your backend pull request?
Why is this point included in the pull request description?
Could you please provide more information about these points? This will help me better understand your pull request.

what you done in this pr...please give information(high level implementation in words please) in description
easy for us to review your pr as a reviewer.

what you done in this pr...please give more information in description(high level implementation in words please else not understand what you done in this pr) easy for us to review your pr as a reviewer.
Where is your design doc???

While reviewing the issue ticket @iamitprakash suggested that design doc is not required as it just an UI update

Implement infinite scroll and pagination in the backend, instead of handling it in the frontend

Hi @ardourApeX, before I review your pull request, I'd like to clarify a few things to avoid any confusion:
Where is your backend pull request? Why is this point included in the pull request description?
Could you please provide more information about these points? This will help me better understand your pull request.

Also please add more description. Currently, it's not really clear what this PR does.

There are no backend changes related to this PR as I have just implemented an existing API. Also, I have updated both the PR and issue description. Let me know if anything else is required. Thank You!

Some of the cards may appear red even though the progress is 100%. This is because I've implemented the same logic that is currently used on the status side.

Status site task card colours are broken. Please don't implement same here.
Issue for the right task colour in cards : [Design Doc] -  Tasks card enhancement in status site website-status#931

Given that we have not yet decided on the criteria for determining if a task will be considered overdue before the ETA, I have not implemented it at this time. The same we have discussed today.

Additionally, the progress bar will persist no matter what the status is. This is because this page does not allow anyone to change it; it is only in read-only mode. Also, hiding it wouldn't make sense because the Super User wouldn't be able to find out the task completion percentage.

We cannot update the status to DONE or VERIFIED unless the percentage is 100%. So even if you display that percentage it will be always 100% providing no extra value. That is the reason why it's proposed to remove the progress bar for other statuses in my-site and status site.

Done

Test stats are useless showing coverage for mock data

Can you please elaborate it? What else is required from the tests?


listing restaurants on browse page

comma sepearation on times could help

comma sepearation on times could help

good idea. planning to fix in future pr, lets get this in now.


Question: how to use generated classes in same eclipse project

I'm using the gradle plugin in a java project where I want to use the generated classes. I'd prefer not to have to create a separate project containing just the generated pojos and make that a dependency of the project where I want to use the generated classes.
The problem I'm having is that the gradle build completes successfully, the generated classes are placed in build/generated-sources/j2sp, but the Eclipse IDE can't resolve the import statements I have for the generated classes, so it generates all these noisy nags everywhere about types that it can't find.
I suppose there is a simple fix, but I've tried manipulating the build path to no avail - is there a way to use the generated classes within the same project without Eclipse complaining about the imports?

Think I figured it out. In Configure Build Path... I had added the folder containing all of the generated classes (build/generated-sources/j2sp/com/path/to/package) instead of just build/generated-sources/j2sp. Once I changed the build path to include just the top-level folder it worked.


Access is denied to local/appplication data

Hi,
I run this command helm dependency update and it results in an error:
walk.go:74: found symbolic link in path: C:\Users\xyz\AppData\Local\Application Data resolves to C:\Users\xyz\AppData\Local
Error: open C:\Users\xyz\AppData\Local\Application Data: Access is denied.
I realized that the folder does not appear (but exists, cause when i tried to create one, it said there is an existing folder with that name).
Yet, I didn't know what to do with that piece of information so, I tried to give it a different path like so : helm dependency update --repository-config "C:\TEMP\helm\repositories.yaml", but it still prints out the same error
Any thoughts?
Thanks
Output of helm version: v3.7.1
Windows 10 pro

I'm pretty sure Helm is just accessing the filesystem here. And looking for per-user config in this location. Can you provide more info if you think this is a Helm (logic) error? Rather than a fs setup/permissions error.

Ouh sorry,
Thought i replied.
I don't know how i could provide more information on that error aside from the error itself.
I was hoping that would be something trivial, some settings I should enable or diasable.

Error: open C:\Users\xyz\AppData\Local\Application Data: Access is denied.

To start with the basic. Does this directory exist? And is it readable by the user you are running Helm under?

this problem cause you forgot to chdir into chart directory.


making sure that ternary expression does not change context

fixing #737 making sure that ternary expression does not change context + exporting functions for test purposes

@RobLoach please take a look

🚀


nixos/moinmoin: fix module by switching to eventlet worker

Motivation for this change
The service (and consequently nixosTests.moinmoin) was failing with:
gunicorn[2192104]: [2020-12-31 13:35:28 +0000] [2192104] [ERROR] Exception in worker process
gunicorn[2192104]: Traceback (most recent call last):
gunicorn[2192104]:   File "/nix/store/jmc14qf1sfnlhw27xyyj862ghkmdkj5a-python2.7-gunicorn-19.10.0/lib/python2.7/site-packages/gunicorn/arbiter.py", line 586, in spawn_worker
gunicorn[2192104]:     worker.init_process()
gunicorn[2192104]:   File "/nix/store/jmc14qf1sfnlhw27xyyj862ghkmdkj5a-python2.7-gunicorn-19.10.0/lib/python2.7/site-packages/gunicorn/workers/ggevent.py", line 196, in init_process
gunicorn[2192104]:     self.patch()
gunicorn[2192104]:   File "/nix/store/jmc14qf1sfnlhw27xyyj862ghkmdkj5a-python2.7-gunicorn-19.10.0/lib/python2.7/site-packages/gunicorn/workers/ggevent.py", line 65, in patch
gunicorn[2192104]:     monkey.patch_all(subprocess=True)
gunicorn[2192104]:   File "/nix/store/fysf67w3i8iv1hfvp536nl8jbzqyk1s7-python-2.7.18-env/lib/python2.7/site-packages/gevent/monkey.py", line 1160, in patch_all
gunicorn[2192104]:     from gevent import events
gunicorn[2192104]:   File "/nix/store/fysf67w3i8iv1hfvp536nl8jbzqyk1s7-python-2.7.18-env/lib/python2.7/site-packages/gevent/events.py", line 67, in <module>
gunicorn[2192104]:     from zope.interface import Interface
gunicorn[2192104]: ImportError: No module named zope.interface

The gevent package does depend on zope_interface however. I wasn't able to figure out why exactly cannot the module be imported and decided to switch to different gunicorn backend instead.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/441

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/441

this should probably update to python3 instead
zope implements a native namespace so there's not conflict in python3, but this isn't a feature in python2

this should probably update to python3 instead
zope implements a native namespace so there's not conflict in python3, but this isn't a feature in python2

Unfortunately moinmoin probably won't be ported to python3:( http://moinmo.in/Python3

Unfortunately moinmoin probably won't be ported to python3:( http://moinmo.in/Python3

@ofborg test moinmoin

Thanks for the fix! It seems that 20.09 is also effected by this. Could you backport this PR?


Feature Request: Package Manager Support (brew, scoop, apt, etc)

Hey there @sarub0b0,
This is a great tool, thanks for building it!
Could you consider publishing this on brew, ports, scoop, chocolatey, apt, pacman etc as this would be easier for installation rather than cargo.
I usually find package managers more quicker and much safer to install rather than downloading binaries or compiling the entire project on my machine.

Thanks for your feature request! It's much appreciated. I've postponed it because I wasn't sure how to approach it, but now that you've raised it, I'd like to address it gradually.


Problem with Restart emulator command.

Restart emulator command dont work fine.
When i execute this command and i reconnect on client, impossible enter on the room.

The Restart Command isn't 100%.
Will check why.

Okey, thanks


Does not create md5 string

I guess the issue is in Data+Crypto file
I had to write 0 in place of index while interating thorugh pointer
var hex: String { 		var string = "" 		enumerateBytes { pointer, index, _ in 			for i in index..<pointer.count { 				string += String(format: "%02x", pointer[i]) 			} 		} 		return string 	}

Which version of Swift are you on? I recently changed this to make it work in Swift 3.1

I had a similar issue and replacing index with 0 worked for me.
The difference though is that I'm using Swift 3.0.2 which is another problem because I can't install Crypto via Carthage since 0.5.1 requires 3.0.1 and 0.52 requires 0.3.1.
I ended up installing CommonCrypto and manually adding the files from your library to my project.
I don't expect a fix for my specific issue since I can easily move to 3.1 and use the latest version of your library and for now I'm doing just a spike.
Nice work btw :)

Does a86569707ea7fa460fa4ad75a7195824074d259f work for you?

Yes works for me. Although I did not check in Swift  3.1 yet.

Yeah, a865697 works for me too. Thank you.


multiple LLM clients

This generalizes to allow different clients that can do LLM inference.

removes AIPL's dependence on llama.cpp
new assumption is that everything will by default be over HTTP
adds the idea of a client (similar to HELM) which handles the inference
adds clients for OpenAI, GooseAI, and a custom HTTP server that is compatible with the openai library

This is so much better and clearer, Rowan! Tysm for refactoring!
Did you test on a local model? I tested with OpenAI, and once I removed the requirement for the environment variable it worked great!

@anjakefala yep, I was using the testing code at the bottom of clients.py to make sure each of those worked (though didn't test permutations of env, thanks for the catch!).


Hamlet crashes when generating large size pdf report

In some cases with over ~30 images in the report. Python package pdfkit and its dependency wkhtmltopdf cannot properly generate a pdf report.

This error could not be reproduced. In cases where HAMLET fails on the report generation, please attempt to re-run the pipeline, which should resume the pipeline where it left of and generate the PDF report.


Bootstrap 4 Live Search -> Active item is "blank"

The default CSS for an active item in the dropdown on Bootstrap 4 is "white", so consider to add this line to the css file:
.dropdown-item.active{color:#000;}
thanks.

The default background color for an active item is also dark blue...so the text should be readable. Black text would be difficult to read against a dark blue background.


update code for Ruby 3 compatibility

Context

What changes are being made? Why are you making this change? Provide a short description or bulleted list.

Fix ruby 3 deprecation warnings
https://opendoor.atlassian.net/browse/DEVXP-1503
Test Plan
canary-web staging

Mitigation

Checklist

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to external documentation if necessary (READMEs, Confluence, etc.)
[ ] I have checked that other PRs this PR depends on have already been deployed.
[ ] I can confirm that if this PR introduces a DB schema change, I have read the data contract schema change run book and have followed all the steps outlined in the wiki to ensure that the change aligns with the data contract.

Closing, keeping this branch as the new default


Feature/weighted

[X] Closes #422
[X] Tests added
[X] Fully documented, including whats-new.rst for all changes and api.rst for new API

I took a shot at the weighted function - I added a DataArrayWeighted class, that currently only implements mean. So, there is still quite a bit missing (e.g. DatasetWeighted), but let me know what you think.
import numpy as np
import xarray as xr
da = xr.DataArray([1, 2])
weights = xr.DataArray([4, 6])
da.weighted(weights).mean()

# <xarray.DataArray ()>
# array(1.6)

There are quite a number of difficult edge cases with invalid data, that can be discussed.

I decided to replace all NaN in the weights with 0.
if weights sum to 0 it returns NaN (and not inf)

weights = xr.DataArray([0, 0])
da.weighted(weights).mean()

The following returns NaN (could be 1)

da = xr.DataArray([1, np.nan])
weights = xr.DataArray([1, 0])
da.weighted(weights).mean(skipna=False)

It could be good to add all edge-case logic to a separate function but I am not sure if this is possible...

Hello @mathause! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xarray/core/common.py:

Line 542:5: E301 expected 1 blank line, found 0

In the file xarray/core/weighted.py:

Line 2:33: W291 trailing whitespace
Line 21:1: W293 blank line contains whitespace
Line 23:1: W293 blank line contains whitespace
Line 49:1: W293 blank line contains whitespace
Line 51:1: W293 blank line contains whitespace
Line 53:5: E303 too many blank lines (2)
Line 55:80: E501 line too long (82 > 79 characters)
Line 91:1: W293 blank line contains whitespace

In the file xarray/tests/test_weighted.py:

Line 11:1: E302 expected 2 blank lines, found 1
Line 17:1: E302 expected 2 blank lines, found 1
Line 46:1: E303 too many blank lines (3)
Line 48:1: W293 blank line contains whitespace
Line 51:1: W293 blank line contains whitespace
Line 52:6: W291 trailing whitespace
Line 60:1: W293 blank line contains whitespace
Line 69:80: E501 line too long (101 > 79 characters)
Line 82:1: W391 blank line at end of file

I updated the PR

added a weighted sum method
tried to clean the tests

Before I continue, it would be nice to get some feedback.

Currently I only do very simple tests - is that enough? How could these be generalized without re-implementing the functions to obtain expected.
Eventually it would be cool to be able to do da.groupby(regions).weighted(weights).mean() - do you see a possibility for this with the current implementation?

As mentioned by @aaronspring, esmlab already implemented weighted statistic functions. Similarly, statsmodels for 1D data without handling of NaNs (docs / code). Thus it should be feasible to implement further statistics here (weighted std).

Hi @mathause - We really appreciate your contribution. Sorry your PR has stalled!
Do you think you can respond to @fujiisoup's review and add documentation? Then we can get this merged.

Thanks, I am still very interested to get this in. I don't think I'll manage before my holidays, though.

I finally made some time to work on this - altough I feel far from finished...

added a DatasetWeighted class
for this I pulled the functionality our of DataArrayWeighted class in to own functions taking da and weights as input
the tests need more work
implanted the functionality using xr.dot -> this makes the logic a bit more complicated

Questions:

does this implementation look reasonable to you?
xr.dot does not have a axis keyword is it fine if I leave it out in my functions?
flake8 fails because I use  @overload for typing -> should I remove this?
Currently I have the functionality 3-times: once as _weighted_sum, once as da.weighted.sum() and once as ds.weighted().sum: how do I best test this?

I decided to replace all NaN in the weights with 0.

Can we raise an error instead? It should be easy for the user to do weights.fillna(0) instead of relying on xarray's magical behaviour.

if weights sum to 0 it returns NaN (and not inf)

Should we raise an error here?

The following returns NaN (could be 1)

I think NaN is fine since that's the result of 1 + 0*np.nan

I decided to replace all NaN in the weights with 0.
Can we raise an error instead? It should be easy for the user to do weights.fillna(0) instead of relying on xarray's magical behaviour.

I agree, requiring valid weights is a sensible choice.

if weights sum to 0 it returns NaN (and not inf)
Should we raise an error here?

Im not sure... Assume I want to do a meridional mean and only have data over land, this would then raise an error, which is not what I want.

This is now ready for a full review. I added tests for weighted reductions over several dimensions and docs.

I updated this once more. Mostly moved the example to a notebook.

This is going in.
Thanks @mathause. This is a major contribution!

Great! Thanks for all the feedback and support!

Thanks @mathause !

Big time!!!! Thanks @mathause! #422 was opened in June of 2015, amazing.

#422 was opened in June of 2015, amazing.

😂
@mathause props for the persistence...

I realize this is a bit late, but I'm still concerned about memory usage, specifically in https://github.com/pydata/xarray/blob/master/xarray/core/weighted.py#L130 and https://github.com/pydata/xarray/blob/master/xarray/core/weighted.py#L143.
If da.sizes = {'dim_0': 100000, 'dim_1': 100000}, the two lines above will cause da.weighted(weights).mean('dim_0') to create two simultaneous temporary 100000x100000 arrays, which could be problematic.
I would have implemented this using apply_ufunc, so that one creates these temporary variables only on as small an array as absolutely necessary -- in this case just of size sizes['dim_0'] = 100000. (Much as I would like to, I'm afraid I'm not able to contribute code.) Of course this won't help in the case one is summing over all dimensions, but might as well minimize memory usage in some case even if not in all.

We do those sorts of operations fairly frequently, so it's not unique here. Generally users should expect to have available ~3x the memory of an array for most operations.
@seth-p it's great you've taken an interest in the project! Is there any chance we could harness that into some contributions? 😄

tldr: if someone knows how to do memory profiling with reasonable effort this can still be changed
It's certainly not too late to change the "backend" of the weighting functions. I once tried to profile the memory usage but I gave up at some point (I think I would have needed to annotate a ton of functions, also in numpy).
@fujiisoup suggested using xr.dot(a, b) (instead of (a * b).sum()) to ameliorate part of the memory footprint. Which is done, however, this comes at a performance penalty. So an alternative code path might actually be beneficial.
Also mask is an array of dtype bool, which should also help.
It think it should not be very difficult to write something that can be passed to apply_ufinc, probably similar to:
https://github.com/pydata/xarray/blob/e8a284f341645a63a4d83676a6b268394c721bbc/xarray/tests/test_weighted.py#L161
So there would be three possibilities: (1) the current implementation (using xr.dot(a, b)) (2) something similar to expected_weighted (using (a * b).sum()) (3) xr.apply_ufunc(a, b, expected_weighted, ...). I assume (2) is fastest with the largest memory footprint, but I cannot tell about (1) and (3).

@max-sixty, I wish I could, but I'm afraid that I cannot submit code due to employer limitations.

@mathause, ideally dot() would support skipna, so you could eliminate the cand pass the skipna down the line. But alas it doesn't...
(a * b).sum() would likely make things worse, though has the advantage that it does support skipna. I think it would depend on the exact shape/value of da, weights, and dim whether this would be better than the current implementation.
Either way, this only addresses the da = da.fillna(0.0), not the mask = da.notnull().
Also, perhaps the test if weights.isnull().any() in Weighted.__init__() should be optional?
Maybe I'm more sensitive to this than others, but I regularly deal with 10-100GB arrays.

Btw, have you looked at https://docs.scipy.org/doc/numpy/reference/generated/numpy.average.html?

There is some stuff I can do to reduce the memory footprint if skipna=False or not da.isnull().any(). Also, the functions should support dask arrays out of the box.

ideally dot() would support skipna, so you could eliminate the da = da.fillna(0.0) and pass the skipna down the line. But alas it doesn't...

Yes, this would be nice. xr.dot uses np.einsum which is quite a beast that I don't entirely see through. I don't expect it to support NaNs any time soon.
What could be done, though is to only do da = da.fillna(0.0) if da contains NaNs.

(da * weights).sum(dim=dim, skipna=skipna) would likely make things worse, I think, as it would necessarily create a temporary array of sized at least da, no?

I assume so.  I don't know what kind of temporary variables np.einsum creates. Also np.einsum is wrapped in xr.apply_ufunc so all kinds of magic is going on.

Either way, this only addresses the da = da.fillna(0.0), not the mask = da.notnull().

Again this could be avoided if skipna=False or if (and only if) there are no NaNs in da.

Also, perhaps the test if weights.isnull().any() in Weighted.__init__() should be optional?

Do you want to leave it away for performance reasons? Because it was a deliberate decision to not support NaNs in the weights and I don't think this is going to change.

Maybe I'm more sensitive to this than others, but I regularly deal with 10-100GB arrays.

No it's important to make sure this stuff works for large arrays. However, using xr.dot already gives quite a performance penalty, which I am not super happy about.

have you considered using these functions? [...]

None of your suggested functions support NaNs so they won't work.
I am all in to support more functions, but currently I am happy we got a weighted sum and mean into xarray after 5(!) years!
Further libraries that support weighted operations:

esmlab (xarray-based, supports NaN)
statsmodels (numpy-based, does not support NaN)

All good points:

What could be done, though is to only do da = da.fillna(0.0) if da contains NaNs.

Good idea, though I don't know what the performance hit would be of the extra check (in the case that da does contain NaNs, so the check is for naught).

I assume so. I don't know what kind of temporary variables np.einsum creates. Also np.einsum is wrapped in xr.apply_ufunc so all kinds of magic is going on.

Well, (da * weights) will be at least as large as da. I'm not certain, but I don't think np.einsum creates huge temporary arrays.

Do you want to leave it away for performance reasons? Because it was a deliberate decision to not support NaNs in the weights and I don't think this is going to change.

Yes. You can continue not supporting NaNs in the weights, yet not explicitly check that there are no NaNs (optionally, if the caller assures you that there are no NaNs).

None of your suggested functions support NaNs so they won't work.

Correct. These have nothing to do with the NaNs issue.
For profiling memory usage, I use psutil.Process(os.getpid()).memory_info().rss for current usage and resource.getusage(resource.RUSAGe_SelF).ru_maxrss for peak usage (on linux).


SDK no longer works with DPNS-related methods

Expected Behavior
SDK should be able to register/retrieve names using the names.* methods.
Current Behavior
Since https://github.com/dashpay/platform/pull/1984, this error is returned when using names.register():
Error: JsonSchemaError: Additional properties are not allowed ('dashUniqueIdentityId' was unexpected), path: /records
Also receiving errors like this when attempting to retrieve a name:
InvalidRequestError: where clause on non indexed property error: query must be for valid indexes, valid indexes are: {"identityId": Index { name: "identityId", properties: [IndexProperty { name: "records.identity", ascending: true }], unique: false, null_searchable: false, contested_index: None }, "parentNameAndLabel": Index { name: "parentNameAndLabel", properties: [IndexProperty { name: "normalizedParentDomainName", ascending: true }, IndexProperty { name: "normalizedLabel", ascending: true }], unique: true, null_searchable: true, contested_index: Some(ContestedIndexInformation { field_matches: {"normalizedLabel": Regex(Regex("^[a-zA-Z01-]{3,19}$"))}, resolution: MasternodeVote }) }}

Possible Solution
Steps to Reproduce (for bugs)

Attempt to create or retrieve a name using the RC package for JS SDK

Context
Your Environment

Version used: 4.0.0-rc.1 on testnet

Should be resolved by #2001 / #2002

names.register() with v4.0.0-rc.2 worked for me.


Geometric mean for conf

Pull Review Summary
Description
A summary of the change. Please also include relevant motivation and context. This could include links to any docs/Slack threads/Github issues other artifacts.
Type of change

Bug fix (change which fixes an issue)
New feature (change which adds functionality)
This change requires a documentation update

Tests
Please describe the tests that you ran to verify your changes. This could include a test plan you executed locally, unit tests/integration tests that were run to verify the change works as expected.
Make sure to include screenshots, API response, log statements etc that point to the test being successful.
Put closes #XXXX in your comment to auto-close the issue that your PR addresses.

np.geometric 💯 !


Doc Feedback - Doesn't show how to get Dev Tools to appear in the menu

Type of issue
Other (describe below)
Feedback
Hi,
We have received this as a Verbatim feedback, where the user isn't able to view the Dev Tools option as specified in the article. We tested and observed the same issue faced by the user.
Can you please check from your end and help us with a resolution or fix required here?
Thank you!
Page URL
https://learn.microsoft.com/en-us/microsoftteams/platform/tabs/how-to/developer-tools
Content source URL
https://github.com/MicrosoftDocs/msteams-docs/blob/main/msteams-platform/tabs/how-to/developer-tools.md
Author
@surbhigupta
Document Id
a5b5782f-7031-a989-5e42-dd3010e18893

@v-preethah - We checked this at our end in Teams desktop, and we cannot see Open dev tools option. Could you please confirm if you are also checking in Teams desktop only and are you also facing the same issue?

Hi @Meghana-MSFT
As specified already in the issue, I was facing the same issue as you have shared in the above screenshot.


Parsing and saving the outputs into a folder that doesn't exist

Contextually, I ran into an issue where I wanted to parse some Excalidraw files from a certain place somewhere/ and then save the outputs in a directory, let's say somewhere/else/.
# Save outputs in a directory that doesn't exist
excalidraw-cli somewhere/ somewhere/else/

Currently if the directory else/ doesn't exist, excalidraw-cli considers the parameter value as a file name, so it will save the output into a single file named somewhere/else.png, this actually makes sense when aware about it, but may confuse unaware people for hours.
I'm considering adding a warning in case the INPUT parameter value is about a directory, but the OUTPUT parameter value is a directory that doesn't exist. Or maybe throw an Error.

My 2 cents:

Currently if the directory else/ doesn't exist, excalidraw-cli considers the parameter value as a file name

If the string has no "extension" in it (it doesn't end with .png) then it's a folder and not a file.
I think that the CLI shouldn't treat somewhere/else/ or somewhere/else as a file, ever.
somewhere/else.png is a file, though

@lucasavila00 That's a good point. I should consider OUTPUT parameter as a directory entry by default, if there is such extension, then we expect a file as output.

Consider OUTPUT parameter as a directory entry by default, if there is such extension, then we expect a file as output.

@tommywalkie I'd go with this heuristic for sure.

This may temporarily disable custom output file name — but there is always room for a flag, like --outputFile 🤔

Currently what's the reasoning of providing an input folder and an output file at the same time?
Does the CLI concatenate all the folders .pngs into the same file?

what's the reasoning of providing an input folder and an output file at the same time?

This is why some testing still needs to be done. 😁
Implementing the mentionned heuristic should hopefully end this confusion.

Fixed as of 0.4.2 and https://github.com/tommywalkie/excalidraw-cli/commit/fdc00aaece43466aec430675d267533d9897d9fe !
Review and notable changes :

IMPORTANT ! excalidraw-cli my_diagram.excalidraw will now output my_diagram.png (no more using .excalidraw.png file extension !)
excalidraw-cli input some/dir/that/doesnt/exist will create the output directory if doesn't exist
excalidraw-cli folder some-file.png will stop the CLI and output an error ⬇

Cannot process the request.
You passed a folder (.\folder\) as input, and a single file (some-file.png) as output.`)


[Improvement] Add architecture documentation

#85   #95

May also provide a Chinese version

thanks for your time.  recently i am using https://github.com/void-main/fastertransformer_backend   infer llama model in triton server.
It would be excellent if there was a detailed architecture documentation describing the framework


Ensembl workflow

The rest of the repository is under MIT license: https://github.com/TGAC/tgac-galaxytools/blob/master/LICENSE , are you sure we need GPL license for just the workflow?

No we don't, but I had practice of putting GPL licence so did it, we can change it to MIT

OK, can you remove the LICENCE file then? The rest should be ready for merge.

Done...


Navigate to a different dashboard show the header bar

Hi,
I have setup the kiosk-mode on mine dashboard. It work well when I load the dashboard as startup page of Fully Kiosk Browser.
The issue appear when from a button I manage the navigate action to a different dashboard.
In this case the header bar appear at the top.
Here an example of mine button that recall the navigate action:
      - type: icon
        icon: mdi:cog-outline
        ########################################################################
        title: Settings
        ########################################################################
        style:
          top: 75%
          left: 15%
          color: rgb(255, 255, 255)
          opacity: 0.1
          padding: 0.8vw
          --mdc-icon-size: 4vw
        tap_action:
          action: navigate
          navigation_path: /lovelace-settings?kiosk

If after loaded the dashboard I refresh the browser the header will be hidden correctly.
Many thanks for your help
Alberto

That doesn't load kiosk-mode, kiosk-mode only loads on the first load. This could be fixable.

See https://stackoverflow.com/a/33668370/12243248 @matt8707

Right now it only works between views and not dashboards, which makes sense. But yes this should be fixed, help with this is appreciated.

Fixed in 1.2.0


Make Array-of-SArray op Number infer the right output Array type

For some reason, there's special rules using Base.promote_op to infer
the correct type for operations involving Number and arrays of other
types.  This means we need to overload promote_op or we'll get an Array
of Any in these cases.
See https://github.com/SimonDanisch/FixedSizeArrays.jl/issues/156 for a similar issue.

Changes Unknown when pulling 05be4fb04d63064f0f7e461279f7b02261eeffc4 on arrays-of-sarrays into ** on master**.

Changes Unknown when pulling 05be4fb04d63064f0f7e461279f7b02261eeffc4 on arrays-of-sarrays into ** on master**.

Great, thanks!


chore: Upgrade to lit 2.3.0

there is one more lit appearance under the peerDependency..
https://github.com/vaadin/bundles/blob/0816bf0de1ae1d8360e3a2717b99806bc1275696/package.json#L200
guess we need to keep them being synced ?


smtp->GetErrorText() not found

I'm getting compile error at this line
libraries/UPnP/AlarmService.cpp:281:75: error: 'class SmtpClient' has no member named 'GetErrorLine'
Serial.printf("Error text %s, line %d\n", smtp->GetErrorText(), smtp->GetErrorLine());
^
can you point me to which SmtpClient library are you using?

I used https://github.com/gregington/SMTPClient but I appear to have local changes.
I'll upload these shortly.
Apologies for the mess.

I managed to bypass this, but now facing compile error in many other libraries. can you upload your entire "libraries" folder. thanks.

I've just uploaded my current version of the smtpclient at https://github.com/dannybackx/arduino-smtpclient

Which errors ? I'd like to fix things for you and others.

I got an error at this line, something like 'name' is not defined within this scope.
I simply comment it out to make it work.
https://github.com/dannybackx/arduino-upnp/blob/master/libraries/UPnP/BMP180SensorService.cpp#L149
Now, I'm facing errors with Timezone library
http://take.ms/bNqk2

I just committed recent work in BMP180 which should fix the "name" problem but also cope with not having such a sensor.
I don't know what's causing the Timezone errors, the errors all refer to stuff defined in that library.
 Danny

On 06/26/2016 10:28 AM, Torin Nguyen wrote:

I got an error at this line, something like 'name' is not defined within this scope.
I simply comment it out to make it work.
https://github.com/dannybackx/arduino-upnp/blob/master/libraries/UPnP/BMP180SensorService.cpp#L149
Now, I'm facing errors with Timezone library
http://take.ms/bNqk2
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dannybackx/arduino-upnp/issues/1#issuecomment-228590307, or mute the thread https://github.com/notifications/unsubscribe/AJ3PleuHuBLy2qqM_i34hGZtxA8_wUZ9ks5qPjgzgaJpZM4I-P6m.

--
Danny Backx -<EMAIL_ADDRESS>- http://danny.backx.info

can you upload your Timezone library as well? I had to download it from the Internet and not sure if I use the correct library.

I see my mistake. I've merged two libraries in my local copy. Uploaded : https://github.com/dannybackx/timetimezone .

So I deleted Time & Timezone libraries and replaced with your library.
I got this error now http://take.ms/CKMJc

On 06/26/2016 11:59 AM, Torin Nguyen wrote:

So I deleted Time & Timezone libraries and replaced with your library.
I got this error now http://take.ms/CKMJc
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dannybackx/arduino-upnp/issues/1#issuecomment-228593358, or mute the thread https://github.com/notifications/unsubscribe/AJ3PlRjBsEVITbcrK6DtSMPilYPeEKGMks5qPk18gaJpZM4I-P6m.

In my environment, that's in
.arduino15/packages/esp8266/tools/xtensa-lx106-elf-gcc/1.20.0-26-gb404fb9-2/xtensa-lx106-elf/include/time.h
Do you  have something similar ?
 Danny

--
Danny Backx -<EMAIL_ADDRESS>- http://danny.backx.info


Add provider errors

Adds a system for providers to send RPC error responses.

Auto-generates Provider.<method>Error RPC methods for all Providers
Automatically creates a basic error example, so that not all x-error schemas need one
Inserts a temporary schema for the error (should be replaced w/ a link to official OpenRPC schema, but that depends on fix/rpc.discover so we'll do it after that is merged
catches any errors when calling provider's in JS  SDK
calls the RPC handler for errors when one is caught

:tada: This PR is included in version 1.4.1-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.6.2-next.1 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Search is taking to steemit.com

Expected behavior

IMHO the search results should displayed within the busy.org interface even if we are using Google Custom Search.

The search results should take us to the corresponding busy.org URL as opposed to steemit.com URL

Actual behavior

Right now we are taken to a new tab
The new tab shows google search results
The links are pointing to steemit.com  (this can be fixed by site:busy.org or with a google custom search engine ?)

https://www.screencast.com/t/XTPk57pby

How to reproduce

go to busy.org
input a keyword into the search box and search

Browser:  Firfox Nightly 58.0a1 (2017-11-11)  and Firefox 57 Beta
Operating system:  macOS
busy.org is running in a Firefox container for steem & Google is search in a container for Google search

closing this as the bug is posted via https://utopian.io now.


Latest rapi-doc not rendering spec with oauth2 security scheme

Looks like the latest versions of rapi-doc have broken the rendering of a few of our API docs.  The ones that are no longer rendering have an oauth2 security scheme in common.
Forcing rapi doc back down to 9.1.4 resolves the issue for the time being.
Here is the error from the browser console:
Uncaught (in promise) TypeError: Cannot destructure property 'authorizationUrl' of 'a.authorizationUrl' as it is undefined.     at WR.zP (rapidoc-min.js:2:334547)     at rapidoc-min.js:2:341670     at Array.map (<anonymous>)     at rapidoc-min.js:2:341659     at Array.map (<anonymous>)     at WR.UP (rapidoc-min.js:2:339292)     at WR.VR (rapidoc-min.js:2:475614)     at WR.render (rapidoc-min.js:2:488782)     at WR.update (rapidoc-min.js:2:17083)     at WR.performUpdate (rapidoc-min.js:2:14034) zP @ rapidoc-min.js:2 (anonymous) @ rapidoc-min.js:2 (anonymous) @ rapidoc-min.js:2 UP @ rapidoc-min.js:2 VR @ rapidoc-min.js:2 render @ rapidoc-min.js:2 update @ rapidoc-min.js:2 performUpdate @ rapidoc-min.js:2 _enqueueUpdate @ rapidoc-min.js:2 await in _enqueueUpdate (async) loadSpec @ rapidoc-min.js:2 await in loadSpec (async) (anonymous) @ rapidoc-min.js:2 setTimeout (async) attributeChangedCallback @ rapidoc-min.js:2 310 @ rapidoc-min.js:2 a @ rapidoc-min.js:2 (anonymous) @ rapidoc-min.js:2 (anonymous) @ rapidoc-min.js:2

duplicate of #667


Nix shell to run runtime dependencies

Prereview checklist

[ ] The CHANGELOG.md has been updated under the # Unreleased header using the appropriate sub-headings with links to the appropriate issues/PRs.
[ ] All tests pass in CI
[ ] PR was self-reviewed

Tool already exists in partner-chains repo


50+ mAP？wow，that is so high. No idea, maybe something went wrong, or it is the benefit of better object detections. Maybe you can convert the ican's detections to your format, and use the same model to verify the 10+% difference.

50+ mAP？wow，that is so high. No idea, maybe something went wrong, or it is the benefit of better object detections. Maybe you can convert the ican's detections to your format, and use the same model to verify the 10+% difference.

I just made wrong about the evading code,i rewrites it by accuracy method,i then eval it on the val data, it can up to 43 on aprole,, i just read some papers, it's different from each other, do you eval on the val data or on the test data?
Originally posted by @ZHUXUHAN in https://github.com/DirtyHarryLYL/Transferable-Interactiveness-Network/issues/3#issuecomment-486056774

We evaluate on the V-COCO test set.


Auto file extension lookup with multiple view engines

When there is no file extension defined, express will now try to use any extension that is registered with an engine. The default extension set by app.set('view engine') is still observed first.
Related issue: #3485

This seems like a good addition to me.  I would like a few more people opinion before we decide what to do with this for sure.  But in general it has my 👍

Two other notes on both of your open PR's:

Make them against the 5.x branch, not 5.0.
The formatting should follow standardjs

@dougwilson I just went to investigate, and I was sure that the branch was 5.x, but the PR is open from 5.0.  What is your plan with those?  I feel like most of the open PR's are against 5.x, but I could be wrong.

So for something destined for the 5.0 alpha the 5.0 branch is right, otherwise 5.x (5.x is the equivalent of master branch for major version 5). But even then, this seems like an enhancement that could do into 4.x, no?

Yeah, this should be able to land in 4 as well.  I only noticed this was open against 5.0, so I brought it up.

In general as long as the commits are easy to move we can move them to the correct destination upon a merge. Since the view file is identical for 4 and 5 we can always merge to 4 even if the PR is against 5.

Thanks guys, so I'll leave this PR on 5.0.
As for the formatting are you referring to the no-semicolon and space before parentheses and opening brace rules? (More notably in the test file, which I have followed the current formatting)


Can't hit git branches extensibility points

Hi,
I tried to add a sample to this project to hit some supposedly existing extensibility points, according to the documentation. It's a basic copy / paste from the Menu sample, and looks like this:
Actions/Actions.html
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml">
    <body>
        <script type="text/javascript" src="Actions.js" charset="utf-8"></script>
    </body>
</html>

Actions/Actions.ts
import "es6-promise/auto";
import * as SDK from "azure-devops-extension-sdk";
import { CommonServiceIds, IHostDialogService  } from "azure-devops-extension-api";

SDK.register('sample-action', () => {
    return {
        execute: async(context: any) => {
            const dialogSvc = await SDK.getService<IHostDialogService>(CommonServiceIds.HostDialogService);
            dialogSvc.openMessageDialog(`Context: ${JSON.stringify(context)}`, { showCancel:false });
        }
    }
});

SDK.init();

and finally Actions/Actions.json
{
    "contributions": [
        {
            "id": "sample-action",
            "type": "ms.vss-web.action",
            "targets": [
                "ms.vss-admin-web.collection-overview-toolbar-menu",
                "ms.vss-admin-web.projects-grid-menu",
                "ms.vss-admin-web.project-overview-toolbar-menu",
                "ms.vss-admin-web.teams-grid-menu",
                "ms.vss-build-web.completed-build-menu",
                "ms.vss-build-web.build-definition-menu",
                "ms.vss-releaseManagement-web.release-definition-explorer-context-menu",
                "ms.vss-releaseManagement-web.release-definition-explorer-toolbar-menu",
                "ms.vss-releaseManagement-web.release-editor-tool-bar-menu",
                "ms.vss-code-web.source-grid-item-menu",
                "ms.vss-code-web.source-tree-item-menu",
                "ms.vss-code-web.source-item-menu",
                "ms.vss-code-web.git-branches-tree-menu",
                "ms.vss-code-web.git-branches-summary-grid-menu",
                "ms.vss-code-web.git-branches-summary-grid-diff-menu",
                "ms.vss-code-web.pull-request-action-menu"
            ],
            "properties": {
                "text": "Sample action",
                "uri": "dist/Actions/Actions.html",
                "icon": "static/sample-icon.png",
                "registeredObjectId": "sample-action"
            }
        }
    ]
}

Some of these targets are still working (build, release, source), others don't. And I don't understand why the git related ones are not responding (ms.vss-code-web.git-branches-tree-menu, ms.vss-code-web.git-branches-summary-grid-menu and ms.vss-code-web.git-branches-summary-grid-diff-menu).
I checked all possible actions in the following pages without any luck. I even switched to the old UI to be sure:

https://dev.azure.com/MY_ORG/_git/MY_PROJECT/branches
https://dev.azure.com/MY_ORG/MY_PROJECT/_settings/repositories

Did I miss something? Are these extensibility points still valid?
Thanks

Thanks for bringing this to our attention. The "ms.vss-code-web.git-branches-tree-menu" contribution should exist in the branches page, but unfortunately looks like it was removed when the other two contributions were removed as part of a page redesign. The documentation is out of date and we will fix that as well. Sorry for the inconvenience. Expect us to ship the contribution in the branches tree point soon.


Repo sync for protected CLA branch

The pull request is created from master637429610208097272 to master to fix git push error for protected CLA branch

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.


opencv4: cuda: fix libstdc++ errors downstream, plus upkeep

Description of changes
Closes https://github.com/NixOS/nixpkgs/issues/258327
Adds a test for and fixes the following libstdc++ mismatch error:
python3.11-pytlsd> ==================================== ERRORS ====================================
python3.11-pytlsd> _______________________ ERROR collecting tests/tests.py ________________________
python3.11-pytlsd> ImportError while importing test module '/build/source/tests/tests.py'.
python3.11-pytlsd> Hint: make sure your test modules/packages have valid Python names.
python3.11-pytlsd> Traceback:
python3.11-pytlsd> /nix/store/ffll6glz3gwx342z0ch8wx30p5cnqz1z-python3-3.11.5/lib/python3.11/importlib/__init__.py:126: in import_module
python3.11-pytlsd>     return _bootstrap._gcd_import(name[level:], package, level)
python3.11-pytlsd> tests/tests.py:7: in <module>
python3.11-pytlsd>     from skimage.transform import pyramid_reduce
python3.11-pytlsd> /nix/store/zr2pijvixax0say6n6d1s0mg3f970xv2-python3.11-scikit-image-0.21.0/lib/python3.11/site-packages/skimage/transform/__init__.py:38: in <module>
python3.11-pytlsd>     from .radon_transform import (radon, iradon, iradon_sart,
python3.11-pytlsd> /nix/store/zr2pijvixax0say6n6d1s0mg3f970xv2-python3.11-scikit-image-0.21.0/lib/python3.11/site-packages/skimage/transform/radon_transform.py:3: in <module>
python3.11-pytlsd>     from scipy.interpolate import interp1d
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/interpolate/__init__.py:167: in <module>
python3.11-pytlsd>     from ._interpolate import *
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/interpolate/_interpolate.py:12: in <module>
python3.11-pytlsd>     from . import _fitpack_py
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/interpolate/_fitpack_py.py:10: in <module>
python3.11-pytlsd>     from ._bsplines import BSpline
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/interpolate/_bsplines.py:9: in <module>
python3.11-pytlsd>     from scipy.optimize import minimize_scalar
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/optimize/__init__.py:422: in <module>
python3.11-pytlsd>     from ._linprog import linprog, linprog_verbose_callback
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/optimize/_linprog.py:21: in <module>
python3.11-pytlsd>     from ._linprog_highs import _linprog_highs
python3.11-pytlsd> /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/optimize/_linprog_highs.py:20: in <module>
python3.11-pytlsd>     from ._highs._highs_wrapper import _highs_wrapper
python3.11-pytlsd> E   ImportError: /nix/store/n0gf2wl3g6229q275r4cxs2yhnam0xvg-gcc-11.4.0-lib/lib/libstdc++.so.6: version `GLIBCXX_3.4.30' not found (required by /nix/store/nab40hl5f2hk4i1x5rlkimmn20dyqh40-python3.11-scipy-1.11.3/lib/python3.11/site-packages/scipy/optimize/_highs/_highs_wrapper.cpython-311-x86_64-linux-gnu.so)
python3.11-pytlsd> =========================== short test summary info ============================
python3.11-pytlsd> ERROR tests/tests.py
python3.11-pytlsd> !!!!!!!!!!!!!!!!!!!! Interrupted: 1 error during collection !!!!!!!!!!!!!!!!!!!!
python3.11-pytlsd> =============================== 1 error in 0.28s ===============================
python3.11-pytlsd> /nix/store/bbxdw4rgwwl3gnajri82yidr1nlsfskf-stdenv-linux/setup: line 1596: pop_var_context: head of shell_variables not a function context
error: builder for '/nix/store/jfpbrdgvfr2qbac02lml42p1kvh4p1nf-python3.11-pytlsd-0.0.5.drv' failed with exit code 2;

...observed when importing cv2 (built with cuda support and the older nvcc-compatible gcc) before importing a native python extension from nixpkgs (which uses the newer libstdc++) like so: https://github.com/iago-suarez/pytlsd/blob/81e0fee35f1fdaf546ec73c71ad3ed488c8ee83b/tests/tests.py#L3-L7
The test passes for the opencv3 attribute because it's just built with the newer gcc(12). I decided not to touch it further, but my expectation is that opencv3 with CUDA will start failing once we bump gcc.
The tests can be run as follows:
❯ nix build -f . --arg config '{ cudaSupport = true; cudaCapabilities = [ "8.6" ]; enableForwardCompat = false; allowUnfree = true; }' opencv4.tests.no-libstdcxx-errors
❯ nix build -f . --arg config '{ cudaSupport = true; cudaCapabilities = [ "8.6" ]; enableForwardCompat = false; allowUnfree = true; }' opencv3.tests.no-libstdcxx-errors

You can check out the first commit and verify that opencv4.no-libstdcxx-errors fails until it's updated in the next commit
In the longer run we should kill backendStdenv and instead implement https://github.com/NixOS/nixpkgs/issues/226165

Things done

Built on platform(s)

[x] 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/)
23.11 Release Notes (or backporting 23.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.

@NixOS/cuda-maintainers

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1207


fix(it/animeunity): fix video extraction

Code stolen from streamingcommunity
Closes #1808
Checklist:

[x] Updated extVersionCode value in build.gradle for individual extensions
[x] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[x] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[x] Added the isNsfw = true flag in build.gradle when appropriate
[x] Have not changed source names
[x] Have explicitly kept the id if a source's name or language were changed
[x] Have tested the modifications by compiling and running the extension through Android Studio

Hi the extension is still not working :(

Create a new issue instead of commenting on old pull requests


Update lokalize workflow

This improves the Lokalise workflow.

Got it. If you won't be using it in the future you could delete it.


climatisationSettings not working, responding Bad Gateway

After climatisation-Settings by
/wec/vehicles/.../domains/climatisation/climatisationSettings/targetTemperature_C_writetopic 22.0
/wec/vehicles/.../domains/climatisation/climatisationSettings/climatisationWithoutExternalPower_writetopic True
... WeConnect-mqtt is responding with ...
error/code 2105
error/message Bad Gateway
error/group 2
error/info Upstream service responded with an unexpected status. If the problem persists, please contact our support.
after some time is see
/wec/mqtt/error/code -10
/wec/mqtt/error/message Error setting value: Could not set value (Bad Gateway - Upstream service responded with an unexpected status. If the problem persists, please contact our support. - Please retry in a moment)
set climatisation via VW-weconnect-App is working at same time.
Starting climatisation with /wec/vehicles/.../controls/climatisation_writetopic start mostly not starting climatisation

same as in #134.
though i did not yet see „starting climatisation“ fail, but haven’t tested it since i saw the other climatisation related errors.

mostly there is not any "reaction" on mqtt, when setting climatisation-related topics.

just tried turning climatisation …/climatisation_writetopic start  and it worked right away. can’t test any further till i load the battery again, but i got a „in_progress“ status and after a mnute or two climatisation status turned on.

Afternoon I went shopping and tried again to Start climatisation via my mqtt-Client. First attemp was successfull. Some time later at next stop it didn't. On mobile phone i was not able to monitor my mqtt.

There is a problem with climatisation settings right now. Looks like they changed the behavior on the server slightly. Start/Stop should still work.

Can you check with the version 0.37.2 if you can set the climatisation settings again? Everything except for climatisationWithoutExternalPower should work again. I think maybe climatisationWithoutExternalPower is not routed correctly because it is a setting not available for MEB cars.

thanks, it’s working again. after i triggered a toggle to climatizationAtUnlocki first got a status in_progress followed by the actual toggle and status success. toggeling back also created no error.

Great! Thanks for reporting and testing!


Flops Profiler breaks after running

I am able to profile my model by adding the following to my config:
  "flops_profiler": {
    "enabled": true,
    "start_step": 5,
    "end_step": 10,
    "module_depth": -1,
    "top_modules": 3
  }

It returns the profile result, however it seems to break straight after.
Traceback:
  File "pretrain_gpt2.py", line 156, in <module>
    pretrain(train_valid_test_datasets_provider, model_provider, forward_step,
  File "/root/megatron-3d/megatron/training.py", line 97, in pretrain
    iteration = train(forward_step_func,
  File "/root/megatron-3d/megatron/training.py", line 481, in train
    loss_dict, skipped_iter = train_step(forward_step_func,
  File "/root/megatron-3d/megatron/training.py", line 324, in train_step
    return train_step_pipe(model, data_iterator)
  File "/root/megatron-3d/megatron/training.py", line 358, in train_step_pipe
    loss = model.train_batch(data_iter=data_iterator)
  File "/root/anaconda3/lib/python3.8/site-packages/deepspeed/runtime/pipe/engine.py", line 283, in train_batch
    self._exec_schedule(sched)
  File "/root/anaconda3/lib/python3.8/site-packages/deepspeed/runtime/pipe/engine.py", line 1172, in _exec_schedule
    self._exec_instr(**cmd.kwargs)
  File "/root/anaconda3/lib/python3.8/site-packages/deepspeed/runtime/pipe/engine.py", line 521, in _exec_forward_pass
    outputs = super().forward(inputs)
  File "/root/anaconda3/lib/python3.8/site-packages/deepspeed/runtime/engine.py", line 824, in forward
    loss = self.module(*inputs, **kwargs)
  File "/root/anaconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 719, in _call_impl
    result = hook(self, input)
  File "/root/anaconda3/lib/python3.8/site-packages/deepspeed/profiling/flops_profiler/profiler.py", line 49, in pre_hook
    module.__steps__ += 1
  File "/root/anaconda3/lib/python3.8/site-packages/torch/nn/modules/module.py", line 778, in __getattr__
    raise ModuleAttributeError("'{}' object has no attribute '{}'".format(
torch.nn.modules.module.ModuleAttributeError: 'GPT2ModelPipe' object has no attribute '__steps__'

Can you please try out this PR https://github.com/microsoft/DeepSpeed/pull/682 to see if the issue is fixed? If not. Could you please share your deepspeed config file?

I had the same issue. And the PR works for me

Can you please try out this PR #682 to see if the issue is fixed? If not. Could you please share your deepspeed config file?

Thanks for trying the PR. Profile across multiple steps is now changed to a single step. It's now merged into the master.


Self-hosted CSS is reloaded in endless loop starting with version 5.5.0

I am running the Business Text plugin on Grafana 11.3.0 and I configured some panels to load external CSS from a local resource.
It worked perfectly until 5.4.0, but since version 5.5.0, I noticed an endless number of resource requests to the same file, consuming an enormous amount of browser CPU time.
My resource is responding HTTP 304 Not Modified, but the plugin keeps requesting the resource.
Downgrading to 5.4.0 immediately solves the problem.
Response headers:

Using a CDN instead of a local resource also solves the problem, which I cannot explain. Maybe my resource is not providing the correct headers that were previously unused by the plugin?

@BarryLuijten Thank you for letting us know.
We will try to reproduce the issue and get back to you.


Installing on OSX fails

I'm not sure if this is still maintained as its a little old ...
Install on OSX doesn't work - either with npm or yarn
I'll post full log in first comment.
Note - that I did install libvips with brew install vips which succeeded.
Something else i missed ?

mitraglass:iiif-image-server-node mitra$ npm i
npm WARN deprecated<EMAIL_ADDRESS>CoffeeScript on NPM has moved to "coffeescript" (no hyphen)
npm WARN deprecated<EMAIL_ADDRESS>Please note that v5.0.1+ of superagent removes User-Agent header by default, therefore you may need to add it yourself (e.g. GitHub blocks requests without a User-Agent header).  This notice will go away with v5.0.2+ once it is released.

><EMAIL_ADDRESS>install /Users/mitra/git/iiif-image-server-node/node_modules/dtrace-provider
> node-gyp rebuild || node suppress-error.js

  ACTION binding_gyp_ndtp_target_build_ndtp .
starting .bashrc
  TOUCH Release/obj.target/ndtp.stamp

><EMAIL_ADDRESS>install /Users/mitra/git/iiif-image-server-node/node_modules/fsevents
> node-gyp rebuild

  SOLINK_MODULE(target) Release/.node
  CXX(target) Release/obj.target/fse/fsevents.o
  SOLINK_MODULE(target) Release/fse.node

><EMAIL_ADDRESS>install /Users/mitra/git/iiif-image-server-node/node_modules/sharp
> node-gyp rebuild

/bin/sh: starting: command not found
gyp: Call to 'which brew >/dev/null 2>&1 && eval $(brew --env) && echo $PKG_CONFIG_LIBDIR || true' returned exit status 0 while in binding.gyp. while trying to load binding.gyp
gyp ERR! configure error
gyp ERR! stack Error: `gyp` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/usr/local/lib/node_modules/npm/node_modules/node-gyp/lib/configure.js:351:16)
gyp ERR! stack     at ChildProcess.emit (events.js:210:5)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:272:12)
gyp ERR! System Darwin 18.7.0
gyp ERR! command "/usr/local/bin/node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/mitra/git/iiif-image-server-node/node_modules/sharp
gyp ERR! node -v v12.11.0
gyp ERR! node-gyp -v v5.0.5
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/mitra/.npm/_logs/2019-12-17T08_44_33_595Z-debug.log

Obviously this is a problem in your dependency, but probably something you want to know about.

Just investigating .... sharp installs fine by itself mkdir foo; cd foo; npm install sharp; so it appears to be something to do with the dependencies etc, in how iiif-image-server-node is using it.


Poor contrast in dark theme when members are highlighted

Following a link, for example: https://mapdl.docs.pyansys.com/release/0.64/api/_autosummary/ansys.mapdl.core.inline_functions.Query.node.html#ansys.mapdl.core.inline_functions.Query.node
Renders the type, and highlights the member referenced. In dark mode this has very poor contrast:

This seems to be fixed in the mapdl dev docs, I assume it's fixed in a recent release of the theme.


create mouse capture example

What issue is this addressing?
Example used for #2634
What type of issue is this addressing?
bug
What this PR does | solves
Provides an example demo for mouse capture

I'm not sure the purpose of this PR. This PR does not fix the bug, right?

Let me close this as #2634 was resolved.

Sorry it was orignally to provide an example for that bug, so I've opened instead as feature enhancement now: #2636


Lot listings

Currently a work-in-progress. Needs more work, but the main idea of the implementation is included. Here's what we're currently showing:

Doesn't look like much, but all the architecture work is done. Just need to add autolayout constraints.
Most of #972.

2 Errors

:no_entry_sign:
Developer Specific file shouldn't be changed

:no_entry_sign:
There were snapshot errors

2 Warnings

:warning:
PR is classed as Work in Progress

:warning:
Needs testing on a Phone if change is non-trivial

  Generated by :no_entry_sign: danger

Looking into the failing snapshots.
Current status:

OK, so https://github.com/artsy/eigen/commit/5d809e8412fae3b04b9278e15d57ff957147883d got the views rendering properly, which is good! But we don't have the sticky header working currently, either in tests or at runtime:

My theory is that only the masonry layout "knows" about these sticky headers. Instead of writing a new collection view layout, I'm just going to configure a masonry layout where each cell is the full-width. /cc @orta

definitely only the masonry knows about the headers!

:+1:

:green_apple:

:shaved_ice:


The real purpose of nocode

If we think of a machine that does not do nothing, it is quite easy. It just doesn't do nothing. But a machine that does nothing is much, much harder concept. It is a machine that actually does something in order to transform everything to nothing. This was the problem Trurl and Klapaucius found out that if you create a machine to does nothing, it will start to transform everything to nothingness. (you can find more info in Stanislaw Lem's The Cyberiad)
So is the purpose of nocode just to do nothing or to turn all code to nothing?
I'm hoping for latter.

Thank you for the reference, we all love good ol' uncle Lem.
Regarding your sharp observation, IMHO it seems in fact that nocode -must- do nothing, and that turning everything around it into nothingness is a side effect. It is still unclear though if this side effect is of the desirable or of the undesirable kind.
An alternative approach is for nocode to just -not do-, thus avoiding the collateral voiding of Reality that occurs when simply doing nothing. This of course causes troubles in our coding pipeline when handling double negations, as we should not just "not do anything", but, again, just "do nothing". We are working on these issues by not working on them, and will keep you posted with any outcomes we encounter, -or not-.


autocorrect and enableSuggestions not working

This still shows suggestions in both ios and android.
TextField(
autocorrect: false,
enableSuggestions: false,
)
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.3.1 21E258 darwin-arm, locale en-GR)
• Flutter version 3.0.5 at /Users/nikostsesmelis/Tools/flutter/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision f1875d570e (2 days ago), 2022-07-13 11:24:16 -0700
• Engine revision e85ea0e79c
• Dart version 2.17.6
• DevTools version 2.12.2

Hi @NikosTsesmelis, I can reproduce this issue on the latest stable and master channel.

Demo

Android
iOS

Sample code
import 'package:flutter/material.dart';

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

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

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

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
            const Text(
              'You have pushed the button this many times: ',
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headline4,
            ),
            const TextField(
							autocorrect: false,
							enableSuggestions: false,
						),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 3.0.5, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.0.5 at /Users/huynq/Documents/GitHub/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (9 hours ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (3 available)
    • SM T225 (mobile) • R9JT3004VRJ • android-arm64  • Android 11 (API 30)
    • macOS (desktop)  • macos       • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)     • chrome      • web-javascript • Google Chrome 103.0.5060.114

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.1667, on macOS 12.2.1 21D62 darwin-x64, locale en-VN)
    • Flutter version 3.1.0-0.0.pre.1667 on channel master at /Users/huynq/Documents/GitHub/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 0cf9d41fc9 (2 hours ago), 2022-07-15 07:21:11 +0700
    • Engine revision 64f98357ca
    • Dart version 2.18.0 (build 2.18.0-283.0.dev)
    • DevTools version 2.15.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/huynq/Library/Android/sdk
    • Platform android-33, build-tools 31.0.0
    • ANDROID_HOME = /Users/huynq/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 13E113
    • CocoaPods version 1.11.3

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

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Community Edition (version 2020.3.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Community Edition (version 2022.1.1)
    • IntelliJ at /Users/huynq/Library/Application Support/JetBrains/Toolbox/apps/IDEA-C/ch-0/221.5591.52/IntelliJ IDEA CE.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

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

[✓] Connected device (4 available)
    • Pixel 3a (mobile)  • 964AY0WL20                           • android-arm64  • Android 12 (API 32)
    • iPhone 13 (mobile) • 2526BC1A-435D-4B08-B99C-44B928F2517B • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-4 (simulator)
    • macOS (desktop)    • macos                                • darwin-x64     • macOS 12.2.1 21D62 darwin-x64
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 103.0.5060.114

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

But I see there's an open issue addressing the case you described #71679 but that issue is only reported for Android. Could you please add a comment on that issue to update that the issue also occurs with iOS? Then I will close this issue as a duplicate.
Thanks!

I did but this issue that you mentioned is 1.5 years old and doesn't seem likely to be fixed

@NikosTsesmelis please give the detail in the comment and react an upvote to the original comment will help to raise the issue. Closing this issue as a duplicate. I will also post my comment and add relevant label for iOS.
Thanks!


BUG: filtering on non-extisting field 'name' breaks subdataset filtering

https://github.com/psychoinformatics-de/sfb1451-projects-catalog/issues/10

This has not been fixed yet and should be done ASAP.


fix: build missing in 3.0.0 - bump to 3.0.1

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 remained the same at 91.319%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
151

Relevant Lines:
162

💛  - Coveralls

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 remained the same at 91.319%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
151

Relevant Lines:
162

💛  - Coveralls

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 remained the same at 91.319%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
151

Relevant Lines:
162

💛  - Coveralls


Focusing on the animation.  Shape file cells removed.

Proposed changes
Include a brief description of the changes being proposed, and why they are necessary.
Closes issues (optional)

Closes Issue #000
Closes Issue #000

Checklist (replace [ ] with [x] to check off)

[ ] Remove any unused Python packages from Load packages
[ ] Remove any unused/empty code cells
[ ] Remove any guidance cells (e.g. General advice)
[ ] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[ ] Include relevant tags in the final notebook cell and re-use tags if possible
[ ] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated

There's a mention of Digital Earth Australia here:

we can take a time series of Digital Earth Australia satellite imagery

Thanks Rob,
I've removed a few Australian references - and two files I didn't need to check in


test: e2e migrate tests for ember apps

e2e migrate tests for ember apps

There are some highlighted strings in the snapshots. I am trying to figure out why.

Fixed

The PR failed one test which was passing previously. Making changes for debugging purposes.

Comments have been addressed.

@lynchbomb Could you re-review this PR?


replaced all console.log(err) with console.error(err), added most imp…

replaced all console.log(err) with console.error(err), added most important default .prettierrc.json settings as well as .editorconfig

Hi, these look like good additions. The project is moving fast today. Can you pull in the latest changes and resubmit? Also, please submit an issue for the changes you are requesting. Thanks for contributing!!


got a strange IP address shown and not sure how to connect

When I opened the app on the vision pro it says my IP address is
2603:8000:8f00:baf6::a45
Any idea why that might happen? It kind of looks like ipv6 but it is missing values

Nevermind, after changing wifi/closing and opening a few times I got a working one!


Key type

Is it intentional that it has to be string? It's not mentioned in the docs but couldn't get the cache to work with an integer key.

^bump

Hi, internally the key is used as an object key.
The data object will look like:
data = {
  "yourkey": {
    "v": "the cached value",
    "t": 0 // the ttl
  }
}

This is the reason why a integer is not possible.
I created a separate Issue to validate the key's type.
Also see this code position
I hope this helps ;-)

Will be fixed by #48


Update dexterity_reference.rst: wording datagrid field

LGTM. While at it you could convert to one sentence per line. But just as an idea, not as an todo.


Added CODEOWNERS and MAINTAINERS files

Signed-off-by: Toddy Mladenov<EMAIL_ADDRESS>This PR combines all the changes from the following PRs for updating the maintainers for notaryproject/notaryproject sub-project:
Add Milind Gokarn #228
Add Pritesh Bandi #227
Add Shiwei Zhang #234
Add Toddy Mladenov #236
Add Vani Rao #229
Add Yi Zha #237
To keep a balance I dropped Add Feynman Zhou #235
The proposal is to abandon the above PRs and merge only this one.

Linking the issue #224

LGTM but IANAM

LGTM but IANAM

Pinging @justincormack for a unanimous vote of the org maintainers.


different keyword but get same description response

Hi all, I have come across a problem and trying to seek your help. I am going to fine-tune parler-tts in Chinese, use Chinese dataset and choose chatglm-6b as the generating model for text description. The prompt format I am using basically corresponds to the English version. But when the annotation of the keywords are different, all my wav files get the same LM output. this has been bothering me for a long time, and I would like to know how to solve it.
（also I've checked that the keywords are all replaced correctly,and my keywords annotation are also chinese.)

Hey @SongYao2 , have you been able to solve this ? If not, what LLM do you use?

Hi, I'm surprised you noticed and answered my question. 😄🎉🎉 Actually I chose chatglm-6b in order to generate Chinese descriptions with more diverse syntax. And chatglm-6b is proposed by a team in China and supports both English and Chinese.
In the end, I didn't dwell on the data construction part, I specified a fixed description syntax, replacing keyword tags to the specified position. I used such a text description to train parler-tts from scratch with a 1000-hour Chinese dataset. the generated tones were very similar.
I did not perform text-controlled comparison experiments due to misreading and omission of words in the generated speech.

Thanks for the details!

I did not perform text-controlled comparison experiments due to misreading and omission of words in the generated speech.

I've realized that the tokenizer we used doesn't use byte fallbacks, which means that some tokens might be omitted. Expect a new release soon


feat: ContextProvider

TODO: add docs

I imagine we want to make this an entirely separate thing from Context for code-splitting/reducing unnecessary things on the instance?
Feels like this would be pretty tight:
import { Context } from "runed";

export const MenuContext = new Context<{ open: boolean }>('Menu')

<script lang="ts">
	import { MenuContext } from "./context.js"
</script>

<MenuContext.Provider value={{ open: true }}>
	<!-- things... -->
</MenuContext.Provider>


Back Button Wrong Behavior When PageHeader in UserControl

I have three pages (Page1, Page2, Page3). I can navigate from Page1 to Page2 to Page3. With the backbutton I can then navigate back in the stack.
Page2 and Page3 contain a UserControl. The UserControl contains the PageHeader. I bind the Frame from the Page to the UserControl (custom dependency property) to the PageHeader.
When I run the app on a desktop (local machine) I see the following behavior:

Page1
NavigateTo: Page2
NavigateTo: Page3
BackButton: Page2 (correct)
BackButton: Page1 (correct)
NavigateTo: Page2
NavigateTo: Page3
BackButton: Page1 (not correct, should have been Page2)

Same behavior occurs if I omit step 5 and go to Page3 directly. Then the BackButton on Page3 leads me to Page1.
If I run the same app on the mobile emulator, the behavior is as intended.
If I remove the user controls, and put the PageHeaders on the Page directly, also the behavior is as intended.
Any idea what is going wrong here?
Thanks in advance,
Franck

No way. Really?


Remove mentions of environment-dev.yml

Reference Issues or PRs
Fixes #307

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[x] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[ ] Other (please describe):

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Documentation

Access-centered content checklist
Text styling

[ ] The content is written with plain language (where relevant).
[ ] If there are headers, they use the proper header tags (with only one level-one header: H1 or # in markdown).
[ ] All links describe where they link to (for example, check the Nebari website).
[ ] This content adheres to the Nebari style guides.

Non-text content

[ ] All content is represented as text (for example, images need alt text, and videos need captions or descriptive transcripts).
[ ] If there are emojis, there are not more than three in a row.
[ ] Don't use flashing GIFs or videos.
[ ] If the content were to be read as plain text, it still makes sense, and no information is missing.

Any other comments?

Self-merging as a one-off because we're participating in the mentored sprints at PyCon, and it'll be nice to have these docs up-to-date. :)


Add default manufacturer file if missing

I didn't know I needed to add a manufacturer.json to ledworks. This will create such file in the action.

Thanks for implementing @brianegge, I think this will work, as the added file in git will be commited by github action in the workflow.
https://github.com/bramstroker/homeassistant-powercalc/blob/master/.github/workflows/update-profile-library.yml#L35
The cloudflare issue is logical, as that api token is only set as secret in my repo.


linux kernel 4.3 is going to break weave with fastdp

See https://github.com/weaveworks/go-odp/pull/4
We'll need to fix this in go-odp, and then be careful to do future releases with an updated go-odp (go's feeble dependency management strikes).
4.3 isn't out yet, but will be soon.  I guess we'll need a 1.2.1 for this if for no other reason (though #1593 looks like it might be another reason).

Fixed by https://github.com/weaveworks/go-odp/commit/8f0597465f607f2b0945029a8f47baccc183571e


Offset Without Limit Causes SQL Error

The following code:
// Get all but the latest: latest() not strictly needed it just demonstrates a use case.
$model->latest()->offset(1)->get();

// Or for those googling the same bug:
$model->latest()->skip(1)->get();

produces this SQL statement:
SELECT * FROM `models` ORDER BY `created_at` DESC OFFSET 1

This statement will generate an error in that the offset should only be used in combination with LIMIT such as LIMIT n OFFSET 1. The issue is that this query should return all results that remain after skipping the first record. There's no SQL equivalent of this so the internet suggests using the largest possible MySQL integer which exceeds the largest possible PHP integer for unsigned integers. So the solution to this right now is the resulting code:
// Solution to get around integer overflow errors
$model->latest()->limit(PHP_INT_MAX)->offset(1)->get();

// And for those googling the same bug:
$model->latest()->skip(1)->take(PHP_INT_MAX)->get();

Which produces a valid SQL statement like:
SELECT * FROM `models` ORDER BY `created_at` DESC LIMIT<PHONE_NUMBER>854775807 OFFSET 1

While this is a work around, it would be nice if Laravel handled this use case by making sure that the query builder checks the resulting statement when calling offset() to ensure that the statement always includes a LIMIT that is at least as large as PHP can handle. Another alternate solution would be to always default with the call to limit(PHP_INT_MAX) and then subsequent manual chaining of limit(n) would overwrite it similar to how select() works.
If you can point me in the right direction I'll do a pull request: I'm just not sure the best place to tackle it.

i have same issue  and this workaround works but is not right way i guess

Is this an issue we can revisit?


Read config.yaml at startup

Should contain:
secret: site's secret
plugins:
  - name: name of plugin #will be used to find on the file system
    config1: specific to plugin

Decided to use json just because its native and a nicely formatted json isn't hard to keep up


Consider how multiple services of the same product on one node can be configured

Currently we only cater to overriding config setting down to a specific node. In a scenario where multiple services of the same type run on the same node (example: Elasticsearch runs best with heap settings < 32gig, but the servers have 480 gig of ram, so multiple processes are started on the same node).
In that case we'd need to be able to specify different port numbers for every service - that all run on the same node.

We now have a solution for this using our role group concept