Error: entry.content is required for byte cursors

I got this cryptic error from hyperdrive. It happened when I tried to use createReadFileStream on an archive loaded from dat.land.
Error: entry.content is required for byte cursors
    at /Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/cursor.js:128:37
    at Archive.get (/Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/archive.js:108:55)
    at Cursor._open (/Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/cursor.js:126:16)
    at open (/Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/cursor.js:29:10)
    at run (/Users/paulfrazee/my/beaker-browser/app/node_modules/thunky/index.js:13:3)
    at Cursor.open (/Users/paulfrazee/my/beaker-browser/app/node_modules/thunky/index.js:27:3)
    at Cursor._openAndNext (/Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/cursor.js:163:8)
    at Cursor.next (/Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/cursor.js:49:33)
    at read (/Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/archive.js:296:9)
    at /Users/paulfrazee/my/beaker-browser/app/node_modules/hyperdrive/archive.js:305:7

Here's what the entries listing looks like:
"[
  {
    "blocks": 2,
    "content": null,
    "ctime":<PHONE_NUMBER>188,
    "gid": 0,
    "length": 30266,
    "linkname": "",
    "mode": 0,
    "mtime":<PHONE_NUMBER>188,
    "name": "figure-1.1.png",
    "type": "file",
    "uid": 0,
    "path": "/figure-1.1.png"
  },
  {
    "blocks": 2,
    "content": null,
    "ctime":<PHONE_NUMBER>262,
    "gid": 0,
    "length": 22364,
    "linkname": "",
    "mode": 0,
    "mtime":<PHONE_NUMBER>262,
    "name": "figure-1.1a.png",
    "type": "file",
    "uid": 0,
    "path": "/figure-1.1a.png"
  },
  {
    "blocks": 3,
    "content": null,
    "ctime":<PHONE_NUMBER>329,
    "gid": 0,
    "length": 37023,
    "linkname": "",
    "mode": 0,
    "mtime":<PHONE_NUMBER>329,
    "name": "figure-1.2.png",
    "type": "file",
    "uid": 0,
    "path": "/figure-1.2.png"
  },
  {
    "blocks": 4,
    "content": null,
    "ctime":<PHONE_NUMBER>385,
    "gid": 0,
    "length": 53469,
    "linkname": "",
    "mode": 0,
    "mtime":<PHONE_NUMBER>385,
    "name": "figure-1.3.png",
    "type": "file",
    "uid": 0,
    "path": "/figure-1.3.png"
  }
]"

I'm having the exact same issue. Tried calling archive.get and archive.download before archive.createFileReadStream and neither had an effect

going to try to fix this with https://github.com/datproject/dat.land/issues/148

bump, I'm still getting this when I load dat.land

which version of hyperdrive was used to produce this archive do you know?

Whatever is being used in the current dat.land

(That is, the dat.land that's deployed)

Dat.land is using an old version of hyperdrive, we don't have an auto
deploy process
El sábado, 30 de julio de 2016, Paul Frazee<EMAIL_ADDRESS>escribió:

(That is, the dat.land that's deployed)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mafintosh/hyperdrive/issues/87#issuecomment-236393465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmotO5WzrFj7tYSNXH3KKN4_AwGnt5aks5qa9DKgaJpZM4JPOmm
.

--
Karissa McKelvey
http://karissa.github.io/
Dat Data
http://dat-data.com

fixed in 8


Use lambda wrapper scripts to auto-inject secrets as environment variables

Is your feature request related to a problem? Please describe.
This project still requires a fair bit of boilerplate / copied code in order to access secrets in a users lambda.  It would be nice if there was some way to securely make the secrets available as environment variables.
Describe the solution you'd like
As of 10/2020 lambda supports wrapper extensions seem to create the ability to wrap a runtime to inject environment variables.  It would be nice if we could optionally inject the cloud formation and implementation code into a users lambda that 'magically' makes the secrets available in their environment.
Describe alternatives you've considered
When layers were initially introduced, the possibility of this was discussed.  There are two other ways that this could potentially be introduced.  1. By implementing a custom runtime.  Or 2.  By injecting custom code into each runtime such that (runtime specific) it gets invoked.   Both of these solutions would require re-implementation of a solution for every runtime.  In addition, a custom runtime is potentially difficult to maintain.  For option 2, it may not even be possible for every runtime.  I'm aware of facilities in python to pre-load code (via site module), but it may not be possible for node or java.
Additional context
https://docs.aws.amazon.com/lambda/latest/dg/runtimes-modify.html
Specifically see the section on wrapper scripts.

AWS Compute Blog post on this topic:
https://aws.amazon.com/blogs/compute/creating-aws-lambda-environmental-variables-from-aws-secrets-manager/


Change the error message 500 for a wrong token

If I enter the token incorrectly I get an error message [Guard Token] etc.... and the call goes into error 500. But this in the case of the frontend should receive a 401. Is there a way to fix this problem

Are you able to send  a PR?
In any case you can “intercept” that error in your app the handle it.

Good morning,
how do I handle such a generic error with code 500 that can show up for anything and having to send a 401 to the frontend, it doesn't make sense...bye bye


Verify bytecode of instantiated ubeswap pair smart contract (from ubeswap factory)

This task is to verify that the bytecode from the instantiated ubeswap pair (from the deployed ubeswap factory) corresponds to the code at ubeswap's commit a88cca4007e57586a19ccfee3a0968dfbab5c736.

This is verified by doing the following:

Downloaded the ubeswap factory bytecode here: https://explorer.celo.org/address/0x62d5b84be28a183abb507e125b384122d2c25fae/contracts
2). Compile the ubeswap factory locally at the commit in the description (this will also compile the pair code, since it's a dependency on the factory code).
3). Removed the metadata hash from the end of the ubeswap pair bytecode.
4). Verify the resulting bytecode is embedded in the bytecode in step 1).


Nightmare zone glowing orb shadows

Describe the bug
The shadows of the pulsing, glowing orbs at the Nightmare Zone appear and disappear quickly as if they're flashing.
There is also no light coming off of those objects.
To Reproduce
Steps to reproduce the behavior:

Go to Nightmare Zone
See error

Expected behavior
Should these even have a shadow since they are light themselves? Rather than displaying a shadow (or trying to) should it just emit light instead?
Screenshots
https://gyazo.com/4a108e73cadb156f3738ac4f273d0d3e
Environment (please complete the following information):

OS: Windows
RuneLite version: 1.7.24
Launcher version: 2.1.3

An update was released today that should have fixed this. :)


Possible bug in HybridDircon

When trying to use HybridDircon on the Cassie model with springs, line 50 in dircon_position_data.cc and line 35 in dircon_distance_data.cc were causing issues. I'm not sure why it was working for the models without springs, but I think the line should be const auto x = this->plant_.GetPositionsAndVelocities(context) to retrieve the actual state.

The current code looks like an anachronism, from before GetPositionsAndVelocities existed (or I before I knew of it). It seems very reasonable to make the changes your requested.
However, I'm not sure why the current code doesn't work.  The code here is very similar, without some bells and whistles, to what Drake does under the hood for GetPositionsAndVelocities
https://github.com/DAIRLab/dairlib/blob/master/systems/trajectory_optimization/dircon_position_data.cc#L50-L51
From what I can tell, some reasons that it might fail, though there might be others, are:

The plant is discrete, not continuous.
There are actually multiple model instances in the plant.


Too Many Articles for Topics

There too many articles under each topic and they aren’t organized in any particular order - e.g there is a dump of information that isn’t organized on the page
Issue Submitted by Suzanne (SME)

@KellyeDC - Have there been any suggestions on how to organize.  I think this is worthwhile but we’d have to talk about how to organize before we ask Glorium to do any work.


Explain which back ends this works with somewhere obvious

I’ve spent maybe 15 minutes on the website, trying to determine which SQL backends this works with.
It’s not in the FAQ, not that I can see on any page on the website.
Seems like information that should be prominent.

It does. I feel like this is one of the first things someone would want to know, though, so I'd have it on the home page.

Yes, agree. We do have "By compiling to SQL, PRQL is instantly compatible with most databases, and existing tools or programming languages that manage SQL." in the first section — but maybe it's within too much text, or it's not clearly expressed? WDYT?


Cropping Notification and Navigation Bars

Using the cropping function to "hide" the notification bar is possible but I can't get the navigation bar to hide
Is there any way to do it?

Using the cropping function to "hide" the notification bar is possible but I can't get the navigation bar to hide

I'm not sure I understand the question.
Do you want the navigation bar to be cropped, or hidden on the device?
To crop it, suppose you have a device (in portrait) 1080×1920, with a navigation bar of 64px.
You want to remove the 64 top pixels, so you can call:
scrcpy --crop 1080:1856:0:64

On your screenshot, the notification bar height is 53px, while the navigation bar is 108px (the whole image is 1080×1920).
So your target height is 1080-53-108=919, starting at offset 53:
scrcpy --crop 1080:919:0:53

maybe it varies a bit because of the -m 720

The -m/--max-size value is computed after the cropping (so you need to crop relative to the original size).


Ensure HTTP endpoints are accessible outside the cluster

Following #113 we should ensure that the necessary HTTP endpoints for getting catalog contents are accessible from outside the cluster. A user shouldn't have to perform any magic to curl the endpoint containing a Catalog's contents.
In essence, a user should be able to fetch the catalog content via the command line by:

Running kubectl get catalog {catalogName} -o yaml to fetch Catalog details
Running curl {Catalog.Status.contentURL} to fetch all the contents of the Catalog

I was more so trying to convey that a user should be able to do a kubectl get catalog operatorhubio -o yaml and take the URL from the Catalog.Status and put it into a curl command like curl {catalog.status.url} and receive the catalog contents. No kubectl plugin required at this stage

I see, we're talking Ingress here then. That's fine.

What's proposed is likely not possible. The catalog's status.contentURL field will very likely be an https URL whose host:port is resolvable/routable only inside the cluster.
My assumption would be that we would document standard ways of exposing a Service's endpoints outside the cluster and then the curl command would use whatever endpoint was exposed. Options would include:

Using ingress. In this case, the URL that is curled would be resolvable by a DNS resolver outside the cluster, and then the ingress would reverse proxy to the underlying service.
Using a NodePort service. In this case, the URL that is curled would be to a cluster host running kube-proxy. The port would be a high-numbered port, likely not the same as the service port.

What's proposed is likely not possible. The catalog's status.contentURL field will very likely be an https URL whose host:port is resolvable/routable only inside the cluster.
My assumption would be that we would document standard ways of exposing a Service's endpoints outside the cluster and then the curl command would use whatever endpoint was exposed

That's a fair assumption. I didn't think too far into if this approach is possible, but if the way to go about this is to document a standard way to access the service's endpoints using something like curl that is fine as well

/assign @rashmigottipati

