Automated cherry pick of #1495: fix export data

Cherry pick of #1495 on release/2.11.0.
#1495: fix export data

/lgtm
/approve

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11.0@ef717be). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.11.0   #1496   +/-   ##
================================================
  Coverage                  ?    7.7%
================================================
  Files                     ?     411
  Lines                     ?   62834
  Branches                  ?       0
================================================
  Hits                      ?    4839
  Misses                    ?   57586
  Partials                  ?     409

Flag
Coverage Δ

#aFlag
7.7% <0%> (?)

Impacted Files
Coverage Δ

pkg/appsrv/handlerinfo.go
43.33% <0%> (ø)

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 ef717be...8b59048. Read the comment docs.


Rename "Server Profile" context menu entry to something else

Discord has per-server profiles that let you edit your nick/avatar per server. The context menu for that is "Edit Server Profile". The entry for the GuildProfile plugin should be changed to something less similar. Maybe "Server Info"?

Yeah, the "Edit Server Profile" can be confused with "Server Profile" option. But in my opinion, the option name should stay as it is because the "server profile" is intended to look pretty similar to user profiles. Renaming it to "Server Info" (for example) may not make sense. In my opinion.


Update README.md

Google play doesn't offer Cyndia Substrate anymore. Updated with link to APK file

http://www.cydiasubstrate.com/download/com.saurik.substrate.apk

Please merge this


Simplify use of Blockstack

Fixes #216 simplify use of Blockstack by integrating the react-blockstack library.

Awesome @njordhov - I just merged in another PR so once this is rebased I can merge it in. Really appreciate the Blockstack expertise you and @friedger are bringing, it's very much needed!

LGTM.
After the merge, passing the authentication status with authed={authenticated} can be substituted with local uses of the useBlockstack() hook. I'll open a separate issue with the details.


keepDirtyOnReinitialize and enableReinitialize doesn't work with objects

When having a complex form with formSections that starts out as undefined or null the reinitialize doesn't work.
See example here: https://codesandbox.io/s/xj74l9y5r4
countData should update when pressing the "Load Account" button

Might be related #3057

That definitely might be related since I also have the issue in 6.8.0 and after.

Same issue here. It's specially problematic when you have forms that are submitted (and receive new initialValues) when they change.
https://codesandbox.io/s/ywxqk37pzz


How to draw a loss curve

hi！My question is the same as yours, do you know how to do it? If you know, can you give me a copy of your code.Thank you


watchify - very slow

Hi,
We're using watchify in a relatively large project (more than 30k lines of code). We have ES6 features and React JSX code that we transpile with Reactify (tried Babel(ify) but it was even slower). We moved from Grunt to Gulp and also upgraded from Browserify 4.2.3 to 9.0.8, the initial build takes >20 secs, which is fine, but the rebuild is 4 times slower than it was. With the older versions of Browserify and Watchify we've had under 1sec on rebuild, now even if we're using the watchify command it takes longer:
$ watchify src/js/application.js -d -t [reactify --es6] -o dist/bundle.js -v
7833167 bytes written to dist/bundle.js (22.25 seconds)
7833180 bytes written to dist/bundle.js (5.11 seconds)
7833167 bytes written to dist/bundle.js (4.96 seconds)
7833298 bytes written to dist/bundle.js (4.80 seconds)

If I remove the debug option, it goes around 1 sec on rebuild:
$ watchify src/js/application.js -t [reactify --es6] -o dist/bundle.js -v
2692764 bytes written to dist/bundle.js (18.38 seconds)
2692729 bytes written to dist/bundle.js (1.64 seconds)
2692730 bytes written to dist/bundle.js (1.38 seconds)
2692764 bytes written to dist/bundle.js (1.42 seconds)

But we're dead without sourcemaps :).
Any thoughts?
Thanks,
Raul.

It could be changes in reactify. I'm not sure when they turned on sourceMaps by default so that could be yet another slowdown. Combining source maps is really expensive. You could try turning off reactify's source maps by making this false but leaving browserify's source maps on.

You could also test out some of the optimizations discussed here.

Thanks @zertosh - turning Reactify sourceMap off reduced the rebuild time with 2 seconds or more:
$ watchify src/js/application.js -d -t [reactify --es6] -o dist/bundle.js -v
7004983 bytes written to dist/bundle.js (17.88 seconds)
7004974 bytes written to dist/bundle.js (2.71 seconds)
7004897 bytes written to dist/bundle.js (2.32 seconds)

@mattdesl I'll have a lock on Browserify 1208 issue, thanks for the link.

@raulmatei I see you made https://github.com/andreypopp/reactify/pull/64 :+1:.
You could also give babel/babelify a try. The acorn parser is faster. It can even do the equivalent of envify via the utility.inlineEnvironmentVariables transform - saving an extra AST parse (if that's a thing you use).

@zertosh - thanks for closing the issue, I was going to close it myself as it's not related with watchify. Sorry for not doing it in time. I'm using Babelify in other projects (smaller ones), and it fits great. For this project there was not too much time for tweaks so I've just dropped it in, saw that it was slower than Reactify and rolled back. I'll spend some more time on it next days...


if width and height value is small ,the border will have a defect

Thank you @Governhong1205.
The rotation seems to be causing this issue.
[path applyTransform:CGAffineTransformRotate(CGAffineTransformIdentity, M_PI / 2)];

I will investigate more and come up with a fix as soon as possible.

Same issue here

I tried to do like this , it will be ok

Worked perfectly !  Thanks
I have to do this code again every time I update my cocoapods though. If you have time to update your code with a fix it would be great ;)

This fix works, but breaks some of the animations unfortunately.
I need to find time to work on a "clean fix". Thank you your feedback!

Are you talking about the OneStroke type? @Boris-Em
I think using this way is temporarily fit to me because I dont use this type now.^_^.

Yes, that's the one I'm talking about @Governhong1205. It's probably not the most popular one anyway.
So if you're not using it, you should be fine!


If i don't want show tag H1?

hi, if i don't want show tag h1 in all pages of carousel and i don't write nothing why appear tag h1 empty in all slide...
Is possible to eliminate this like if i don't write nothing in admin for 2/3 slides i don't want appear when i read html code...
thank and sorry for my italian english

Do you mean h2 element? I don't see any h1 elements in this template. h2 element is used to show page's title which is almost used in all pages and there is no options to turn it off. If you want to can change the line

{{ page.title }}

in the files in user/themes/x-corporation/templates/modular/ to
{% if page.title %}
{{ page.title }}
{% endif %}

Sorry, my english is very bad..
I want know if in carousel page is possible, if i don't write nothing in header in 2 or 3 slide, don't show in html tag h1 like this

thanks

I see. In user/themes/x-corporation/templates/modular/carousel.html.twig you change the line

{{ slide.heading }}

to

{% if slide.heading %}{{ slide.heading }}{% endif %}

thanks


Animate, Animations pbs

Hello,
I'm currently trying to port your code in typescript/react to have a native typescript ui library... I've already coded many elements without problems (while trying to use typescript features like typing, generics, class hierarchy, ...)... but once I've started with Animate, Animations, Dimmer, Modal, ..., I've encounter big problems to make thinks working...
My question is : is the code on github currently works for the latter components? I see for instance than Animations use the component Transition which doesn't exists in react...
best regards
xavier

@hallister stated he built his own animation library
afaics he's using react-addons-transition-group for Transition.

Yep, and the react-addons-transition-group has some bugs, so it will have to be brought over at somepoint. But basically you need React with addons for now.

I particular replaced your Animation with react-motion-ui-pack (https://github.com/souporserious/react-motion-ui-pack/), only Modal with Popup left (but i'm going to rework them to use new RenderToLayer). warning messages were gone and select labels are now animated :smile_cat:
Are you interested or still preferring to stick with your current animation solution?

Moving to react-motion and react-motion-ui-pack now


perf(ui): use react virtualized in channels management screen

Description:
Resolves #1537
It's recommended to test in production mode

How Has This Been Tested?
Manually
Types of changes:
Perf improvement

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.06%) to 18.658% when pulling 962cdb3eaf9f6806b5d24ae3f8cb01a8ea3317e0 on korhaliv:perf/virtualize-channels into 946312a366a5a397240e70da7fc4d0e7dee2ebb4 on LN-Zap:next.

Coverage decreased (-0.06%) to 18.658% when pulling 962cdb3eaf9f6806b5d24ae3f8cb01a8ea3317e0 on korhaliv:perf/virtualize-channels into 946312a366a5a397240e70da7fc4d0e7dee2ebb4 on LN-Zap:next.


Binding to tableView retains a strong reference to the tableView

I recently created a UITableView that used quite a bit of memory while on screen.  Which made memory shoot up quickly when navigating around.  It seemed that in the callback function for updates to the tableView it was calling beginUpdates() on the tableView that was passed in, creating a strong reference.  I made a fork of the repo to test a solution which seemed to work.  Moved the call for beginUpdates() into the .didChangeContent callback since all changes are tracked via Sets anyways.  Also, passed in tableView as weak instead of unowned.  Was there a reason it should be unowned?  Seemed like that could cause a crashing race condition if the UITableView is getting cleaned up and FetchedRequestController's callbacks happen.
Here's the changes I made that seemed like a fix to me:
https://github.com/Alecrim/AlecrimCoreData/compare/master...Maxwellimus:master
Any guidance on how to get these changes reviewed and back into the develop branch would be great.  Thanks!

Hello.
Thank you for catching this issue. I did not have it on my projects, but the reference being weak instead unowned is a more secure approach.
I'm committing a new version fixing this issue (Swift 2.3 compatible version.)
About to get your contributions reviewed and added to ACD please take a time to review the instructions in this link: https://help.github.com/articles/creating-a-pull-request.

Hey,
Thanks for integrating the changes.  Sorry I didn't set it up as a pull request the first time.  There was one thing missing in the code integration though.  Moving the beginUpdates call to the weak referenced tableView.
I created a pull request here with a fix:
https://github.com/Alecrim/AlecrimCoreData/pull/156


oauth2/consent: id token expiry should be configurable

closed by https://github.com/ory-am/hydra/commit/24d34b312ee95f82def54d9f6a48e1cf3f5e3902


Possible Error 409 from Tove if submitting too many requests at a time

Potential "Save Data" Bug

It's possible to get an Error 409 from Tove if you send too many POST/PUT/PATCH/whatever requests in one go.
Alice does not know how to handle an Error 409.

Dev Notes
Replication: (At least, what I did to consistently get the error above)

