mu / Sigma (scale-independent) modified gravity support

This PR implements support for the scale-independent version of the mu / Sigma parameterisation of modifications to GR (for a definition of these functions see, for example, Simpson et al. 2013, the CFHTLenS modified gravity constraints paper).
At the moment this is not quite ready for review but it is close. Here is what is still missing:

benchmarks for growth factor, P(k), and Cl's (but test code of the C library is there waiting)
python tests
updates to existing example scripts and notebooks.

If you can contribute benchmarks please let me know (@TessaBaker perhaps?)

Pull Request Test Coverage Report for Build 1906

4 of 7 (57.14%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 94.097%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyccl/core.py
2
3
66.67%

pyccl/background.py
2
4
50.0%

Totals

Change from base Build 1869:
-0.5%

Covered Lines:
526

Relevant Lines:
559

💛  - Coveralls

@c-d-leonard  Just following up that I submitted comments for the review last week (just in case they fell under the stack) :)

@mishakb Thank you for the nudge! I will make it a priority to address these by early next week.

@mishakb Just to update you: so, I've merged master into this branch and got it working again. To address your above questions:

Good catch about CMB lensing. I am pretty sure I did at some point have the appropriate Sigma factors in the CMB lensing integral, however the cl code has been seriously altered since then and I think it's gotten lost. That said, I've just been staring at ccl_cls.c for more than an hour and I can't understand where the CMB lensing integral is calculated (except in the limber case where I can track it). Maybe @beckermr or @damonge could chime in pointing me to where this integral is specified for the non-limber case?
The number counts tracer is affected by the mg parameters in the following ways:

the power spectrum which feeds in to it is modified by mu0 (implemented in ccl_power.c)
the magnification effect is altered by Sigma, you can see this in the function clt_init_wM.
I tried to add a few comments to clarify this.

Yes, the growth rate should be implemented in the way you say following our previous discussions.

@c-d-leonard thanks for sending the update. I will keep an eye open on how the cmb lensing will be addressed.  For the growth, I will check if such a redshift/time "switch" is there. I don't recall coming across it in the code but will check again.

@mishakb re your comment on redshift / time dependency of growth - I think I totally misunderstood your question the first time. I thought you were asking about whether we normalized the growth rate to GR at early times, but now I think you were asking about the assumed redshift dependence of mu. Is the second one correct?
Assuming yes, I'll clarify: currently this PR assumes that mu(z) = mu0 * OmegaL(z) / OmegaL_0 and similarly for Sigma. I think we said that someone in your group was interested in doing another PR after this one gets merged to generalize this. Does that answer your question?

@c-d-leonard : CMB (and non-CMB) lensing are not implemented in the non-Limber case. Does that make sense?

@damonge oh okay, that explains why I couldn't find where it was implemented :) Has that always been true? Is it something we think is going to change in the future or are we happy with limber-only for those cases?

@mishakb, given @damonge's reply above I will just go ahead and make sure that the appropriate factors are implemented for CMB lensing in the Limber case.

@c-d-leonard Good for CMB-Lensing then (@damonge ). I checked again for clt_init_wM and fine too. For the early times recovery of GR, you got my question right the first time.  So by the parametrization above, at early times Omega_L(z) becomes smaller and smaller until vanishing so mu=0 and GR is recovered as well as it growth at early times. So this addresses this question.

@c-d-leonard incidently, I was not able to see your new comments about the K-g correlation window?

@c-d-leonard incidently, I was not able to see your new comments about the K-g correlation window?

@mishakb The way it's set up, any correlation is made up of two tracers; here it would be one lensing-type tracer (for K) and one number counts tracer (for g). The MG effect on the K tracer is through the modified power spectrum and through the factors of Sigma in clt_init_wL (around like 422 of ccl_cls.c). The MG effect of the number counts tracer is through the modified matter power spectrum alone (unless there is magnification); this comes in either in transfer_nc (around line 723) for the limber case or within CCL/build/ANGPOW/src/ANGPOW/inc/Angpow/angpow_ccl.h in the angpow case.

@c-d-leonard sorry for not making my question clear enough. I understand and agree with your comment above and that is indeed how auto-cross-correlations are implements.  What I really wanted to mention is that I did not see the comments in the code itself and recall that you said/wrote that you added some. (maybe I am just missing them).

@c-d-leonard it would be great to put an "MG" label or "flag" everywhere where you made changes. I think there are already at almost each place so just to make sure none are missing.

@mishakb I can do this, but I'm not sure if we actually want to. I understand that it would be useful for reviewing, but you can imagine that if we had different flags for every PR we merged into master, the code in master would get pretty messy pretty fast. Maybe @elisachisari can comment on this stylistic issue?

@c-d-leonard sorry for not making my question clear enough. I understand and agree with your comment above and that is indeed how auto-cross-correlations are implements. What I really wanted to mention is that I did not see the comments in the code itself and recall that you said/wrote that you added some. (maybe I am just missing them).

Right, I added some at e.g. lines 739 and 952.

@c-d-leonard it would be great to put an "MG" label or "flag" everywhere where you made changes. I think there are already at almost each place so just to make sure none are missing.

@mishakb I can do this, but I'm not sure if we actually want to. I understand that it would be useful for reviewing, but you can imagine that if we had different flags for every PR we merged into master, the code in master would get pretty messy pretty fast. Maybe @elisachisari can comment on this stylistic issue?
@c-d-leonard I see your point and it probably makes sense. So perhaps just the comments are good enough to trace the changes plus the git of course. @elisachisari
@c-d-leonard let me know when you are done with all changes.

@mishakb I think it's ready to look at - I am just fixing some issues with the Travis build but I don't have any more substantial changes planned at the moment.
What's the status on understanding at which scales we expect what agreement with benchmarks?

@elisachisari Thanks so much for the review! I addressed your comments one by one above but in general:

I need to add a tests file to include the benchmarks I got recently for real space two point correlation functions, rather than Fourier space (see comment above)
I am very confused about what's going on with the tests. It looks like they are passing on linux and failing on mac, which makes it very hard for me to debug because I don't have a mac ... I may need some help with this.
We still need to get the final word on the expected tolerance at a given scale of the difference in the benchmarks and CCL for the matter power spectrum, @mishakb will have more information on that status of this.

@c-d-leonard I have been working on the comparison rationales/expectations but got slowed down by the collaboration wide presentation today and the last days of teaching this week.  I will work on it again tomorrow.  Glad that @elisachisari looked into the tests/benchamarks code changes for MG as that is the part I wanted to ask about before to look into.  Are the only related directories examples and benchmarks?