Thanks for the callout @rashmigottipati! Feel free to close directly when you see stuff like this.
And also (you're probably aware, but I know its easy to forget) there's some GH syntax you can put in PR descriptions/comments that will get GH to do this for you automatically: https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

@joelanford yeah, my bad, I forgot linking the GH issue in the PR description when I'd created it but added it after the PR was merged, so I don't think it was able to pick it up. :sweat_smile: And I was unable to close it directly as I don't see the "Close" button, I think it's because I am not a maintainer on this project.

I think it's because I am not a maintainer on this project.

Ah! Well we should fix that!

@joelanford thank you!


bad tilecounts in util.py

It's seems like tilecounts are outdated.
It must be like [1,1,1,40,40,80,80,320,1000,2000,2000,4000,8000,16000,16000,32000] .
Works fine with that numeric array.

Now fixed and thanks for your help~


[Cleanup] Remove unused hmap

This is a followup to address the issue raised in #688. It looks like we have an unused hmap which removing it will cleanup many actions

Maybe we can delete this -iquote as well, https://github.com/bazel-ios/rules_ios/blob/master/rules/library.bzl#L898

Closing because of failing tests and merge conflicts. Please rebase/reopen if this is still desired.


Add movie name if statement on search Query for modal and search function calls

Added if statement.  To check if search history buttons still work after an incorrect input


Add draw() method for trees

This adds the String draw() method to the Tree interface. It pretty prints a tree, like Data.Tree.drawTree from Haskell, in a nice two dimensional way.

Current coverage is 91.46%

Merging #949 into master will increase coverage by +0.01% as of d7b5f2e

@@            master    #949   diff @@
======================================
  Files           80      80
  Stmts         8819    8831    +12
  Branches      1706    1709     +3
  Methods          0       0
======================================
+ Hit           8065    8077    +12
  Partial        175     175
  Missed         579     579

Review entire Coverage Diff as of d7b5f2e

Powered by Codecov. Updated on successful CI builds.

Very nice addition :) Thx!


Get the Markdown text back from UITextView

I convert the markdown to HTML and put it in a UITextView, i need get the markdown text back from UITextView, how i get this text back?
Obs: I can´t get html text from UITextView and convert it in MarkDown back, because when i get the html from UITextView, the html have CSS styles, so i can´t convert in markdown.

You can't. You'll need to save the markdown separately.

Thanks.


[Documentation]: Custom Environment Variables

Overview
The documentation wasn't clear on how a developer could set custom environment variables in CRNA.  I updated the baseline template's README.md to reflect what I saw.  Please feel free to make any edits/suggestions for content.  Thanks!

@lucasbento made updates as suggested, and also added the TOC link.  Let me know if there's anything else you'd like in this content block.

@ericnograles thanks so much for the PR! And @lucasbento thanks for providing some feedback before I was able to drop by :grin:
I implemented this in the Expo tools CRNA uses under the hood, but this won't work yet if you eject without using ExpoKit. As a result, I don't think we want to steer people towards using this environment variable setup yet, because it will break immediately upon ejecting, and there're are also some rough edges around the packager cache that would need to be resolved first.
If it's alright with you I'd like to leave this open but not merge until we can work out a solution to expose these environment variables in upstream React Native. It's really important that code changes are very minimal (ideally nonexistent) to have your app continue working after ejecting.
cc @ericvicenti @mkonicek re: upstream support

@dikaiosune makes sense, and yes, perfectly fine with this.  Let me know if this is something I can assist with on the coding side as well, I'd just need some guidance on the affected modules and whatnot.

@ericnograles @dikaiosune
Many people like me want to store environment variables in .env file instead of passing on command line or package.json. May you give us the proper way to do it.
I have tried package like zoo , react-native-config but not working.

@nsisodiya I can definitely see how that'd be useful. Unfortunately, it would still be a problem to implement it for CRNA without also doing so for "upstream" React Native. In the meantime I'd recommend installing a tool like direnv which reads from an .envrc file to populate environment variables. Bonus: it works for all tools, not just ones that specifically read from a .env file!

not much activity here. feel free to use something like https://github.com/zetachang/react-native-dotenv with crna!


Improve Image Uploading Experience

Shows the base64 version of the image as a preview while the upload is in progress, and when it's done, replaces the base64 URL with the uploaded URL #36

@hernancito incredible work! Planning to take a look at this over the weekend!
P.S.: Huge fan of Aerolab btw – I visited the HQ in Buenos Aires and worked from there a couple of times! Really love the vibes & culture 😁

Incredible work @hernancito!! 👏
I'm trying this out now and it looks like the image is getting uploaded at the location of the cursor – and not at the location of the image drop:
https://github.com/steven-tey/novel/assets/28986134/1f3de3ef-6bed-46b0-b5f1-bab27c1edbd2

I fixed it by adding a 3rd prop to startImageUpload: pos that's the position where the image should be inserted

Just realized this doesn't handle invalid file types either:

Will think about it!

Incredible work on this @hernancito 👏 thank you so much for the PR!
Will be announcing this on Twitter tomorrow – stay tuned!

Nice catch @steven-tey I didn't test it dropping pictures, my bad. Thank you for the improvements

hey guys, thank you for your amazing works.
quick question: How can we handle removing the placeholder image when there is an error on uploading the image to the DB? I have tried various ways, but it seems a little bit tricky to me. The only way to remove the placeholder image now is to refresh which is kind of bad user experience.

Hi @lumpinif! I submitted a PR https://github.com/steven-tey/novel/pull/390 to address this scenario. The key is to remove the image placeholder when the promised is rejected.

Hi @lumpinif! I submitted a PR #390 to address this scenario. The key is to remove the image placeholder when the promised is rejected.

Nice work!
Appreciate it very much
Also, do we have a solution for something like onDeleteImage? Because we currently only have a plugin to upload images and get an image-upload function for uploading an image to the backend, however, I was trying to remove the image both from the client side and the backend side for database scaleable and billing reasons. But I found that is quite difficult for me to make that happen

In that case, you'll need to keep track of the data in your editor. Although deleting an image from the backend is not a great solution as a user can undo the deletion and the image (already cached in the browser) will be displayed in the editor but will not be saved in the database.

In that case, you'll need to keep track of the data in your editor. Although deleting an image from the backend is not a great solution as a user can undo the deletion and the image (already cached in the browser) will be displayed in the editor but will not be saved in the database.

fair point! thank you @hernancito

In that case, you'll need to keep track of the data in your editor. Although deleting an image from the backend is not a great solution as a user can undo the deletion and the image (already cached in the browser) will be displayed in the editor but will not be saved in the database.

just checked the PR https://github.com/steven-tey/novel/pull/390 still not merged, can't wait 👍

hey just merged the PR, thank your for your work @hernancito


Bridges not appearing in Port Piscarilius Sewers

Please check if your issue is not a duplicate by searching existing issues
Describe the bug
Bridges aren't appearing in the Port Piscarilius sewers.  These were discovered while doing the Queen of Thieves quest.
To Reproduce
Steps to reproduce the behavior:

