[Feature Request] Group blog posts by year and allow for changing the year.

Feature Description

When someone has many blog posts, then having them grouped makes it easier to navigate.
Currently all the blog post links are listed in the same 1 page for a "type: blog" pages.
Problem/Solution

Having some sort of grouping by year would allow to split the list into consumable short lists.
Some sort of Sidebar navigation with list of possible years, would allow to navigate between them.
The list of years would need to be extracted by going through each post and then making a list with distinct years. Once the year is know, using the navigation the list would be filtered out to show only the posts from that year.
Alternatives Considered

While there is some possibility to do this manually, it would be nice if they was possible via the engine itself.
A manual solution would like:

Create a type: docs page
Create sub pages manually for each year
Add each post manually as a link.

In this process a lot of the feature that comes automatically with a blog type page would be lost i.e Readmore links, partial abstracts, RSS etc.
Additional Context

Nevermind, I played around and I was able to achieve this using the some directory structuring and using
cascade:
  type: docs

I'll add some comment later for anyone else wanting to do this. Closing the issue now.


Added case for dos file endings in tests

In reference to #33. The Base64 shader uris in these tests needed to be updated and were the cause of the failing tests.
Edit: This works locally, I'm looking into why travis is failing; I'm pretty sure its a windows->unix line-ending problem

Coverage remained the same at 90.253% when pulling 8bb3d34b39b03caa01dd59b539e31d3394ff401f on shader-test-failures into 56ea6d3f53ea361e2c2b3626773ef134b3b292d0 on master.

So, the issue was that the uri produced is different if your git checks out the files with dos line endings. I've modified the tests to allow either.

