aframe-xr won't load in WebXR-Viewer w/ aframe v0.8.2

Hey everyone,
Ever since upgrading our app from aframe v0.8.1 to v0.8.2, we are experiencing issues with running aframe-xr.
With further debugging we found out that the "enterAR" method won't load the scene correctly and will eventually crash it.
I suspect that it might be caused by an event emitter, but I'm still trying to find out what might cause that.
(note that the app works fine on the WebARonARCore browser)
Tested on iPhone 7, iOS v. 11.3.1 (15E302).
Latest https://github.com/mozilla-mobile/webxr-ios build.
Any ideas where to start looking?

Not sure where to start.  The version in vendor is still showing 0.7.x, I've never tried upgrading to even 0.8.
If you figure it out, I'd love to updating the version here to 0.8.x, but I (alas) don't have time to look into this.

I can confirm that the 0.8.1 version of aframe works with aframe-xr - I am using it as dependency and not from vendor. You still need to add three.xr.js, though.
About the 0.8.2 version, I am looking into that - still not sure if the problem lies in the WebXR Viewer or somewhere in between aframe-xr to aframe (or both).

With 0.8.2 I get
aframe-xr.js:1 Uncaught TypeError: Cannot assign to read only property 'enterVR' of object '#'
    at o.wrapSceneMethods (aframe-xr.js:1)
    at Object.module.exports.fireEvent (index.js:120)
    at HTMLElement.value (a-node.js:228)
    at a-node.js:118

This is apparently this line:
https://github.com/mozilla/aframe-xr/blob/50326e45f17fdfc87c1704cc7a16510058fba4e8/src/systems/xr.js#L29
maybe enterVR is now a read only property in aframe 0.8.2?

This seems to be a random issue, I don't have the error anymore...

I updated my last two comments to say the error I saw was on WebARonARCore.
On WebXR Viewer with aframe 0.8.2, I saw the same stacktrace via the react exception catcher because I have some part of the application with react, not the aframe part. Sometime I don't have the stacktrace, but the background is white, sometime it even works and I have a VR button at the bottom, sometimes I have the camera feed and the AR and VR buttons overlapping.

Glad to see some progress ... sorry I don't have much insight here!

Same here, tapping doesn't work for me on WebARonARCore, no error. Tapping works on WebXR Viewer.

For the error https://github.com/mozilla/aframe-xr/issues/17#issuecomment-387994835 on WebARonARCore, it was actually on 0.7.1 when I got this error, not 0.8.2. In aframe 0.7.1, the enterVR, exitVR are using writable: window.debug, in 0.8.2, it is writable: true.
For the click not working with WebARonARCore, with aframe 0.8.0 or 0.8.2, can it be related to a dom element that is above the aframe canvas? I had a similar issue with a full width and height transparent div above the aframe canvas in my project. I'll look closer into webxr-polyfill. Maybe you have some pointers @blairmacintyre
For the issue on WebXR Viewer with aframe 0.8.2 where the background stay white (do you have this behavior @Danpollak ?), it is probably a conflict or race condition between latest webvr-polyfill and webxr-polyfill? Actually I have sometime the camera feed working when I enable some of
aframe-ar code I'm trying to port, if I do window.webkit.messageHandlers.initAR.postMessage(data) myself for example (see code in https://github.com/chenzlabs/aframe-ar/blob/master/src/mozilla-xr-ar.js). From the webxr-polyfill code I read, it may not be a good idea to use window.webkit.messageHandlers.initAR and watchAR myself though, it may lead to other issues I think. I'll try to use updateFrame event instead when porting this code, like in the reticle component.

Ah I'm going somewhere. My intuition about the element above the canvas seems correct. It seems the a-scene element if above the canvas (child of a-scene) on WebARonARCore.
Looking at the click event with
document.addEventListener('click', function(e) {
        console.log(e.target);
});

without aframe-xr, no logs, with aframe-xr, I have logs saying the click is on a-scene.
With aframe-xr and the following code:
      document.addEventListener('DOMContentLoaded', function() {
        setTimeout(() => {
          console.log(AFRAME.scenes[0].canvas);
          AFRAME.scenes[0].canvas.style.zIndex = 50;
        }, 4000);
      });

the click event doesn't go to document (so no logs) and tapping works correctly, the issue is fixed. I think this has to do with the way the canvas is created for arcore, but I don't really know what is the difference between simple aframe and aframe+aframe-xr though.

With aframe alone (no three.xr), the canvas is created as a child of a-scene. With three.xr and aframe-xr, the canvas is created as a child of div.webxr-realities which is the first element in body.

So I continued to debug with WebXR Viewer
with
...

debug

...

In aframe-xr/src/systems/xr.js I put in cameraActivated the following line to debug:
document.getElementById('debugMessage').innerHTML = navigator.XR;
navigator.XR is actually undefined, this weird because ./dist/three.xr.js should contain vendor/webxr-polyfill.js and so define navigator.XR. So I finally edited my index.html file to include webxr-polyfill like this

and included three.xr.js by editing aframe-xr/src/systems/xr.js and put the following lines at the top
import WebXRManager from '../../../three.xr.js/src/WebXRManager';
import WebXRUtils from '../../../three.xr.js/src/WebXRUtils';

THREE.WebXRManager = WebXRManager;
THREE.WebXRUtils = WebXRUtils;

and rebuilt aframe-xr.
Now navigator.XR is defined and I go in aframe-xr/src/systems/xr.js initXR, I have two displays, the first supports ar, the second one doesn't support ar. I have the iPad 2018.
The code in THREE.WebXRManager https://github.com/mozilla/three.xr.js/blob/644c8489dd091f02ed9972c8f14f0a57aa405d26/src/WebXRManager.js#L205-L212 verify
if we have 1 ar and 0 vr or 0 ar and 1 vr, so it doesn't work for me, and this.startSession is never called.
If I modify the first condition to
if (arSupportedDisplays === 1 && this.options.AR_AUTOSTART) {

it works properly.

This fix is not enough, WebXRManager this.startPresenting go to the VR branch instead of AR branch because displayVR is defined. startPresenting should have a reality parameter I think.
My camera position and rotation are still not updated properly.

And in aframe-xr/src/systems/xr.js, disable look-controls on camera to not update the orientation on every tick (because it's enabled now with #18):
    if (this.supportAR && this.data.arAutostart) {
      sceneEl.camera.el.setAttribute('look-controls', 'enabled', false);
    }
    sceneEl.renderer.xr = new THREE.WebXRManager(options, displays, sceneEl.renderer, sceneEl.camera, sceneEl.object3D, this.updateFrame);

Well we should probably enable it back when ending the AR session, similar to what aframe-ar does https://github.com/chenzlabs/aframe-ar/blob/ff7202b838355be603ecd0722093f3c7586f4e08/src/ar-camera.js#L15-L20
@blairmacintyre in sessionEnded https://github.com/mozilla/aframe-xr/blob/50326e45f17fdfc87c1704cc7a16510058fba4e8/src/systems/xr.js#L129-L136
there is an error no?
Shouldn't it be the following?
sessionEnded: function (data) {
    if (this.activeRealityType === 'ar') {
      // Remove the transparency set in sessionStartedon on iOS devices
      document.documentElement.style.backgroundColor = '';
      document.body.style.backgroundColor = '';
    }
    this.activeRealityType = 'magicWindow';
    this.el.emit('realityChanged', this.activeRealityType);

I'll clean up my changes (removing all debug info :)) and do a PR for three.xr.js and aframe-xr to be easier to review.

@vincentfretin amazing work!!
I won't be able to test it till Sunday (no iOS available), but it seems like you nailed it! Will give a more in-depth feedback once I get my hands on an iPhone.

For the look-controls disabling and sessionEnded fix, I did it in the following commit
https://github.com/mozilla/aframe-xr/commit/a24c87450f9d8549ecde646054e72e4ef1e276bf
in my PR #18

Oh man I included dist/three-xr.js instead of dist/three.xr.js this is why navigator.XR was undefined. :(

For the raycaster click that wasn't working on both WebXR Viewer and WebARonARCore, I did this https://github.com/mozilla/aframe-xr/pull/18/commits/c0544e71d437f7e4cd604f76243c38783ddd5607 to fix it.
I believe #18 and https://github.com/mozilla/three.xr.js/pull/11 fixes all the issues we discussed here.


1st install - getting error 500, windows server 2016 datacenter edition

I tried with:

letsencrypt-win-simple.V<IP_ADDRESS>
letsencrypt-win-simple.V<IP_ADDRESS>-beta2
same result. So at least it is consistent.
The webserver is located behind a NAT router with port forwarding.
I checked that using regular folders and testfiles that the webserver works ok.
The server is running on a VM under Windows Server 2016 Datacenter edition.
And I believe that the problem arise from the web.config which somehow makes the server malfunction.

Anyone have an idea???
--- content of web.config in the acme-challenge folder ---

---
[INFO] Authorizing identifier certauth.fs.dataproces.dk using http-01 challenge
[INFO] Answer should now be browsable at http://certauth.fs.dataproces.dk/.well-known/acme-challenge/e4KEV5t7C_Z8IW01M6DYkzWkx1XE2yUN5kBcwyJfuyM
[INFO] Authorization result: invalid
[EROR] ACME server reported type urn:acme:error:unauthorized
[EROR] ACME server reported detail Invalid response from http://certauth.fs.dataproces.dk/.well-known/acme-challenge/e4KEV5t7C_Z8IW01M6DYkzWkx1XE2yUN5kBcwyJfuyM: "
Would you like to start again? (y/n):

I'm getting this list from the directory, so the files appear to be there, however, if I try at access any of the files themselves I'm getting a 404. This would normally be because there isn't a web.config file to enable the extensionless files. In this case you appear to have something at a higher level overriding the web.config file. The fact that I'm actually getting a listing is also an indicator of this as IIS doesn't normally do this sort of directory listing unless you specifically enable it.
BTW: We can't see the webconfig in your message as you haven't surrounded it with ``` lines.
NB: I don't seem to have a Windows 2016 demo machine handy so I can't tell if this is a new default or something you've changed though.
                        certauth.fs.dataproces.dk - /.well-known/acme-challenge/
                                              certauth.fs.dataproces.dk - /.well
     ___________________________________________________________________________

[To Parent Directory]
  9/7/2017  4:40 PM           87 e4KEV5t7C_Z8IW01M6DYkzWkx1XE2yUN5kBcwyJfuyM
  9/7/2017  4:56 PM          162 web.config
  9/7/2017  4:57 PM           87 YL_I1GHUGtOcp-FtfUEaC62YZSYku7KVeU3N5lgEtDQ
  9/7/2017  4:56 PM           87 z8BkuNmsqi2VQfoYoITm6Rbr_Chtumzs-IkM2gMjwAQ
     ___________________________________________________________________________

Noobs - huh? :-)
OK didn't see that. However you gave me the search criteria to go with: "extentionless" and it turns out that someone you might know already had an answer for this https://github.com/ebekker/ACMESharp/issues/15 So I followed those instructions:

Goto Site/Server->Mime Types
Add a mime type of .* (application/octet-stream)
and that solved the connection issue.

Now the next one is:
Do you want to specify the user the task will run as? (y/n): - yes
Enter the username (Domain\username): myDomain\administrator
Enter the user's password: NahNotGonnaGiveThatAway
[EROR] Exception Value cannot be null.
Parameter name: source
Would you like to start again? (y/n): - no
Any Idea what that could be if it is an issue at all ???

I guess that is https://github.com/Lone-Coder/letsencrypt-win-simple/issues/532#issuecomment-326778347

Did you by any chance install ASP.NET between encountering this problem and solving it? I'm kind of sceptical about the mimeType solution because that is already added by the default web.config. But it might be related to the user report in #550.

I'm going to close this due to it being quiet for a long time. Please feel free to post again though if you keep having issues with the latest release.


Search doesn't return particle phrases consistently

If I search e.g. kotak, I get all forms containing kotak (phrase and single-word), but if I search e.g. eka or namôya, I only get the particle, not any of their many phrases---I can get the phrase if only if I search it in its entirety, which is unhelpful to a learner.

This seems to have gotten implemented, cf.

@katieschmirler Does this now work better for you?

Yes, much better! I do notice when I search namôya now, it also returns (following all the previously missing particle phrases) a number of nouns where I'm struggling to see the formal or semantic connections (in most cases)--not an issue, I'm just curious as to what aspects of the search functionality result in this.

The semantically unrelated tail of the search results should be turned into an issue of its own, thus closing this issue.


No inpaint UI

I'm not getting any errors, but when I send my img to inpaint it's not loading the inpaint UI. Mask options  etc is missing. It looks the same as the img2img page.

Update:sometimes it's working after running the cell again and then again it's not.


Implement Undo for tasks and events add & delete

Context
What is done:

undo delete, undo add, undo mark
sorting both events and tasks.

tasks are sorted: floating tasks always on top, and the rest from earliest to latest
events are sorted: by their starting time.

What to take note:

Notice the change in Command class, as well as the UndoableCommand abstract class.
There are some minor changes to the AddTask/Event, DeleteTask/EventCommand, and MarkCommand, please verify those.
Take note of some test cases may behaviour differently with the sorting function.

More specifically, when a task or event is deleted and undo, it may appear at a different position in the filtered list. It is because of the sorting only takes into account deadline and duration, also because it is not adding into an absolute index (which can be implemented, but it may result in me changing many or ur codes.) @daredevil999 if you are to implement AddCommand with index, it is then possible.

Links
#55
Reviewers
@CS2103AUG2016-F09-C4/developers

Looks good, do you mind updating the diagrams to reflect the undocommand inheritance?

Since this branch has a lot of changes to the files, can we merge this during our meeting tmr? By then, everyone should be done with their respective tasks. Thanks!

Okay cool.
I have updated the developer guide for my part in the master as well.


Update regression.js

updated:
1.included r Square output from linear and logarithmic function
2.increased precision of calculated value to 4 place
3.included moving average.

coefficient of determination and precision configuration has been added in #48


Templates clean-up

Issue Type
Code related
Description
Shouldn't templates directory (internals/templates) be cleaned-up after initialization? Is there any reason for them to stay there for the future and be further commited with the project?
Steps to reproduce
Doesn't apply 📝
Versions

React-Boilerplate: 3.4.0
Node/NPM: -
Browser: -

Feel free to open a PR with your change


Added static redirect for HTML-to-JSX page

Relates to reactjs/react-magic/pull/145 and facebook/react/issues/11080.
Verified that Gatbsy redirects still work:

/docs/index.html -> /docs/hello-world.html

Verified that this new static redirect works:

https://deploy-preview-6--reactjs.netlify.com/html-jsx.html -> http://magic.reactjs.net/htmltojsx.htm

cc @Daniel15 😄

This is uncontroversial enough... YOLO

Thanks! You could probably make it a permanent (301) redirect.

So picky 😆  29835da


Lint documentation directory

This PR is part of #236 endeavor.

@Digipalvelutehdas/apika-tiimi please review


Module 4 exercises [Ampadu Boateng Daniel]

WS2_M4E1_Template.docx

Well done, Daniel. You clearly grasped the material outlined in Module 4 and successfully applied it to these exercises. Thank you for your participation! Please be sure to fill out the post-Module survey before you begin the next Module.


Sendgrid v3

This needs to me merged BEFORE of email changes to spoton2, else new email stuff will not work in hotspot.

Travis CI build fails ... I think I need to update setup.py etc to account for changes ...

setup.py now calls for sendgrid 6.1.0. the travis build seems to be ok now.

@tomohara Can you bump the major version of this library to 2.0.0


Implement an ARC and/or WARC file writer

For archiving crawlers

https://github.com/iipc/webarchive-commons
we can work on it together :)


XSS vulnerability in  label,SVG include attack vector.

This label and attack vector will cause dom-based XSS.
if you type
<EMBED SRC="data:image/svg+xml;base64,PHN2ZyB4bWxuczpzdmc9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIiB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMjAwMC9zdmciIHhtbG5zOnhsaW5rPSJodHRwOi8vd3d3LnczLm9yZy8xOTk5L3hsaW5rIiB2ZXJzaW9uPSIxLjAiIHg9IjAiIHk9IjAiIHdpZHRoPSIxOTQiIGhlaWdodD0iMjAwIiBpZD0ieHNzIj48c2NyaXB0IHR5cGU9InRleHQvZWNtYXNjcmlwdCI+YWxlcnQoIlhhYVMiKTs8L3NjcmlwdD48L3N2Zz4=" type="image/svg+xml" AllowScriptAccess="always"></EMBED>，the xss vulnerability will be triggered.And other markdown online editors that use this component are also affected by this vulnerability,like MdEditor.

+1

Are there any updates on this and will this be fixed?


Allow Users to Create and Trigger Arbitrary Hooks

What happened (or feature request):
Feature Request
What you expected to happen:

End-users have been using our pre/post hook functionality (documented here https://ddev.readthedocs.io/en/latest/users/extending-commands/)
Additional use cases are popping up where end-users wish to trigger a predetermined series of commands that are not specifically tied to an existing DDEV event like starting the containers OR importing a database.
It would be ideal to allow for additional entries that could be invoked stand-alone, such as an installation profile (see #5) or for triggering a series of unit or integration tests.
A sample command might be something like ddev hook insert-task-key where "insert-task-key" would be stored in .ddev/config.yml under the "hooks" key.

Please use a complexity rating of 1-5 (5 is high) for a feature request. (High complexity implies more PR planning)
3

The other approach would be to allow to extend ddev commands pool, or to create aliases for complex ddev exec.
Our use case is e.g. to be able to easily run  tests (unit/functional...) in the container.
right now I need to call:
ddev exec ./bin/phpunit -d memory_limit=1G -c vendor/typo3/testing-framework/Resources/Core/Build/UnitTests.xml

and I would love to be able to shorten that to:
ddev exec unitTests

or sth similar.
Yet another solution would be to allow creating custom shell aliases from configuration, this way the command would be available in both ssh console (ddev ssh) or ddev exec <alias>

just to provide an idea how we would might use it:
commands:
    gerritStart:
      - exec-host: "ddev run gerritConfig"
      - exec-host: "ddev run gerritCommitMesssageHook"
      - exec-host: "ddev run gerritPreCommitHook"
      - exec-host: "ddev run gerritReset"

    gerritConfig:
      - exec-host: "git config user.name \"Your Name\""
      - exec-host: "git config user.email<EMAIL_ADDRESS>
    gerritCommitMesssageHook:
      - exec-host: "mkdir .git/hooks"
      - exec-host: "cp Build/git-hooks/commit-msg .git/hooks/commit-msg"
      - exec-host: "chmod +x .git/hooks/commit-msg"

    gerritPreCommitHook:
      - exec-host: "mkdir .git/hooks"
      - exec-host: "cp Build/git-hooks/unix+mac/pre-commit .git/hooks/"
      - exec-host: "chmod +x .git/hooks/pre-commit"

    gerritReset:
      - exec-host: "git reset --hard origin/master"
      - exec-host: "git pull"

    validateRst:
      - exec: "Build/Scripts/validateRstFiles.php"

    unitTests:
      - exec: "bin/phpunit -c vendor/typo3/testing-framework/Resources/Core/Build/UnitTests.xml"

    functionalTest:
      - exec: "export typo3DatabaseName=\"func\" typo3DatabaseUsername=\"db\" typo3DatabasePassword=\"db\" typo3DatabaseHost=\"localhost\" typo3InstallToolPassword=\"klaus\" && bin/phpunit -c vendor/typo3/testing-framework/Resources/Core/Build/FunctionalTests.xml"

    # enableXdebug:
    # disabledXdebug:
    # https://docs.typo3.org/typo3cms/ContributionWorkflowGuide/DebuggingTypo3/Index.html

    openGerrit:
      - exec-host: "open https://review.typo3.org"

    openContribGuide:
      - exec-host: "open https://docs.typo3.org/typo3cms/ContributionWorkflowGuide/"

This would make it very easy for developers to onboard, as very common tasks are abstracted into a very transparent way

I don't think you'd want to run ddev inside an exec-host, but just to mention here as in Slack: Some of these things are easily done by adding scripts in the repo. So in a scripts directory for example, a gitsetup.sh script could do your git setup, and you could have exec: scripts/gitsetup.sh

the idea was to provide an consistent entry point for devs ... so maybe i will stick with composer scripts for now ... but this feels weird too ...

This is a big topic and related to customer requests on our hosting product. Tagging "needs decision" because there are likely other considerations that need to be addressed as well.

currently we changed it to integrate it into composer.json ... so it might be not important anymore (atleast for TYPO3)

We are currently running into the same problem. We have some commands that needs to run in a container for our frontend developers. Especially webpack or linting commands. Our frontend developers are not experienced with command line, so we want to provide simple commands for them.
Currently, we put all the necessary commands in a readme file so that they can copy&paste the commands. We do not want to create a separate shell script for the commands, because we want to support the major platforms because of freelancers. To do so, we would have to create a shell and batch file and keep them in sync.
So it would be nice to define custom commands inside ddev config to improve working with ddev for people, that are not experienced with command line. They need to focus on their work and not learning what containers the project provides and what options the exec command has and also what's the document root inside the container.

@rfay about
    gerritStart:
      - exec-host: "ddev run gerritConfig"
      - exec-host: "ddev run gerritCommitMesssageHook"
      - exec-host: "ddev run gerritPreCommitHook"
      - exec-host: "ddev run gerritReset"

I agree that running ddev inside an exec-host is not good. But in some cases this example might make sense. What about a ddev-host command which eventually can only be run inside an exec-host hook and is aware of that context? With that it might be possible to provide some dynamic environment variables extracted from host information into the containers before they are started as well. That for example would make my setup we discussed much easier where m pre-start host script writes the extracted information in some files inside the .ddev folder which then can be accessed and applied by the pre-start exec script inside the container. I would not even need a exec hook then because I could use getenv() function from PHP directly.

It looks to me like custom commands have met the needs expressed here, people use them extensively.
Also, there are now pre/post hooks for many, many things.
Closing, happy to continue the conversation or reopen if I've missed the point.


Typo in pyes/query.py

Found a tiny typo in pyes/query.py.
As per http://www.elasticsearch.org/guide/reference/query-dsl/nested-query.html, shouldn't this be 'score_mode'?
Pasting the patch here,
https://gist.github.com/1320540

Fixed in cf44521b52881d0afe221a3630e397f796d6a011. Thank you!


[NDTensor] Create extension folder so functions can be accessed.

Description
Create a extensions folder in NDTensors and define some functions for GPU extensions
Checklist:

[ ] Decide on storagemode/buffertype
[ ] AbstractGPUArray extensions
[ ] Generic adapt_storagetype
[ ] Default parameters for GPUs
[ ] Create extensions folder
[ ] All tests pass

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

Project coverage is 53.78%.  Consider uploading reports for the commit ae6b65a to get more accurate results

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1352       +/-   ##
===========================================
- Coverage   84.40%   53.78%   -30.62%
===========================================
  Files         100       99        -1
  Lines        8581     8528       -53
===========================================
- Hits         7243     4587     -2656
- Misses       1338     3941     +2603

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

The new NDTensors/src/lib/*Extensions libraries should be organized like the other libraries in NDTensors/src/lib, with src and test directories and source code organized into a module *Extensions.

As discussed I am moving all of the using statements from NDTensors.jl to import.jl and defining things like using .CUDAExtensions: cu so operations like NDTensors.cu still work.
I am also going to move cpu from Expose to GPUArraysCoreExtension

@mtfishman I pushed my changes here, I am stepping out this afternoon and will look through the changes later this afternoon to verify that everything is inline. Thanks!


Avoid error message regarding cacerts when running with read-only file system

Running containers with read-only file system increases runtime security.
While this is possible with images created by this build pack, and error message like the following is logged on container startup:
  E 2020-07-17T12:34:01.498115853Z unable to open /layers/paketo-buildpacks_bellsoft-liberica/jre/lib/security/cacerts
  E 2020-07-17T12:34:01.498122355Z open /layers/paketo-buildpacks_bellsoft-liberica/jre/lib/security/cacerts: read-only file system

Are there any consequences when the cacerts file is not being modified as intended?
Can this be avoided or the message deactivated?

Supporting this functionality effectively prevents you from ever contributing non-OS-included CA certificates to the JVM.  We can make this clearer (probably with a large warning), but running with read-only filesystem will result in a demonstrable functional deficiency.


Fix argument issue on Makefile for clang plugin

In clang_plugin of makefile, some of arguments aren't escaped with "". Then it's build break on some environment.
The following is error case's example. Then this change will solve the issue with "" escape.
[01:22:22][82676] Building clang plugin OCaml interface... [*ERROR**][82676] *** ERROR 'Building clang plugin OCaml interface' [*ERROR**][82676] *** command: ' make -C /Users/hidenorly/work/infer/facebook-clang-plugins/clang-ocaml build/clang_ast_proj.ml build/clang_ast_proj.mli CC=clang CXX=clang++ -std=gnu++11 CFLAGS="-g -O2" CXXFLAGS="-g -O2" CPP="clang -E" LDFLAGS="" LIBS="" LOCAL_CLANG=/Users/hidenorly/work/infer/facebook-clang-plugins/clang/install/bin/clang CLANG_PREFIX=/Users/hidenorly/work/infer/facebook-clang-plugins/clang/install CLANG_INCLUDES=/Users/hidenorly/work/infer/facebook-clang-plugins/clang/install/include SDKPATH=/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator.sdk ' [*ERROR**][82676] *** CWD: '/Users/hidenorly/work/infer' [*ERROR**][82676] *** stdout:
[*ERROR**][82676] *** stderr:
[*ERROR**][82676] make: invalid option -- =
[*ERROR**][82676] make: invalid option -- g
[*ERROR**][82676] make: invalid option -- u
[*ERROR**][82676] make: invalid option -- +
[*ERROR**][82676] make: invalid option -- +
[*ERROR**][82676] make: invalid option -- 1
[*ERROR**][82676] make: invalid option -- 1
[*ERROR**][82676] Usage: make [options] [target] ...

Since this is makefile, I did build test with ./build-infer.sh clang and ./autogen.sh; ./configure; make And build was success with this modification.

Thank you @hidenorly , I wish I'd seen this sooner! I did something similar and a bit more extensive in c30c04535058f78d2e7770d1f2cbdd6c180462ea so I believe we can close this one


summary.out - add column for crop status at end of season

Possible crop statuses:
1 - crop harvested at maturity
2 - crop harvested on reported date
11 - failure to plant (automatic planting)
12 - failure to germinate
21 - crop mature due to slow grain filling
31 - crop died due to heat stress
32 - crop died due to cold stress
33 - crop died due to deficit water stress
34 - crop died due to excess water stress
51 - crop died due to pest damage

Should be able to set this value whenever MDATE is set because MDATE controls the simulation end date. (Is this true in every case?)

In MZ_GROSUB.for, line 1668, added the first example of CropStatus. It doesn't go anywhere yet, but this is what we need to do if we agree on the list of crop statuses.

Maybe we need a more generic RunStatus to also include things like:

weather data missing
soils data missing
management data missing
crop genetics data missing
DSSAT code files missing
Simulation options invalid
etc.

There are 719 instances of MDATE in the code, but most of these are passing the variable around or documenting it in comments. Less than 20% are actually setting the value and have to be handled. This is manageable.
There are 2323 instances of "CALL ERROR" in the code. Each would have to be handled! Maybe we don't want this, but keep it at crop status only.

commit 52a0bc4


Release/8.0.1

:tada: This PR is included in version ara-client-v1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Change suggestion_for_argument to return ['<file>', '<directory>'] for PATH arguments

completely allows completing both files and directories. If an argument is named PATH or *_PATH then suggestion_for_argument should return ['<file>', '<directory>'].

Implemented by 7c80a9a.


Commonalities release 0.2.0

Problem description
We have introduced changes to API Design Guidelines: mainly related to notifications in CloudEvents format, but also some other updates and corrections. Should we have the next release?
Expected action
Agree with TSC on Commonalities release schedule.
Define PRs and Issues that should be closed before the next release.
Additional context

Fully agree. Since the design guidelines included the CloudEvents format, I definitely support this idea to have a new release.

In order to finalize changes related to CloudEvents format we need to close:

PR#43 #43
PR#83  #83
and  decide on issues: https://github.com/camaraproject/Commonalities/issues/89 and https://github.com/camaraproject/Commonalities/issues/90

I propose to focus on PRs: #43 and #83 as directly related to CloudEvents.
Issue #89 was agreed to be closed and issue #90 extends the Subscription functionality and needs more discussion.


Error Java: java.lang.NoClassDefFoundError...

i have error java on my server  because of plugin quests v 2.5.1-b82

my server running on spigot 1.8.

do you have vault installed? if not, add Vault plugin.

Closed due to lack of response.


How to run benches for all runtimes?

E.g. https://github.com/paritytech/polkadot/pull/6155#issuecomment-1278639790
Instead of specifying "westend", is it possible to run benchmarks for all runtimes with one command?

Currently the only way to do it is to post multiple lines:
/cmd queue -c bench-bot $ pallet westend-dev runtime_parachains::hrmp
/cmd queue -c bench-bot $ pallet kusama-dev runtime_parachains::hrmp

This is far from ideal, since it compiles twice and therefore blocks the runner and queue for much longer.
I think @mordamax is working on some improved github-external UI.

I guess this is similar https://github.com/paritytech/pipeline-scripts/issues/68 ?
but indeed, we'll try to include this case into requirements to improvements

Will be improved within https://github.com/paritytech/command-bot/issues/131


No speedup for advertised eduction example

Following the example in the 0.3 announcement post. Was hoping to see ~10x performance improvement.
Was there a regression with 0.4?
(transduce) pkg> status
    Status `~/temp/transduce/Project.toml`
  [6e4b80f9] BenchmarkTools v0.4.3
  [28d57a85] Transducers v0.4.0

julia> itr = (y for x in 1:1000 for y in 1:x);

julia> @btime foldl(+, eduction(itr))
  222.099 μs (4 allocations: 96 bytes)
167167000

julia> @btime sum(itr)
  198.473 μs (1 allocation: 16 bytes)
167167000

So the new fix #45 makes it ~10x faster than 0.3 and so ~100x faster than the iterator.

Excellent! And record-setter for fastest bugfix I've experienced.
Wondering if benchmark testing should compare against non-transducer implementations and ensure a minimal level of performance improvement. This could also be a nice way for users to see the performance improvements on their systems.
I assume the PkgBenchmark judge is for comparing the same functions on the same hardware. Still helpful for catching this sort of regression though.

Wondering if benchmark testing should compare against non-transducer implementations and ensure a minimal level of performance improvement.

I'm actually doing this in TransducersBenchmarksReports.jl: https://github.com/tkf/TransducersBenchmarksReports.jl/blob/master/docs/src/gemm.md#transducers-vs-manual-implementation
I haven't hooked it up Transducers.jl's CI as it's a bit tedious to set it up (compared to just calling judge).

Still helpful for catching this sort of regression though.

Yep, I should have added it in the benchmark :)


Issue with "Migrating MineMeld to Cortex XSOAR" in @site/docs/reference/articles/minemeld-migration.md

Describe the problem

Page: Migrating MineMeld to Cortex XSOAR
Source: https://github.com/demisto/content-docs/blob/master/content-repo/extra-docs/articles/minemeld-migration.md

The section around Tor Exit Node has the confidence score of greater than or equal to 75.  This value is not available from the TOR feed.
type:IP and sourceBrands:"Tor Exit Addresses Feed" and **confidence:>75** and trafficlightprotocol:Green
Screenshots
Indicators from Tor Feed:

If you run that search from the page:

Environment

OS: Mac
Browser: Chrome
Browser Version: Something.

Suggested fix
Modify the search to take the confidence score out, and also call out if they want to use the TLP of green, they need to set that on the Feed to use it in search, but it's not really required so you can do either of these:

Setting TLP Green:


Dragonrage: Watch japaneseknotweedkillers.com

Dragonrage requests the watch of the watch_keyword japaneseknotweedkillers\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

japaneseknotweedkillers\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Rejected by Jeff Schaller in Charcoal HQ. Reason: 'already watched; spare filler text from SO'


Use the newer API Update init.py

Starting from version 9.0.0 of Pillow, Image.ANTIALIAS has been deprecated and the use of Image.Resampling.LANCZOS is recommended instead.

之前已经有所修复，该pr关闭


Autoupdater

Add autoupdater. This will be done using hazel and electron's autoupdater feature.

Added, currently testing (1.0.0-alpha.2 and .3)

Fucked it up, trying again now

Doesn't seem to work, but I'm not getting any errors either. Not sure what's going on.

It is done


Feature/add tags small card

Instructions
See #551

closes #551

Hey Dane, Edward asked me about why I thought the build might be failing on this one.  I'm frankly not sure, because when I commitlint locally, it's fine:
~/dev/design-system(feature/add-tags-small-card ✔) echo "feat(tag): adds tag to small card for issue #549" | commitlint

⧗   input: feat(tag): adds tag to small card for issue #549
✔   found 0 problems, 0 warnings
    (Need help? -> https://github.com/conventional-changelog/commitlint#what-is-commitlint )

~/dev/design-system(feature/add-tags-small-card ✔) echo "Merge branch 'master' of github.com:pluralsight/design-system into fe" | commitl
int

⧗   input: Merge branch 'master' of github.com:pluralsight/design-system into fe
✔   found 0 problems, 0 warnings
    (Need help? -> https://github.com/conventional-changelog/commitlint#what-is-commitlint )

~/dev/design-system(feature/add-tags-small-card ✔) echo "test: updates snapshots of card" | commitlint

⧗   input: test: updates snapshots of card
✔   found 0 problems, 0 warnings
    (Need help? -> https://github.com/conventional-changelog/commitlint#what-is-commitlint )

~/dev/design-system(feature/add-tags-small-card ✔) echo "Merge branch 'master' of github.com:pluralsight/design-system into fe" | commitl
int

⧗   input: Merge branch 'master' of github.com:pluralsight/design-system into fe
✔   found 0 problems, 0 warnings
    (Need help? -> https://github.com/conventional-changelog/commitlint#what-is-commitlint )

Can you make sense of the mondo build log output?

no idea. let's just merge


bump-formula-pr fails when attempting to sign commit

[x] Confirmed this is a problem with running a brew command and not brew installing or the post-install behaviour of one or more formulae? If it's a formulae-specific problem please file this issue at the relevant tap e.g. for Homebrew/homebrew-core https://github.com/Homebrew/homebrew-core/issues/new
[x] Ran brew update and retried your prior step?
[x] Ran brew doctor, fixed all issues and retried your prior step?
[x] Ran brew config and brew doctor and included their output with your issue?

$ brew doctor
Your system is ready to brew.

brew-config
To help us debug your issue please explain:

What you were trying to do (and why)
attempting to bump a formula with bump-formula-pr
What happened (include command output)
Error when it attempting to git-commit.

$ HOMEBREW_DEVELOPER=true brew bump-formula-pr --url=https://github.com/nodenv/node-build/archive/v2.6.20.tar.gz --sha256=af89b9eb0086324a78674d31e7d74e750d30ac8565b27058431b1b854b068b9a node-build

Already up-to-date.
==> replace "https://github.com/nodenv/node-build/archive/v2.6.19.tar.gz" with "https://github.com/nodenv/node-build/archive/v2.6.20.tar.gz"
==> replace "ed4b5aa3e5d6c474f7e53e9a78b6080bb7e57497ff1d2c67ddbc2700d0c5c2d0" with "af89b9eb0086324a78674d31e7d74e750d30ac8565b27058431b1b854b068b9a"
M	Formula/node-build.rb
Switched to a new branch 'node-build-2.6.20'
error: gpg failed to sign the data
fatal: failed to write commit object
Error: Failure while executing: git commit --no-edit --verbose --message=node-build\ 2.6.20 -- /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core/Formula/node-build.rb

What you expected to happen
expected it to commit and open a PR
Step-by-step reproduction instructions (by running brew commands)

My git configuration has user.signingkey set and commit.gpgsign=true.
 So I'm presuming the issue is that the brew process is running under a different user so it doesn't have access to gpg-agent, thus it's unable to sign the commit. However, I'm not sure what the remedy should be here. (This isn't the first issue I've encountered with homebrew respecting the user's gitconfig, but then failing due to some configuration therein.)
Is there some configuration that other homebrew devs use? I assume I'm not the only one who has git configured to sign commits.

We now filter environment variables by default in your configuration. It's likely we may want to passthrough some GPG variables to enable signing to work.

I couldn't reproduce this today, FWIW. I'm not immediately sure why, I expected it to fail based on your report and failures seen with brew pull.
Relevant section of config:
[commit]
	gpgsign = true
[gpg]
	program = gpg

@DomT4 still failed with unset GPG_TTY (which I rather expected because it isn't normally set in my environment, anyway)

I tried @woodruffw 's workaround (export GPG_TTY=$(tty))... same error.

Keep discussing this here but given the issue is affecting one person and others can't reproduce this: it warrants closing, sorry.

I'm not really sure how to keep debugging this. For anyone else who encounters this:
to work around the issue, I just disabled commit signing in homebrew-core's repo (tap):
$ pwd
/usr/local/Homebrew/Library/Taps/homebrew/homebrew-core
$ git config commit.gpgsign
false


✨Blockcode

Feature Request
Add a code block to the editor.
See with Blocknote first if it is a feature they are working on.

See: https://github.com/TypeCellOS/BlockNote/issues/721

Added on Blocknote side: https://github.com/TypeCellOS/BlockNote/releases/tag/v0.18.1


Contactform branch

project requirements
In that section, I built for this milestone:

the contact form: Form
and the footer (if present in the design)

I made sure that:
The Name field accepts up to 30 characters.
The Email field only accepts emails.
The message field accepts up to 500 characters.
All fields have placeholder text.
All fields have the correct type.
A user cannot submit the form if one or more fields are empty.
Collect data using the Formspree service.
To lay out the elements on the page, you must use Flexbox.

No need to implement any functionality that requires JavaScript.
Keep the design as much as possible (e.g. font, colors, images, text, margins) using Figma's templates.
Implemented the button interactions (activate, hover, etc.).
The site must be responsive following these breakpoints:
The minimum mobile screen size must be 375px
The minimum desktop screen size must be 768px
I've used responsive values in your CSS rules like percentages (i.e. width: 80%) instead of pixels.

Remind me about the big picture of this project.
Mode of work and submission
I have implemented the above requirements in all repositories in your pair programming group.
We made and worked together with commits to guarantee joint participation in the project.

I made and fixed the optional suggestion prompts.


Possible to manually update x, y, w and h properties?

I want to set the coordinates and size of the drag-item manually using javascript, besides dragging/resizing.
The x,y, w and h properties seem to be only default values.
Is there any way to update the drag-item with new values and display it on its new position?

@Dylan190774 show your code or use :x="some_number" :y="some_number".

It's a Scrabble like word-game.
I have an array letters that hold the position, size and letter.
      <VueDragResize
        v-for="(letter,letterindex) in letters"
        v-bind:key="'letter_'+letterindex"
        :isDraggable="true"
        :isActive="true"
        :isResizable="false"

        :x="letter.x"
        :y="letter.y"
        :w="letter.width"
        :h="letter.height"
        :minw="letter.width"
        :minh="letter.height"

        @clicked="onActivated(letterindex)"
        @dragging="(rect)=>onDragging(letterindex, rect)"
      >
        <div
          :class="'letter letter_'+letterindex"
          @touchstart="onTouchstart(letterindex)"
          @touchend="onTouchend(letterindex)"
        >
          {{ letter.letter }}
        </div>
      </VueDragResize>

In onTouchstart and onTouchend I decide if the letter is placed right on the board or that it should go back to the rack. I do the animation with jQuery. This works, but when I try to drag the same letter again, the x and y are still in the same place where I stopped dragging. This should be updated to the new position it got after animating to the rack.
Hope you understand :)

Here is the template for the demo. Could you please add your code to show the problem?

I made a simplified example
The tile can be dragged, and when you click the button it is moved to a new location. But this only works once.
Is there some way to reset it, so I can keep on moving it with Javascript as well as being able to drag it ?

Hi @Dylan190774
From what I understand taking into account your 2 posts here is the basic fixes.
Regarding

In onTouchend I decide...

you did not provide any if statements, so I can not comment it.
If you need any additional help, add additional code where needed and write step by step your use case.

Thanks for taking the trouble to help me with this.
Eventually of course I don't want to use the buttons. The return of the tile to the rack should happen when the tile is placed wrong, so this should be triggered when the dragging stops (or touchend, but that doesn't seem to work well either).
Also , in touch mode (Chrome Toggle device toolbar) the buttons work only once, and then do nothing anymore. I'm wondering what is the cause of that.

I don't want to use the buttons. The return of the tile to the rack should happen when the tile is placed wrong, so this should be triggered when the dragging stops (or touchend, but that doesn't seem to work well either).

I can help you to fix this, but you need to provide an appropriate code (function which checks wrong/right place of tile and in case of wrong place return tile to the rack).

Could you please respond to my other question about the dragstop event in touch-mode (#37)  ?  If I could catch the moment the dragging stops in touch mode, I think I can get it working the way I want. Somehow if I use a touchend-event to do this, the tile is not reactive anymore.

Im trying to update the width and heoght with a pinch movement, resizing the variables in touchmove event. It only refreshes the size on touchend though... Any ideas?


CeskaTelevize - no video found

Checklist

[ ] I'm reporting a broken site support issue
[X] I've verified that I'm running youtube-dl version 2020.01.01
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log

PASTE VERBOSE LOG HERE

youtube-dl --verbose --add-metadata --ignore-errors --keep-video -o '%(title)s.%(ext)s' https://www.ceskatelevize.cz/ivysilani/12673563853-lidstvo-pribeh-nas-vsech/219382567420001
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'--add-metadata', u'--ignore-errors', u'--keep-video', u'-o', u'%(title)s.%(ext)s', u'https://www.ceskatelevize.cz/ivysilani/12673563853-lidstvo-pribeh-nas-vsech/219382567420001']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.01.01
[debug] Python version 2.7.17 (CPython) - Linux-5.3.0-24-generic-x86_64-with-Ubuntu-19.10-eoan
[debug] exe versions: ffmpeg 4.1.4-1build2, ffprobe 4.1.4-1build2, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[CeskaTelevize]<PHONE_NUMBER>20001: Downloading webpage
[CeskaTelevize]<PHONE_NUMBER>20001: Downloading JSON metadata
[CeskaTelevize]<PHONE_NUMBER>20001: Downloading JSON metadata
[CeskaTelevize]<PHONE_NUMBER>20001: Downloading JSON metadata
[CeskaTelevize]<PHONE_NUMBER>20001: Downloading JSON metadata
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "/home/user/.local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
ie_result = ie.extract(url)
File "/home/user/.local/bin/youtube-dl/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/home/user/.local/bin/youtube-dl/youtube_dl/extractor/ceskatelevize.py", line 201, in _real_extract
self._sort_formats(e['formats'])
File "/home/user/.local/bin/youtube-dl/youtube_dl/extractor/common.py", line 1327, in _sort_formats
raise ExtractorError('No video formats found')
ExtractorError: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Description

Neither
https://www.ceskatelevize.cz/ivysilani/12673563853-lidstvo-pribeh-nas-vsech/219382567420001
or
https://www.ceskatelevize.cz/porady/12673563853-lidstvo-pribeh-nas-vsech/219382567420001
are working.
It seems this affects only particular series, other on iVysilani are working well

OK, I found #23403 and it's exactly the same issue. Is there any way how to "emulate" enabling DRM like the browsers does?

No.


Update to Python 3.7+ Django 3.2+

Fix #29

I've updated my PR with the fewest changes to support Django 3.2+ and Python 3.7+
I will leave any other non-essential change proposals (eg: black, precommit, ...) after the merge of this PR to speed up the release of version 2.0 which is necessary to migrate the djangoproject.com site to Django > 3.2.

Hi Folks,
Thanks for the great work, and sorry for not being very active over here.
Would it be OK for me to transfer this repository to https://github.com/django? As well as the PyPI rights… I don't have a lot of interest in maintaining this project anymore, and it seems it's holding you back.
Let me know what you think, thanks!

Hi @brutasse
That is a great idea. Maybe @carltongibson / @felixxm can help with the ownership transfer.
If taking this project inside the Django organization isn't feasible, we're also happy to take care of it without the ownership transfer. We'll need the push access to this repo and PyPi.

Would it be OK for me to transfer this repository to https://github.com/django? As well as the PyPI rights… I don't have a lot of interest in maintaining this project anymore, and it seems it's holding you back.
Let me know what you think, thanks!

Hi @brutasse and thank you for your message.
I would suggest that you consider moving the package to https://github.com/jazzband , it's a switch that many packages that are used in the Django ecosystem have made.
If you think this might work for you here you will find a guide ready with the steps to do and if you want I'm available to help you
https://jazzband.co/about/guidelines
Could I still ask you to take a look at the open PR and if you deem it appropriate, merge it?
This would allow us to generate a new version on PyPI right away, for use in djangoproject.com, while simultaneously turning on the ownership transfer.

I agree with @pauloxnet, moving to @jazzband is a better idea.

Hi @brutasse (sorry for the ping) hope you're well 👋
How do you feel about @pauloxnet's suggestion of merging this PR and then moving this to @jazzband?
I guess will have to move the contributing section from the readme to a contributing file (with some jazzband lines on top) but the project seems very suitable so I think should be quite straight forward 🙂

Alright @pauloxnet merge away, this looks great to me! ⭐ I can work on a PR for the contributing guide today but also tell me if there's anything I can do to support 👍

@sarahboyce this PR is one year old, I'll review it today and eventually, I update it and then ask for a review from you.

@sarahboyce @CuriousLearner Since the pipelines on Travis no longer work I added the Github workflows to run tests in all versions.
You can see the results here:
https://github.com/pauloxnet/django-push/actions/runs/7051709347
I would ask you for further review


Adding container based filtering for get ports

Signed-off-by: Mohammed Zeeshan Ahmed<EMAIL_ADDRESS>What type of PR is this?
/kind bug
What does this PR do / why we need it:
This PR adds the ability to filter ports by container name (interface change) thereby fixing the case where more than 1 ports are spread across containers resulting in need to specify a port
Which issue(s) this PR fixes:
Fixes #4330
PR acceptance criteria:

[x] Unit test

[ ] Integration test

[ ] Documentation

[ ] Update changelog

[x] I have read the test guidelines

How to test changes / Special notes to the reviewer:
See linked issue for error case. As for checking how it is fixed
❯ ls
devfile.yaml  node-ex  package.json  sample  script-validator  server.js  test-repo
❯ cat devfile.yaml
schemaVersion: 2.0.0
metadata:
  name: nodejs
  version: 1.0.0
  attributes:
    alpha.build-context: src
    alpha.build-dockerfile: Dockerfile
components:
  - name: buildguidance
    container:
      image: buildguidanceimage-placeholder
      memoryLimit: 1024Mi
      endpoints:
        - name: http-3001
          targetPort: 3001
  - name: runtime
    container:
      image: registry.access.redhat.com/ubi8/nodejs-12:1-45
      memoryLimit: 1024Mi
      mountSources: true
      sourceMapping: /project
      endpoints:
        - name: http-3000
          targetPort: 3000
commands:
  - id: install
    exec:
      component: runtime
      commandLine: npm install
      workingDir: /project/src
      group:
        kind: build
        isDefault: true
  - id: run
    exec:
      component: runtime
      commandLine: npm start
      workingDir: /project/src
      group:
        kind: run
        isDefault: true
❯ odo url create --host com --container buildguidance --ingress
 ✓  URL nodejs-testbench-qjcp-3001 created for component: nodejs-testbench-qjcp

To apply the URL configuration changes, please use `odo push`
❯ odo url create --host com --container runtime --ingress
 ✓  URL nodejs-testbench-qjcp-3000 created for component: nodejs-testbench-qjcp

To apply the URL configuration changes, please use `odo push`

Unrelated failure in windows UT https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_odo/4732/pull-ci-openshift-odo-main-psi-unit-test-windows/1395286598446223360
/test psi-unit-test-windows

Again failed due to same issue. Not related to code of this pr
/retest

/lgtm

Recurrence of the same issue https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_odo/4732/pull-ci-openshift-odo-main-psi-kubernetes-integration-e2e/1395704113907372032
/retest

/approve


Set RTL minimal dash size to 2.0, add invalid dash size error to the draw_dashed_line.

Follow up to #73444

Adds a log error for p_dash ≤ 0 case.
Sets RichTextLabel min. dash size to 2 (it's calculated for line width, which can be 0 if it's not set by the font).

Thanks!


Eliminate cache requirement (resolves #586)

Resolves #586, continued from #642.

one cache state file that contains the state of the cache.
All tests passing with shared caching (mutable and immutable read global files), and with non-shared caching (previous Toil default)

Looking at the failure from the previous PR (pretending the super(wrongly named class) error didn't happen), I now remember why we can't completely do away with FileStore. Only Mesos and SingleMachine support shared caching because we haven't figured out how node-specific batch system cleanup would work on parasol and gridengine (and lsf)

Jenkins, test this please.

@hannes-ucsc this is ready! :)

@arkal, there are conflicts. They must be small because I didn't merge anything major.

Fixed. Tests running now

@hannes-ucsc this is ready! I added more tests too

Reviewing now.

Note to self: Made it to _testValidityOfCacheEvictTest

Continued in #682


PWG render has gray background

I noticed that when printing using a driverless queue, some documents have a gray background (it should be white). This is reproducable on several printers and happens consistently with this type of document. We use PostScript files as the input for CUPS. This is one of the problematic files:
test.zip
When I manually convert the file to PWG raster and look at the output in rasterview, I can see that the background is gray.
cupsfilter -m image/pwg-raster ./print_link_1.ps > test.pwg
stdout.txt
pwg.zip

When I convert the file to PDF using cupsfilter, the background looks fine in a PDF viewer. But when I convert that pdf to pwg, it is gray again.
test.pdf
When I try to convert the PostScript file to Apple raster using cupsfilter, rasterview cannot open it at all. cupfilter prints an error message.
cupsfilter -m image/pwg-raster ./print_link_1.ps > test.pwg
ERROR: Unsupported raster data.
urf.zip
stdout_urf.txt
The version of cups-filters used is 1.28.10.

I forgot to mention: The PostScript file is generated by the "Microsoft PS Class Driver" on Windows. When printed using a raw queue, the background is white as expected.

I tried to reproduce the problem on several other systems.

Raspberry Pi OS 11 with cups-filters 1.28.7 and ghostscript 9.53.3: not reproducable
Ubuntu 20.04 with cups-filters 1.27.4 and ghostscript 9.50: not reproducable (although the logo and table headers are "dotted" instead of properly filled)
EndeavourOS with cups-filters 1.28.15 ghostscript 9.56.1: not reproducable

It seems to be a problem specific to our Linux System which is based on BuildRoot 2022.02 and uses cups 2.4.0, cups-filters 1.28.10 and ghostscript 9.55.0. Possibly a missing dependency or a bad combination.

When invoking GhostScript directly using the same parameters as cupsfilter, the issue can be reproduced:
gs -dSAFER -dNOPAUSE -dBATCH -dNOINTERPOLATE -dNOMEDIAATTRS -sOutputFile=./gs.pdf -sDEVICE=pwgwrite -dDoNumCopies -dCompatibilityLevel=1.3 -dAutoRotatePages=/None -dAutoFilterColorImages=false -dNOPLATFONTS -dColorImageFilter=/FlateEncode -dPDFSETTINGS=/default -dColorConversionStrategy=/LeaveColorUnchanged -r600x600 -dDEVICEWIDTHPOINTS=612 -dDEVICEHEIGHTPOINTS=792 -I/usr/share/cups/fonts ./test.ps

gs -dSAFER -dNOPAUSE -dBATCH -dNOINTERPOLATE -dNOMEDIAATTRS -dShowAcroForm -sOutputFile=./gs.pwg -sDEVICE=cups -sMediaClass=PwgRaster -sOutputType=Automatic -r600x600 -dDEVICEWIDTHPOINTS=612 -dDEVICEHEIGHTPOINTS=792 -dcupsBitsPerColor=8 -dcupsColorOrder=0 -dcupsColorSpace=19 -dcupsBorderlessScalingFactor=0.0000 -dcupsInteger0=1 -dcupsInteger1=1 -dcupsInteger2=1 -scupsPageSizeName=na_letter_8.5x11in -I/usr/share/cups/fonts ./gs.pdf

Changing the dcupsColorSpace parameter to a grayscale one (3 or 18) produces an output with white background. Most of the other ones I tested produce the gray background.

Which Ghostscript version is that? I do not remember that there was ever one with a "pwgwrite" output device. I had actually called it "pwgraster" by the way.

According to the other command line options the first Ghostscript line should probably be the one to turn your PostScript input file into PDF and so creating the input file for the second line. The correct output device for that is "pdfwrite". Please copy and paste these lines and do not write them from the top of your head, so that we can also actually reproduce your problem.

The Ghostscript version is 9.55.0.
Sorry about the incorrect command. This was something I tried before but didnt work for obvious reasons. I edited my comment above to contain the commands I can reproduce the problem with.

I have now repeated the test with your corrected command lines, at least for Ghostscript 9.55.0 and I still get the correct, white background.

Not really, looks like a bug in a particular distribution's package. Please check on which distribution and which release of it you can reproduce this bug and report a bug to there. Please post the link to that report here.

I found a similar issue here but it looks like nobody found the root cause.
Is this likely an issue with Ghostscript only or does Ghostscript use something from CUPS to render the files? I am asking because some of the parameters sound very CUPS specific.

It is principally only Ghostscript, as you can reproduce it with only Ghostscript command lines. The CUPS-specific parameters are due to the fact that the rasterizing (second command line) is done with the CUPS output device.

Thank you for your help. I will link the bug report.

I figured out what the root cause is. GhostScript contains sources for a number of its dependencies in its own source code. When building GhostScript, BuildRoot deletes these sources which results in GhostScript using the libraries from the system instead. One of these libraries is Little-CMS. The current LTS release of BuildRoot contains Little-CMS 2.13 which has a known issue with gray backgrounds. It is fixed in version 2.13.1.
I did not open an issue at BuildRoot because they already updated to the fixed version of Little-CMS. The fix has not been included in the latest LTS release 2022.02.3 but should land in the next one.


Please put on NPM

Searching npmjs.com does not return your plugin.

https://www.npmjs.com/package/de.appplant.cordova.plugin.printer

Its coming soon as I am working on a new release. With that the new id will be cordoba-plugin-printer

https://www.npmjs.com/package/cordova-plugin-printer


Set up automation to deploy staging server

Here's steps to deploy staging:

Make a pull request to dev branch
Merge the PR into dev branch
ssh into staging server
git pull on dev branch
docker build  to build all the images you need
docker compose up to start vim

Goals:

Automate step 3 to step 6
Be able to partially reuse the pipeline when deploying production
Can reuse the pipeline on other projects (e.g. rodan and linked music)

Great idea!
On Oct 13, 2023, at 10:35 AM, softcat477 @.***> wrote:
Here's steps to deploy staging:

Make a pull request to dev branch
Merge the PR into dev branch
ssh into staging server
git pull on dev branch
docker build to build all the images you need
docker compose up to start vim

Goals:

Automate step 3 to step 6
Be able to partially reuse the pipeline when deploying production
Can reuse the pipeline on other projects (e.g. rodan and linked music)

—
Reply to this email directly, view it on GitHubhttps://github.com/DDMAL/VIM/issues/71, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAH342JLU77BPYSAEPGXKETX7FGTZAVCNFSM6AAAAAA57IC5N6VHI2DSMVQWIX3LMV43ASLTON2WKOZRHE2DEMBYGMZTKOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Permission denied when reading msg on Windows.

In the CI https://github.com/iterative/dvc/actions/runs/3310681625/jobs/5465204030
Permission denied: 'C:\\Users\\runneradmin\\AppData\\Local\\Temp\\pytest-of-runneradmin\\pytest-0\\popen-gw0\\test_queue_remove_done_2_0\\.dvc\\tmp\\exps\\run\\85b382c77c5350b90c992924313ad1745fbcef17\\85b382c77c5350b90c992924313ad1745fbcef17.json'

That means the file is still opened in another thread or another process. It usually means the file is not being closed properly, or we are not waiting for some other task to completely finish (and we try to open+read the file before the other task has closed it)

We probably need to lock the processinfo files in the dvc-task process manager when reading/writing to avoid potential simultaneous reads on windows

Our old naive solution in #53 greatly reduced the possibility of this kind of case, but can not prevent it. Time to solve it.


Fix twoslash generics

Fixes #452

Still broken in the v2 site. How long does a commit take to be deployed?

Production deploys are manual - staging isn't though: http://www.staging-typescript.org/docs/handbook/basic-types.html#type-assertions


No smoke sensor device, and unable to see any docs if its off by default.

First, thank you for a fantastic plugin!
I do not see a smoke created by default - should this be the case or is there just not documentation yet?

Should be created, unless you set hidden for the host in config.json.  Are you running v1.1.11 and did you restart Homebridge?  If you want me to have a look, please attach a debug log file, see https://github.com/ebaauw/homebridge-rpi#debug-log-file.

Not much there.... Im not sure debugging is working properly:
root@rpi-netmgr:/home/pi# cat homebridge.log
[7/24/2020, 14:42:33] config.json (/root/.homebridge/config.json) not found.
[7/24/2020, 14:42:33] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:33] Registering accessory 'homebridge-chamberlain.Chamberlain'
[7/24/2020, 14:42:33] ---
[7/24/2020, 14:42:33] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:33] Registering platform 'homebridge-config-ui-x.config'
[7/24/2020, 14:42:33] ---
[7/24/2020, 14:42:34] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:34] Registering platform 'homebridge-http-webhooks.HttpWebHooks'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookSensor'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookSwitch'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookPushButton'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookLight'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookThermostat'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookOutlet'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookSecurity'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookGarageDoorOpener'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookStatelessSwitch'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookLockMechanism'
[7/24/2020, 14:42:34] Registering accessory 'homebridge-http-webhooks.HttpWebHookWindowCovering'
[7/24/2020, 14:42:34] ---
[7/24/2020, 14:42:34] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:34] Registering platform 'homebridge-nest.Nest'
[7/24/2020, 14:42:34] ---
[7/24/2020, 14:42:35] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:35] Registering platform 'homebridge-ring.Ring'
[7/24/2020, 14:42:35] ---
[7/24/2020, 14:42:35] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:35] Registering platform 'homebridge-rpi.Lib'
[7/24/2020, 14:42:35] Registering platform 'homebridge-rpi.RPi'
[7/24/2020, 14:42:35] ---
[7/24/2020, 14:42:35] Loaded plugin<EMAIL_ADDRESS>[7/24/2020, 14:42:35] Registering platform<EMAIL_ADDRESS>[7/24/2020, 14:42:35] ---
Setup Payload:
X-HM://0023ISYWYO8AE
Scan this code with your HomeKit app on your iOS device to pair with Homebridge:
Or enter this code with your HomeKit app on your iOS device to pair with Homebridge:
[7/24/2020, 14:42:35] Homebridge is running on port 40943.

Please attach a full debug log file, see https://github.com/ebaauw/homebridge-rpi#debug-log-file.  I need the output of homebridge -D.

Here. as above.
homebridge.log.zip

Indeed, the same.  Note that homebridge didn't pick up your config.json (sorry, I missed that earlier), you probably need to specify -U /var/lib/homebridge next to -D (assuming you installed using hb-service).

Much better :-)
homebridge.log.zip

`[7/24/2020, 16:19:48] [RPi] warning: heartbeat 12, drift 259
[7/24/2020, 16:19:51] [RPi] rpi-netman: exec vcgencmd measure_temp
[7/24/2020, 16:19:51] [RPi] rpi-netman: exec vcgencmd measure_clock arm
[7/24/2020, 16:19:51] [RPi] rpi-netman: exec vcgencmd measure_volts
[7/24/2020, 16:19:51] [RPi] rpi-netman: exec vcgencmd get_throttled
[7/24/2020, 16:19:51] [RPi] rpi-netman: command HWVER (17) 0 0
[7/24/2020, 16:19:51] [RPi] rpi-netman: command HWVER (17) => 11546897
[7/24/2020, 16:19:51] [RPi] rpi-netman: state: {"date":"2020-07-24T15:19:51.000Z","boot":"2020-07-23T16:41:48.000Z","load":0.12,"temp":46,"freq":1000265600,"volt":0.8375,"throttled":0}
[7/24/2020, 16:19:51] [RPi] rpi-netman: set Current Temperature from 43°C to 46°C
[7/24/2020, 16:19:51] [RPi] rpi-netman: set Frequency from 750MHz to 1000MHz
[7/24/2020, 16:19:51] [RPi] rpi-netman: set Load from 0 to 0.12
[7/24/2020, 16:19:51] [RPi] rpi-netman: set Last Updated from "Fri Jul 24 2020 16:19:09" to "Fri Jul 24 2020 16:19:51"
[7/24/2020, 16:19:51] [RPi] rpi-netman: set Last Boot from "Thu Jul 23 2020 17:41:49" to "Thu Jul 23 2020 17:41:48"
[7/24/2020, 16:20:06] [RPi] flush cachedAccessories
[7/24/2020, 16:20:06] [RPi] rpi-netman: exec vcgencmd measure_temp
[7/24/2020, 16:20:06] [RPi] rpi-netman: exec vcgencmd measure_clock arm
[7/24/2020, 16:20:06] [RPi] rpi-netman: exec vcgencmd measure_volts
[7/24/2020, 16:20:06] [RPi] rpi-netman: exec vcgencmd get_throttled
[7/24/2020, 16:20:06] [RPi] rpi-netman: command HWVER (17) 0 0
[7/24/2020, 16:20:06] [RPi] rpi-netman: command HWVER (17) => 11546897
[7/24/2020, 16:20:06] [RPi] rpi-netman: state: {"date":"2020-07-24T15:20:07.000Z","boot":"2020-07-23T16:41:49.000Z","load":0.17,"temp":45,"freq":1500398464,"volt":0.8375,"throttled":0}
[7/24/2020, 16:20:06] [RPi] rpi-netman: set Current Temperature from 46°C to 45°C
[7/24/2020, 16:20:06] [RPi] rpi-netman: set Frequency from 1000MHz to 1500MHz
[7/24/2020, 16:20:06] [RPi] rpi-netman: set Load from 0.12 to 0.17
[7/24/2020, 16:20:06] [RPi] rpi-netman: set Last Updated from "Fri Jul 24 2020 16:19:51" to "Fri Jul 24 2020 16:20:07"
[7/24/2020, 16:20:06] [RPi] rpi-netman: set Last Boot from "Thu Jul 23 2020 17:41:48" to "Thu Jul 23 2020 17:41:49"
[7/24/2020, 16:20:16] [Ring] Jeremy Detected Motion
[7/24/2020, 16:20:21] [RPi] rpi-netman: exec vcgencmd measure_temp
[7/24/2020, 16:20:21] [RPi] rpi-netman: exec vcgencmd measure_clock arm
[7/24/2020, 16:20:21] [RPi] rpi-netman: exec vcgencmd measure_volts
[7/24/2020, 16:20:21] [RPi] rpi-netman: exec vcgencmd get_throttled
[7/24/2020, 16:20:21] [RPi] rpi-netman: command HWVER (17) 0 0
[7/24/2020, 16:20:21] [RPi] rpi-netman: command HWVER (17) => 11546897
[7/24/2020, 16:20:21] [RPi] rpi-netman: state: {"date":"2020-07-24T15:20:21.000Z","boot":"2020-07-23T16:41:48.000Z","load":0.13,"temp":46,"freq":1000265600,"volt":0.8375,"throttled":0}
[7/24/2020, 16:20:21] [RPi] rpi-netman: set Current Temperature from 45°C to 46°C
[7/24/2020, 16:20:21] [RPi] rpi-netman: set Frequency from 1500MHz to 1000MHz
[7/24/2020, 16:20:21] [RPi] rpi-netman: set Load from 0.17 to 0.13
[7/24/2020, 16:20:21] [RPi] rpi-netman: set Last Updated from "Fri Jul 24 2020 16:20:07" to "Fri Jul 24 2020 16:20:21"
[7/24/2020, 16:20:21] [RPi] rpi-netman: set Last Boot from "Thu Jul 23 2020 17:41:49" to "Thu Jul 23 2020 17:41:48"
[7/24/2020, 16:20:35] [PingHosts] [Router] success for <IP_ADDRESS> with session 245
[7/24/2020, 16:20:35] [PingHosts] [Keyfob Module] success for <IP_ADDRESS> with session 304
[7/24/2020, 16:20:36] [RPi] rpi-netman: exec vcgencmd measure_temp
[7/24/2020, 16:20:36] [RPi] rpi-netman: exec vcgencmd measure_clock arm
[7/24/2020, 16:20:36] [RPi] rpi-netman: exec vcgencmd measure_volts
[7/24/2020, 16:20:36] [RPi] rpi-netman: exec vcgencmd get_throttled
[7/24/2020, 16:20:36] [RPi] rpi-netman: command HWVER (17) 0 0
[7/24/2020, 16:20:36] [RPi] rpi-netman: command HWVER (17) => 11546897
[7/24/2020, 16:20:36] [RPi] rpi-netman: state: {"date":"2020-07-24T15:20:36.000Z","boot":"2020-07-23T16:41:48.000Z","load":0.1,"temp":45,"freq":750146496,"volt":0.8375,"throttled":0}
[7/24/2020, 16:20:36] [RPi] rpi-netman: set Current Temperature from 46°C to 45°C
[7/24/2020, 16:20:36] [RPi] rpi-netman: set Frequency from 1000MHz to 750MHz
[7/24/2020, 16:20:36] [RPi] rpi-netman: set Load from 0.13 to 0.1
[7/24/2020, 16:20:36] [RPi] rpi-netman: set Last Updated from "Fri Jul 24 2020 16:20:21" to "Fri Jul 24 2020 16:20:36"
`

I see no weird things in log, but also no evidence whether the smoke sensor is exposed.  Could you please attach cachedAccessories?

What HomeKit app are you using?  By default, Home combines the Smoke Sensor and Temperature Sensor in a single tile.  Too change that, select Show as Separate Tiles in the settings.  In Eve, you just see an additional Smoke characteristic in the Pi accessory.

Apples own app. Let me go dig for the temp one and maybe its now compound

Right you are! I expected a new, separate smoke sensor.


THRIFT-5666: lib: cpp: prefer poll.h over sys/poll.h

The <sys/poll.h> header is not actually where poll(2) must be defined according to the spec. However, it may be the case that some random UNIX-like OS uses <sys/poll.h> instead.
Both use cases can be supported simply by including the proper header, if it exists.
Tested with both the autotools.sh and cmake.sh build strategies, and is fine.

[x] Did you create an Apache Jira ticket?  (not required for trivial changes)
[x] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?

@Jens-G - can you please review? I can rebase if necessary, but otherwise, this is a relatively trivial change (poll() is declared in <poll.h> not <sys/poll.h>

Actually, I'll respin this to just add #ifdef HAVE_POLL_H in a few places.

@emmenlau - are you able to review? It's pretty trivial.


Add support for functions with multiple arguments

Sorry guys for the delay. I had a lot going on at work lately. Thank you @blakeohare for the pull request. I will package this one as soon as I have more time.


SAK-33295: syllabus item title requirement/validation not visible to user

https://jira.sakaiproject.org/browse/SAK-33295
The title of a Syllabus Item is required, however has the following bugs:

no required field indicator (asterisk)
validation is not presented to the user

Steps to reproduce:

create a Syllabus Item
provide some content in the CKEditor, but do not specific a title
click either of the save options
notice the modal is gone, nothing was created, and no error/warning was presented to the user

There is actually validation and a validation message in place, but immediately after presenting the validation message, the JavaScript code closes the modal. The result is that the user has no opportunity to see the error, no opportunity to rectify the error, and loses all the data they may have entered in the CKEditor.
The solution to this is to:

mark the field as required in the UI (asterisk)
don't close the modal if the validation fails so the user can actually see the validation error and correct the issue themselves

Travils failure unrelated (site-stats tests fail)


Bitmap function improperly displaying when tft.rotation(1) or (3)

When using the image_bitmap.py example, The logo.py bitmap displays improperly, it seems to be read in incorrect order, leading to it displaying "split" diagonally, as if someone cut the image from (x1,y0) to (x0,y1). I added some gradient lines to help me understand the coordinate system:
import tft_config
import logo
import gc
import time

def main():
    tft = tft_config.config()

    tft.reset()
    tft.init()
    tft.rotation(1)
    tft.bitmap(0, 0, logo.WIDTH, logo.HEIGHT, logo.BITMAP)

#for testing coordinates:
    # Red to yellow gradient on x axis, starting at 0,0
    for i in range(0,240,2):
        color = tft.colorRGB(255, i, 0)
        tft.pixel(i, 0, color)

    # Green to cyan gradient on y axis, starting at 0,0
    for i in range(0,240,2):
        color = tft.colorRGB(0, 255, i)
        tft.pixel(0, i, color)
    time.sleep(0.1)
    del tft

gc.enable()
main()
gc.collect()

That results in:

I then decided to test tft.rotation(0), using an image that should fit properly, a 240x240 picture of a cat. This was also to rule out the original logo.py being improperly formatted.

That worked okay, but when trying  tft.rotation(1)  with that same image, the image doesn't appear to be read/displayed entirely:

I was a little confused, so I tried adding an offset to the start/end points in different permutations until something worked, the best I got was with tft.bitmap(0, 0, logo.WIDTH, logo.HEIGHT-1, logo.BITMAP), which resulted in a "split" image similar to before with logo.py:

Not sure how I should proceed here. I can probably get around it for now by purposely adding the split in my original images to negate the split when displayed, but having to subtract from y1 still isn't making sense to me. Otherwise thanks for the driver!

Hi,
Thanks for pointing out the mistake in the example. I have fixed it, and now it should be working properly.
The reason for this issue is due to the fact, of how panel controllers work. Typically, you set a valid area first, then you fill all the content as linear data (a linear byte array) to this area, line by line. By this nature, if you have set an invalid area, the picture is going to shift.
For example: You have a picture of a size 8x8.
# And this is how the data will be sent to the display.
b'\x00\x00\x00\x00\x00\x00\x00\x00\x11\x11\x11\x11\x11\x11\x11\x11\x00\x00\x00\x00\x00\x00\x00\x00\x11\x11\x11\x11\x11\x11\x11\x11\x00\x00\x00\x00\x00\x00\x00\x00\x11\x11\x11\x11\x11\x11\x11\x11\x00\x00\x00\x00\x00\x00\x00\x00\x11\x11\x11\x11\x11\x11\x11\x11'

In case you have set the coordinate wrong, e.g. (0,1,8,8). Which is a 9x8 area, (8 - 0 + 1) x (8 - 1 + 1). What actually will be displayed is as the follows, say 0x00=*, 0x11=#.
********#
#######**
******###
#####****
****#####
###******
**#######
#********
# and a blank line

So the picture is not what you wanted, which would be:
********
########
********
########
********
########
********
########

So basically, you have to calculate the correct value of the start point (x0, y0) to the end point (x1,y1). Note that, there are (x1 - x0 + 1) x (y1 - y0 + 1) pixels in total.
Hopefully this helps.
Best regards.

Thank you, I'll give this a shot. This also confirms that my img_to_bitmap.py script is indeed working properly, would that be something you'd be interesting having in the repository? I can make a PR for it.

If you write the code on your own, or you know the source of the code, you are free to make a PR. Otherwise will be difficult for me to get the license righ.

It's my own, I'll clean it up and PR!


Support for taking screenshots on remote systems to Capture Screenshot keyword

Originally submitted to Google Code by Andreas.EbbertKarroum on 4 Feb 2010

In order to use the screenshot as String feature from Selenium, which
allows for taking screenshots on a remote system, I added this to
init.py. I'm sure it can be done better, but I hope you get the idea :)
One of the problems is that the String that holds the binary image, is
encoded in the default encoding of the platform on which the screenshot is
taken. Since this can vary, we have to provide a parameter for that:
def _absnorm(self, path):
    return os.path.normpath(os.path.abspath(path.replace('/', os.sep)))

def _write_to_file(self, path, content, encoding, mode):
    path = self._absnorm(path)
    parent = os.path.dirname(path)
    if not os.path.exists(parent):
        os.makedirs(parent)
    f = open(path, mode+'b')
    f.write(content.encode(encoding))
    f.close()
    return path

def capture_remote_screenshot(self, codec, path=None):
    """Captures a screenshot and returns it as a string

    Given path must be relative to Robot Framework output directory,
    otherwise the embedded image is not shown in the log file. If path is
    not given, file with name similar to 'selenium-image-x.png' is created
    directly under the output directory.
    """

    # configure path
    if path and os.path.isabs(path):
        raise RuntimeError("Given path must be relative to Robot outpudir")
    if not path:
        path = self._namegen.next()
    outdir = NAMESPACES.current.variables['${outputdir}']
    fullpath = os.path.join(outdir, path)
    if not os.path.exists(os.path.split(fullpath)[0]):
        os.makedirs(os.path.split(fullpath)[0])

    # retrieve remote screenshot
    self._info("Retrieving Screenshot")
    screenshot =

self._selenium.capture_entire_page_screenshot_to_string("background=#CCFFDD")
screenshot=base64.b64decode(screenshot)
try:
screenshot = screenshot.decode(codec)
except UnicodeDecodeError:
raise RuntimeError("Got error when decoding Screenshot-String
with '%s'" % codec)
    # save screenshot
    self._info("Saving screenshot to file '%s'" % fullpath)
    self._write_to_file(fullpath, screenshot, codec, 'w')
    self._html('

'
               '
' % (path,

path))

Originally submitted to Google Code by Andreas.EbbertKarroum on 10 Feb 2010

I improved the en-/decoding bits. would be nice, if this is included in the upcoming
release:
def _absnorm(self, path):
    return os.path.normpath(os.path.abspath(path.replace('/', os.sep)))

def _write_to_file(self, path, content, mode):
    path = self._absnorm(path)
    parent = os.path.dirname(path)
    if not os.path.exists(parent):
        os.makedirs(parent)
    f = open(path, mode+'b')
    f.write(content)
    f.close()
    return path

def capture_remote_screenshot(self, path=None):
    """Captures a screenshot and returns it as a string

    Given path must be relative to Robot Framework output directory,
    otherwise the embedded image is not shown in the log file. If path is
    not given, file with name similar to 'selenium-image-x.png' is created
    directly under the output directory.
    """

    # configure path
    if path and os.path.isabs(path):
        raise RuntimeError("Given path must be relative to Robot outpudir")
    if not path:
        path = self._namegen.next()
    outdir = NAMESPACES.current.variables['${outputdir}']
    fullpath = os.path.join(outdir, path)
    if not os.path.exists(os.path.split(fullpath)[0]):
        os.makedirs(os.path.split(fullpath)[0])

    # retrieve remote screenshot
    self._info("Retrieving Screenshot")
    screenshot =

self._selenium.capture_entire_page_screenshot_to_string("background=#CCFFDD")
screenshot=base64.b64decode(screenshot)
    # save screenshot
    self._info("Saving screenshot to file '%s'" % fullpath)
    self._write_to_file(fullpath, screenshot, 'w')
    self._html('

'
               '
' % (path, path))

Originally submitted to Google Code by @pekkaklarck on 10 Feb 2010

Do I get it right that the existing Capture Screenshot keyword cannot be used in
this case because it saves the image to the host where Selenium server is running?
And it's not a good idea to use capture_entire_page_screenshot_to_string with the
local screenshots because that approach doesn't work fully with all browsers?
Instead of adding a new keyword, could we enhance Capture Screenshot so that it
would use capture_entire_page_screenshot_to_string method when not connected to a
server on localhost? Could you Andreas try to implement a patch using this approach?
Notice that I just refactored the capture_screenshot method so I recommend you to
update your svn checkout. I also added some tests for the existing functionality and
you can run them e.g. with command atest/run_tests.py python ff --suite screenshots.

Originally submitted to Google Code by @yanne on 11 Feb 2010

Andreas, the latter patch omits the codec argument and decoding with that.
Is that a mistake or was it not needed in the original patch?

Originally submitted to Google Code by @yanne on 11 Feb 2010

I modified the current Capture Screenshot so that if the server host is anything else
than 'localhost', capture_entire_page_screenshot_to_string method is used to create
the screenshot. I also made some tests which start the server to <IP_ADDRESS> and they
seem to pass and screenshots are created.
I cannot easily test this with a real remote, can you Andreas do that?
Just run the latest trunk from svn.

Originally submitted to Google Code by @pekkaklarck on 11 Feb 2010

I did some further tuning in r364.

Originally submitted to Google Code by @pekkaklarck on 12 Feb 2010

We noticed that capture_screenshot_to_string works with all browsers regardless was
the server running on local or remote machine and the latest version of the Capture Screenshot uses that. The acceptance tests for this functionality pass and
documentation has been updated so I'll close this issue. Andread, could you still
please test the latest version in your system with a real remote server?
We also decided to implement a separate Capture Page Screenshot keyword ( issue 93 )
that uses capture_entire_page_screenshot_to_string.

Originally submitted to Google Code by Andreas.EbbertKarroum on 12 Feb 2010

Hi,
from a review of the sources, it looks good. Nice refactoring! One hint to the
"Capture Screenshot": yes it works with all browsers, because it does not need the
browser to take the screenshot. The selenium server is using java to access the
screen content. The concrete problem with this approach is: we are running the
selenium server on windows, and as soon as you log out (but leave the process
running), you will get screenshots that are all black. (see my blog post about that)
the capture entire page also works with IE, but only if you use "*ieproxy" as browser
profile and run the server in -singleWindow. Maybe you want to include that in the
documentation for the keyword.
I'll now checkout the code and give it a test!

Originally submitted to Google Code by @pekkaklarck on 12 Feb 2010

Do you Andreas have a concrete proposal on how to enhance the documentation of these
Capture Screenshot keywords so that they would include the information about black
screenshots when logged out on Windows and the need to use *ieproxy and
-singleWindow? Do you know is this stuff documented somewhere other than in the
source so that we could add a like to the kw doc too?

Originally submitted to Google Code by Andreas.EbbertKarroum on 12 Feb 2010

Hi,
and here are the results of the german jury ;)

Capture Screenshot & localhost & *iexploreproxy => OK

Capture Page Screenshot & localhost & *iexploreproxy/-singleWindow => OK, but
Why do I have to provide a filename? Why not:
def capture_page_screenshot(self, filename=None, css='background=#CCFFDD'):
I would have expected to have the file written to the default directory with
default, as with Capture Screenshot

Capture Screenshot & localhost & ff => OK

Capture Page Screenshot & localhost & ff => OK

Capture Screenshot & <IP_ADDRESS> & *iexploreproxy => not tested

Capture Page Screenshot & <IP_ADDRESS> & *iexploreproxy => not tested

Capture Screenshot & remote & ff => NOK

Capture Page Screenshot & remote & ff => NOK
There is a file created, but it's not a valid png, because the file is not opened in
binary mode: open(path, 'wb').

7 & attached patch applied => Expected NOK: screenshot is all black
8 & attached patch applied => OK

Originally submitted to Google Code by @pekkaklarck on 12 Feb 2010

Thanks a lot for testing Andreas! I fixed 2), 7), and 8) in r412. Could you still
test this version to verify how everything works?
Notice also that there's nowadays a separate screenshot.py module. The code was
accidentally added back to init.py in r408 but I re-removed it in r411.

Originally submitted to Google Code by @pekkaklarck on 15 Feb 2010

I enhanced the documentation in r420. The link to the latest version is below and
comments are appreciated:
http://robotframework-seleniumlibrary.googlecode.com/svn/trunk/doc/SeleniumLibrary.html#Capture%20Screenshot

Originally submitted to Google Code by @pekkaklarck on 15 Feb 2010

After the latest fixes and doc enhancements this ought to be done.


Enable strictFunctionTypes in tsconfig

My project uses strictFunctionTypes and I see a lot of type errors from ng1-shift due to this option

I think would be better if we enable full strict mode in tsconfig via strict: true


D3-svg-legend is causing issues with the new sidebar behavior, remove and rewrite

Closed via #460


turn platform ios build setting Require only App-extension safe to YES

I will receive a warning when build with Today Extension
linking against a dylib which is not safe for use in application extensions: /Users/mrche/Documents/teambition/Teambition-Universal/Carthage/Build/iOS/SwiftDate.framework/SwiftDate

@Hout   I'm sorry to ask that is this pull request possible to merge?  because our project use this as a framework by Carthage , and It's not good to see a warning every day.

Thank you. Can you post the same request on develop please?


Does not pass weight or environment from Javascript Buttons

When used with the PayPal Javascript buttons (https://github.com/paypal/JavaScriptButtons) the cart does not pass on the data-weight and data-host attributes. You can see this by looking at the hidden inputs in the cart popup. Price, shipping, weight units, merchant, and number all go through, but weight and host (eg. Sandbox) do not.
Other attributes may not work,  but these are the ones that I've found.

Here is an implementation of PayPal Minicart were shipping costs are based on item weight. Shipping calculation happens instantly and is shown as an item in the main Cart.
Any feedback is welcome.
http://htmlpreview.github.io/?https://github.com/fabriziotappero/minicart-plus/blob/master/minicart.html


Vagrant installation is not working

vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'flynn-base' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
    default: Adapter 2: hostonly
==> default: Forwarding ports...
    default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
    default: SSH address: <IP_ADDRESS>:2222
    default: SSH username: vagrant
    default: SSH auth method: private key
    default: Warning: Remote connection disconnect. Retrying...
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
    default: The guest additions on this VM do not match the installed version of
    default: VirtualBox! In most cases this is fine, but in rare cases it can
    default: prevent things such as shared folders from working properly. If you see
    default: shared folder errors, please make sure the guest additions within the
    default: virtual machine match the version of VirtualBox you have installed on
    default: your host and reload your VM.
    default:
    default: Guest Additions Version: 5.1.8
    default: VirtualBox Version: 5.0
==> default: Configuring and enabling network interfaces...
The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

/sbin/ifdown eth1 2> /dev/null

Stdout from the command:

Stderr from the command:

mesg: ttyname failed: Inappropriate ioctl for device

prime@SpArK:~/flynn/demo$ flynn -a dashboard env get LOGIN_TOKEN
13:27:10.346865 main.go:173: no clusters configured

This looks like a problem with your Virtualbox installation. Are you successfully able to use other Vagrant environments at this time?

@vaibhav-jain what version of Vagrant are you using? The Flynn demo box is Ubuntu 16.04 which uses network device names like ensXX, so vagrant running ifdown eth1 looks suspicious.

Vagrant 1.8.1 VIrtual box  5.0.24_Ubuntu r108355

After upgrading vagrant to 1.9.1
vagrant up
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Checking if box 'flynn-base' is up to date...
==> default: Clearing any previously set forwarded ports...
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
    default: Adapter 2: hostonly
==> default: Forwarding ports...
    default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
==> default: Waiting for machine to boot. This may take a few minutes...
    default: SSH address: <IP_ADDRESS>:2222
    default: SSH username: vagrant
    default: SSH auth method: private key
==> default: Machine booted and ready!
==> default: Checking for guest additions in VM...
    default: The guest additions on this VM do not match the installed version of
    default: VirtualBox! In most cases this is fine, but in rare cases it can
    default: prevent things such as shared folders from working properly. If you see
    default: shared folder errors, please make sure the guest additions within the
    default: virtual machine match the version of VirtualBox you have installed on
    default: your host and reload your VM.
    default:
    default: Guest Additions Version: 5.1.8
    default: VirtualBox Version: 5.0
==> default: Configuring and enabling network interfaces...
==> default: Machine already provisioned. Run `vagrant provision` or use the `--provision`
==> default: flag to force provisioning. Provisioners marked to run always will still run.
prime@SpArK:~/flynn/demo$ vagrant provision
==> default: Running provisioner: shell...
    default: Running: inline script
==> default: 11:37:09.008990 check online-hosts
==> default: 11:39:09.065980 check online-hosts error: timed out waiting for 1 hosts to come online (currently 0 online)
==> default:
==> default: The following hosts were discovered but remained unreachable:
==> default:
==> default: http://<IP_ADDRESS>:1113
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

I suggest running vagrant destroy followed by vagrant up this will re-create the Vagrant environment cleanly.

Now getting
L=/usr/local/bin/flynn && curl -sSL -A "`uname -sp`" https://dl.flynn.io/cli | zcat >$L && chmod +x $L
bash: /usr/local/bin/flynn: Permission denied
curl: (23) Failed writing body (0 != 12888)

Finally got it working phew . . . .


[FLINK-31963][state] Fix rescaling bug in recovery from unaligned checkpoints.

Backport of #22584 to 1.17
This commit fixes problems in StateAssignmentOperation for unaligned checkpoints with stateless operators that have upstream operators with output partition state or downstream operators with input channel state.
(cherry picked from commit 354c0f455b92c083299d8028f161f0dd113ab614)

What is the purpose of the change
(For example: This pull request makes task deployment go through the blob server, rather than through RPC. That way we avoid re-transferring them on each deployment (during recovery).)
Brief change log
(for example:)

The TaskInfo is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
TaskManagers retrieve the TaskInfo from the blob cache

Verifying this change
Please make sure both new and modified tests in this PR follows the conventions defined in our code quality guide: https://flink.apache.org/contributing/code-style-and-quality-common.html#testing
(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (100MB)
Extended integration test for recovery after master (JobManager) failure
Added test that validates that TaskInfo is transferred only once across recoveries
Manually verified the change by running a 4 node cluster with 2 JobManagers and 4 TaskManagers, a stateful streaming program, and killing one JobManager and two TaskManagers during the execution, verifying that recovery happens correctly.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure


Dataset viewer issues

I was looking through the dataset viewer and I like it a lot. Version numbers, citation information, everything's there! I've spotted a few issues/bugs though:

the URL is still under nlp, perhaps an alias for datasets can be made
when I remove a feature (and the feature list is empty), I get an error. This is probably expected, but perhaps a better error message can be shown to the user

IndexError: list index out of range
Traceback:
File "/home/sasha/streamlit/lib/streamlit/ScriptRunner.py", line 322, in _run_script
    exec(code, module.__dict__)
File "/home/sasha/nlp-viewer/run.py", line 316, in <module>
    st.table(style)
File "/home/sasha/streamlit/lib/streamlit/DeltaGenerator.py", line 122, in wrapped_method
    return dg._enqueue_new_element_delta(marshall_element, delta_type, last_index)
File "/home/sasha/streamlit/lib/streamlit/DeltaGenerator.py", line 367, in _enqueue_new_element_delta
    rv = marshall_element(msg.delta.new_element)
File "/home/sasha/streamlit/lib/streamlit/DeltaGenerator.py", line 120, in marshall_element
    return method(dg, element, *args, **kwargs)
File "/home/sasha/streamlit/lib/streamlit/DeltaGenerator.py", line 2944, in table
    data_frame_proto.marshall_data_frame(data, element.table)
File "/home/sasha/streamlit/lib/streamlit/elements/data_frame_proto.py", line 54, in marshall_data_frame
    _marshall_styles(proto_df.style, df, styler)
File "/home/sasha/streamlit/lib/streamlit/elements/data_frame_proto.py", line 73, in _marshall_styles
    translated_style = styler._translate()
File "/home/sasha/.local/share/virtualenvs/lib-ogGKnCK_/lib/python3.7/site-packages/pandas/io/formats/style.py", line 351, in _translate
    * (len(clabels[0]) - len(hidden_columns))

there seems to be an encoding issue in the default view, the dataset examples are shown as raw monospace text, without a decent encoding. That makes it hard to read for languages that use a lot of special characters. Take for instance the cs-en WMT19 set. This problem goes away when you enable "List view", because then some syntax highlighteris used, and the special characters are coded correctly.

Thanks for reporting !
cc @srush for the empty feature list issue and the encoding issue
cc @julien-c maybe we can update the url and just have a redirection from the old url to the new one ?

Ok, I redirected on our side to a new url. ⚠ @srush: if you update the Streamlit config too to /datasets/viewer, let me know because I'll need to change our nginx config at the same time

9


Update user service

Update user service to create project if it does not exist, when adding a new user

Depends on #71

What was the need for this?
If a project does not exist, then it must be first created (this case will be minimally reached since there are only a few number of projects).
Ideally the workflow for the client should be, try to find a project -> create one if 404 -> try to find user within project -> create one if 404 -> schedule the notifications using the project and the user.
UserService should not be in-charge of making modifications to the projects repo unless absolutely necessary.

Sorry for the confusion, yes you're right. I thought it should be done in the server. I've added this in the client now. https://github.com/RADAR-base/RADAR-Questionnaire/pull/872/files
Closing this.