Setting user-agent via page.setUserAgent along with page.setRequestInterception does not correctly change the user-agent.

Steps to reproduce
Tell us about your environment:

Puppeteer version: 1.9.0
Platform / OS version: Ubuntu 16.04
URLs (if applicable):
Node.js version: 11.0.0

What steps will reproduce the problem?
Setting user-agent via page.setUserAgent, then allow request interception via page.setRequestInterception(true) and surcharge headers with whatever you want.
Please include code that reproduces the issue.
var express = require('express');
const puppeteer = require('puppeteer');

(async () => {
    //Set small express server to print headers
    new Promise((resolve, reject) => {
        let port = 3001;

        let app = express();
        app.use(express.json());
        app.listen(port);
        app.route('/test').get((req, res) => {
            console.log(JSON.stringify(req.headers));
        });
    });

    //Create browser
    let puppeteerArgs = [
        '--no-sandbox',
        '--disable-setuid-sandbox',
        '--enable-features=NetworkService'
    ];
    let browser = await puppeteer.launch({
        headless: true,
        args: puppeteerArgs
    });

    //Create page and set user-agent and interception
    var page = await browser.newPage();

    await page.setUserAgent('newUserAgent');
    await page.setRequestInterception(true);
    page.on('request', request => {
        request.continue({headers: {}});
    });

    //Call the api
    await page.goto('http://localhost:3001/test');

})();

What is the expected result?
Printed headers object should contain the user-agent set by page.setUserAgent.
What happens instead?
Print headers object contains the puppeteer user-agent.
So we have to explicitely change it in the returned header of request.continue().
This should either be changed or written in the doc.

So we have to explicitely change it in the returned header of request.continue().

@rqueraud So what are you trying to achieve when doing request.continue({headers: {}});?
P.S. I expected a different pattern to be used with header overrides - sent a PR with an example for the documentation.

@aslushnikov  With that example, I was showing that not overriding any fields in the headers still resulted to the userAgent header to be reinitialized as if no page.setUserAgent() were used.
But I imagine that it was not working because i wasn't using the correct pattern. Following the new example you provided in the doc made it work. Thanks.


map not initialized when partial view recreated?

http://stackoverflow.com/questions/29925661/ng-map-initializing-map-in-partial-view

Allen, I stepped through the map directive initialization code and noticed what may be a problem. Please check out the details over on stackoverflow (http://stackoverflow.com/questions/29925661/ng-map-initializing-map-in-partial-view). Thanx!

Turns out it was my problem: I'd defined my own lat/lng object to hold the map's center, and then assigned a Google Maps LatLng object to it in the dragend event handler. I've posted the answer over on stackoverflow.


Everything breaks when deleting a remote server

author is None when checking is_remote

I have a guess as to the cause of this, it appears the author generic foreign doesnt appear to be set to cascade on a delete. Therefore there might be author lookups happening after a delete on authors that no longer exist.

Above comment is exactly what happens, when a server is deleted its authors are also deleted but the comments are orphaned. Going to make a routine on a delete of authors or remote authors to clean up comments.


Simple terrain generation

Procedural Terrain Mesh Generation

This is hidden under an experimental feature

This uses perlin and opensimplex functions to generate terrain data such as Height, Temperature and Moisture.
Should we create a branch 0.4 and start working on that branch instead of main?

Yeah, maybe. Then bug fixes can be added to the main for version 0.3 without delaying their appearance until 0.4.

I use space_editor to create my space sandbox. So I can also invest in a terrane generator in the future too

I finished code separation. Can you fix conflicts? Or I could do it tomorrow

I use space_editor to create my space sandbox. So I can also invest in a terrane generator in the future too

This is mostly a MVP version to start.
Will fix the issues

Do you don't mind if the pull request is uploaded only after 0.3 is ready? Additionally, the land generation should definitely be in the modules folder, as it is definitely not the core of the editor. (I consider the core of the editor to be the functionality that is needed to interact with entity component values, quick access to entities, easy entity placement on a level or prefab, and everything that is needed to link optional modules).

No problem, it gives me tive to improve it a bit

Also might want to add an example to the examples folder.

It seems that the terrain generation library should also be consistent with the #94

Closing due to conflicts


How to use this package with Laravel 5.6?

Versions:

ide-helper Version: v.2.9.0
Laravel Version: 5.6.39
PHP Version: 7.3.22

Description:
I try to install this package with composer
composer require --dev barryvdh/laravel-ide-helper
but I get:

sammy@33f31594dd34:/var/www$ composer require --dev barryvdh/laravel-ide-helper
Warning from https://repo.packagist.org: You are using an outdated version of Composer. Composer 2.0 is now available and you should upgrade. See https://getcomposer.org/2
Using version ^2.9 for barryvdh/laravel-ide-helper
./composer.json has been updated
Loading composer repositories with package information
Warning from https://repo.packagist.org: You are using an outdated version of Composer. Composer 2.0 is now available and you should upgrade. See https://getcomposer.org/2
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for barryvdh/laravel-ide-helper ^2.9 -> satisfiable by barryvdh/laravel-ide-helper[v2.9.0].
    - Conclusion: remove laravel/framework v5.6.39
    - Conclusion: don't install laravel/framework v5.6.39
    - barryvdh/laravel-ide-helper v2.9.0 requires illuminate/filesystem ^8 -> satisfiable by illuminate/filesystem[v8.0.0, v8.0.1, v8.0.2, v8.0.3, v8.0.4, v8.1.0, v8.10.0, v8.11.0, v8.11.1, v8.11.2, v8.12.0, v8.12.1, v8.12.2, v8.12.3, v8.13.0, v8.14.0, v8.15.0, v8.16.0, v8.16.1, v8.17.0, v8.17.2, v8.18.0, v8.18.1, v8.19.0, v8.2.0, v8.20.0, v8.20.1, v8.21.0, v8.22.0, v8.22.1, v8.23.1, v8.24.0, v8.25.0, v8.26.0, v8.26.1, v8.27.0, v8.28.0, v8.28.1, v8.3.0, v8.4.0, v8.5.0, v8.6.0, v8.7.0, v8.7.1, v8.8.0, v8.9.0].
    - don't install illuminate/filesystem v8.0.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.0.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.0.2|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.0.3|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.0.4|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.1.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.10.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.11.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.11.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.11.2|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.12.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.12.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.12.2|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.12.3|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.13.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.14.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.15.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.16.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.16.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.17.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.17.2|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.18.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.18.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.19.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.2.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.20.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.20.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.21.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.22.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.22.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.23.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.24.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.25.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.26.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.26.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.27.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.28.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.28.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.3.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.4.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.5.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.6.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.7.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.7.1|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.8.0|don't install laravel/framework v5.6.39
    - don't install illuminate/filesystem v8.9.0|don't install laravel/framework v5.6.39
    - Installation request for laravel/framework (locked at v5.6.39, required as 5.6.*) -> satisfiable by laravel/framework[v5.6.39].

Installation failed, reverting ./composer.json to its original content.

Is current version not compatible with Laravel 5.6? If so, which one would be?

Is current version not compatible with Laravel 5.6? If so, which one would be?

Correct, it's not. We dropped support for unsupported Laravel versions.
Please check packagist, by the looks of it it seems v2.8.0 should work -> https://packagist.org/packages/barryvdh/laravel-ide-helper#v2.8.0

(I wonder though, I thought composer would be able to pick a compatible version on its own based on the constraints… ah, guess it was just wishful thinking :)


Update reference-connect-adconnectivitytools.md

Providing the location of the adconnectivitytools module.

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

#sign-off


Proxy in config file

The default config.json contains a proxy definition with hard-coded address http://proxy.dortmund.isst.fraunhofer.de:3128. I cannot reach this proxy and I don't think it is necessary for users outside the ISST. In fact, not being able to connect to the proxy causes issues when trying to run the connector. Would it be possible to remove the proxy definition? Or did I oversee another use case for this? Thank you :)

Hi, thanks for the comment. We decided to leave this proxy settings, as it is always easier to remove something than having issues with adding new properties to a JSON string. The only reason is to ensure an easy entrance to setting up the connector.

Hi, thanks for the comment. We decided to leave this proxy settings, as it is always easier to remove something than having issues with adding new properties to a JSON string. The only reason is to ensure an easy entrance to setting up the connector.


Suggestion for an avatar redirect route

A proposal is not written for this. There's community interest in having a route to take a matrix user ID and get their profile photo without having to convert between MXC and the thumbnail API

Discussion for this happend around this message

(also 9 +1s in the first 2 hours ._. )

This wouldn't solve that problem, I'd imagine. It would be another route on the homeserver, which means you'd need to authenticate yourself with the homeserver. The deployment of that homeserver may affect how CORS works though.

An auth requirement is a hard block for us implementing this in IRCCloud. That’s hardly of much concern to the matrix project I expect but just a point of information. I was imagining matrix avatars were public information that we could use to augment the display of irc bridged matrix users. But if that’s not the case then there’s not really any way around it.

There could probably be some sort of rule like "if you're in a public room, your profile is public" or similar, therefore making the route potentially unauthenticated. I still don't really understand what IRCCloud is trying to do where the client won't have an access token though.

The client isn't a matrix client, it just shares channels with other users who are.

Also, I was under the impression the information was already publicly visible via existing apis, just not in a redirect/cross-origin friendly form.

Ah, alright then. So you're trying to provide a richer experience for those on the matrix side of the bridge?
The APIs are currently public, although I'm working on a proposal to fix that. The cross-origin issue isn't something the spec can really solve and is more a deployment concern. The redirect (as discussed here) is more as a convenience for not having to do the conversion dance.

A richer experience for those on the IRC side of the bridge.

For example, I'm in the #matrix channel on freenode as a plain IRC user "jwheare". With an appropriate API, I could display the avatars of any user in that channel connecting via the matrix side, by detecting a matrix-style ID in the realname field e.g. "@travis:t2l.io"
I have no strong feelings on how that API should or shouldn't made available, a redirect derived from the home server and id is just one solution that would conveniently satisfy our use case as a browser based client. Other web clients like thelounge, kiwiirc, etc could also make use of the same API, and while desktop clients like Quassel can use the existing multiple API calls without need for CORS headers, it would probably be more convenient for them to use a redirect too.
Whether this is a use case the matrix project cares about, I don't know. I acknowledge that you're reconsidering the privacy implications here too.

Seeing as per-room avatars are possible, maybe it makes sense to have the route take both the room ID and the user ID. Then if the room is public enough, then it can serve up the avatar without auth.

The room ID isn't exposed over IRC.

@jwheare That you cannot use it in your use case is another topic.
This is for adding such endpoint in general for matrix. And also considering rooms is reasonable I think.
Although from a privacy PoV I am not sure if I want to have such endpoint in general...


Initial implementation of the Card component for Octuple

SUMMARY:
Create Card component for Octuple:
Base Card:

Custom Card:

GITHUB ISSUE (Open Source Contributors)
JIRA TASK (Eightfold Employees Only):
https://eightfoldai.atlassian.net/browse/ENG-28741
CHANGE TYPE:

[ ] Bugfix Pull Request
[X] Feature Pull Request

TEST COVERAGE:

[ ] Tests for this change already exist
[ ] I have added unittests for this change

TEST PLAN:

nit can we include the screenshots of the components in description?

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@c3c5f32). Click here to learn what that means.
 Consider uploading reports for the commit 5e1285e to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #406   +/-   ##