It's a bit awkward to have to keep both of those strings, and this is not ideal maintenance for adding future tests.
Can we write a utility function that wraps bufferEqual(dataUriToBuffer(.. and encapsulates this?  Also include a comment about why we are doing this since it will be non-obvious.
If you do some Node research, there may be a way to normalize this when reading the file.

@pjcozzi Is there a strong argument for keeping the hard-coded uris at all? Something like this would probably be a lot cleaner:
var fragmentShaderUri = 'data:text/plain;base64,' + new Buffer(fragmentShaderData).toString('base64');

Sounds good, give it a try!

@pjcozzi This is ready for another look

Coverage remained the same at 90.253% when pulling a0710571541f7f034e457e2f0a0a2fa804e02dcf on shader-test-failures into 56ea6d3f53ea361e2c2b3626773ef134b3b292d0 on master.

Very nice, thanks @lasalvavida!

Changes Unknown when pulling af3e0a4a6e1bddfc1ccc2912caf3b52a2292ff57 on shader-test-failures into ** on master**.


Patch 1.10.0

@rstrahan are there any plans to patch 1.10.0?

No current plans, @imranismail .. I see you sent a PR for this - thanks!
Can you share more info please:

why is patch needed? What broke in 1.10.0?
briefly enumerate the fixes did you made in the PR
With this info i'll try to find time soon to validate and merge. Thank you!
Cheers
Bob

I just tried with the current master and it seems to work. I figured there's three changes in the files RR patches and maybe redoing the patch would be needed.


Handle other events

I tried to use RawValue but failed to obtain satisfactory results.
Also in every "other event" I have added tag (as it was removed), removed "seq" field as it is duplicated , also "cleaned" Greetings event as is outlier and I do not think it complies with other events.
I also left alone
"tag":"SnapshotConfirmed"

as if I understand well it is going to be addressed in separate PR.
All events tested. Testing does not use assert_eq but String's contain as the resultant string field order is not deterministic.

closing in favor of https://github.com/Anviking/oura/pull/5


[PINOT-6696] Remove http connection pool in table-size api path

It appears that MultiThreadedHttpConnectionManager is buggy - especially when it encounters a connection refused error on any connection, it fails to "release" the underlying connection and subsequent connections to other servers over the pool encounter connection-refused due to the stale connection. This causes failures in the TableSize API which is invoked during segment upload, failing the upload.
Also, in the current code, we use default size for max connections which is 20; for one of large clusters (with ~80 servers hosting segments of a table), the 20 max connection essentially means that the connections are established and torn down, essentially rendering the pool ineffective.
If we do need to use a pool, we should consider using PoolingHttpConnectionManager - this requires a httpclient version bump to 4.x.x from the existing 3.1. That is a larger change - for this phase, we'll consider the simpler case of bypassing the pool completely.
Testing done:
Deployed updated binary in integration cluster and hit table/size api on all tables hosted on the controller - checked that the basic functionality worked, there are no resource leaks (via netstat/lsof). Also, checked that there are no connection refused errors to hosts that are up.

Codecov Report

Merging #3051 into master will decrease coverage by 0.01%.
The diff coverage is 87.5%.

@@             Coverage Diff              @@
##             master    #3051      +/-   ##
============================================
- Coverage     70.37%   70.36%   -0.02%
  Complexity        4        4
============================================
  Files           934      934
  Lines         43504    43496       -8
  Branches       5907     5906       -1
============================================
- Hits          30617    30607      -10
- Misses        10827    10831       +4
+ Partials       2060     2058       -2

Impacted Files
Coverage Δ
Complexity Δ

...m/linkedin/pinot/controller/ControllerStarter.java
60.94% <ø> (-0.69%)
0 <0> (ø)

...edin/pinot/controller/api/resources/TableSize.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...i/resources/PinotSegmentUploadRestletResource.java
57.89% <100%> (ø)
0 <0> (ø)
:arrow_down:

...ontroller/api/resources/ServerTableSizeReader.java
80% <100%> (+1.95%)
0 <0> (ø)
:arrow_down:

...n/pinot/controller/helix/SegmentStatusChecker.java
83.87% <100%> (-0.11%)
0 <0> (ø)

...om/linkedin/pinot/common/http/MultiGetRequest.java
94.44% <100%> (-0.56%)
0 <0> (ø)

...inkedin/pinot/controller/util/TableSizeReader.java
91.52% <100%> (-0.08%)
0 <0> (ø)

...lix/EmptyBrokerOnlineOfflineStateModelFactory.java
86.66% <0%> (-13.34%)
0% <0%> (ø)

...impl/dictionary/DoubleOnHeapMutableDictionary.java
75.55% <0%> (-6.67%)
0% <0%> (ø)

...r/helix/SegmentOnlineOfflineStateModelFactory.java
59% <0%> (-6%)
0% <0%> (ø)

... and 25 more

Continue to review full report at Codecov.

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

@mcvsubbu good point about deleteClosedConnections()! I looked through the code a bit more and it seems like the issue is more with how the "freeconnection" list is handled. When the application tries to execute a method, the HttpMethodDirector fetches a connection to satisfy the request. The pool-mgr should return a connection with the host-configuration passed in. But it looks like thats not happening correctly in some cases. This a bit of FUD at this point, but its possible that we get an incorrect connection when we don't see the error too (as there is nothing so far that points me to the fact that this is related to conn-refused; its just more noticeable in that particular case).
I think we should upgrade httpclients and work with newer connection pools than chase issues with the older clients.
Appreciate the review and suggestions!


Chunkers

This is a very WIP chunker.
Will address #36

@andreibondarev TextSplitter is a very simple chunker that will split on delimeter. I'm not currently limiting chunks to a max size or doing overlap. But I first want to see if this is acting as you'd expect it.

@andreibondarev TextSplitter is a very simple chunker that will split on delimeter. I'm not currently limiting chunks to a max size or doing overlap. But I first want to see if this is acting as you'd expect it.

I've fleshed this out a little more. Does this help create a clearer picture?

(There's generally a ton of extra space that gets add when a PDF is loaded so we may need to do some gsub-ing, strip-ing and compact-ing to clean it up.)
I had tested out loading and splitting a contract and it looks like it worked alright!

Also -- whoever want to quickly test out the Markdown splitter:
str = Loaders::HTML.new("https://github.com/andreibondarev/langchainrb/blob/main/README.md").load
chunks = Chunkers::MarkdownSplitter.new.split_text(str)

@rickychilcott I think, really importantly, is your implementation allows to easily throw your custom Chunker:: in the mix. We should definitely document how it can be done and maybe test it out in a Rails app:
# app/chunkers/custom_chunker.rb
class Chunkers::CustomSplitter < Chunkers::Base
   def split_text(text)

Again -- not sure everything needs to be this ^^ flexible right out of the gate but just putting some ideas out there.

@rickychilcott Do you think you could upmerge this and resolve conflicts to see where we're at? Perhaps this could be the first iteration that processes simple string outputs from Langchain::Processors::* before we figure out what that return object looks like.
cc @alchaplinsky

I actually created a quick PR for that https://github.com/andreibondarev/langchainrb/pull/70. Let me y'all know what you think!

+1 for this getting merged. Having issues with loading PDFs (This model's maximum context length is 8191 tokens, but the given text is 267572 tokens long.)

I went ahead and added a basic text chunker: https://github.com/andreibondarev/langchainrb/pull/188. I'd like to get some feedback and then we can figure out what the next iteration looks like. For example we want may to automatically pick a chunking strategy based on the original file format (if it's a file).
I'm closing this stale PR.
@rickychilcott Thank you for the original effort!
@moekidev Thank you for the baran gem!


How to automatically close FPS after dragging a certain distance instead of dragging it to the bottom

That's right, @VisualDeceit!


Kindly explain the magic behind composer/installers

Composer's customer installers doc states:

A Custom Installer is defined as a class that implements the Composer\Installer\InstallerInterface and is usually distributed in a Composer Plugin

But Composer\InstallersBase\BaseInstaller does not implement Composer\Installer\InstallerInterface. So what's magic behind it's working? Does composer check for this installer and give it special treatment?

The magic is in Installer.php which extends LibraryInstaller as per https://getcomposer.org/doc/articles/custom-installers.md#the-custom-installer-class
LibraryInstaller implements InstallerInterface: https://github.com/composer/composer/blob/master/src/Composer/Installer/LibraryInstaller.php

Thanks @shama. I should have checked better :)


Star Rating carn't select checked="checked" stars

Hey,
https://ampbyexample.com/advanced/star_rating/preview/
https://ampbyexample.com/playground/#url=https%3A%2F%2Fampbyexample.com%2Fadvanced%2Fstar_rating%2Fsource%2F
It is impossible to vote 2 stars. When your product have a 5 star best rating und 5 stars displayed via checked="checked" no one can rate the product with 5 stars!?!
Best,
Daniel

Hi,
could you explain this better?
I tried to:

open https://ampbyexample.com/advanced/star_rating/preview/
click on 5 stars and it worked
then I clicked on 2 stars and it worked

Could you list the steps you are doing?

Hi,

open https://ampbyexample.com/advanced/star_rating/preview/

click first on 2 stars and it didnt work

The Problem is: You carnt vote the default value checked="checked".
The example has 2 stars by default and when you try to vote the example with 2 stars, it isnt possible.
Best,
Daniel

Hi,
what is not exactly possible?
For me:

open https://ampbyexample.com/advanced/star_rating/preview/
click first on 2 stars
2 stars are clicked

What exactly happens after you click the 2 stars? could you send a snapshot?

When you click on a different star rating, a network request is made. I think the problem might be that if you click on the current star rating, there is no network request made.
One use case might be to  have the current rating (i.e. the average rating, not the rating you gave) as the initial selected value. The problem for that use case would be that you couldn't vote for the current rating.
I think in that case, there might be a better way to show the initial state. Maybe a separate rating component would be useful for this use case.

@sparhami Thats exact my use case. Display the average rating e.g. 5 stars as the initital selected value and none couldn't vote 5 stars.

Clearing a rating is also a valid use case (perhaps more so if you just wanted a simple thumbs up/down), which simple radios don't address. You might also want to partially fill in stars (to distinguish an average rating of something like 4.5 from 5.0 or 4.0).


after adding a tile, move cursor to next emtpy tile

Creates a better flow for creating programs

I made a start on this already


MastodonEmbed

Implements detection of mastodon toots from the mastodon fediverse. We are allowing for embedding toots from any of the possible instances by checking for a class of mastodon-embed on the embed code. For URL detection, we will have to rely on the oEmbed API for these services to pull the HTML code and then automatically pick the correct embed.
nb. If we would like to store the content of the status, we will need to use an API token to get the status. This is not surfaced on the embed code.

<iframe src="https://mastodon.social/@arstechnica/112440642771928896/embed" class="mastodon-embed" style="max-width: 100%; border: 0" width="400" allowfullscreen="allowfullscreen"></iframe><script src="https://mastodon.social/embed.js" async="async"></script>

{
  "text": "\uFFFC",
  "blocks": [{
    "id": "B00000000",
    "type": "mastodon-embed",
    "parents": [],
    "selfClosing": false,
    "attributes": {
      "url": "https://mastodon.social/@arstechnica/112440642771928896"
    }
  }],
  "marks": []
}

Quick question about the content slice on this annotation, since I don't see where that data would come from. Perhaps you expect this to be empty when creating the annotation from an embed code, but might allow us to fetch/set this from somewhere else? If so that is fine but I would maybe update the comment on that attribute to make that clear.

Yeah, I expect that the content will likely be manually set by people or dynamically fetched. (Likely manual in this case)


dlm interval issue

Current on version 2.4.2,
The DLM  (data lifecycle manager) interval values aer 2,3,4,6,8,12,24, while in code https://github.com/cloudtools/troposphere/blob/master/troposphere/dlm.py#L11, could you please fix this issue, thank you in advance.

This interval documentation only shows 12 and 24. Could you link documentation for that expanded range of values?

sorry for missing the doc, please reference here https://docs.aws.amazon.com/dlm/latest/APIReference/API_CreateRule.html, also it is confirmed on AWS UI, since I can modify the interval value in UI.


Make current context available as placeholder in field options

Summary
This issue only concerns multi-context-mode. This is more of an enhancement than a bug. In some fields it's possible to make snippet calls so tags get processed and you can for example create your own set up input options by calling pdoResources to populate them. This is not context-aware.
Step to reproduce
Try to find a context placeholder and fail at it.
Observed behavior
Me not finding the context placeholder :)
Expected behavior
Me having a [[+clientconfig.context]] placeholder of some sort, so I can do this:
[[pdoResources?
   &tpl=`@INLINE [[+pagetitle]]==[[+id]]`
   &parents=`0`
   &limit=`0`
   &context=`[[+clientconfig.context:default=`web`]]`
   &outputSeparator=`||`
   &showHidden=`1`
]]

Environment
ClientConfig 2.1.0, MODX 2.7.1, Debian, MySQL 5.7.25, PHP 7.0.33

This would likely need to be added in core/components/clientconfig/controllers/home.class.php around line 44-48. Perhaps simply $this->modx->setPlaceholder('context', $context) where $context is a validated key (currently handled around line 57-68.
Would be a cool enhancement but don't have time to build (and more importantly) test that right now.


Add <sys/timex.h> bindings for linux-musl

Adds bindings for adjtimex() & co. for linux-musl. Also adds clock_adjtime() to linux-glibc.

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

AWS OSPO review, no OSPO concerns.

@bors r+

:pushpin: Commit 530b4cae868e7d1a66df4b4185b776e03b954b43 has been approved by JohnTitor

:hourglass: Testing commit 530b4cae868e7d1a66df4b4185b776e03b954b43 with merge de9c20783abda27b2e9a0411f58dfaacb8ccc63e...

:broken_heart: Test failed - checks-actions

Looks like a CI test is failing because http://ftp.debian.org/debian/dists/testing/main/installer-s390x/20200314/images/generic/initrd.debian has gone 404 (maybe releated to the release of Debian 11)? Obviously unrelated to this PR.

Fixed on master, @bors retry

:hourglass: Testing commit 530b4cae868e7d1a66df4b4185b776e03b954b43 with merge 5f5f51cb992957a50bdbdb482d8392a8c3e0d26b...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing 5f5f51cb992957a50bdbdb482d8392a8c3e0d26b to master...


problem of Store XML file into riak [JIRA: CLIENTS-816]

Dear Mancini,
Recently, I use basho/riak-php-client between php program and riak server. And I want store a xml file into riak, I set content-type to text/xml,but it doesn't work. I check the source code, and find "rawurlencode($data)" in riak-php-client/src/Riak/Command/Object.php line 50. This causes xml label no longer has any effect.
I wish to konw the necessity of the code. I delete urlencode and store xml file successfully, but I'm afraid about influencing the correctness of the project.
Best Wishes!
Xiaolong Tang

Hi @txl1290, thanks for submitting the issue, I will try to trouble shoot it. Do you have a snippet of code so that I can reproduce the issue?

Hi @christophermancini ,
I found the reason. It's my mistake. I use storeObject to store xml. But I use the xml through riak http request. This doesn't contain the urldecode handle.
I use fetchObject and now it's ok.
sorry to waste your time.
Best Wishes!
Xiaolong Tang


Use Metal Detector when boss is dying in 10 seconds or less

This still isn't working, it needs more work in its own branch before pushing into development.

We should redo the way we get the boss seconds to death. I've been debugging it, and it returns 0, then returns some values such as 4.606632096124567e-9.


Installation broken?

Hi,
Trying to install the plotly extension by jupyter labextension install @jupyterlab/plotly-extension, I'm getting an UnboundLocalError: local variable 'proc' referenced before assignment Error.
I'm on jupyter 4.4.0, python 3.5 and anaconda installation, am I doing something wrong?

I get a different error:
kwargs = {'name': self.extra_args[1]}
IndexError: list index out of range

I had this error: error Couldn't find match for "83b1a3e5837d785af582efd5ed1a212f2df6a4ae" in "_c" for "https://github.com/mapbox/mapbox-gl-style-spec.git".
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.
I installed nodejs via pip and using https://anaconda.org/conda-forge/nodejs with the same result
I'm using anaconda,
plotly 2.4.1
jupyterlab 0.31.10
this issue occurs in windows and macos

@blink1073 Does this sound familiar?

Great!
@jvaldiviezo9 Can you try installing npm via conda install nodejs and see if that resolves you issue?

@gnestor The solution in my case was that Git was not installed, that solved my problem

@jvaldiviezo9 Ah, git. Would not have been one of my first guesses 👍


Add support for highlighting functions

Hi @Nek-12.
Please provide an example of this feature, perfectly in PR

Functions can be highlighted with a different color as on the screenshot based on the curly braces after the declaration. Granted, we can't be sure a function will be highlighted properly with some languages that have function syntax sugar, such as kotlin trailing lambda foo { }. However, we could try to highlight what we can.

I mark this as "nice to have" as many languages do not have "function keyword" and it would require structure analysis


Razzle 4 server callback

❓Question
Hello,
I am upgrading our application from razzle 3 to razzle 4 to fix some issues. Currently we are using C# backend and calling a callback function on server.tsx that renders the html as explained on this tutorial
On razzle 3 everything worked find and I could use
node -p "var fn = require('./build/server'); fn((_,x)=>console.log(x), '/');"
and the html would show on the console (in either mode, yarn build or razzle start)
You can find the repository for razzle 3 server callback here: https://github.com/EvanCoh/razzle3callback
When I try the above command on razzle 4 (while running yarn start) after the exact same changes (repository is here: https://github.com/EvanCoh/razzle4callback ) I get an error:
node -p "var fn = require('./build/server'); fn((_,x)=>console.log(x), '/');"

sswp> Handling Hot Module Reloading
[eval]:1
var fn = require('./build/server'); fn((_,x)=>console.log(x), '/');
                                    ^

undefinedTypeError: fn is not a function
    at [eval]:1:37
    at Script.runInThisContext (node:vm:131:12)
    at Object.runInThisContext (node:vm:308:38)
    at node:internal/process/execution:77:19
    at [eval]-wrapper:6:22
    at evalScript (node:internal/process/execution:76:60)
    at node:internal/main/eval_string:23:3

If I run 'yarn build' and try the above, then it works fine, but the problem is that I cant get it to work in development (= yarn start).
Both of the above repositories were created with
npx create-razzle-app --example with-typescript razzle4callback
and
npx create-razzle-app@three --example with-typescript razzle3callback
to be as close to your examples as possible, while adding the minimum changes to achieve the callback functionality.
you can take a look at the complete code of both repositories and maybe add any commit.
Can you tell me how to fix the error above or at least how to debug what's causing it?

Try fn = require('./build/server').default;

Thanks but the issue persists even so:
node -p "var fn = require('./build/server').default; fn((_,x)=>console.log(x), '/');"
sswp> Handling Hot Module Reloading
[eval]:1
var fn = require('./build/server').default; fn((_,x)=>console.log(x), '/');
                                            ^

undefinedTypeError: fn is not a function
    at [eval]:1:45
    at Script.runInThisContext (node:vm:131:12)
    at Object.runInThisContext (node:vm:308:38)
    at node:internal/process/execution:77:19
    at [eval]-wrapper:6:22
    at evalScript (node:internal/process/execution:76:60)
    at node:internal/main/eval_string:23:3

Run build first, that is the dev build.

Actually using webpack 5 solved the issue.


add a build/ folder symlink to bin_dist

Description
symlink the bin folder to build/ in root directory for easier bin-dist.

Codecov Report

Merging #7719 (adf94ae) into master (91d2a95) will decrease coverage by 56.97%.
The diff coverage is n/a.

:exclamation: Current head adf94ae differs from pull request most recent head e60eb91. Consider uploading reports for the commit e60eb91 to get more accurate results

@@              Coverage Diff              @@
##             master    #7719       +/-   ##
=============================================
- Coverage     71.50%   14.52%   -56.98%
+ Complexity     4033       80     -3953
=============================================
  Files          1581     1535       -46
  Lines         80472    78598     -1874
  Branches      11954    11751      -203
=============================================
- Hits          57540    11417    -46123
- Misses        19037    66345    +47308
+ Partials       3895      836     -3059

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
?

unittests2
14.52% <ø> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

any thoughts on this? it would've simplify our document/quickstart a bit. also personally it makes myself debugging binary distribution easier

thx @xiangfu0 --> will create another PR to update pinot-docs


ABLoss Implentation

 loss_AT4 = ((self.Connect4(res4_s) > 0) ^ (res4_t > 0)).sum().float() / res4_t.nelement()
           loss = self.criterion_active_L2(self.Connect4(res4_s), res4_t.detach(), margin) / self.batch_size
I read your paper, this is a good work.   but I am a little  confused about the implementation, I could not find the corresponding implementation of Formula 7 in your paper. Could your give me a explanation, waiting your reply. thanks

Hi
loss_AT is only used for loss visualization
You can find the loss function for back-propagation here
https://github.com/bhheo/AB_distillation/blob/a7701d3506e13da87bda75aba1cee6397408144f/cifar10_AB_distillation.py#L111-L113
and criterion_active_L2 is here https://github.com/bhheo/AB_distillation/blob/a7701d3506e13da87bda75aba1cee6397408144f/cifar10_AB_distillation.py#L16-L20
In detail,
r(S(I)) is d_net.Connectx(d_net.resx)
\rho(T(I)) is (target > 0) and 1 - \rho(T(I)) is (target <= 0)
(source + margin)**2 * ((source > -margin) conduct role of \sigma(\mu 1 + r(S(I))) and L2 norm
I hope it helps you to understand the implementation


generate docs for request builders and send_request

Generate docs for the request builders.
Include snippet in send_request that imports a request builder and sends the request
Will save adding information from x-ms-example to the send_request doc for a later issue: https://github.com/Azure/autorest.python/issues/917

added to prepare_request branch


Feat/291 fix for windows

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Database instance

Hello! I'm trying to execute method auth->register, but getting error «Uncaught InvalidArgumentException: The database connection must be an instance of either PdoDatabase, PdoDsn or PDO» in vendor\delight-im\auth\src\UserManager.php on line 88.
I have a instance of PDO in my own singleton class:
class Db
{
private static $instance = null;

private function __construct() {}

private function __clone() {}

public static function getInstance() {
    if (!self::$instance) {
        try {
            self::$instance = new PDO('mysql:dbname= ... etc

and require it as $this->auth = new \Delight\Auth\Auth($this->dbh);
where
$this->dbh
is a instance of PDO. what am I doing wrong?

sorry, i solved that

Thanks, no problem :)


#47 Connect FootprintLineGraph component with endpoints and adjust display

Updated FootPrintLineGraph.tsx to call endpoints and display live user data. Added dual Y-axis in apex chart options to visualize the large difference in carbon score data and green transaction data. Added dynamic Y-Axis in both cases in case data changes.

@benrockehenderson This looks awesome! Thanks a lot! I was thinking it might be good to add some test cases for when the data.green_transactions returns N/A? Even though we don't have any N/A values, I think it'd be a good idea to also handle it in case we use some other JSON file as our data in the future. We can create a ticket for that in our Kanban :)

Looks amazing, thank you! Good idea @zzadxz, I can add cases for that in backend.


update material-ui 0.15.0

there are many changes that need to made in material-ui 0.15 beta
you'll currently get errors like this due to the paths being wrong. It's a quick fix.
ERROR in ./~/formsy-material-ui/lib/FormsyCheckbox.js
Module not found: Error: Cannot resolve module 'material-ui/lib/checkbox' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsyCheckbox.js 13:16-51

ERROR in ./~/formsy-material-ui/lib/FormsyDate.js
Module not found: Error: Cannot resolve module 'material-ui/lib/date-picker/date-picker' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsyDate.js 13:18-68

ERROR in ./~/formsy-material-ui/lib/FormsyRadioGroup.js
Module not found: Error: Cannot resolve module 'material-ui/lib/radio-button-group' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsyRadioGroup.js 13:24-69

ERROR in ./~/formsy-material-ui/lib/FormsySelect.js
Module not found: Error: Cannot resolve module 'material-ui/lib/select-field' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsySelect.js 13:19-58

ERROR in ./~/formsy-material-ui/lib/FormsyText.js
Module not found: Error: Cannot resolve module 'material-ui/lib/text-field' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsyText.js 13:17-54

ERROR in ./~/formsy-material-ui/lib/FormsyTime.js
Module not found: Error: Cannot resolve module 'material-ui/lib/time-picker/time-picker' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsyTime.js 13:18-68

ERROR in ./~/formsy-material-ui/lib/FormsyToggle.js
Module not found: Error: Cannot resolve module 'material-ui/lib/toggle' in /Users/hawk/Documents/hopecommerce/node_modules/formsy-material-ui/lib
 @ ./~/formsy-material-ui/lib/FormsyToggle.js 13:14-47

0.4.0 released.


Merge to master

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5c1ffcb). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage          ?   69.72%
=========================================
  Files             ?       16
  Lines             ?      109
  Branches          ?       11
=========================================
  Hits              ?       76
  Misses            ?       31
  Partials          ?        2

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 5c1ffcb...dc82bbc. Read the comment docs.


List current api key

Closes #13556

I think we should make Basic Auth and the ?api_key= parameter behave exactly the same. To be consistent with engine, we should check the basic auth first, and then the api_key (relevant if both are provided).

That will be done here: https://github.com/CartoDB/cartodb/issues/13565

Acceptance like a :rocket: Waiting for last test run after merging master before deploying.


Version naming of Coq Platform releases

The current implied Coq Platform version naming conventions have potential to cause counterintuitive version ordering in OPAM, so I propose that the Coq Platform follows Coq's version naming conventions:

X.YY+alphaZ for alpha (or beta) releases, e.g., 8.12+alpha1,  8.13+alpha2.
X.YY.Z for stable releases, e.g., 8.12.0 and 8.13.1.

Note in particular that OPAM sorts:

8.12.0+alpha1 after 8.12.0
8.13.0.alpha1after 8.13.0
... hence we should avoid conventions that lead to such version names.

For example,
$ dpkg --compare-versions 8.12.0 lt 8.12.0.alpha1 && echo true
true

For the record, the current policy set by @MSoegtropIMC is to have the complete Coq version name, followed by a further number. E.g., 8.13+beta1.0, <IP_ADDRESS>, etc. However, @gares expressed concern about the last digit being important, e.g., if we can add new packages in such a release. I agree with this concern. My proposal would be to separate the platform versioning scheme from the Coq versioning scheme. Since it seems important to highlight the Coq version, it could be done in a later component. E.g., 1.0.0+8.13. Another option which was proposed by @ejgallego is to use date-based version numbers, like is common for distributions.

For the record, the current policy set by @MSoegtropIMC is to have the complete Coq version name, followed by a further number. E.g., 8.13+beta1.0, <IP_ADDRESS>, etc. However, @gares expressed concern about the last digit being important, e.g., if we can add new packages in such a release. I agree with this concern. My proposal would be to separate the platform versioning scheme from the Coq versioning scheme. Since it seems important to highlight the Coq version, it could be done in a later component. E.g., 1.0.0+8.13. Another option which was proposed by @ejgallego is to use date-based version numbers, like is common for distributions.

I still prefer the Coq version + extra digit variant because:

the Coq platform should be tightly coupled to the Coq releases
it is an exceptional startup situation that we might add many packages from one minor version to the next
I don't see why we should pick a version scheme which is optimized for this startup situation

I still prefer the Coq version + extra digit variant because:

the Coq platform should be tightly coupled to the Coq releases
it is an exceptional startup situation that we might add many packages from one minor version to the next
I don't see why we should pick a version scheme which is optimized for this startup situation

* it is an exceptional startup situation that we might add many packages from one minor version to the next

If this becomes a rule, then I'm fine with your reasoning. It was not clear to me that packages were supposed to enter/exit the platform only at major release time.

* it is an exceptional startup situation that we might add many packages from one minor version to the next

If this becomes a rule, then I'm fine with your reasoning. It was not clear to me that packages were supposed to enter/exit the platform only at major release time.

the Coq platform should be tightly coupled to the Coq releases

IIUC, that's not entirely true since the platform may be released a few weeks after a Coq release. Also, as we just discussed, new packages can be added in between major Coq releases.
OTOH, it seems important to everyone here to clearly highlight what is the major version of Coq that's shipped in a given platform version. However, where we disagree (both @gares and me) with you is with the claim that the patch-level number of Coq is important enough to be shown in the version of the platform. We think it doesn't deserve to be shown more than any other relevant piece of data like the patch-level number of the math-comp version that is shipped.
That being said, it could be confusing to have Coq platform's 8.13.1 version to actually include Coq 8.13.0. So a solution to this issue would be to have another component separator. I understand that one would want to put the Coq version first, but then the platform version could be 8.13-1.0. And then, you would go on to 8.13-1.1, 8.13-1.2, etc., every time you update any package (Coq, math-comp, etc.) to a new patch-level (bugfix) release (this could happen every time such a package gets a new bugfix release). And when you add a new package that was not included before you would go to 8.13-2.0. And when you bump to a new major Coq release, you could reset the platform version component: 8.14-1.0.

the Coq platform should be tightly coupled to the Coq releases

IIUC, that's not entirely true since the platform may be released a few weeks after a Coq release. Also, as we just discussed, new packages can be added in between major Coq releases.
OTOH, it seems important to everyone here to clearly highlight what is the major version of Coq that's shipped in a given platform version. However, where we disagree (both @gares and me) with you is with the claim that the patch-level number of Coq is important enough to be shown in the version of the platform. We think it doesn't deserve to be shown more than any other relevant piece of data like the patch-level number of the math-comp version that is shipped.
That being said, it could be confusing to have Coq platform's 8.13.1 version to actually include Coq 8.13.0. So a solution to this issue would be to have another component separator. I understand that one would want to put the Coq version first, but then the platform version could be 8.13-1.0. And then, you would go on to 8.13-1.1, 8.13-1.2, etc., every time you update any package (Coq, math-comp, etc.) to a new patch-level (bugfix) release (this could happen every time such a package gets a new bugfix release). And when you add a new package that was not included before you would go to 8.13-2.0. And when you bump to a new major Coq release, you could reset the platform version component: 8.14-1.0.

Another suggestion, in line with what I was proposing above, as been made independently by @erikmd in https://github.com/coq/ceps/pull/52#issuecomment-768669866:

Hi all, I know that it has been mentioned above that the precise naming convention could be discussed in a later thread.
However I just wanted to raise a suggestion on a detail (that might impact the suffix that is mentioned in the CEP):
as the Coq platform can be seen as a Coq distribution (core + contributions), would it make sense to always append the suffix with a + instead of just a .? a bit like how some GNU/Linux distributions such as Debian tag the packaged versions… I mean, that would amount to release for Coq 8.14.0:

either coq-platform 8.14.0+beta; coq-platform 8.14.0+0; coq-platform 8.14.0+1 (assuming we keep Coq's whole 3-place version)

or coq-platform 8.14+beta; coq-platform 8.14+0; coq-platform 8.14+1 (assuming we want to "hide" Coq's point release to users, while avoiding any ambiguity w.r.t. what "8.14.1" could mean).

WDYT? (anyway, if you agree with this +-suffix idea, the precise naming convention, e.g., the choice between the two items above, could still be discussed apart, obviously)

Both @gares and me prefer the second bullet point.

Another suggestion, in line with what I was proposing above, as been made independently by @erikmd in https://github.com/coq/ceps/pull/52#issuecomment-768669866:

Hi all, I know that it has been mentioned above that the precise naming convention could be discussed in a later thread.
However I just wanted to raise a suggestion on a detail (that might impact the suffix that is mentioned in the CEP):
as the Coq platform can be seen as a Coq distribution (core + contributions), would it make sense to always append the suffix with a + instead of just a .? a bit like how some GNU/Linux distributions such as Debian tag the packaged versions… I mean, that would amount to release for Coq 8.14.0:

either coq-platform 8.14.0+beta; coq-platform 8.14.0+0; coq-platform 8.14.0+1 (assuming we keep Coq's whole 3-place version)

or coq-platform 8.14+beta; coq-platform 8.14+0; coq-platform 8.14+1 (assuming we want to "hide" Coq's point release to users, while avoiding any ambiguity w.r.t. what "8.14.1" could mean).

WDYT? (anyway, if you agree with this +-suffix idea, the precise naming convention, e.g., the choice between the two items above, could still be discussed apart, obviously)

Both @gares and me prefer the second bullet point.

I think this depends a bit on the timing we will see. In case a patch release of Coq comes after a platform release, it is likely we will do a patch release of the platform pretty immediately. Say we are at Coq platform 8.14+1 with Coq 8.14.0 and then comes a patch to 8.14.1 and we would release Coq platform 8.14+2 immediately. Then this relation is not obvious from the version numbers - one would have to read the release notes.
But I don't have a very strong opinion here. I agree that running out of letters is not the only downside of X.Y.Z.W version numbers.

I think this depends a bit on the timing we will see. In case a patch release of Coq comes after a platform release, it is likely we will do a patch release of the platform pretty immediately. Say we are at Coq platform 8.14+1 with Coq 8.14.0 and then comes a patch to 8.14.1 and we would release Coq platform 8.14+2 immediately. Then this relation is not obvious from the version numbers - one would have to read the release notes.
But I don't have a very strong opinion here. I agree that running out of letters is not the only downside of X.Y.Z.W version numbers.

have to read the release notes.

That is the crux IMO.
I see the platform as a distribution, its version can hardly provide all the info about what it contains.
Sure, Coq is a key component, but I expect users to care more about the rest. E.g. I care more about the version of my browser or mail client than the version of my Linux kernel. I'm stretching it a little, I admit it, but being able to remove even the Coq version from the platform version would be a sign of success IMO.

have to read the release notes.

That is the crux IMO.
I see the platform as a distribution, its version can hardly provide all the info about what it contains.
Sure, Coq is a key component, but I expect users to care more about the rest. E.g. I care more about the version of my browser or mail client than the version of my Linux kernel. I'm stretching it a little, I admit it, but being able to remove even the Coq version from the platform version would be a sign of success IMO.

Thinking about it, also taking your proposed release process into account, I think using just 8.14+X or 8.14.X is the better option. If the platform becomes the publicly announced "product", people will become less aware of the Coq patch releases and probably don't care how Coq platform X.Y.Z relates to Coq release X.Y.
So we take this as agreed?

Thinking about it, also taking your proposed release process into account, I think using just 8.14+X or 8.14.X is the better option. If the platform becomes the publicly announced "product", people will become less aware of the Coq patch releases and probably don't care how Coq platform X.Y.Z relates to Coq release X.Y.
So we take this as agreed?

Totally.
I prefer the + because it does not risk of being confused with the Coq version, which stays X.Y.Z

Totally.
I prefer the + because it does not risk of being confused with the Coq version, which stays X.Y.Z

Fine, so the bottom line is that Coq platform will be named 8.X+Y, where the 8.X matches the Coq version and Y is either "beta" or an increasing number.

Fine, so the bottom line is that Coq platform will be named 8.X+Y, where the 8.X matches the Coq version and Y is either "beta" or an increasing number.

Yes, I'll update the CEP about the new release cycle with this info ASAP!

Yes, I'll update the CEP about the new release cycle with this info ASAP!

About Y=beta, I suggest Y=beta1 to make sure that we account for the (hopefully infrequent) case where we need to release a second beta of the platform (beta and beta2 are not ordered as we would like in all packaging systems).

About Y=beta, I suggest Y=beta1 to make sure that we account for the (hopefully infrequent) case where we need to release a second beta of the platform (beta and beta2 are not ordered as we would like in all packaging systems).

Thanks for noting this. Yes we should use +beta1.

Thanks for noting this. Yes we should use +beta1.

This discussion was concluded on Zulip and the related CEP. In the end, we will use the YYYY.MM.X scheme where X is an increasing number starting at 0, beta versions take the version number YYYY.MM+beta1 and YYYY.MM is only increased for major updates, mostly of Coq, but possibly also of an included package, and in case of removal of a package. Minor updates will only require an increase of X and may include new packages or updates of packages with only bug fixes.


UMAP Clustering of OOD samples based on user filtering, fixed model summary page bug

Steven and I did a code review to approve this PR


Documentation to filter via URL parameter is missing

Filtering can be done via query parameters for MVC related stuff. This is undocumented.

Done.

Done.


Ifix 19

@Deepamulimani-eGov we have new repo configs-mgramseva. please raise pr to that repo not here.


Hangs while "initializing..."

I am using mod_md v1.1.16 with the following simple configuration:
MDomain MY_DOMAIN
MDCertificateAgreement https://letsencrypt.org/documents/LE-SA-v1.2-November-15-2017.pdf

ServerName MY_DOMAIN

<VirtualHost *:443 *:80>
  ServerName MY_DOMAIN
  DocumentRoot /var/www/default/

  SSLEngine on
  DocumentRoot /var/www/default/

  ErrorLog ${APACHE_LOG_DIR}/error.log
  LogLevel debug
  CustomLog ${APACHE_LOG_DIR}/access.log combined

</VirtualHost>

Unfortunately, mod_md fails for my domain. Any idea why? I have no clue.
The error.log shows the following debug messages:
[Wed Aug 15 20:15:12.927264 2018] [ssl:info] [pid 14:tid<PHONE_NUMBER>37024] AH01914: Configuring server MY_DOMAIN:443 for SSL protocol
[Wed Aug 15 20:15:12.927341 2018] [ssl:debug] [pid 14:tid<PHONE_NUMBER>37024] ssl_engine_init.c(1701): AH10083: Init: (MY_DOMAIN:443) mod_md support is available.
[Wed Aug 15 20:15:12.927347 2018] [md:debug] [pid 14:tid<PHONE_NUMBER>37024] mod_md.c(1110): AH10076: MY_DOMAIN: manages server MY_DOMAIN
[Wed Aug 15 20:15:12.927350 2018] [md:debug] [pid 14:tid<PHONE_NUMBER>37024] mod_md.c(1162): AH10113: md_get_certificate called for vhost MY_DOMAIN.
[Wed Aug 15 20:15:12.927425 2018] [md:debug] [pid 14:tid<PHONE_NUMBER>37024] mod_md.c(1224): AH10116: MY_DOMAIN: providing fallback certificate for server MY_DOMAIN
[Wed Aug 15 20:15:12.927431 2018] [ssl:warn] [pid 14:tid<PHONE_NUMBER>37024] AH10085: Init: MY_DOMAIN:443 will respond with '503 Service Unavailable' for now. This host is part of a Managed Domain, but no SSL certificate is available (yet).
[Wed Aug 15 20:15:12.927630 2018] [ssl:debug] [pid 14:tid<PHONE_NUMBER>37024] ssl_engine_init.c(479): AH01893: Configuring TLS extension handling
[Wed Aug 15 20:15:12.927864 2018] [ssl:debug] [pid 14:tid<PHONE_NUMBER>37024] ssl_util_ssl.c(476): AH02412: [MY_DOMAIN:443] Cert matches for name 'MY_DOMAIN' [subject: CN=Apache Managed Domain Fallback / issuer: CN=Apache Managed Domain Fallback / serial: EF8F93D2A131D6977651E437793E94C4D29C1799 / notbefore: Aug 15 19:54:26 2018 GMT / notafter: Aug 29 19:54:26 2018 GMT]
[Wed Aug 15 20:15:12.927875 2018] [ssl:info] [pid 14:tid<PHONE_NUMBER>37024] AH02568: Certificate and private key MY_DOMAIN:443:0 configured from /etc/apache2/md/domains/MY_DOMAIN/fallback-cert.pem and /etc/apache2/md/domains/MY_DOMAIN/fallback-privkey.pem

[Wed Aug 15 20:15:12.983230 2018] [md:info] [pid 14:tid<PHONE_NUMBER>37024] AH10071: mod_md (v1.1.16-git), initializing...
[Wed Aug 15 20:15:12.995211 2018] [ssl:info] [pid 14:tid<PHONE_NUMBER>37024] AH01914: Configuring server MY_DOMAIN:443 for SSL protocol
[Wed Aug 15 20:15:12.995237 2018] [ssl:debug] [pid 14:tid<PHONE_NUMBER>37024] ssl_engine_init.c(1701): AH10083: Init: (MY_DOMAIN:443) mod_md support is available.
[Wed Aug 15 20:15:12.995243 2018] [md:debug] [pid 14:tid<PHONE_NUMBER>37024] mod_md.c(1110): AH10076: MY_DOMAIN: manages server MY_DOMAIN
[Wed Aug 15 20:15:12.995248 2018] [md:debug] [pid 14:tid<PHONE_NUMBER>37024] mod_md.c(1162): AH10113: md_get_certificate called for vhost MY_DOMAIN.
[Wed Aug 15 20:15:12.995322 2018] [md:debug] [pid 14:tid<PHONE_NUMBER>37024] mod_md.c(1224): AH10116: MY_DOMAIN: providing fallback certificate for server MY_DOMAIN
[Wed Aug 15 20:15:12.995327 2018] [ssl:warn] [pid 14:tid<PHONE_NUMBER>37024] AH10085: Init: MY_DOMAIN:443 will respond with '503 Service Unavailable' for now. This host is part of a Managed Domain, but no SSL certificate is available (yet).
[Wed Aug 15 20:15:12.995476 2018] [ssl:debug] [pid 14:tid<PHONE_NUMBER>37024] ssl_engine_init.c(479): AH01893: Configuring TLS extension handling
[Wed Aug 15 20:15:12.995969 2018] [ssl:debug] [pid 14:tid<PHONE_NUMBER>37024] ssl_util_ssl.c(476): AH02412: [MY_DOMAIN:443] Cert matches for name 'MY_DOMAIN' [subject: CN=Apache Managed Domain Fallback / issuer: CN=Apache Managed Domain Fallback / serial: EF8F93D2A131D6977651E437793E94C4D29C1799 / notbefore: Aug 15 19:54:26 2018 GMT / notafter: Aug 29 19:54:26 2018 GMT]
[Wed Aug 15 20:15:12.995988 2018] [ssl:info] [pid 14:tid<PHONE_NUMBER>37024] AH02568: Certificate and private key MY_DOMAIN:443:0 configured from /etc/apache2/md/domains/MY_DOMAIN/fallback-cert.pem and /etc/apache2/md/domains/MY_DOMAIN/fallback-privkey.pem
[Wed Aug 15 20:15:12.997337 2018] [mpm_event:notice] [pid 14:tid<PHONE_NUMBER>37024] AH00489: Apache/2.4.34 (Ubuntu) OpenSSL/1.1.0h configured -- resuming normal operations
[Wed Aug 15 20:15:12.997649 2018] [core:notice] [pid 14:tid<PHONE_NUMBER>37024] AH00094: Command line: '/usr/sbin/apache2 -D FOREGROUND'
[Wed Aug 15 20:15:13.319433 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] MY_DOMAIN: setup staging
[Wed Aug 15 20:15:13.320322 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] MY_DOMAIN: need certificate
[Wed Aug 15 20:15:16.600584 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] registered new account https://acme-v01.api.letsencrypt.org/acme/reg/40309724
[Wed Aug 15 20:15:16.624442 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] MY_DOMAIN: check Terms-of-Service agreement
[Wed Aug 15 20:15:16.624471 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] MY_DOMAIN: setup new authorization
[Wed Aug 15 20:15:17.379528 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] MY_DOMAIN: setup new challenges
[Wed Aug 15 20:15:18.357895 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] updated authz https://acme-v01.api.letsencrypt.org/acme/authz/q1JZoD_RdJ6cY-p-622tHYcY2j6m3N0CixafGx1xSTU
[Wed Aug 15 20:15:18.358552 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] MY_DOMAIN: monitoring challenge status
[Wed Aug 15 20:15:18.959532 2018] [ssl:info] [pid 15:tid<PHONE_NUMBER>85824] [client <IP_ADDRESS>:57164] AH01964: Connection to child 5 established (server MY_DOMAIN:443)
[Wed Aug 15 20:15:18.960195 2018] [ssl:info] [pid 15:tid<PHONE_NUMBER>85824] [client <IP_ADDRESS>:57164] AH01996: SSL handshake failed: HTTP spoken on HTTPS port; trying to send HTML error page
[Wed Aug 15 20:15:18.960271 2018] [ssl:info] [pid 15:tid<PHONE_NUMBER>85824] SSL Library Error: error:1408F09C:SSL routines:ssl3_get_record:http request
[Wed Aug 15 20:15:19.295672 2018] [md:error] [pid 16:tid<PHONE_NUMBER>43392] ACME server authz: challenge 'invalid' for MY_DOMAIN at https://acme-v01.api.letsencrypt.org/acme/authz/q1JZoD_RdJ6cY-p-622tHYcY2j6m3N0CixafGx1xSTU. Exact repsonse was: {"combinations":[[2],[0],[1]],"challenges":[{"type":"tls-alpn-01","uri":"https://acme-v01.api.letsencrypt.org/acme/challenge/q1JZoD_RdJ6cY-p-622tHYcY2j6m3N0CixafGx1xSTU/6504127061","status":"invalid","token":"tiqQ7Y4Wwwm1sJdfQZ4ZG3sarXPF_Ec4uRxno2Kdfzk"},{"type":"dns-01","uri":"https://acme-v01.api.letsencrypt.org/acme/challenge/q1JZoD_RdJ6cY-p-622tHYcY2j6m3N0CixafGx1xSTU/6504127062","status":"invalid","token":"kOOoaapxuLiXSdHqiOAoagJjSskmuN7Vc53V9wzPy_A"},{"keyAuthorization":"gMR7DMZdCBBw7h6pa6mLkd9M-uO_l23TwiMbAT_m4HQ.Dz3q6DWV1E-KWnbJQN-DpsDDzuslrt03xXvAYkF2a5g","type":"http-01","uri":"https://acme-v01.api.letsencrypt.org/acme/challenge/q1JZoD_RdJ6cY-p-622tHYcY2j6m3N0CixafGx1xSTU/6504127063","status":"invalid","error":{"detail":"Invalid response from http://MY_DOMAIN/.well-known/acme-challenge/gMR7DMZdCBBw7h6pa6mLkd9M-uO_l23TwiMbAT_m4HQ: \\"<!DOCTYPE HTML PUBLIC \\"-//IETF//DTD HTML 2.0//EN\\">\\n\\n<title>400 Bad Request</title>\\n\\n
Bad Request</h1\\"","type":"urn:acme:error:unauthorized","status":403},"token":"gMR7DMZdCBBw7h6pa6mLkd9M-uO_l23TwiMbAT_m4HQ","validationRecord":[{"hostname":"MY_DOMAIN","url":"http://MY_DOMAIN/.well-known/acme-challenge/gMR7DMZdCBBw7h6pa6mLkd9M-uO_l23TwiMbAT_m4HQ","port":"80","addressesResolved":["<IP_ADDRESS>"],"addressUsed":"<IP_ADDRESS>"}]}],"expires":"2018-08-22T20:15:17Z","identifier":{"type":"dns","value":"MY_DOMAIN"},"status":"invalid"}
[Wed Aug 15 20:15:19.296260 2018] [md:error] [pid 16:tid<PHONE_NUMBER>43392] (22)Invalid argument: MY_DOMAIN: unexpected AUTHZ state 3 at https://acme-v01.api.letsencrypt.org/acme/authz/q1JZoD_RdJ6cY-p-622tHYcY2j6m3N0CixafGx1xSTU
[Wed Aug 15 20:15:19.296289 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] (22)Invalid argument: MY_DOMAIN: checked all domain authorizations
[Wed Aug 15 20:15:19.296567 2018] [md:error] [pid 16:tid<PHONE_NUMBER>43392] (22)Invalid argument: AH10056: processing MY_DOMAIN
[Wed Aug 15 20:15:19.296589 2018] [md:info] [pid 16:tid<PHONE_NUMBER>43392] AH10057: MY_DOMAIN: encountered error for the 9. time, next run in  0:21:20 hours

This:
<VirtualHost *:443 *:80>

will not work. You enable SSL on port 80 and Let's Encrypt will want to talk plain http: on that port to your server. Check with a browser if your server can be reached. If you do not get a connection, something is wrong in your setup.

Removing the port 80 from the VitualHost worked:
[Thu Aug 16 07:59:16.139675 2018] [md:info] [pid 15:tid<PHONE_NUMBER>94432] MY_DOMAIN: creating certificate request
[Thu Aug 16 07:59:17.723996 2018] [md:info] [pid 15:tid<PHONE_NUMBER>94432] MY_DOMAIN: received certificate
[Thu Aug 16 07:59:17.724829 2018] [md:info] [pid 15:tid<PHONE_NUMBER>94432] MY_DOMAIN: retrieving certificate chain
[Thu Aug 16 07:59:17.932627 2018] [md:notice] [pid 15:tid<PHONE_NUMBER>94432] AH10059: The Managed Domain MY_DOMAIN has been setup and changes will be activated on next (graceful) server restart.

On another server I am using <VirtualHost *:443 *:80> to enable MDRequireHttps permanent. Without *:80 I'm getting the default apache page. With the additional *:80 the forward works:
curl -I http://MY_DOMAIN
HTTP/1.1 301 Moved Permanently
Location: https://MY_DOMAIN/

How can I achieve MDRequireHttps permanent without adding *:80? Do I miss something?

I understand, sure. Usually I do my forwarding to https with a separate VirtualHost.
In this case, I'm pointing you to an unclear documentation with MDRequireHttps. it says here and here:

This is a convenience directive to ease http: to https: migration of your Managed Domains. With:
MDRequireHttps temporary

you announce that you want all traffic via http: URLs to be redirected to the https: ones [...]

From a user perspective it seems, that I only need to add the MDRequireHttps and I am done. Thus, mod_md cares about the rest. That assumption seems wrong. Maybe you could add a note, that a VirtualHost on port 80 is still required for MDRequireHttps?

Wrt to https://github.com/icing/mod_md/issues/100#issuecomment-413467695: On my testing machine I had an older apache and with an older mod_md with <VirtualHost *:443 *:80>. Probably my configuration relied on this rejected migration advice.

I can add the statement that a vhost on :80 continues to be necessary, yes.


$state.current is sometimes 'empty'

I encountered this strange bug today. Sometimes (I don't know why) $state.current returns an "empty" object:
There's at least two stackoverflow issues about this bug:
http://stackoverflow.com/questions/25370775/exposing-the-current-state-name-with-ui-router
http://stackoverflow.com/questions/24895908/how-do-return-the-state-current-name-from-ui-router-statechange
I can't explain the bizarre behaviour I'm seeing: When I debug $state I can see that $state.current is fine, but when I print $state.current directly I get this:
{ name: "", url: "^", views: null, abstract: true }

$state.$current.self is also the same empty object., even though it looks fine when I inspect $state.
I don't quite understand how this can be happening. How can $state and $state.current return different values? $state.current doesn't have a getter.
EDIT: Okay, the problem was that the debugger doesn't show the object when it was printed, rather it shows it's current state. Now I just need to figure out why the state is empty.

Just encountered this bug tonight... we could see the proper $state.current values in the HTML but they were not accessible in the corresponding controller. Very strange!

Same here , seems it's the bug in 0.2.13.

I tried this and it works:
In controller:
$scope.uiState = $state

In view:
uiState.current

Or:
In Controller:
$rootScope.$state = $state

In View:
$state.current

Same thing

This is the root state. This happens when your initial page load targets a nested URL that depends in a resolve. The root state is the initial state of the application pending a successful transition to the desired state. Feel free to add this to the FAQ if it was unclear to you, and you think it might confuse other people as well.

I don't understand.
If i console.log($state) it shows correct state, if i console.log($state.current) then it shows empty string, so how can I get current nested state ?

@ddctd143 i think this can help you
https://github.com/angular-ui/ui-router/wiki/Nested-States-%26-Nested-Views#parent-property-using-state-name-string
Maybe your state dont't know anything about your parent.

Same thing as @ddctd143 is happening to me. @nateabele what if my state doesn't have a parent and doesn't depends in a resolve?. I'm trying to get the $state.current in a directive, if i set a timeout works as i expect, if not i get the $state object with empty values. How do i wait until $state.current resolve, is there a promise?, Am i approaching this incorrectly?

I faced the same issue, This is happening when I reload the page, but not in normal navigation. Anyways,  I used $timeout and it is working fine for me.          $timeout(function () {           console.log($state.current, 'this is working fine');         }, 100);

I have this bug too.
My opinion. We have "abstract: true" param. I think this is a base/root state which has been loaded.

Unfortunately this is not yet supported
Answer

Anyways, I used $timeout and it is working fine for me.

I faced the same issue, but felt that $timeout is bad.
So I looked for some special event from API docs, and '$stateChangeSuccess' is what you need to get nested current state (while parent is abstract):
$scope.$on('$stateChangeSuccess', function (event, toState, toParams, fromState, fromParams) {
   // toState === $state.current
});

Thanks @last-indigo.
This solution works for states, but it doesn't work on with History events.  I'll edit my comment if I figure it out.

+1
Ok with fake timeout.
setTimeout( () => { console.log($state.current)}, 0);

@nateabele I understand what you said, but I don't understand how to deal with it. And the FAQ doesn't answer the question...
I need to get the current state name from a component's controller to initialise the component accordingly to the current state (when user reload the page for example). But in some cases I've got the { name: "", url: "^", views: null, abstract: true }. How to wait for the resolve of the state to be finished in that case? Events are deprecated and $transition$ is not recognize in a controller. So I don't see how to handle this kind of cases...

resolveBinding: function($transition$) {   console.log($transition$.to()); }


XML config provider cannot differentiate an empty section from a key with an empty string value

See #600
For example:

This results in consistencies with other providers in that something that would normally be treated as an empty section is treated as a key with an empty string value.

Yeah, but doesn't really matter, so closing.


Artist metrics development

Hi There!
I have made the following:

Configure React and Redux 🔧
Fetch data from last.fm API ⬇
Make components to render API content 🗃
Implement Search functionality 🔍
Add styling 🎨
Record a video showing the project 📸
Added tests ✅
Deploy the page 🚀

Thank you so much for taking the time to review my code! :zap:

Dear CR,
Please approve this PR as the above requested change should have been raised as an optional change.
Thank you!


SLi-GRU + Li-GRU fixes

Rebased against tip of develop

For the record, this is not ready to merge as the LS test recipe does not seem to converge for me (Adel and I are trying to figure that one out).


Console warnings

On starting up the app, I get this warning several times in my console. Doesn't stop anything from working as it's just a warning, but worth looking into.
GitPoint[1251:177712] Warning: Failed prop type: Invalid props.style key `color` supplied to `RCTView`.
Bad object: {
  "color": "#000000",
  "padding": 15,
  "fontFamily": "AvenirNext-Bold"
}
Valid keys: [
  "display",
  "width",
  "height",
  "top",
  "left",
  "right",
  "bottom",
  "minWidth",
  "maxWidth",
  "minHeight",
  "maxHeight",
  "margin",
  "marginVertical",
  "marginHorizontal",
  "marginTop",
  "marginBottom",
  "marginLeft",
  "marginRight",
  "padding",
  "paddingVertical",
  "paddingHorizontal",
  "paddingTop",
  "paddingBottom",
  "paddingLeft",
  "paddingRight",
  "borderWidth",
  "borderTopWidth",
  "borderRightWidth",
  "borderBottomWidth",
  "borderLeftWidth",
  "position",
  "flexDirection",
  "flexWrap",
  "justifyContent",
  "alignItems",
  "alignSelf",
  "alignContent",
  "overflow",
  "flex",
  "flexGrow",
  "flexShrink",
  "flexBasis",
  "aspectRatio",
  "zIndex",
  "direction",
  "shadowColor",
  "shadowOffset",
  "shadowOpacity",
  "shadowRadius",
  "transform",
  "transformMatrix",
  "decomposedMatrix",
  "scaleX",
  "scaleY",
  "rotation",
  "translateX",
  "translateY",
  "backfaceVisibility",
  "backgroundColor",
  "borderColor",
  "borderTopColor",
  "borderRightColor",
  "borderBottomColor",
  "borderLeftColor",
  "borderRadius",
  "borderTopLeftRadius",
  "borderTopRightRadius",
  "borderBottomLeftRadius",
  "borderBottomRightRadius",
  "borderStyle",
  "opacity",
  "elevation"
]
    in RCTView (created by View)
    in View (created by SectionList)
    in RCTView (created by View)
    in View (created by SectionList)
    in RCTView (created by View)
    in View (created by SectionList)
    in SectionList (created by AuthProfile)
    in RCTView (created by View)
    in View (created by ParallaxScrollView)
    in RCTScrollContentView (created by ScrollView)
    in RCTScrollView (created by ScrollView)
    in ScrollView (created by ParallaxScrollView)
    in RCTView (created by View)
    in View (created by ParallaxScrollView)
    in ParallaxScrollView (created by ParallaxScroll)
    in ParallaxScroll (created by AuthProfile)
    in RCTView (created by View)
    in View (created by ViewContainer)
    in ViewContainer (created by AuthProfile)
    in AuthProfile (created by Connect(AuthProfile))
    in Connect(AuthProfile) (created by SceneView)
    in SceneView (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in RCTView (created by View)
    in View (created by AnimatedComponent)
    in AnimatedComponent (created by Card)
    in Card (created by Container)
    in Container (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in CardStack (created by Transitioner)
    in RCTView (created by View)
    in View (created by Transitioner)
    in Transitioner (created by CardStackTransitioner)
    in CardStackTransitioner (created by Unknown)
    in Unknown (created by Navigator)
    in Navigator (created by NavigationContainer)
    in NavigationContainer (created by SceneView)
    in SceneView (created by TabViewAnimated)
    in RCTView (created by View)
    in View (created by TabViewAnimated)
    in RCTView (created by View)
    in View (created by TabViewPagerPan)
    in RCTView (created by View)
    in View (created by AnimatedComponent)
    in AnimatedComponent (created by TabViewPagerPan)
    in TabViewPagerPan (created by TabViewAnimated)
    in RCTView (created by View)
    in View (created by TabViewAnimated)
    in TabViewAnimated (created by TabView)
    in TabView (created by withCachedChildNavigation(TabView))
    in withCachedChildNavigation(TabView) (created by Unknown)
    in Unknown (created by Navigator)
    in Navigator (created by NavigationContainer)
    in NavigationContainer (created by SceneView)
    in SceneView (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in RCTView (created by View)
    in View (created by AnimatedComponent)
    in AnimatedComponent (created by Card)
    in Card (created by Container)
    in Container (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in RCTView (created by View)
    in View (created by CardStack)
    in CardStack (created by Transitioner)
    in RCTView (created by View)
    in View (created by Transitioner)
    in Transitioner (created by CardStackTransitioner)
    in CardStackTransitioner (created by Unknown)
    in Unknown (created by Navigator)
    in Navigator (created by NavigationContainer)
    in NavigationContainer (created by App)
    in Provider (created by App)
    in App
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer

This occurs here. It is impossible to apply color to View, only to Text, apparently.
Why do you need a condition here, can not we just leave only a View?

Yup just found that @lex111 :+1: The problem is that we need a <Text> element if title is a string, because we'll get a nasty crash if we have raw text inside a <View>. We should only need the padding: 15 style there, so I'm in favor of just getting rid of all the styles on that view and keeping only the padding.

While we have this open, there is another place that a warning pops up and that's in the PullRequestDiff screen. Currently there's a warning that keys aren't applied to a list and will appreciate if someone who takes this on can take a look there too 🙏

I think we have these warnings resolved now 💃


⚠ Processors has degraded performance

In f402577, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 76 ms

Resolved: Processors performance has improved in ca2277b.


Event that starts and ends on next day is not shown in the view by day and week

If I create an event that starts on one day and ends, for example, at dawn the next day, it is not shown in the week and day view, since the dates are well set.

Duplicate of #171 .


core_sign_update: use pkcs11 openssl engine

To be tested with an actual payload.
TODO:

[ ] add the openssl configuration to consume the PCKS11 engine (https://github.com/OpenSC/libp11#using-the-engine-from-the-command-line)

[openssl_init]
providers = provider_sect

engines=engine_section

[engine_section]
pkcs11 = pkcs11_section

[pkcs11_section]
engine_id = pkcs11
dynamic_path = /usr/lib64/engines-3/libpkcs11.so
MODULE_PATH = /usr/lib64/pkcs11/opensc-pkcs11.so
init = 0

[ ] include the "generate_payload" in this repo ? NOTE: we need to update the key URI to pass the pin-code
[ ] include the "download_payloads" in this repo?
[ ] provide some documentation (like the pcscd daemon that needs to run in foreground)
[ ] remove the hack around USB group for acct-user/pcscd (see the comment: https://github.com/flatcar/scripts/pull/1149#discussion_r1426745596)

The workflow would be the follow:

start the SDK
start the pcscd daemon
plug the device
./download_payloads alpha:3815.0.0 ...
./generate_payloads ./data/ keys (with a for loop on the releases in ./data/)

+1 for adding download_payloads/generate_payload to this repo. (What I don't like is making it mandatory to run inside the SDK because of of common.sh)

start the pcscd daemon

Inside the SDK or can I use the one from the host? Would it conflict with the one from the host? That was the case before and one had to manually stop it for successful signing.

./download_payloads alpha:3815.0.0 ...

./generate_payloads ./data/ keys (with a for loop on the releases in ./data/)

In the end a big wrapper would be nice that downloads and uploads.

Let's merge and then remove the scripts from flatcar-build-scripts, otherwise things can get out of sync


Redesign footer

Redesigned footer and made it completely responsive !!
Issue No = #90

@Jagroop2001 Please review this PR !!


[REACT VERSION] The Research Page and Reference Pages are not visible

The given pages are not visible.

I want to work on this under JWOC

@anushbhatia kindly have a look at this issue and the PR


gam-4.2-windows.zip treated as a threat by Symantec

I tried to configure GAM in one my office machine and I got a warning from Symantec antivirus that the gam-4.2-windows version has traces of SONAR.Heuristic.159. I've not seen this with earlier versions of GAM and I was pretty surprised to see it!
Attaching the screenshots for your reference.

AV vendors flag legit files as viruses all the time. There's nothing GAM can really do about this. Thanks for raising though.


Module org.dozer.mapper cannot find javax.el implementation

During the camel-dozer component startup 2 warnings are printed in the log:

2018-12-05T22:42:44.076Z WARN [org.dozer.el.ELExpressionFactory] (MSC service thread 1-7) > javax.el is not supported; Failed to resolve ExpressionFactory, com.sun.el.ExpressionFactoryImpl from [Module "org.dozer.mapper" version 6.2.0 from local module loader @37e547da (finder: local module finder @2b6856dd (roots: /wildfly/provided_modules,/wildfly/modules,/wildfly/modules/system/layers/fuse,/wildfly/modules/system/layers/base))]
2018-12-05T22:42:44.076Z WARN [org.apache.camel.converter.dozer.MapperFactory] (MSC service thread 1-7) > Expressions are not supported by Dozer. Are you missing javax.el dependency?

The missing dependency is forcing dozer to use the NoopELEngine. Adding the org.glassfish.javax.el dependency directly to the module.xml is resolving the issue:
<?xml version="1.0" encoding="UTF-8"?>
<module name="org.dozer.mapper" xmlns="urn:jboss:module:1.1">
    <properties>
        <property name="jboss.api" value="private"/>
    </properties>
    <resources>
        <resource-root path="dozer-core-6.2.0.jar"/>
    </resources>
    <dependencies>
        <module name="javax.api"/>
        <module name="javax.el.api"/>
        <module name="org.glassfish.javax.el"/>
        <module name="javax.xml.bind.api"/>
        <module name="org.apache.camel.component.dozer"/>
        <module name="org.apache.commons.beanutils"/>
        <module name="org.apache.commons.io"/>
        <module name="org.apache.commons.lang3"/>
        <module name="org.objenesis"/>
        <module name="org.slf4j"/>
    </dependencies>
</module>

This is likely out of date. In the current master, we have [this|https://github.com/wildfly-extras/wildfly-camel/blob/master/feature/modules/src/main/resources/modules/system/layers/fuse/com/github/dozermapper/main/module.xml] dozer module. Note, it is using com.github.dozermapper. In the Camel dependency tree, there is no org.glassfish.javax.el
Perhaps you can reproduce what you are seeing, by modifying this [test case|https://github.com/wildfly-extras/wildfly-camel/tree/master/itests/standalone/basic/src/test/java/org/wildfly/camel/test/dozer].

IIRC the Glassfish dependency is defined as both optional and scoped as provided. Hence we don't include it in our distro by default:
https://github.com/DozerMapper/dozer/blob/6.4.1/core/pom.xml#L101-L106
Also, the camel-dozer Karaf feature is including the glassfish javax.el:
https://github.com/apache/camel/blob/master/platforms/karaf/features/src/main/resources/features.xml#L554

It's also in the component POM, so maybe we're excluding it somewhere:
https://github.com/apache/camel/blob/camel-2.23.0/components/camel-dozer/pom.xml#L57-L61


Fix transfort.rs: correct multiplicative inverse.

The multiplicative inverse of 3 in Z/2^64Z is<PHONE_NUMBER>2473034411.
It can be shown quickly:
ϕ(2^64) =<PHONE_NUMBER>854775808 (Euler's totient function)
3^(ϕ(2^64) - 1) ≡<PHONE_NUMBER>2473034411 (mod 2^64).
Obviously, the same result can be found with the extended Euclidean algorithm.

You're right. My question: How did you find this project?

Twitter ;-) (https://twitter.com/jedisct1/status/802810501851971588)

Nice!


toggle-button doesn't input project name automatically

Environment details

Operation system: Microsoft Windows 10 Pro
Extension version:  1.7.0

Expected behavior
toggle-button input project name automatically when click button.
Actual behavior
toggle-button doesn't input project name automatically when click button.
Steps to reproduce the behavior

Move https://todoist.com/app
Click toggle-button

Additional details
I investigated and found that class name had changed from .pname to .project_item__name on todoist web page.

Maybe related code is below.
https://github.com/toggl/toggl-button/blob/e7b1d28d277dcc1bc2822e1347af2a24554209b0/src/scripts/content/todoist.js#L7

Another Toggl Button user complained about this.
See case 567912 for details.

I have the same issue.
This kind of makes the button not useful.

Another complaint. See Desk case 575965

Another user mentioned this problem. See case 580867

Aaaand another one. Case 584650

Another user came across the issue case/649981

Another user reporting the issue - case/654935

case/665225

Fixed by #973. Needs to be deployed, though.


Function .centers() returns only 3 items

Hello :)
I wanna use .centers() but when i use this I get only 3 points

0: [0, 0]
1: [1.7320508075688772, 0]
2: [0.8660254037844386, 1.5]

Why visible and not visible hexagon centers points is not return ?

You should probably set the hexbin's .extent() before calling .centers()
See https://github.com/d3/d3-hexbin#hexbin_centers


Jobs: Show HP number in Healer

Healer's HP is as important as Tank's.
Many times I saw healers are busy at restoring other's HP,
but ignore their own HP is not enough and got killed in AoE, causing party wipe.
Also healer who makes barrier sometimes want to know how much damage can it nullify.
So I think it's necessary to show HP number in Healer.

That seems reasonable to me, thanks!


user-guide/configmap does not document self-updating

I can't find anything that documents the ability to have configmap volumes self-updated.
Secrets documents it with two sentences: "When a secret being already consumed in a volume is updated, projected keys are eventually updated as well. The update time depends on the kubelet syncing period."
@pmorie

/remove-lifecycle stale
/lifecycle frozen

/sig apps

/priority backlog
#16085 should have an nice high priority though


Is there a method to cap shortest path search distances?

I am using csgraph.shortest_path to return an all-pairs distance and predecessor array.
Is there a manner to cap the search distance?
If not, is this something that could be added?

I see that the dijkstra shortest path method does have a limit input which will work for my purposes.
I would close this except that it seems that the limit flag is missing from the method heading.

Indeed it is missing, I'll open a PR momentarily to fix it


nacos开启权限控制后request: /nacos/v1/ns/operator/metrics failed

nacos开启权限控制后，用新增的非管理员账号，启动时报错ERROR com.alibaba.nacos.client.naming - request: /nacos/v1/ns/operator/metrics failed，
msg: Whitelabel Error PageThis application has no explicit mapping for /error, so you are seeing this as a fallback.Thu Apr 23 10:54:42 CST 2020There was an unexpected error (type=Forbidden, status=403).authorization failed!

请提交到nacos社区，Nacos


Different styling for unsupported file types

The files from the File Tree which cannot be opened/selected should be styled differently, or maybe hidden?

decided to:

excluded files: italic + grey, arrow cursor
unsupported files: grey, arrow cursor


Add support for mlirExternalPassSignalFailure

In #251, I overlooked to mlirExternalPassSignalFailure function. This requires a wrapper around MlirExternalPass, which is a CAPI pointer for the ExternalPass subclass of ::mlir::Pass that represents any external pass and can expose some private methods from Pass.
I decided to call this wrapper ExternalPassHandle, since we are already using ExternalPass for the trait.
I originally made these changes a while ago when experimenting with extending the CAPI for analyses, but decided to make this PR, since mlirExternalPassSignalFailure is part of the MLIR CAPI already.

Thank you for the PR!


Ignore custom class name attribute

Is there a possibility ignore custom named attributes in HTML?  Or that you want these removed from your css?
Uncss looks at pages the same as a web browser looks at them. If the web browser doesn't understand css-class, then no, uncss will not understand it either, and the attributes will be ignored. (Is this what you want?) As a result, it will see those classes as being not-in-use, and so will delete  custom from the resulting css.
If your web page is using clientside javascript to turn that css-class into a proper class attribute, then uncss should also process/run that clientside javascript, and create an HTML with those class elements in-use. And given that they would then be in-use, uncss would then not remove them from the generated trimmed css files, and you would see custom in the resulting css.
Those are basically your options, based on how Uncss works. Think of uncss like a web browser loading a webpage. And then walking the HTML tree to figure out what's in use. That's basically it.

As you say, my web page is using clientside javascript to turn that css-class into a proper class attribute, so this a right answer. Thank you.


Use consistent names for objects in comments.

Until now the code is not consistent about naming classes or objects.
For example entity attribute type instance is sometimes called instances of entity attribute types.
Remove all unconsistent comments.

The quality of the comments has been improved.
See: https://github.com/inexorgame/entity-system/commit/5c8f9ecc93565dee05452dc1236e548cc277dd5f.


build collector NPE.

Hi,
I am getting java.lang.NullPointerException errors on trying to start the hudson collector. Other collectors are working fine. But I am not able to get the jenkins collector to work.
2015-07-31 09:28:27,591 ERROR o.s.boot.SpringApplication - Application startup failed
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'hudsonCollectorTask': Invocation of init method failed; nested exception is java.lang.NullPointerException
    at org.springframework.beans.factory.annotation.InitDestroyAnnotationBeanPostProcessor.postProcessBeforeInitialization(InitDestroyAnnotationBeanPostProcessor.java:136) ~[spring-beans-4.1.4.RELEASE.jar!/:4.1.4.RELEASE]
    at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.applyBeanPostProcessorsBeforeInitialization(AbstractAutowireCapableBeanFactory.java:408) ~[spring-beans-4.1.4.RELEASE.jar!/:4.1.4.RELEASE]

Full stacktrace of the error is here: https://gist.github.com/bimlendu/82a5be5606464fd17743
Thanks.

can you provide your properties file for build collector here

Its same as in the readme.
spring.data.mongodb.database=dashboarddb
spring.data.mongodb.host=<IP_ADDRESS>
spring.data.mongodb.port=27017
spring.data.mongodb.username=dashboarduser
spring.data.mongodb.password=dA$hb0Ardu$eR
hudson.cron=0 0/5 * * * *
hudson.servers[0]=https://jenkins.host.com/
hudson.saveLog=true

Hi @amitmawkin , any help on this?
The hudson collector jar that gets built has following dependecy jars in lib:
[root@001 tmp]# ls lib/ | grep spring
spring-aop-4.1.4.RELEASE.jar
spring-beans-4.1.4.RELEASE.jar
spring-boot-1.2.1.RELEASE.jar
spring-boot-autoconfigure-1.2.1.RELEASE.jar
spring-boot-starter-1.2.1.RELEASE.jar
spring-boot-starter-data-mongodb-1.2.1.RELEASE.jar
spring-boot-starter-logging-1.2.1.RELEASE.jar
spring-context-4.1.4.RELEASE.jar
spring-core-4.1.4.RELEASE.jar
spring-data-commons-1.9.1.RELEASE.jar
spring-data-mongodb-1.6.1.RELEASE.jar
spring-expression-4.1.4.RELEASE.jar
spring-test-4.1.4.RELEASE.jar
spring-tx-4.1.4.RELEASE.jar
spring-web-4.1.0.RELEASE.jar

Do you think, the version of spring-web (4.1.0) is causing the issue. Most others are 4.1.4. Or this is acceptable?

@bimlendu the prefix hudson in applications.properties is wrong. Use this instead:
Jenkins.cron=0 0/1 * * * *
Jenkins.servers[0]= https://jenkins.host.com/
Jenkins.saveLog=false

Thanks , I have fixed the issue and corrected the case too, If you pull the latest changes
jenkins.cron=0 0/1 * * * *
jenkins.servers[0]= https://jenkins.host.com/
jenkins.saveLog=false
Let me know if this fixed it and you can close the issue

Also updated readme.md to reflect the same

Thanks @rogerwelin and @amitmawkin . Jenkins collector is starting up ok now. Closing this issue.

Hi @rogerwelin and @amitmawkin
Can you please have a look at issue "cannot run jenkins collector #1793" opened by me.
I am not able to run any collector and the java -jar command for the collector just does not get executed.
Could you please help!
It has been a week i am struggling now.
Thanks
Abhishek Somani


efConstruction and M

In the paper, to find the neighbors of each node, you need to first find efConstruction candidate neighbors by SEARCH_LAYER function. Then find M neighbors among the candidate neighbors for connection by SELECT_NEIGHBORS_HEURISTIC function. So you need efConstruction to be always greater than M. But why in the experiment, your ef Construction appears to be less than M.

When efContrcution is less than M, how is it finding neighbors


Error when uploading in xiao esp32s3

c:\Users\user\Arduino\libraries\MPU9250_WE\src/MPU6500_WE.h:337:28: error: section attribute not allowed for 'accOffsetVal'
337 |     RTC_DATA_ATTR xyzFloat accOffsetVal; // to be able to keep offsets after deep sleep
|                            ^~~~~~~~~~~~
c:\Users\user\Arduino\libraries\MPU9250_WE\src/MPU6500_WE.h:338:28: error: section attribute not allowed for 'gyrOffsetVal'
338 |     RTC_DATA_ATTR xyzFloat gyrOffsetVal;
|                            ^~~~~~~~~~~~

@giannisj5 Something I tried found its way into the the release 1.2.8. Now it should work again (1.2.9).

thank you very much, very fast fix. Everything is working perfect


GoogleDrive API v3 usage limit

I came across this the following error after roughly testing if packratt could pull all my measurement sets from my google drive folder. There were 48 in total so I ran 48 separate packratt get methods in a python3 script:
packratt.get(f'/MSC_DATA/MS/KAT7_{NTIME}_7_{NCHAN}.tar.gz', 'datasets/ms/')

where NTIME and NCHAN are indicators for the specific MS. Nearing the end of the downloads, packratt stopped working due to google throwing a request error code 403 for the download url:
Traceback (most recent call last):
  File "/home/brian/Code/brianWelman/final/lib/python3.6/site-packages/packratt/downloads.py", line 91, in requests_partial_download
    response.raise_for_status()
  File "/home/brian/Code/brianWelman/final/lib/python3.6/site-packages/requests/models.py", line 943, in raise_for_status
    raise HTTPError(http_error_msg, response=self)
requests.exceptions.HTTPError: 403 Client Error: Forbidden for url: https://doc-0c-74-docs.googleusercontent.com/docs/securesc/ha0ro937gcuc7l7deffksulhg5h7mbp1/n46bea6cf1caaduraqda42ptdlcbmo96/1613487750000/00827075692661388493/*/1rwrycA2AQ6jPZU15JZYHikIfGdqRs3i7?e=download

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "main.py", line 80, in <module>
    packratt.get(f'/MSC_DATA/MS/KAT7_{t}_7_{c}.tar.gz', 'datasets/ms/')
  File "/home/brian/Code/brianWelman/final/lib/python3.6/site-packages/packratt/interface.py", line 58, in get
    sha256_hash = downloaders(entry['type'], key, entry)
  File "/home/brian/Code/brianWelman/final/lib/python3.6/site-packages/packratt/dispatch.py", line 19, in __call__
    return fn(*args, **kwargs)
  File "/home/brian/Code/brianWelman/final/lib/python3.6/site-packages/packratt/downloads.py", line 124, in download_google_drive
    params=params)
  File "/home/brian/Code/brianWelman/final/lib/python3.6/site-packages/packratt/downloads.py", line 93, in requests_partial_download
    raise Exception(e)
Exception: 403 Client Error: Forbidden for url: https://doc-0c-74-docs.googleusercontent.com/docs/securesc/ha0ro937gcuc7l7deffksulhg5h7mbp1/n46bea6cf1caaduraqda42ptdlcbmo96/1613487750000/00827075692661388493/*/1rwrycA2AQ6jPZU15JZYHikIfGdqRs3i7?e=download

Which as I see now is api usage limit reached error. See the following article for more. This is based off of my google drive api I have enabled and to increase the quota, I would have to add a super admin account (which I do not know how to do) to manually change those usage limits. Otherwise, I have to wait for the quota to reset. No packratt get command will work with this specific google drive folder until it does.
Anyway around this? It is a slight problem when I try to pull multiple MS to perform tests. Otherwise, I will attempt to move to Rhodes servers instead.

Thanks for reporting this @brianwelman2, I think you're the first person trying to set up this quantity of downloads, so it's not surprising that you're testing the limits.

Which as I see now is api usage limit reached error. See the following article for more. This is based off of my google drive api I have enabled and to increase the quota, I would have to add a super admin account (which I do not know how to do) to manually change those usage limits. Otherwise, I have to wait for the quota to reset. No packratt get command will work with this specific google drive folder until it does.

I think what we're seeing here are the limitations of Google Drive as it's not really a content delivery network (CDN) as suggested by  this stackoverflow answer. I wasn't aware of these limits so thanks.

Anyway around this? It is a slight problem when I try to pull multiple MS to perform tests.

I guess that it's complaining about many small requests. Perhaps you could Tar them all into one big file for download?

Otherwise, I will attempt to move to Rhodes servers instead.

This may be the short term answer. Otherwise, S3 buckets or some other dedicated CDN service may be the way forward.

Okay, thank you for the clarification. I realised I would be pushing it for
this kind of use, but with the cache in use with packratt, it can be
avoided. I will have a look at the suggested CDN services, but for now it
works perfectly fine for me.
On Wed, 17 Feb 2021, 09:20 Simon Perkins<EMAIL_ADDRESS>wrote:

Thanks for reporting this @brianwelman2 https://github.com/brianwelman2,
I think you're the first person trying to set up this quantity of
downloads, so it's not surprising that you're testing the limits.
Which as I see now is api usage limit reached error. See the following
article for more. This is based off of my google drive api I have enabled
and to increase the quota, I would have to add a super admin account (which
I do not know how to do) to manually change those usage limits. Otherwise,
I have to wait for the quota to reset. No packratt get command will work
with this specific google drive folder until it does.
I think what we're seeing here are the limitations of Google Drive as it's
not really a content delivery network (CDN) as suggested by this
stackoverflow answer https://stackoverflow.com/a/10313416. I wasn't
aware of these limits so thanks.
Anyway around this? It is a slight problem when I try to pull multiple MS
to perform tests.
I guess that it's complaining about many small requests. Perhaps you could
Tar them all into one big file for download?
Otherwise, I will attempt to move to Rhodes servers instead.
This may be the short term answer. Otherwise, S3 buckets or some other
dedicated CDN service may be the way forward.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ratt-ru/packratt/issues/19#issuecomment-780359952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE5OS5DS5G4PE2XFDD6KFNLS7NU3RANCNFSM4XWU2YAA
.


Add DoSomething.org into our App Store description

If you search on app store for dosomething.org, we don't show up, because there's no mention of dosomething.org anywhere in the description (our seller name is listed as Do Something, Inc per our DNB number).
This won't be handled through code, but is a task for when we submit our next release. cc @lkpttn

I added it into the description for now, but I'm going to leave the issue open to remind us to change the keywords in the next version.

@lkpttn Should we close this out -- did you add as keyword?

Done and done


feat: updated for Oyster Demo

updated for Oyster demo

Will open a new PR later with needed updates


Environment variables not intended for confidence might interfere with configurations

As confidence will read any ennvar that happens to match a name being loaded, things might interfere 'normal' operation. Should we maybe require a prefix to envvars to make the reader pick them up? Allow a prefix to be specified by the user?

Envvar loading can be turned off, maybe not overcomplicate this further...


flow annotations + arrow-parens?

Not sure if this is the appropriate place for this but I figure i'll start here. If i have
const log = (a: string) => console.log(a)
log('hi')

and eslintrc
{
    "env": {
        "es6": true
    },
    "parser": "babel-eslint",
    "parserOptions": {
        "ecmaVersion": 6,
        "sourceType": "module"
    },
    "rules": {
        "arrow-parens": [2, "as-needed"]
    }
}

Then eslint complains that I have an unnecessary parens around the parameter 'a'. But in order to use the flow type (I believe) i need to wrap it like this. Is there some kind of workaround for this?

I think this should be brought up with eslint itself. As type annotations are now finally specified (https://github.com/estree/estree/blob/master/extensions/type-annotations.md), eslint is now also considering to respect type annotations in the rules (see here for example https://github.com/eslint/eslint/issues/5211#issuecomment-213064355)

Did you guys create an issue about this on the ESLint repo ? I can't find anything.

+1, seeing this a lot, so I end up adding a bunch of eslint-disable-next-line arrow-parens annotations.
Use case, something like:
handleChange = (e: Event) => this.props.onChangeFn(this.props.id, e);

There's a PR open with ESLint fixing this: https://github.com/eslint/eslint/pull/7436

https://github.com/eslint/eslint/pull/7436 is fixed


Fix react-script package directory resolution

Given that the react-scripts package has no main field i am resolving to package.json
This will work on every OS so the code after could be even removed
Fix #58

Hi @remorses, if this will work for all operating systems, can we remove the unnecessary code too?
Thanks for this!

OK i removed the unused code

Hi @remorses, this behaviour is complex - agreed - but it's on purpose. People are (kind of) encouraged to fork react-scripts when they want to add custom behaviour, and this logic resolves those custom forks.
This PR unfortunately breaks that behaviour... I hope that makes sense?


Map marker isn't showing

When I click on the open location icon of a specific talk or something, the map opens without a marker, which could be confusing to users.

I would like to work on this issue.

Go ahead! @saketkumar95

@Elgayar @saketkumar95 Any progress?

@mananwason Yes. I'll send the PR today. Was busy in my university exams.

@mananwason Looks like @saketkumar95 is still inactive. Can I work on this?

@anantprsd5 Yeah i just missed this issue. If you want to work on this then go ahead. :)

@saketkumar95 @Elgayar  I can see the marker in the map for every location. Can you please tell me how to reproduce this issue?

Map development has taken place in meantime. Therefore it is unclear if this issue still persists. Open if necessary and provide more details and info for reproducing it.