Go to 'Port Piscarilius sewers'
Walk west.
See missing bridges (or don't in this case).

Expected behavior
Bridges to appear such that I know how to walk to the west to the sewer camp.
Screenshots
See screenshots here: https://imgur.com/a/ow9U7KG
Environment (please complete the following information):

OS: Windows 10
RuneLite version: 1.6.23
Launcher version: 2.1.0

turn off low detail

Perfect!  Issue solved!


Error running build.cmd

Trying to clone and build the Orchard2 repo for the first time... running build.cmd gives an error that it is "unable to find package 'KoreBuild'". I get the same when running this:
.nuget\nuget.exe install KoreBuild -ExcludeVersion -o packages -nocache -pre
VS2015 and DNVM setup and installed... but I must be missing something.

I experienced the same issue trying orchard2 for the first time late last week. It seems that aspnet vnext/core is changing significantly and is Lumley causing this and then other issues once you get past this one.
I know this doesn't help you per se but maybe it's good advice since all this tech is so new. I'm going to wait for aspnet core to buy rc2 before trying out orchard2.

Thanks @jeremycook that helps a lot actually. I thought that maybe I messed something up installing aspnet core for the first time. I'd like to get started with Orchard2 with some experimental sites, but don't have the resources to track down issues down to the vnext level right now.  Hopefully someone can come through here with a fix.

Fixing build when you move to dotnet cli. I make the transition to rc2

@sebastienros @Jetski5822  Maybe create a branch dev? (To work on unstable code)


Element.Insert AdjacentPosition.AfterEnd

I believe that when using Element.Insert with AdjacentPosition.AfterEnd and the referenced element is the last element of the parent, then an "index out of bounds" exception is thrown.
Dom/Element.cs line 570
Parent.InsertChild(Parent.IndexOf(this) + 1, nodes);
If 'this' is the last child of Parent, then that +1 makes it out of range in
Dom/Node.cs line 311
public INode InsertChild(Int32 index, INode child)
{
return this.PreInsert(child, _children[index]);  -- Line 311
}
I believe _children[index] will throw

Thanks for the report - I'll look into it!

Honestly, I forgot to look into this. Also I therefore skipped placing a Milestone (my bad!). I will take care of this in v0.9.7.

Awesome.  Thanks!

Fixed, see PR #335.


Add logpmf/pdf for all distributions except for PLN

PLN has a more complicated form than the others, and it will take me some more time to figure out if it's worthwhile to add the logpmf.

My current feelings about pln:
Correctness:

I believe the pln likelihood is currently correct

Length/complexity:

I believe the pln likelihood function could be shortened dramatically by replacing the two separate paths for large versus small counts (which are both kind of complicated in themselves) with a single Gauss-Hermite quadrature routine.
Shorter code is better than longer code, all else equal, but there's also some merit to "if it an't broke, don't fix it"

LogPMF:

I think the integral has to be done on the likelihood scale, not the log scale. I'm not sure there is a way to go straight to the log-likelihood.
My recollection is that pln has such long tails that we won't often get super-tiny likelihoods. For this reason, I think an explicit logpmf is less important than for the other distributions.  It would be good to double-check that, though

Thanks for the insights! I don't know enough about Gauss-Hermite quadrature to comment, but I'm with you on shorter code is better code.
It would definitely be good to double check. I'll leave it as it is for now, and come back to it when the other more preeminent issues are fixed.

I've read through the code and all of the "code" changes look good to me. I haven't checked the optimizer changes or the equations for the pmfs since @davharris will be able to do a better job with those anyway.
@davharris - when you're happy with this things I think this is ready to go in.
Thanks to both of your for all of your work on this.

I don't see any problems.  Should we write a few tests before merging, though?

Sorry for being the time-limiting step here. I did some rudimentary tests with made-up numbers to make sure that the functions returned the same outputs before/after the changes, but it's probably a good idea to do it more systematically. We already have the tests for pmf and cdf of PLN, gleaned from the original paper. @davharris what would you suggest for the other distributions and the solvers? Should we try to replicate the functions in R?

Replicating in R sounds good to me, unless it substantially complicates the testing infrastructure.
Dave

On Mar 7, 2016, at 12:56 PM, Xiao Xiao<EMAIL_ADDRESS>wrote:
Sorry for being the time-limiting step here. I did some rudimentary tests with made-up numbers to make sure that the functions returned the same outputs before/after the changes, but it's probably a good idea to do it more systematically. We already have the tests for pmf and cdf of PLN, gleaned from the original paper. @davharris what would you suggest for the other distributions and the solvers? Should we try to replicate the functions in R?
—
Reply to this email directly or view it on GitHub.

Looks like there are still numeric problems with the negative binomial solver.
import macroeco_distributions as md

ab = [42L, 21L, 7L, 2L, 21L, 2L, 3L, 1L, 3L, 7L, 2L, 8L, 6L, 7L,
2L, 2L, 1710L, 38L, 8L, 2L, 5L, 72L, 50L, 6L, 208L, 6L, 24L,
4L, 3L, 1L, 2L, 1L, 40L, 52L, 503L, 50L, 47L, 232L, 100L, 30L,
95L, 8L, 4L, 67L]

md.nbinom_lower_trunc_solver(ab)

md.nbinom_lower_trunc_solver(ab)
/Users/davidharris/anaconda/lib/python2.7/site-packages/macroeco_distributions/macroeco_distributions.py:611: RuntimeWarning: overflow encountered in exp
  return -nbinom_lower_trunc_ll(ab, exp(x[0]), expit(x[1]))
/Users/davidharris/anaconda/lib/python2.7/site-packages/scipy/stats/_discrete_distns.py:167: RuntimeWarning: invalid value encountered in subtract
  coeff = gamln(n+x) - gamln(x+1) - gamln(n)
(6.9645032936894593e-16, 1.8722355604094302e-09)

I don't know why scipy's optimizers get stuck in these low-likelihood areas. My R code finds the optimum at n=1.84667069193733e-06, p=0.00202186468733917, which is about 50 log-likelihood points better.  Maybe we need better starting values for these extreme cases?

It does look like the starting values aren't very good for the case I posted in my previous comment.  Using the code I linked to, I get a starting value of 0.99887 as the starting value for p0, when the optimum as at p=0.0020.
I've also confirmed that---at least for this abundance vector---I get good results starting p0 closer to 0.001 instead of .99887.

The starting value is consistent with that in fitdistr() in R though. Given how sensitive this optimization is to the starting value, I'm not sure if fixing the starting value at 0.001 would not cause problems when p is at the other end.
The negative binomial in R also takes a different form (mu, size instead of n, p). Do you think that would make any difference (making it easier or harder for the optimizer)?

Okay, thanks.  I was wondering where those starting values came from.  To clarify, I wasn't suggesting that we fix the starting value there, just noting that it worked better as a starting value for this particular problem.
Given that the negative binomial is known to be hard to optimize, I wouldn't be opposed to trying several starting values and returning the best set of results.
I don't know which parameterization would be better in general.  We could probably figure it out by looking at the condition number, but I forget exactly what that means.  Maybe it's the ratio of the first eigenvalue of the Hessian matrix to the second eigenvalue?

I'm afraid that I'm not familiar with the condition number; but having
multiple starting values sounds like a brilliant idea. Let me modify the
function to include all three values (one computed from data, one close to
zero, one close to 1), and see how that works.
Thanks for doing the tests!
On Tue, Mar 8, 2016 at 2:08 PM, David J. Harris<EMAIL_ADDRESS>wrote:

Okay, thanks. I was wondering where those starting values came from. To
clarify, I wasn't suggesting that we fix the starting value there, just
noting that it worked better as a starting value for this particular
problem.
Given that the negative binomial is known to be hard to optimize, I
wouldn't be opposed to trying several starting values and returning the
best set of results.
I don't know which parameterization would be better in general. We could
probably figure it out by looking at the condition number, but I forget
exactly what that means. Maybe it's the ratio of the first eigenvalue of
the Hessian matrix to the second eigenvalue?
—
Reply to this email directly or view it on GitHub
https://github.com/weecology/macroecotools/pull/39#issuecomment-193920436
.

@davharris - I think the new function works for the example you posted. If you run into other issues, please let me know.
@ethanwhite - what's the level of robustness we'd want in tests for this module? Do we want to make sure that no error is introduced with each update (then having 5 or so random values for each distribution/solver/likelihood function from R is probably fine)? Or do we want to check the extreme cases as well (in which case we'd probably want to add the cases from sad-comparisons that caused us trouble)?

@rueuntal If we can fix #40, we might not need multiple starting values.  Multiple starting values certainly wouldn't hurt, though.

@rueuntal I vote for including some extreme cases in the test suite.

I added three simple test cases for the pmf/pdf and cdf of each distribution. These are more like "sanity checks", making sure that the distributions behave as expected if/when we make changes. All tests have passed.
There are still no tests on the solvers though, which are probably more likely to be broken by future updates. I'm hoping that @davharris has already coded up some of them in R so that I don't have to duplicate the work :) Dave would you mind putting a few tests in if you already have the code?

@davharris - can you give this PR and discussion another look through and see if it's good to go in. Since it's the only open PR it would be nice if we could merge it before tagging 0.3 and publishing on Zenodo.

In case cleaning is important
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R616
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R537
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R236
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R242
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R239
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R236
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R233
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R212
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R607
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R608
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R616
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R173

https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R232

to

https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R242

It would be nice to include a default return.
but it is also perfect assuming only these two cases happen.
def _argcheck(self, mu, sigma, lower_trunc):
	if lower_trunc is True:
		self.a = 1
	else:
		self.a = 0
	return sigma > 0

Hi @henrykironde  - what do you mean by a default return?

sorry  I missed a link
https://github.com/weecology/macroecotools/pull/39/files#diff-0c22aa27a8370a07c9f00d7c8e7f53a2R173

    def _logpmf(self, x, p, upper_bound):
        x = np.array(x)
        if p[0] < 1:
            return stats.logser.logpmf(x, p) - stats.logser.logcdf(upper_bound, p)
        else:
            ivals = np.arange(1, upper_bound[0] + 1)
            normalization = sum(p[0] ** ivals / ivals)
            logpmf = x * log(p[0]) - log(x) - log(normalization)
            return logpmf

def logpmf(p):

    if p < 1:
        return "yes"
    else:
        return "not"
    return "this value may need to be debugged"

#test drive
logpmf(None)

But this may not be important if you are sure that p[0] will always be a number

Ah I see. Are you trying to catch the case where any of the inputs are not numeric?
In that case I'd be inclined to keep the current version, which throws a TypeError if that happens (this is also the style of scipy.stats).

Wow, I haven't thought about this in a long time.  I'll see what I can do.

Okay, here are my thoughts:

[ ] Could we get the R code used to calculate the test values (or at least a more complete description of what was done? The Pareto distribution, for example, isn't in the stats package in R, so it would be good to know which function was used.  Even better, could the tables be generated in R and saved as a CSV or something? Copy-pasting from the R console to the test suite makes me a little jumpy.

[ ] There's a lot of repetitive code in the test_distribution_Xdf functions like test_nbinom_lower_trunc_cdf and test_trunc_weibull_pdf. Could some of that be abstracted away? It would be a lot less brittle and much easier to read.

I think these changes might work well together (e.g. if all the R results are in a single table, it might be easier to write the tests with a single function).

Hi @davharris - Good call. Let me get those fixed.

Sorry I'm not familiar with Travis CI. What does the above error message mean?

@rueuntal, I think that is just Travis failing occasionally.
I don't see anything that you changed leading to this failure.
try git commit --amend save and force push with -f. git push -f ......
...... represents whatever branch you where pushing eg. git push origin master ->  git push -f origin master

I’ve restarted Travis’s tests, so I don’t think the amend/force push will be necessary.

On Aug 11, 2016, at 1:17 AM, Henry Senyondo<EMAIL_ADDRESS>wrote:
@rueuntal https://github.com/rueuntal, I think that is just Travis failing occasionally.
I don't see anything that you changed leading to this failure.
try git commit --amend save and force push with -f. git push -f ......
...... represents whatever branch you where pushing eg. git push origin master -> git push -f origin master
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/weecology/macroecotools/pull/39#issuecomment-239075814, or mute the thread https://github.com/notifications/unsubscribe-auth/AAzdCefW0RHy9hkBVESL9WM5LwOHt1OLks5qerCHgaJpZM4Hj93W.

Nope.  Still failing.  Not sure if this is the reason or not, but it might be related: https://www.traviscistatus.com/incidents/11hp8bhkrkn7 https://www.traviscistatus.com/incidents/11hp8bhkrkn7

On Aug 11, 2016, at 1:20 AM, Dave Harris<EMAIL_ADDRESS>wrote:
I’ve restarted Travis’s tests, so I don’t think the amend/force push will be necessary.

On Aug 11, 2016, at 1:17 AM, Henry Senyondo<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
@rueuntal https://github.com/rueuntal, I think that is just Travis failing occasionally.
I don't see anything that you changed leading to this failure.
try git commit --amend save and force push with -f. git push -f ......
...... represents whatever branch you where pushing eg. git push origin master -> git push -f origin master
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/weecology/macroecotools/pull/39#issuecomment-239075814, or mute the thread https://github.com/notifications/unsubscribe-auth/AAzdCefW0RHy9hkBVESL9WM5LwOHt1OLks5qerCHgaJpZM4Hj93W.

yap, I did restart it too and it failed. Let me check the last time this repo was  run.
cos you mentioning sudo, reminds me that there were some changes on how and when to use sudo

okey i think this is it
https://www.traviscistatus.com/incidents/11hp8bhkrkn7

Thanks everyone. I tried the simple fix for:
https://www.traviscistatus.com/incidents/11hp8bhkrkn7
in #41, but that didn't work.
It looks to me like the instructions for using conda have changed a fair bit since we set it up in this repo:
http://conda.pydata.org/docs/travis.html
So the .travis.yml probably needs to be updated.

This is probably a super simple bug - what's the directory for the input test_data.csv file? Neither 'test_data.csv' nor './test_data.csv' worked.

Very cool.  Do we still want to add tests for the solvers before merging?

This is probably a super simple bug - what's the directory for the input test_data.csv file? Neither 'test_data.csv' nor './test_data.csv' worked.
My guess is that you want macroeco_distributions/test_data.csv.

Great! The checks have finally passed, thanks everyone!

Do we still want to add tests for the solvers before merging?
What's your feeling for it?

Seems like a good idea.    I'm leaning yet.  But I don't think it has to be on this PR, so I think we can merge.

Yeah I agree, especially given python's suboptimal behavior for optimization.
I'll go ahead and merge this PR then, and start an issue to discuss our options regarding the solvers.


vendorize dependencies using Godep

tracking dependencies is hard, that's why there is a tool called godep (https://github.com/tools/godep) since go1.5 which takes care of it.
this fork contains the required dependencies tracked in the Godeps.json and additionally everything in vendor/ as suggested by the godep documentation.

add Godeps file to track dependencies
commit dependencies in vendor/ folder

I love it. +1 for vendoring.

I don't like it.
go get installs dependencies.  It's enough for libraries.
I use vendoring only for building executable, not libraries.

i'm also still not sure about this, as i had installation issues when i installed it first, but it worked after removing everything and go get it again.
Also the point regarding vendoring of executables vs. libraries sounds valid.

Presumably this would require pushing the use of godep on users of logrus?
I use Glide, an alternative to godep.
I'd sooner not have to keep 2 different vendoring tools up to date.
I agree with the stance on only vendoring executables.
Perhaps if dependency versions are critical for logrus releases then they could somehow be pinned down in a Makefile?

On 13 Mar 2016, at 20:54, Christian Becker<EMAIL_ADDRESS>wrote:
i'm also still not sure about this, as i had installation issues when i installed it first, but it worked after removing everything and go get it again.
Also the point regarding vendoring of executables vs. libraries sounds valid.
—
Reply to this email directly or view it on GitHub.

10,000s of clones are done of this repo every day. Each clone would now include all these test-related dependencies and deploy them to their production environments. None of these are run-time requirements.
Thanks for the suggestion though.


[Snyk] Fix for 28 vulnerabilities

Snyk has created this PR to fix one or more vulnerable packages in the `rubygems` dependencies of this project.
Changes included in this PR

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

alpha-blog/Gemfile
alpha-blog/Gemfile.lock

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

539/1000  Why? Has a fix available, CVSS 6.5
Cross-site Request Forgery (CSRF) SNYK-RUBY-ACTIONPACK-569599
No
No Known Exploit

539/1000  Why? Has a fix available, CVSS 6.5
Information Exposure SNYK-RUBY-ACTIONPACK-569600
No
No Known Exploit

539/1000  Why? Has a fix available, CVSS 6.5
Cross-site Scripting (XSS) SNYK-RUBY-ACTIONVIEW-560837
No
No Known Exploit

479/1000  Why? Has a fix available, CVSS 5.3
Cross-site Request Forgery (CSRF) SNYK-RUBY-ACTIONVIEW-569601
No
No Known Exploit

449/1000  Why? Has a fix available, CVSS 4.7
Cross-site Scripting (XSS) SNYK-RUBY-ACTIONVIEW-632514
No
No Known Exploit

834/1000  Why? Mature exploit, Has a fix available, CVSS 8.1
Deserialization of Untrusted Data SNYK-RUBY-ACTIVESUPPORT-569598
No
Mature

539/1000  Why? Has a fix available, CVSS 6.5
Cross-site Scripting (XSS) SNYK-RUBY-BOOTSTRAPSASS-174549
No
No Known Exploit

539/1000  Why? Has a fix available, CVSS 6.5
Cross-site Scripting (XSS) SNYK-RUBY-BOOTSTRAPSASS-450237
No
No Known Exploit

539/1000  Why? Has a fix available, CVSS 6.5
Cross-site Scripting (XSS) SNYK-RUBY-BOOTSTRAPSASS-450238
No
No Known Exploit

539/1000  Why? Has a fix available, CVSS 6.5
Cross-site Scripting (XSS) SNYK-RUBY-BOOTSTRAPSASS-450239
No
No Known Exploit

636/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 6.3
Cross-site Scripting (XSS) SNYK-RUBY-JQUERYRAILS-565439
No
Proof of Concept

679/1000  Why? Has a fix available, CVSS 9.3
Denial of Service (DoS) SNYK-RUBY-JSON-560838
No
No Known Exploit

579/1000  Why? Has a fix available, CVSS 7.3
Cross-site Scripting (XSS) SNYK-RUBY-LOOFAH-474102
No
No Known Exploit

344/1000  Why? Has a fix available, CVSS 2.6
XML External Entity (XXE) Injection SNYK-RUBY-NOKOGIRI-1055008
No
No Known Exploit

619/1000  Why? Has a fix available, CVSS 8.1
Command Injection SNYK-RUBY-NOKOGIRI-459107
No
No Known Exploit

659/1000  Why? Has a fix available, CVSS 8.9
Uncontrolled Memory Allocation SNYK-RUBY-NOKOGIRI-534637
No
No Known Exploit

600/1000  Why? Has a fix available, CVSS 7.5
Denial of Service (DoS) SNYK-RUBY-NOKOGIRI-552159
No
No Known Exploit

624/1000  Why? Has a fix available, CVSS 8.2
Denial of Service (DoS) SNYK-RUBY-PUMA-536835
No
No Known Exploit

479/1000  Why? Has a fix available, CVSS 5.3
HTTP Response Splitting SNYK-RUBY-PUMA-559020
No
No Known Exploit

479/1000  Why? Has a fix available, CVSS 5.3
HTTP Response Splitting SNYK-RUBY-PUMA-559100
No
No Known Exploit

550/1000  Why? Has a fix available, CVSS 6.5
HTTP Request Smuggling SNYK-RUBY-PUMA-570205
No
No Known Exploit

550/1000  Why? Has a fix available, CVSS 6.5
HTTP Request Smuggling SNYK-RUBY-PUMA-570206
No
No Known Exploit

479/1000  Why? Has a fix available, CVSS 5.3
Information Exposure SNYK-RUBY-RACK-538324
No
No Known Exploit

589/1000  Why? Has a fix available, CVSS 7.5
Directory Traversal SNYK-RUBY-RACK-569066
No
No Known Exploit

646/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 6.5
Cross-site Request Forgery (CSRF) SNYK-RUBY-RACK-572377
No
Proof of Concept

686/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 7.3
Arbitrary Code Injection SNYK-RUBY-RAKE-552000
No
Proof of Concept

635/1000  Why? Has a fix available, CVSS 8.2
Denial of Service (DoS) SNYK-RUBY-RUBYZIP-469156
No
No Known Exploit

731/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 8.2
Regular Expression Denial of Service (ReDoS) SNYK-RUBY-WEBSOCKETEXTENSIONS-570830
No
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
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

fix issues

fix issues


ui2 sync node version in travis testing with compose setup

Feature Request
Desired behaviour
It seems that ui2 travis tests may be testing a different version of node than our deployment.
Please downgrade and test the deployment version temporarily.
┆Issue is synchronized with this Jira Story
┆Issue Type: Story
┆Issue Number: DOCK-606

Node version can be downgraded in a separate branch that never gets merged... but it's a mystery how Travis gets a global @angular/cli without installing one

There's a way to install and run Angular CLI non-globally. I think we should ideally do that, as today there is no guarantee that all of us are running the same Angular CLI locally.
Definitely post 1.6 though.

Oh, we're installing it non-globally.  The problem is that we need a global version (pretty much any version) to run ng commands (which the local one then overrides)

I think you can do it without even the global version: https://medium.com/@starikovs/how-to-use-angular-cli-locally-729dbb6707dd

Yes, now we run into the problem of install npx globally :)

Isn't npx part of npm, and we already are installing that globally.
But, good point about needing to change all the commands.
And if global Angular CLI is using the local Angular CLI version anyway, which I didn't realize, then never mind. :)

npx is only parts of certain npm versions, no idea which ones.  The global CLI uses the local CLI is most versions I can think of, but there's no guarantee

be3c0478800dafbc0fb7a074d329338cc4bf240a to be reverted

Does this even need to be "tested" or should we just change this issue to something that reminds us to revert it later.

Does this even need to be "tested" or should we just change this issue to something that reminds us to revert it later.

Doesn't really need review. Create a new issue for 1.7.0 to revert these

https://github.com/ga4gh/dockstore/issues/2279


Fix border bottom and button

What is issue is being fixed? #34
What changed?
I added missing border bottom to contact us section.  Also, added darker color on quick links at footer matching nav background color and styled send button on contact us form according to request.
Please see on the screenshot below

@KanjiyaJinesh Please revise and approve if requirements are being met.  Thank you in advance!

LGTM


Add Columns in Sidebar

Hello,
How can I add columns in sidebar?
I am trying to achieve this functionality:

columns with a timeline that expands and collapses
thanks

Hi @sabina4 , re: the columns in sidebar, take a look at the readme as it pertains to:
sidebarContent prop - component that allows you to control whats rendered at top of left sidebar
groupRenderer prop - component that allows you to control whats displayed for each item in the left sidebar

@mcMickJuice i tried the groupRenderer prop however it doesn't take into consideration the content size for the line height to grow and shrink accordingly.

here is a link that briefly shows the problem
https://codesandbox.io/s/o4pzy4mlz6


Location of the ECS Constant -- ship NEVER leaves

Location of the ECS Constant -- ship NEVER leaves Porrima, no matter how much in game time has passed since you hailed the ship and told them it's time for them to move on before the Paradiso Group gets upset. Not sure if this is a script bug or a quest design bug. This has been the case since game launch has not yet been fixed by Bethesda. If side quests are necessary to make the ship leave Porrima, there are no in-game hints to make it progress.

Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
Yes
Reproduction Steps
Do the Paradiso quest, and once you get the "Location of the ECS Constant" activity, hail the ship and tell it to move on. No matter how much in-game time passes, the activity will forever be in your quest log and the ship will never leave Paradiso.

#431


Ignore not found error when rotating log file

Do not return not found error when log file was deleted before rotate.
Signed-off-by: Xiaoguang Sun<EMAIL_ADDRESS>
What have you changed? (mandatory)
Do not return error when log file is deleted before rotating. Although I doubt if we really need to return any error at here. It is definitely better to be conservative and only skip std::io::ErrorKind::NotFound at this time.
Please explain IN DETAIL what the changes are in this PR and why they are needed:

Summarize your change (mandatory)
TiKV is going to panic during rotating if current log file is deleted. This PR changes log rotating behavior and ignore NotFound error kind.

Please NOTE that:

Do not assume reviewers understand the original issue

What are the type of the changes? (mandatory)

Misc (other changes)
NA

How has this PR been tested? (mandatory)
Unit test has been updated to validate the change.
Does this PR affect documentation (docs) or release note? (mandatory)
No

If there is a document change, please file a PR in ([docs]
No
If this PR should be mentioned in the release note, please update the [release note]
No

Does this PR affect tidb-ansible update? (mandatory)
No
If there is a configuration or metrics change, please file a PR in [tidb-ansible]
No
Refer to a related PR or issue link (optional)
NA
Benchmark result if necessary (optional)
NA
Add a few positive/negative examples (optional)
NA

Test failed mysteriously at not related part. How to trigger a new run of it please?

Test failed mysteriously at not related part. How to trigger a new run of it please?

Merging from master did the trick.

/run-integration-tests

/run-integration-tests


Batching/scheduling

This PR (originally part of PR #266) implements the logic that allows for queries to be batched together automatically. It is still WIP and I will now be implementing the changes as discussed with @helfer.
TODO:

[ ] Update CHANGELOG.md with your change
[ ] Make sure all of the significant new logic is covered by tests
[ ] Rebase your changes on master so that they can be merged easily
[ ] Make sure all tests and linter rules pass

Can you rebase this on top of master now that the other PR is merged? It looks like they have some overlap now.

I assume the reason it's done as part of the network interface is because when we build batching at the transport level, we'll want to have a special network interface for that transport?

Yes, the idea is that different servers may not all support the same kind of batching. There's a "batcher" in the client that batches up requests and sends them to the network interface at the same time, but it's up to the network interface to decide whether it sends the queries batched or not.
Scheduling for polling queries is going to be another separate module, which just aligns polling intervals.

Fixed.


Natural Neighbor Interpolation

So help me, I'm finally going to climb/conquer this mountain. Notes:

The scipy.spatial module has Delaunay triangulation: https://docs.scipy.org/doc/scipy-0.17.0/reference/generated/scipy.spatial.Delaunay.html (based on QHull)
There is also high quality Delaunay triangulation from the Python triangles package: https://github.com/drufat/triangle (based on http://dzhelil.info/triangle/).

Other useful links for when I circle back here:

newer Natural Neighbor paper
https://blancosilva.github.io/post/2014/10/28/Computational-Geometry-in-Python.html

scipy.intepolate.griddata might have something similar using the 'linear' option.  Specifically it calls the cython function LinearNDInterpolator:
http://docs.scipy.org/doc/scipy/reference/generated/scipy.interpolate.LinearNDInterpolator.html#scipy.interpolate.LinearNDInterpolator
https://github.com/scipy/scipy/blob/v0.17.0/scipy/interpolate/interpnd.pyx
Could be something to explore, but maybe not exactly the same.

Regardless, a good option to exploit in some fashion (even if only an example).

@ahaberlie has a working implementation. The nice part is no NCAR license weirdness; it just might need some further optimization for runtime.

Implemented in #186.


add docs on using H3_ALLOC_PREFIX

Coverage remained the same at 99.124% when pulling 866670cc36250538cbeae06d28e81aa0def90378 on isaacbrodsky:alloc-docs into 029a3b79e15dc190740710b6db5513504482d004 on uber:master.


I implemented an Android client.

https://github.com/wuhongsheng/S2SAndroidClient

This is so cool!


18.6.0

Codecov Report

Merging #67 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #67   +/-   ##
=======================================
  Coverage   49.73%   49.73%
=======================================
  Files           9        9
  Lines         189      189
  Branches        2        2
=======================================
  Hits           94       94
  Misses         95       95

Continue to review full report at Codecov.

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


[forms] Update "Pay with" UI on RETIRE form (pool listing)

Currently on a Pool Listing, the Pay with: option on the /retire form is a drop down:

The objective of this ticket is to change the UI to present both pay with options so they are always visible and more intuitive to use.
There are two states that need to be considered, when the user is logged in or not.
Logged In
The entire flow is here.
UI updated design here.

Selected state.

NOTE

The RETIRE CARBON button is now active.

Logged Out - Pay with CC
The entire flow is here.
UI updated design here.

NOTE:

CTA to Login to pay with USDC moved to the pay with option
The current SIGN IN / CONNECT TO BUY button is replaced with RETIRE CARBON

Selected state.

NOTE:

CC is selected and RETIRE CARBON button now active (i.e. there is no requirement to login to CM to retire carbon with CC)

Logged Out - Pay with USDC
The entire flow is here.
Selected state

NOTE:

User is logged in after wallet connect step
USDC is selected
RETIRE CARBON button is active

@0xMakka hopefully the design flows and my break down above are clear. Let me know if any questions.
One note, this change is for /retire form from a pool listing. I still need to get professor to update the /buy flow. And then we will need to update for seller listings down the road as well.


Unable to load Asset - wrong indentation in pubspec.yaml

Steps to Reproduce
I downloaded some images to put into my app as Assets instead of Network images. I saved those images in a directory on my project called "Images". I saved those images as assets in the pubspec.yaml
assets:
  - images/default.png
  - images/cat.jpg.

I try to load the cat.jpg image with this line of code new Image(image: new AssetImage('images/cat.jpg')). But I get this error
══╡ EXCEPTION CAUGHT BY SERVICES ╞═════════════════════════════════
The following assertion was thrown resolving a single-frame image stream:
Unable to load asset: images/cat.jpg
When the exception was thrown, this was the stack:
#0      PlatformAssetBundle.load (package:flutter/src/services/asset_bundle.dart:221:7)
<asynchronous suspension>
#1      AssetBundleImageProvider.loadAsync (package:flutter/src/services/image_provider.dart:370:44)
<asynchronous suspension>
#2      AssetBundleImageProvider.load (package:flutter/src/services/image_provider.dart:356:7)
#3      ImageProvider.resolve.<anonymous closure>.<anonymous closure> (package:flutter/src/services/image_provider.dart:253:61)
#4      ImageCache.putIfAbsent (package:flutter/src/services/image_cache.dart:82:22)
#5      ImageProvider.resolve.<anonymous closure> (package:flutter/src/services/image_provider.dart:253:38)
(elided 8 frames from package dart:async)
Image provider: AssetImage(bundle: null, name: "images/cat.jpg")
Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#f1859(), name: "images/cat.jpg", scale:
1.0)
══════════════════════════════════════════════════════════════════

Network images load fine but any assets don't.
Logs
flutter run -d all
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Running 'gradle assembleDebug'...                     1.3s
Built build/app/outputs/apk/app-debug.apk (21.6MB).
Installing build/app/outputs/apk/app.apk...           1.8s
Launching lib/main.dart on iPhone 7 Plus in debug mode...
I/flutter ( 9705): ══╡ EXCEPTION CAUGHT BY SERVICES ╞══════════════════════════════════════════════════════════════════
I/flutter ( 9705): The following assertion was thrown resolving a single-frame image stream:
I/flutter ( 9705): Unable to load asset: images/cat.jpg
I/flutter ( 9705):
I/flutter ( 9705): When the exception was thrown, this was the stack:
I/flutter ( 9705): #0      PlatformAssetBundle.load (package:flutter/src/services/asset_bundle.dart:221)
I/flutter ( 9705): <asynchronous suspension>
I/flutter ( 9705): #1      AssetBundleImageProvider.loadAsync (package:flutter/src/services/image_provider.dart:370)
I/flutter ( 9705): <asynchronous suspension>
I/flutter ( 9705): #2      AssetBundleImageProvider.load (package:flutter/src/services/image_provider.dart:356)
I/flutter ( 9705): #3      ImageProvider.resolve.<anonymous closure>.<anonymous closure> (package:flutter/src/services/image_provider.dart:253)
I/flutter ( 9705): #4      ImageCache.putIfAbsent (package:flutter/src/services/image_cache.dart:82)
I/flutter ( 9705): #5      ImageProvider.resolve.<anonymous closure> (package:flutter/src/services/image_provider.dart:253)
I/flutter ( 9705): (elided 8 frames from package dart:async)
I/flutter ( 9705):
I/flutter ( 9705): Image provider: AssetImage(bundle: null, name: "images/cat.jpg")
I/flutter ( 9705): Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#b3e4e(), name: "images/cat.jpg", scale:
I/flutter ( 9705): 1.0)
I/flutter ( 9705): ════════════════════════════════════════════════════════════════════════════════════════════════════
Running Xcode build...
Jul 13 10:07:53 Admins-MacBook-Pro com.apple.CoreSimulator.SimDevice.2A665704-95E9-4CD2-86DF-9510A136125F.launchd_sim[47623] (com.apple.videosubscriptionsd): Service only ran for 0 seconds. Pushing respawn out by 10 seconds.
CoreSimulatorBridge: Requesting installation of file:///Users/zeke/IdeaProjects/flutter_shopping/build/ios/Release-iphonesimulator/Runner.app/ with options: {
            CFBundleIdentifier = "com.yourcompany.flutterShopping";
            PackageType = Developer;
            SimulatorRootPath = "/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator.sdk";
            SimulatorUserPath = "/Users/zeke/Library/Developer/CoreSimulator/Devices/2A665704-95E9-4CD2-86DF-9510A136125F/data";
        }