=======================================
  Coverage        ?   85.48%
=======================================
  Files           ?      656
  Lines           ?    13912
  Branches        ?     4741
=======================================
  Hits            ?    11892
  Misses          ?     1966
  Partials        ?       54

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


error when serving compiled project from dist

Hey, i ran
NODE_ENV=development npm run compile

and served the directory then by a simple static koa server.
in the browser console i got the following error.
i didnt changed a file.
Resource interpreted as Stylesheet but transferred with MIME type text/html: "http://localhost:3041/app.f274289f0fc6393db73ed95c1e158221.css".
vendor.f72d2a38e02b954cdf0e.js:1 Uncaught SyntaxError: Unexpected token <
app.8fef3419aa6737a759ef.js:1 Uncaught SyntaxError: Unexpected token <

any hints?
running
npm start

gave no problems.

I need to document this, but this changed after a recent refactor. Currently you cannot compile with a NODE_ENV of development, since HMR will be enabled.  Are you also running npm start with NODE_ENV as production? I didn't see that command in your terminal log.
Installing deps for 1.0.0 atm, will get back as soon as that's done and I've tested it.

my current workflow looks like that:

i cloned master
i ran NODE_ENV=production npm run compile
i start a docker-container with a simple (my own) koa-static server (node 5.4.0 running). the container links the dist folder. i am starting the koa-server with NODE_ENV=production npm start
i am loading localhost:3000 in the browser and getting the message mentioned before
my docker-container output looks like that

http_1    | Listening on Port 3000
http_1    |   <-- GET /
http_1    |   --> GET / 200 71ms 554b
http_1    |   <-- GET /app.f274289f0fc6393db73ed95c1e158221.css
http_1    |   <-- GET /vendor.f72d2a38e02b954cdf0e.js
http_1    |   <-- GET /app.8fef3419aa6737a759ef.js
http_1    |   --> GET /app.f274289f0fc6393db73ed95c1e158221.css 200 20ms 554b
http_1    |   --> GET /vendor.f72d2a38e02b954cdf0e.js 200 19ms 554b
http_1    |   --> GET /app.8fef3419aa6737a759ef.js 200 7ms 554b

i think the server-side i ok here.
it seems that something goes wrong while compiling?

Hm, ok, so you're on master, that helps. I'm not too experienced with Docker so I'm wondering if it's related to that somehow. Thank you for the detailed workflow, I will play around with it this afternoon and see what I can uncover.

i think it is related with my koa server. its is running with
NODE_ENV=production npm run compile && npm start
i will look into it and give feedback if i can find something.
thx for your feedback!

i used koa 1.x and switched to 2.x with which it is running.


MUP without docker?

Hi guys,
is there any chance to deploy a meteor project without the need of using docker?
I am referring to the old mup, which is not being maintained anymore.
I am still using the old one, but I'm afraid to update meteor from 1.3, to 1.4, as it is using a newer node version.

Take a look at this fork of the old mup. Looks like it's maintained. https://github.com/aadamsx/meteor-up-classic

oh wow!
thanks a lot, I'll try opening an issues, to ask whether it works with meteor 1.4


Update JwtGuard.php

支持hyperf 3.1

@haozing thanks for your patch. But could you extend it to fix all issues?

[ ] src/AuthManager.php - add use function Hyperf\Support\make;
[ ] src/Guard/SsoGuard.php - add use function Hyperf\Support\make;
[ ] src/Annotation/Auth.php - replace parent::__construct($value); with $this->value = $value; or change constructor to public function __construct(public ?string $value = null) (base constructor was deprecated)
[ ] src/helper.php - replace use Hyperf\Utils\ApplicationContext; with use Hyperf\Context\ApplicationContext;


Qualified name is set from resolved IP address (db2)

Egeria: 3.14 (main, 992952d015f09624c00bf3d1e0f8ba79f28d15b2)
IBM DB2 (IBM Cloud)
Coco pharma lab deployment
test notebook
Problem

Connecting is resolving IP address (from DNS name)
host may have multiple IP addresses/endpoint (which may change)

I specified my jdbc connection as 'jdbc:db2://blahblahblah.databases.appdomain.cloud:31929/bludb:sslConnection=true;'
(where blahblahblah is a unique part of the name)
However after pointing the jdbc connector at this database the database entry's qualified name is
qualifiedName :jdbc:db2://<IP_ADDRESS>:31929/bludb
(again numbers fudged)
The problem is that when checking that original hostname, it in fact maps to 6 different IP addresses (as seen with nslookup), which likely get resolved in a round robin fashion. It could also change if a service moves, or gets exposed in another way
As such that conversion to IP means we do not have a unique identifier for the database - it's not a stable value
I think a better qualified name would be the host string specified in the jdbc URL, or some other stable information we can retrieve from the metadata.

this appears to be fixed in the latest connector. closing


Valid Sudoku (Leetcode)

Aim

Details
I would like to solve Valid Sudoku question in C++ language under LGMSOC'21.
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

@ravikr126 Kindly assign me this issue under LGMSOC'21

I would like to work on this issue in Python under LGMSoC'21.
Please assign me.

Please assign me this issue under LGMSOC'21.
I will like to contribute in Java .


Updated the wiki links to links to the md files

Re line 10 Not sure where that link is suppose to go so I'm going to leave that one alone
Re line 27 I looked at the seattle page and it redirects to build instructions so thats what I did here too and linked the page to the build instructions page
I added some of the unused links regarding RePyV2

Thanks as always for the contribution.


Sending "Ctrl-C" hotkey combination does not work

I've been searching for a solution to this for several days, but am beginning to wonder if I'm just looking at a bug.
I have a script that needs to copy to the clipboard using Ctrl + C, but somehow sending this key combination has lost functionality in pyautogui.
There is no exception thrown, and it appears everything is executing as it should be. However, the desired text is not copied to the clipboard. Copying the text manually using Ctrl + C (on my actual keyboard) does cause the desired text to be copied. However, when running pyautogui the clipboard remains unchanged.
I found this question on Stack Overflow that verifies the issue:
http://stackoverflow.com/questions/38899566/python-pyautogui-and-ctrl-c?newreg=4a6bdd511c314b129438d6d79f427600
I am also running Windows 7 and Python 3.5, as is Todd Lewden, the author of the Stack Overflow question.
The "Cheat Sheet" on Read the Docs indicates that this functionality should be present in pyautogui:

Keyboard hotkeys like Ctrl-S or Ctrl-Shift-1 can be done by passing a list of key names to hotkey():

pyautogui.hotkey('ctrl', 'c')  # ctrl-c to copy
pyautogui.hotkey('ctrl', 'v')  # ctrl-v to paste

I'm not sure at what point sending the "Ctrl + C" combination became broken, but it looks like it worked at some point, according to the above reference to the documentation.
I had a thought that perhaps it has to do with "Ctrl + C" being a Python keyboard interrupt, but I don't know that that is the case. If it were, Python should come to a screeching halt when the Ctrl + C is sent. It appears to execute without error, except that it doesn't work.

same here, when i tried to use "Ctrl + a" as pyautogui.hotkey('ctrl', 'a') nothing happened, is there any particular way of using it?

Both Ctrl+C and Ctrl+A work out of the box for me on Windows 7 with Python 3.5.
Does it make a difference if your script is running as administrator?

I can confirm that pyautogui.hotkey('ctrl', 'c') works for me on Windows 7 64-bit with Python 3.4.4, 3.5.2, and 3.6.0.

This person seems to have this same problem on Python 3, Windows 7 64 bit: https://stackoverflow.com/questions/41683334/doesnt-works-ctrlv-in-pyautogui-python-3-on-windows

I'm going to close this issue and combine it with https://github.com/asweigart/pyautogui/issues/93

I had the same problem trying to use both hotkey() and the separated keyDown(), press() and keUp() methods. I discovered that the methods did not work in Idle3 (running on MacOS Sierra 10.12.4) but they did work when executed via the terminal in a .py file.
So the problem might be with idle, strangely it's not even possible to type a backslash \ in idle (with alt, shift, 7), at least not on my german keyboard...

I had this issue as well. Solved by using the python 64bit libraries (Python35-64 vs Python35-32). Check your system environment variables.

Using the 64 bit installation of Python actually does work for me as well. Thanks for pointing this out.

None of the hotkeys work in Spyder 3.2.6, Python 3.6.4 64bits on windows 10:
import pyautogui as pat
import pyperclip
import webbrowser
webbrowser.open('https://www.digitalocean.com/community/tutorials/how-to-scrape-web-pages-with-beautiful-soup-and-python-3')
pos=(50,1000)
pat.moveTo(pos)
pat.hotkey('ctrl','u')
pat.hotkey('ctrl','a')
pat.hotkey('ctrl','c')
print(pos)