Go to http://local.zooniverse.org:3000/projects/1764/workflows/3085/groups/12345/subjects/85068/edit
Observe first transcribed line is "Boston March 3 - 1837", second line is "Friday noon"
Click & drag to move "Boston March 3 - 1837" to second line (action 1)
Click on "Boston March 3 - 1837" to open popup (action 2)
Write new text: "BOSTON MARCH 3 - 1837". Click "Replace with Selected" (action 3)
Click "Close" (action 4)
Now RAPIDLY CLICK ON THE UNDO BUTTON 💥 💥 💥 💥 💥 💥 💥 💥
Expected: all 4 actions should be undone and saved to Tove
Actual: it looks like actions 1 to 4 are undone in on the front end. Then, after a few seconds of blissful silence, the whole app crashes.

Console error message: PATCH https://tove-staging.zooniverse.org/transcriptions/85068 409
Further adventures in console error messages:getError.js:25 Uncaught (in promise) TypeError: Cannot read property 'error' of undefined
    at getError (getError.js:25)
    at TranscriptionsStore.js:309

After reloading, it's apparent that actions 2 to 4 are undone, but action 1 (rearranging to second line) remains saved to the database.

This is likely a race condition where two PATCH requests are occurring near-simultaneously (for action 3 and action 1, to be specific) before the first one could be fully processed by the database.
Other ways to replicate the problem:

Shuffle the order of the first few lines of any Subject REALLY REALLY FAST.

Status
Major problem (crashes application and leaves state of Subject transcriptions in a potential limbo) but possibly low probability of happening? I mean, what maniac would perform so many changes in one go?
...but if Tove ever takes longer than it currently does to confirm a PATCH request (e.g. user has a slow network, or the payload is megahuge), or if users act faster than the PATCH request can resolve, then we've got a problem.
Note that one sensible solution is to make the the "save changes to database" system action to trigger only once every X seconds (and only if there's no active "save changes to database" action currently happening) but that would require a major rethink of the ALICE design.

Great catch!

Major problem (crashes application and leaves state of Subject transcriptions in a potential limbo) but possibly low probability of happening? I mean, what maniac would perform so many changes in one go?

"what maniac" - this will for sure happen. I'll plan accordingly in the frontend. Maybe when patches occur in a quick succession, only the final one should hit the backend? The state should be the same as MobX is maintaining it.


Prod URL and SSL certificate

Describe the task
A production URL for the CIF application, e.g. cas-cif.nrs.gov.bc.ca, should be ordered via the NRM service desk.
If required, an Entrust certificate will need to be ordered (first, an attempt to issue the certificate needs to be made, leading to a failed deployment, payment must be done via the NRM service desk, and once the request is approved, the application can finally be deployed).
Alternatively, a Let's Encrypt SSL cert could be used, offering a free and fully-automated process
Checklist:

[x] Determine if we can use Let's Encrypt
[x] Seek budget approval for Entrust if we can't use Let's Encrypt
[x] Budget approved
[x] Get input from program area on what url type should be secured (vanity vs nrs)
[x] Submit required Domain name request & web property application form in order to secure cif.gov.bc.ca

Emailed program area (March 1, 2022) to get input on whether we should secure a vanity or nrs url for the CIF app.

Program area would like to proceed with the vanity url.

secure the vanity URL
configure the DNS
deploy to prod
order the certificate

Steps to secure vanity url: https://apps.nrs.gov.bc.ca/int/confluence/display/DEVGUILD/Vanity+URL's

Submitted domain name request form and web property application form to CITZ GDX.

The DNS was setup, so cif.gov.bc.ca, dev.cif.gov.bc.ca and test.cif.gov.bc.ca all point to the silver cluster. Developers now need to configure the app (helm chart) to use the hosts above. Given that SSL certificates are only issued on installation of the app with our current script, the easiest path in dev and test is probably to uninstall and then re-install the application with the new hosts

@LindsayMacfarlane Entrust certificate request was issued. An email was sent to the GGIRCS inbox on Thursday April 14. The tracking ID in that email will have to be submitted as part ot the iStore request to approve the certificate.

Request for SSL certificate has been submitted to the NRM Business Service Desk.

Have followed up to see status of request on:

April 29
May 3

Certificate order appears to be complete. See GGIRCS inbox for email.

@LindsayMacfarlane certificate was issued, backed up to 1Password, and the prod cif application was turned down to 0 pods, as it's not ready to be online yet


[Bug] Logout on "python3 app.py"

Hi,
When I run python3 app.py, it justs logs out after like 10 seconds
I'm running Pop!_OS 22.04 LTS
I captured the output :
Welcome to Spotlightify

Starting auth process...

[CACHE] [MANAGER] Initialised

[CACHE] [SONG] Initialised
[CACHE] [IMAGE] Initialised
[CACHE] [PLAYLIST] Initialised
[CACHE] [LIKED] Initialised
[CACHE] [ALBUM] Initialised
[CACHE] [ARTIST] Initialised
[CACHE] [SONG] Starting
[CACHE] [IMAGE] Starting
[CACHE] [PLAYLIST] Starting
[CACHE] [LIKED] Starting
[CACHE] [ALBUM] Starting
[CACHE] [PLAYLIST] Skipping caching, last updated jeu 02 jun 2022 18:38:31
[CACHE] [ARTIST] Starting
[CACHE] [ARTIST] Skipping caching, last updated jeu 02 jun 2022 18:38:32
[Error] could not select default device.
[CACHE HOLDER] Reloading JSON File
arguments did not match any overloaded call:
  QPoint(): too many arguments
  QPoint(int, int): argument 2 has unexpected type 'float'
  QPoint(QPoint): argument 1 has unexpected type 'int'
Starting auth process...

threads can only be started once
Starting auth process...

And then there's like a hundred
threads can only be started once
Starting auth process...

And I'm logged out of my session
I checked config.json, the fields are exactly what they need to be
Did I do something wrong? I don't really understand those errors

same problem here

Hey there,
I'm closing this issue because Spotlightify has been completely rewritten with a new Golang backend and React Typescript frontend. The latest version is much improved—give it a try and let me know what you think!
Cheers,
Peter


Add CHANGELOG

Updating from 5.1.x to 5.2.x causes my app to break in a few places. I'm fine with with fixing those, but having a CHANGELOG where I can see the changes makes it easier to know what I'll be hunting down.

@cesarandreu could you comment on what breaks you experienced?

Indeed I think  a changelog might be helpful.
All the rule changes are captured on eslint-config-standard

https://github.com/feross/eslint-config-standard/commits/master anything beyond 4.1.0 is probably what affected you.

I got the following new errors / warnings:

Block must not be padded by blank lines.
'?' should be placed at the beginning of the line.
':' should be placed at the beginning of the line.
Requires a space before "}".
Infix operators must be spaced.

Perhaps https://github.com/rvagg/changelog-maker might be useful for generating changelogs.

Block must not be padded by blank lines.

This was a long standing rule that was broken in eslint for a small while.  It was technically always a standard idiom.

Requires a space before "}".

There is a requirement for even/consistent spacing,  is this what you mean?

Ah cool, I did not know this existed, @yoshuawuyts :) It looks like its a piece of the puzzle for adding to the node.js changelog (their release instructions have the full details)
I'm currently hand-crafting a changelog with the last couple releases just to get something out here (using vmd to preview as I craft them :wink:). I'll post them in the node.js format above and we can gather feedback from folks.

Actually, I am leaning towards something simpler like the format proposed at http://keepachangelog.com/

Added a changelog:
https://github.com/feross/standard/blob/master/CHANGELOG.md
It follows http://keepachangelog.com/ conventions. I decided to not list the raw commits and instead tried to make the change log more human readable.
A lot of standard is now broken out into dependencies, so a link to its CHANGELOG/history is provided along with a list of changes that affected standard.
There is also some redundancy; if a dependent package had "Fixed" and "Added" things, its shown twice with a specific list of things for each section. I feel this adds some clarity, especially if there are multiple changes in a dependency.
If you have time, please take a look and give some feedback! Thanks!
(would love feedback from @ngoldman and @bcomnes too!)