Jul 13 10:08:04 Admins-MacBook-Pro com.apple.CoreSimulator.SimDevice.2A665704-95E9-4CD2-86DF-9510A136125F.launchd_sim[47623] (com.apple.videosubscriptionsd): Service only ran for 0 seconds. Pushing respawn out by 10 seconds.
CoreSimulatorBridge: Requesting launch of com.yourcompany.flutterShopping with options: {
            arguments =     (
                "--enable-dart-profiling",
                "--flx=/Users/zeke/IdeaProjects/flutter_shopping/build/app.flx",
                "--dart-main=/Users/zeke/IdeaProjects/flutter_shopping/lib/main.dart",
                "--packages=/Users/zeke/IdeaProjects/flutter_shopping/.packages",
                "--enable-checked-mode",
                "--observatory-port=8108",
                "--diagnostic-port=8109"
            );
            environment =     {
            };
        }
CoreSimulatorBridge: Beginning launch sequence for bundle 'com.yourcompany.flutterShopping'
                retryTimeout: 120.000000 (default write com.apple.CoreSimulatorBridge LaunchRetryTimeout <value>)
                bootTimeout: 300.000000 (default write com.apple.CoreSimulatorBridge BootRetryTimeout <value>)
                bootLeeway: 120.000000 (default write com.apple.CoreSimulatorBridge BootLeeway <value>)
                Note: Use 'xcrun simctl spawn booted defaults write <domain> <key> <value>' to modify defaults in the booted Simulator device.
        Simulator booted at: 2017-07-13 14:59:27 +0000
        Current time: 2017-07-13 16:08:04 +0000
        Within boot leeway: NO