pushed wrong button by mistake.  :(

@mishakb the directories relevant for the tests are benchmarks/ and tests/
@elisachisari To update: I have started on a script to include tests against the correlation function benchmarks Cristhian sent me but (as you will have seen in the chat Mustapha) I am waiting on some information from him (e.g. the dNdz's he used), so I'm not going to push that script to the repo yet. I will hopefully be able to work on this at the beginning of next week a little but like I said I am traveling for the next two weeks so I won't have that much time.
Also the Travis build seems to be failing erratically (it's failing C tests on mac which it passes on linux, which is super weird and hard for me to debug). If someone with a mac could run the tests on this branch, that would be a help.

The C tests are failing in my Mac:
TEST 87/95 power_MG:MG_nonlin_error ERROR 1040: ccl_background.c: ccl_cosmology_compute_growth(): Support for the growth rate in cosmologies with massive neutrinos is not yet implemented.
The python tests fail here:
Test halo model and supporting functions. ... mu0=0.000000, sig0=0.000000 ok ccl_test_pyccl_interface.test_halomod(pyccl.Cosmology(wa=0.0, n_s=0.96, Neff=3.046, sigma8=0.8, w0=-1.0, bcm_etab=0.5, bcm_log10Mc=14.079181246047625, A_s=None, h=0.67, sigma_0=0.1, mu_0=0.1, Omega_k=0.0, bcm_ks=55.0, Omega_c=0.27, Omega_b=0.0490086879038, Omega_g=None, m_nu=0.0, mnu_type=None, z_mg=None, df_mg=None, matter_power_spectrum='linear', baryons_power_spectrum='nobaryons', halo_concentration='duffy2008', transfer_function='boltzmann_class', mass_function='tinker10', emulator_neutrinos='strict'),) Test halo model and supporting functions. ... Abort trap: 6
The Travis build that is successful doesn't seem to be running the C tests (I can't find them in the output, which is very strange): https://travis-ci.org/LSSTDESC/CCL/jobs/530873780
The Travis build that fails, fails both for C and python: https://travis-ci.org/LSSTDESC/CCL/jobs/530873782 The C error is the same as above, but there seems to be some difference in the python failure. I don't know why the python tests aren't failing in some of the Travis builds.
When the tests run (both in C and in python), I get prints to screen of "mu0=0.0000" and "sigma=0.0000". We should remove this before merging but for now they seem useful to debug.

@mishakb (cc @elisachisari) - Thanks a lot for giving careful thought to the question of how well the benchmarks for P(k) should agree. Given your statement that you wouldn't expect agreement above 1e-2 and that we should only compare at subhorizon scales (which I agree with), I have made the tolerance for the MG P(k) tests 1e-2 and I have only checked the ones with MG parameters nonzero at scales smaller than k_h =5e-3. Of course, this is sort of reverse-engineered, because I checked to see at what larger scales the benchmarks started to differ from CCL at greater than 1e-2. Nevertheless I think the fact that we get this agreement is a pretty good check.
With regards to your request for the P(k)'s from CCL to which I am comparing - I'm not comparing to an output file of P(k) but rather CCL gets called and compared to the output from MGCAMB / ISiTGR actively when the tests are run. If you do want to output what is being compared, you could do that pretty easily though - you'd find it in CCL/benchmarks/ccl_test_MG.c.
I still haven't heard back from Cristhian on information I need to incorporate the correlation function benchmarks. I will ping him again.

@elisachisari I have no clue what's going on with the tests. The C tests at least are passing locally for my on my laptop. Let's see how these latest commits fare with travis I guess....

Hmmmmm. I am looking at the tests. A few things

The linux tests for the C code are not actually running! My first clue was the short testing time. Our test suite takes ~40 minutes and it finished in ~25. See here https://travis-ci.org/LSSTDESC/CCL/jobs/531997653#L2311

For the OSX tests, one of the failures looks like something in the C test suite.

TEST 91/99 power_MG:MG_pk_model3 ERROR 1040: ccl_background.c: ccl_cosmology_compute_growth(): Support for the growth rate in cosmologies with massive neutrinos is not yet implemented.

I don't understand this one, but presumably, there is a section of code that is calling the growth function with a neutrino model. This will need to be tracked down in the file. I am guessing we assumed this was working due to the linux tests but actually it is not since those did not run.

For the OSX python tests, I don't know at all what is going on.

Short of digging into the code itself, I think you will need to run them all locally and investigate in detail. If you cannot get them to fail locally, try clearing out the build dir etc or use a fresh installation.

Thanks @beckermr and sorry about the slow reply (I've been travelling). I did a new clone and build and still can't get the C checks to fail. So, I think I'm going to have to proceed with the world's slowest debugging strategy and debug via Travis.

@elisachisari @beckermr @mishakb The Travis tests are passing now! What was happening was that there was a bug introduced in getting the neutrinos masses in setting up a temporary equivalent GR cosmology object; only one of the three masses were getting set. Locally on my laptop the other masses were also by default being initialized to zero for some reason, but on the Travis build they were being set to nonsense non-zero values.
I think now the only thing is to get the correlation benchmarks included. I'm still going back and forth with Cristhian on understanding how he produced those so I can do the right comparison.

@cdleonard:  We discussed yesterday the problem here at UTD and we compared ISITGR, MGCAMB (and CAMB) using GR limit but also for the case MG params=0.1 and all checks just fine with mostly around 10^{-3} differences. Therefore, the difference has to be in the way that we have changed in ISiTGR/CAMB/MGCAMB the lensing from DES bins and  specs (as implemented in CAMB/ISiTGR/MGCAMB) to those of the CCL code and benchmarks. We are doing some further checks and will be in touch.

Attached and below are the results of comparison from @gqcristhian and I for the for the comparisons between CCL and ISiTGR (CAMB) for the GR and MG cases for the 4 correlation functions:
So basically the relative errors  are :
w(theta)  ------> the relative error is less than 1.8x10^-2 in all cases
gammat  ------> the relative error is less than 6x10^-2 in all cases
xim  ------------> the relative error is less than 6.8x10^-2 in all cases
xip  -------------> the relative error is less than 1.6x^-2 in GR, but less than 2.5x10^-1 in MG cases due some points.
Note also that for xip in bin(1,2) for GR and MG the last value of theta causes a large relative error for some reason. The correlation functions with the CCL data, provides in total, 180 theory predictions points. Of these 180 points, 179 have a reasonable relative error. However, 1 data point corresponding to xip, in bin (1,2), for the last value of theta (we have 15 values for theta, just the highest one) is different from CCL.  Since this issue is present in GR case as well, I am not sure if this is worth further consideration but we would like to hear what you think.
plotsMG.zip
plotsGR.zip

Here are the plot for xip where we removed the last point of integration.  For all cases xip relative error is less than 2.1x10^-2
NewPlots.zip

As agreed at telecon, here is a brief summary of exchanges or files related to this PR but have been done via slack-channel:
Below is part of an initial file for possible implementations of MG parameters in CCL prepared in early 2018 by Weikang Lin and Mustapha Ishak that was discussed with Danielle Leonard in May 2018.  The Method I below was adapted/completed and implemented (see CCL note):
Method I. Some general steps for quasi-static limit :

Growth function (and growth rate). The file is in src/ccl_background.c.
The linear matter power spectrum is obtained by multiplying the growth factor. The growth factor is affected by μ.
If the matter power spectrum is scaled and stored in the spline, the total growth factor needs to be normalized to 1 today and ∝ a in the distant past as usual.
The total growth factor (and growth rate) should be used in the RSD transfer function. Just need to make sure the total one is being used in it.
Three “transfer functions” (weak lensing, magnification and CMB lens). The file is in scr/ccl_cl.c. They are affected by Σ via a modified Poisson equation for relativistic particles. 1/k^2—>Σ/k^2.
Declare any additional parameters in include/ccl_core.h

Method II. Another way and σ_R.

Another way is not to scale the matter power spectrum in the spline, but use whatever CLASS gives.
The normalization in this way will be different. The unmodified growth function is normalized to 1 today and ∝ a in the distant past. The modified growth (additional part) should be 0 in the distant past, which means the total growth function is not normalized to 1 today.
If someone needs to output the matter power spectrum, multiply the output with a modified growth factor.
In this way, additionally one needs to change σ_R to include the modified growth factor today. The file is in scr/power.c. This can be done by multiplying the sigmaR_integrand by a growth factor (scale dependent or not). Doesn’t seem to have a σ_R(z) at a arbitrary redshift, so we don’t need to worry about that.

February 2019 – March 2019: a number of discussions and file exchanges between Danielle, Cristhian, Weikang and Mustapha on slack channel to get the MG matter power spectrum P(k) from CCL to agree with that from ISiTGR/CAMB. The implementation of MG in CCL is different than ISiTGR. In CCL the matter power spectrum of GR is rescaled using the ratio of growth factors MG/GR while in ISITGR the potentials are modified to MG and the power spectrum calculated accordingly. Results reported in benchmarks and expected and found to be at the 10^{-2} level. Results available in the benchmarks.
April 2019 -- July 9th 2019: a number of discussion and file exchanges between Danielle, Mustapha and Cristhian on slack channel to get the MG 3 X 2pts power spectra and correlations to agree between CCL and ISiTGR.  Different integration methods and configuration of test data required some good deal of work. This included the implementation of common redshift bins and distributions and other data settings. After some intense debugging and fixing of integration limits and some conventional pre-factors for the cls and correlations, an agreement between the correlation between CCL and ISiTGR of ~1-5 X 10^{-2} was reached, in accordance with the expectation. Results available in the benchmarks.

@elisachisari @mishakb I just added the benchmark files to the test script and the tests are passing! locally anyway, let's hope the travis build agrees. I did change the factor by which we multiply the forecast errors to be 0.2 for both fftlog and bessel (before it was 0.1 for bessel). This still means that the discrepancy is less than 20% of the forecast error on the correlation functions, which is not too bad at all. Are we okay with this?
I still need to add a comment to the note stating not to use the power spectrum on large scales outside of the quasistatic regime with mu / Sigma != 0. I will probably end up pushing that tomorrow but I think code wise this is ready for a final look over.

Oh also I renamed the benchmark files because the names were long enough that they were causing the test script to fail flake8 tests for python formatting, and I removed the files not actually used to keep the data repository cleanish.

My previous comments and suggestions have been addressed. Benchemarks for the power spectra and correlation functions for clustering, lensing and their cross-correlation against external code ISiTGR are at the expected theoretical level of agreement. These have been documented in the wiki with up to date references to ISiTGR repository. CHANGELOG and note have been updated. I approve this PR.


Debugger unexpectedly skips lines when stepping through code

Godot version
3.4.4.stable
System information
Windows 11
Issue description
When using "Step Into" or "Step Over" in the debugger when a breakpoint has triggered, sometimes lines are skipped. For example when I press "Step Over" I expect the debugger to land in the next line, but that doesn't happen sometimes.
Steps to reproduce
For some reason, now that I am trying to reproduce it, it doesn't happen anymore. I am sorry, I know this looks bad.
Once, I thought it had something to do with where my cursor was placed when pressing "Step Over", but that doesn't consitently reproduce the bug either.
I'll try to describe a scenario where this issue would occur. Let's say I have the following code:
var x = 3
x = 4
if x = 3:
  print("3")

The debugger stops at the first line and I press "Step Over". I expect it to land on x=4, but it lands on the if-statement.
Edit: It just skipped from line 60 to 62 when I pressed F10, which is "Step Over" for me.

Minimal reproduction project
No response

@Calinou I'll try to come up with a minimal reproduction project, but so far I haven't found a way to reproduce this reliably. It just happens sometimes.
Is it possible I am using threads without being aware of it? Because I haven't actively done anything like described in the documentation regarding threads.


Make the package not dependent on alegreya package

If one uses xelatex or lualatex and already has the Alegreya font installed system-wise, then they would not need the alegreya tex package to use the Arguelles theme. Most importantly, installing the alegreya tex package in this case would lead to having the font in two different places, which is really not a good idea.
I believe that, instead of forcing the use of this font via the package, it would be better to provide concrete instructions in the README.md file for using it via specific Beamer commands for the various tex engines.

Hello! I see your point but I have a few remarks. In order of importance:

As far as I am aware, the theme's dependency on the Alegreya package does not cause the font to be in two different places on your system. In fact, the Alegreya package is already part of your TeX distribution, so you have it regardless of whether the theme uses it or not. And if it uses it, it does not make another copy of it.
Alegreya is a core aesthetic feature of the theme. I do not want to support the use of just about any font: if people want to use something ludicrous in a presentation, say, the sans-serif version of Computer Modern, they will do it without my help. That said, the use of Alegreya is not forced in any way. You are free to load another font package in your own TeX file and override the theme's default.
You are proposing an approach that would be convenient to users who (a) prefer XeLaTeX/LuaLaTeX, and (b) already have the Alegreya font installed system-wide. This seems a rather small group of users, especially because Alegreya is not a system font on any operating system I know. I am not sure if the specific needs of this group of users should drive design choices. Still, I am interested in making the theme more usable to this group: what would be needed exactly? An alternative code that does not load the Alegreya package but uses fontspec?

I'd be happy to know what you think.

Thanks a lot for replying. If I understand correctly, you would be well served by an option that causes the theme to skip loading the Alegreya package, so that the font may be used via fontspec.
It is also possible to code what the system should do if the option is used, like actually loading fontspec or checking if a font is installed, but I am not an expert on fontspec and I have no idea how it might interact with other things. Therefore, I'd be more inclined to just skip Alegreya and leave the rest up to the user. Would this work in your case?
As a side note, if you know fontspec well enough to write up the commands that should be used to load Alegreya that way, you're more than welcome to make a pull request. I would rather not endorse any other font, but allowing the same font to be loaded in different ways sounds great.


[2.0] Remove decodeLast.

Motivation:
decodeLast is nothing more or less than a bug magnet. It by default
assumes that it is safe to re-enter decode synchronously, despite its own
execution model ensuring that this is not the case. Additionally, it does
this quietly, without notifying most naive users of B2MD that the method
exists at all.
This is done to ensure that parsers successfully parse everything, but the
implementation is silly. In particular, it assumes that this uncommon
behaviour (needing to drain the parser on EOF) is sufficiently common
to justify burdening all implementations with it. Most protocol parsers
do not need to do anything special on EOF, because EOF is not a meaningful
signal for their protocol.
On top of this, B2MD does not even properly handle this behaviour becuase
it is not half-close aware. If a TCP half-close is received, B2MD will
not call decodeLast. Of course, protocols that treat EOF as semantic
are also the protocols that are most-likely to benefit from half-closure
support, meaning that the protocol types B2MD is working to support here
need custom code to work properly anyway.
Given that, and the long sequence of bugs caused by decodeLast, we should
remove it. In the extremely unusual case that EOF is semantic for the
protocol, users can reimplement something like decodeLast using
channelInactive and userInboundEventReceived.
Modifications:
Deleted decodeLast.
Result:
Fewer bugs. Happier users.
Resolves #108.

By the way, I think it's hugely meaningful that this patch is only deletions. There are exactly zero tests and zero protocols relying on the behaviour of decodeLast.

@Lukasa shouldn't we remove the definitions of decodeLast too because that code won't ever be run now, right?

In the master that existed when I wrote this patch there were no user definitions of decodeLast. There is now one, in the WS decoder, but the HTTP decoder doesn't implement decodeLast because it was literally impossible for it to have a correct implementation. It implements channelInactive instead. ;)
That said, yes, I'll rebase on the current master and remove the WS implementation.

We fixed this in a different way, B2MD2 is not done yet but the bug-magnetiness of decodeLast is gone


Add support for Zen Audio Player

Zen Audio Player is one of my side projects.
It lets you listen to YouTube videos without the distracting visuals. It's really useful for listening to EDM mixes at work because the videos are often NSFW slideshows.
The logic here is mostly based off of the YouTube connector. Zen Audio Player makes use of YouTube's iframe API, there's no magic going on :smile:

Thanks @david-sabata, do you know the ETA for the next release?


Remove speculative macOS crash fix

This neither fixed https://github.com/ppy/osu/issues/12944 nor peppy's crashes, so pretty safe to disable. I don't want to be causing too many unnecessary flushes.

I'm just hoping that my reduction in crashes isn't related to this being present 😅 .


Sector - Public Finance - Content Review

[x] Verify project/sector descriptions
[x] Check project/sector images
[x] Check all resources that are added/missed
[x] Check partner links
[x] Check social-media links
[x] Check if all bandhus are added on the project/sector pages

Thanks @apoorv74 . Have reviewed the content and made a few corrections. Added resources (blogs, videos etc.). Haven't added a cover image. Please let me know if there's anything missing @akkisagiraju

OBI project image - new suggestion
PF Sector image - new suggestion - this one I'm not sure how we can use - we can freely use it with attribution.
Requesting @preethical to check if the images for public procurement are appropriate. There seems to be some missing resources in the public procurement page, under the project name.

@aruncdl can you share the source for the PF sector image ?

@aruncdl could you elaborate? I see the resources, i had put in there in the project area. But there is not twitter, url, insta, facebook currently for public procurement to add.

@apoorv74 @aruncdl can someone tell me where I can put the appropriate image if need be?

@apoorv74 these are some of the options with their source
Option 1
Option 2

@apoorv74 @aruncdl can someone tell me where I can put the appropriate image if need be?

Here

@akkisagiraju please check the images and close.

@aruncdl Can we have a different image for PF sector? The license from Freepik says we have to explicitly include their website url and the text "Background vector created by rawpixel.com - www.freepik.com" in our footer or somewhere close to the image.

@akkisagiraju let's go with the image that's on right now for the launch imo, unless @gggodhwani can suggest something that works


Add CONTRIBUTING.md

Summary
This commits adds a CONTRIBUTING.md to be used by GitHub, which is displayed when creating a pull-request.
My aim was to keep it simple and include the most important information to consider for new PRs and mostly link to the documentation.
Impact
None
Testing
Not applicable

LGTM


Cross Over Grade Final Grade

SHRUTHI
Live Review with Student Crossovers (7 Points Total)
Runtime (binary criteria below, 3 points + bonus)

[x] Observe Team project quality,  runs on cross domain, performs all evaluations without crashing.
[x] Observe, Individual UI contains inputs that are validated for garbage, data cleaned (ie validate DOB or Password)
[x] Observe, Individual UI contains actions for Create, Read,
[ ] Observe, Individual UI contains actions for Update, Delete (above requirement)

Code Review (binary criteria below,  4 points  + bonus)

[x] Individual code quality, review and observe comments in every function, class, or method reviewed.
[ ] BE review, observe if  Individual API handles error conditions (ie duplicate user id)
[ ] BE review,  observe if Individual API contains GET and POST methods
[x] BE review, observe that Individual code contains database operations that support Create, Read
[ ] BE review, observe that Individual code contains database operations that support Update, Delete (above requirement)

ANNIKA
Live Review with Student Crossovers (7 Points Total)
Night of the Museum (4 points Total)

[ ] Attendance and Capture the Moment: 2 points
[ ] Presentation to Teacher: 2 points (1 minute)
[ ] Frontend
[ ] Behind the Scenes
[ ] JavaScript Inspect
[ ] Database Viewer

Runtime (binary criteria below, 3 points + bonus)

[x] Observe Team project quality,  runs on cross domain, performs all evaluations without crashing.
[x] Observe, Individual UI contains inputs that are validated for garbage, data cleaned (ie validate DOB or Password)
[x] Observe, Individual UI contains actions for Create, Read,
[ ] Observe, Individual UI contains actions for Update, Delete (above requirement)

Code Review (binary criteria below,  4 points  + bonus)

[x] Individual code quality, review and observe comments in every function, class, or method reviewed.
[ ] BE review, observe if  Individual API handles error conditions (ie duplicate user id)
[ ] BE review,  observe if Individual API contains GET and POST methods
[x] BE review, observe that Individual code contains database operations that support Create, Read
[ ] BE review, observe that Individual code contains database operations that support Update, Delete (above requirement)

Silent Review by providing Review Ticket to Teacher (4 Points Total)
Team review ticket 1 point.  Participation of Individuals, 75% of individuals completed college board requirements with a crossover score on each of 6 criteria.
Individual Review Ticket 3 points.   Provide me individual review ticket that is linked from Team Summary.

[ ] Highlight your accomplishments. (.5 each with Bonus)
[ ] Review Ticket / Presentation
[ ] Runtime Links
[ ] Technical Achievements, backend focus
[ ] Guides for Review (include Blog)
[ ] Video
[ ] GitHub analytics
[ ] Bonus, substitute and personal achievement for .5 more

ANNIKA
Live Review with Student Crossovers (7 Points Total)
Night of the Museum (4 points Total)

[x] Attendance and Capture the Moment: 2 points
[x] Presentation to Teacher: 2 points (1 minute)
[x] Frontend
[x] Behind the Scenes
[x] JavaScript Inspect
[x] Database Viewer

Runtime (binary criteria below, 3 points + bonus)

[ ] Observe Team project quality,  runs on cross domain, performs all evaluations without crashing.
[x] Observe, Individual UI contains inputs that are validated for garbage, data cleaned (ie validate DOB or Password)
[x] Observe, Individual UI contains actions for Create, Read,
[x] Observe, Individual UI contains actions for Update, Delete (above requirement)

Code Review (binary criteria below,  4 points  + bonus)

[x] Individual code quality, review and observe comments in every function, class, or method reviewed.
[x] BE review, observe if  Individual API handles error conditions (ie duplicate user id)
[x] BE review,  observe if Individual API contains GET and POST methods
[x] BE review, observe that Individual code contains database operations that support Create, Read
[x] BE review, observe that Individual code contains database operations that support Update, Delete (above requirement)

Silent Review by providing Review Ticket to Teacher (4 Points Total)
Team review ticket 1 point.  Participation of Individuals, 75% of individuals completed college board requirements with a crossover score on each of 6 criteria.
Individual Review Ticket 3 points.   Provide me individual review ticket that is linked from Team Summary.

[ ] Highlight your accomplishments. (.5 each with Bonus)
[ ] Review Ticket / Presentation
[ ] Runtime Links
[ ] Technical Achievements, backend focus
[ ] Guides for Review (include Blog)
[ ] Video
[x] GitHub analytics
[ ] Bonus, substitute and personal achievement for .5 more


[Snyk] Security upgrade browser-sync from 2.29.1 to 3.0.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

676/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.1
Cross-site Request Forgery (CSRF) SNYK-JS-AXIOS-6032459
Yes
Proof of Concept

658/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 5.3
Regular Expression Denial of Service (ReDoS) SNYK-JS-AXIOS-6124857
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: browser-sync
    The new version differs by 22 commits.

c0556cb v3.0.0
5d5e665 v3.0.0-alpha.2
b7bf92e remove package
2e8dd31 v3.0.0-alpha.1
23f1754 optiona
ab5f75b v3.0.0-alpha.0
aa23814 moving localtunnel to peer dependency
f267cb6 Merge remote-tracking branch 'origin/09-09-adding_playwright_tests'
97f08e7 removing cypress
8731b98 removing cypress
7f2dc62 adding playwright tests
d787281 Update README.md (#2051)
2e2fdc6 fixing nx caching, run tests on nonde 16, 18 & 20 (#2050)
b2441b1 build(deps): bump json-schema and jsprim in /packages/browser-sync-ui (#2042)
86fd7fd build(deps): bump json5 from 1.0.1 to 1.0.2 (#2014)
a95410e build(deps-dev): bump webpack in /packages/browser-sync-client (#2027)
1e4fb68 build(deps): bump engine.io in /packages/browser-sync (#2038)
daa8e8a build(deps): bump nanoid and mocha in /packages/browser-sync-ui (#2015)
02efdff v2.29.3
62d906e fix: append to head if body not present yet - fixes #2031 (#2041)
f91440e v2.29.2
d0c50e0 deps: drop qs (#2040)

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Request Forgery (CSRF)
🦉 Regular Expression Denial of Service (ReDoS)

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis.


refactor: use return statement only once with ternary if

The _get_queryset() function from django/shortcuts.py file use two return statements, We could refactor this to use only one return statement by using the ternary if statement.

@th3h4dyy  Thanks for this patch :+1:, however we try to avoid trivial code reformatting unless it's a big problem as it creates extra noise in the history.


Strip big blobs from /config_dump

Consider an xDS which sends a Listener config with the gRPC-JSON transcoder filter.
Which has the proto_descriptor_bin field which is some megabytes of protobuf descriptors.
This wall of base64 text makes  /config_dump hard to read, so it would be nice to replace it with something shorter, e.g. a sha-1 of that blob.

I think it should be controlled by an URL option, e.g. /config_dump?short


Photo thumbnail data is unintentionally included in all query results

All PhotoFields, including photo thumbnail data BLOBs, are included in queries that include Fields.all. The library currently does not use this BLOB data contained in query result cursors. Therefore, it might be slowing down queries for no reason. CPU and memory usage may temporarily spike during cursor processing until the cursor is closed.
We should not be including photo thumbnail data as part of query APIs! This is the whole reason why the Photo entity is effectively unused.

This is a part of #132.


New Translation: Українська мова (Ukrainian language)

My name: Vladyslav ZVLAD
URL: https://github.com/zvlad
Please add the Ukrainian language.
Thank you!
uk-UA.zip

Thanks for your contribution!
@all-contributors please add @zvlad for translation

I attached the localization file to the first message. There is a zip archive

I attached the localization file to the first message. There is a zip archive

No worries, your translation will be available on next release ❤

Thank you! Also I can support localization.


(chore) Adjust Send/Receive to Address

Description

Disable Send to Address
Modify styles
Add Icon

Web

iOS

Android

Animation

About #2515 #2523

@MathiasMoralesDixtra @sirpy
As i've understood the 'send to address' screen is available by the direct url, you have agreed with just to remove 'send to address' button ?


Geo-referenced images AKA Ground Overlay

This is more a feature request than a bug.
I would like to be able to draw what Google at least call Ground Overlays  onto the map.
I thought I found a way to do it by putting a raster layer with just the image in the right place in the tiles over the vector data. However outside of the few tiles I need the raster layer draw black pixels over the rest of the map.
Thanks

Hi @ac87, thanks for your feedback.
Supporting Google-style image overlays is an interesting suggestion which we will consider, though probably won't be able to add in the near term. @blair1618: if we added this, we could consider slicing the image into raster tiles on the fly, similar to what geoson-vt does, but for Raster sources (I spoke with someone at State of the Map doing this).
In the meantime, your suggestion of a bounded raster tile source makes sense. We are currently  considering the ability to apply a bounding box to any data source (vector or raster), outside which no tiles will be requested. See https://github.com/tangrams/tangram/issues/366.

Thanks for the feedback, just to be clear this is not for using huge GeoTiffs as a raster overlay (which you have to pre-slice to be able to render) but more small images such as a building floorplan.
I see it being more a mapdata object, perhaps a polygon with image source where the image stretches between the top left and bottom right points in the polygon. Having looked through the issues now, I think this is related to custom markers #763 #653, only the marker is a polygon

For any other newbie trying to do this, Tangram is more than capable already.
I am still just playing with Tangram and there is a lot to absorb in the docs. Hadn't spotted Geometry Masking
I created tiles for the zooms where I want the image to appear and used a GeoJson file to define the location to render the tiles at.
The other thing I had missed was why transparent pixels in the tile were rendering as black, the raster draw style needed to have blend added, and blend must be a parameter of a style not added to a draw block.
styles:
    raster-blended:
        base: raster
        blend: overlay

sources:
    image-overlay-rasters: { type: Raster, url: 'http://localhost:8888/{x}/{y}/{z}.overlay.png' }
    image-overlay-source:
        type: GeoJSON
        url: image_overlay.json
        rasters: [image-overlay-rasters] # attach image layer

layers:
    overlay-image:
        data: { source:  image-overlay-source }
        draw:
            raster-blended:
                order: 220

Can now see why you had mentioned bounded tile source. That would make this a perfect solution.

There is indeed a lot to absorb :)  We need to improve the learning curve for sure.
Tile bounding box is coming soon (in Tangram JS, then easy to add to ES as well)! https://github.com/tangrams/tangram/issues/366

Tangram JS has recently added a much easier way of achieving this effect: https://github.com/tangrams/tangram/pull/688
I expect that the best resolution to the need expressed in this issue is to implement this feature, as captured now in https://github.com/tangrams/tangram-es/issues/2032. To consolidate the issues, I'll close this one.


refactor: Remove redundant methods in instrumentation/rack

While I do not know the precise provenance of these methods (nor do I
care to futz around with git any further to find out), the current API
contains ~similar~ identical methods that we can rely upon for this functionality.
Specifically, we remove:

OpenTelemetry::Instrumentation::Rack.current_span
OpenTelemetry::Instrumentation::Rack.context_with_span
OpenTelemetry::Instrumentation::Rack.with_span

The only method actually being used was with_span; the others were
just dead code as far as I can tell.
This PR also removes the associated tests, and updates the Gemfiles that
are auto-generated by appraisal upon install.

I knew that was too easy; the rails tests are broken now. I'll fix.

Ah, dang. Okay - so the rails instrumentation will modify the name of the top-level rack span if it can, and that would account for why the duplication is still needed. Creating our own Context.key in the rack instrumentation allows us to fairly easily get at that specific span.
That said - I'd really like to remove the duplication still. It feels like the kind of thing where bugs will creep in if we're not careful, and it feels like it's violating the way instrumentation is supposed to only rely on publicly exposed bits of the API.
Do we have any strong feelings on altering our approach here? I feel like one way would be storing a reference to the context object explicitly in the rack env, and then using that in the rails instrumentation to modify the span name.

We chose the current approach because it is somewhat similar to what we do with HTTP and Redis instrumentation - using Context to provide additional information. I don't think this is a violation of the way instrumentation is supposed to work - Rails is pretty dependent on Rack, and can provide more details than Rack can discern, so it makes sense to me for Rails to be able to pickup the Rack span from the Context in this way and augment it.

it makes sense to me for Rails to be able to pickup the Rack span from the Context in this way and augment it.

I agree with that - I think what I was concerned about was duplicating methods from the API into the rack instrumentation to achieve it. I wonder if there's another way we could obtain the "rack span" from within the thread-local context? If the context key wasn't opaque, it would seem straightforward...
We could expose the CURRENT_SPAN_KEY constant from the Rack instrumentation, and perhaps that would allow us a more straightforward way to accomplish it?
(Also: this is definitely not some huge, pressing issue - I just saw an opportunity to refactor a bit and wanted to see what others thought!)

After today's discussion, I'm just going to close this. I started to refactor the PR to only remove the "redundant" methods that weren't necessary for the rails instrumentation and started to end up with a PR that did very little, if anything. 😆


Fix test_rebuild_with_restoration on master head

Signed-off-by: Chris Chien<EMAIL_ADDRESS>Increase volume/data size make volume restoration time longer
20 times test result
https://github.com/longhorn/longhorn/issues/4488

Please backport this to 1.2.x to ensure the test case consistent.
1.1.x will not be maintained, so no need to backport there.


Add Bindings in IIS Web App Manage and IIS Web App Deploy bindings options suck!

I can't begin to tell you how terrible the deployment tasks are!  If you're lucky enough to find these options buried deep in hell, then you're pain isn't over.  The bindings section of the stage sets up http as expected, but for https it's useless.  I have to manually type in 443 for the SSL port number for it, but not for http?  What's going on here?
The SSL certificate thumbprint forces you to find this mysterious number which is also called a fingerprint.  There are two of them.  One has 40 characters while the other has 256 characters. It would be nice if you clued us into exactly which one you wanted before cutting and pasting it!   Why use this long code instead of a friendly name that can be easily obtained?  Once you've copied the thumbprint you then have to manually remove all the spaces so that it's acceptable by the field.
  After all, this is all free stuff and we should be grateful!

@person Thank you for the feedback, we apologize for the issues you're experiencing.
To solicit feedback from the product team, please use Report a problem or Suggest a feature in the Azure DevOps community and provide a link back to this issue.
Otherwise, please add a reference to the documentation in question and we can assign an author for review.


The Music system  keeps droping songs after  playing half

see image with what the console is saying when the song is requested and playing
http://puu.sh/nxenC/0b123188c6.png
also when using the pre release version which works better the songs end to early says finished and they arnt finished at all

im still look into this, sorry for not replying. There is something weird happening with the music stream.

Nothing to be sorry for its a big thing programing and thought id drop a message so you know :)

Should add on to this, just installed your bot today and I love it already. Seem to be having the same issue with the music streaming tho.
http://i.imgur.com/OBPiEkU.png

Just wondering if you are any closer to fixing this issue.
keep up the good work tho, this bot is great except for this little flaw.

It also seems to trigger when playing the video after the previous video has been skipped. I do not recall the same errors as in the images so it might be a separate bug, but often the video stops early.

This bug is still apparent in the latest build :/

What usualy fixes this for people is changing discord server location to amsterdam or singapore.

I tried switching channels and problem still occurs. Any updated news? (Installed and running latest as of 5/18)

@kp0815 I saw in the Discord chat @Kwoth mention moving the Discord server physically closer to the server hosting the bot helps, but doesn't fully resolve the issue.

@Kwoth @xnaas It's working perfectly now that I'm using an older version of the bot. I tried v0.91 ~ v0.98 and they all "stopped" the stream after half wayish - now that I'm using NadekoBot.v0.9-alpha5.3 It hasn't stopped playing once. I tried sooo many versions but this one hit the spot. (To use this one you need bot account set up instead of the token way - so just can't copy and paste credentials from the newer versions)

This issue will only get resolved on the next iteration of music code, which is after 1.0 version of nadeko - it won't be tracked specifically since it will probably "just work" after moving to discord.net 1.0 and rewriting music code.


fix issue #250

This is a start at fixing issue #250
I looked into this and the problem seems to be that by passing a mapping, the normalizer goes through the "mapping normalizer" code path, which leads to all sorts of problems once the boolean value of "allow_unknown" is encountered.
The "schema" keyword means something different when set in the context of a sequence (in a mapping, the top level of the schema dictionary defines keys of the schema, in a sequence, it defines the top level is equivalent to the second level of a mapping schema), resulting in the exceptions thrown at the normalization stage.
This has probably been discussed before but if not, it might be sensible to use a different keyword for the lists "schema" like "itemschema" or something like that?

cc @funkyfuture

Looks good to me!

yeah, the problem with the ambiguity is known.
you should also add a test case where type is not defined in the schema.
here's a solution for that issue in validating: 7dc632866a2ea6956dc65f42d03e3ab45c295605

I didn't know it was possible to do "implicit" list validation without specifying the type at all depending on how the schema looks.  This seems kind of counter intuitive.  Just for clarification:
As I understand this implicit schema parsing works like this:
schema = {'list': {'schema': {'allowed': ['a','b','c']}}}
v = Validator(schema)
v.validate({'list': ['a', 'b']})
# True, interprets the schema type of 'list' as a list
v.validate({'list': {'a': 'something'}})
# error {'list': ['must be of dict type']}, because cerberus sees the 'allowed' in the schema an assumed you meant a list

However, I can also do:
v.validate({'list': 4})
# also True, apparently because the schema is only checked if a list or dict is passed?

I've restructured the checking with the help of your commit (thanks) @funkyfuture .  And I think I've covered the cases I described above... assume I've understood the mechanics correctly.
To add to the discussion:
Instead of the TypeError check of the list comprehension, I could also go further with the error checking and only raise the _SchemaRuleTypeError if 'allow_unknown' is the problem.  However I don't know if this is more help or harm depending on what other rules you could throw in there that don't have valid mappings as values.

on the train I can't really browse through the code and docs. but concerning your last question I think, it's okay as it is. rules are usually very fault tolerant outside their purpose. with that ambiguity and possible complementing rules to schema though, this can't be applied consequently.
so, lgtm. maybe some note to the docs to prevent misunderstanding. you know best. :-)
(and all your assumptions are correct, it think)

I added a note to the docs, wasn't sure whether to put something under the schema rule, but since it really only gets hairy when you don't define a type rule I would think that's the better place for it.


feat: enable to use static service name for activation in k8s

https://issues.redhat.com/browse/AAP-21464
Follow https://github.com/ansible/eda-server/issues/381,

I think we should have a internal ticket ref to track this feature.

Closes https://github.com/ansible/eda-server/issues/616
Closes https://github.com/ansible/eda-server/issues/381

We need a internal ticket to track this feature.

https://issues.redhat.com/browse/AAP-21464


Add raw tags

Fabio currently uses specially crafted Consul tags (urlprefix-host/path opt opt ...) to generate route commands which build the fabio routing table. There is a certain impedance mismatch between the syntax of the urlprefix- tags and the route add commands they generate, e.g.
urlprefix-/
route add <svcname> / http://<svcaddr>/

urlprefix-/ strip=/foo
route add <svcname> / http://<svcaddr>/ opts "strip=/foo"

urlprefix-:1234 proto=tcp
route add <svcname> :1234 tcp://<svcaddr>/

urlprefix-/ proto=https
route add <svcname> / https://<svcaddr>/

It would be useful if the urlprefix- tags are just a convenient shorthand notation for common more complex route commands but users could choose to express the same thing as a route command as well. This would limit the syntax of the urlprefix- commands to what is necessary for the most common uses cases.
For this to work, the code would need to distinguish between two types of tags: short and raw
raw tags are added verbatim (without the tag prefix) to the routing table whereas the short tags provide a special shorthand notation syntax which needs to be parsed and transformed.
Additionally, the raw tags need pseudo-variables to inject the service name and service address which are currently generated from the urlprefix- tags to be fully equivalent.
urlprefix-/ could then also be written as fabio route add $service / http://$addr/ assuming that fabio  is the raw prefix tag.
The open questions are whether there should be a different prefix for the raw tags or whether this should be rolled into the urlprefix- syntax. Since the urlprefix- syntax is already used for defining TCP services I think we shouldn't overload it even further.
I'd like to gather some ideas on the syntax of a fabio tag for defining routes and allowing raw route commands and then we start with the route commands.
Note
There is an open issue/pr somewhere which has these ideas but I can't find it right now. Will update when I do.
Note2
Part of the discussion is in https://github.com/fabiolb/fabio/issues/87#issuecomment-369008796

I think you would want to use a different prefix for the raw tags.
What is the real impetus for adding this capability? Are you intending for there to be new extensions to the route syntax that can't be expressed using the "short" urlprefix- style tags?

I could see this being useful as fabio gradually moves to supplant the major reasons people give for deploying another more traditional reverse proxy like nginx or haproxy before fabio.  These primarily seem to revolve around header mangling which could be tricky to express with the short tags unless there it becomes yet another csv style option.
Take the following example for header manipulation with pseudo variables:
route add <svc> / http://<addr>/ opts "headers_front=x-forwarded-for=$remote,x-my-header=my static value,remove-some-header,append-some-header+=with this value,headers_back=remove-from-response-"

Versus something maybe more legible and dynamic...
route add <svc> / http://<addr> opts fe.x-forwarded-for=$remote fe.x-my-header="my static value" fe.remove-some-header- fe.append-some-header+="with this value" be.remove-from-response-

Maybe not the best example?

@ctlajoie My main motivation is that there are two ways of expressing the same thing and there is a slight difference in syntax. Also, urlprefix- doesn't sit well with tcp services. Tech debt from the early days :)
Maybe it is just me wanting to clean up but in the process I make things more complex. What do you think?

You can think about the difference between the two as being declarative vs imperative.
If I add a tag urlprefix-example.com/ to a service, I am declaring that I want requests for example.com to be routed to this service.
A tag like fabio route add $service / http://$addr/ is imperative.
Allowing route commands like that in tags also implies other things are also possible to do with the tag, such as fabio route del someothersvc.
IMO it makes more sense to use a declarative syntax for service tags.

I am probably biased because I have never needed to manually add routes using the route add syntax. I consider the urlprefix- tags to be the "normal" method of adding routes.

So we've starting using route add syntax for lots of stuff right now, but we shove it into the fabio.config in consul.  I agreee that it doesn't make sense to support route add in a tag for the declarative vs imperative split.
I guess I'm back to my original request of just wanting a catch-all https-redirector, and I'd prefer putting route add command in fabio.config to putting it in a job's tags - it applies to everything anyway, not just a single job, and definitely "feels" right to put it in fabio.config.

Hey @magiconair had a chance to think any more about a global https redirector?

@holtwilkins you can already do the "catch all" HTTP->HTTPS redirect.  We're doing that today with the following route:
route add http-redirect host.domain.tld:80 https://host.domain.tld$path opts "redirect=301"

To get exactly what you are wanting we could probably add another pseudo-variable for $host that could be used similarly to the existing $path variable.  Then you could basically add a route for :80 that would target https://$host$path.

Yup @leprechau that would be great.

So, after actually looking at the code I think we could do this without introducing another variable.  I think we could just add a single code block to the end fo the BuildRedirectURL function in route/target.go that would use the request host as the redirect host if the redirect host is empty.  This would allow you to specify something like: route add :80 / http://$path opts "redirect=301"

@holtwilkins would you care to open a new issue around this specifically?

Hey @leprechau I'm happy too, but not sure how to word it.  It seems like the old PR at https://github.com/fabiolb/fabio/pull/451 was meant to do something similar to this but was seemingly forgotten about?  Should I open an issue calling out host-less redirects?


181725890 nav updates

This was a refactor of the app's navigation. The three problems it sought to address are:

Navigation was split into multiple classes, some with redundant functions.
Most of the routing passes through the MainActivity and HomeFragment.
Invalid routes when navigating between fragments.

We now only have two classes:

NavUtil:  contains helper functions to validate origin and destination fragments before navigating. Also has a function to inflate TransactionDetails from anywhere in the app.
NavHelper (renamed from StaxNavigation): initializes MainActivity navigation, checks permissions and handles all nav functions declared in MainActivity.

Changes here include:

Removing the NavigationInterface.
Stripping the StaxNavigation of all nav functions that don't require permissions and keep only the ones that originate from the MainActivity class.
All fragments handle their own navigation.

@androidmaven @davkutalek These commits introduce changes to how the app handles deep links. Deep links are now declared at fragment level in the navigation graph like below:
<fragment
        android:id="@+id/navigation_transfer"
        android:name="com.hover.stax.transfers.TransferFragment"
        android:label="@string/nav_transfer"
        tools:layout="@layout/fragment_transfer">
    <action
            android:id="@+id/action_navigation_transfer_to_accountsFragment"
            app:destination="@id/accountsFragment" />
    <action
            android:id="@+id/action_navigation_transfer_to_navigation_home"
            app:destination="@id/navigation_home"
            app:popUpTo="@id/navigation_home"
            app:popUpToInclusive="true" />
    <argument
            android:name="transactionType"
            app:argType="string" />
    <deepLink
            android:id="@+id/transferDL"
            app:mimeType="text/plain"
            app:uri="stax.me/transfer/{transactionType}" />
</fragment>

Android now handles the deep links automatically whenever a valid URI is clicked outside of Stax.
In the example above, the deep link handles routing to the TransferFragment allowing us to specify what type of transfer action we want to execute. Since we're using safeargs, the transactionType has to be passed so that the fragment can load successfully. We cannot use the previous method of using two deep links and since Android handles the navigation for us. Therefore, all send money and buy airtime deep links have to be updated to reflect this.
An advantage of this, is that we can declare routes quite easily like we do on web e.g https://stax.me/{channels}/{country}/{type} if we want.


EB activating Crusher

he is also a multiclass warlock with Eldritch Blast and Repelling invocation, and the Crusher option appears when he hits someone with Eldritch Blast!
Save file attached, from the Lost Valley campaign..
AutoSave_1.zip

fixed shield bash but this EB thing is really odd.

fixed on dev.


[BUG] go-mode not autoloaded for go files

Go language enabled in private init.el, doom refresh run. When opening a .go file it doesn't automatically load the go-mode. Running go-mode manually works perfectly. It's entirely possible there's a step I've missed but I'm not sure what.
System information

- OS: gnu/linux (x86_64-pc-linux-gnu)
- Shell: /usr/bin/fish
- Emacs: 26.2 (Apr 12, 2019)
- Doom: 2.0.9 (HEAD -> develop, origin/develop, origin/HEAD f754d4ff 2019-07-23 18:23:17 +0200)
- Graphic display: t (daemon: nil)
- System features: XPM JPEG TIFF GIF PNG RSVG IMAGEMAGICK SOUND GPM DBUS GSETTINGS GLIB NOTIFY ACL GNUTLS LIBXML2 FREETYPE M17N_FLT LIBOTF XFT ZLIB TOOLKIT_SCROLL_BARS GTK3 X11 XDBE XIM MODULES THREADS LIBSYSTEMD LCMS2
- Details:
  ```elisp
  env bootstrapper: envvar-file
  elc count: 0
  uname -a:  Linux 5.2.8-arch1-1-ARCH #1 SMP PREEMPT Fri Aug 9 21:36:07 UTC 2019 x86_64
  modules:   (:completion company ivy :ui doom doom-dashboard doom-quit hl-todo modeline nav-flash ophints (popup +all +defaults) treemacs vc-gutter vi-tilde-fringe window-select workspaces :editor evil file-templates fold multiple-cursors rotate-text snippets :emacs dired electric vc :tools eval flycheck (lookup +docsets) magit :lang data emacs-lisp go markdown (org +dragndrop +ipython +pandoc +present) sh :config default)
  packages:  n/a
  exec-path: (/usr/local/sbin /usr/local/bin /usr/bin /usr/lib/jvm/default/bin /usr/bin/site_perl /usr/bin/vendor_perl /usr/bin/core_perl ~/go/bin/ ~/go/bin/ ~/go/bin/ /usr/lib/emacs/26.2/x86_64-pc-linux-gnu/)
  ```

I have the same problem for Elixir files

Hmm, this seems to have fixed itself somehow. After a clean reboot it works fine. Strange.

For me, too. Really weird. I am absolutely sure that I restarted Emacs multiple times.

Well I reckon since it seems both of our issues are solved, we may as well close this. I've been thinking about it, and I reckon the issue was that I opened the go file, then exited emacs and installed go-mode, then reopened the previous session when I relaunched emacs. That probably cached the modes instead of just reopening the files. When I reopened the file from scratch after a restart of emacs it worked perfectly....

I can now reproduce this issue.
rm -rf .local
./bin/doom install

Start Emacs, open an Elixir file. Everything is fine. Exit Emacs.
./bin/doom upgrade

Start Emacs, open an Elixir file, fundamental mode.
rm -rf .local/straight
./bin/doom install

Start Emacs, everything is fine again.

*Messages* is empty…


Remove support for string templates in component Template method

https://github.com/Fyord/fyord/commit/390c1dfe7dbdd0b40c8d8e58411f577caadcb6fd


Merged

This PR contains all the changes made to Baleen by NCA to date, including the changes from the other PRs as well as changes and dependency updates not in those PRs. If accepted, the other NCA PRs can be deleted.
Fixes #51, #50, #49

Thank you for taking the time to share these updates with us.


[5.7] Resolve Blade facade to named service

I am unsure whether the manual resolution was added for a specific purpose. I am assuming it was just there because the first version of the ViewServiceProvider did not register a binding for the Blade compiler.

The only other facade that does not return a string from getFacadeAccessor() is the Schema facade.
If we replace that logic with a binding (we'd have to register a new one), we could get rid of some complexity because we could require all facades to return a string from that method.

Hmm, I'm scared to change this in case I forgot some reason I did it this way. 👀

@taylorotwell I don't see why there should be a reason other than what I wrote:

I am assuming it was just there because the first version of the ViewServiceProvider did not register a binding for the Blade compiler.

We can target it at 5.8, then there's enough time to find out?


Update function is necessary when obj get from LokiJs reference

My case is with LokiJs version 1.3.4:
var accountInfo = _db.addCollection('AccountInfo', { indices: ['AccountId'] });
....
var item = accountInfo.findOne({ 'AccountId': 123}); //AccName='Account123'
item.AccName = 'test'; //set new dummy data
and without using function Update like: accountInfo.update(item). The data in Loki still changed by re-load data again:
var itemClone = accountInfo.findOne({ 'AccountId': 123 });
return item.AccName == 'test'); ==> return true
And we wonder why we must call function update for the change? Is there any way to get a clone value only (for all function get data: find, findOne, get....).
Thanks for support,

Quote from documentation:

When you call update on an object though you force a resync of the indexes of the collection

@alex21 thanks for clarifying. I will add that you can obtain the desired behaviour by using the clone property when creating the collection, this will return object copies instead of the object reference.


Add jappanese lrnguage

Is possible?

yes adding a japanese keyboard would be possible, although it would probably be very hard due to 3 alphabets larger than the english one this is based on, and having to make the code to determine what alphabet to use for each word would be hard to say the least.

@canochento i will look at adding this for you (and anybody else using japanese) i will look at chinese however it wont work very well, default would be hiragana, shift/caps lock would get you katakana, and the function menu would give you kanji

thanks.
I love your app.


Stackdriver Dashboard URI should contain projectID

When creating a new Sandbox, the instructions contain a list of URLs of "next steps".
In dashboard URI, we need to add a project parameter so the link opens correctly.
https://app.google.stackdriver.com/accounts/create?**project=<CREATED_PROJECT?>**

Resolved in PR #57


LCFS - Port Organization Django DB Models to SQLAlchemy Models in FastAPI

Describe the task
Port the Django database models for organization, organization address, organization type, and organization status to SQLAlchemy models in FastAPI.
Purpose
The purpose of this task is to migrate the existing Django database models to SQLAlchemy models in FastAPI. This will enable the application to utilize SQLAlchemy's features and benefits, such as improved performance, database abstraction, and compatibility with different database backends.
Acceptance Criteria

[ ]  Create SQLAlchemy models for organization, organization address, organization type, and organization status.
[ ]  Make sure to use SQLAlchemy's declarative base class for defining the models.
[ ]  Map the SQLAlchemy models to the corresponding database tables.
[ ]  Define the relationships and foreign keys between the models.
[ ]  Implement CRUD (Create, Read, Update, Delete) operations for the organization model.
[ ]  Ensure the SQLAlchemy models and database operations are correctly integrated into the existing FastAPI application.
[ ] Test the functionality of the ported models by creating, reading, updating, and deleting records in the organization, organization address, organization type, and organization status tables.
[ ]  Ensure all tests pass successfully.
[ ]  Update the documentation or code comments as necessary to reflect the changes made during the porting process.

Additional context
The existing Django database models should serve as a reference for creating the SQLAlchemy models.Pay attention to the table relationships and foreign keys when designing the models. It's recommended to follow the existing coding style and conventions in the FastAPI project.

No need for testing as this is set up for new LCFS


Not Working On Ios platform

First great work , and its awesome.
But today i was trying to use this for ios platform, and its not working for me.
i tried it for UWP before and its perfect. But suddenly its not working now, even if i try to switch the platform for same project.
i think i always get null instance though i'm making a instance of it, also tried to pass prefab from unity editor layout.
Please help me , is it related to platform ?

I'll test it again and , let you know.


Fix HVAC modes missing the friendly name and having an extra °C appended

With the unit_of_measurement=UnitOfTemperature.CELSIUS constructor parameter set, the current state and the hvac_mode selection of the HVAC device is not rendered properly in the frontend:

hvac_mode is showing the raw message instead of the friendly name
an errornous °C is appended

removing the parameter fixes both:

Nice, wonder if this isn't a bug in HA... seems strange that setting the measurement unit would affect the modes 🙃

Thanks a lot! This has been bugging me for a while :-)


Getting error while running client.client_login() function

I have installed using
pip install IIFLapis
My code is:
from IIFLapis import IIFLClient
client = IIFLClient(client_code=CLIENT_CODE, passwd=PASSWORD, dob=DOB) # DOB is in 'YYYYMMDD' format.
When I am trying to run client_login() function like
client.client_login()
I am getting an Error which is like:

Please, solve this issue...

For me IIFLClient works and I pass 5 parameters ( 2 more parameters: email_id and contact_number )
You should try adding those too - your registered email and number along with client code, password and DOB (YYYYMMDD)

Thanks for the reply.I have added those parameters, but still getting same error.  Thanks & Regards,Tejas NavalkheSoftware (Python, Web, GUI, Data Science) Developer__________________ From: divjainbtSent: Thursday, December 16, 2021 1:56 AMTo: IIFLSecurities/TradingAPIsCc: Tejas Navalkhe; AuthorSubject: Re: [IIFLSecurities/TradingAPIs] Getting error while running client.client_login() function (Issue #22) For me IIFLClient works and I pass 5 parameters ( 2 more parameters: email_id and contact_number )You should try adding those too - your registered email and number along with client code, password and DOB (YYYYMMDD)—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

Use the latest IIFLapis package.

Use the latest IIFLapis package.

Can you please provide me the links or documentation for the same?

Use the latest IIFLapis package.

Can you please provide me the links or documentation for the same?

https://pypi.org/project/IIFLapis/    version 2.1.1 make sure you are using python 3.7 or higher.
Secondly the directory of the python file you first call to execute the program must have the keys.conf file. Python checks the first execution entry folder for the keys file.

I am using the same package and getting the same error. I also have keys.conf file.


BorderRadius doesn't work with missing BorderSide.

When I run this code
import 'package:flutter/material.dart';

void main() async {
  runApp(
    MaterialApp(
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: TestWidget(),
      ),
    ),
  );
}

class TestWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Container(
      color: Colors.red,
      height: 500,
      width: 500,
      alignment: Alignment.center,
      child: Container(
        width: 50,
        height: 50,
        decoration: BoxDecoration(
          color: Colors.black,
          border: Border(
            left: BorderSide(width: 1, color: Colors.blue),
            top: BorderSide(width: 1, color: Colors.blue),
            bottom: BorderSide(width: 1, color: Colors.blue),
          ),
          borderRadius: BorderRadius.only(
            bottomLeft: Radius.circular(10),
            topLeft: Radius.circular(10),
          ),
        ),
      ),
    );
  }
}

I get this error: A borderRadius can only be given for a uniform Border.
When i add  BorderSide right on Border class i didn't get any error. But when i remove right BorderSide value i get error. I tried add right value on border with transparent color, it didn't worked again. Is there any mistake or out of rule programming on my code?

Hi, @MuhammedAlmaz  Thanks for filing the issue, There is already a similar issue describing this case https://github.com/flutter/flutter/issues/12583, You may follow up there for further updates closing this issue as duplicate, If you disagree feel free to write in the comments and I will reopen it.
Thank you.


behave tests will run from a seperate test folder so existing instrum…

…ents on the sftp wont impact the tests

In the blaise terraform you'll need to change the env var there
https://github.com/ONSdigital/blaise-terraform/blob/28ed5b84d42c8f924fecd9487d4b55b42dd7b124/ci/tasks/nisra-case-mover-integration-test/task.sh#L35
SURVEY_SOURCE_PATH="ONS/OPN/"

good spot, done


Transfers fail from Linux to Android

Sender: Dell Latitude running Debian 12 with FC installed via .deb.
Receiver: Samsung Galaxy A03 running Android 13 with FC installed via Google Play.
On the sender I open FC, select "Send", "Android", and the file I want to transfer. On the receiver I open FC, select "Receive", "Linux", and the destination folder. The receiver turns on its camera, and I click "Start Transfer" on the sender. Immediately the sender says:
Start the transfer on the other device and scan the QR code when prompted.
Starting hotspot flyingCarpet_934b
Error connecting to peer: Could not start hotspot: Error: Connection activation failed: No suitable device found for this connection (device eno1 not available because profile is not compatible with device (mismatching interface name)).

The sender displays a QR code only for a split second, but this is still long enough for the receiver to register it. The receiver says Searching for devices... and after a few seconds, No devices found. Make sure the device you want to connect to is turned on and ready to install.
Both devices are able to connect to my home wifi without issues. I've also tried transferring in the other direction, and that fails in a similar manner.
I'd appreciate any insight you might have on this. Thank you.

Thanks for the great report. The message Connection activation failed: No suitable device found for this connection (device eno1 not available because profile is not compatible with device (mismatching interface name)) is what I've seen when using a WiFi card on Linux whose driver lacks support for running a hotspot. I have a wireless USB dongle that can host a network on Windows, but when I use the same one with Linux, I had to compile and install the drivers for it, which let me use it as a WiFi client, but not host a hotspot. So I assume that's what's going on here. Are you able to start a hotspot on that Latitude otherwise? The direction of the transfer won't make a difference: when it's Linux and Android, the Linux machine will always host the hotspot.

Thank you for your fast response. I've now tried creating a hotspot on the Latitude using nmcli (following this guide) and I get the exact same error, so it seems that is the problem.
I've installed FC on my Windows desktop as well, which is able to transfer to the Android, so that solves the problem for me.


In tsconfig update allowJS so it is processing both js(x)/ts(x) files

To guard against potential future issues with hybrid usage of js(x) and ts(x) files in same project, update 'allowJs' to true.
currently commented out, defaults to 'false'
https://github.com/fusor/mig-ui/blob/master/tsconfig.json#L7

// "allowJs": true,                       /* Allow javascript files to be compiled. */

We're only building the front-end with typescript, so it's not a hybrid app. We can discuss the other js files that exist, but I think that's a distinct issue. Closing this and actually setting allowJs explicitly to true. #337 renames the only "js" file that was part of src.


'min' property of datetimepicker is not working property

I have set the min property of datetimepicker in a .ts file as following
min = new Date(1900, 0, 1);
When i select the year less than 1900 then it redirected me to 1899 year-view instead of 1900 year-view. It is working as expected for 'max' property.
Question : When we are setting max and min date then why we are allowing user to select any other date which is not in max-min date range???

I could not reproduce the issue you described.
Pleas use StackBlitz to build an app and show the issue you mention about
I would close this until it is confirmed as an issue.
Answer to your question: No, you could not select any date out of your max-min range.


The issue of figure generation

Hi,
Thanks a lot for your suggestion! Now I can generate the figure after deleting the plt.tight_layout(). But there are two questions. One is the figure overlapped. I double-check the source code, there is a self.figure.tight_layout() in the backend_interagg.py. Here is the figure that I got.
Another question is I cannot get the figure in the plug-in from the PyCharm and I can only use the argument --outf to save the figure in the drive. Do you have any idea about this issue?

Besides, how do you get the figure 2 from the paper? I am curious how to get backward chaining and forward chaining from the IMA with the softmax attention model. I found from the source code using imasm as backward chaining and fwimarsm as forward chaining. Here is the diagram that I got. Backward chaining is more evident than forward chaining. Cause for the forward chaining, step 2 from the left first forward chaining table might show r(X):-s(X) rather than p(X):-q(X).

Is the deepest reasoning depth is 4? I see there are some questions finished before step 4. What if the question cannot be solved in 4 steps? Besides, for the test cases, Can the model solve some more complex problems? The model is trained by multi-task learning, but testing on each test cases. I see there are different difficulty level of question, from easy to hard. What is the main difference between them? Can the model solve a problem that has all of the property from those 12 different tasks?
Many thanks!

I'm not sure about PyCharm related issues, they don't sound related to any bugs in the source code since you are able to reproduce the results and the diagrams. I would recommend looking at PyCharm related documentation and similar to tight_layout adjust the visualisation code.
The details for forward chaining case are explained in the paper, end of experiments section. There are certain training conditions under which that is achieved.
Yes, at training time the models are iterated at most 4 times to cover 4-step reasoning. Questions regarding longer chains of reasoning at test time and difficulty of questions are evaluated and discussed in the paper, namely in section analysis and details of the easy, medium, hard datasets are given in section named experiments.


Example api code updates

This adds some more complete code examples and usage of various pants goals

lgtm! Thanks for the great examples.
The lambda context fixture could be shared between the tests in a conftest.py test util file, but that's minor.

I went ahead and added this now. Thanks for the review!


linkis datasource module remove origin datasource manager add new dat…

Linkis Datasource Manager Common Module
What is the purpose of the change
In order to better manage data sources, the original LIinkis DataSource Manager module was transformed, the original DataSourceManager module was deleted, and increase the Linkis-DataSource-Manager-Common module and the Linkis DataSource-Manager-Server module were added. The PR is Linkis - DataSource-Manager-Common Code
Brief change log

remove origin module datasource manager;
add module linkis datasource manager common;
define linkis datasource manager auth context
defline datasource ,the main purpose is store the data source information
defline datasource env,the main  purose is store the data source environment information
defline datasource param,the main purpose for init the datasource
defline datasource type,the main purpose to differentiate between different types of data sources
add some util to support
defline some exception class

Verifying this change
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
Anything that affects deployment: (yes)
The MGS(Microservice Governance Services), i.e., Spring Cloud Gateway, OpenFeign, Eureka.: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? (not documented)

link issue:https://github.com/apache/incubator-linkis/issues/1332


ev6 custom stock longitudinal control can't run

Describe the bug
ev6 2022 with hda2
custom stock longitudinal control can't run

Which car does this affect?
ev6 2022 with hda2
Provide a route where the issue occurs
staging-c3
openpilot version
staging-c3
Additional info

i would like to use it

Custom Stock Longitudinal Control is temporarily disabled for HKG CAN-FD cars since sunnypilot <IP_ADDRESS> due to an upstream openpilot issue with the Red Panda firmware. comma mentioned that the fix is targeted to be implemented in 0.9.4 but no ETA yet. If you would like to use the feature without issues, you can install the test-c3-vw-custom-stock-long branch for the time being (it is still on 0.9.2).
You can follow the thread in the sunnypilot discord under the #dev-feedback channel or the #hyundai-kia-genesis channel for updates on this feature being reimplemented.
DISABLED: Hyundai/Kia/Genesis CAN-FD platforms
https://github.com/sunnyhaibin/sunnypilot/blob/7ec767f53fbc18fa6b45f72dc478f24196a124c6/CHANGELOGS.md?plain=1#L45-L46
https://github.com/sunnyhaibin/sunnypilot/blob/7ec767f53fbc18fa6b45f72dc478f24196a124c6/CHANGELOGS.md?plain=1#L67-L71

thanks you
you veryvery nice


cherry-pick to 4.1.5: pin ms-rest-js and ms-rest-azure-js in connector and schema

Fixes #643 (In the 4.1.5 branch)

Pull Request Test Coverage Report for Build 2167

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

Totals

Change from base Build 2078:
0.04%

Covered Lines:
2633

Relevant Lines:
3088

💛  - Coveralls


Paper for OWCAL

[ ] Analyze dates and relative age of cited authorities (see first proposal here)
[ ] Consider if there are meaningful measures of judicial influence from "author" or "judge" tables in CAP--pull judge from list of most cited cases? or try to track counts to particular judges like Lemuel Shaw, James Kent, Joseph Story?
[ ] Take Wheeler, Practical Law of Slavery (F0105347590): Where else do the cited cases show up? Are they all "property" cases or what other "genre" do they emerge in?

Not sure if they'll have useful material in time for the paper, but we could see if RAs have analytical ideas if they get a database by mid-month. See #86.

Working draft can be found here: https://docs.google.com/document/d/1zb-0SGr69mHc6PIlVEwnwWCkXBzvG7vrd6Qxn-M6ILk/edit?usp=sharing
I will flesh it out after other writing deadlines on 2/17.

Another obvious thing to do:

[ ] Create a co-citation network of cases as seen from the treatises.

[ ] Create bibliographic coupling network of treatises as linked by the cases they cite

Suggested plan to proceed:

[ ] Update whitelist (https://github.com/lmullen/legal-modernism/issues/81)

[ ] Rerun linking over MOML

[ ] Work on analysis of textbooks

[ ] Updates to paper visualizations/tables as needed

[ ] Update single-volume library and regexes (https://github.com/lmullen/legal-modernism/issues/69)

[ ] Import English Reports database (https://github.com/lmullen/legal-modernism/issues/42)

[ ] Import Code Reporter database (https://github.com/lmullen/legal-modernism/issues/80)

[ ] Re-run detection and linking process over MOML, with linking to new databases

[ ] Update paper visualizations with the new citational databases

[ ] Attempt coupling/co-citation ideas above and/or canon-cloud idea from https://github.com/lmullen/legal-modernism/issues/66

[ ] Run detection and linking process over CAP (https://github.com/lmullen/legal-modernism/issues/74)

Just putting the updates hitlist here for simplicity:

[ ] Add smoothed curve of age of cases cited in cases
[ ] Add histogram of age of cases cited in treatises
[ ] Standardize y-axis on top ten case cite counts (axis of 0 to 20)
[ ] Update second paragraph count of unique cases cited (currently 326,000)


question about using 'sep_example.tf'

when using 'sep_example.tf' to test the train procedure of shift_net.
tf file is from  "http://people.eecs.berkeley.edu/~owens/multisensory/sep_example.tf"
i got messege like:
File "train_shift.py", line 3, in
shift_net.train(shift_params.shift_v1(num_gpus=3), [0,1,2], restore = False)
File "multisensory-master/src/shift_net.py", line 315, in train
model.make_model()
File "/multisensory-master/src/shift_net.py", line 155, in make_model
self.inputs = read_data(pr, self.gpus)
File "/multisensory-master/src/shift_net.py", line 17, in read_data
lambda : shift_dset.make_db_reader(
File "/multisensory-master/src/tfutil.py", line 484, in on_cpu
return f()
File "/multisensory-master/src/shift_net.py", line 19, in
num_db_files = pr.num_dbs))
File "/multisensory-master/src/shift_dset.py", line 283, in make_db_reader
ims, flows, samples, sfs, labels, ytids = tf.train.batch(example_list[0], batch_size)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/training/input.py", line 927, in batch
name=name)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/training/input.py", line 722, in _batch
dequeued = queue.dequeue_many(batch_size, name=name)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/ops/data_flow_ops.py", line 464, in dequeue_many
self._queue_ref, n=n, component_types=self._dtypes, name=name)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/ops/gen_data_flow_ops.py", line 2418, in _queue_dequeue_many_v2
component_types=component_types, timeout_ms=timeout_ms, name=name)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2956, in create_op
op_def=op_def)
File "/data/wen/anaconda2/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1470, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
OutOfRangeError (see above for traceback): FIFOQueue '_1_batch/fifo_queue' is closed and has insufficient elements (requested 15, current size 0)
is there some setting in shift_params.py to change or set?
thanks.

i guess this may be reason:
"
2019-03-22 15:45:37.811825: W tensorflow/core/framework/op_kernel.cc:1192] Invalid argument: Name: , Feature: im_0 (data type: string) is required but could not be found.
2019-03-22 15:45:41.945426: W tensorflow/core/framework/op_kernel.cc:1192] Out of range: FIFOQueue '_1_batch/fifo_queue' is closed and has insufficient elements (requested 5, current size 0)
"
im_0 needed is not found.

Hello, have you slove the problem? I also meet the problem when try to train shift_model. I don't kown what the input is for training shift_model.


[object Object] is returned from a binding when resolver throws an error

I've noticed that when a query/mutation is executed via a graphql binding an [Object object] error is returned. This behaviour is observed when an error is thrown in resolver as a part of a business logic.
const resolvers = {
  Query: {
    test: async (parent, args, context) => {
      // some random _id that is passed as a query arg
      const _id = '507f1f77bcf86cd799439011';
      const binding = await context.bindings.organisations.query.organisation({ _id }, '{ _id }', { context: { foo: 'foo' } });
      return 'foo';
    }
  }
};

organisation query throws an Apollo error when organisation is not found by id.

Funny enough the cause is an old version of graphql-tools and the PR for it was already made:  https://github.com/graphql-binding/graphql-binding/pull/161
When will it be merged?

Hi, it's still happening with the latest version. There's no way how to find out what went wrong. I tried to inspect the error but there's nothing helpful.
Error: [object Object]
  at new CombinedError (/app/node_modules/graphql-tools/dist/stitching/errors.js:82:28)

// err.originalError.errors[0]
{ Error: [object Object]
    at asErrorInstance (/app/node_modules/graphql/execution/execute.js:489:43)
    at process._tickCallback (internal/process/next_tick.js:68:7)
  message: '[object Object]',
  locations: [],
  path: [ 'testMutation' ]
}

I found a workaround at https://github.com/apollographql/graphql-tools/issues/480#issuecomment-444318754.
For anyone interested:
import { onError } from 'apollo-link-error'
import { createHttpLink } from 'apollo-link-http'

// ...

const errorLink = onError(args => {
	if (args.graphQLErrors && args.graphQLErrors.length === 1) {
		args.response.errors = args.graphQLErrors.concat(new GraphQLError(''))
	}
})

class XXXBinding extends Binding {
	constructor(uri: string) {
		const schema = makeRemoteExecutableSchema({
			schema: fs.readFileSync(schemaPath, 'utf-8'),
			link: errorLink.concat(
				createPersistedQueryLink().concat(createHttpLink({ uri, fetch }))
			),
		})
		super({ schema })
	}
}

Hey @almostprogrammer 👋,
Thank you for filling the issue. Could you provide me with a small reproduction repository or CodeSandbox? That way I can find the bug a lot more quickly.

Hi @maticzav here's the reproduction https://github.com/almostprogrammer/graphql-binding-173-reproduction/tree/master

Hey @almostprogrammer 👋,
The reason you are experiencing this problem is because the first server localhost:4000 is not yet available when you already try to generate binding from it on localhost:5000. Simply changing the starting script to two separte instances fixed the code for me and I no longer recived [Object Object].
Could you try doing the same thing and verify it's working for you as well. If you give me push access to your reporoduction repo, I can commit the changes I made 🙂

Hey @almostprogrammer 👋,
It seems like this issue stems from graphql-tools. Would you mind opening up the issue there and reference this one? We can keep this one open as an indicator. Currently, graphql-binding doesn't opinionate error handling and we would like to keep it that way. Having graphql-tools team sort this out would benefit us greatly.
Hope you understand 🙂

You could consider switching within graphql-binding to graphql-tools-fork, where this bug has been fixed.

@yaacovCR why is the fork not merged into the official graphql-tools yet?

Not for me to say, would love it!

Is there an open PR?

I stopped submitting PRs as they were not reviewed. I have not seen the maintainers give input on any issues in quite a while. That is why I created the fork.
You could almost say that the project was abandoned, however there was a recent PR, possibly from someone internal to Apollo, that was submitted and merged within about a day.
All kind of mysterious. Probably has something to do with Apollo's move from schema stitching to federation, but I think that there would be no downside to fixing some of these bugs...

Could you try opening a PR nevertheless?

I am not sure worth the time of resolving the two different changelogs! If the maintainers of the upstream repository would chime in that they are interested, I would obviously feel differently.
Also, the code is public, though, so if you think it is worthwhile, please feel free!
And if you are interested in keeping the changelogs up to date, from what I have kept track, the fork addresses (among others?) graphql-tools issues 454, 480, 819, 862, 997, 1048, 1056, 1061, 1074, 1075, 1121, 1147, 1159, and 1171.

Anyways:
https://github.com/apollographql/graphql-tools/pull/1206

Thank you for reporting.
In the last few months, since the transition of many libraries under The Guild's leadership, We've reviewed and released many improvements and versions to graphql-cli, graphql-config and graphql-import.
We've reviewed graphql-binding, had many meetings with current users and engaged the community also through the roadmap issue.
What we've found is that the new GraphQL Mesh library is covering not only all the current capabilities of GraphQL Binding, but also the future ideas that were introduced in the original GraphQL Binding blog post and haven't come to life yet.
And the best thing - GraphQL Mesh gives you all those capabilities, even if your source is not a GraphQL service at all!
it can be GraphQL, OpenAPI/Swagger, gRPC, SQL or any other source!
And of course you can even merge all those sources into a single SDK.
Just like GraphQL Binding, you get a fully typed SDK (thanks to the protocols SDKs and the GraphQL Code Generator), but from any source, and that SDK can run anywhere, as a connector or as a full blown gateway.
And you can share your own "Mesh Modules" (which you would probably call "your own binding") and our community already created many of those!
Also, we decided to simply expose regular GraphQL, so you can choose how to consume it using all the awesome fluent client SDKs out there.
If you think that we've missed anything from GraphQL Binding that is not supported in a better way in GraphQL Mesh, please let us know!

In the context of that particular issue - thanks to the awesome work of @yaacovCR , GraphQL Mesh uses the new schema stitching introduced in GraphQL Tools v6.
We're looking forward for your feedback of how we can make your experience even better!


Add alias for Hue LWB004

Add this alias to map an additional model number for Hue Lux A19 bulb E27. When used with Zigbee2MQTT, the model is reported as Hue Lux A19 bulb E27 (433714) which based on the below is the same bulb as LWB004:
https://zigbee.blakadder.com/Philips_LWB004.html
https://www.zigbee2mqtt.io/devices/433714.html

@henworth Thanks for the addition!


How to handle audio focus change?

I'm trying to use your library to play YouTube videos, which are muted when no headphones are on or when user's music is playing. The problem is that even if I mute the video with youTubePlayer.setVolume(0) in onStateChange, music stops when the video starts playing. If I continue playing the music, the video stops.
I tried this code, but this doesn't help. Do you have any suggestions how this can be solved?
override fun onStateChange(youTubePlayer: YouTubePlayer, state: PlayerConstants.PlayerState) {
    if (state == PlayerConstants.PlayerState.BUFFERING) {
            val audioManager = context.getSystemService(Context.AUDIO_SERVICE) as AudioManager
            if (!Utility.isHeadsetConnected(context) || audioManager.isMusicActive) {
                audioManager.requestAudioFocus (object : AudioManager.OnAudioFocusChangeListener {
                    override fun onAudioFocusChange(focusChange: Int) {
                        if (focusChange == AudioManager.AUDIOFOCUS_LOSS) {
                            audioManager.abandonAudioFocus(this)
                        }
                    }
                }, AudioManager.STREAM_MUSIC, AudioManager.AUDIOFOCUS_GAIN)
                youTubePlayer.setVolume(0)
            }
        }
}

Hi, that's how Android handles multiple sources playing at the same time, by default: one gets paused.


Question about the distance in ply

Hi,Thanks for your greate work!
I find that when measure the distance near the postion i capture the image(~2m) is lager than position far away (~17m);
the near postion's difference with ground truth is about 0.1m;
the far postion's difference with ground truth is about 0.05m;
I am very confused about this?

Hi,Thanks for your greate work! I find that when measure the distance near the postion i capture the image(~2m) is lager than position far away (~17m); the near postion's difference with ground truth is about 0.1m; the far postion's difference with ground truth is about 0.05m; I am very confused about this?

It is not the common case (Further positions should suffer from larger errors). But in general I think the error in your case is acceptable (around 5%)?

Emmm,the far position is acceptable, the truth is ~1.12m, the .ply measured is ~1.17m, and the near position is had to say, the truth is ~0.42m, the .ply measured is ~0.56m

Emmm,the far position is acceptable, the truth is ~1.12m, the .ply measured is ~1.17m, and the near position is had to say, the truth is ~0.42m, the .ply measured is ~0.56m

If you can collect a small number of depth maps by depth sensors, we recommend that you use scale alignment or scale+shift alignment. Afterward, you could use such factors for further inference. We can do that because normally for the same camera, the error distribution should be similar.
If not, you might wait for a few weeks then. We will release stronger models.


No .shx and .dbf file for Mumbai wards

Could you please add .shx and .dbf file, so that the shapefiles could be put to use. Thank you.

Added, and also included geoJSON. https://github.com/HindustanTimesLabs/shapefiles/tree/master/city/mumbai/ward