Awesome CHANGELOG work @Flet  Looks great.
If a change is only supposed to be a minor, but in reality turns out to be major, it might be grounds for a re-versioned release with a deprecate on the incorrect semver bump (at least in theory, I've never actually gone through this process).

@Flet looks excellent!

I'm usually not a fan of changelog files because they take extra effort to keep up-to-date and this is an all-volunteer effort already, but I do understand the need for end users to quickly find out what changed.
Let's try it out and see how it works.

If a change is only supposed to be a minor, but in reality turns out to be major, it might be grounds for a re-versioned release

The ternary (? :) and the 'blocks must not be padded' rules have been part of standard even though they haven't been enforced consistently due to eslint bugs and ditching jscs at one point. But now that we can enforce them again, they're basically bugfixes, though it's true that users experience them as breaking changes.
Sorry about that folks -- hopefully won't happen again. I don't think we have any more of these.

Thanks @feross. I'll happily try to keep it up to date if folks forget :innocent:  The standard releases seem to be kept pretty small these days anyways, so that should make it easier to maintain.

An :angel: indeed -- thanks!

That was really quick, thanks!

@flet: lgtm :+1:
I'm a fan of the change log. I like to add a link to the diff between each release (e.g. 5.2.1 header links to https://github.com/feross/standard/compare/v5.2.0...v5.2.1) but that's a lot of effort for a small reward. @bcomnes and I worked on some stuff related to change logs (https://github.com/bcomnes/changelog-init & https://github.com/ngoldman/gh-release) and I've wanted to create something to automate adding those diff links. Maybe I'll take a crack at that this week.
@feross: re:

I'm usually not a fan of changelog files because they take extra effort to keep up-to-date and this is an all-volunteer effort already, but I do understand the need for end users to quickly find out what changed.

I understand it can be a bit of a chore, but it definitely helps keep contributors and collaborators more aware of user-facing changes and makes it easier to make a call about patch vs. minor vs. major releases. If it's just a small part of each contribution it starts to take care of itself. :)

@bcomnes I've gone through the process you described above, it's not that bad. Just npm publish the major version and npm deprecate<EMAIL_ADDRESS>"warning message" and that message will show up on npm install if anyone's still installing that version. Definitely a bad scenario for users though, which I think keeping a change log helps avoid.

Nice, love the diff link idea. I went ahead and added them (plus diff links going back to 4.0.0).

The diff links are handy!


Ce 1053

see PR#13 in cumulus-license-puppet for module fix. see if that works in your setup instead of this workaround.

Merged PR #8 with a different fix.


Empty Bag

If I download all content as a zipped file for the following resource I get an empty zip file.
https://www.hydroshare.org/resource/27a7553d7b9f4d08a8d055c258cf6efa/

@dtarb It is fixed. You should be able to download the bag now. What I did is to set bag_modified to true so that the bag can be regenerated since the existing bag zip file is empty with 0 byte in size. I did not know how this happened in the first place, but this specific issue is resolved. I suggest to close this issue for now if bag download for this resource works now.

OK. Closing for now.  We need to be vigilant as to why this happened and whether there is a systemic problem to address.


ci(eslint): migrate to eslint + prettier

Coverage remained the same at 88.301% when pulling 4928953e77a710ced7db1f638bcdabaf09abc42c on eslint-typescript into c8cd3ced13314a401f1e41a01687cce5c280823f on master.


drivers: ethernet: eth_xmc4xxx: fix get_phy

This fixes the eth_xmc4xxx_get_phy function.
Fixes: #76402

ping @ifyall
pls approve, your are the Assignee.


Fix missing kubeconfig issue in daily release

in the daily release KUBECONFIG is not and therefore the TestTempWorkspace was failing. We should not rely on the env only. kube.ResolveConfig("") does the right thing. let's us is.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sebastienvas
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS [sebastienvas]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

cc @rkpagadala @hklai

@sebastienvas: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
125ce0bc59ed98d6f8cf6c59760bdaa99da3a8ab
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: hklai, sebastienvas
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/OWNERS [hklai,sebastienvas]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sebastienvas: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
125ce0bc59ed98d6f8cf6c59760bdaa99da3a8ab
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-unit-tests
doh these mixer tests are annoying

@sebastienvas thank you for fixing this!  Not sure why we need it here, but appreciate your help in getting the test pass.


Add Task dataset to dataset endpoint

Description of change

This PR adds a tasks-dataset to the dataset endpoint, in preparation for a new pipeline to move it over to Data Workspace.
More information can be found in ticket CLS2-772.
Checklist

[x] Has this branch been rebased on top of the current main branch?

Explanation
The branch should not be stale or have conflicts at the time reviews are requested.

[ ] Is the CircleCI build passing?

General points

Other things to check

Make sure fixtures/test_data.yaml is maintained when updating models
Consider the admin site when making changes to models
Use select-/prefetch-related field lists in views and search apps, and update them when fields are added
Make sure the README is updated e.g. when adding new environment variables

See docs/CONTRIBUTING.md for more guidelines.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 96.54%. Comparing base (6bd5529) to head (89726ae).
 Consider uploading reports for the commit d957a92 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5369      +/-   ##
==========================================
+ Coverage   96.53%   96.54%   +0.01%
==========================================
  Files         926      927       +1
  Lines       21998    22009      +11
  Branches     1964     1964
==========================================
+ Hits        21235    21248      +13
+ Misses        620      619       -1
+ Partials      143      142       -1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


WIP: Added Lupio-RTC device and new LupV platform

Change-Id: Ibd7cb3a36d9e1a897a20c462a90e04a0d9066a1a

For the commit messages, I would use "dev,riscv: ..." for the lupv platform and "dev:" for the RTC.

Looks all good to me! I'll let you all handle the switch from develop to lupio for the branch to be merged in.
Happy Friday :smiley:


Integrate the FERC EQR Data

The FERC Electric Quarterly Report (EQR) reports the details of more than a billion electricity transactions that have taken place since 2013, at up to 5 minute resolution, including many detailed attributes of each transaction. It's an underappreciated goldmine of electricity market data.
More details from FERC here.

Pre-2013 too!

Well... I was kind of hoping not to have to look at any more FERC DBF
files. :)
On Tue, Jun 4, 2019 at 7:44 PM Karl Dunkle Werner<EMAIL_ADDRESS>wrote:

Pre-2013 https://www.ferc.gov/docs-filing/eqr/q2-2013/data/database.asp
too!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/catalyst-cooperative/pudl/issues/296?email_source=notifications&email_token=AAERSN6WXHL6EPCRVD57AODPY4LAZA5CNFSM4HSSBP6KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW6LJ2Y#issuecomment-498906347,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAERSN5BMYUDU6QPUA2FYOTPY4LAZANCNFSM4HSSBP6A
.

--
Zane A. Selvans, PhD
Chief Data Wrangler
Catalyst Cooperative
https://catalyst.coop
<EMAIL_ADDRESS>Signal/WhatsApp/Telegram/SMS: +1<PHONE_NUMBER>
Twitter: @ZaneSelvans https://twitter.com/ZaneSelvans
PGP https://www.gnupg.org/: 0x64F7B56F3A127B04

This is bigger than an "Epic" task really.


Support required tags for questionnaires

When starting a record, show the required tags as form fields to be completed before starting.

 Required Tags


Prompt slow after ls

I have change my bash profile to bash-it. Question is every time when I type ls, I must wait 1s to 2s.
Looks like the video .
In the video, ls is waiting something like node /usr/local/bin/which ng completed.
Have no idea about this.

A couple of questions:

Which Bash-it theme are you using?
Which Bash-it plugins have you enabled?
Does this happen in all directories, or just ones where you are in a Git repo or similar?

My Bash-it theme is Envy.
My Bash-it plugins:

○ → bash-it show plugins
Plugin              Enabled?  Description
alias-completion      [x]     Automatic completion of aliases
autojump              [x]     Autojump configuration, see https://github.com/wting/autojump for more details
aws                   [ ]     AWS helper functions
base                  [x]     miscellaneous tools
battery               [ ]     display info about your battery charge level
boot2docker           [ ]     Helpers to get Docker setup correctly for boot2docker
browser               [ ]     render commandline output in your browser
chruby-auto           [ ]     load chruby + auto-switching (from /usr/local/share/chruby)
chruby                [ ]     load chruby                  (from /usr/local/share/chruby)
direnv                [ ]     load direnv, if you are using it: https://direnv.net/
dirs                  [ ]     directory stack navigation
docker-compose        [x]     Helper functions for using docker-compose
docker-machine        [x]     Helpers to get Docker setup correctly for docker-machine
docker                [x]     Helpers to more easily work with Docker
edit-mode-emacs       [ ]     Enable emacs editing mode
edit-mode-vi          [x]     Enable vi editing mode
explain               [ ]     mankier.com explain function to explain other commands
extract               [ ]     one command to extract them all...
fasd                  [ ]     initialize fasd (see https://github.com/clvv/fasd)
fzf                   [ ]     load fzf, if you are using it
gh                    [ ]     load gh, if you are using it (DEPRECATED, use hub instead)
gif                   [ ]     video to gif, gif to WebM helper functions
git-subrepo           [x]     load git-subrepo if you are using it, and initialize completions
git                   [x]     git helper functions
go                    [x]     go environment variables & path configuration
gradle                [ ]     Add a gw command to use gradle wrapper if present, else use system gradle
hg                    [ ]     hg helper functions
history               [x]     history manipulation
hub                   [ ]     load hub, if you are using it
java                  [ ]     Java and JAR helper functions
javascript            [x]     download jquery files into current project
jekyll                [ ]     manage your jekyll site
jenv                  [ ]     load jenv, if you are using it
jgitflow              [x]     Maven jgitflow build helpers
jump                  [x]     initialize jump (see https://github.com/gsamokovarov/jump)
latex                 [ ]     use mactex
less-pretty-cat       [x]     pygmentize instead of cat to terminal if possible
nginx                 [ ]     manage your nginx service
node                  [x]     Node.js helper functions
nodenv                [ ]     load nodenv, if you are using it
nvm                   [ ]     node version manager configuration
osx-timemachine       [x]     OS X Time Machine functions
osx                   [x]     osx-specific functions
percol                [ ]     Search&Select history and fasd with percol
pipsi                 [ ]     load pipsi, if you are using it
plenv                 [ ]     plenv plugin for Perl
postgres              [ ]     postgres helper functions
powerline             [x]     enables powerline daemon
projects              [ ]     add "export PROJECT_PATHS=~/projects:~/intertrode/projects" to navigate quickly to your project directories with `pj` and `pjo`
proxy                 [ ]     Proxy Tools
pyenv                 [ ]     load pyenv, if you are using it
python                [x]     alias "shttp" to SimpleHTTPServer
rails                 [ ]     Helper functions for Ruby on Rails
rbenv                 [ ]     load rbenv, if you are using it
ruby                  [x]     ruby and rubygems specific functions and settings
rvm                   [ ]     load rvm, if you are using it
sdkman                [ ]     Load Software Development Kit Manager
ssh                   [x]     ssh helper functions
sshagent              [ ]     sshagent helper functions
subversion            [ ]     svn helper functions
textmate              [ ]     set textmate as a default editor
tmux                  [x]     make sure that tmux is launched in 256 color mode
tmuxinator            [ ]     sources tmuxinator script if available
todo                  [x]     Todo.txt integration
virtualenv            [ ]     virtualenvwrapper and pyenv-virtualenvwrapper helper functions
visual-studio-code    [ ]     Defines the `code` executable for Visual Studio Code on OS X
xterm                 [ ]     automatically set your xterm title with host and location info
z                     [ ]     maintains a jump-list of the directories you actually use
                              z is DEPRECATED, use fasd instead
z_autoenv             [ ]     source into environment when cding to directories

to enable a plugin, do:
$ bash-it enable plugin  <plugin name> [plugin name]... -or- $ bash-it enable plugin all

to disable a plugin, do:
$ bash-it disable plugin <plugin name> [plugin name]... -or- $ bash-it disable plugin all

It happens in all directories, after all commands.

@edubxb I havn't enabled ng completion. 🌚


qiita_pet in setup.py package_data is wrong

While doing a clean install via: pip install -e . @josenavas found that the package_data description of qiita_pet is wrong (it's not copying all the files), see here. I think we can implement something like this: http://stackoverflow.com/a/22260596

@josenavas, could you confirm if this is still an issue?

I've done a couple of clean installs and I have not hit this issue again.


Adds more validation output with invalid arrays.

Closes #5834
Also includes:

For isValidGuidArray, returns all faulty values and modified the error output.
For isValidUserPrincipalNameArray, returns all faulty values and modified the error output.
For isValidUserPrincipalNameArray, applied this validation logic to planner plan add, planner plan set, planner task add, planner task set, and teams meeting add.

Merged manually, thank you! 🎉


More specific null/empty str handling in IndexMerger

Addresses a concern raised in PR #2006 about dimension cardinalities.
Changes the conditions for when an entry for the null/empty str is added to the value dictionary during index merging.
Old behavior:

If dim has values in at least one index, ensure the null/empty str is in the dictionary and convert missing columns to empty strings for merging

New behavior:

If dim is entirely missing from an index, but has values in another index, ensure the null/empty str is in the dictionary and convert missing columns to empty strings for merging

@navis does this address your concerns from #2006 ?

:+1:

What will happen if a dim has values in all index, but all index do not have null/empty value.

@binlijin The null/empty str will not be added to the dictionary in that case (i.e., dimAbsentFromIndex will be false)
Off the top of my head, testNonLexicographicDimOrderMerge() in IndexMergerTest is an example where the dims don't have null/empty values

@jon-wei  good

@jon-wei It seems like the ideal condition would be:

If dim is entirely missing from one index, but has mixed nulls and values in another index, ensure the null/empty str is in the dictionary and convert missing columns to empty strings for merging

Checking for mixed nulls/values shouldn't add additional expense as isNullColumn already iterates over all the dim values. So I think it's feasible to do this check the ideal way.

@gianm It's still necessary to add the null/empty str to the dictionary for the case where a dim is missing from one index, and has no null values in another; when they merge.
In the legacy merger this was handled by bumping the dictionary in IndexIO:
          if (onlyOneValue) {
            log.info("Dimension[%s] is single value, converting...", dimension);
            final boolean bumpedDictionary;
            if (nullsSet != null) {
              log.info("Dimension[%s] has null rows.", dimension);
              final ImmutableBitmap theNullSet = bitmapFactory.makeImmutableBitmap(nullsSet);

              if (dictionary.get(0) != null) {
                log.info("Dimension[%s] has no null value in the dictionary, expanding...", dimension);
                bumpedDictionary = true;
                final List<String> nullList = Lists.newArrayList();
                nullList.add(null);

                dictionary = GenericIndexed.fromIterable(
                    Iterables.concat(nullList, dictionary),
                    GenericIndexed.STRING_STRATEGY
                );

:+1:


Display profile picture

:tada: This issue has been resolved in version 7.1.0 :tada:
The release is available on:

GitHub release
v7.1.0

Your semantic-release bot :package::rocket:


Apply java-library plugin in samples, javadoc, tests

Gradle's general advice is to use the java-library plugin over the java plugin in most cases. The java plugin will eventually go away. As such, we should set an example by updating our migrating our own tests to follow our recommendations. This step is necessary in order to deprecate the java plugin.
We also update all (or most) public-facing snippets and javadoc to use non-java plugins. Most have been migrated to java-library but some have been migrated to application.
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

Not nearly ready


BUG: Fix wrapping for TransformBaseTemplate class

The class was re-named from TransformBase to TransformBaseTemplate, but
the wrapping name was not updated.
Also wrap double transform's, the most common.

How did you notice this error?

I made an attempt to write transforms in Python and ran into this issue. I believe this does not fully address that use case, yet, but it is a step in the right direction.

Do you think this PR was merged to late to fix these errors for the last nightly?

	/home/lorensen/ProjectsGIT/Dashboards/ITK-SkullStrip/Modules/Filtering/FastMarching
Exit Condition	1
Standard Error
../../../lib/libITKSpatialObjects-5.1.so.1: error: undefined reference to 'vtable for itk::TransformBaseTemplate<double>'
../../../lib/libITKSpatialObjects-5.1.so.1: error: undefined reference to 'typeinfo for itk::TransformBaseTemplate<double>'
collect2: error: ld returned 1 exit status

This does not look related.


AWS blog guide ain't updated accordingly to current situation in a repository

source: https://github.com/nirmata/kyverno-notation-aws/issues/124
AWS blog guide ain't updated accordingly to current situation in a repository, as processing changed
              - key: images
                value: "{{ request.object.spec.[ephemeralContainers, initContainers, containers][].image }}"

Doesn't work anymore, which may result in confusion further for whoever tries to implement it and provide bad user experience
More details: https://aws.amazon.com/blogs/containers/announcing-container-image-signing-with-aws-signer-and-amazon-eks/
From my side, it was assumed initially will work, but created a lot of confusion & time in order to implement working PoC

Thanks @vponoikoait for opening these issues, I will fix them as soon as I can

@vponoikoait, That blog was published when this service was in early stages and we added a lot of features after that. Unfortunately, some of those required us to break backwards compatibility with 0.x alpha versions.
You can follow this blog: https://nirmata.com/2023/11/20/verifying-images-and-attestations-using-aws-signer-notation-and-kyverno/
We will see if we can get that AWS blog updated

@vishal-chdhry would you be kind to include there that current auth requires for kyverno to stay in kyverno namespace and have specific SA name? So nobody would potentially go confused.
Referencing: https://github.com/nirmata/kyverno-notation-verifier/issues/27

It would be also a valid mentioning regarding that it's available since Kyverno version 1.10+, when service calls started to be a thing


Accept copy from a part of the log and paste as text in the search

Prerequisites
affected version: 2.26.3
make sure to check this before filing an issue:

[ x] I checked the documentation and found no answer
[ x] I checked to make sure that this issue has not already been filed

Describe what you would like to have
When I'm looking into the logs I see a text that I want to search, there is already an option to copy:

But there is no option to paste in the Search request.
Potential Workaround
So far I have to type the text.
More Information
Using Mac

Hello @cdamken , sorry for being late with the feedback. Am I right, you would like to see the option Paste in the context menu of search-input?

@cdamken  Does it mean that ctrl + V (cmd + V) doesn't work for you as a way to paste copied text?

The issue seems to be that the RMB doesn't create a context menu in the search input leave alone show the Paste option. This seems to be confusing as we show the Copy option inside the main view.
I think it'd be good if we not only provide the Paste option in the search input but anywhere we can type (e.g. editing filter name, filtering notifications, etc.)

related to #1575


Add function which allows for sound to play only if current autotype isn't certain character(s)

i've found a temporary solution to this using .typewriter_function() (i also opened a new issue which if it gets fixed, this will change slightly since it's accounting for the issue).
maybe something similar to this can be wrote into scribble itself without the need of a parser rewrite? (i have no clue of course)
typewriterIsFormatter = false;
typewriterOffset = 0;
typewriterBlacklist = [ " ", ".", ",", "!", "?" ];

play_char_sounds = function(_scribble, _textPosition)
{
	var _text = _scribble.text;
	var _char = string_char_at(_text, _textPosition);
	switch (_char)
	{
		case "[": typewriterIsFormatter = true; break;
		case "]": typewriterIsFormatter = false; break;
	}

	typewriterOffset += typewriterIsFormatter;

	var _realChar = string_char_at(_text, (_textPosition + typewriterOffset));
	var _totalBlacklist = array_length(typewriterBlacklist);
	for (var i = 0; i < _totalBlacklist; i++)
		if (_realChar == typewriterBlacklist[i]) return;

	// from here you can play your sounds
}

<scribble>.typewriter_function(play_char_sounds);

Implemented for v8


Task/78 add warning not to put git repos into onedrive folder

PR details
What is the purpose of this pull request? (put an "X" next to an item)
Please delete options that are not relevant.

[x] Other, please explain:

What changes did you make? (Give an overview)
there is no easy way to see if folder is being backed up,
i retrieve the absolute path to the git repo and then scan the path for words used in the cloud services('onedrive','google drive'...), in case a word is found a warning is presented to the user
Is there anything you'd like reviewers to focus on?
the way to retrieve the absolute path, is there a better way that dont need an extra dependency?
Checklist

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] ESLint & Prettier are not outputting errors or warnings

Tried path.resolve? That's part of nodejs and hence not an additional dependency
https://stackoverflow.com/a/42600768

Tried path.resolve? That's part of nodejs and hence not an additional dependency
https://stackoverflow.com/a/42600768
Also, I think there was __dirname?

Tried path.resolve? That's part of nodejs and hence not an additional dependency
i did use path.resolve, i expressed myself incorrectly, i didn't use any new dependency.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test

Docs Build status updates of commit a42ef9d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Archive_master_2019-09-26-02-53-21 has been deleted or has been force pushed: fatal: Couldn't find remote ref Archive_master_2019-09-26-02-53-21

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


query-string 4.3.0  is breaking protractor-http-mock

After release of query-string 4.3.0 we started seeing issues in our automated tests:
Error: objectAssign is not defined
anonymous/anonymous/queryStringParseFunction
matchQueryString
match
matchExpectation
httpMock
processQueue
scheduleProcessQueue
$RootScopeProvider/this.$get</Scope.prototype.$eval
$RootScopeProvider/this.$get</Scope.prototype.$digest
$RootScopeProvider/this.$get</Scope.prototype.$apply
done
completeRequest
requestLoaded
nrWrapper
angular.js:12808:18

Reverting back to query string 4.2.3 fixes the problem above.

Thanks for the heads up. I just published 0.9.2 which fixes the issue. I had to lock the down the version of query-string unfortunately since their new code is not compatible.


Drop TLS 1.0 and 1.1 support from ALB's

Trello: https://trello.com/c/C9hginSe/166-remove-tls-10-and-11-support-for-publishing-apps
This changes the default TLS security policy used in the application
load balancers to be a newer rule that drops support for TLS 1.0 and
1.1.
As no ALBs set a non-default listener_ssl_policy this rule will apply to
all of the public ALBs in GOV.UK infrastructure. So this will apply to
anything on publishing.service.gov.uk which is not behind the CDN (www,
assets and transition sites). I decided that it'd be better to apply
this broadly rather than individually as this change has been something
that has been put off for a while and it'd be good to set any exceptions
based on reasoning rather than caution.
This has been done because TLS versions below 1.2 are susceptible to
man-in-the-middle attacks and by continuing to allow support for the
older TLS versions we risk downgrade attacks 1.
The policy ELBSecurityPolicy-TLS-1-2-2017-01 only has support for TLS
1.2 (at the time of writing there isn't an ALB security policy that
supports TLS 1.3) and drops a number of weaker ciphers
(ECDHE-ECDSA-AES128-SHA, ECDHE-RSA-AES128-SHA, ECDHE-RSA-AES256-SHA,
ECDHE-ECDSA-AES256-SHA, AES128-SHA and AES256-SHA) 2
The impact of this change is expected to be minimal, every browser in
the service manual 3 supports TLS 1.2 4.
To analyse the impact I downloaded the 30 day log files for the following
public load balancers to compare:

load balancer
TLS 1.2
TLS 1.1
TLS 1
Percentage TLS 1.2

govuk-account-public-elb
6460
83
89
97.337%

govuk-backend-public-elb
10178549
71
2649
99.973%

govuk-bouncer-public-elb
0
0
0
N/A

govuk-ckan-public-elb
3408231
75
2428
99.927%

govuk-draft-cache-public-elb
984077
57
547
99.939%

govuk-email-alert-api-public-elb
33348728
55
58
100%

govuk-graphite-public-elb
14504
0
0
100%

govuk-jumpbox-public-elb
0
0
0
N/A

govuk-licensify-frontend-public-elb
33838
58
70
99.622%

govuk-monitoring-public-elb
6751
0
0
100%

govuk-prometheus-public-elb
144
0
0
100%

govuk-sidekiq-monitoring-public-elb
290
0
0
100%

govuk-whitehall-backend-public-elb
1442461
79
117
99.986%

licensify-backend-public-elb
792912
71
322
99.950%

I also analysed govuk-public-elb but, since that is massive, I only did
it for the last 5 days:

load balancer
TLS 1.2
TLS 1.1
TLS 1
Percentage TLS 1.2

govuk-cache-public-elb
106137686
0
0
100%

The results show almost all connection are made using TLS 1.2, from my
brief investigations it seems that most of the TLS 1 and TLS 1.1 are
from probing.
E.g:
https 2022-01-14T01:12:05.115375Z app/govuk-email-alert-api-public/05a186ca4b2bde26 <redacted>:51796 - -1 -1 -1 400 - 0 0 "- https://govuk-email-alert-api-public-403570493.eu-west-1.elb.amazonaws.com:443- -" "-" ECDHE-RSA-AES128-SHA TLSv1.1 - "-" "-" "arn:aws:acm:eu-west-1:172025368201:certificate/063a18cc-d4bf-4eaf-b1c1-41e86bbdd69d" - 2022-01-14T01:12:05.115000Z "-" "-" "-" "-" "-" "-" "-"
https 2022-01-14T20:12:05.148956Z app/govuk-backend-public/169e0e0d325925ed <redacted>:41680 <IP_ADDRESS>:80 0.001 0.001 0.000 500 500 194 198 "GET https://<IP_ADDRESS>:443/ HTTP/1.1" "Mozilla/5.0 (Windows NT 6.1; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/81.0.4044.138 Safari/537.36" AES128-SHA TLSv1 arn:aws:elasticloadbalancing:eu-west-1:172025368201:targetgroup/govuk-backend-public-HTTP-80/e592c497dedb83be "Root=1-61e1d915-5267f5b907490f4f6069ad77" "<IP_ADDRESS>" "arn:aws:acm:eu-west-1:172025368201:certificate/063a18cc-d4bf-4eaf-b1c1-41e86bbdd69d" 0 2022-01-14T20:12:05.146000Z "waf,forward" "-" "-" "<IP_ADDRESS>:80" "500" "-" "-"

ELB Cloud policy details here: https://docs.aws.amazon.com/elasticloadbalancing/latest/classic/elb-security-policy-table.html

Thanks for the reviews/input/suggestions - super helpful.
I've dug out some further analysis into the TLS 1 and TLS 1.1 connections for licensing, backend and ckan and put that into a spreadsheet. I've shared it with Replatforming team, let me know if you have access issues and I'll add you: https://docs.google.com/spreadsheets/d/1trbMKNXOxkbey2g_a_qz1iB4hPW5OWyzI9SrA-PXY-w/edit
There is evidence of potential real users amongst the noise of scanning/opportunistic stuff. The only potentially user changing state journey I could find was a confusing case in Licensing where they seem to be using TLS 1 despite the user indicating IE 11 on Windows 10 🤔 . For non state changing journeys, there's ~1000 requests from a user agent claiming to be IE6 - which has got to be quite the grim experience.
I like the idea of status page to share information. I'm talking with some PMs about whether to warn people. I'm a bit worried a warning might over-inflate the impact it could have on people and risk delaying things, but I'll see what PM's involved think.
As a rough plan I'll see if I can get things co-ordinated so that I can plan to apply this to production next Wednesday (it's already applied to Integration).
For sharing the information with third parties of what we're doing I wanted a good link to explain why it's been done. This advice from NCSC seems a good resource to me: https://www.ncsc.gov.uk/guidance/using-tls-to-protect-data but let me know if you know of better.


update README

This PR updates the README to include more information regarding Tack, including potential applications for it's usage.

I think this was meant for another project?
Tack is C# and mostly an IDE tool. The applications you listed look ML-heavy, so I believe this PR is meant for another repo :)


Peerinvalid

Hello,
When I'm trying to install ng2-dnd, I get the following errors:
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "ng2-dnd" "--save"
npm ERR! node v4.4.1
npm ERR! npm  v2.14.20
npm ERR! code EPEERINVALID

npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>
Can someone help me solve this ? :)

I'm busy fixing your problem now.
On 30 March 2016 at 17:24, pwasilewski<EMAIL_ADDRESS>wrote:

Hello,
When I'm trying to install ng2-dnd, I get the following errors:
npm ERR! Darwin 14.5.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "install" "ng2-dnd" "--save"
npm ERR! node v4.4.1
npm ERR! npm  v2.14.20
npm ERR! code EPEERINVALID
npm ERR! peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!
npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>npm ERR! peerinvalid Peer<EMAIL_ADDRESS>wants<EMAIL_ADDRESS>Can someone help me solve this ? :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/akserg/ng2-dnd/issues/7

I have updated library to Angular 2.0.0-beta.12.
Can you check and confirm that the issue was fixed.

I have updated the library to Angular 2.0.0-beta.13.

I'm closing this issue. Please reopen if you need it.


jarek/8015: fix TimePlot with ConstantLine and pandas timestamp

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


GLPI and Keycloak - "Found an Attribute element with duplicate Name"

Hello,
I've managed (I think) to set up phpsaml with Keycloak. I had some problem with the different terms but the main authentication seems to be working - except when the user gets redirected back to GLPI, they don't get logged in.
Instead there is a message - "Found an Attribute element with duplicate Name" and a lonk to log in again that brings them back to the login page.
I have debug enabled in the plugin but the only thing I see in the glpi php-errors.log is the exact same message, so I'm having trouble troubleshooting the issue.

Can you provide a screenshot?  You may need to step through the plugin_post_init_phpsaml function in setup.php to see where this message is generated.


Port rename

Description
Rename command now works. I also added an option to change a specific column's name, e.g. [[ a b]; [1 2]] | rename -c [a new_name]. The spans also work nicely, pointing to the specified column's area if the specified column does not exist.
If more columns are given than exist in the record, it skips the extra columns.
Tests
Make sure you've run and fixed any issues with these commands:

[x] cargo fmt --all -- --check to check standard code formatting (cargo fmt --all applies these changes)
[x] cargo clippy --all --all-features -- -D warnings -D clippy::unwrap_used -A clippy::needless_collect to check that you're using the standard code style
[x] cargo build; cargo test --all --all-features to check that all the tests pass

looks good


python_raft: framework simliication for use with L3 frameworks

Goal:
Minor changes to support integration with the level 3 testing suites to support common unifications

resolved #89 is merged


feat(avm): improve interpreter errors and tests

Interpreter only handles AvmInterpreterError type.
AvmMessageCallResult now has a revertReason: Error. It's useful for testing but might be useful in general. Happy to change otherwise.
Added /*param=*/ on calls where params are not clear from context or variable name.

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (8): 38.3 (+80%)
note_trial_decrypting_time_in_ms (32): 95.6 (+93%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 14d39edb and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,412
179,524
716,068

l1_rollup_calldata_gas
221,680
866,944
3,448,444

l1_rollup_execution_gas
314,099
984,616
3,668,033

l2_block_processing_time_in_ms
1,449 (-9%)
5,445 (+3%)
21,513 (+3%)

note_successful_decrypting_time_in_ms
324 (+6%)
968 (+5%)
3,693 (+4%)

note_trial_decrypting_time_in_ms
:warning: 38.3 (+80%)
:warning: 95.6 (+93%)
148

l2_block_building_time_in_ms
16,404 (+2%)
65,064 (+2%)
260,648 (+2%)

l2_block_rollup_simulation_time_in_ms
11,925 (+2%)
47,360 (+2%)
190,003 (+2%)

l2_block_public_tx_process_time_in_ms
4,442 (+2%)
17,609 (+2%)
70,338 (+2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
15,818 (-4%)
30,276 (-4%)

note_history_successful_decrypting_time_in_ms
2,151 (-5%)
4,220 (-9%)

note_history_trial_decrypting_time_in_ms
107 (-63%)
165 (-32%)

node_database_size_in_bytes
3,514,404
3,543,978

pxe_database_size_in_bytes
29,923
59,478

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
214
44,681
25,561

private-kernel-ordering
166
43,753
14,809

base-rollup
1,200
128,338
881

root-rollup
79.6
4,088
677

private-kernel-inner
293
71,208
25,561

public-kernel-private-input
207
31,863
25,561

public-kernel-non-first-iteration
205
31,905
25,561

merge-rollup
7.74 (+1%)
2,608
881

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
15,455
32,219

Transaction processing duration by data writes.

Metric
0 new commitments
1 new commitments

tx_pxe_processing_time_ms
380 (+1%)
1,092

Metric
1 public data writes

tx_sequencer_processing_time_ms
540


Join SHA dash 256 to one word

@vanillajonathan : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.

@vanillajonathan : Thanks for your contribution! The author, @spelluru, has been notified to review your proposed change.

Thank you @vanillajonathan
#sign-off


Not Working with Format from Article

I created the CSV using the 6 columns needed.  I'm getting an error when trying to upload the file stating the header id could not be found.  I was able to get this to work with csv using Source Path,Destination Path as the headers and just using the destination URL with folders ndded.

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Fix shadow

Some variables shadow global, which is difficult to handle when porting to other platforms with shadow warnings turned on

Hold on, are the global variables in these modules themselves, rather than in your application? In that case probably the global variables should be renamed, not the local ones.

Yeah, please ignore my first comment. I checked the source of these files, and they've got globals variables in them that could be named better. Those should be renamed rather than the local ones, and please also (for this PR as well as future ones) provide bit more verbose commit messages explaining the problem your fixing, since now I wrongly assumed that the globals were coming from the other platform that you're porting to, rather than these modules themselves.

Yeah, please ignore my first comment. I checked the source of these files, and they've got globals variables in them that could be named better. Those should be renamed rather than the local ones, and please also (for this PR as well as future ones) provide bit more verbose commit messages explaining the problem your fixing, since now I wrongly assumed that the globals were coming from the other platform that you're porting to, rather than these modules themselves.

Thank you for your suggestion.
This is a warning inside zephyr. zephyr disable the shadow flag, but on our platform, the shadow flag is forcibly enable, so a warning is raised.
I will provide some details for this commit.

The global variable should have a compound naming rule. It is not good to define only one uuid. According to the definition in Bluetooth or Bluetooth mesh, the prefix bt_  bt_mesh_

The global variable should have a compound naming rule. It is not good to define only one uuid. According to the definition in Bluetooth or Bluetooth mesh, the prefix bt_ bt_mesh_

Just to be clear, I think the variables in question don't have global scope (which would require using the officially allocated namespaces) but they're global (static) within these modules. Therefore, it should be sufficient to just put more context into their names.
In the case of the mesh_provisioner sample app, it doesn't look like net_idx is ever modified beyond its initial value (0), so it could just as well be a const or a preprocessor define. Maybe a prov_ prefix would be suitable for these variables?
As for uuid in the GATT shell, I think the Bluetooth stack requires the variable's lifetime to be long enough to last the duration of asynchronous operations, but I don't think it needs to be global for the entire file, rather each command handler could have its own static variable declared within the scope of each function (unless I've misunderstood and the discovery and read handlers need to manage the exact same UUID variable instance).

The global variable should have a compound naming rule. It is not good to define only one uuid. According to the definition in Bluetooth or Bluetooth mesh, the prefix bt_ bt_mesh_

Just to be clear, I think the variables in question don't have global scope (which would require using the officially allocated namespaces) but they're global (static) within these modules. Therefore, it should be sufficient to just put more context into their names.
In the case of the mesh_provisioner sample app, it doesn't look like net_idx is ever modified beyond its initial value (0), so it could just as well be a const or a preprocessor define. Maybe a prov_ prefix would be suitable for these variables?
As for uuid in the GATT shell, I think the Bluetooth stack requires the variable's lifetime to be long enough to last the duration of asynchronous operations, but I don't think it needs to be global for the entire file, rather each command handler could have its own static variable declared within the scope of each function (unless I've misunderstood and the discovery and read handlers need to manage the exact same UUID variable instance).

Got it, I'll submit it later.

The global variable should have a compound naming rule. It is not good to define only one uuid. According to the definition in Bluetooth or Bluetooth mesh, the prefix bt_ bt_mesh_

Just to be clear, I think the variables in question don't have global scope (which would require using the officially allocated namespaces) but they're global (static) within these modules. Therefore, it should be sufficient to just put more context into their names.
In the case of the mesh_provisioner sample app, it doesn't look like net_idx is ever modified beyond its initial value (0), so it could just as well be a const or a preprocessor define. Maybe a prov_ prefix would be suitable for these variables?
As for uuid in the GATT shell, I think the Bluetooth stack requires the variable's lifetime to be long enough to last the duration of asynchronous operations, but I don't think it needs to be global for the entire file, rather each command handler could have its own static variable declared within the scope of each function (unless I've misunderstood and the discovery and read handlers need to manage the exact same UUID variable instance).

Got it, I'll submit it later.

The net_idx is global (static) within the modules, Could we only change the name to ensure that there is no conflict? Because other variables in this module do not have a uniform name, such as prov_.
The global uuid shall be in function scope with static.

Any plans to merge this PR?


Update appendix link in spring-cloud-openfeign doc

Current appendix link redirect to a 404 page

@terceser Thank you for signing the Contributor License Agreement!

Hello, @terceser . Thanks for submitting the PR. Could you please link to where you are getting a 404? I was not able to reproduce this issue.

Hello @OlgaMaciaszek, here is the link on the main branch https://github.com/spring-cloud/spring-cloud-openfeign/blob/main/docs/src/main/asciidoc/spring-cloud-openfeign.adoc
(screenshot bellow)

the Appendix page redirect to a 404 page
(screenshot bellow)

Ah, I see. That is because it's accessed from the README in GH. The actual reference site is here. When you access it from there, you'll get HTML.

ah I see ^^ please disregard my PR


Backport UCSCParser, UCSC_human_parser and UCSC_mouse_parser from ensembl-xref

Description
Port changes made to UCSCParser (as well as the two thin wrappers around it used for human and mouse) in ensembl-xref, including unit tests, back to ensembl:feature/xref_sprint.
Use case
See ENSCORESW-3231.
Benefits
Salvage more work from the 2018 xref sprint. Actually get UCSC xrefs back (their file format has changed) once ensembl:feature/xref_sprint has been merged into master.
Possible Drawbacks
Increased test-suite run time (note: Travis does not automatically execute tests from misc-scripts/xref_mapping yet).
Testing
Have you added/modified unit tests to test the changes?
Yes.
If so, do the tests pass/fail?
They pass.
Have you run the entire test suite and no regression was detected?
Yes (including tests from misc-scripts/xref_mapping), no regression detected.

Output comparison with release/98: the old UCSCParser erroneously uses Ensembl stable IDs as accessions (this is due to the change of format of UCSC files we use to generate their xrefs), the new version correctly handles the new format. Otherwise, no differences.

Coverage remained the same at 81.539% when pulling<PHONE_NUMBER>32e231b19f9acab78f5217e7957a95 on backport/ENSCORESW-3231 into 87fcf0fa496a8f25939bb44a9564197d9eace7dd on feature/xref_sprint.


sing-box core

plz update the sing-box core

done

plz update the sing-box core

nothing special in new beta core...wait for new stable


SAM with DeploymentPreference configuration does not create CodeDeploy deployment

I have a simple SAM with a basic DeploymentPreference setup that, after deployment, in the CloudFormation console shows me the various resources it has created.

As the above screenshot notes, no CodeDeploy "deployment" has been created. When I go to CodeDeploy in the console, I can see the Application and Deployment Group have been created... but Deployment is empty.
CodeDeploy Deployment is where the S3 bucket reference, Lambda details, and other deployment configuration should be referenced. So without this piece, when I upload a new zip to S3, no new Lambda version is created.
Here is my SAM for reference. (The S3 bucket is created in a different CF template and its ARN exported. I tried with the explicit S3 bucket + path string with the same result.)
AWSTemplateFormatVersion: '2010-09-09'
Transform: AWS::Serverless-2016-10-31
Resources:
  DummyFunction:
    Type: AWS::Serverless::Function
    Properties:
    ¦ FunctionName: BigOnion_dummyFunction
    ¦ Role: arn:aws:iam::XXXXX:role/BigOnion_Lambda_Logs_S3_Dynamo_SNS

    ¦ AutoPublishAlias: live
    ¦ DeploymentPreference:
    ¦   Type: AllAtOnce
    ¦   Role: arn:aws:iam::XXXXX:role/LambdaDeploy

    ¦ Handler: index.handler
    ¦ Runtime: nodejs6.10
    ¦ CodeUri:
    ¦   Bucket: !ImportValue s3BigOnionFunctionsHolder
    ¦   Key: 'BigOnion_dummyFunction.zip'

I have tried this in both us-east-1 and us-west-2 with the same result.

Thanks for the update @jfuss that CodeDeploy does not deploy but routes traffic between two versions. You may want to update some of the documentation pages as I foolishly went on a multi-day wild goose chase to try to get this working... :(

With just a few lines of configuration, SAM will do the following for you: [...] Deploy new versions of your Lambda function and automatically create aliases that point to the new version.

@bluepeter Couple of questions:

What exactly are you expecting?
Is this the first deployment? What happens when another deployment through CloudFormation happens?

So that is referring what 'SAM' will do for you not what CodeDeploy does. While SAM is using CodeDeploy, you should think about this as an implementation detail.

@jfuss I was expecting that any change to the underlying zip file would trigger a new Lambda version deployment. This way, I can initially deploy code + settings via CF once; and then do subsequent code deploys more simply via aws s3 cp ... to upload and replace the underlying zip (which CD would then deploy to Lambda).
I guess I can have a Lambda trigger on S3 do the same thing, but I was interested in explore an AWS recommended approach, and I mis-read CodeDeploy integration.

@bluepeter I would like to pick your brain here a little. Curious, why not do something more like this:

CI/CD system uploads to S3
CI/CD system calls update-change-set and then passes the new s3 location in as a parameter
CI/CD system executes change-set if the change set was created successfully

Why do you need this extra Lambda? Is there something here missing from SAM that would make it much more valuable for CI/CD systems?
It sounds to me like there is this gap you would like to be filled and I am trying to understand what exactly is missing or you feel is missing.

I suspect that we will eventually go exactly that route. We're in a heavy dev focused stage, with a manual deployment process using http://apex.run/
We are moving to CloudFormation to better document the increasingly complex relationships between resources. But we're finding it difficult to quickly and easily deploy to a dev environment with CF, particularly for Lambda functions. We have to i) push a new zip to S3 and ii) reflect the new name and aws deploy SAM. Slow.
So a thought is to use SAM to reflect the settings and resource dependencies initially. Deploy. And then, for future code deploys, use aws cp to replace the underlying zip. Or aws lambda. (Or even apex deploy.)
Eager to hear any suggestions/thoughts. CF/SAM is making things easier for Lambda (yet to get to API Gateway which is going to be messy), but, since we're in deep dev territory, we need a way to do quick deploys.

@bluepeter So if your team is going to move to CloudFormation eventually (by the sounds of it, you are moving over to it now), I would encourage you to take the time and move over now. The SAM team here has recognized the pain point you are describing. We have nothing concrete in solving it as of now, but is something we are thinking about. To help in making this faster for devs, you could write a simply script that does what you are describing (zip, upload, reflect). Once you have that, the iteration time will increase and will help in reflecting what will go to production (at whatever time that would be).
What are you trying to test though? Is it changes to a Lambda function or the entire system? Have you looked or heard about sam-local? The README says it best: "SAM Local can be used to test functions locally, start a local API Gateway from a SAM template, validate a SAM template, and generate sample payloads for various event sources." I think this may help reduce some of the churn you are seeing, even more so than my suggestion above.

Thanks @jfuss I had not heard of sam-local, and I will definitely check it out... later. Just finished the huge task of moving (mostly) everything to CF/SAM... need a bit of a break before onboarding new tech!


[Helm] Add OSM, Kuma, NSM adapters to the Meshery Helm Chart

Current Behavior
The Meshery Helm chart is missing adapters.
Desired Behavior
Meshery Helm chart should have all adapters.

This issue on my schedule

We can close this issue because of the referenced issues above.


ZED SDK v2.8.3 not working with CUDA 10.1 on Ubuntu 16.04

I get the following error message:
ERROR : Found CUDA 10.1 but the ZED SDK requires 10.0
The ZED will NOT work with this CUDA version (10.1), Continue  [Y/n] ?

Zed needs CUDA 10


patch to allow simai negative offsets

negative offset is unchecked for cases and results error from lexer itself.

Thanks for the PR; merging. &first is currently ignored by the parser btw


Fix middleware copying when creating new group

When creating new group when copying middlewares from parent router we loose some in the process. It seems that when allocating space for middlewares by mistake we allocate space for the NEW middlewares instead of the parent ones. This should fix the problem

Oh wow thanks @harnash


ESPSomfy AP back online

Hi, my device has been runnig well for a couple of days.
Yesterday it became unavailable, looking on the wifi netwoks available from my laptop, I sudenly realised that the AP I used to first setup the device was there.
The network config, shades, groups, rooms, everything was still there, but the AP keeps on and I cant access the device from my wifi networks.
I have made no changes to the device, nor to thw router or networks.

I am guessing that you have a computer that has the setting to automatically reconnect to the access point set.  This means that if your network goes down and the AP network comes back up again that computer will connect to it.  ESPSomfy RTS will consider that a connected device and not shut down the AP.
Check the computer that you used to configure ESPSomfy RTS and forget the AP.  It likely has the automatically connect setting turned on.

hi, it is having strange behavior, later yesterday worked again, and now the AP is back again.
Have to say that I think I´m having issues with the operator router and its leasing times, in fact some smart relays I have installed are going offline sometimes, maybe this is what´s going on with the device, we have to name it same way hehe, but what I can´t understand is why the AP goes online again....

The AP for ESPSomfy RTS comes online whenever it cannot connect to the currently defined network connection after some period of time.  It will not stop that AP if something connects to it but it will go away again when a connection can be made.  Likely you have whatever device you used to connect initially set up to automatically connect to the AP when it is visible.  In this case it will not shut down the AP which is the desired behavior.
The pre-release of v2.4.4 will perform this operation much quicker and will attempt to connect even when the AP is available.  Still it will not shut down the AP if there is a computer connected to it.  Please change your connection so that it does not automatically connect to ESPSomfy RTS AP when it is available.

Hi, well i have tried every device at home and the ap still online and i am unable to connect to the device vía my home wifi, vía its ap i can access It and have to say It can't see any wifi

Gotta ask.  Are you one of those folks that use a hidden SSID?

🤣🤣🤣
Nope! What for? Hehehehe

If you had said that you use hidden SSIDs then I would have aske you the same question... what for?  Anyhow ESP32 only supports 2.4ghz wifi so is your router set up for 2.4ghz?

Well, I have a dual band wifi and a 2.4 for smart devices

I'll try to get it set up where I have a failed connection for the AP and the soft AP is open.  In the meantime you can enter the AP manually then restart the device.

Ok, I'll fome back later

Done, i have changed host name, and set ip to dhcp, now I connect via host.local address

Hi, something happens, i went out and everything worked fine. Just came back hours later and nothing works.

Do you think something happens with the device? any malfunction, or defective, or something?

What do you mean nothing works?  Did the device drop off the network and the Soft AP reappear?

Yes and the device does not see any wifi

The first thing that comes to mind is the power supply.  If this is connected to a data cable you can see the logs in web.esphome.io to see what it is doing.  If ESPSomfy RTS locks up it will reboot as there is a watchdog timer.  However, if the power supply drops below power output it will not boot.

Hi, usually I use to plug It to a mobile charger, but i Will plug tomlaptop and send log

[22:26:53] 1 NTP Settings  [22:26:53]pool.ntp.org TZ:CET-1CEST-2,M3.5.0/02:00:00,M10.5.0/03:00:00 [22:26:53]WIFI Settings [22:26:53] SSID: [gus] PassPhrase: [Josesalo123] [22:26:53] [22:26:53]Launching web server... [22:26:53]Creating Web MicroServices... [22:26:53]WiFi Mode: 0 [22:26:53]WiFi interface ready [22:26:53]WiFi station mode started [22:26:53]Completed scan for access points [22:26:53]Scanned 2 Networks... [22:26:53]*0: gus (-75dBm) CH:1 MAC:78:94:B4:2F:38:BB [22:26:53] 1: vodafonegus24 (-76dBm) CH:1 MAC:8A:94:B4:2F:38:BE [22:26:53]App Version:2.4.3 shades.cfg exists so we are using that [22:26:53]Reading header at 0 version:24 len:76 roomSize:29 roomRecs:2 shadeSize:276 shadeRecs:4 groupSize:200 groupRecs: 1 pos:76 Setting S3 Transceiver Defaults... [22:26:53]Applying radio settings Setting Data Pins RX:4 TX:3 Setting SPI Pins SCK:7 MISO:8 MOSI:9 CSN:6 Radio Pins Configured! [22:26:53]Successfully set up the radio [22:26:53]Enabled receive on Pin #4 Timing: 1 Initializing RX Queue [22:26:53]Connecting to AP [22:26:53]WiFi clients stopped [22:26:53]WiFi interface ready [22:26:53]WiFi station mode started [22:26:53]Set hostname to:espsomfygus [22:26:58]Completed scan for access points [22:27:04]Found strongest AP 1 78:94:B4:2F:38:BB Timing Net: 5930ms Connected to access point [22:27:07]Got WiFi IP: <IP_ADDRESS> [22:27:07]Socket Server Started... [22:27:07]Max Heap: 204788 [22:27:07]Free Heap: 218428 [22:27:07]Min Heap: 215664 [22:27:07] [22:27:07]Successfully Connected to WiFi!!!!<IP_ADDRESS> (-75dbm) [22:27:07]MDNS Responder Started: serverId=97633C Max Heap: 196596 [22:27:07]Free Heap: 211988 [22:27:07]Min Heap: 211944 [22:27:48]Check github for updates... [22:27:48]Max Heap: 196596 [22:27:48]Free Heap: 212192 [22:27:48]Min Heap: 209124 [22:27:51]Internet is Available: 1244ms [HTTPS] GET... code: 200 [HTTPS] GET... code: 200 - 32481 Max Heap: 155636 [22:27:51]Free Heap: 209140 [22:27:51]Min Heap: 153076

after another devicec reboot,back to normal function,and this is the log
[22:30:48]s (-75dBm) CH:1 MAC:78:94:B4:2F:38:BB
[22:30:48] 1: vodafonegus24 (-76dBm) CH:1 MAC:8A:94:B4:2F:38:BE
[22:30:48]App Version:2.4.3
shades.cfg exists so we are using that
[22:30:53]Reading header at 0
Connected to access point
[22:30:55]Got WiFi IP: <IP_ADDRESS>
[22:30:55]Socket Server Started...
[22:30:55]Max Heap: 204788
[22:30:55]Free Heap: 218680
[22:30:55]Min Heap: 215720
[22:30:55]
[22:30:55]Successfully Connected to WiFi!!!!<IP_ADDRESS> (-87dbm)
[22:30:55]MDNS Responder Started: serverId=97633C
Max Heap: 196596
[22:30:55]Free Heap: 212036
[22:30:55]Min Heap: 211988
[22:30:58]Loading file /index.html
[22:31:16]Timing WebServer: 2631ms
Socket [0] Connected from <IP_ADDRESS> url: /
Initializing Socket Client 0
Timing Socket: 277ms
Max Heap: 172020
[22:31:16]Free Heap: 193992
[22:31:16]Min Heap: 187928
[22:31:32]Check github for updates...
[22:31:32]Max Heap: 188404
[22:31:32]Free Heap: 207840
[22:31:32]Min Heap: 180228
[22:31:35]Internet is Available: 1275ms
[HTTPS] GET... code: 200
[HTTPS] GET... code: 200 - -1
Max Heap: 147444
[22:31:35]Free Heap: 204948
[22:31:35]Min Heap: 149004

Both logs are perfect boot sequences.  Do you have an ip reservation for the device?

@gralfj has this problem been resolved for you?

Sorry, but no. Just very busy right now, ill come back.

I am going to close this for now but feel free to re-open it when you get time.


Fix inclusion of input source maps sourcesContent in JSModule compilations

It looks like sourceFileContentMap was incorrectly being cleared. sourceFileContentMap is populated during the compile initialisation, not when the sourceMaps are generated. So clearing it was preventing sourcesContent from being included in the source map.
Our pattern for jsmodule compilations is:
compiler.compileModules(EXTERNS, modules, options); // sourceFileContentMap populated here
for (JSModule module : modules) {
   compiler.getSourceMap().reset(); // sourceFileContentMap dropped here
   String source = compiler.toSource(module);
   SourceMap = compiler.getSourceMap();
   writeOutput(source, sourceMap);
}

Internal Google issue b/120608357 created.

Re-triggering Travis

Not hearing anything on this PR, I created #3166 as an alternative to address @shicks concerns.

Thanks @ChadKillingsworth , I'm travelling at the moment so can't follow up on this.

Let's go with #3172.  Thanks for bringing this issue to our attention!


Legendary Yumako Fruit Has Incorrect Recipe

Describe the bug
Factorio lab incorrectly believes that you can accept legendary yumako fruit to make legendary materials.
You cannot change recipes on a per recipe basis. This produces a behavior where it can suggest that you can create infinite legendary plastic easily.
Yumako seed originate from the ground, and if they do go legendary they do so at a low rate. You have to pass the fruit itself through a recycler to upgrade it. Factorio lab "knows" this in the sense that the recipe it has is to recycle a legendary fruit for 1/4 of a legendary fruit and it is using this to produce infinite yumako.
Jellynut also appears to be affected.
This is a difficult issue to fix because modules affect the recipe on a per module basis but quality 3 normal modules suggest a rate of return of 0.0001 legendaries on a recipe from normal-to legendary. Each tier is 10 percent higher.
To Reproduce
Open Factoriolab. Input recipe for coal. Set quality to legendary. Alternatively compare to iron, copper and coal.
Click to change recipe.
Go to yumako legendary. Clicking the drop down and going to the recipes will show that the recipe selection functionality has been moved. If you follow those instructions you'll see that the legendary recycling is recycling a legendary for a 25% rate of return.
Expected behavior
I expect to be able to get a coal from drills recipe that allows me to show how much coal from that resource it would take to make a single legendary coal. Even if not all tiers where provided for given the mechanical difficulty of coding that, a single recipe would show what realistic behavior is.
Screenshots

Additional context
A naive user could be misled into believing it is not 1 in thousands odds for any legendary recipe through this route.
A malicious one can this information to produce malicious output.
This is difficult. To provide all recipes appears to be computationally difficult since to model quality you would have to choose 4 modules from the set of all possible modules.

I don't think I'm following what you consider to be a bug here. If you want to model quality mining, you can. That's not what you're doing here, you're just asking for quality coal with no quality modules involved, so the calculator is doing its best to do that.
Perhaps there is a fundamental misunderstanding - the calculator is not capable of selecting modules for you, you have to specify which modules you want to use. If you do not tell the calculator you want to use quality modules, it will not allow them to be used anywhere.


Testing the new artoolkit5-js with Allow Memory Growth

⚠ All PRs have to be done versus 'dev' branch, so be aware of that, or we'll close your issue ⚠
What kind of change does this PR introduce?

This is a bugfix, i  am experimenting with the ES6 feature dev branch.
Can it be referenced to an Issue? If so what is the issue # ?
See issue #357 and https://github.com/AR-js-org/artoolkit5-js/issues/11
How can we test it?

Test the examples in the aframe and three.js folder, in particular nft examples are much affected
Summary

Sometimes running the examples (ES6 feature in dev branch) with a Mobile device i get a out of Memory issue as described in https://github.com/AR-js-org/artoolkit5-js/issues/11 this occur especailly with the NFT examples.
Does this PR introduce a breaking change?
Not to the final user. Changes occur internally arjs-context.js and the different artooolkit5-js package.
Please TEST your PR before proposing it. Specify here what device you have used for tests, version of OS and version of Browser
Tested with Oppo A72 running Android 11.1 and Chrome 95
Other information
This issue occurs only in Mobile devices.
This PR also remove the double artoolkit init in the Context class and in the worker. This is not necesssary because we init artoolkit already in the ARController _initialize https://github.com/AR-js-org/artoolkit5-js/blob/aa21c379d51b08e9a0d52b26f9f6928084ddb844/src/ARController.js#L1260
task to complete

[ ] make other tests
[ ] publish the new artoolkit5-js package and change the ref in AR.js package.json
[ ] restore old build path in nft examples

I need to test also with another Mobile device, but i think this should solve the issue.


Ability to load variable from file

This demonstrates a potential solution for https://github.com/urfave/cli/issues/613. If there is interest in possibly adding this feature we can flush out the implementation

This demonstrates a potential solution for https://github.com/urfave/cli/issues/613. If there is interest in this feature we can flush out the implementation

The failed unit test is because I updated the documentation in the readme without realizing there were validations. I wasn't exactly sure how to fix, but if someone can point me in the right direction I can update the pr

Awesome, that looks great

@bradrydzewski 👍 this is looking good, thanks for the contribution!
My only thought is that we may want to more explicitly document in the README what the order of precedence of the sources is (I was surprised we didn't already, but it feels prudent to do so now given the addition of the new source).

I am wondering if we should try to integrate this into the help output for flags with the FilePath defined.

Thanks for the review! I'll plan on updating the pr accordingly this coming week

My only thought is that we may want to more explicitly document in the README what the order of precedence of the sources is (I was surprised we didn't already, but it feels prudent to do so now given the addition of the new source).

I can certainly add this to the documentation.
I also meant to solicit feedback on precedence. Should it be Flag > Env > File? Or Flag > File > Env? Given the primary use case for loading parameters from file (secrets) it made me wonder whether or not it should take precedence over environment, although I'll admit I'm having difficulty articulating why one would be more important than the other. Any thoughts?

My only thought is that we may want to more explicitly document in the README what the order of precedence of the sources is

I added a section to document precedence for the file path. Let me know if you have any thoughts or feedback on the wording (writing good documentation is harder than writing code, IMO 😄 )

I am wondering if we should try to integrate this into the help output for flags with the FilePath defined.

I also added the filepath to help. Below is an example output. Please let me know if you have any feedback regarding the formatting.
GLOBAL OPTIONS:
   --password value  password for the mysql database [/var/secret/password]

ok, updated the PR based on feedback. Please note that a few of the Travis builders are failing for reasons that I can't explain. I noticed these same builders (Go 1.5 and Go 1.6) are also failing for #616 as well.

friendly ping @jszwedko ... I'm sure you are busy so I want to do as much as I can on my end to help push this through. I think the PR addresses most (all?) of your comments, but it looks like the pipeline is acting up. For the life of me I can't figure out what is going on. Any thoughts?
Thanks so much!!

Hi @bradrydzewski,
Apologies for the delay in reviewing this! This looks good to me.
Regarding the build failures, this appears to be due to some incompatibilities in the golang.org/x/tools tree for some Go versions. I'm working on fixing this over in #624. I'll drop a note here when that is merged.

#624 has been merged, updating this branch

And we're green, thanks again @bradrydzewski !

Actually, I have one more thought :smile:
The other fields that are like this, e.g. EnvVar, support multiple values. What do you think about having FilePaths be a []string so that users could specify multiple files (e.g. []string{"~/.mysql_password", "/etc/mysql/password"}) where it would try them until it found one to read from.

yes, I'll update the pull request next week with that capability. Thanks!

closing now that https://github.com/urfave/cli/pull/675 was merged (thanks @jmccann )


Remove "kludge" code that came from PIC32

Summary
Remove "kludge" code that came from PIC32
Impact
Improve code quality
Testing
ESP32 Ethernet board

#1274

#1274

Yes, but we cannot remove it from RISC-V without testing it on some RISC-V board.

#1274

Yes, but we cannot remove it from RISC-V without testing it on some RISC-V board.

I just want it to show as a reference in the issue's discussion.  I didn't want to link it to the PR, because if I do, it will be closed.
It'd be great if someone who uses RISC-V can verify that and remove the kludge as well.

Hi @acassis,
I noticed that you removed the following code which did not come from PIC32.
Actually the code was added to reduce power consumption and is the same as esp-idf's.
#if XCHAL_HAVE_INTERRUPTS
  __asm__ __volatile__ ("waiti 0");
#endif


Update library

Please update http library to latest version 1.1.0 actually
This will solve incompatibility issues with others libraries
Because aggregator_news depends on any_link_preview ^3.0.1 which depends on http ^1.0.0, http ^1.0.0 is required.
So, because aggregator_news depends on http ^0.13.6, version solving failed.

This will be available in Version 3.0.2

any_link_preview: ^3.0.2 is Live now.
Feel free to open this ticket if the issue persists. Thank you!


May I be a contributor of your project?

I am an IT student who studying Open Source Software.
I have to contribute and communicate with people on the GitHub.
Actually, I am a newbie about coding, but I could to contribute and help your in different ways.
For examples, writing documentation, design and error correction.
I am also doing the project that blurred images restoration with MATLAB.
The reason why I want join is, I want to learn and understand your method.
Sorry for this
I could not find your public email, so I have to put a issue for asking about that at the end.
Here is my public email by the way<EMAIL_ADDRESS>

Hello, thanks for your support!
This project comes from my course homework, actually i'm not going to maintain it because image-deblurring is not my research interest. If you have any questions, please email me by<EMAIL_ADDRESS>


Bluetooth: GATT: Write Without Reponse to invalid handle asserts

Describe the bug
Issuing a Write Without Response command to an invalid handle (i.e. this handle does not exist). This line of code asserts
To Reproduce
Steps to reproduce the behavior:
u8_t payload[5];
memset(payload, 1, sizeof(payload));
bt_gatt_write_without_response(pConn, 0x1234 payload,
			sizeof(payload), FALSE);

Expected behavior
The previous behavior was that the write command returned 0 and no callback etc. was triggered. In case that the remote (G)ATT is notifying of the error, it would be great to trigger the write callback.
Impact
Our Zephyr system is behaving according to remote user input, so we cannot assume that the user passes a valid handle (think typo) hence this issue is crucial to us.
 Zephyr SDK 0.10.1
Commit SHA or Version used:  SHA 386fcf3b53f523398c1d980ab3f691b8ab505607

Additional context
Compiled for nrf52_bsim

@cvinayak I do not have rights to assign you.

Not sure if this is better fixed at the Host or Controller level

All we do in the host is to convert to over the byte order if needed:
cmd->handle = sys_cpu_to_le16(handle);

Im surprised this even compile though.

"In case that the remote (G)ATT is notifying of the error, it would be great to trigger the write callback."
There shall be no response (even ATT Error Response to this command).
Core 5.1 Vol 3, Part G, 4.9.1 Write Without Response
"If the Characteristic Value write request is the wrong size, or has an invalid
value as defined by the profile, then the write shall not succeed and no error
shall be generated by the server."

Core 5.1 Vol 3, Part G, 4.9.1 Write Without Response
"If the Characteristic Value write request is the wrong size, or has an invalid
value as defined by the profile, then the write shall not succeed and no error
shall be generated by the server."

It's not wrong size, nor invalid value. It is a nonexistent handle but I am not that familiar with the GATT spec so maybe it is defined somewhere else.

Basically you have two types of ATT transactions:
ATT Request <-> ATT Response
ATT Command
the difference is that there is not response to ATT Command.

Cannot reproduce with the bug-description.
Root cause must be something other than the call to GATT Write without Response with Invalid Handle.
The handling of the invalid handle seems to be correct when receiving, and when sending it's not used for anything.
Also the thread seems to be the IDLE thread.
[00:00:00.672,546] <err> os: Current thread: 0x566e0cc0 (idle)

We need an updated way to reproduce this issue

Cannot reproduce it anymore as well, might have been something else indeed.


Add reference to the newly released PowerShell Module which contains …

…the status monitor
Last week, I used the legacy approach to setup Status Monitor.  Was wishing there was a reference to the new module that was just released so I added one in the hopes that it may help someone else.

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

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

#sign-off


abcjs player soundfont error

The abcjs webplayer isn't working for me on either folkrnn.org or themachinefolksession.org. Console error:

Hmm.
It works for me, here. But it shouldn't be accessing the soundfonts on github. That could be a symptom of the problem.
To get any further, I'll need more information. To start: to be precise, the page appears to load normally (i.e. as per the demo video on the homepage), with score. You can download the midi. But when hitting play on the audio player, you hear nothing?

Sorry for the delay.
To clarify: nope, the page loaded but the "compose" button had no effect: no url change, no content change in the main section of the page.
Looks like the issue was my Ghostery setup. But what, exactly, I'm not sure. I marked https://folkrnn.org/ as trusted, and the the sidebar "compose" app started working.
Surprisingly, I then told Ghostery to "restrict" the site… and the app still works. Thought maybe it was because I still had the csrftoken, but it works even after I delete it. Oh well 🤷♂ weird that I can't get it to stop working, but always working isn't so bad!

Sorry, that wasn't accurate. Misremembered the full problem. The tune generation thing was about Ghostery.

the page appears to load normally (i.e. as per the demo video on the homepage), with score. You can download the midi. But when hitting play on the audio player, you hear nothing?

Correct, I hear nothing and there is a console error.
My previous comment was off, but again it's something about a Chrome extension: it works fine in an incognito window. I should have tried that first. User error!

Thanks for the update.