CoreSimulatorBridge: Launch attempt #0 for bundle 'com.yourcompany.flutterShopping', have been trying to launch for 0 seconds. Modern launch services = YES
CoreSimulatorBridge: Got LSBundleProxy for 'com.yourcompany.flutterShopping', sequenceNumber 272, cacheGUID 8461006D-5286-45F2-849F-2F22600D1BBB
CoreSimulatorBridge: [Common] [FBSSystemService][0x5217] Sending request to open "com.yourcompany.flutterShopping"
CoreSimulatorBridge: [Common] [FBSSystemService][0x5217] Request successful: <BSProcessHandle: 0x7f80a4146df0; Runner:69456; valid: YES>
CoreSimulatorBridge: Launch successful for 'com.yourcompany.flutterShopping'
You've implemented -[<UIApplicationDelegate> application:didReceiveRemoteNotification:fetchCompletionHandler:], but you still need to add "remote-notification" to the list of your supported UIBackgroundModes in your Info.plist.
══╡ EXCEPTION CAUGHT BY SERVICES ╞══════════════════════════════════════════════════════════════════
The following assertion was thrown resolving a single-frame image stream:
Unable to load asset: images/cat.jpg
When the exception was thrown, this was the stack:
#0      PlatformAssetBundle.load (package:flutter/src/services/asset_bundle.dart:221:7)
<asynchronous suspension>
#1      AssetBundleImageProvider.loadAsync (package:flutter/src/services/image_provider.dart:370:44)
<asynchronous suspension>
#2      AssetBundleImageProvider.load (package:flutter/src/services/image_provider.dart:356:7)
#3      ImageProvider.resolve.<anonymous closure>.<anonymous closure> (package:flutter/src/services/image_provider.dart:253:61)
#4      ImageCache.putIfAbsent (package:flutter/src/services/image_cache.dart:82:22)
#5      ImageProvider.resolve.<anonymous closure> (package:flutter/src/services/image_provider.dart:253:38)
(elided 8 frames from package dart:async)
Image provider: AssetImage(bundle: null, name: "images/cat.jpg")
Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#f1859(), name: "images/cat.jpg", scale:
1.0)
════════════════════════════════════════════════════════════════════════════════════════════════════

