Error attempting to claim book from newsletter

~ $ python script/spider.py --config config/prod.cfg --notify ifttt --claimOnly

                      __   __              __                                   __
    ____  ____ ______/ /__/ /_____  __  __/ /_        ______________ __      __/ /__  _____
   / __ \/ __ `/ ___/ //_/ __/ __ \/ / / / __ \______/ ___/ ___/ __ `/ | /| / / / _ \/ ___/
  / /_/ / /_/ / /__/ ,< / /_/ /_/ / /_/ / /_/ /_____/ /__/ /  / /_/ /| |/ |/ / /  __/ /
 / .___/\__,_/\___/_/|_|\__/ .___/\__,_/_.___/      \___/_/   \__,_/ |__/|__/_/\___/_/
/_/                       /_/

Download FREE eBook every day from www.packtpub.com
@see github.com/niqdev/packtpub-crawler

[*] 2017-01-31 10:30 - fetching today's eBooks
[*] configuration file: /app/config/prod.cfg
[*] getting daily free eBook
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/account/my-ebooks
[+] book successfully claimed
[+] notification sent to IFTTT
[*] getting free eBook from newsletter
[*] fetching url... 200 | https://www.packtpub.com/packt/free-ebook/practical-data-analysis
[-] <type 'exceptions.IndexError'> list index out of range | spider.py@123
Traceback (most recent call last):
  File "script/spider.py", line 123, in main
    packtpub.runNewsletter(currentNewsletterUrl)
  File "/app/script/packtpub.py", line 160, in runNewsletter
    self.__parseNewsletterBookInfo(soup)
  File "/app/script/packtpub.py", line 98, in __parseNewsletterBookInfo
    title = urlWithTitle.split('/')[4].replace('-', ' ').title()
IndexError: list index out of range
[+] error notification sent to IFTTT
[*] done
~ $