Sorry for digging up an old thread but it appeared to most relevant place to post.
Windows 10 (x64)
Python 3.6.3 (Anaconda) (I appreciate that this is beyond the versions listed as tested)
There appears to be a bug where by hotkeys involving the clipboard do not work (ie. Ctrl-C and Ctrl-V). In addition, and more worryingly, if a script using pyautogui is running, the clipboard becomes non-functional in its entirety. I cant copy and paste in other applications whilst the script is running.

I had the same issue but I've got around it using:
pyautogui.keyDown('ctrl')
pyautogui.press('c')
pyautogui.keyUp('ctrl')

"Ctrl-C" hotkey combination will work when you run your script from Command Prompt

"Ctrl-C" hotkey combination will work when you run your script from Command Prompt

I found the solution!
pyautogui.keyDown('ctrl')
pyautogui.keyDown('c')
pyautogui.keyUp('c')
pyautogui.keyUp('ctrl')
In my script I had to use root.update() in the middle.

I found the solution!
pyautogui.keyDown('ctrl') pyautogui.keyDown('c') pyautogui.keyUp('c') pyautogui.keyUp('ctrl')
In my script I had to use root.update() after.

Does not feel any of them works for my Ubuntu 22: #93
pyautogui.keyDown('alt')
time.sleep(0.2)
pyautogui.keyDown('tab')
time.sleep(0.2)
pyautogui.keyUp('tab')
time.sleep(0.2)
pyautogui.keyUp('alt')

or
time.sleep(2)
pyautogui.hotkey('alt','tab',interval = 0.1)

Putting this here because it might help someone out; I was trying to figure out why pyautogui.hotkey('ctrl', 'c', interval=0.2) wasn't working with win32clipboard and I figured out that if I already had the clipboard open (i.e. win32clipboard.OpenClipboard()), the data wouldn't get copied to the clipboard. My solution was to close the clipboard, copy the data using pyautogui, then reopen it and extract the copied data, something like:
pyautogui.hotkey('ctrl', 'c', interval=0.2)
time.sleep(0.1)
win32clipboard.OpenClipboard()
data = win32clipboard.GetClipboardData()
win32clipboard.CloseClipboard()

Putting this here because it might help someone out; I was trying to figure out why pyautogui.hotkey('ctrl', 'c', interval=0.2) wasn't working with win32clipboard and I figured out that if I already had the clipboard open (i.e. win32clipboard.OpenClipboard()), the data wouldn't get copied to the clipboard. My solution was to close the clipboard, copy the data using pyautogui, then reopen it and extract the copied data, something like:
pyautogui.hotkey('ctrl', 'c', interval=0.2)
time.sleep(0.1)
win32clipboard.OpenClipboard()
data = win32clipboard.GetClipboardData()
win32clipboard.CloseClipboard()

This solution worked. Note, to get win32clipboard you need to install pywin32. But you still import it as win32clipboard


Timezone support in DateTime()

Timestamps are stored in an undefined timezone in EXIF. In DateTime() imagemeta assumes UTC. There is a note mentioning missing support for OffsetTimeOriginal and OffsetTimeDigitized, but I have yet to see those in the wild.
The current implementation assuming UTC is almost always wrong.
I have a few suggestions for fixing this.

Add a timezone argument to DateTime(). This will break API-compatibility and will push the problem to the user of the package.

Add an additional function like DateTimeZone(tz *time.Location). This will provide a more correct function while maintaining backwards compatibility for using using DateTime().

Add a SetTimeZone() method to exif.Data, and let it default to time.UTC. This will provide backwards compatibility and not expose another helper function.

Assume local timezone using time.Local. This will not provide backwards compatibility, and not really fix the problem, but it will probably guess correct more often than assuming UTC.

Would you be interested in a PR implementing one of those suggestions?

Thanks for setting out those 4 options.

It's important that time zones are well represented, using only the local timezone or a set timezone would be insufficient.
I have been working on a solution with timezoneLookup to match latitude and longitude with timezone.
I am ok with breaking changes in API at this stage if it moves toward a good standard, more intuitive/readable, and better performance.

Either option 1 or 2 would be feasible. Do you have other suggestions?

I'll like to add that I have some thoughts about GPS timestamps, I think it's outside the scope of this issue, but for completeness, I'll include it here. The EXIF GPS timestamp should represent the time in UTC, but I have observed the following:

GPSTimestamp in UTC as expected.
GPSTimestamp in UTC without accounting for leap seconds.
GPSTimestamp in GPS time.
GPSTimestamp in TAI (!).
GPSTimestamp corrected for device time (!).
GPSTimestamp representing the time of "location lock", not capture time.
Strange errors around the first and last week of the year.

So in summary, not even the GPS timestamp can be used as any meaningful source of truth.
I think it's possible to infer a timezone based on coordinates (and a date). It would be useful if timezoneLookup provided a method returning a time.Location based on a coordinate (and a date) to easily interface with imagemeta.
I'll try to implement 1) as I think that makes the most sense, and it makes it very explicit that the package user has to make a choice.

GPS timestamps are useful if you know what standard the cameras you are using have. Many people who travel often forget to update their date and time stamps on their DSLRs and Point and shoots.
Canon seems to use GPS timestamps based on GMT0

Well. Since #27 got merged, we're in a much better place, and I'll close this.

FWIW, Google Pixel phones use OffsetTime, OffsetTimeOriginal, and OffsetTimeDigitized for time zone information. Looks like the value is, e.g. -04:00 but I don't know if that's the "raw" value in the file or if that's an interpreted value. (Multiple exif tools have showed this value.)
The current exif lib I'm using, cozy/goexif2, doesn't read those. Will/does this package support those fields?

@mholt thanks for mentioning this. It appears to be the raw value. 7 bytes long ending in a null byte.
I have just pushed an update to develop branch that includes these 3 Tags. Due to the way time.Location works I am struggling to convert -04:00 to a Location to be included in a time.Time. Have you been able to do this?

@evanoberholster Thanks for the reply!
Yep, I can confirm. I also found this confusing but after a bit of fiddling, you can see what I got in the linked PR above:
// timeOffsetVal is the EXIF field value as a string string, e.g. "-04:00"
t, err := time.Parse("-07:00", timeOffsetVal)
if err != nil {
	return nil, err
}
return t.Location(), nil

Hope that helps :) Thanks so much again for working on this lib. Really excited for it.

Hi @mholt, thanks for contributing ideas. I made some of the changes you asked for. Took a deeper look at the time package to find the most efficient way to implement offsets. I still think it could be improved, right now it has 4 allocations per tag.
Take a look at 5fb96b9. Kindly fork the develop branch when you get the chance and test it.
I hope to soon be able to push develop to main after the rewrite. That way PRs can be smaller with changes instead of major rewrites.

I went ahead and tested the timestamp values from the develop branch to make sure the timezone is properly applied, and sure enough it looks correct. Thank you so much!

Closing this issue as it appears resolved. Please reopen if it needs to be further addressed.


Zoomed/streched images with IE11 but not with Chrome/Firefox

This is a (multiple allowed):

[x] bug (?)

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 3.4.1

Platform/Target and Browser Versions: IE 11.953.14393.0 on Windows 10

What you did
I tried to mess with the width of the items, remove it, add some.
Expected Behavior
Show the full pictures, not just a part of it
Actual Behavior
When I view the slideshow with IE, It gets all zoomed or streched.
Screenshot Live Example

This fixed it.
.swiper-container img {
    width: 100%;
 }


[Tracking/Action] Post-KubeCon EU 2024 Comms Ideas

Description
After the event it is good to hype up the amazing work we did and people we interacted with. I've been working on a few things for this which we can track here. We can also come up with any other things we might want to do e.g. blogs.
To-Do
Current action items to work on that I can think of!

[ ] Compile clips into a youtube video as a trailer and for social media from the event (longer format and maybe youtube short too?)
[ ] Compile all the sustainability talks into a youtube playlist.
[x] Photo folder for the TAG members to share photos of talks/meetups
[ ] Update the conference checklists and create runbooks to help with this!

All great ideas! +1

related #266 #86 #353


[Skype, BotFramework]My Chat Bot don't responds on Skype only for some users

Hello @stevkan
we have a chat bot approved on Skype, everything has been working fine until today it stopped responding in the following mobile versions <IP_ADDRESS>, <IP_ADDRESS>
Is there a bug or update by the Skype team?