Flutter Doctor
[✓] Flutter (on Mac OS X 10.12.5 16F73, locale en-US, channel alpha)
    • Flutter at /Users/zeke/flutter
    • Framework revision d36e2f6191 (3 weeks ago), 2017-06-21 15:09:10 -0700
    • Engine revision b0dee695ec
    • Tools Dart version 1.24.0-dev.6.7

[✓] Android toolchain - develop for Android devices (Android SDK 26.0.0)
    • Android SDK at /Users/zeke/Library/Android/sdk
    • Platform android-26, build-tools 26.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] iOS toolchain - develop for iOS devices (Xcode 8.3.3)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 8.3.3, Build version 8E3004b
    • ios-deploy 1.9.1
    • CocoaPods version 1.2.1

[✓] Android Studio (version 2.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Gradle version 3.2
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b06)

[✓] IntelliJ IDEA Ultimate Edition (version 2017.1.5)
    • Flutter plugin version 15.1
    • Dart plugin version 171.4694.29

[✓] Connected devices
    • Android SDK built for x86 • emulator-5554                        • android-x86 • Android 8.0.0 (API 26) (emulator)
    • iPhone 7 Plus             • 2A665704-95E9-4CD2-86DF-9510A136125F • ios         • iOS 10.3 (simulator)

Is your "assets" key underneath a flutter: key as well?
https://flutter.io/assets-and-images/

Man do I feel dumb. I just needed to tab the assets over so that they are under flutter: instead of on the same level. Thanks.

What is a flutter key ? I right clicked project file to make a directory called assets ? Is this not the correct way ?

@Dano79 flutter: in pubspec.yaml
https://flutter.io/assets-and-images/

flutter:
uses-material-design: true
assets:

images/xxx.png

and not
flutter:
uses-material-design: true
assets:

images/xxx.png

Thank you @shashankrshukla ...U saved my Day....!!!!

@shashankrshukla fixed my problem as well , thx

I am getting the same error when I run my code initially on my device but once I hot reload/restart the app, the app works just fine. Please help me out.

I am getting the same error when I run my code initially on my device but once I hot reload/restart the app, the app works just fine. Please help me out.

@gsunit
I had the same problem. I had my images inside assets/ and in the pubspec.yaml I had this.
flutter:
 assets:
  - assets/sz-logo.png
  - assets/comodo-1.png
  - assets/iso27001.png

All I needed to do is change it to this:
flutter:
assets:
  - assets/
  - assets/sz-logo.png
  - assets/comodo-1.png
  - assets/iso27001.png```

@daanvandenhoek that shouldn't be necessary and just
flutter:
assets:
  - assets/

should have the same effect.

i am not able to load/upload images..
assests:

assests/img.jpg
it is always showing error

@shashankrshukla Thank you so much! problem was just about a "Tab". I mean it didn't work when it was like this:
assets:
    - assets/food.jpg

but then i just added a tab space before the "assets:" then it worked
  assets:
    - assets/food.jpg

@Dano79
flutter:
  uses-material-design: true
  assets:
   - images/xxx.png

and not

flutter:
  uses-material-design: true
assets:
   - images/xxx.png

in pubspec....one tab makes all the difference...

I had done the same thing but still the image is not loading. Please help!

@rohanmittal53  Please consider asking support questions in one of the other channels listed at http://flutter.io/support .

My issue was for those with same problem as mine
If you have sub-directories in the assets folder, then using
flutter:
  assets:
    - assets

will not include sub-directories. Use instead:
flutter:
  assets:
    - assets/icons
    - assets/whatever
    - assets/foo
    - assets/bar

Spaces are significant in YAML
assets is indented too far

Spaces are significant in YAML

assets is indented too far

flutter:
  uses-material-design: true
  assets:
   - images/default.png
   - images/cat.jpg

✅

😊

If you get this error only when first installing the app and then it disappears in the sequential hot restart, you need to Tools =>Flutter => Flutter Clean

Just as a comment because Google's first hit for the error is this issue...
You get the same error when you didn't include a file in your pubspec.yaml's assets section:
flutter:
  assets:
   # Only one asset image is included...
   - assets/first.png

and in the Dart code
Image.asset('assets/second.png'),

flutter: ══╡ EXCEPTION CAUGHT BY IMAGE RESOURCE SERVICE ╞════════════════════════════════════════════════════
flutter: The following assertion was thrown resolving an image codec:
flutter: Unable to load asset: assets/second.png

To fix this, make sure that all your assets are included

If you get this error only when first installing the app and then it disappears in the sequential hot restart, you need to Tools =>Flutter => Flutter Clean

tks you are right！

it worked out with me

@erikkubica comment regarding sub-directories worked for me.
Also, it seems like a flutter clean (possibly with packages get?) is needed after every change to the yaml in relation to the assets to make a change.

Also, it seems like a flutter clean (possibly with packages get?) is needed after every change to the yaml in relation to the assets to make a change.

It shouldn't be, but this was broken for a while and only fixed in the last 2 or 3 dev releases. Can you confirm which flutter version you are on?

Flutter 1.5.9-pre.43 • channel master • https://github.com/flutter/flutter.git Framework • revision 3cd15b52d4 (2 days ago) • 2019-04-27 04:52:18 -0400 Engine • revision 3b1bebf289 Tools • Dart 2.3.0 (build 2.3.0-dev.0.3 c46deebfb6)
Do you remember which version exactly it was fixed on?

I did a test on another project with the same problem and it worked, I was wrong.

Anyone getting this issue with embedded android + flutter?

So am I missing something here?
I have the correct indentation in my pubspec.yaml file but I'm still unable to load any images/assets. Extremely frustrating.

I changed "F" to "f" and it worked for me :)
Flutter:
  assets:
    -  assets/

flutter:
  assets:
    -  assets/

same problem here

I tried to change indentation in pubspec.yaml
I tried to clean flutter

until now it hasn't worked

@arvie86 You should write like this in your pubspec.yaml：
Flutter:
  assets:
    -  assets/images/

and write like this in your ***.dart:
Image(
        image: AssetImage(
          'assets/images/******.jpg',
        ),

maybe you wrote in
images/******.jpg
I hope that can help you.Sorry about my poor English. Wish you can understand it.

I am getting the same error when I run my code initially on my device but once I hot reload/restart the app, the app works just fine. Please help me out.
@gsunit

clean flutter it will solve the problem
In Android Studio,
Tools->Flutter->Flutter Clean

sir, i love you.

Thanks for the answer, but Im successfull solve this problem just add "lib" before assets. Like this :
flutter:
uses-material-design: true
assets:

lib/images/monkey.png
lib/images/cat.jpg

Image.asset('lib/assets/monkey.png')

A common mistake is when you create the assets directory in the wrong place.
Note: the assets folder should be in the same folder with the android, ios and lib folders.

flutter:
uses-material-design: true
assets:
- images/
and
this not
flutter:

uses-material-design: true

assets:
- images/

Mentioning full path including lib/assets/images/ worked for me.
Here's some code!
pubspec.yaml
flutter:
  uses-material-design: true
  assets:
    - lib/assets/images/

Dart code
Image.asset("lib/images/background.jpg")

Looks like the conversation here has converged on making sure that folks use the correct indentation and syntax in their pubspec. Since this isn't a Flutter issue, I'm closing this issue.

@kf6gpe Just to clarify, this is not what's actually happening.
at the time I had the issue, I tried all those suggestions and none worked for me. I'm not sure if people are confusing the issue with something else and then thinking it's the solution, or perhaps people experience the problem differently, but either way none of the above was the solution.
However, flutter has been upgraded and I haven't had that issue since, so I think closing the issue until someone else will experience it (and all above suggestions won't work for him) is a good idea at the moment.

@Dano79
flutter:
  uses-material-design: true
  assets:
   - images/xxx.png

and not

flutter:
  uses-material-design: true
assets:
   - images/xxx.png

in pubspec....one tab makes all the difference...

I had done the same thing but still the image is not loading. Please help!

Try tab instead of space between "-" & image tag

@Dano79
flutter:
  uses-material-design: true
  assets:
   - images/xxx.png

and not

flutter:
  uses-material-design: true
assets:
   - images/xxx.png

in pubspec....one tab makes all the difference...

I had done the same thing but still the image is not loading. Please help!

Indentation error.
Check Pubspec.yaml file
`> > flutter:

uses-material-design: true
assets:
- images/xxx.png
`

When I first ran the program, I received this message :
Unable to load asset: assets/words_time/Moein - TarkamNakon.json
but after hot reload Everything worked fine.
so I noticed that the problem was with the space in the file address
after change
assets/words_time/Moein - TarkamNakon.json
to
assets/words_time/Moein-TarkamNakon.json
I no longer had a problem with the first performance

When I first ran the program, I received this message :
Unable to load asset: assets/words_time/Moein - TarkamNakon.json
but after hot reload Everything worked fine.
so I noticed that the problem was with the space in the file address
after change
assets/words_time/Moein - TarkamNakon.json
to
assets/words_time/Moein-TarkamNakon.json
I no longer had a problem with the first performance

I tried all above but that worked thank you.

  Here are some more people being screwed by YAML:

https://stackoverflow.com/a/54334073/265521
https://stackoverflow.com/a/53659937/265521
https://stackoverflow.com/a/54931576/265521

Probably too late to choose a saner format for pubspec, but it would be nice if the "Unable to load asset" error listed the assets that are present, at least in debug mode. It might help people (especially people like me who have indented things properly but still get this error :-/ ).

for me the problem was, that I was using dart package instead of flutter.
I was using this:
import 'dart:io';
var fullFile =  new File(fullPath).readAsStringSync();
instead of:
import 'package:flutter/services.dart';
var fullFile =  await rootBundle.loadString(fullPath);
after changing that, it worked for me.

@Dano79
flutter:
  uses-material-design: true
  assets:
   - images/xxx.png

and not

flutter:
  uses-material-design: true
assets:
   - images/xxx.png

in pubspec....one tab makes all the difference...

You beauty,it worked like a charm

I was using \ instead of / in the path so I was getting this problem
Now it has been solved

If you still get  an error after trying all of the above, check if there is no slash infort of the path in your dart file:
Image(image: AssetImage('/assets/image.jpg'))
this is not correct.
It shoul be:
Image(image: AssetImage('assets/image.jpg'))

Here is all possible solution added [Solved] Unable to load asset Flutter

I'm sure there is nothing wrong with my code or file path, I tried all those suggestions and none worked
Then I try close the project and reopen it, Now it has been solved and images appears
(Android Studio v.3.1.4)

I'm sure there is nothing wrong with my code or file path, I tried all those suggestions and none worked
Then I try close the project and reopen it, Now it has been solved and images appears
(Android Studio v.3.1.4)

The following assertion was thrown resolving an image codec:
Unable to load asset: images/google.png
When the exception was thrown, this was the stack:
#0      PlatformAssetBundle.load (package:flutter/src/services/asset_bundle.dart:227:7)

#1      AssetBundleImageProvider._loadAsync (package:flutter/src/painting/image_provider.dart:667:14)

Image provider: AssetImage(bundle: null, name: "images/google.png")
Image key: AssetBundleImageKey(bundle: PlatformAssetBundle#ca559(), name: "images/google.png", scale: 1.0)
same error and my assets is arrange
//pubsec.yaml
flutter:
uses-material-design: true
assets:
- images\google.png
- images\facebook-logo.png

flutter:
uses-material-design: true
assets:
- lib/assets/images/


json: invalid number literal "1.7.3" / Issue with value for build-blocker-plugin version value

I am seeing this error constantly in the logs for the operator pod. It seems to be complaining of the string "1.7.3" in my jenkinsinstance yaml file. This error is repeated constantly.
ERROR:
E1010 19:04:52.310718       1 jenkinsinstance_controller.go:324] Error updating JenkinsInstance default/jenkinsinstance-team1: json: invalid number literal "1.7.3"
E1010 19:06:20.492016       1 jenkinsinstance_controller.go:324] Error updating JenkinsInstance default/jenkinsinstance-team1: json: invalid number literal "1.7.2"
The issue appears to only affect the value for the version of the 'build-blocker-plugin'
No matter what I change the value to, or if I change the double quotes to single quotes or no quotes, it still complains with that error in the Operator log.

For the sake of interest, please try "v1.7.3" if you haven't already.


[Proposal] Units of Repositories

I would like to see Gogs is modulable to resolve some questions. We can define one module I named it Unit. If you have more suitable name, please comment.
type UnitType int

const (
    UnitCode UnitType = iota
    UnitIssues
    UnitPRs
    UnitCommits
    UnitReleases
    UnitWiki
    UnitSettings
)

type Unit struct {
    UnitType
    NameKey string
    URI string
    DescKey string
    Sequence int
}

Now we have 6 units. And of course we maybe have more units in future such as Pluse, Graphs just like github's.
var (
    Units = map[UnitType]Unit{
        {
            UnitCode,
            "repo.code",
            "/",
            "repo.code_desc",
            0,
        },
        {
            UnitIssues,
            "repo.issues",
            "/issues",
            "repo.issues_desc",
            1,
        },
        {
            UnitPRs,
            "repo.pulls",
            "/pulls",
            "repo.pulls_desc",
            2,
        },
        {
            UnitCommits,
            "repo.commits",
            "/commits/master",
            "repo.commits_desc",
            3,
        },
        {
            UnitReleases,
            "repo.releases",
            "/releases",
            "repo.releases_desc",
            4,
        },
        {
            UnitWiki,
            "repo.wiki",
            "/wiki",
            "repo.wiki_desc",
            5,
        },
        {
            UnitSettings,
            "repo.settings",
            "/settings",
            "repo.settings_desc",
            6,
        }
    }
)

Then we can add one new field on Repository struct
type Repository struct {
...
Units []UnitType `xorm:"json"`
...
}

One repository could be assigned one or more units when it was created. And the old repositories could be assgined all the current units. When user visit repository page, firstly we could show units according the repository.
That is the first step.
The second we can add units on Access and Organization, Team,
type Access struct {
...
RepoID int64
UserID int64
Units []UnitType `xorm:"json"`
}

So for one person who visit repository it could only visit all allowed units.
We can define Team units, so when someone be added to this team, it will copy the team's units to this user's repository access.
type Team struct {
...
Units []UnitType `xorm:"json"`
...

I think the effect of this proposal is small. We can complete it on 2 to 3 commits.

This has been discussed at lengths over here https://github.com/gogits/gogs/issues/2438 . Might wanna continue there instead :)

I think maybe this is a different concept although I used the name plugin. This is not a big idea. It is easy to impletment. This proposal's is aims to let developer, tester and operator work together in Gogs. And every role has own plugins. Developer could view code but tester and operator will not.

Is this is about making modules for everything? in that case gogs is already built like that. Though it doesn't have the per-user, per-module access-part...

Yes, this is only a new organization for implementing per-user, per-module access. So we can let redmine , jira out. Different users (designers, testers, operators) can operate each other on Gogs, not only developers.

How units affects users? I'm not quite get this part.

This first-step has been merged by go-gitea/gitea#742. So I close this issue.


problem after deploy to Heroku (Cookie is missing in the request header)

Hi, thanks for this great tool.
I used PR #52 code and also add headers['Access-Control-Allow-Credentials'] = true; into withCORS method for including cookies in the CORS request
everything works properly in dev mode (hosted CORS Anywhere on <IP_ADDRESS>:8080)
but not working anymore when hosted on Heroku
problem is cookie missing in the request header
in dev mode, req header contains Cookie: GAMEC=Wed Aug 09 2017 15:30:18 GMT+0800 (CST)
but on Heroku, there is no 'Cookie' in the req header
the site is setting cookie like below
try {
    document.cookie = "GAMEC=" + new Date() + ";path=/";
} catch (e) {
    document.cookie = "GAMEC=" + Date().valueOf() + ";path=/";
}

Im using fetch to make a CORS request
fetch(url, {
    method: 'POST',
    headers: {
      'Content-Type': 'application/x-www-form-urlencoded;',
    },
    credentials: 'include',
    body: formData,
})

is it Heroku's problem ?
or did I do anything wrong ?
please, help me solving the problem, thanks

My issue might be releated, thats why I don't open another one.
I got kind of the same problem, but my cookies are not in the document.cookies varibable, they should be only http cookies set by set-cookie response headers.
So the custom REST API I'm currently working on sets a session token with the set-cookie header and uses it in further requests... for any reason this works without problems in postman, even without cors-anywhere, but there are cors errors when I use fetch.
Is there any option to enable setting cookies or something?

@DevCubeHD Your comment does not contain enough details to give a proper answer, as it is not clear whether you use the default CORS Anywhere (e.g. the public demo), a fork or custom config.

Is there any option to enable setting cookies or something? @Rob--W

No there is not. There is no safe or reliable way to offer cookie support; see my most recent comment at https://github.com/Rob--W/cors-anywhere/pull/154#issuecomment-468649353

My issue might be releated, thats why I don't open another one.
I got kind of the same problem, but my cookies are not in the document.cookies varibable, they should be only http cookies set by set-cookie response headers.
So the custom REST API I'm currently working on sets a session token with the set-cookie header and uses it in further requests... for any reason this works without problems in postman, even without cors-anywhere, but there are cors errors when I use fetch.
Is there any option to enable setting cookies or something? @Rob--W

How to solve this problem ?


WebCam Output is NOT showing

Hello. Im university student who study Unity.
I found your great NNCam and want to use it, but have some problem.
When I run(play) this project, I check the WebCam is Truely running cuz my laptop-camera light turn on.
However, NNcam output doesn't showing the WebCam.
I have been put release-onnx file

s on Assets/ONNX folder, and check another NNCam project, ex. Effect Unity project.
When I run Effect Unity project, WebCam success.
How Can I run NNCam (remove background) as right way?
Thnks.
My Unity version is 2020.3.37f1, Barracuda version is 1.3.2
Thank You.

and check another NNCam project, ex. Effect Unity project.

Did it run successfully?

and check another NNCam project, ex. Effect Unity project.
Did it run successfully?

Yes. No Error 😅

Does your project run when you set a background texture?

Maybe Yes.
When I change background of Camera and Compositor(Script), its change well

Then it should run without the background texture.
How it breaks when you unset the background texture? What is the symptom?
By the way, you can ignore the warning messages (messages with the yellow "!" icon).

Then it should run without the background texture.
How it breaks when you unset the background texture? What is the symptom?
By the way, you can ignore the warning messages (messages with the yellow "!" icon).

Actually, Warning or Error Message is zero. Just Camera is not Working only at this project. It doesn't break when i do not setting background either.

Just Camera is not Working only at this project.

I think the Camera worked in the first screenshot because it drew the gray background (null texture works as a gray texture).

Just Camera is not Working only at this project.

I think the Camera worked in the first screenshot because it drew the gray background (null texture works as a gray texture).

Oh How can I put Background Texture? Its mean 'Target Texture' on Camera Component?

Set the Background property of the Compositor script.

Yes I did it. But jest background layer is showing. No Human😂 However Cam state is turnin on

It comes back to the first question. Do the sample scenes (Room, Effect, etc.) work correctly? Does it show a person correctly?

It comes back to the first question. Do the sample scenes (Room, Effect, etc.) work correctly? Does it show a person correctly?

Yes. Effect scenes work correctly, with show a person.
When I build & run Room scene, just Room background texture is showing, not a person.

So, the Compositor works in the sample scenes, but it doesn't work in scenes you newly created.
Did you set the Resources and Shader properties correctly?

Yes I did. This is Room scene. No Human

And This is Effect Scene. Human is showing too

And This is Effect Scene. Human is showing too

No. The "Effect" scene is an example that adds swirling effects to people in front of the camera. I can't see any effect on your screenshot, so it isn't working correctly.
In summary, nothing work correctly on your side.

And This is Effect Scene. Human is showing too

No. The "Effect" scene is an example that adds swirling effects to people in front of the camera. I can't see any effect on your screenshot, so it isn't working correctly.
In summary, nothing work correctly on your side.

OK I see.
Could you tell me about what is the problem about Effect Scene setting at above image..?

Could you tell me about what is the problem about Effect Scene setting at above image..?

There is no problem. It should run.


resolve "ld: warning: linking against dylib not safe for use in applicat...

resolve "ld: warning: linking against dylib not safe for use in application extensions" when setting as AppExtensions' target.

Thank you for your pull request!
I've  merged PR and published new version as 0.0.4


utils: avoid using 'gem' as an identifier

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes?
[x] Have you successfully run brew tests with your changes locally?

Avoid breaking code highlighting of some editors (e.g. Sublime Text) when gem is used as an identifier. This messes up the highlighting below the first use and makes working with the file rather annoying.
This is a bit of a self-serving PR, but I hope it is acceptable nonetheless. I believe TextMate users might be affected, too, as they rely on the same syntax definition files for code highlighting.

Seems fine 👍


...

COM model is c++ itself. And you just have no chances to understand C++ to C transition in near future

script fixed

Sorry @alexeyneu, but #5 was almost done by the time you submitted this, and I also don't think it improves the readability much. You're add somewhat simpler syntax, but also a lot of boilerplate. If I were to simplify it, I would rather rewrite it in Rust than C++, TBH.

C code you see is generated by MIDL . C++ is native for COM


[#7 TES 1.1 Spec compliance] Validation errors for TesServiceInfo

Describe the bug
ServiceInfo does not include the new required properties in the TES 1.1 spec
Steps to Reproduce
Run the TES compliance test for TES 1.1
Expected behavior
The response_schema_validation test case passes.
Additional context
Currently, the test case message says: Response Schema validation failed for GET /service-info
and the log_messages contains:  validation errors for TesServiceInfo\nid\n  field required (type=value_error.missing)\ntype\n  field required (type=value_error.missing)\norganization\n  ...

Also see #164


Extract preprocessor and babel plugin into their own packages

I think we should take the preprocessor  and the babel-plugin and extract them from this package into their own libraries.
We don't necessarily want every app into the future to be locked into having a dep on ember-template-imports, for a few reasons:

we will probably want a better name for it before it goes into the default blueprint. The user-facing feature is "template tag", not "template imports". Managing this transition is harder if multiple packages (like ember-template-lint and estlint-plugin-ember) have peer deps on it. Those peer deps are only needed because they need access to the preprocessor, and would be happy to just have deps on the library if it existed.
we want people to be able to see and manage their own build plugins in environments like embroider's upcoming vite support. In that situation, having ember-template-imports is unhelpful -- it has as bunch of legacy broccoli things that aren't relevant. The user really only wants the babel-plugin to put their babel config and a packager-specific plugin that can depend on the preprocessor library.

List of consumers' locations that would need updating after the extraction occurs:

The rollup plugin

https://github.com/NullVoxPopuli/rollup-plugin-glimmer-template-tag/blob/main/packages/rollup-plugin-glimmer-template-tag/src/rollup-plugin.js#L10

ember-template-lint

https://github.com/ember-template-lint/ember-template-lint/blob/b4433e9439f3c555b3c4beb56c34bfed18a423b5/lib/extract-templates.js#L1
https://github.com/ember-template-lint/ember-template-lint/blob/b4433e9439f3c555b3c4beb56c34bfed18a423b5/test/unit/extract-templates-test.js#L2

eslint-plugin-ember

https://github.com/ember-cli/eslint-plugin-ember/blob/a714b468832b7fc26da020513c7ef491959e211b/lib/preprocessors/glimmer.js#L5
https://github.com/ember-cli/eslint-plugin-ember/blob/a714b468832b7fc26da020513c7ef491959e211b/lib/config/recommended.js#L2

ember-repl

https://github.com/NullVoxPopuli/limber/blob/273f1626927d6d467de45c9d08c406adb353fcdc/packages/ember-repl/addon/cjs/index.ts#L7

Glint

https://github.com/typed-ember/glint/blob/3b0d3ab78765c2b39e25e09788597609740cacad/packages/environment-ember-template-imports/-private/environment/preprocess.ts#L2

Should the babel plugin become part of https://github.com/emberjs/babel-plugin-ember-template-compilation/ ?

(It would not a separate plugin, it would be a feature in the existing plugin.)

So, started extracting preprocess-embedded-templates.ts and started to realize that there are a number of inter-dependencies to the files I'm trying to extract and between those files themselves. I'm currently taking the bullish approach and just extracting and duplicating to get things to work but going to need to work out the most appropriate way to do this before shipping.

a good chunk of stuff is simpler if you delete all the template-literal-transform code, as in: https://github.com/ember-template-imports/ember-template-imports/pull/146
you'll start to see that some abstractions (consts, functions, etc) are no longer needed, or only do a very focused thing.
Hope this helps!

[UPDATE] Here's an attempt at extracting the preprocess-embedded-templates code: https://github.com/achambers/preprocess-ember-template
I have verified this locally against ember-template-imports and ember-template-lint but still have some questions. Need to also verify against the other repos that depend on template imports currently.
One thing to note, have gone back to making this change against master instead if against https://github.com/ember-template-imports/ember-template-imports/pull/146 as removing support for hbs breaks ember-template-import as it still needs to be able to render a template that uses inline templates as per it's tests that use inline hbs in test files which, as far as I understand, is still a valid use case, for the time being at least.

Latest updates here:

we have the preprocessor shipping in https://github.com/embroider-build/content-tag/
babel-plugin-ember-template-compilation 2.1.0 shipped with built-in support for the babel part, since RFC 931 is merged
we are using them in @embroider/addon-dev

Next steps:

PR updates to ember-cli-babel bumping minimum babel-plugin-ember-template-compilation-plugin to 2.1.0 and adding @babel/plugin-transform-class-static-block alongside the other plugins that become mandatory when transpiling decorators.

Hi @ef4 . Started to take a look at these next steps but am a little confused. Can you please clarify a couple of things?

PR updates to ember-cli-babel bumping minimum babel-plugin-ember-template-compilation to 2.1.0

I couldn't find babel-plugin-ember-template-compilation referenced in ember-cli-babel. It does seem to be used in ember-cli-htmlbars. Is that the repo I should be bumping the package?

adding @babel/plugin-transform-class-static-block alongside the other plugins that become mandatory when transpiling decorators.

Should this still happen in ember-cli-babel? I see that the likes of @babel/plugin-proposal-class-properties are also referenced in ember-cli-htmlbars however they seem to just be used in tests and are devDependencies where as they are dependencies in ember-cli-babel. I suspect this item remains as is but I just wanted to clarify.

once those changes are in ember-cli-babel, we can update ember-template-imports

Assuming we are making the changes to both ember-cli-babel and ember-cli-htmlbars, I guess we'll need to bump both packages in ember-template-imports. Confirm?

should we close this issue and track our work in https://github.com/ember-template-imports/ember-template-imports/issues/143 now?

We still have the other tools to propagate changes to, but we did it! Go team!


n_tokens_all <= cparams.n_batch

it appears that the n_batch in sessions remain the default value of 512. I have had some large inputs of over 800 tokens testing Hermes mistral v0.2 and Dolphin Phi 2 and both had the same error of n_tokens_all <= cparams.n_batch. The fix was to cut until the 500th token.
Setting the sessions params' n_batch did not solve this issue as the default still remains 512 by the llama. Rust's layer does recognize the new value yet the cpp layer keeps the default value.
Taking a look at llamacpp source, there are conflicting default values too, 2048 and 512... so I'm not sure where the issue lies.

I couldn't really reproduce the issue, but regardless I have updated the llama.cpp in the feat/llava, branch. Could you check if the error persists in that branch?

I just ran into this, you need to set both n_batch and n_ctx, because the batch size is set to the minimum of those two options.
https://github.com/edgenai/llama_cpp-rs/blob/55eb9691b13d07f56eaa68992d5e6accaa281691/crates/llama_cpp/src/model/mod.rs#L541


chinese translate update

Aftre several rules update, here comes new Simplified Chinese translate.

Hey @Ooopz , I will see about reviewing this this week. Hopefully it will not take too long for me to take a look. Thanks for opening the PR!

Hey @Ooopz , thanks for creating this PR to update the Chinese translation!


Revert replacing NMAgent client in CNS (#1643)

This reverts commit 9cc0b0ba3e60aea9986ad5bc365c07324edc5d17.
Performing these changes uncovered unknown dependencies that clients had on specific formations of responses from CNS. Until these can be properly documented and understood, the legacy implementation must remain.
A subsequent approach may instead introduce alternative endpoints with known properties that clients can migrate to piecemeal.

Lints will need to be bypassed for this, since it is, as much as possible, a pure revert. The goal is restoring the old behavior, warts and all. Lints can be noted and applied in a subsequent cleanup PR, but we should try to preserve the atomicity of this PR by adding to it as little as we can.
Also, beware looking at this patch through diff goggles. This code is not de novo logic--it already existed in the codebase prior to the revert. I may not be able to answer questions about why things are the way they are. It is also not as good as it could be, so we may need to suspend our better taste in favor of getting a clean release.

no objection as longs as the functional checks pass

It was decided that we will not revert, and instead add functionality in #1825 to try to repair the regressions. Closing.


Password to protect changing the setting only

Description
Currently the password feature will protect the site from being accessed, is it possible that we can configure the password to protect changing the setting only?
I'd love to share my homepage with my friends, but I don't want them to change any setting, that's the use case.
Priority
Medium (Would be very useful)

Thank you for this issue ! This is being worked on as #259
Certain configs will be password protected and other won't. The ones that are not protected can will be accessible with homarr/configName

Oops, I missed that one when searching. Close as duplicate then. Great work :).


Auth implementation

Description
Our app will require later to connect users and to hide certain information based on roles. This is known as authentication and authorization or in short Auth.
In NextJs we have a time-saving library called NextAuth, this PR is to leverage this library and explore how its use of it can be on our app.
Screenshots
Type of change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How to test it
If applicable, describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

Go to '...'
Click on '....'
Scroll down to '....'

Checklist:

[x] Doesn't break the current code.
[x] Passes linters and test, also is building.
[x] Doesn't have spelling or grammatical problems.
[x] Doesn't have unnecessary comments or debugging code.
[x] The branch is updated with the last dev branch changes.

I've received this error on click sign in button

So the email implementation is too much involved and means a lot of changes to make, so I decided to ditch that solution and use an alternative: current user verification plus a magic link to the email.
Also, people can be whitelisted to avoid extra people entering as admin. With magic, we can use a lot of fun stuff like several auth methods or modify the look and feel of the pop-up.
Please tell me how do you feel about the experience and how it can be improved.

@victor-heliomar @e-azocar I think you 2 have to create credentials to complete the flow. We can talk about that in a meet.

I added test users in Hasura, we can check on that on a meet

@Israel-Laguan I think it's good to see him at a meet.
Today in the afternoon I try to run the code to test the PR

After clicking Magic Link I received this error


host-bmc: Implemented Enable interface for Core objects

Upstream Gerrit link for the same patch

add to approvelist


Clean up retain/autorelease noise

There are numerous functions that produce a lot of noise in Objective-C code, examples include:

_objc_retainAutoreleasedReturnValue
_objc_retainAutorelease

These are not often the interesting bits of the function, and it might be helpful if these could be hidden. This should be investigated first to confirm that:

this is actually possible;
this is reliably performable; and
there are no side effects to doing this.

Since these functions effectively "pass through" a value, I'd expect this to be feasible, but I need to do some testing to confirm.

This should be possible by just replacing the LLIL call instructions with simple assignments, passing through the parameter?