It has successfully claimed the book from the newsletter already, but on subsequent days I'm getting the above error.
And it sends an IFTTT notification for the second one :(

Updated to the latest code from master of this repo. Issue still present.
~ $ python script/spider.py --config config/prod.cfg --claimOnly

                      __   __              __                                   __
    ____  ____ ______/ /__/ /_____  __  __/ /_        ______________ __      __/ /__  _____
   / __ \/ __ `/ ___/ //_/ __/ __ \/ / / / __ \______/ ___/ ___/ __ `/ | /| / / / _ \/ ___/
  / /_/ / /_/ / /__/ ,< / /_/ /_/ / /_/ / /_/ /_____/ /__/ /  / /_/ /| |/ |/ / /  __/ /
 / .___/\__,_/\___/_/|_|\__/ .___/\__,_/_.___/      \___/_/   \__,_/ |__/|__/_/\___/_/
/_/                       /_/

Download FREE eBook every day from www.packtpub.com
@see github.com/niqdev/packtpub-crawler

[*] 2017-01-31 10:36 - fetching today's eBooks
[*] configuration file: /app/config/prod.cfg
[*] getting daily free eBook
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/account/my-ebooks
[+] book successfully claimed
[*] getting free eBook from newsletter
[*] fetching url... 200 | https://www.packtpub.com/packt/free-ebook/practical-data-analysis
[-] <type 'exceptions.IndexError'> list index out of range | spider.py@123
Traceback (most recent call last):
  File "script/spider.py", line 123, in main
    packtpub.runNewsletter(currentNewsletterUrl)
  File "/app/script/packtpub.py", line 160, in runNewsletter
    self.__parseNewsletterBookInfo(soup)
  File "/app/script/packtpub.py", line 98, in __parseNewsletterBookInfo
    title = urlWithTitle.split('/')[4].replace('-', ' ').title()
IndexError: list index out of range
[*] done

Can confirm, I'll look into it. For a quickfix, create a file named "lastNewsletterUrl" containing "https://www.packtpub.com/packt/free-ebook/practical-data-analysis" in the config folder. This should stop the errors for now.

Fixed in https://github.com/niqdev/packtpub-crawler/pull/48

nice. thanks 👍

Getting this error now after updating : on Version  - 2.2.4
[+] new download: /home/david/packtpub-crawler//home/david/packtpub-crawler/ebooks/extras/MySQL_for_Python.zip
[*] getting free eBook from newsletter
[*] fetching url... 200 | https://www.packtpub.com/packt/free-ebook/practical-data-analysis
[-] <type 'exceptions.IndexError'> list index out of range | spider.py@123
Traceback (most recent call last):
  File "script/spider.py", line 123, in main
    packtpub.runNewsletter(currentNewsletterUrl)
  File "/home/david/packtpub-crawler/script/packtpub.py", line 160, in runNewsletter
    self.__parseNewsletterBookInfo(soup)
  File "/home/david/packtpub-crawler/script/packtpub.py", line 98, in __parseNewsletterBookInfo
    title = urlWithTitle.split('/')[4].replace('-', ' ').title()
IndexError: list index out of range
[*] done

Yes, this is the same error. The fix should be merged tonight hopefully or you can change the line yourself.

Sorry just noticed the comment above about created the "lastNewsletterUrl" that worked.

Thanks @juzim , the problem above is fixed, but actually there is a bug, see the log below (I've hidden some variables/paths)

upload 2 times the ebook from the newsletter with 2 different name (but the file is the same)
I received 2 email, the first one is correct, the second again contains 2 link with 2 pdf but are the same

python script/spider.py -c config/prod.cfg -u drive -s firebase -n gmail

                      __   __              __                                   __
    ____  ____ ______/ /__/ /_____  __  __/ /_        ______________ __      __/ /__  _____
   / __ \/ __ `/ ___/ //_/ __/ __ \/ / / / __ \______/ ___/ ___/ __ `/ | /| / / / _ \/ ___/
  / /_/ / /_/ / /__/ ,< / /_/ /_/ / /_/ / /_/ /_____/ /__/ /  / /_/ /| |/ |/ / /  __/ /
 / .___/\__,_/\___/_/|_|\__/ .___/\__,_/_.___/      \___/_/   \__,_/ |__/|__/_/\___/_/
/_/                       /_/

Download FREE eBook every day from www.packtpub.com
@see github.com/niqdev/packtpub-crawler

[*] 2017-01-31 18:45 - fetching today's eBooks
[*] configuration file: XXX/github/packtpub-crawler/config/prod.cfg
[*] getting daily free eBook
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/account/my-ebooks
[+] book successfully claimed
[-] downloading file from url: https://www.packtpub.com/ebook_download/12890/pdf
[################################] 9985/9985 - 00:00:01
[+] new download: XXX/packtpub-crawler/ebooks/MySQL_for_Python.pdf
[+] new file upload on Drive:
[+] uploading file...
-[+] updating file permissions...
-       [path] XXX/packtpub-crawler/ebooks/MySQL_for_Python.pdf
	[download_url] UUU
	[name] MySQL_for_Python.pdf
	[mime_type] application/pdf
	[id] III
[+] Stored on firebase: KKK
[+] notified to: ['aaa', 'bbb']
[*] getting free eBook from newsletter
[*] fetching url... 200 | https://www.packtpub.com/packt/free-ebook/practical-data-analysis
[*] fetching url... 200 | https://www.packtpub.com/account/my-ebooks
[+] book successfully claimed
[-] downloading file from url: https://www.packtpub.com/ebook_download/12890/pdf
[################################] 9985/9985 - 00:00:01
[+] new download: XXX/packtpub-crawler/ebooks/Practical_Data_Analysis.pdf
[+] new file upload on Drive:
[+] uploading file...
|[+] updating file permissions...
-       [path] XXX/packtpub-crawler/ebooks/MySQL_for_Python.pdf
	[download_url] https://drive.google.com/uc?id=ZZZ
	[name] MySQL_for_Python.pdf
	[mime_type] application/pdf
	[id] LLL
[+] new file upload on Drive:
[+] uploading file...
|[+] updating file permissions...
\       [path] XXX/packtpub-crawler/ebooks/Practical_Data_Analysis.pdf
	[download_url] https://drive.google.com/uc?id=DDD
	[name] Practical_Data_Analysis.pdf
	[mime_type] application/pdf
	[id] YYY
[+] Stored on firebase: WWW
[+] notified to: ['aaa', 'bbb']
[*] done

Actually I started the script again and it seems that the 3 book are identical and the daily ebook is ignored and I have 3 copy of the same book (newsletter). Moreover I checked on firebase and the data uploaded are mixed e.g. different filename but same author (of the newsletter).

I think we have to reset the Packtpub instance before handling the newsletter but then this would have been broken for weeks. Could you check?
Also, could you comment out spider.py:22 so we can see the contents of each handled packtpub.info?

Yep,
so first log (see the author is wrong) and noticed now paths is empty
...
[*] getting daily free eBook
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/packt/offers/free-learning
[*] fetching url... 200 | https://www.packtpub.com/account/my-ebooks
{
  "url_source_code": "https://www.packtpub.com/code_download/12891",
  "paths": [],
  "description": "MySQL for Python is the essential ingredient for building productive and feature-rich Python applications as it provides powerful database support and will also take the burden off your webserver. This eBook shows how to boost the productivity and maintainability of your Python apps by integrating them with the MySQL database server. It will take you from installing MySQL for Python on your platform of choice all the way through to database automation and administration. Every chapter is illustrated with a practical project that you can use during your own app development process. This eBook is free for today only so don\u2019t miss out!",
  "title": "MySQL for Python",
  "author": "Hector Cuesta",
  "filename": "MySQL_for_Python",
  "book_id": "12890",
  "url_claim": "https://www.packtpub.com/freelearning-claim/5286/21478",
  "url_image": "https://dz13w8afd47il.cloudfront.net/sites/default/files/imagecache/dotd_main_image/0189OS_MockupCover_0.jpg"
}
[+] book successfully claimed
...

and second log
...
[*] fetching url... 200 | https://www.packtpub.com/account/my-ebooks
{
  "url_source_code": "https://www.packtpub.com/code_download/12891",
  "paths": [
    "XXX/packtpub-crawler/ebooks/MySQL_for_Python.pdf"
  ],
  "description": "Get started in data analysis with this free 360 page eBook guide\nFor small businesses, analyzing the information contained in their data using open source technology could be game-changing. All you need is some basic programming and mathematical skills to do just that. This free data analysis eBook is designed to give you the knowledge you need to start succeeding in data analysis. Discover the tools, techniques and algorithms you need to transform your data into insight.\n\nVisualize your data to find trends and correlations\nBuild your own image similarity search engine\nLearn how to forecast numerical values from time series data\nCreate an interactive visualization for your social media graph",
  "title": "Practical Data Analysis",
  "author": "Hector Cuesta",
  "filename": "Practical_Data_Analysis",
  "book_id": "12890",
  "url_claim": "https://www.packtpub.com/promo/claim/12891/27564",
  "url_image": "https://d1ldz4te4covpm.cloudfront.net/sites/default/files/B02731_Practical Data Analysis.jpg"
}
[+] book successfully claimed
...

About the first question, we may have to reset everything before the claim, but is this the first newsletter since they reactiveted the free ebook? The other book seems correct, I checked also the logs on heroku

Also the field url_source_code is wrong

Can you add packtpub = Packtpub(config, args.dev) to spider.py:123 and see if it fixes it? Sorry, but somehow the tests on my machine are broken...

Did you by any chance delete the lastNewsletterUrl file? Because if the script tries to grab an already claimed newsletter book from the archive, it won't find it on the top position and overwrite the data for whatever book is currently on there.
We have to either check if the book was already claimed (would be a nice feature anyways) or find the book in the list by name/id/etc.
I'll try to look into it tomorrow but it shouldn't happen again unless you delete the file.

I'm testing now, using docker, with the change that you suggested i.e. XXX I have the following error
...
[+] book successfully claimed
[+] created new directory: /packtpub-crawler/ebooks
[-] downloading file from url: https://www.packtpub.com/ebook_download/12890/pdf
[+] new download: /packtpub-crawler/ebooks/MySQL_for_Python.pdf
[+] new file upload on Drive:
\[+] uploading file...
\[+] updating file permissions...
/Traceback (most recent call last):
  File "script/spider.py", line 124, in main
    packtpub.runNewsletter(currentNewsletterUrl)
  File "/packtpub-crawler/script/packtpub.py", line 160, in runNewsletter
    self.__parseNewsletterBookInfo(soup)
  File "/packtpub-crawler/script/packtpub.py", line 105, in __parseNewsletterBookInfo
    self.info['url_claim'] = self.__url_base + claimNode[0]['href']
IndexError: list index out of range
	[path] /packtpub-crawler/ebooks/MySQL_for_Python.pdf
...

When we reset the whole packtpub we also loose the login information, so it won't work. I added a method to reset the packtpub.info but this won't solve the other issue.

We should probably just reset in Packtpub.py
self.info = {
  'paths': []
}

No same problem, we lose the session

I think the solution is to see if the book is already claimed before further processing it. The claim response page contains an error message that we can parse. I try to submit a patch tomorrow.

About your solution I just don't like the fact that we have to do another request, we should be able to reset all the fields before.
This is just my thought, but this is were mutable state sucks (we are also missing tests) and a purely functional approach would help us a lot. By the way, I'm not gonna rewrite anything..haha

We don't need another request, the claim request returns the archive with
the error message (if the book was already claimed). So it's just another
check in get_claim that will throw an exception that prevents further
processing.
This has some small downsides but will fix your issue.
niqdev<EMAIL_ADDRESS>schrieb am Mi., 1. Feb. 2017, 09:14:

About your solution I just don't like the fact that we have to do another
request, we should be able to reset all the fields before.
This is just my thought, but this is were mutable state sucks (we are also
missing tests) and a purely functional approach would help us a lot. By the
way, I'm not gonna rewrite anything..haha
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/niqdev/packtpub-crawler/issues/47#issuecomment-276597493,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEmPPA9AkfIBUjuV_QpYgZTOPphG9nYfks5rYD7rgaJpZM4Lydlb
.

While I'm not a fan of flow control by exceptions, I think it works just fine in this case. Your code is solid and can surely handle quite some more beating before a rewrite would be necessary.
Fixed in https://github.com/niqdev/packtpub-crawler/pull/50
This fix looks for a specific error message on the claim result page (which curiously only exists for the newsletter, not the dailies) which should work for now. No additional requests are made.
Assuming that the first entry in the archive is always the book we are processing at the moment might cause further trouble (packtpub might switch to alphabetic sorting for example). But the only way to see if the book was already claimed in the archive right now is searching by name which is prone to errors since we parse the title from the claim URL.
In my opinion, the right way to do this is not parsing the latest entry, but synchronizing the archive with the local downloads folder (stepping through all books on the page and downloading those that are missing) after a claim. Since we can generate the file name from the list entry title instead of the claim URL this way, we can securely match them.
This would also resolve https://github.com/niqdev/packtpub-crawler/issues/23
Any volunteers? :)

In my opinion, the right way to do this is not parsing the latest entry, but synchronizing the archive with the local downloads folder (stepping through all books on the page and downloading those that are missing) after a claim.

How would that work if you're running with --claimOnly?

The script would just claim the book and you can download it later manually or run it with a "downloadAll" parameter that only syncs the archive with the local folder. Notifications etc are handled on claim, not download.

@juzim , I'll keep monitoring until next newsletter. I'll create tag 2.2.5.
About your solution with the local search is fine, about me, how you have seen, I haven;t much time in this period and working on other projects aswell. If you leave it there I may do it, just don;t know when.
Thanks

Just to keep track, it needs further investigation.
Anyway sometimes the script is able to download the newsletter, while for example this week there is this error
[*] fetching url... 200 | https://www.packtpub.com/packt/free-ebook/what-you-need-know-about-angular-2
[-] <type 'exceptions.IndexError'> list index out of range | spider.py@125
Traceback (most recent call last):
  File "script/spider.py", line 125, in main
    packtpub.runNewsletter(currentNewsletterUrl)
  File "PATH/packtpub-crawler/script/packtpub.py", line 169, in runNewsletter
    self.__parseNewsletterBookInfo(soup)
  File "PATH/packtpub-crawler/script/packtpub.py", line 101, in __parseNewsletterBookInfo
    urlWithTitle = div_target.select('div.promo-landing-book-picture a')[0]['href']
IndexError: list index out of range

I bet they are doing a/b tests which makes it hard to reproduce. I think
claiming still works despite the error, can you confirm?

No, unfortunately the claiming is not working too.

The div promo-landing-book-picture doesn't  exists

That's it?! I'll try to fix it soon but it might take till next week,
sorry.
niqdev<EMAIL_ADDRESS>schrieb am So., 2. Apr. 2017, 11:10:

The div promo-landing-book-picture doesn't exists
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/niqdev/packtpub-crawler/issues/47#issuecomment-290974349,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEmPPB4hdhLsEjGopseM72lUW5HhNEgvks5rr2YVgaJpZM4Lydlb
.

Looks like some of the divs has been renamed on the newsletter's landing page. I compared the page for an older book:

Claim your free 416 page Amazon Web Services eBook!

This book is a practical guide to developing, administering, and managing applications and infrastructures with AWS. With this, you'll be able to create, design, and manage an entire application life cycle on AWS by using the AWS SDKs, APIs, and the AWS Management Console.

with the current one:

What you need to know about Angular 2

Get to grips with the ins and outs of one of the biggest web dev revolutions of this decade with the aid of this free eGuide! From setting up the very basics of Angular to making the most of Directives and Components you’ll discover everything you need to get started building your own web apps today.

                                What You Will Learn

and came up with this hotfix:
https://github.com/niqdev/packtpub-crawler/compare/master...mkarpiarz:fix_newsletter_divs
I haven't tested email notifications yet, so I'm not sure how the description would look like, but claiming a newsletter ebook seems to work now. Happy to submit a PR if @juzim haven't started working on this yet.

That would be great, thanks!

Hi Guys,
I'm creating google script that parsing PacktPab tweets(it comes from @juzim google script). I'm not sure but there is a chance that all books from newsletters also will be published on their Twitter and no needs to fix it :) joking.
It's not finished - should exclude duplicates and check does link still available or not. If you have time, please look on output if it's fine for crawler or not https://goo.gl/AXtAC8

The link doesn't work for me, can you create a pull request please?
Also, while there are tons of free books on the feed, they repeat a lot so we have to make sure the duplication check works.

Is there a reason for the newsletter spreadsheet being empty even though this week's free ebook is still available under https://www.packtpub.com/packt/free-ebook/what-you-need-know-about-angular-2?

@mkarpiarz before merge the PR can you please confirm that the email/notifications are still working? Thanks

@mkarpiarz I removed it to prevent error messages until the issue is fixed

@juzim - that's fine for now since there is an option to self-host the file.
I haven't tested email notification yet, @niqdev, but I printed out all the variables in this __parseNewsletterBookInfo method and I noticed this in the output:
self.info['title']: u'5612_Wyntkangular_Ebook_500X617.Jpg'
self.info['filename']: '5612_Wyntkangular_Ebook_500X617.Jpg'

This is because the code from:
https://github.com/niqdev/packtpub-crawler/blob/e604cc1138c5934f7cbe8c210ce1fa6f2caa80b3/script/packtpub.py#L101
is extracting book titles based on the url inside the class where the book cover is and for this week's free ebook the relevant part looks like this:

So there is no link with the book title and instead the url points to the location of the cover image.
I'll create a separate thread for this title parsing issue.


Displays disconnect after some time and become CPU-intensive

I use this driver to get 2 extra monitors in addition my one hardware monitor. It is very useful in VR, and typically works without issues. Since installing the driver on 17 May, I've ran into this issue at least 3 times.
After playing VR for some time (4d15h according to task manager), I will get a short lag spike and all displays turn gray on my VR overlay (OVR Toolkit). The "display disconnected" noise is played by Windows, and roughly a second later, they all automatically reconnect again. After reconnecting, the 'Windows Driver Foundation' process tends to hover between 1% and 16% CPU usage for a minute (taskmanager) (intel i7-14700K). Some time later, this decreases to between 3% and 7% CPU. It also uses 13-18MB of memory.
Disabling the driver in device manager drops the CPU usage to a consistent 0%, and memory to 2.2MB. Reactivating it brings it back to 3-7% CPU and 13-18MB memory.
Restarting the PC typically resets this issue, but it tends to happen again within a few days.
I added some resolutions and framerates in my options.txt file but didn't change anything else.
I'm using Windows 11 (Home?), not activated yet.
Connected monitors:

1920x1080@60Hz | S24R35x (Hardware Samsung display)
1024x768@60Hz
1280x800@30Hz

Hardware:

Motherboard: Z790 Pro RS
CPU: Intel Core i7-14700K
GPU: NVIDIA RTX 4070 Super
RAM: 32GB DDR5
SSD: 2TB M.2
Connected through ethernet

Typically open applications:
Task manager, File Explorer, Firefox, Discord, Spotify, Steam, SteamVR, Visual Studio (Code), VRCX, VRChat, OVR Toolkit, fpsVR, Command Prompt. Sometimes Unity
These programs tend to have been open for a while. This issue occurred while simply scrolling through a webpage.

After a restart, the CPU consumption is 1.5 - 5% (with spikes of 8%) with 12-14MB memory.
A morning after, the CPU-usage seemed steady around 0-1.1% (with spikes of 1.7%) with 14-18.5MB memory.
In the afternoon, now using the overlay more intensely after sleeping in vr, the CPU usage rose to be an average 1-5% (with spikes of 8%) with 14-17MB memory.
The day after, after idling my pc outside of vr, the CPU of the Windows Driver Foundation was at 14-25% again.

By any chance if your boot GPU set to your IGPU instead of your actual gpu ?

On my desktop pc, I only have my 4070 Super, no iGPU

If you can reproduce this issue on the latest beta release, that would be greatly appreciated!
@MysticMia


Reactive/RxSwift support

Are there any plan to support Reactive/RxSwift for APIKit?

APIKit doesn't include Rx support at this time, but you can implement it easily like this.


fix jukebox issues introduced from mypy pr

Codecov Report

Merging #1387 (de0614e) into main (dcf5f0a) will decrease coverage by 0.12%.
The diff coverage is 41.17%.

@@            Coverage Diff             @@
##             main    #1387      +/-   ##
==========================================
- Coverage   48.68%   48.57%   -0.12%
==========================================
  Files         294      294
  Lines       16562    16644      +82
==========================================
+ Hits         8064     8085      +21
- Misses       8498     8559      +61

Impacted Files
Coverage Δ

lnbits/extensions/jukebox/views.py
51.72% <0.00%> (-3.84%)
:arrow_down:

lnbits/extensions/jukebox/views_api.py
16.81% <0.00%> (-0.64%)
:arrow_down:

lnbits/extensions/jukebox/models.py
100.00% <100.00%> (ø)

lnbits/extensions/lnurlp/migrations.py
81.48% <0.00%> (-14.82%)
:arrow_down:

lnbits/db.py
81.20% <0.00%> (-10.07%)
:arrow_down:

lnbits/commands.py
73.61% <0.00%> (-1.39%)
:arrow_down:

lnbits/core/views/generic.py
57.34% <0.00%> (-0.21%)
:arrow_down:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


feat : Add functionality in test file to store crawler results

Description
In the current approach, the test_res file is over written while testing all the crawlers. While this is an efficient approach, it becomes hard to debug at times when the tests fail as developer does not have access to the test_res file which is compared with the sample output.
This PR adds an additional parameter in verify function called store_results. If this parameter is set to True by developer during function call, it stores the result of the test crawler, which can be useful to analyze if test results fail.
Below is an example of how the new verify function can be called.
def test_compute_instance_name(self):
    """Test compute instance name."""
    self.assertTrue(
      verify(
        CrawlerFactory.create_crawler(
          "compute_instances",
        ).crawl(
          PROJECT_NAME,
          ClientFactory.get_client("compute").get_service(self.credentials),
        ),
        "compute_instances",
        True,
        True,
      )
    )

NOTE : In this PR, store_results has been set to True for 2 resources - compute_instances and services

Hi @mshudrak, based on the experience in #305 where it was difficult to figure out why the tests were failing, I thought of coming with this PR to modify the verify function, so that it is easier for the developer to debug and compare files.
Do let me know your thoughts on this PR, on whether it adds any value or not.

Hi, thanks for the PR. We currently just print to console when the test is failing. No need to store that in the fail.

Got it. Closing this PR.


Organization support?

Hi,
I see you're filtering the repos only owned by the current user, any reasons behind this reasoning?

Hey, I'd like to see Oraganization support too, since I am coding mostly on a org and only like testing stuff on my private profile

Hi. In general, technically there are no restrictions on getting organizations. I need to somehow understand if the current user has access to traffic api or not. I'll add this soon.

@Noxits @DTheIcyDragon
Hi, again. Just released new version – it should now load organisations repos too (if repo is public).
Check please

Sweet, thanks, works like a charm now!


Correct punctuation, capitalization and grammar (English)

Rename "Image reverse search" to "Reverse Image Search" or "Image Reverse Search" to adhere to proper punctuation in English

Thanks! English is not my native language, so just say when you find other errors.


Add Series.transform

This seems virtually an alias of Series.apply (if it's in Series). I revised the doc a bit so that it can be used in DataFrames too in the future.

Codecov Report

Merging #428 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #428      +/-   ##
==========================================
- Coverage   94.73%   94.69%   -0.05%
==========================================
  Files          42       42
  Lines        4746     4747       +1
==========================================
- Hits         4496     4495       -1
- Misses        250      252       +2

Impacted Files
Coverage Δ

databricks/koalas/missing/series.py
100% <ø> (ø)
:arrow_up:

databricks/koalas/series.py
93.22% <100%> (+0.03%)
:arrow_up:

databricks/koalas/tests/test_indexes.py
94.33% <0%> (-1.89%)
:arrow_down:

Continue to review full report at Codecov.

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

let me merge. Thanks @ueshin!


The map print result can not zoom to page size setting

Steps to reproduce

Add a shape layer to map
click "print layout"
click "Choose Printer"
set the page size with 600mm*600mm
5.apply the printer setting
6.print map

DotSpatial version:  1.9
when i choose my printer and set the pagesize as 600mm*600mm, then i print the map,and the print result is wrong. the screen shot is show as bellow:
1.

2.

Expected behaviour
may be, the print map should be fill the 600*600 page,someone can help me?thanks!

maybe your page orientation is landscape

@skylning did you solved issue?

No


[ASK] How to create test submission for MIND competition

Description
I've been trying to use the nrms_MIND notebook to create test submission by doing the following:
test_news_file = os.path.join(data_path, 'test', r'news.tsv')
test_behaviors_file = os.path.join(data_path, 'test', r'behaviors.tsv')
group_impr_indexes, group_labels, group_preds = model.run_fast_eval(test_news_file, test_behaviors_file)

with open(os.path.join(data_path, 'prediction.txt'), 'w') as f:
    for impr_index, preds in tqdm(zip(group_impr_indexes, group_preds)):
        impr_index += 1
        pred_rank = (np.argsort(np.argsort(preds)[::-1]) + 1).tolist()
        pred_rank = '[' + ','.join([str(i) for i in pred_rank]) + ']'
        f.write(' '.join([str(impr_index), pred_rank])+ '\n')

f = zipfile.ZipFile(os.path.join(data_path, 'prediction.zip'), 'w', zipfile.ZIP_DEFLATED)
f.write(os.path.join(data_path, 'prediction.txt'), arcname='prediction.txt')
f.close()

The problem is that when I submit those to the competition, I get a weird error which doesn't tell much on how I can solve this problem. The error is the following:
WARNING: Your kernel does not support swap limit capabilities or the cgroup is not mounted. Memory limited without swap. /tmp/codalab/tmpDabQNd/run/program/evaluate.py:17: VisibleDeprecationWarning: boolean index did not match indexed array along dimension 0; dimension is 61 but corresponding boolean dimension is 7   false_score = score[label==0] /tmp/codalab/tmpDabQNd/run/program/evaluate.py:18: VisibleDeprecationWarning: boolean index did not match indexed array along dimension 0; dimension is 61 but corresponding boolean dimension is 7   positive_score = score[label==1] Traceback (most recent call last):   File "/tmp/codalab/tmpDabQNd/run/program/evaluate.py", line 125, in      auc, mrr, ndcg, ndcg10,mrrs = scoring(truth_file, submission_answer_file)   File "/tmp/codalab/tmpDabQNd/run/program/evaluate.py", line 91, in scoring     mrr = mrr_score(y_true,y_score)   File "/tmp/codalab/tmpDabQNd/run/program/evaluate.py", line 36, in mrr_score     y_true = np.take(y_true, order)   File "/opt/conda/lib/python2.7/site-packages/numpy/core/fromnumeric.py", line 124, in take     return take(indices, axis, out, mode) IndexError: index 19 is out of bounds for size 7
What am I doing wrong and how can I fix that?
Thank you,
Nuno

@leavingseason FYI

@Leavingseason @miguelgfierro Any help would be really appreciated. Would like to submit my predictions to the competition but it always fails with the above error :(

Hi @ngcferreira, I have downloaded your submission. I find that its size is very small (only 300+kb), while a valid submission should be much larger. Your prediction file only contains 7,538 lines, while the entire behavior.tsv file contains 2,370,727 lines. Thus, you may need to check your code to see whether the prediction for each line in the behavior.tsv file is included. Thanks.

@wuch15 Thank you for your help. Unfortunately I went on vacations before I've seen your answer so I couldn't submit my results anymore.


urgent requirement. for having options for private relief camps and collection centres

private relief camps and collection centres

register private relief camps

view private relief camps

register private collection camps
view private collection centres

3 and 4 will be done by @naveenpf
1 and 2 to be done similar to government relief camps

Task 1 and 3 looks same

I think 3 is register private collection centres?

Are there difference in fields? If it is the same I think it is enough to add new field type

@naveenpf please comment
yes without making conflicts in the model.

I can write tests, once the functionality is ready.

How do you want to see this? With the same relief camp listing we just add column to show if it is pblic or private? Or a separate page itself?

@vignesh is working on 1 and 2

Done

@naveenpf  can you tag the PR?


Is there an ETA for the next release?

I'm using features in the vnext branch and I'm wondering when they will be available in a NuGet package.

Hey John!  Christmas is coming.  I'm hoping to get some time to work on this in the next couple of weeks while it is quiet at work.


inherit EventEmitter

Also emit event start, stop, local, remote, target and maybe another. That do you think

What's the goal?

@pstadler i use own build system based on promises and parallels, inside build system i use flightplan. I need start and end (etc) event for my own solution.

You can do this on your own, no need for flightplan to emit these events directly.
Related: https://github.com/pstadler/flightplan/issues/74

@pstadler i use directly run, but i don't know when flightplan is finished.

@pstadler after flightplan is finished i need push message to developer


since 2.3000.xxx version，It's only available in English，Is there any way I can get html in other languages?

hello @wangsen2020  please do you have any whatsapp web html version code stable to link it with the cachversion please the version 2.2413.51 is not working


(#89) - nail down pouchdb/express-pouchdb deps

Issues like #99 show why it's a bad idea to rely
on Github dependencies in npm packages. It's nice
when everything can just come from npm's servers.

@marten-de-vries gave a +1 to https://github.com/pouchdb/pouchdb-server/pull/89, so gonna merge this and publish since it's green.


Connect to local host kernel

It seems like Hydrogen allows connecting to remote kernel now. I wonder if I can connect to a jupyter notebook kernel e.g. on http://localhost:8888 or a local spyder kernel? I tried to input this url in the gateway configuration, but it does not work out.

I connect to a local notebook server all the time, on my machine. Here's what my gateways setting look like:
[{"name":"Local gateway","options":{"baseUrl":"http://localhost:8888"}}]

If you're still having issues getting this working, could you provide more details about what you tried and exactly how it does not work?

Thanks @nikitakit ! I updated all my packages and tried again today and was able to connect to jupyter notebook kernel from hydrogen! My gateways setting is exactly as what you have above. Weirdly it did not connect the last time.
On the other hand, what I really wanted is to connect to a kernel started in spyder. It does not come with a url address like what notebook does in the browser, so I am not sure what to put in the baseUrl. There is connect_info that gives an ip address and many port numbers, but that did not work out. Any idea it is possible/how to do this?

I haven't used Spyder myself, but it seems like spyder is spawning kernels directly instead of through a notebook server.
Connecting to them requires using a different feature of hydrogen. See here (except ignoring all the stuff about docker).
Essentially, you need to add a hydrogen/connection.json file in your project folder (the top folder you added in the tree view), and make sure this file contains the output of connect_info. Then hydrogen will connect to that kernel instead of spawning its own.

Worked like a charm! Thank you so much!!!


[QUESTION] Prevent Tile animation on nested component click

Hi, nice project!
I was wondering if it's possible to avoid the tile animation when I click on a nested button.
I'm able to prevent the event propagation so that only the nested button handler is executed, but I would also need to prevent the animation.
Thanks!

Thank you. At the moment, there is no such possibility. According to the Microsoft developers guide, Tile is an independent object that provides information and does not need to contain controls. This is how he works in Metro 4. In another job is not planned.


Update references.mdx

 correct what I understand to be a typo: int &ref = var. x does not refer to anything here
Description

Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[ ] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 Code refactor
[ ] ✅ Build configuration change
[ x] 📝 Documentation
[ ] 🇺🇸 Translation
[x ] 🗑 Chore

That indeed looks like a typo, thanks for noticing and contributing!


Add new Glossary section to Docs

Anthony Lead

[x]  Decided not to use a glossary section on the Docs - will cover all the keywords in the existing sections. Glossary will be heard to manage in the long run.

Decided not to use a glossary section on the Docs - will cover all the keywords in the existing sections. Glossary will be heard to manage in the long run.


Validation: email test missed some cases

I found some cases that was missed by both the js regexp as well as the php regexp. @jimdelois is looking at updating the php side.

:+1:

:+1:


docs(ldap): Fixed mistake on group lookup in docs

Gate does a lookup fallback to the user.  Fiat does NOT.  Need to probably mention this in AuthN docs vs. Authz docs, but lets remove this first...

@ttomsu, is there any additional input you wanted to add or should we merge this?

ping @ttomsu


Refactor salt

This PR:

Updates the fractal core contracts package version to 0.1.7
Updates the mock contracts to use creator and msg.sender as create2 salt instead of tx.origin
Updates the Metafactory to support the new create2 salt pattern

@tbwebb22 do you mind going through the changeset and leaving comments explaining why things changed the way they did?


v8 and nodejs: paxmark mksnapshot

Motivation for this change
paxmarking
Things done

[x] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Is this working?

The first commit works. I decided to try making nodejs use the v8 nixpkgs
package as it is highly related (nodejs gets killed by pax while building
its internal copy of v8).I'm still trying to figure out this, but the first
commit can be merged.
On Thu, 8 Dec 2016 at 22:07 Graham Christensen<EMAIL_ADDRESS>wrote:

Is this working?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/20995#issuecomment-265725361, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AB2crMmZkjBL39zXNWoucTRp7EdvBx-nks5rF_MFgaJpZM4LHZxo
.

grsecurity is no longer supported.


Fix for OWLS-65309

The change delete voyager when there are no voyager ingress left.

Simon, please create a public issue here to mirror the internal bug.  We are going to better support external customers who are watching this project by describing our work publicly.

Fixes issue #275


I cant get supergroup

I am receiving this error: Error { code = 400 message = "Invalid supergroup identifier" } on the code: TdApi.GetSupergroup getSupergroup = new TdApi.GetSupergroup(); getSupergroup.supergroupId = chatId; this.client.send(getSupergroup, re -> {, the used id is correct... I am using this id: -1001516291509

The id is wrong. chat id != supergroup id


Cookie-based affinity setting not working when using same cookie-name on http/https Httpsettings for same host

Describe the bug
The browser-client is not being routed to the same server of the same backend pool when arriving at http port 80 and being redirected to the http port 443 of the same host.
To Reproduce
All statements below are created by az network application-gateway ..

create 1 backend pool "backendPool-1" with say 8 machines
create HttpSetting for protocol http port 80. Enable Cookie-based affinity and use cookie name "afck-somehost"
create HttpSetting for protocol https port 443. Enable Cookie-based affinity and use the same cookie name "afck-somehost"
create HttpListener for port 80 and host name "www.somehost.com" (Listener type = Multi site)
create HttpListener for port 443 and host name "www.somehost.com" (Listener type = Multi site)
create Rule 1 for HttpSetting port 80, HttpListener port 80 and for the backend pool "backendPool-1"
create Rule 2 for HttpSetting port 443, HttpListener port 443 and for the same backend pool "backendPool-1"
Request the host with http://www.somehost.com (http 80) with a redirect page
User redirects to https://www.somehost.com (http 443), but with a different value in the cookie "afck-somehost"

Expected behavior
The value of the cookie "afck-somehost" is the same and the client is being routed to the same server from the backend pool.
Environment summary
Azure Portal - Application Gateway
Additional context
None

Not CLI related issue


Settings Page broke in IE 11

In 2018.2 in IE 11 if you go to the Settings page you will see a blank canvas. The browser console shows the following error

The new UI also doesn't work in ie11. I remember some polyfills had to be done to get the new UI working for ie11 in 6.6
Not sure why it's broken again in 2018.2 and 2019.1

Fixed along with a number of other issues in IE11 UI


Make seek() throw an exception when using NaN value

When seek() is used with a NaN value, a very hard to debug error is generated somewhere deep in React Native, which makes it hard to find the cause of the error (especially when the seek value is the result of a calculation that goes wrong in some cases but not others):

This pull request improves this by throwing an error early, which makes the issue clearer and provides a stack trace in user space:

Ok this makes a lot of sense, going ahead and merging this is a big improvement over the arcane error. Thanks.


Sorting of search results

There is some concern over the ordering of the listings on search results and services pages. We don't wish to appear biased and promote some businesses over others.
Some solutions that have been discussed:
Alphabetical ordering
PROS: it is standard and could be employed without people feeling slighted.
CONS: It may encourage people to make their business names start with A to place themselves higher up in the lists
Last edited/visited by owner
PROS: People more involved with their business page on our site would be rewarded for their visits.
CONS: It adds an unnecessary task to already busy people and wastes their time better spent on caring for clients. It could also add up to more costs for hosting.
Randomized ordering
PROS: Nobody can complain, since it would always be different for everyone, which would hopefully result in a reasonably fair distribution of clients to businesses.
CONS: It will be harder to implement, because there needs to be some consistency, at least when it comes to paging.

I've found this article that explains how random ordering can be done:
http://khanduri.github.io/learn/2016/02/26/fetch-rows-in-random-order-with-seed-support
Using that, I've moved all of the logic into a single SQL query call that returns the already sorted distinct list of businesses that match the search. I've also implemented some paging logic, but it requires a few more changes to be ready for use.
Done so far:

sorting query
session based seed for sorting

To do:

Paging in templates
Service listings

Changes to config.py
At the top:
from datetime import timedelta

And then new values:
    PERMANENT_SESSION_LIFETIME =  timedelta(days=1)
    RESULTS_PER_PAGE = 10

On the server, ordering is always the same, regardless of the seed.

It's been fixed with a tweak in the query.


Update rust version for CI

Description
Try fixing CI

@lvpeschke Do you have the rights to change the expected status from ci/circleci: bls-tests to run-tests?

@lvpeschke Do you have the rights to change the expected status from ci/circleci: bls-tests to run-tests?

Yes, let's see if the pass with the smaller instances.

@lvpeschke Do you have the rights to change the expected status from ci/circleci: bls-tests to run-tests?

Yes, let's see if the pass with the smaller instances.

The tests consistently fail with the smaller size runners. But I don't expect much work in this repo, so I guess it's fine until CI is moved to GH actions.


#203 #254 #231 add icon to asar content, change styling to allow copying the content

add icon to asar content, change styling to allow copying the content
The resources folder in bruno-electron is not packaged to the asar archive. So the image could not be loaded.

Nice!
Thanks for taking care of this @mirkogolze !


Wired behaviour : Closure overwriting function argument

Hi,
I stumbled upon a very strange behavior, tried to make a simple code to demonstrate it
this is the original js code :
function fadeTo(value, time = 250, delay = 0) {
    let contextResolve;

    const promise = new Promise((resolve, reject) => {
        contextResolve = resolve;
    });

    const tween = new Game.utils.Tween({})
        .to({ alpha: value }, time);

    tween.onComplete(contextResolve);
    tween.start();
    return promise;
}

When compiled using closure I obtain :
'use strict';
function fadeTo(a, d=250, b) { //<-- my "value" argument translated to "a"
    let c;
    b = new Promise((a,b)=>{
        c = a
    }
    );
    a = (new Game.utils.Tween({})).to({  //<-- Here is line 8
        alpha: a
    }, d);
    a.onComplete(c);
    a.start();
    return b
}
;

if you compare the codes I have a variable called "value" passed as first argument to fadeTo() function.
this variable got translated to "a" by closure compiler.
but if you see line 8, this same variable is used to reference the Tween object, which should use a different variable (my original code was const tween) .
this breaks the behavior of the code since "a" variable don't hold the original value anymore, and if I pass it to an internal object (like the alpha value in my code) it's the tween object which got passed instead of the original value.
I tried to move the code inside the promise but got similar broken code.
also tried with both : --language_out ECMASCRIPT6/ECMASCRIPT5 ... still broken.
is this an intended behavior or something wrong with my code ? or is it a bug ?

for information this is the command line I used to compile the code
java -jar closure-compiler.jar  --js /path/to/source.js  --js_output_file out.js --jscomp_off=internetExplorerChecks --language_in ECMASCRIPT6 --language_out ECMASCRIPT6

Hi,
Both your examples look like they are compiled correctly to me. This is something the compiler does in CoalesceVariableNames.
In your original example, in a = (new Game.utils.Tween({})).to({alpha: a}, d);, first the current value of a is taken and used to create an object {alpha: a} that gets passed into the to method. Only after that method completes does a get reassigned to a Tween. There's no difference between a = foo({alpha: a}); use(a); and const tween = foo({alpha: a}); use(tween);
Marking as WAI - please reopen if you find an example that we can run in the console that produces different output post-compilation.


get_key_references is not finding certain key reference

Look at the 1st and only active key of jlightwallet3 (GPH7iDi...)
cli wallet>>>
get_account jlightwallet3
Now, try to get get_key_references with that active key, it will be empty:
browser console>>>
var db = Apis.instance().db_api()
resolve(db.exec("get_key_references", [["GPH7iDijoYQSroHwh6EhDkQfchziW7EUpGtss8wadx89JVNLRHztY"]]))
The owner key works..

@theoreticalbts @bytemaster As soon as users start modifying account keys they are going to start seeing accounts disappear form the graphene-ui wallet...

I think I need a look ahead pool of keys anyway for other reasons.  That will work around this issue for now by checking past the unfound active key and finding the next owner key.

This could leave missing keys in a wallet that was recovered by a brainkey or restored from a stale backup.

Here is another scenario (graphene-ui) -> ```I updated dummy0's account permission so it had no public keys but used other accounts that also belonged to me.  It disappears from the dashboard.
jcalfee [4:21 PM]
I can however transfer still from the account and the page does recognize it as "My Account"
jcalfee [4:22 PM]
This is due to ChainStore.getAccountRefsOfKey not finding that account.

Please find by memo key too...


Fix warnings on channel info row Formatted Text

Summary
Channel Info row is expecting a textID and default message in order to create a <FormattedText> component for the element label. This created lots of warnings for bindings, since they are not system defined, and therefore have not a translation in the i18n bundle. This also could create some issues if coincidentally, the formed text id that was being formed became the same as some translation id we have in the system.
In order to solve this, we have removed the requirement of the text id, and only create the <FormattedText> field when it is given.
Another solution would have been to move the localization out of this component, and let the elements be directly the component to render (or the translated text), but that solution would imply a bigger changelog.
Ticket Link
None
Release Note
None

@mickmister Friendly reminder to review this.

@DHaussermann For this PR, not sure what exactly to test. The error messages should only be visible on a development build, so you won't see them neither now nor before.
If you want, you can conduct some regression testing seeing if all the channel menu elements translate properly (apart of the app related ones).

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/17981
https://circleci.com/gh/mattermost/mattermost-mobile/17980

Artifact links:
https://17981-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.ipa
https://17980-70265724-gh.circle-artifacts.com/0/tmp/artifacts/Mattermost_Beta.apk


Bug duplication when a server doesn't have to sync inventories

How to reproduce the bug:
configure 2 servers (A and B) to sync inventories and a third server (C) with inventory sync disabled.
1: In server A drop something then move to server C (the inventory is different as expected);
2: Move from server C to server B (the inventory here is the same from server A but what you dropped is still in the inventory)
3: Move from server B to server C (the item you dropped are still there, but you have the same items in the inventory)
config for server A and B: https://pastebin.com/49ZhSF9H
config for server C: https://pastebin.com/ZH9yu6ar
Using latest version: 2.2.8

hi there :)
due to the nature of this issue, further troubleshooting is needed. Please make a ticket on Discord and verify your purchase so we can help you resolve this.


[AIO] Support SO_REUSEPORT for AsyncIO Server

As title.

Should this be "Support SO_REUSEPORT in AsyncIO Server"? We don't necessarily want it on by default, right?

@gnossen Title renamed as suggested.

In the attempt of #21607, I found supporting SO_REUSEPORT is more complicated than expected due to the isolation of IO manager. IO manager could not map each socket to corresponding server, so the socket options cannot be applied properly.
There are three potential ways to solve this problem:

Not allow mixing enabled socket and disabled socket (otherwise raise exceptions);
Add an API to set a global flag SO_REUSEPORT that valid for entire process;
Plumb through the channel argument in tcp_server_custom.cc (might be hard to get approval).

@lidizheng I fully support option 3. Configuring this option via a per-process global is a recipe for race conditions and bugs. Let's go down the right technical path and worry about the political friction later.

Just for having me on the same page as you. The main "problem" right now for doing it in the right way is the no visibility of the channel args in the custom IOMGR interface, am I right? Which theoretically should be doable to do it since the core when works with its own IO implementation can make usage of these parameters. am I missing something?
Seems that there could be other parameters besides the REUSEPORT one that could impact in the future in the custom IOMGR provided by Aio - like GRPC_ARG_TCP_MAX_READ_CHUNK_SIZE ?. So I would advocate also for trying to solve this in the right way.


Xiaomi Mijia-720P dont works

Xiaomi Mijia-720P camera with latest firm: 3.3.6_2018062014
Starting proxy...
Starting RTSP server on "rtsp://<IP_ADDRESS>:8554/"
(Using port 8000 for optional RTSP-over-HTTP tunneling.)

Got error when receiving: timed out
Traceback (most recent call last):
  File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 257, in send
    data, addr = s.recvfrom(1024)
socket.timeout: timed out

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/pi/.local/lib/python3.5/site-packages/miio/device.py", line 257, in send

    data, addr = s.recvfrom(1024)
socket.timeout: timed out
...

xiaomi gateway camera works fine!!!

Did try on several camera:
chuangmi.camera.xiaobai
Starting proxy...
Unable to discover a device at address <IP_ADDRESS>
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.7/site-packages/miio/device.py", line 224, in send
    self.do_discover()
  File "/usr/local/lib/python3.7/site-packages/miio/device.py", line 162, in do_discover
    raise DeviceException("Unable to discover the device %s" % self.ip)
miio.exceptions.DeviceException: Unable to discover the device <IP_ADDRESS>
Can't read output line 1 from command: python3 -c "import miio;result = miio.device.Device('<IP_ADDRESS>', '4234306857534d527930425043544f4e').send('get_ipcprop', ['all']);print(result['p2p_id']);print(result['avID']);print(result['avPass']);"
Error connecting to camera, make sure ip and token are correct.
Error, waiting 10 seconds and trying again.

Setting that I got from MiHome for that camera:
{
			"did": "[REDACTED]",
			"token": "[REDACTED]",
			"longitude": "0.00000000",
			"latitude": "0.00000000",
			"name": "Caméra chambre Ethan",
			"pid": "0",
			"localip": "[REDACTED]",
			"mac": "[REDACTED]",
			"ssid": "[REDACTED]",
			"bssid": "[REDACTED]",
			"parent_id": "",
			"parent_model": "",
			"show_mode": 1,
			"model": "chuangmi.camera.xiaobai",
			"adminFlag": 1,
			"shareFlag": 0,
			"permitLevel": 16,
			"isOnline": true,
			"desc": "Camera online",
			"extra": {
				"isSetPincode": 0,
				"fw_version": "3.3.10_2018061309",
				"needVerifyCode": 0,
				"isPasswordEncrypt": 0
			},
			"uid":<PHONE_NUMBER>,
			"pd_id": 75,
			"password": "",
			"p2p_id": "",
			"rssi": 0,
			"family_id": 0,
			"reset_flag": 0
		}

#isa.camera.isc5c1
Starting proxy...
Unable to discover a device at address <IP_ADDRESS>
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/local/lib/python3.7/site-packages/miio/device.py", line 224, in send
    self.do_discover()
  File "/usr/local/lib/python3.7/site-packages/miio/device.py", line 162, in do_discover
    raise DeviceException("Unable to discover the device %s" % self.ip)
miio.exceptions.DeviceException: Unable to discover the device <IP_ADDRESS>
Can't read output line 1 from command: python3 -c "import miio;result = miio.device.Device('<IP_ADDRESS>', '5774556d486241553746565452425943').send('get_ipcprop', ['all']);print(result['p2p_id']);print(result['avID']);print(result['avPass']);"
Error connecting to camera, make sure ip and token are correct.
Error, waiting 10 seconds and trying again.

Setting that I got from MiHome for that camera:
{
			"did": "[REDACTED]",
			"token": "[REDACTED]",
			"longitude": "0.00000000",
			"latitude": "0.00000000",
			"name": "Caméra Balcon Ethan",
			"pid": "0",
			"localip": "[REDACTED]",
			"mac": "[REDACTED]",
			"ssid": "[REDACTED]",
			"bssid": "[REDACTED]",
			"parent_id": "",
			"parent_model": "",
			"show_mode": 1,
			"model": "isa.camera.isc5c1",
			"adminFlag": 1,
			"shareFlag": 0,
			"permitLevel": 16,
			"isOnline": true,
			"desc": "Device online ",
			"extra": {
				"isSetPincode": 0,
				"fw_version": "<IP_ADDRESS>",
				"needVerifyCode": 0,
				"isPasswordEncrypt": 0
			},
			"uid":<PHONE_NUMBER>,
			"pd_id": 434,
			"password": "",
			"p2p_id": "",
			"rssi": -56,
			"family_id": 0,
			"reset_flag": 0
		}


Update git-dolt to use cobra for better UI

A video about Cobra came on YouTube autoplay and I was bored so I decided to try migrating git-dolt to it. It was easy and comes with some nice benefits. Look how pretty!
$ git-dolt
Run a git-dolt subcommand.
Valid subcommands are: fetch, help, install, link, update.

Usage:
  git-dolt [command]

Available Commands:
  fetch       Fetches the Dolt repository referred to in the given git-dolt pointer file
  help        Help about any command
  install     Installs the git-dolt smudge filter for this Git repository
  link        Links the given Dolt repository to the current Git repository
  update      Updates the reference in the given git-dolt pointer file to the given revision

Flags:
  -h, --help   help for git-dolt

Use "git-dolt [command] --help" for more information about a command.

$ git-dolt fetch
Error: accepts 1 arg(s), received 0
Usage:
  git-dolt fetch <pointer-file> [flags]

Flags:
  -h, --help   help for fetch

$ git-dolt help install
Installs the git-dolt smudge filter for this Git repository.
After this, when git-dolt pointer files are checked out in this repository, the corresponding Dolt repositories will automatically be cloned.

Usage:
  git-dolt install [flags]

Flags:
  -h, --help   help for install

Great, this finally passes CI. @bheni, @reltuk, either of you want to have another look before I merge?


Version 2.4.0 - Explicit termination method 'end' not called

It looks like it happens when reading of ZipInputStream is interrupted.
I do ZipInputStream.close() but it is not enough.
D/StrictMode: StrictMode policy violation: android.os.strictmode.LeakedClosableViolation: A resource was acquired at attached stack trace but never released. See java.io.Closeable for information on avoiding resource leaks.
at android.os.StrictMode$AndroidCloseGuardReporter.report(StrictMode.java:1786)
at dalvik.system.CloseGuard.warnIfOpen(CloseGuard.java:264)
at java.util.zip.Inflater.finalize(Inflater.java:398)
at java.lang.Daemons$FinalizerDaemon.doFinalize(Daemons.java:250)
at java.lang.Daemons$FinalizerDaemon.runInternal(Daemons.java:237)
at java.lang.Daemons$Daemon.run(Daemons.java:103)
at java.lang.Thread.run(Thread.java:764)
Caused by: java.lang.Throwable: Explicit termination method 'end' not called
at dalvik.system.CloseGuard.open(CloseGuard.java:221)
at java.util.zip.Inflater.(Inflater.java:114)
at net.lingala.zip4j.io.inputstream.InflaterInputStream.(InflaterInputStream.java:21)
at net.lingala.zip4j.io.inputstream.ZipInputStream.initializeDecompressorForThisEntry(ZipInputStream.java:206)
at net.lingala.zip4j.io.inputstream.ZipInputStream.initializeEntryInputStream(ZipInputStream.java:187)
at net.lingala.zip4j.io.inputstream.ZipInputStream.getNextEntry(ZipInputStream.java:103)
at net.lingala.zip4j.io.inputstream.ZipInputStream.getNextEntry(ZipInputStream.java:77)

Is this specific to 2.4.0? Do you get this error also with older versions?

It looks like it happens when reading of ZipInputStream is interrupted.

I have fixed this issue. I will include the fix in the next release.

I included this fix in a release, v2.5.0, I had to do today. I think the code change I did will fix this issue. If not, please reopen this issue again with your findings.
And also: a couple of more features have been added in this release. Please have a look at release notes.

Thank you so much! The issue is fixed in v2.5.0


cannot import third module correctly？

"eslint-plugin-import": "^2.2.0"
"antd": "0.11.3"
ESLint: 'antd' should be listed in the project's dependencies. Run 'npm i -S antd' to add it (import/no-extraneous-dependencies)
but i truely have listed third modules in my package.json dependencies,and why throw the eslint error?but React module is working all right

eslint-plugin-import (like all eslint stuff) should be a devDependency. Can you share your entire package.json?

I found the problem,it's definitely my fault,and i am sorry for openning this issue,and after you see my apologize, you can delete this issue.


Export from elastic cloud instance

Is there any plan to make elasticdump work against a cloud instance of elasticsearch, i.e. using a token or api key?

It already supports this. You can use basic auth or bearer token header as suggested in the docs


Web builds do not work with Safari Browser anymore

Godot version
3.5.1.stable
System information
MacOS 13.0, Safari Browser 16.1
Issue description
After the latest update of MacOS and it's Safari Browser, my web build stopped working - when loading it will crash, then reload the page, crash again and display "A problem repeatedly occured on ". If the game is embedded into the page, the whole page will crash.
I tried to figure out is wrong and then eventually tried other projects, of which all had the same behaviour. I then went to Godot Discord and tried to open projects of others from the Release channel: not a single one of them loaded.
I asked a friend who also got a Mac to try a few of them and he had exactly the same issues. So it looks like no web build is working with the Safari Browser anymore.
Steps to reproduce
Web builds crash with Safari Browser
Minimal reproduction project
No response

Duplicate of #67949

Closing as duplicate of #67949.


Fix flaky test_prim_inplace_copy_fwd

The problem with bf16 dtype inputs is that we were testing different functions as eager upcasts and downcasts around each arithmetic op while the nvFuser executor does upcasting and downcasting once and doesn't do it for intermediates. To test the behavior of copy_ we don't need several arithmetic operations, so I reduced it to just one. On 1000 test runs there are now zero failures.
Fixes #295

Supergood, thank you @IvanYashchuk @carmocca


Added linting script, run lint on PRs, and fix lint errors already in the codebase

In order to ensure that code is properly formatted, this PR adds linting with Ruff and a GitHub action to run on every commit.  Once merged, we'll add the action as a required check in order to merge PRs.

Closing and re-opening to verify #15


Modal Scrollbar Disappears in Chrome 85.0.4183.83

Hi @robinparisi,
The latest version of Chrome has a bug that hides the scroll bars on Windows 10 as seen on this screenshot.
It appears that this issue was also present in Chrome 41.
There are a couple of workarounds that I was able to find, but nothing that is acceptable:

Disable hardware acceleration. This resolves the problem but hardly a fix for users/visitors.
Apply transform:translateZ(0); to the modal. This shows the scroll bar but breaks the DOM flow and causes footer content that are attached to the bottom of the screen to lose their position.

Apart from a re-write to contain the scrollbar within the modal content div I'm not really sure what can be done here.
Thoughts?

Hi @eihabi,
Thanks for filling out an issue. I can't test on Windows for now but I will try to do so as soon as possible.


Adding Helm Chart

Adding helm chart for demo and test

@ViniciusSouza - thanks for this contribution. I apologize that this was not noted here, but we have a chart for this application located here:
https://github.com/Azure-Samples/helm-charts/tree/master/chart-source/azure-vote


hi

nice to meet you,can you be frind with me?

Maybe. This is not the right place to ask this kind of question.


AttributeError: type object 'Color' has no attribute 'to_tuple'

I think in commit 487ed18 you got rid of to_tuple() and that broke the tkinter adapter. I am now getting the error
  File "RGBMatrixEmulator/adapters/tkinter_adapter.py", line 48, in draw_to_screen
    self.__canvas.itemconfig(shape_id, fill=Color.to_hex(pixel))
  File "RGBMatrixEmulator/graphics/color.py", line 18, in to_hex
    return "#%02x%02x%02x" % Color.to_tuple(pixel)
AttributeError: type object 'Color' has no attribute 'to_tuple'

Jsut confirmed, it works in version 0.10.0, but broken in 0.11.0

Thanks for the report, I was able to reproduce easily and fix is forthcoming soon in 0.11.1.

Resolved by #65


Fix cri validation test to use new log format.

Skip the tag field in cri validation test.
I've validated with cri-containerd:
Ran 57 of 58 Specs in 180.102 seconds
SUCCESS! -- 57 Passed | 0 Failed | 0 Pending | 1 Skipped PASS

Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

/lgtm


Release/judas priest

Try out this version of the Leather — download extension builds.

Contents:

https://github.com/leather-wallet/extension/issues/4185
https://github.com/leather-wallet/extension/issues/4162
https://github.com/leather-wallet/extension/issues/4151
https://github.com/leather-wallet/extension/issues/4195
https://github.com/leather-wallet/extension/issues/4187
https://github.com/leather-wallet/extension/issues/3710

👍🏼
remember to merge, not rebase

I took in Anas's fix for BG colour so I need to re-review please

@pete-watters this is being stretched

@pete-watters this is being stretched

Thanks, I'll fix it https://github.com/leather-wallet/extension/issues/4212


requirement: R22, Perform structural decompilation passes on LLVM IR

Depends on #11.

The requirement is satisfied for now, but the design may be improved in the future. Further discussion is taking place in issue #11 and #5.


Do not silently restart a new tailable cursor after maxDocs is reached.

Should resolve failing test of https://github.com/ReactiveMongo/ReactiveMongo-Streaming/pull/56.

Any such change would need to update some cursor tests or to add some to demonstrate the validity of the change and the non-regression.

I will try to come-up with some tests. But since this change impact only an api exclusively used by ReactiveMongo-Streaming (currently not tested in driver project), validating non regression would require running tests of that project against this PR. Is that possible?

Tailable cursor is not only used by streaming. If there is an issue only in streaming, then no change there should fix it.

Right, the method is used, it was just intellij "find usage" failing me.

Added a test that (fails) without this fix.


Parse error: syntax error

Hi there,
I was trying to install package which installs perfectly fine without errors, but than afterwards getting a parse error as I refresh the page.
Parse error: syntax error, unexpected ':', expecting '{' in /var/www/localhost/api/vendor/spatie/laravel-activitylog/src/helpers.php on line 6
Am I the only one getting this error?

Please upgrade to PHP 7 to use this package.


Clean up admin handlers into a REGISTER_FACTORY-based initialization

There's a mess of makeHandler calls, with a corresponding mess of member variables, #includes and initializers. If we replace this with a FactoryRegistry, the dependencies will be cleaner, less duplicate code, and it will open it up to third-party extensions adding admin features without having to touch their config (the current best option for extending admin, BootstrapExtensionFactory, would require updating both binary and config).
Maintainers have okayed this idea, so it's now in my queue.

jmarantz didn't like this change, so it's aborted.


UHF-9675: Add cookie check for siteimprove to be able to see inside accordions

UHF-9675

What was done

Allow helfi_accordions_open cookie so that it can be used by Siteimprove.

How to install

Check installation instructions from the HDBT PR

How to test

[ ] Check testing instructions from the HDBT PR
[ ] Check that code follows our standards

Other PRs

https://github.com/City-of-Helsinki/drupal-hdbt/pull/919
https://github.com/City-of-Helsinki/drupal-helfi-platform-config/pull/707

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

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

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #707   +/-   ##
=========================================
  Coverage     13.01%   13.01%
  Complexity      242      242
=========================================
  Files            28       28
  Lines           945      945
=========================================
  Hits            123      123
  Misses          822      822

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


Automatic error report

Unable to open and view the variables in the Variable explorer.

This is fixed in our latest version (3.3.4). Please update by opening the Anaconda Prompt and running there
conda update spyder


Cannot open folder

VSCode Version: Code 1.15.1 (41abd21afdf7424c89319ee7cb0445cc6f376959, 2017-08-16T18:07:25.676Z)
OS Version: Windows_NT x64 10.0.15063
Extensions:

Extension
Author (truncated)
Version

markdown-toc
Ala
1.5.6

jquerysnippets
don
0.0.1

auto-close-tag
for
0.4.3

auto-rename-tag
for
0.0.14

beautify
Hoo
1.1.1

path-autocomplete
ion
1.6.1

easy-less
mrc
1.4.5

vetur
oct
0.9.5

vscode-icons
rob
7.12.0

markdown-preview-enhanced
shd
0.2.7

open-in-browser
tec
1.1.0

(2 theme extensions excluded)

Steps to Reproduce:

Reproduces without extensions: Yes/No

@nosizi How can this bug be reproduced? Could you provide some more information and/or some screenshots?

GIF

Gif image

@nosizi is that folder on OneDrive?

@bpasero no, on E Disk

@bpasero Console Img

@nosizi hm nothing in particular. Something in the terminal maybe where you started with?


docs: update ReadMe

Rendered: https://github.com/google-github-actions/deploy-cloudrun/blob/9fcb98a53ee71791a97312034ca42eb9ffc0221a/README.md

@averikitsch I was thinking we could consolidate these into a single .github repo, so this can be easily maintained. This along with issue templates. WDYT?
https://docs.github.com/en/free-pro-team@latest/github/building-a-strong-community/creating-a-default-community-health-file#supported-file-types

@averikitsch I was thinking we could consolidate these into a single .github repo, so this can be easily maintained. This along with issue templates. WDYT?
https://docs.github.com/en/free-pro-team@latest/github/building-a-strong-community/creating-a-default-community-health-file#supported-file-types

That sounds great. Should I remove the contributing and .github/ISSUE_TEMPLATE files?

yeah I will send a PR to create that new repo in infra.


CON-3221 add TRUSTED_PROXIES support

PIn to v4 of the API, v1 of rack

I'll investigate adding IPv6 support.

It seems to me that something like
def trusted_proxy?(ip)
    (trusted_proxies && trusted_proxies.any? { |cidr| cidr.include?(ip) }) || super
end

should do the trick.

This would get you the default rack behavior for all IPv6 addresses, though, right? That doesn't seem like what we want.

After doing a bit of cursory testing, it seems like the code works fine for IPv6 as-is. Can you give an example of something that wouldn't work?

I think I'm a bit confused as to what this patch is supposed to do. Is the idea to untrust private address ranges (that are trusted by default by Rack)?
Re IPv6, I don't have an IP6 network to test at hand, but I'd expect you need to add "::1" (which is localhost6) along with <IP_ADDRESS>.

Thanks for the clarification!
fd00::/8 is the private subnet (like <IP_ADDRESS>/8). Localhost is ::1,
which should get filtered IIUC, but it doesn't look like it would.
23 sie 2016 17:19 "apotterri"<EMAIL_ADDRESS>napisał(a):

On Tue, Aug 23, 2016 at 9:21 AM, Rafał Rzepecki<EMAIL_ADDRESS>wrote:

I think I'm a bit confused as to what this patch is supposed to do. Is
the
idea to untrust private address ranges (that are trusted by default by
Rack)?

Yeah, that's basically the idea. There are more details here:
https://docs.google.com/document/d/1nh5xx0Z8GGpZRni-
yxTB80aA96Z4I6_T6-bT8TpXLyw/edit

Re IPv6, I don't have an IP6 network to test at hand, but I'd expect you
need to add "::1" (which is localhost6) along with <IP_ADDRESS>.

Yesterday, turned on my docker daemon's IPV6 support. I started up a couple
of linked containers and used them to test out this change. Even with only
TRUSTED_PROXIES=<IP_ADDRESS>/32 http://<IP_ADDRESS>/32, I got the desired
behavior -- IPV6
addresses that start with fd (which is what rack looks for) didn't get
filtered out.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<https://github.com/conjurinc/conjur-rack/pull/6#issuecomment-241727821
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEBHAxao_
lkKp4iYAP7V2twfh9Om1AHIks5qivPVgaJpZM4JmXl7>
.

--
Alan Potter
Engineer
p:<PHONE_NUMBER>
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/5ff664c66c5990e35bd8d1f3bfc0b4
83/www.conjur.net/>
Identify, Authorize, Audit... Everything
Demo
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/13deaa1654e874c3dccc0acce4e701
93/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nM
JW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Finfo.
conjur.net%2Fdemo&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32>
| Resources
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/cc9a5a824c8143be8a0bab0dcc7f1c
20/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nM
JW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Finfo.
conjur.net%2Fresources&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32>
| Blog
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/c59449660ac6f9ab10a1787fd59884
90/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nM
JW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Fblog.
conjur.net%2F&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/conjurinc/conjur-rack/pull/6#issuecomment-241767052,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAyRVFRWhJp_leAS-qOw2tsxsjBY5MbPks5qiw-ZgaJpZM4JmXl7
.

On Tue, Aug 23, 2016 at 11:58 AM, Rafał Rzepecki<EMAIL_ADDRESS>wrote:

Thanks for the clarification!
fd00::/8 is the private subnet (like <IP_ADDRESS>/8). Localhost is ::1,
which should get filtered IIUC, but it doesn't look like it would.

Right. So, without TRUSTED_PROXIES set, fd00:/8 addresses get mapped to
localhost. With TRUSTED_PROXIES set, they don't.

23 sie 2016 17:19 "apotterri"<EMAIL_ADDRESS>napisał(a):

On Tue, Aug 23, 2016 at 9:21 AM, Rafał Rzepecki <
<EMAIL_ADDRESS>wrote:

I think I'm a bit confused as to what this patch is supposed to do. Is
the
idea to untrust private address ranges (that are trusted by default by
Rack)?

Yeah, that's basically the idea. There are more details here:
https://docs.google.com/document/d/1nh5xx0Z8GGpZRni-
yxTB80aA96Z4I6_T6-bT8TpXLyw/edit

Re IPv6, I don't have an IP6 network to test at hand, but I'd expect
you
need to add "::1" (which is localhost6) along with <IP_ADDRESS>.

Yesterday, turned on my docker daemon's IPV6 support. I started up a
couple
of linked containers and used them to test out this change. Even with
only
TRUSTED_PROXIES=<IP_ADDRESS>/32 http://<IP_ADDRESS>/32 <
http://<IP_ADDRESS>/32>, I got the desired
behavior -- IPV6
addresses that start with fd (which is what rack looks for) didn't get
filtered out.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
<https://github.com/conjurinc/conjur-rack/pull/6#
issuecomment-241727821
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEBHAxao_
lkKp4iYAP7V2twfh9Om1AHIks5qivPVgaJpZM4JmXl7>
.

--
Alan Potter
Engineer
p:<PHONE_NUMBER>
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/5ff664c66c5990e35bd8d1f3bfc0b4
83/www.conjur.net/>
Identify, Authorize, Audit... Everything
Demo
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/13deaa1654e874c3dccc0acce4e701
93/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nM
JW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Finfo.
conjur.net%2Fdemo&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32>
| Resources
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/cc9a5a824c8143be8a0bab0dcc7f1c
20/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nM
JW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Finfo.
conjur.net%2Fresources&si=3680029&pi=47d6aafc-d455-4a13-
9ffc-a8737ef09b32>
| Blog
<http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/
f59ded25d9f0fae3e973353a29205cf4/c59449660ac6f9ab10a1787fd59884
90/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nM
JW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Fblog.
conjur.net%2F&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<https://github.com/conjurinc/conjur-rack/pull/6#issuecomment-241767052
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAyRVFRWhJp_leAS-
qOw2tsxsjBY5MbPks5qiw-ZgaJpZM4JmXl7>

.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/conjurinc/conjur-rack/pull/6#issuecomment-241782555,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEBHA_a0TWAVfstuQzAXzkozk1f6XETcks5qixiKgaJpZM4JmXl7
.

--
Alan Potter
Engineer
p:<PHONE_NUMBER>
http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/f59ded25d9f0fae3e973353a29205cf4/5ff664c66c5990e35bd8d1f3bfc0b483/www.conjur.net/
Identify, Authorize, Audit... Everything
Demo
http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/f59ded25d9f0fae3e973353a29205cf4/13deaa1654e874c3dccc0acce4e70193/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nMJW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Finfo.conjur.net%2Fdemo&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32
| Resources
http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/f59ded25d9f0fae3e973353a29205cf4/cc9a5a824c8143be8a0bab0dcc7f1c20/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nMJW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Finfo.conjur.net%2Fresources&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32
| Blog
http://t.yesware.com/tt/58d6ef3f2477232db08043fa9198ec0f5a0fde51/f59ded25d9f0fae3e973353a29205cf4/c59449660ac6f9ab10a1787fd5988490/t.sidekickopen03.com/e1t/c/5/f18dQhb0S1KL8bGch0W2n0x6l2B9nMJW7t6jwn3LqWkjN2BgL6d1vVSKf69FLZ203?t=https%3A%2F%2Fblog.conjur.net%2F&si=3680029&pi=47d6aafc-d455-4a13-9ffc-a8737ef09b32


Don't set completion text to full signature unless its override or partial method completion

This change fixes a regression where overloads would not be collapsed in the VS Code completion list. In addition, I added a couple of tweaks to ensure that we report the "EnumMember" and "Const" kinds.

Note: This isn't quite right. Some tests failing.

thanks!

aaaaah mystery solved!
I never saw this on the original PR because I tested against the version of C# for VS Code which had the magic completion Regex still present (which we remved here https://github.com/OmniSharp/omnisharp-vscode/commit/01aa52e197d37e6832006376bf8da96eb1bb5591). Turns out the purpose of this was to collapse overloads together on the client side.
It absolutely makes sense to do this on the server side correctly instead - thanks @DustinCampbell for fixing 😅


Support Provider as a return type for OptionValues annotation

For consistency sake.
Fixes https://github.com/gradle/gradle/issues/29341

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have been cancelled:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test this

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have passed:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test this

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have been cancelled:

Pull Request Feedback (Trigger) (Check)

Build Scan

@bot-gradle test this

@bot-gradle clean

I've triggered the following builds:

Pull Request Feedback (Trigger) (Check)

Build Scan

The following builds have passed:

Pull Request Feedback (Trigger) (Check)

Build Scan

WARN: Based on labels, this pull request addresses notable issue but no changes to release note found.


Overall fixes 2

Fixes wrong chunkGoal in ExecutionServices.getNextExecutableMemosChunk;
Updates AssetIconButton to mimic IconButton default properties when there is none.

All updates made and merged in the last PR (#76) in this nested chain of collection execution PRs


Added contributor to the ©

Please review and confirm that you agree with being added
Please review that you want to be added with your real name

@Tierhon please review this PR, thanks!!

@matuskalas This looks fine to me!

Perfect, thank you! 😊


Fixes #464 - Do not include 'upload_attachment' in widget settings if deactivated

@lqez, may I ask you to review this patch, please? I cannot upgrade some project of mine because of this.

@lqez thanks for the thumb up! What about merging :-)


Localization not working for list instance

Using schema "http://schemas.dev.office.com/PnP/2017/05/ProvisioningSchema"
template files has localization attributes
pnp:Localizations
<pnp:Localization LCID="1033" Name="sitevariant" ResourceFile="sitevariant.en-us.resx" />
<pnp:Localization LCID="1035" Name="sitevariant" ResourceFile="sitevariant.fi-fi.resx" />
</pnp:Localizations>
template file having list instance
<pnp:ListInstance Title="{localization:sitevariant:Documents}"
However title is not getting correctly set. it is setting as {localization:sitevariant:Documents}

Hi @prafullapatil
Maybe @PaoloPia  has an answer for us, I am unable to make this localization working on my templates. For example, I also have localization defined in my template, and then when I use this in my navigation =>
<pnp:CurrentNavigation NavigationType="StructuralLocal">
  <pnp:StructuralNavigation RemoveExistingNodes="true">
    <pnp:NavigationNode Title="{localization:core:NavHome}" Url="{site}" />

When applying template on a site, navigation contains : localization:core:NavHome text instead of localized text in NavHome node of the resx file.

Hi @prafullapatil
I just give another try after reading the article of @erwinvanhunen (https://www.erwinmcm.com/multilingual-provisioning-templates-with-the-office-365-developer-patterns-and-practices-provisioning-engine/)
<pnp:NavigationNode Title="{localization:NavHome}" Url="{site}" />
<pnp:NavigationNode Title="{localization:core:NavHome}" Url="{site}" />
<pnp:NavigationNode Title="{res:NavHome}" Url="{site}" />
<pnp:NavigationNode Title="{res:core:NavHome}" Url="{site}" />

Here is the result after applying the template...

For some reason, it seems the way @PaoloPia is using in the fullexample for example here is not working...
 <pnp:SiteFields>
        <Field ID="{23203E97-3BFE-40CB-AFB4-07AA2B86BF45}" Type="Text" Name="ProjectID" DisplayName="{localization:intranet:ProjectID}" Group="Base.Foundation.Columns" MaxLength="255" AllowDeletion="TRUE" />
        <Field ID="{B01B3DBC-4630-4ED1-B5BA-321BC7841E3D}" Type="Text" Name="ProjectName" DisplayName="{localization:intranet:ProjectName}" Group="Base.Foundation.Columns" MaxLength="255" AllowDeletion="TRUE" />
        <Field ID="{A5DE9600-B7A6-42DD-A05E-10D4F1500208}" Type="Text" Name="ProjectManager" DisplayName="{localization:intranet:ProjectManager}" Group="Base.Foundation.Columns"  MaxLength="255" AllowDeletion="TRUE" />
        <Field ID="{F1A1715E-6C52-40DE-8403-E9AAFD0470D0}" Type="Text" Name="DocumentDescription" DisplayName="{localization:core:Description}" Group="Base.Foundation.Columns" MaxLength="255" AllowDeletion="TRUE" />
</pnp:SiteFields>

or maybe it does work only in some elements ?
@PaoloPia , @erwinvanhunen can you confirm that synthax {localization:name:key} does not work or work only in specific cases ?

Hi all,
After reviewing the code, seems the TokenParser is never considering the Name from the Localization entries, so actually the code only supports one set of resx files (sample in the Provisioning Schema is not working), and the use as: {localization:NavHome}
I have submitted a Pull Request to the PnP Core library (https://github.com/SharePoint/PnP-Sites-Core/pull/2367) that should fix this issue. As I don´t want to change too many lines of code, what I´m doing when the token is added to the Token collection in TokenParser, is prefixing it with the Localization Name value, so the Token Key is always something like: intranet:ProjectID
However, this code forces you to use the Token including the Name (i.e: {localization:intranet:ProjectID}), and this is not good if you just have one set of resx files, and this likely breaks most of the existing templates. To avoid that and adding compatibility backwards, the code is checking if there´s only one set of resx files, and if so, the key is built without the Name prefix.
I´d appreciate if any of you can test my PR, cos although I think I´m covering all the scenarios, I might be missing something.
Thanks!


feat: database cleanup (resolves #935)

[x] Removes formats (#935)
[x] Removes phases (#935)
[x] Removes consulting_methods (#935)

Prerequisites
If this PR changes PHP code or dependencies:

[x] I've run composer format and fixed any code formatting issues.
[x] I've run composer analyze and addressed any static analysis issues.
[x] I've run php artisan test and ensured that all tests pass.
[x] I've run composer localize to update localization source files and committed any changes.

If this PR changes CSS or JavaScript code or dependencies:

[x] I've run npm run lint and fixed any linting issues.
[x] I've run npm run build and ensured that CSS and JavaScript assets can be compiled.

@greatislander where can I see the changes being used when running the application?

@greatislander so far I've only seen the Consultations settings in the "Communication and consultation preferences" page. There is a bug here. If you change any of the checkboxes they all change together.

Just pushed a fix for this, thanks for catching. This is the only user-facing change. Phases and Formats are attached to resources which are not fully implemented yet, so those changes don't impact any user-facing UI.

@greatislander in testing out the latest fix, I'm getting an error after saving the Consultations change.
resources / views / settings / communication-and-consultation-preferences.blade.php : 105
htmlspecialchars(): Argument #1 ($string) must be of type string, array given

I had made a couple of selections, including "workshop" and selecting "In person" from the second set of checkboxes.

I tried again just selecting the first checkbox and saving, the same issue occurred.


Can't reply long messages

As you can see, there's no space for me to send the message (couldn't even with "send pressing INTRO" option)

Ad-hoc solution to this problem is to turn your device into landscape orientation. Then large Send button should be visible.
I will find better solution to this...

Maybe we could Auto hide snd show Action bar somehow.
See e.g. http://www.devexchanges.info/2015/09/android-showhide-actionbar-when.html?m=1

Implemented in v.26.01


Wrong flag given.

After running VMAware in terminal i got this output along with a positive result due to what is ebing said below as a result of this i decided to copy paste the flag --discard-hyperv-host but upon running i was given Unknown argument provided, consult the help menu with --help which is what i had copy and pasted from below upon checking the help menu with --help i relasied you had run with the flag --disable-hyperv-host so all in all small typo to be chnaged in commits thanks for the work outside of this works a charm.
[    NOTE    ] If you know you are running on host, Hyper-V virtualises all applications by default within the host system. This result is in fact correct and NOT a false positive. If you do not want Hyper-V's default virtualisation enabled, run with the "--discard-hyperv-host" argument. See here https://github.com/kernelwernel/VMAware/issues/75
====== CONCLUSION: Running inside a Microsoft Hyper-V VM ======

fixed https://github.com/kernelwernel/VMAware/pull/115


contextcompat.checkselfpermission not working

Hey @pbakondy I am facing error while building my app, at this line, in sim.java file at line number 233,
ContextCompat.checkSelfPermission(this.cordova.getActivity(), type));

not enough information
what is the error? environment? etc
On 25 Nov 2016 11:24, "Premkumar Agrawal"<EMAIL_ADDRESS>wrote:

Hey @pbakondy https://github.com/pbakondy I am facing error while
building my app, at this line, in sim.java file at line number 233,
ContextCompat.checkSelfPermission(this.cordova.getActivity(), type));
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pbakondy/cordova-plugin-sim/issues/44, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAej2XBgE6qlFnjjf-oz5SXoDXfDOnFmks5rBrdWgaJpZM4K8TL6
.

After hitting ionic run android
I am getting following response on command prompt,
`F:\A_LHS_Projects\DynamicApp\DA_Code\DynamicApp_V2\platforms\android\src\com\pbakondy\Sim.java:233: error: cannot find symbol
ContextCompat.checkSelfPermission(this.cordova.getActivity(), type));
^
symbol:   method checkSelfPermission(Activity,String)
location: class ContextCompat
Note: Some input files use or override a deprecated API.
FAILED
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
1 error
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':compileDebugJavaWithJavac'.

Compilation failed; see the compiler error output for details.

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

BUILD FAILED
Total time: 13.068 secs
Error: Error code 1 for command: cmd with args: /s,/c,"F:\A_LHS_Projects\DynamicApp\DA_Code\DynamicApp_V2\platforms\android\gradlew cdvBuildDebug -b F:\A_LHS_Projects\DynamicApp\DA_Code\DynamicApp_V2\platforms\android\build.gradle -Dorg.gradle.daemon=true -Pandroid.useDeprecatedNdk=true"
F:\A_LHS_Projects\DynamicApp\DA_Code\DynamicApp_V2>`

Hey @pbakondy  did you understood the issue..

Did you install Android requirements?
https://github.com/pbakondy/cordova-plugin-sim#requirements

@pbakondy  I have following support files of Android SDK installed

What is the version of Android Support Library?
Please run Android SDK Manager and install latest packages.
Which SDK are you using?

@pbakondy My Android SDK version is 6.0 (API 23)
Android Support Library 23.2.1 ![capture](https://cloud.githubusercontent.com/assets/9786096/20625194/b951f8c4-b337-11e6-9f53-96a9c3b9e2dd.PNG)

As I see Android Support Library is not installed yet

Ok, let's try something else.
I will make a few code changes, could you try it for me by installing it
from github?
On 25 Nov 2016 13:21, "Premkumar Agrawal"<EMAIL_ADDRESS>wrote:

@pbakondy https://github.com/pbakondy My Android SDK version is 6.0
(API 23)
Android Support Library 23.2.1 ![capture](https://cloud.
githubusercontent.com/assets/9786096/20625194/b951f8c4-
b337-11e6-9f53-96a9c3b9e2dd.PNG)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pbakondy/cordova-plugin-sim/issues/44#issuecomment-262945962,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAej2ZEL28JOV7kdA_5D2uRyPfTzC0NDks5rBtKxgaJpZM4K8TL6
.

Yaa Sure... @pbakondy

Please try install the latest:
cordova plugin rm cordova-plugin-sim
cordova plugin add https://github.com/pbakondy/cordova-plugin-sim.git

Should i run both the commands

Hey @pbakondy have some urgent work..
will try later and get back to you...
Thanks for your instant support....

Yes, both commands
The first one removes the installed plugin from npm, the second one installs the latest code

Hey @pbakondy  I installed SIM plugin successfully, & its working finely. Thanks for your support..

Hi. what changes have you made in the code. I am facing the same issue.. please mention the changes, it will be useful for me.
Thanks in advance


Progress Bar support

Any plans on adding support for a progress bar?

A progress bar block should be relatively trivial to support.
Would you be able to elaborate more on your use case?  I know dunst has this, but I'm unclear on the specifics, or if you want anything different to what they have over there.
From what I can tell, with dunst, you normally send a notification with a progress hint; notify-send -h int:value:42 "Working ...", and then the notification displays some progress bar if that hint is present.
We'd probably want to use render_criteria to decide when to draw the progress bar, and just have a block that reads that value and shows it.  You'd have to replace the notification using its ID if you wanted to update a volume slider or something like that, which I don't think notify-send supports.  notify-call does support that but it's less prominent.
Would simply a block like that obeys a hint satisfy you, or is there anything else you want to add?
The only thing I'm wanting to avoid is making a custom wired-send tool or something that supports custom functionality.  I think wired should just be a notify receive tool and other things are better suited to sending notifications, like notify-call.
Apologies for the wall of text.

Dunst doesn't support styling the progress bar and renders it in a fixed position. This kinda defeats the purpose, for my use case, as I'm trying to have it blend with the body of the notification.
I should be able to place the notification wherever I'd like to, whether that's the body or header.
For example, I'd live the bar volume to be in body:

This is available on master.  You can add a progress block to your config with the name ProgressBlock:
        (
            name: "progress",
            parent: "body",
            hook: Hook(parent_anchor: BL, self_anchor: TL),
            offset: Vec2(x: 0.0, y: 0.0),
            params: ProgressBlock((
                padding: Padding(left: 3.0, right: 3.0, top: 3.0, bottom: 3.0),
                border_width: 3.0,
                border_rounding: 10.0,
                border_color: Color(hex: "#ebdbb2"),
                background_color: Color(hex: "#ff0000"),
                fill_color: Color(hex: "#00ff00"),
                size: Vec2(x: 100.0, y: 30.0), // x: -1.0, and y: -1.0 will match parent rect size.
                border_color_hovered: Color(hex: "#fbf1c7"),
                background_color_hovered: Color(hex: "#3c3836"),
                fill_color_hovered: Color(hex: "#fbf1c7"),
            )),
        ),

It reads the the value hint, which you can send like the following:
notify-send -h int:value:<progress> "Working..." "I'm working on something."
Where <progress> is an integer from 0-100.
If you get a chance to try it, let me know if you come across any issues or things you would like changed.

Released in 0.9.2.