Which mobile version of Skype is this issue occurring in?  (iOS?  Android?  Other?)
Also, which BotFramework SDK are you using? (NodeJS? C#?)

Hello, searching a Little bit we found that the "trouble" was that Skype (a lot of versions but not all), including the version for desktop of Windows 10, don't send in it activity the name of the user, for this reason, my bot didn't response.
But now, we remove this opción from our code, and all right!
Thanks   @nodeomega  for response.


Except all errors on task==key check in subs

cc @eriknw your feedback on this would be appreciated.
@cowlicks this resolves at least part of your recent issue

@mrocklin which recent issues?

Merging soon if no comments


[CELEBORN-1226][IMPROVEMENT] Make all thread to use ThreadUtils

What changes were proposed in this pull request?
Make all single thread use standard ThreadUtils to simplify the code
Why are the changes needed?
Does this PR introduce any user-facing change?
How was this patch tested?

ping @FMX @SteNicholas

also ping @mridulm


Pool revm::Evm to share among execution threads

Won't need this anymore once we remove revm dependency with #382.


Upgrading to PyPy 5.9.

See: https://morepypy.blogspot.com/2017/10/pypy-v59-released-now-supports-pandas.html
Fixes #1.

Dang, I forgot about this...thanks!! I'll probably be able to submit a new package tomorrow or Sunday.

Thanks! Is there a link to follow for the package update?

@dhermes You can see it here.

w00t!

@dhermes FYI it's been approved!


Help need to be implemented

We need context aware help.
Help button  - already there
Need to do:

retranslate the request to State
State read form some storage, better in HTML and display it.

Help code is done. Need to fill the content. It is in the progress.


Microsoft Edge shows a black preview screen then crashes

Hycov works great with most windows except with Edge.
When I have a Edge browser open, hycov toggle shows:

very briefly the Edge window content
then it's replaced by black
a few seconds later, Edge window content appears again

If I use hycov a few times, Edge freezes and is not responsive until I do killall msedge
Edge has been very stable before I used hycov.
The cause is unknown. I can try to log more information if you tell me what would help

I've been using edge and there's no problem
https://github.com/DreamMaoMao/hycov/assets/30348075/5fd96824-1921-4886-a63f-e124edf480c2

That's strange, I will try to do a OBS record of when the issue happens!

i saw the problem, is your ms edge on the latest version? coz mine's not and it freezes in hycov

Unfortunately, I was not able to reproduce it, which may have something to do with the fact that different machines use different graphics drivers


Add check access database for system tables

I'm not sure it's necessary. It helps, though, to shield messy data from multiple user scenarios.
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

I think system database should be generally available for all the users - as it contain system.number, system.one and other very handy tables. Only few tables can be considered as potentially unsafe: system.query_log, system.zookeeper.  System.processes can be limited to current user only for users with limited access.

@filimonov This pull request just filters content of system tables, that contain information about another tables. All system tables remain available, but only with information about tables that the user is allowed to see.

@zhang2014 We can't filter system.processes, system.query_log, because they don't contain information about database. Even more, the query can use multiple databases. Lets just leave it as is.

@zhang2014 The only missing part is integration test.


Update Google Cloud Bigtable client to the lastest version, beyond Google Cloud BOM to support new features

Newest Google Cloud Bigtable clients include support for the new Bigtable Changestream API https://github.com/googleapis/java-bigtable/releases/tag/v2.19.0. It was released 2 days ago and not in the latest Google Cloud BOM https://github.com/googleapis/java-cloud-bom/releases/tag/v26.7.0. We will remove this override once Google Cloud BOM includes 2.19.0

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @pabloem

LGTM


[WIP] Text to speech mixin

When finished, will close https://github.com/pi-top/pi-top-Python-SDK/issues/376
pyfestival Python dependency is provided here into our apt repository, with a patch that is needed in order for import to work on RPi. Workflow run that added it into the apt repo is here. There should be no need to keep https://github.com/pi-top/pyfestival/ at this point.
Architecture
Uses a generalized object factory that registers backend services we wish to make available - currently we only have "DEFAULT" and "FESTIVAL" but in future we'll ideally have "GOOGLE", "AMAZON" etc. The builders will accept the kwargs relevant to that service and ignore the rest, this should make it flexible when we add new services that require API keys etc.
One downside to this approach is that the speech service can't be changed easily from the Pitop() class, one way to do it is as follows:
from pitop import Pitop
from pitop.processing import tts

my_speech_service = speech.services.get("FESTIVAL")

pitop = Pitop()
pitop.speech = my_speech_service

Alternatively, we could pass in the speech service to Pitop class:
pitop = Pitop(tts_service_id ="FESTIVAL")

But it feels a shame to add parameters to the Pitop() class. We could also use a class method:
pitop = Pitop.using_speech_service("FESTIVAL")

which is probably a cleaner approach.
To do

Use a factory pattern for allowing different speech backends to be swapped in as it's very likely we'll want to change the TTS engine in future (for compatibility or general improvement)
Discuss if speech service settings will be included in the Pitop.from_config() usage
Fix SIOD ERROR: the currently assigned stack limit has been exceeded error when using the non-blocking mode (thread)

I am not convinced with the design pattern. Why should the user need to know anything about festival? Why does the user need to handle the importing and management of TTS directly?
from pitop import Pitop

pitop = Pitop()
pitop.speak("Hello World")
# Another option might be a 'speaker' property of the pi-top:
# pitop.speaker.speak("Hello World")

seems like a much more sensible way to approach this.

This is currently failing to build the package as there is a spelling mistake in this commit's message that is propagating into the snapshot version changelog.
This draft PR provides an easy way for all package builds to ignore changelog spelling mistakes for snapshot builds with a simple boolean input. This hasn't been tested, and this PR is a good test of this functionality.
We should update this PR to use this experimental branch until it passes, then merge in and build against this new master commit. Once this is working here, we can update ALL package build workflows to ignore typos for snapshot builds moving forwards.

I am not convinced with the design pattern. Why should the user need to know anything about festival? Why does the user need to handle the importing and management of TTS directly?
from pitop import Pitop

pitop = Pitop()
pitop.speak("Hello World")
# Another option might be a 'speaker' property of the pi-top:
# pitop.speaker.speak("Hello World")

seems like a much more sensible way to approach this.

Yeah this is exactly how it is now, though I do like the “speaker” idea as we can add more speaker-specific commands to that too (beeps, songs, pre-recorded voices etc)

pyfestival is currently failing to build: https://github.com/pi-top/pi-topOS-Upstream-Package-Build/runs/3455683743?check_suite_focus=true
We will need to create a patch for the changes needed for this build to pass in CI.


Improve website style

the website is generated from /docs, but there are a few things need to be fixed

[ ] navigation bar in narrow screen, namely when the width is less than a threshold, the navbar becomes a symbole "[=]", then the dropdown menu should be aligned vertically rather than horizontally
[ ] search doesn't work. it works well when build on local machine, so it's probably due to readthedocs.org using different version software or failed to load the index.js
[ ] the doc body has no padding when the screen width is less than a threshold

we definitely need to change more things to make the website looks better

ping @yuweiw823

will work on this :)

could this be something from sphinx instead of docs natively

it's the search index generated from all docs, should be located at the
root directory.
http://mxnet.readthedocs.io/en/latest/searchindex.js
On Mon, Jun 20, 2016 at 1:01 AM, yuwei wang<EMAIL_ADDRESS>wrote:

[image: screen shot 2016-06-20 at 00 58 49]
https://cloud.githubusercontent.com/assets/8573604/16186927/3a1ffe20-3682-11e6-8db3-a33b5ca31b5c.png
Seems that we missed a .js file...Never see it before in the docs folder.
@tqchen https://github.com/tqchen
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dmlc/mxnet/issues/2464#issuecomment-227074706, or mute
the thread
https://github.com/notifications/unsubscribe/AAZv4YAY_Yq_lTqyHcYZW5VAP3Xycp_Yks5qNkjYgaJpZM4I5BTg
.

Closing it for now since we have new website design


[feat] Adding Design on the buttons

Would you like to have a new feature? Please describe.
Want to add some design on the buttons.
See this video once
https://user-images.githubusercontent.com/70086051/197324529-3d9c75a6-6080-4f18-a33f-370708224f5f.mp4
https://user-images.githubusercontent.com/70086051/197327062-8a9d1c98-1848-4d03-8a4a-17bafa078bbb.mp4
How would this feature be beneficial for the project
No response
Additional context
No response

@SuperAayush @bhavyastar please take a look and confirm if this change conflicts with the changes in the roadmap

Hey @bishal7679!!
You can go for a PR.


validate_version! doesn't exist

but it is mentioned in the README

Me too:

Hey, this method was actually removed in version 2. Just merged a change so that it's no longer referenced in the README!


feat: reduce the Cloud Run Hello World image size

With this change the image size is about 1.3MiB compressed, and GCR
says that it is less than 0.5MiB compressed. That makes for faster
startup times.

@coryan Oh sorry I overlooked this one, but it seems like it's already in the repo?

@tmatsuo no, this change is not yet in the repository.

@coryan So we don't need the ca-certificate package at all?

@tmatsuo for a pure server that does not access GCP, nope, we do not need the ca-certificates.


Does not word with other language

Where I have to change to support UTF-8. so that I can train it on other languages

It actually should work with utf-8 if you're using the latest version.
What are your versions:

char-rnn-tensorflow
tensorflow
python

Thanks.

Actually the sample outputs my Greek text as raw utf-8 , "  \xcf\xce\x83,  \xb1\xb9\  .........."

@lowtronik that hex format. just decode it
result.decode("utf-8", "replace")

@ShuvenduBikash  I just deleted  .encode('utf-8')  and it works

I have the same problem, it generates raw text like this

\xc3\xa8p

However if I follow your suggestion and delete .encode('utf-8') it fails with this error:

UnicodeEncodeError: 'ascii' codec can't encode character '\u201c' in position 444: ordinal not in range(128)


fix power after snapshot load

Description

Checklist

[x] This PR has only one purpose or idea.
[ ] Tests have been added/updated to verify that the new/changed code works.

[x] The code style follows good practices.
[x] The commit message(s) follow good practices.

[x] The release notes (location doc/release/0.X.rst) are up-to-date with any bug fixes or new features.
[x] The documentation is still up-to-date in the doc folder.
[x] The dependencies are still up-to-date in setup.py.

@john-science , we have no unit tests for the main interface at all and its part of db load which is a pretty heavy lift.  i think we should rely on internal in-use tests.


Adding basic sanity test.

This adds a basic script to test if we didn't forget to run the templates renderer. We can add more to that as we proceed to add more sanity checks.

I will merge this once it's updated :)

Updated.

Looks like it fails?

I don't know what you're talking about :-P

One more merge?

grunts
On Tue, Apr 14, 2015 at 7:41 AM, Craig Tiller<EMAIL_ADDRESS>wrote:

One more merge?
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/pull/1124#issuecomment-92883954.


Bring back spy.reset()

A previous issue https://github.com/chaijs/chai-spies/issues/27 introduced the reset method to this library which we use extensively. It was removed in 1.0 for no clear reason. I'd like to see it put back

@AdamMcCormick can you use restore instead?
chai.spy.restore()

@dabutvin No. Please see the thread at #27. Restore is not the same thing and does not satisfy my use case. It also makes my tests significantly slower given my test setup.

Any update on this issue?

Hey @AdamMcCormick thanks for the issue.
It was removed as it was seen to be an anti pattern because it swaps out internal state which can cause issues in tests - certainly compounding issues if your tests rely on external state. The preferred alternative is to restore and make a new spy.
@AdamMcCormick could you please give us some numbers as to how much slower your tests are by using restore? I'd be surprised to see even 100ms delta between restore/reset.

You can just do
const spy = chai.spy()
let funcToBeSpied

beforeEach(() => {
  funcToBeSpied = spy()
})

With proxyquire you have to clear the require cache and re-proxyquire the module, in order to do a full reset.
That is quite messy. This was my solution for resetting the spy, in lieu of spy.reset():
let barSpy;
const foo = proxyquire("../lib/foo.js", {
  "./bar.js": (...args) => barSpy(...args),
});

describe("foo bar", () => {
  beforeEach(() => {
    barSpy = chai.spy();
  });

  it("bar is called once by foo", () => {
    foo();
    expect(barSpy).to.have.been.called.once;
  });

  it("bar is called once by foo, again", () => {
    foo();
    expect(barSpy).to.have.been.called.once;
  });
});


Merging next to main for release 2.0.3

Notes

Merging next to main for the latest release
Please review the app version and generated release notes,
and make any necessary updates
Additional next to main changes can be found here: !3

closing this PR per https://github.com/splunk-soar-connectors/berryio/pull/6 so it doesn't get accidentally merged.


Add Riot Account support (using Riot Account-V1)

Hi there!
I wanted to use Orianna for a small personal project but while I was using it, I noticed there was no way to query Account-V1. I checked the discord for any information on the topic but couldn't find any, and it was pretty quiet.
Riot will be moving away from Summoner names on Nov 20, according to their API docs. Looks like they will support the Summoner-V4 endpoints for awhile after, but they're deprecated at this point. https://developer.riotgames.com/docs/lol#summoner-names-to-riot-ids_summoner-names-post-migration
I was really unfamiliar with the Orianna repo but I got it working as I needed for my personal project. This pull request includes:

Adding Account object and related pipeline information to query the Account-V1 API
Addition of regionalRoute to the Platform enum to account for the different endpoint that Account-V1 uses
New constructors to support the previous change
4 new methods in Orianna base class

accountWithPuuid
accountsWithPuuid
accountWithRiotId
accountWithRiotId

I realize this is nowhere near a complete PR but it's working enough for my personal project for now, with no breaking changes to the existing codebase to my knowledge. I figured I'd open it since this seems to be something that will be needed in the future anyway, and if anyone wants to use some of the work here it's here to use. I figure this might need a more comprehensive implementation.
There are some questionable decisions (especially in the APIs) that I made because I don't think the pattern of Riot Ids gameName and tagLine fit in with any existing design patterns. I just wanted to get something working in an afternoon.

Thanks for taking a look! I made some of the changes that were noted above. This is also my first time looking at the internals of Orianna so I don't have the fullest grasp of how everything fits together.
One thing about URL encoding - I didn't see any other endpoints (like summoner name) that did it. I think it's encoded as part of the URLBuilder in the HTTPClient (here).
If this is a special case and needs to be encoded otherwise, I can make the change!

Good catch, you're correct that URL encoding is handled elsewhere. LGTM, thanks for the contribution!


GDF Files NaNs read as lowest negative number + other  data 1e6 smaller

When reading GDF files, in some cases NaNs are converted to the lowest negative number, is this intended behavior?
>>> raw_edf = mne.io.read_raw_edf('/home/schirrmr/data/bci-competition-iv/2b/B0101T.gdf')
>>> data = raw_edf.get_data().T
>>> data[52250:52352,:3] # there should be nans within that range
array([[ -2.13321126e-06,  -2.71305409e-06,  -6.13412680e-07],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [...]
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [ -1.00001526e-04,  -1.00001526e-04,  -1.00001526e-04],
       [  3.66216526e-08,   2.79240101e-06,  -3.98260472e-06]])

Dataset is from this: http://www.bbci.de/competition/iv/#dataset2b
The description given there about NaNs is:

Note that the runs are separated by 100 missing values, which are encoded as not-a-numbers (NaN) by default.  Alternatively, this behavior can be turned off and the missing values will be encoded as the negative maximum values as stored in the file with [...]

But I didn't see this behavior (negative minimum value) documented in mne?
Also, at least on my PC, these minimum values are not identical for different channels, which could cause confusion?
>>> np.min(data[:,:3])
-0.00010000152590218967
>>> np.min(data)
-0.0010000152590218965

Also, non-NaN values are factor 1e6 smaller than the values I get from biosigtoolbox in matlab.

I looked at the data and indeed, the values are negative maximum values (int16, -32768). So nothing is actually converted, the data just is encoded to be small, which turns into some other small values when calibrated accordingly. You can work around this by setting the small values to NaNs yourself using some threshold. The small differences are probably caused by floating point imprecisions.

Also, non-NaN values are factor 1e6 smaller than the values I get from biosigtoolbox in matlab.

Biosigtoolbox is is probably presenting the data as microvolts. In MNE the EEG data are in volts.

I think this is just a feature of Biosigtoolbox in how it treats minimal values. Closing.

Thanks for the explanations :)


Issue with ngFor

The datepicker popup is not working when used in  a ngFor loop. I am displaying the datepicker controls using ngFor. The control is showing up but when I click on the button, popup is not showing up.

Hi I am also facing the same issue.I want to display Datetimepicker using ngFor loop inside modal window.
Can you please help..


Add logo

I had a go at putting together a logo for the Cross Gov community. I've run these past one or two people and they seemed to think they were not terrible.
We can use these:

on our website
on social media
in our email comms
probably other places

Just pushing these files up to github so they're available

hmm, turns out that I have forgotten my schoolboy maths and hexagons don't fit exactly inside a square anyway


Publish wasm

Hi, I would like to know if there is a reason to close this issue. Are there any plans to release wasm version?

Hi, I would like to know if there is a reason to close this issue. Are there any plans to release wasm version?

I haven't found the time to properly maintain the wasm version. What's your usecase?

I'd like to add Oxc parser to my AST Explorer

I noticed that the playground's package.json loads it as such:

https://github.com/oxc-project/oxc/blob/main/website/package.json#L32C5-L32C19

"@oxc/wasm-web": "link:../npm/wasm-web",

Yet there doesn't seem to be a npm/wasm-web in the main repo?

https://github.com/oxc-project/oxc/tree/main/npm

What's your usecase?

We were also exploring whether it would be a suitable parser in wakaru:

https://github.com/pionxzh/wakaru/issues/35#issuecomment-1835413696
https://github.com/pionxzh/wakaru/issues/35#issuecomment-1835430137

What's your usecase?

@Boshen We were also exploring whether it would be a suitable parser in wakaru (cc // @pionxzh):

Split code and save progress? pionxzh/wakaru#35 (comment)
Split code and save progress? pionxzh/wakaru#35 (comment)

I haven't found the time to properly maintain the wasm version

@Boshen Off the top of your head, what are the current issues with it that require further maintenance/etc? Do you consider it not up to date/usable/etc?

Things could work if you intend to write everything in Rust, otherwise it's not going to work on the JS side. We lack all the things you need on the JS side after looking at your requirements.

We will need the wasm binding for js parser and the AST query and run them on JS side.
So the main blocker is that there is no enough resources and maintainers for wasm version?

AST query and run them on JS side
We don't have a AST visitor on the JS side.

So the main blocker is that there is no enough resources and maintainers for wasm version?

Kind of, it's gonna be a lot harder to maintain because there are currently no TypeScript types for the AST on the JS side.

We don't have a AST visitor on the JS side.

Is using a WASM plugin a good idea?

To be honest I'm still not sure whether the scope of this project should expand to supporting wasm and napi ...

Appreciate trying to keep the scope small but I was thinking that the the oxc ecosystem could benefit from supporting multiple use cases with its components. That way it would gain new contributors and would ease some of the maintaining burden. The community would also benefit by not needing to duplicate effort redoing lexers, parsers, traversers...etc.
My usecase involves rewriting some JS/TS from both js and external systems and oxc came to mind.

Done! https://www.npmjs.com/package/@oxc-parser/wasm
I used wasm-pack build --target web for the build, probably need something like https://github.com/nshen/vite-plugin-wasm-pack to get it working with vite, otherwise vite will load the wasm file as a HTML file causing a CompileError: WebAssembly.instantiate(): expected magic word error.

Thanks for your excellent work! However, the usage of docs may be wrong.
parseSync(code, { filename: "test.ts" });
> Error: expected instance of ParserOptions

The parser options cannot be a plain object.

If we pass a class, there's an error once get/set value.
const parserOptions = new ParserOptions()
parserOptions.filename = options?.filename || 'foo.ts'
> Error: null pointer passed to rust

@sxzz I released<EMAIL_ADDRESS>and updated the README https://www.npmjs.com/package/@oxc-parser/wasm
wasm FFI is really confusing, I'm still unsure whether I fixed it or not, but "it works on my machine" at least.

Great, it works now.
P.S. Oxc parser is already supported by my AST Explorer version.


scanner: Fix filtering of project scan results

If the definition file of a project is in the root of the repository,
all license findings which were not in the definition file itself or in
a license file were removed by filterProjectScanResults, because
substringBeforeLast("/") returns the definition file path itself
instead of an empty string.
To fix this, do not filter at all if the definition file is in the root
of the repository.
Signed-off-by: Martin Nonnenmacher<EMAIL_ADDRESS>

This change is

/rebase


Fix formatter to correctly convert invalid doc comments

Background

Fixes an issue where a doc-comment above a trait-statement and member may get converted to a line-comment
Updates the expected formatting of test case file (.formatted.smithy)
Updates tests to use assertEquals, since it provides diff functionality in IDEs
Caveat:

A trailing doc-comment won't get converted to a line-comment

TODO was added to handle this case, if it is possible

Testing

Ran the tests, no failures

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I pushed a commit that fixes the regression, which is captured by what I added to the test case here. The TODO here is a separate issue that occurs when the comment is after a default value assignment. We will need to follow up to address that.

Followup work: https://github.com/smithy-lang/smithy/issues/2279


System generated comments for refunds and fee waivers

As an IAO analyst and lead
I want a system generated comment when a refund or fee waiver has occurred
so that I have an audit trail of who issued the refund or waiver

Assumptions & Scope
What are the assumptions for this story?
If a subsequent waiver or refund is updated, then a new comment will be posted.
What is IN scope?
System generated comments.
What is NOT in scope?
Notifications are not required.
Entering the fee waiver (#2772) or refund (#2679) amounts
Emailing CAS to issue the refund and notifying the applicant.
Acceptance Criteria
Scenario 1: System generated comment for fee waiver

GIVEN a fee waiver has been approved by the ministry or IAO (outside system)
WHEN the IAO analyst enters the Fee Waiver Amount and saves the CFR form
THEN a system generated comment will be posted in 'Request History Comments' that says "[Username] entered a fee waiver for the amount of $xxxx"
AND the comment will have the date and time.

Scenario 2: System generated comment for refund

GIVEN a refund has been approved following the refund process (outside system)
WHEN the IAO analyst enters the Refund Amount and saves the CFR form
THEN a system generated comment will be posted in 'Request History Comments' that says "[Username] entered a refund for the amount of $xxxx"
AND the comment will have the date and time.

Dependencies? What is the impact of this dependency? (If so, link dependency in the ticket, make it visible in a team´s backlog)
Validation Rules? (If yes, list here)
Design
@xxx - please link the Design here
Definition of Ready

[ ] Is there a well articulated User Story?
[ ] Is there Acceptance Criteria that covers all scenarios (happy/sad paths)?
[ ] If there is a user interface, is there a design?
[ ] Does the user story need user research/validation?
[ ] Does this User Story needs stakeholder approval?
[ ] Design / Solution accepted by Product Owner
[ ] Is this user story small enough to be completed in a Sprint? Should it be split?
[ ] Are the dependencies known/ understood? (technical, business, regulatory/policy)
[ ] Has the story been estimated?

Definition of Done

[ ] Passes developer unit tests
[ ] Passes peer code review
[ ] If there's a user interface, passes UX assurance
[ ] Passes QA of Acceptance Criteria with verification in Dev and Test
[ ] Confirm Test cases built and succeeding
[ ] No regression test failures
[ ] Test coverage acceptable by Product Owner
[ ] Ticket ready to be merged to master or story branch
[ ] Developer to list Config changes/ Update documents and designs
[ ] Can be demoed in Sprint Review
[ ] Tagged as part of a Release
[ ] Feature flagged if required
[ ] Change Management activities done?

@aparna-aot, for the system generated comment could we please show two places after the decimal:

Thanks for the fix, @aparna-aot


[Python docs feedback] when building the image in docker tutorial, it can't install package from requirements.txt (Flask)

When i follow this tutorial(https://docs.docker.com/language/python/build-images/) in docker guide, i have some error. this tutorial is how to build image in python code. When i write "docker build --tag python-docker .", Error occurs.
bash can't install flask package in requirements.txt. Especially, it can't install this format like " @ file :/// ~~~ "
my error log is here
$ docker build --tag python-docker .
[+] Building 9.4s (12/13)
=> [internal] load build definition from Dockerfile                                                          0.0s
=> => transferring dockerfile: 37B
...
...
=> ERROR [4/5] RUN pip3 install -r requirements.txt                                                          3.0s

[4/5] RUN pip3 install -r requirements.txt:
#12 2.247 Processing /opt/concourse/worker/volumes/live/893742d6-dd9e-4693-6ad8-7402592041f5/volume/aiohttp_1607109646206/work
#12 2.250 ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/opt/concourse/worker/volumes/live/893742d6-dd9e-4693-6ad8-7402592041f5/volume/aiohttp_1607109646206/work'
#12 2.250

my requirements.txt is here

my docker file is here

Actually, i can't solve this problem. many package in requirement.txt are "@ file:/// ~~" format. So i can't edit to == in manually.
what should i do? i need your help!
version
docker : 20.10.7
flask installed : 1.1.2
python : 3.8.2(anaconda, virtual name is python_pro2net)

Thanks for reporting this issue. It's now fixed as part of https://github.com/docker/docker.github.io/pull/13507.


ignore linting

Description
Fixes #15
Checklist

[ x] I have performed a self-review of my own code
[ x] I have followed the name conventions for CSS Classnames and filenames, Components names and filenames, Style filenames, if you are in doubt check the the project README.MD and here https://github.com/HackYourFuture-CPH/curriculum/blob/master/review/review-checklist.md
[x ] I have commented my code, particularly in hard-to-understand areas, if you code was simple enough mark the box anyway
[ x] I have made corresponding changes to the documentation, if you code was simple enough mark the box anyway
[x ] This PR is ready to be merged and not breaking any other functionality

Hi @senner007 !
Could we add the same fix to boilerplate-for-fp 👍 ?

Hi @senner007 ! Could we add the same fix to boilerplate-for-fp 👍 ?

Definitely 👍


Add transactions manually into database

We should add the following transactions into the database manually:

[ ] https://etherscan.io/tx/0x12d699a9c3eb3605aebbeeea9453286a16d2772738a837efe36cd4a891f75893

@divine-comedian The fromAddress of this user doesn't have a profile in our DB, should I create profile for it?

Hey guys... I dont think I can add transactions in AdminBro yet. Right @mohammadranjbarz?
I can make them GIVbacks eligible or not if they already exist however, the last time I asked, I couldnt add any that didnt already exist.
Happy to help with these things in the future once that is an option for me.

@divine-comedian The fromAddress of this user doesn't have a profile in our DB, should I create profile for it?

Wow really? Hmm that's very suspicious, there's a chance the user never actually made this donation through our UI.
If we can't find this donation anywhere in our back-up service nor does their profile exist (they never signed into the app) my guess is they never made this transfer through Giveth - plus I can confirm this is the project owner donating to their own project..

Hey guys... I dont think I can add transactions in AdminBro yet. Right @mohammadranjbarz?
I can make them GIVbacks eligible or not if they already exist however, the last time I asked, I couldnt add any that didnt already exist.
Happy to help with these things in the future once that is an option for me.

No unfortunately we can't add donations in the DB right now, we just can make them givback eligible or givback not eligible by admin panel


Add GitHub actions workflow to build and publish package to PyPI

You will soon be able to do pip install vyxal with no further details. Closes https://github.com/attempt-this-online/attempt-this-online/issues/6 (CC @Lyxal)
This needs to be pulled into main, tested with a placeholder tag (I suggest v2.7.2post1), and if/when it works:

remove the repository_url so it uses the real PyPI
replace the PyPI token with one that has permission to upload to the vyxal package namespace

I suggest I own that package initially and then once I've got it all working I'll grant permissions to people (you'll need PyPI accounts!)

Also it looks like I already have a pypi account if that helps you at all.


Still getting an error with the new release

This error shows up when i try to build my project
A problem occurred configuring project ':fluttertoast'.

Failed to notify project evaluation listener.
java.lang.AbstractMethodError (no error message)

Me too.
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':fluttertoast'.
> Failed to notify project evaluation listener.
   > java.lang.AbstractMethodError (no error message)

Flutter:
Running flutter doctor...
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v0.9.4, on Mac OS X 10.14 18A391, locale en-TH)
[✓] Android toolchain - develop for Android devices (Android SDK 28.0.3)
[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
[✓] Android Studio (version 3.2)
[✓] IntelliJ IDEA Community Edition (version 2018.2.4)
[✓] VS Code (version 1.28.0)
[✓] Connected devices (1 available)

• No issues found!

Me too. All version don't work)

Me too. All version don't work)

Error running Gradle:
Exit code 1 from: E:\Project\cc\android\gradlew.bat app:properties:
WARNING: The specified Android SDK Build Tools version (27.0.3) is ignored, as it is below the minimum supported version (28.0.2) for Android Gradle Plugin 3.2.0.
Android SDK Build Tools 28.0.2 will be used.
To suppress this warning, remove "buildToolsVersion '27.0.3'" from your build.gradle file, as each version of the Android Gradle Plugin now has a default version of the build tools.

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':fluttertoast'.

Failed to notify project evaluation listener.
java.lang.AbstractMethodError (no error message)

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 3s
Please review your Gradle project setup in the android/ folder.

same here

same here

same……

Hi all,
I had the same problem. I fixed it by updating a few version strings in my project:
android/app/build.gradle file:
compileSdkVersion 28
android/build.gradle file:
classpath 'com.android.tools.build:gradle:3.2.0'
classpath 'org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.71'
android/gradle/wrapper/gradle-wrapper.properties file:
distributionUrl=https\://services.gradle.org/distributions/gradle-4.10.2-all.zip

2.0.7  workaround .
make sure the version in your pubspec.lock and pubspec.yaml

same here, solved with version 2.0.7, thank you!@CarGuo

I am with version 2.0.7 and it doesn't work for me. What else can be missing? Thank you

still in 2.0.9
Launching lib\main.dart on Android SDK built for x86 64 in debug mode...
Initializing gradle...
Resolving dependencies...

Error running Gradle:
Exit code 1 from: C:\Users\Administrator\AndroidStudioProjects\esma\android\gradlew.bat app:properties:

FAILURE: Build failed with an exception.

What went wrong:
A problem occurred configuring project ':fluttertoast'.

Failed to notify project evaluation listener.
java.lang.AbstractMethodError (no error message)

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 2s
Finished with error: Please review your Gradle project setup in the android/ folder.

Please check with v2.1.0 and let me know if still exists

Please check with v2.1.0 and let me know if still exists

Wow, fixed.


Preserve the bounds of the type parameters of used classes

This should be part of the modularity model. The previous test case that showed we didn't do this was wrong, and I should not have approved it.
Should fix JDK-8319461.

If this passes CI, I will override approval requirements and merge it without review from @LoiNguyenCS. He should review it later when he gets a chance.

I just assumed that removing the unused type bound should not affect an analysis tool.

No worries, I should have noticed earlier. This is why we weren't preserving one of the JDK bugs, though, so I wanted to fix it.

@LoiNguyenCS next week you should look through the other changes I made in this PR to keep CF-6388 compiling. They're pretty extensive; we had the modularity model for interfaces wrong, too.


chore: address rootulp feedback on PR 1095

Addresses my late feedback on https://github.com/celestiaorg/celestia-app/pull/1095

Codecov Report

Merging #1145 (0455dfc) into main (d24af41) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1145   +/-   ##
=======================================
  Coverage   48.26%   48.26%
=======================================
  Files          71       71
  Lines        3999     3999
=======================================
  Hits         1930     1930
  Misses       1899     1899
  Partials      170      170

Impacted Files
Coverage Δ

x/blob/keeper/keeper.go
81.48% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Assume updated CI role

Follows https://github.com/wellcomecollection/platform-infrastructure/pull/31
Asks the weco-deploy tool to assume the CI role in order to publish images.

For: https://github.com/wellcomecollection/platform/issues/4627


Major rewrite: Get rid of dependencies, combine flavors, match other editors better, offer more customizablity

Rewrite to avoid dependencies for both regular Emacs and Doom Emacs, contain all flavors in the same theme, make it closer to the other editor themes, and make it more customizable (allow user to overwrite any colors).
Since this is such a big rewrite, we should create a tag of before this gets merged.

Should I merge this and raise issues for these messages?

Should I merge this and raise issues for these messages?

I believe @nyxkrage will fix based on the feedback given, no need to rush :D

Tested the theme myself and it looks great ❤

@nyxkrage , thanks for your amazing work on it! Really happy to see the development continues on this version of the lovely theme for Emacs. Since this is a major rewrite, could I suggest that you consider the issue I recently opened about Solaire mode ? https://github.com/catppuccin/emacs/issues/41
Thanks for your work everyone

Added some basic support for solaire-mode which should close #41 and should now be good to merge.

This PR would also close #22 and #18

Did the MELPA release get tagged as 1.0.0? If so, we should probably bump this to 2.0.0 and tag the previous commit.

No, it didn't. I can now though however. At the time I remember saying something along the lines of "who will actually miss it" which was very naive of me


gstreamer needs manual migration to 1.21

Comment:
I noticed that the bot previously failed to migrate to gstreamer 1.21
You'll need to debug it and manually push it through.

@hmaarrfk, the issue was the gst-python package pins against gstreamer weren't very good - I fixed that yesterday.

Nice


Allow for different E2E test directories

This will allow for a cleaner separation between the external network
tests and regular E2E tests, avoiding leaking of the skip/focus details
to other places.
Signed-off-by: Miguel Angel Ajo<EMAIL_ADDRESS>

@mkimuram  ^

I'm breaking something, isn't the default working? https://github.com/submariner-io/shipyard/pull/521/checks?check_run_id=2304358600#step:3:989
It triggers the "test with subctl" somehow.


Renewal problem

Hey there, I tried to test the renewal process by specifying
node.default['acme']['renew'] = 89

The first certificate I got had was issued on:
Monday, 13. February 2017 at 09:52:00
And had an expiry date of:
Sunday, 14. May 2017 at 10:52:00
So I supposed, that if I waited one day, I would get a new certificate, but this did not happen.
Am I getting something wrong, or is this a problem with the renewal process?
Thanks in advance for any help!
japel

Hi Japel,
I could imagine it being rounded down in a way. If it still doesn't work after Wednesday, 15 February 2017 at 10:51:59 then there might be a problem.
Best,
Thijs

@thoutenbos thanks for the quick reply, I'll take a look tomorrow then.
Best regards
japel

Sadly no renewal happened 😭
Is there any logs I could take a look at to gain more information?

It looks like the recepie has to be run to get a renewed certificate.
Is there a way to have a cron job doing the renewal?

@zurchpet oh, that would explain why it did not happen on my case

@japel To clarify, are you not running chef-client on a regular basis on the server in question?

@jldugger no, I wasn't... Haven't tried running it as a daemon, but I will soon:
https://www.packtpub.com/mapt/book/networking-and-servers/9781785287947/1/ch01lvl1sec25/running-chef-client-as-a-daemon

The resource reads the certificate on each converge to check if the "days till expiration" criteria is met.
There are many ways to deploy chef, usually you have a chef-server running (DIY hosting/cloud or as a service at https://manage.chef.io/login). When you provision your nodes in such a setup, by default, chef-client will run as a daemon and reconverge once or twice per hour (you can configure that).
If you use tools like knife-solo or knife-zero, which provide tooling around chef-solo and chef-zero, then you are responsible yourself for reconverging the nodes. If you think about it: each chef resource checks the current state before applying a change, so running chef on a regular interval is like an "integration test" of your production node. If something changes due to failure or manual intervention, chef is "healing" it automatically.
Most people of the recent Ansible bandwaggon don't get this concept and also the official let's encrypt client (now called certbot) does not get it: While it is responsible updating the certificates it does not ensure that your http is running as configured, it just violates the principle of the chef-server philosophy of "single source of truth". Certbot is mainly a tool for people who manually setup nodes (like all the tinkerers out there) but not for serious business/devops.


victor as devDependency - not really maintained - action needed?

What do you think of victor as dev dependencies?
It is pretty unmaintained (last update 6 years ago, some issues open, some PR open)

Thank you for your feedback.
It has been very helpful in speeding up the development process, but I actually had some trouble with unfixed bugs.
When the development progresses a little further, I would like to consider having vector calculation code inside, or changing to another library.
If you know of any good 2d vector library, please let me know.

Thanks for the clarification.
Unfortunately, I don't know any good 2d libraries.

It took some time, but I implemented the calculation process internally and removed victor as a dependency. (v0.5.0)
Thank you for bringing up the issue!


Exception starting Zatsu with Flask v3.0.0

It seems that the app currently is not compatible with the current version of Flask. The following exception occured when trying to start Zatsu:
Traceback (most recent call last):
  File "D:\Users\Andrea\Downloads\ZatsuViewer-master\zatsu.py", line 12, in <module>
    from api import _singletonApp
  File "D:\Users\Andrea\Downloads\ZatsuViewer-master\api.py", line 4, in <module>
    from app import App
  File "D:\Users\Andrea\Downloads\ZatsuViewer-master\app.py", line 32, in <module>
    from message import Message
  File "D:\Users\Andrea\Downloads\ZatsuViewer-master\message.py", line 4, in <module>
    from flask import escape
ImportError: cannot import name 'escape' from 'flask' (C:\Users\Andrea\AppData\Roaming\Python\Python312\site-packages\flask\__init__.py)

As a workaround, it's possible to install a previous version with pip install flask==2.3.3.

This seems to be a duplicate of an issue reported on Discord.

This discord answer fixes it yes?


GIVGarden page - improvements on UX

Needs more info on different types of projects , may be a question mark and a secific icon for each type can added and also used in the project section
We can have the icon of the supporters too with percentage of participation at the bottom of each project (when the supporters number exceeds a particular amount a vertical three dot appears to see more supporters)
this section is a bit confusing need more info on the percentage bar
I think the name of the project should comes at the top then name of the creator (????)
5.We can search only based on the name of the creators. i think it is beneficial to have a section to be able to search by name of the project
this two toggles just sort based on the top project and newest, but should it filter out them (????) as it has toggle and it implies that you can see A or B (top or new) and in that case what is the definition of new (Last 24 hours??)
These are different sections that should be more clear (With a logo and may be  as a separate section in the page not the top menu)

I would also add that clicking for details shouldnt be so hard... It would be nice that if you clicked anywhere in the 4 box that it would take you to the details page

Great feedback. Thanks

Related to #333

We should split them in separate issues.


Potential inspiration bot

Ran across this, it seems to do about 80% of what you want already:
https://github.com/skiprox/eat-shit-bot
There's a few forks that are ahead of this too so it might be worth seeing what they've added

I think for the sole purpose of optimized streaming, we should absolutely read through their code and changes. As far as I understand, ESB doesn't have an easy way to change who you do/not retweet, unless I misread it.

Yeah, I think that's right. I like that they've thought
through daemonization & figured adding an allow/block list would be a
pretty easy next step with moderated tweets being some LOE beyond that.
On Wed, Jul 29, 2020 at 3:42 AM motcreux<EMAIL_ADDRESS>wrote:

I think for the sole purpose of optimized streaming, we should absolutely
read through their code and changes. As far as I understand, ESB doesn't
have an easy way to change who you do/not retweet, unless I misread it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DefendPDXCode/DefendPDX-TwitterBot/issues/7#issuecomment-665493550,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACHJCECFGZIUTY5FRQQNTDDR57HHBANCNFSM4PLAJ6KA
.


Fix Broken AdminAppPropertiesActivity UI test

This is a pull request raised for issue https://github.com/odk-x/tool-suite-X/issues/470
Reason for Fix:
The UI test for the settings screen was broken following a rework of the settings screen for Admin Mode. As a result, the function that sets up the admin mode to enable the test run was broken, the strings to be matched weren't referenced correctly and the position of the item to be clicked were out of order. this raised the need for some refactoring to fix the broken UI test.

**Changes made to the BaseUITest class
---Fixed the enableAdminMode() function to correctly locate the enable admin setting button, click on it and enter the admin password
**Changes made to the AdminAppPropertiesActivityTest class
1 Corrected the referenced to the matched strings in the following tests:
-checkIfChangeAdminPasswordScreen_isVisible()
-checkIfManageAbilityToChangeServerSettingScreen_isVisible()
-checkIfManageAbilityToChangeDeviceSettingScreen_isVisible()
-checkIfManageAbilityToChangeTableSpecificSettingScreen_isVisible()
-checkIfResetConfigurationScreen_isVisible()
-checkIfExitAdminModeScreen_isVisible()
-checkIfVerifyUserPermissionScreen_isVisible()
2 Corrected the position of item on the list for
-checkIfChangeAdminPasswordScreen_isVisible()
-checkIfManageAbilityToChangeServerSettingScreen_isVisible()
-checkIfManageAbilityToChangeDeviceSettingScreen_isVisible()
-checkIfManageAbilityToChangeTableSpecificSettingScreen_isVisible()
-checkIfResetConfigurationScreen_isVisible()
-checkIfExitAdminModeScreen_isVisible()
-checkIfVerifyUserPermissionScreen_isVisible()
3 Expanded the scope of test to match the summary description for
-checkIfChangeAdminPasswordScreen_isVisible()
-checkIfManageAbilityToChangeServerSettingScreen_isVisible()
-checkIfManageAbilityToChangeDeviceSettingScreen_isVisible()
-checkIfManageAbilityToChangeTableSpecificSettingScreen_isVisible()
-checkIfResetConfigurationScreen_isVisible()
-checkIfVerifyUserPermissionScreen_isVisible()

@Redeem-Grimm-Satoshi awaiting your review

Great job!
Haven't tested this yet, provide a screenshot showing how it works

Hi @Redeem-Grimm-Satoshi I've  added the screenshot(s) as requested.


feat(comment-edit): Auto-close comment block when hitting enter after /*.

Automatically close the comment block by inserting *, */ when pressing enter after opening a block comment /*.
The new behaviour looks like this:

Also added move_ methods to TextEditor façade and removed now obsolete tricks to close the comment by adding a / on the last line and hitting enter.
Closes #827

lgtm
fix https://github.com/dart-atom/dartlang/issues/350


Broken link in README

The link to https://cloud.google.com/showcase/experiments is a 404! Oh no!

It's now showcase.withgoogle.com/experiments


Website fails on https and also does not work on ip address.

The static site which when I try to open on my localhost, works without any hiccups but it is under http only so no camera accessibility or mic. Whereas when I try to use my ip address, it seemingly crashes on loading into https and never reloads at all. Is it only with me? I followed the steps which are provided in the documentation  for installation.

@Teja-sai-08
Just run npm start  and navigate to http://localhost:1337 unless you dont have process.env set up .
localhost should not give you probelms accessig camera etc .
you can also run npm run start:secure , that fires an https:// host on local using the self-signed certs,  in this case server is started on port 4433 (see server.ts) , then navigate to https://localhost:4433/  or https://your-ip::4433/

@Teja-sai-08 as no comment from you , i'm closing this isssue (question ) as i concider it working, as described above.


Update description of JsonApiFramework client and server side implementations to latest version

Signed-off-by: Scott McDonald<EMAIL_ADDRESS>

Thanks!


Return probabilities (not just best match)

What do you suggest is the best way to get all probability values from the classifier rather than the best match only? For instance an Array or Object of all classifications sorted by probability.
classifier.classify('there is something wrong with my computer system')
// software: 0.8, hardware: 0.9

Use the getClassifications function instead:

classifier. getClassifications('there is something wrong with my computer system')


Add logic to fetch prometheus rules from clusters

Prometheus rules are obtained from managed clusters to subsequently fill out the alarm dictionaries

/approve


Move to GitHub Actions

Closes https://github.com/exercism/ceylon/issues/100
The Exercism organisation is moving to GitHub Actions.
It runs faster.

Nice work. Thanks @petertseng!!


[stable/jenkins] Allow templating of ServiceAccount annotations

What this PR does / why we need it:
Allow templating of serviceAccount.annotations and serviceAccountAgent.annotations by rendering them with the tpl function
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
A use case for templating ServiceAccount annotations is the use of a ServiceAccount as an OAuth client in OpenShift.
In this example, the Route name is templated so the OAuth redirect can point to a Route in the release rather than being pinned to a hardcoded Route.
serviceAccount:
  annotations:
    serviceaccounts.openshift.io/oauth-redirectreference.jenkins: |-
      {
        "kind": "OAuthRedirectReference",
        "apiVersion": "v1",
        "reference": {
          "kind": "Route",
          "name": "{{ template "jenkins.fullname" . }}"
        }
      }

helm template test
Given the above yaml in stable/jenkins/sa-values.yaml
helm template stable/jenkins \
--show-only templates/service-account.yaml \
--values stable/jenkins/sa-values.yaml

# Source: jenkins/templates/service-account.yaml
apiVersion: v1
kind: ServiceAccount
metadata:
  name: RELEASE-NAME-jenkins
  namespace: default
  annotations:
    serviceaccounts.openshift.io/oauth-redirectreference.jenkins: |-
      {
        "kind": "OAuthRedirectReference",
        "apiVersion": "v1",
        "reference": {
          "kind": "Route",
          "name": "RELEASE-NAME-jenkins"
        }
      }
  labels:
    "app.kubernetes.io/name": 'jenkins'
    "helm.sh/chart": "jenkins-1.17.2"
    "app.kubernetes.io/managed-by": "Helm"
    "app.kubernetes.io/instance": "RELEASE-NAME"
    "app.kubernetes.io/component": "jenkins-master"

Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

@wmcdona89 Are you interested in becoming a maintainer of this chart?

@torstenwalter Yes, I'm interested in helping maintain this chart. BTW...I'm nearing the end of my list of PRs to open and thanks for helping me get all of these merged :-)

@wmcdona89 great to hear. Please add yourself as maintainer in Chart.yaml and OWNERS file.

@torstenwalter I added myself.

/ok-to-test

Added a templated description.
  annotations:
    description: "Used by release {{ .Release.Name }} for role-based access control"


Feature/consume history

Fixes #126
Changes proposed in this PR:

user's orders history table created and displayed in History

Looks good, remove the title column.
We removed some auto renaming of the classes generated by apollo so you need to fix this  has no exported member named 'OrdersDataTokenOrders'. Did you mean 'OrdersData_tokenOrders'
The big question now is should we leave this as a table with 2 columns or change it to a tile grid like publish (both will be paginated when the other pr is done)

#126 should be linked already, isn't it showing?
What about adding a 'Datatoken' column instead of title?

I'm sorry, didn't see the issue in the description and i presumed it's not linked, my bad.
Sure, Datatoken may have some value

This works great!
I would at least switch the Compute Jobs & Orders tables since almost nobody will have anything listed in Compute Jobs.
And for now the table title could be Downloads which makes it more clear what this table is.

You can't show the price, you don't know when the dt was bought and when the asset was consumed. Basically i can buy from the pool 10 dt and after a month i use 1 dt to consume the asset. What is the price? The price at the moment of consumption or when i actually bought the dt? Furthermore you can't actually link the event in the background, only lets say by time proximity.

good points. So then I think this PR is ready-to-go after we remove the Orders.module.css file. If we don't need any further styling we can always leave out css files for a component

done! sorry i forgot about that


Update container-instances-virtual-network-concepts.md

Fixing spelling error on line 38, "Government" instead of "Governemment"

Looks good. Thanks @reigntastic


Have a special account type for accounts that don't hold state.

There is often a need to pass accounts which don't hold any state (e.g. when passing a mint authority). Using AccountInfo type for these accounts is not the best because there are no checks done and since anchor v0.22.0 it's marked as unsafe and should be avoided. It's better to use SystemAccount which will check that the account is owned by the system program.
But SystemAccount won't check that the account is uninitialized (space and lamports are 0).
In theory it probably doesn't make a difference, but to be extra pendantic I thought it might be useful to have an account type which would check that the account is uninitialized.
Also because using SystemAccount for this use case kind of seems off because by the sound of it implies that we require this account should be owned bye the system program, instead of the real intention - this is an uninitialized account that doesn't hold any state, we just use it for the address (we don't care that system program owns it, just that it's uninitialized). Again, in theory it's probably not any different but could make the intention more clear.

could you elaborate on your use case with the mint authority?
So what youd like to have is an UnitializedAccount that does the following checks?:

space = 0
lamports = 0

Should it also check that owner = SystemProgram?

I thought about this a bit more and would like to formulate it a bit differently than in the original post.
The use case is any time you need to pass an account which doesn't necessarily store anything but is used for signing purposes.

For example you have a program that has a vault and in order to be able to withdraw from this vault you need to sign the tx with the vault authority/admin/etc. This authority key is an account which doesn't store any data, we just use it's keys for signing. Of course in the "withdraw" instruction you would mark it as Signer, but for the "initialize vault" instruction you might not require it to sign the instruction, you only  need to pass the public key to store it (in order to check against it in the "withdraw" instruction). So which type do you give to this account in the ctx? AccountInfo? SystemAccount?

Another similar example are PDA authorities. Suppose you have a PDA that has a mint authority on a mint. In order for the program be able to issue a mint CPI call you need to pass the PDA address as an account in the instruction. But this account doesn't store anything, it's only used to sign the CPI call. Which type should this account have in the ctx? AccountInfo? SystemAccount?

If you use AccountInfo you will get a compiler warning and it's use is discouraged.
SystemAccount could work, but there are some cases where this account also stores data in which case it isn't owned by the system program. So you could argue that in this case you would use Account<'info, ...>, but suppose in this instruction we don't care about the data it stores, we only care that it signs the instruction and we would like to avoid deserializing this data or making it accessible in this instruction for simplicity's sake. Then you say "OK in that case you use Signer". But Signer doesn't work if the account is a PDA (use case 2), or when we just want to pass it to store it (use case 1).
So, of course, you can work your way around it in these scenarios, but what I'm saying is, there doesn't seem to be any account type to signal the intent: "I'm passing this public key, it may or may not be initialized and store data - I don't care don't deserialize it, all I care is it it's a key. It could be a PDA pubkey that I need to make a CPI call or I'm passing it to store it to do Signer checks against it later on".

So, of course, you can work your way around it in these scenarios, but what I'm saying is, there doesn't seem to be any account type to signal the intent: "I'm passing this public key, it may or may not be initialized and store data - I don't care don't deserialize it, all I care is it it's a key. It could be a PDA pubkey that I need to make a CPI call or I'm passing it to store it to do Signer checks against it later on".

This is what UncheckedAccount is for. We could add types like UncheckedAccountForCPI but we thought that the /// CHECK: would be better than introducing more types that are the same as UncheckedAccount except in their signaled intention


Wrong angles in acffeature.cpp:computeChannels

cartToPolar return angles between (0,360) but this function works around cartToPolar returning (-180,180)

Can you please check if this is still the case for your code and provide a sample please? Because I have it running right here without any problems and returning angles correctly.

Can't test the code at the moment but this fix https://github.com/Itseez/opencv_contrib/commit/d76b976012cf47eb90720b1cd6a06fa664f20425#diff-64782f0669713a552a2e3c9c2f61524c should have fixed it.

:+1: then this should be closed down :)

Yep! Hadn't seen that commit :+1: