Unable to create a new plex-pass container

I am using this command to check the good start of a new container : docker run --name=plex -v /shyrka/docker/plex/config:/config -v /Volumes/music:/media --net=host -p 32400:32400 -e PLEXPASS_LOGIN='snip' -e PLEXPASS_PASSWORD='snip' wernight/plex-media-server:autoupdate
And it seems there is a problem during the generation of the deb url ?
8192
+ /retrieve_plex_download_url.py
Retrieving the latest Plex release for PlexPass users...
Traceback (most recent call last):
  File "/retrieve_plex_download_url.py", line 51, in <module>
    print(retrieve_latest_download_url(login, password))
  File "/retrieve_plex_download_url.py", line 40, in retrieve_latest_download_url
    assert len(links) == 1
AssertionError
+ curl -L -o plexmediaserver.deb
curl: no URL specified!
curl: try 'curl --help' or 'curl --manual' for more information

I am new to docker, so maybe I miss something.

Ok, I tried to find the problem, and found that this part of the script curl -Ls https://plex.tv/downloads | grep -o '^"'"'"'*amd64.deb' | grep -v binaries doesn't works, and return empty strings.
In a shell, I have tried using this command instead : curl -Ls https://plex.tv/downloads | grep -o 'http.*amd64.deb' | grep -v binaries and it return a full valid url : https://downloads.plex.tv/plex-media-server/<IP_ADDRESS>.1620-e0b7243/plexmediaserver_<IP_ADDRESS>.1620-e0b7243_amd64.deb
The problem, is I am not able to udpate the script as I don't know how to modifiy it.

Ok, I tried to find the problem, and found that this part of the script curl -Ls https://plex.tv/downloads | grep -o '^"'"'"'*amd64.deb' | grep -v binaries doesn't works, and return empty strings.
In a shell, I have tried using this command instead : curl -Ls https://plex.tv/downloads | grep -o 'http.*amd64.deb' | grep -v binaries and it return a full valid url : https://downloads.plex.tv/plex-media-server/<IP_ADDRESS>.1620-e0b7243/plexmediaserver_<IP_ADDRESS>.1620-e0b7243_amd64.deb
The problem, is I am not able to udpate the script as I don't know how to modifiy it.

Found the problem : the links text for plex pass have changed, 64-bit is now 64 bits here is the change :
links = [link for link in browser.links() if '.deb' in link.url and (link.text == '64-bit' or link.text == '64 bits')]

Found the problem : the links text for plex pass have changed, 64-bit is now 64 bits here is the change :
links = [link for link in browser.links() if '.deb' in link.url and (link.text == '64-bit' or link.text == '64 bits')]

Thanks for the notification, I'll update once I'm on a stable network.
On Fri, 1 Jan 2016 at 11:13 iXo<EMAIL_ADDRESS>wrote:

Found the problem : the links text for plex pass have changed, 64-bit is
now 64 bits here is the change :
links = [link for link in browser.links() if '.deb' in link.url and (link.text == '64-bit' or link.text == '64 bits')]
—
Reply to this email directly or view it on GitHub
https://github.com/wernight/docker-plex-media-server/issues/8#issuecomment-168297042
.

Thanks for the notification, I'll update once I'm on a stable network.
On Fri, 1 Jan 2016 at 11:13 iXo<EMAIL_ADDRESS>wrote:

Found the problem : the links text for plex pass have changed, 64-bit is
now 64 bits here is the change :
links = [link for link in browser.links() if '.deb' in link.url and (link.text == '64-bit' or link.text == '64 bits')]
—
Reply to this email directly or view it on GitHub
https://github.com/wernight/docker-plex-media-server/issues/8#issuecomment-168297042
.


Bounty computers are not making list items properly.

Description
I'm unsure on replication as it only happens sometimes, but in the screenshot example after the bowl of chili example all the print buttons print the bowl of chili, it seems like the list is not being populated correctly. When skipping the chilly all the following ones are off by one.
Reproduction
Try adding bowl of chilli could be the broken bounty proto as a start.
Screenshots

Additional context
Have heard also from other aHelps when bounties being sold the bounty not being removed/ completed from the computer list.

I was never able to replicate this locally or on another server and when I was testing it, getting the reports at the time it only happened on grass hopper. I'm going to close this for now and see if it comes back.


Custom Recipe Development, JS Error: Unexpected Token Import

Attempting to create a custom discord recipe. After dragging files into the dev folder inside of library. Once I go to run the service the js console outputs:
eafb5fe485657e3e10aa.js:17 SyntaxError: Unexpected token import
    at createScript (vm.js:53:10)
    at Object.runInThisContext (vm.js:95:10)
    at Module._compile (module.js:543:28)
    at Object.Module._extensions..js (module.js:580:10)
    at Module.load (module.js:488:32)
    at tryModuleLoad (module.js:447:12)
    at Function.Module._load (module.js:439:3)
    at Module.require (module.js:498:17)
    at require (internal/module.js:20:19)
    at EventEmitter._electron.ipcRenderer.on (/Applications/Franz.app/Contents/Resources/app.asar/webview/plugin.js:30:5)
    at emitTwo (events.js:106:13)
    at EventEmitter.emit (events.js:194:7)

you need to replace import foo from 'foo'; with const foo = require('foo');
This needs to be updated in the docs as we are transpiling the recipes before deployment.


Force remove SSH keys

Right now we hard code force=false for deleting keypairs. This means that we will always orphan a keypair in AWS if a user creates a cluster with a unique keypair and then destroys it after.
Can we please expose this flag via an environmental variable, document the environmental variable in /docs/envvar.md, and patch this?
Suggested variable name: KUBICORN_FORCE_DELETE_KEY
The system as it stands today in AWS is here: https://github.com/kris-nova/kubicorn/blob/master/cloud/amazon/resources/keypair.go#L137-L151

I will take over this one!


Fix compression parameter to compress in Judges::Push#run for BazaRb.new

Closes #127

@yegor256 please check

@rultor release, tag is 0.26.1

@rultor release, tag is 0.26.1

@yegor256 OK, I will release it now. Please check the progress here

@Suban05 Hey there! 👋 Thanks for your review! You've earned +4 points, which breaks down like this: +4 base points, -10 for having only 2 comments (policy suggests at least 6), and +10 to keep you motivated. While more comments could've boosted your score (up to 8 points!), we appreciate your effort. Remember, even reviewing your own code can earn points! Your current balance is +387. Keep up the good work and aim for more detailed reviews next time! 💪

@rultor release, tag is 0.26.1

@yegor256 Done! FYI, the full log is here (took me 29min)


Update docs: vagrant up order

Vagrant up examples place boot at the end of the host list. This cases the agents to fail when attempting to install dcos agent.
This merge puts the boot host after the master(s) on all vagrant up examples.
Fixes: Following documentation causes failed deployment.

Duplicate: https://github.com/dcos/dcos-vagrant/pull/69
I haven't been able to duplicate this problem, but @onlydole's issue seemed to be fixed by doing a vagrant destroy -f and trying again.
There was a long slack discussion and debug thread that started here: https://dcos-community.slack.com/archives/dcos-vagrant/p1464711342000884
If you can supply more steps to duplicate after a complete destroy, I might be able to help more, but the proposed solution is technically wrong and cannot be accepted because it makes deployment slower.

Looks like https://dcosjira.atlassian.net/browse/VAGRANT-20 is caused by DNS hijacking. So this is not the right solution. Tracking in the JIRA ticket.

Kudos on this one @karlkfi!


Conditional requirements are broken with wheel

in win7
C:\Users\Lenovo>pip install --upgrade httpie
Requirement already up-to-date: httpie in c:\python27\lib\site-packages
Cleaning up...

C:\Users\Lenovo>http httpie.org
Traceback (most recent call last):
  File "C:\Python27\lib\runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "C:\Python27\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "c:\Python27\Scripts\http.exe\__main__.py", line 5, in <module>
  File "C:\Python27\lib\site-packages\httpie\__main__.py", line 6, in <module>
    from .core import main
  File "C:\Python27\lib\site-packages\httpie\core.py", line 23, in <module>
    from .downloads import Download
  File "C:\Python27\lib\site-packages\httpie\downloads.py", line 15, in <module>

    from .output import RawStream
  File "C:\Python27\lib\site-packages\httpie\output.py", line 19, in <module>
    from .models import HTTPRequest, HTTPResponse, Environment
  File "C:\Python27\lib\site-packages\httpie\models.py", line 8, in <module>
    class Environment(object):
  File "C:\Python27\lib\site-packages\httpie\models.py", line 34, in Environment

    from colorama.initialise import wrap_stream
ImportError: No module named colorama.initialise

This should be fixed in HTTPie v0.9.0. See:
https://github.com/jakubroztocil/httpie/issues/255#issuecomment-72316343


chore: use gitattributes

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


make package doesn't package up a couple of qt5 libraries

I did a make package on the trunk and the distribution it built was missing the following required qt5 libraries: libQt5Svb.solibQt5Concurrent.so
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2769
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: make package doesn't package up a couple of qt5 libraries
Assigned to: -
Category: -
Target version: 3.0
Author: Eric Brugger
Start: 02/17/2017
Due date:
% Done: 0%
Estimated time:
Created: 02/17/2017 01:14 pm
Updated: 02/28/2017 06:30 pm
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: trunk
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
I did a make package on the trunk and the distribution it built was missing the following required qt5 libraries: libQt5Svb.solibQt5Concurrent.so
Comments:

The first library should probably be Svg, not Svb.

I just checked the third party libraries on quartz and libQt5Svg and libQt5Concurrent are both present. I then checked the installation of 3.0.1 and those libraries were there as well. The installation was installed from a distribution created by "make package". Therefore, this is no longer an issue.


File size / file name limitation?

Is there any known file size or file name limitation that would prevent loading certain files?
Thank you
Cross-issue https://github.com/xlfe/spiffy/issues/7

File name is limited to 32 chars, IIRC.

Thank you for your reply, @raz123
Our file names are less than 32 chars. See spiffy log output:
Creating rom spiff_rom.bin of size 196608 bytes
Adding files in directory files
Unable to read file .
Unable to read file ..
bootstrap.css.gz added to spiffs (15615 bytes)
index.html added to spiffs (2876 bytes)
jquery.js.gz added to spiffs (30153 bytes)

The default config for filename size is 32 chars, true. I do not think it is changed on the nodemcu project.  File sizes - well, the max length depends on pretty many things. The length field is 32 bits, so that's not it.
Then, I think the span field config on nodemcu is 16 bits with pages of 256 bytes, giving roughly 250*65536 bytes max, so that's cool also.
And then I think, filesystem has roughly 40k of spi flash to play with. The spiff_rom.bin of 196608 bytes seems large in this sense.
I'll have to check what spiffy actually does.

Ping?

Pong? :)

Sorry for undirected ping, raz :) It was intended to @kireevco

we are using spiffs on sming. not nodemcu. spiffy using "cfg.log_block_size = SPI_FLASH_SEC_SIZE" main problem is this. if we change "cfg.log_block_size = SPI_FLASH_SEC_SIZE*2" finally we can use ~142kb on filesystem. also spiffs failing if total files bigger than ~142kb. (we have free 192kb on filesystem)
btw: here is sming source. https://github.com/anakod/Sming

Ah I see. What errorcodes do you get? I'll see if I can browse your code
this evening.
2015-04-17 9:04 GMT+02:00 alonewolfx2<EMAIL_ADDRESS>
we are using spiffs on sming. not nodemcu. spiffy using
"cfg.log_block_size = SPI_FLASH_SEC_SIZE" main problem is this. if we
change "cfg.log_block_size = SPI_FLASH_SEC_SIZE*2" finally we can use
~142kb on filesystem. also spiffs failing if total files bigger than
~142kb. (we have free 192kb on filesystem)
btw: here is sming source. https://github.com/anakod/Sming
—
Reply to this email directly or view it on GitHub
https://github.com/pellepl/spiffs/issues/12#issuecomment-93923475.

"write errno -10001" its appearing after ~141kb

Take this opportunity to say thank you for the excellent work, pellepl!

@anakod agree :) @pellepl thank for excellent work

blush Thanks for the kind words! Perhaps you should de-emphasize the praise until I've found the problem ;) It occurs to me that this might not be a bug, but what you see is a replica of  https://github.com/nodemcu/nodemcu-firmware/issues/309 (see lengthy comment by me somewhere in the middle)
Anyhow, in SmingFramework/Services/SpifFS/spiffs.c you have a spiffs_mount() function. I need to know exactly what is in you cfg struct before you send it to SPIFFS_mount().
Also, are you running the latest and greatest spiffs version? Seems to be some parts missing in the config for the new stuff.
My hopes are that spiffs could be tuned a wee bit better for the ESP8266 altogether, for both nodemcu and sming by simply configuring it differently.

@pellepl I have a question:
https://github.com/anakod/Sming/blob/master/Sming/Services/SpifFS/spiffs.c#L56
Thisout that change (* 2) I can't write to the end of filesystem (it's crashed). Why? Is it right fix, or we should change something else?

Hmm, it should definitely not crash. Did you get a dump or a trace? If there is a bug you've found, any info would help.
In my testbench it works filling the FS with what I presume are the same params - I still need to know the exact contents of the config struct Sming sends into SPIFFS_mount().
Other than that, it is fully legal to have a logical block size that is two times the size of a physical sector. It's all in the https://github.com/pellepl/spiffs/blob/master/docs/TECH_SPEC. In short, it works like this:
The SPI flash has sector boundaries - say 4096 bytes as in the ESP8266 case. Spiffs groups the SPI flash into logical blocks, which must be on sector size boundary, in our case e.g. 4096, 8192, etc. Each block is then divided into pages, in your config 256 bytes. So, having a sector size of 4096, a block size of 8192, a page size of 256 gives 4096*2/256 = 32 pages.
Also, each logical block contains a meta data look up table, stealing at least one page per block. In the case of having a block size of 4096 and a page size of 256, 1/16th or 6.25% of the SPI flash is used for LUTs. Having a block size of 8192 instead, 1/32th or 3.125% if the SPI flash is used for LUTs. The former case is not a very optimal configuration, because only 25% of the LUT will be used as there are only 15 actual file data pages per block to index. The latter case is a better solution.
Still it must not crash. It is confusing, because the nodemcu project uses this configuration (4096 bytes block, 256 bytes pages) without problems (?). Is sming using the latest version? Any details you could provide would be great.

here is cfg config. https://github.com/anakod/Sming/blob/master/Sming/Services/SpifFS/spiffs_config.h#L168

Yes, the static config I've found. The runtime I need from someone with a board.

@pellepl  if you preffer any change i can compile and test on the board

Ok great, before SPIFFS_mount at  https://github.com/anakod/Sming/blob/master/Sming/Services/SpifFS/spiffs.c#L108, insert
debugf("spiffs.phys_size: %d\n", cfg.phys_size); debugf("spiffs.phys_addr: 0x%08x\n", cfg.phys_addr); debugf("spiffs.phys_era_blk: %d\n", cfg.phys_erase_block); debugf("spiffs.log_blk: %d\n", cfg.log_block_size); debugf("spiffs.log_pg: %d\n", cfg.log_page_size);
and attach the output here I can start digging for real.

fs.start: size:192 Kb, offset:4024C000

spiffs.phys_size: 196608

spiffs.phys_addr: 0x4024c000

spiffs.phys_era_blk: 4096

spiffs.log_blk: 8192

spi

Thanks! But what happened to the log_pg output?

it seems module trying to connect network before that line. i will disable network and try again. wait a minute please

here is it.
fs.start: size:192 Kb, offset:4024C000
spiffs.phys_size: 196608
spiffs.phys_addr: 0x4024c000
spiffs.phys_era_blk: 4096
spiffs.log_blk: 8192
spiffs.log_pg: 256

Cheers! Will dig into it asap.

I don't know if it independent problem or same, therefore I can add this into this thread.
I would like to test SPIFFS with small erase sector. It is test for Micron NOR flash 4kB subsectors. But SPIFFS tests didn't pass.
I change ONLY params_test.h and only this lines:
#define SECTOR_SIZE         8192
#define LOG_BLOCK           8192
#define LOG_PAGE            256
After "make test" I see to message about failed "long_run_config_many_medium"
And with real 4kB sector it is total lost. Let's have this config:
#define SECTOR_SIZE         4096
#define LOG_BLOCK           131072
#define LOG_PAGE            256
and result:
TEST 33/56 : running test file_uniqueness
creating 3072 files
checking 3072 files
removing 1536 files
creating 1536 files
make: *** [test] Segmentation fault
What I doing wrong?

@dobryj In the first case with 8k blocks, the test fail since the fs runs out of space. The testcase is one of the most complicated ones, and was written to totally fill the system using 64k blocks. Having 8k blocks will add more metadata which steals more actual data, and presumably I was too lazy to cover all possible configs in the test case when writing it.
I would say you can safely ignore this failure.
As for the 128k block size with 4k sectors, that must be a bug in the test setup. I'll have to check it. It works with 128k block size with 64k sectors, which makes no sense, as the sector size should not affect in this way.

Segmentation fault problem, from my comment  can be solved by change one line in test_spiffs.c to
static int erases[PHYS_FLASH_SIZE / SECTOR_SIZE];
Problem was fixed size of array to 256. I it was enought only for default test configuration.

@anakod Regarding the max file size of 141kB when using 192kB of space: this is not a bug.
Parts of the "lost" space is eaten by metadata. Other parts are reserved as free buffer in order to do garbage collecting. And the remaining data that cannot be written is due to the same reason as is mentioned in https://github.com/nodemcu/nodemcu-firmware/issues/309. I was fooled a bit there, I thought the free size of the spiffs system was 192kB, not the physical free space.

@anakod Regarding the crash you mentioned: was this in the testbench (same as @dobryj found) or was it on target? If in testbench, then it is fixed and pushed to master.
If it is on target, things are worse. I've been testing it quite hard here, both on another target and in the testbench - but I cannot reproduce it. Without more info it's hard to straighten out what could be wrong. I would at least need a stack trace or something to begin with.
In any case, that is another issue - if there was a target crash, please open another issue and provide all info you can give me. I get the feeling the ESP8266 is not very transparent, can anyone jtag it or are you guys bound to printf-debugging only?


Rename gas security checker to gosec

cc @alecthomas

Tests are not working.

@alecthomas I had to rework a bit the test. gosec expects now the files to be in the GOPATH, because it analyses the entire package. It doesn't work any more with temporary files.
cc @gcmurphy

This should also fix issue reported in https://github.com/alecthomas/gometalinter/pull/499#issuecomment-407133522

Thanks for this, I appreciate it.

@alecthomas Thanks for your suggestion. I fixed the test code accordantly.

Brilliant, thanks!

Could it be that this change breaks usage of // nolint: gosec?
I can still disable using // nolint, but not when I only want to disable gosec.

@alecthomas @ccojocar
This is a breaking change. We were running the linter with --disable=gosec, which then started giving issues when the rule was renamed.
Could you please bump the major version to v3.0.0?
Tools like http://labix.org/gopkg.in are designed so that breaking changes have to go into major versions.

This change is not yet part of a release. If you're concerned about breaking changes use a tag. v2.0.6 is the latest.

Hello @ccojocar, this change appears to be having some issues. See #510 for example. Mind taking a look?


Change the name of the committer on the Hub

Currently, the parquet files are committed by https://huggingface.co/francky (see https://huggingface.co/datasets/glue/commits/refs%2Fconvert%2Fparquet).
Create a dedicated user like parquet-converter or converter or datasets-server, or...
Alternative: use an app token to do the commits. I understand that we should soon have the feature, right @SBrandeis?

Alternative

this is orthogonal, no? (You'll still need your commits linked to a specific user)

as discussed let's just try renaming the existing user to parquet-converter and hope it works out of the box:)

By the way, as we only rely on the token, no change is required here in the datasets server itself. I'll keep open only to track the user renaming.

Alternative: use an app token to do the commits. I understand that we should soon have the feature, right @SBrandeis?
this is orthogonal, no? (You'll still need your commits linked to a specific user)

See https://github.com/huggingface/datasets-server/issues/1283


[ForwardRef in snapshot

When I upgrade to @material-ui/core 4 and start using @material-ui/styles, every snapshots failed.
 Received value does not match stored snapshot "[Component] Menu should render correctly 1".

    - Snapshot
    + Received

    @@ -1,6 +1,6 @@
    - <WithStyles(Paper)
    + <WithStyles(ForwardRef(Paper))
        square={false}
        style={
          Object {
            "left": 0,
            "position": "absolute",

The code didn't change at all.
const Menu = (props: MenuProps<OptionType>): ReactElement => (
  <Paper square={false} style={{ position: 'absolute', zIndex: 1, left: 0, right: 0 }} {...props.innerProps}>
    {props.children}
  </Paper>
);

Is it normal ? Can we do something about it ? I think the ForwardRef in the snapshot add useless complexity.
I even have sometime have both WithStyle and ForwardRef appearing in the snapshot since the update.
 @@ -1,6 +1,6 @@
    - <TextField
    + <WithStyles(ForwardRef(TextField))
        InputLabelProps={
          Object {
            "shrink": true,
          }
        }

That is an inherent flaw with snapshot testing. If you serialize your snapshot with implementation details (or test them in general) they can fail at any time.

I think the ForwardRef in the snapshot add useless complexity.

Depends. They do change the API of the component. They're only useless if you don't care about refs.

@eps1lon So If I correctly understand, you changed the implementation of all MaterialUi Component and add ForwardRef to every one, that's why I see this now in the snapshot ?
Thanks for your answer

@eps1lon So If I correctly understand, you changed the implementation of all MaterialUi Component and add ForwardRef to every one, that's why I see this now in the snapshot ?

Yes. It wasn't tagged as a breaking change in the release notes though. The PR properly marked itself as breaking though. Will fix that.
It's hard to recommend a silver bullet to snapshot testing with 3rd party libraries. Depending on your test it's probably best to serialize them shallow with the name  of the identifier i.e. if you import it as MuiPaper it should be serialized with that name instead of its displayName. I don't know if there are existing solutions for this.
Serializing the whole component tree of 3rd party components is brittle. Imagine those components only introduce a headless component that is essentially an identity function i.e. const MeaningLess = props => props.children. Nothing changed about the public API (unless perf is part of your public API) but the snapshot will fail regardless.
I used snapshot testing personally but it gives you a false sense of security. It leads to throwaway test like it renders correctly that only give you a nice coverage number but no confidence in the behavior. They can be done right but just like any other test they need some setup.

I made a makeStyles mock to display all css properties passed to the component

@VincentLanglet We have a Jest issue focusing on improving the snapshot support: #14357. Do you think that you would be able to share the solution there?


Readme Updates

List of possible Readme updates is described in #40
Initial commit for points 1, 2

The note about the testing with the Ubuntu distros should be updated. Remove 14.04 and add 18.04
As a side note, maybe a Travis job can also be added for Ubuntu 18.04 OpenCV installation

@rsnk96 are these additions fine? Any changes for the grammar, description, etc.
I'll squash them once approved


Simplifying idtac;; k into k yields "Inconsistent implicit qualifier" after a long pause

I posted a repro on cpitclaudel_slow_idtac.
In the code I posted, the call to repeat succeeds quickly if I write it as (repeat (idtac;; …)), but fails slowly if I write only (repeat (…)).
Is this a known issue?

(repro is commit bc3ed50e3273e5c39c3db8d8be270c0237cf87a2)
As far as the error goes, it is ./examples/pattern-matching/PatternMatching.fst(552,0-570,25): (Error) Inconsistent implicit qualifier (see also ./examples/pattern-matching/PatternMatching.fst(569,59-569,63)), which is bizarre. It arises when checking that implicits where solved, and typechecking a solution.
As for the slowness, the proofterm generated by the tactic (which, note, succeeds) is absolutely huge, and seems to contain a lot of redexes. While with the extra bind, it is much smaller.
Smells like funky normalizer business.

Also caused by tactic re-typechecking, which is now gone.


example improvement?

is this the correct import?
import MetaMaskProvider from 'metamask-react';
×
Error: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. You likely forgot to export your component from the file it's defined in, or you might have mixed up default and named imports.
*** Fixed it ***
import {MetaMaskProvider} from 'metamask-react';
import {useMetaMask} from 'metamask-react'

Hey! Thanks a lot for reporting the issue!
There is not default export indeed, only named exports :)
I'm gonna update the example part in the README in order to clarify that

Also thank you for making a great library I appreciate it makes testing easier for Dapps


Some candidates appear to exist more than once in the database

For instance, Tom Berryhill has two different filer IDs and two different candidate pages on the CAL-ACCESS website.
http://cal-access.ss.ca.gov/Campaign/Candidates/Detail.aspx?id=1294245
http://cal-access.ss.ca.gov/Campaign/Candidates/Detail.aspx?id=1254687

This has been managed by our merge command.


Generate treq-using Python client code

requests is great for synchronous http
treq is great for asynchronous http

requests is great for asynchronous http too, with a help from gevent. Socket implementation should not affect the awesomeness of a higher-level interface :)

requests is great for asynchronous http too, with a help from gevent. Socket implementation should not affect the awesomeness of a higher-level interface :)

Okay.  Sure.  Sounds like that would make a good, separate ticket too.

Okay.  Sure.  Sounds like that would make a good, separate ticket too.


bluetooth: fix GATT service reregistering

Fixed issue with reregistering of a GATT services.
Added unit tests covering the GATT reregistering scenario.

Please add Fixes #67377 to the commit message.

That should be in the PR description, not necessarily in the commit message (in fact, references like that in commit messages loose their meaning if the project ever moves to a different location or if someone looks at the commit history in some fork of upstream).

Please add Fixes #67377 to the commit message.

That should be in the PR description, not necessarily in the commit message (in fact, references like that in commit messages loose their meaning if the project ever moves to a different location or if someone looks at the commit history in some fork of upstream).

Ok, I just find it useful to myself at least when going through the git history through git log.

Ok, I just find it useful to myself at least when going through the git history through git log.

Sure, which is why I wasn't trying to outright forbid it. However, GitHub doesn't look at commit messages when it links PRs and issues together, which is why it's important that the reference is at least in the PR description (a contributor might forget to update that if they just modify the commit message and push).

Ok, I just find it useful to myself at least when going through the git history through git log.

Sure, which is why I wasn't trying to outright forbid it. However, GitHub doesn't look at commit messages when it links PRs and issues together, which is why it's important that the reference is at least in the PR description (a contributor might forget to update that if they just modify the commit message and push).

I see. Usually, when PR has a single commit, it content is put into the description. But yes, you are right, this should be in the PR description for GitHub to match the PR with the issue.


Update TRN endpoint for new overseas regs

Context
Regs have changed regarding overseas qualified teachers. This updates the TRN endpoint with new rules.
Changes proposed in this pull request

Removes the EuropeanEconomicArea route.
Add validation for InitialTeachingTraining.TrainingCountry.

Checklist

[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

https://trello.com/c/UF2GlW6c/1191-update-api-endpoint-for-new-regulation-changes


Elvish crashes in completion filtering

if you input something as filter and press tab elvish crashes with
runtime error: index out of range
goroutine 1 [running]:
github.com/elves/elvish/sys.DumpStack(0xc4205c8918, 0x1)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/sys/d2
github.com/elves/elvish/program/shell.rescue()
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progr5
panic(0x87f0e0, 0xbc02a0)
        /usr/lib/go/src/runtime/panic.go:502 +0x229
github.com/elves/elvish/edit/completion.(*completion).selectedCandidate(...)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/4
github.com/elves/elvish/edit/completion.(*completion).Replacement(0xc42023c000, 0x975800, 0xc42023c000, 0x7fee4d4a8d68)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/9
github.com/elves/elvish/edit/edcore.(*editorRenderer).Render(0xc4204ee900, 0xc4203f20a0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b
github.com/elves/elvish/edit/ui.Render(0x971740, 0xc4204ee900, 0x77, 0x972680)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b
github.com/elves/elvish/edit/edcore.(*editor).refresh(0xc4201e8000, 0x7fee4d420101, 0xc4203740a0, 0xc4203428c0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b
github.com/elves/elvish/edit/edcore.(*editor).CallFn(0xc4201e8000, 0x7fee4d422040, 0xc4203740a0, 0xc4203428c0, 0x1, 0x)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/e
github.com/elves/elvish/edit/edcore.callHooks(0xc4201e8000, 0x9787c0, 0xc4202a2900, 0xc4203428c0, 0x1, 0x1)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/7
github.com/elves/elvish/edit/edcore.init.1.func1.2(0xc4204db178, 0x3)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/a
github.com/elves/elvish/edit/edcore.(*editor).finishReadLine(0xc4201e8000, 0x0, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/6
github.com/elves/elvish/edit/edcore.(*editor).ReadLine.func1(0xc4201e8000)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/f
panic(0x87f0e0, 0xbc02a0)
        /usr/lib/go/src/runtime/panic.go:502 +0x229
github.com/elves/elvish/edit/completion.(*completion).selectedCandidate(...)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/4
github.com/elves/elvish/edit/completion.(*completion).Replacement(0xc42023c000, 0x975800, 0xc42023c000, 0x7fee4d4a8d68)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/9
github.com/elves/elvish/edit/edcore.(*editorRenderer).Render(0xc420529d80, 0xc420352550)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b
github.com/elves/elvish/edit/ui.Render(0x971740, 0xc420529d80, 0x77, 0x972680)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b
github.com/elves/elvish/edit/edcore.(*editor).refresh(0xc4201e8000, 0x970101, 0xc420234500, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b
github.com/elves/elvish/edit/edcore.(*editor).CallFn(0xc4201e8000, 0x9717c0, 0xc420234500, 0x0, 0x0, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/e
github.com/elves/elvish/edit/edcore.(*editor).ReadLine(0xc4201e8000, 0x0, 0x0, 0x0, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/f
github.com/elves/elvish/program/shell.interact.func1(0x896ea0, 0xc420504700, 0x16, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progrf
github.com/elves/elvish/program/shell.interact(0xc4200f5040, 0xc4200d2300, 0x16, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progr9
github.com/elves/elvish/program/shell.(*Shell).Main(0xc4201ca340, 0xc4200a2170, 0x0, 0x0, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progr7
github.com/elves/elvish/program.Main(0xc4200a2170, 0x1, 0x1, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progr0
main.main()
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/main.5

goroutine 19 [syscall]:
os/signal.signal_recv(0x975320)
        /usr/lib/go/src/runtime/sigqueue.go:139 +0xa6
os/signal.loop()
        /usr/lib/go/src/os/signal/signal_unix.go:22 +0x22
created by os/signal.init.0
        /usr/lib/go/src/os/signal/signal_unix.go:28 +0x41

goroutine 21 [chan receive]:
github.com/elves/elvish/eval.init.13.func1()
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/eval/3
created by github.com/elves/elvish/eval.init.13
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/eval/9

goroutine 23 [select, locked to thread]:
runtime.gopark(0x937b88, 0x0, 0x910514, 0x6, 0x18, 0x1)
        /usr/lib/go/src/runtime/proc.go:291 +0x11a
runtime.selectgo(0xc42004e750, 0xc420098300)
        /usr/lib/go/src/runtime/select.go:392 +0xe50
runtime.ensureSigM.func1()
        /usr/lib/go/src/runtime/signal_unix.go:549 +0x1f4
runtime.goexit()
        /usr/lib/go/src/runtime/asm_amd64.s:2361 +0x1

goroutine 34 [chan receive]:
github.com/elves/elvish/program/shell.handleSignals.func1(0xc420098240)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progr8
created by github.com/elves/elvish/program/shell.handleSignals
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/progrc

goroutine 50 [chan receive]:
github.com/elves/elvish/edit/prompt.(*prompt).loop(0xc42024a000)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/1
created by github.com/elves/elvish/edit/prompt.makePrompt
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/4

goroutine 51 [chan receive]:
github.com/elves/elvish/edit/prompt.(*prompt).loop(0xc42024a080)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/1
created by github.com/elves/elvish/edit/prompt.makePrompt
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/4

goroutine 2305 [syscall]:
syscall.Syscall6(0x10e, 0x7, 0xc420642280, 0x0, 0x0, 0x0, 0x0, 0x42add4, 0x937328, 0xc4201f1578)
        /usr/lib/go/src/syscall/asm_linux_amd64.s:44 +0x5
github.com/elves/elvish/vendor/golang.org/x/sys/unix.Pselect(0x7, 0xc420642280, 0x0, 0x0, 0x0, 0x0, 0xc42017e0f0, 0x0,)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/vendo2
github.com/elves/elvish/sys.Select(0x7, 0xc420642280, 0x0, 0x0, 0x2, 0xc4201f17c0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/sys/s1
github.com/elves/elvish/sys.WaitForRead(0xc4201f16e8, 0x2, 0x2, 0x0, 0x1, 0x0, 0x0, 0x0)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/sys/w4
github.com/elves/elvish/edit/tty.(*runeReader).run(0xc42018c140)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/4
created by github.com/elves/elvish/edit/tty.(*runeReader).Start
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/f

goroutine 2354 [select]:
github.com/elves/elvish/edit/tty.(*reader).send(0xc420180240, 0x972560, 0xc420536008)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/d
github.com/elves/elvish/edit/tty.(*reader).run(0xc420180240)
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/f
created by github.com/elves/elvish/edit/tty.(*reader).Start
        /builddir/elvish-b24e4a73ccd948b8c08d4081c2bcfb7cf603a02b/_build-elvish-xbps/src/github.com/elves/elvish/edit/b

execing recovery shell /bin/sh

I cannot seem to reproduce this problem.  Perhaps you can give more details on how to reproduce?

Start completion, input string that matches nothing, press tab.

I'm also seeing this with 0.12-1-gb24e4a7.


Option to remove a backup ?

Is there also a command to remove / delete a specific backup by the id ?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: aae944b4-aa1b-4bc7-5f3c-1e645c792e86
Version Independent ID: 06e361cb-035c-0b20-1b85-6532e551a35a
Content: az webapp config backup
Content Source: latest/docs-ref-autogen/webapp/config/backup.yml
Service: app-service-web
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

@AzureAppServiceCLI, @antcp, please help take a look.

@StefH - at this time it looks like you only have the ability to delete using the name via the az storage blob delete command - https://docs.microsoft.com/en-us/cli/azure/storage/blob?view=azure-cli-latest#az_storage_blob_delete
Routing to storage team for confirmation.

Hello @seligj95, thanks for your reply, however I know to delete a backup from storage, I did that:

However, this deleted backup is still visible in the backup list from the web-app:

@StefH - were you able to restore from the backup you deleted from storage? I deleted the relevant storage blobs and the backup is still visible as you pointed out, but I get an error when trying to restore that backup since the underlying files no longer exist.

There doesn't seem to be a way to delete a specific backup from the App Service portal but you can delete the underlying files. I will take this back to the team to confirm and see if this is something on the roadmap.

@seligj95

There doesn't seem to be a way to delete a specific backup from the App Service portal but you can delete the underlying files. I will take this back to the team to confirm and see if this is something on the roadmap.

This is what I also would like to know. If you have any news on this, please keep me informed.

Please find the CLI command to delete the backup by id here : https://learn.microsoft.com/en-us/cli/azure/webapp/config/backup?view=azure-cli-latest#az-webapp-config-backup-delete
Thanks,
Ashok.


Add experts and possible mentors for OLS-2

for the ones who answered the form

looks good and works on my machine - we'll need to update it again in a day or two to synchronise with the most recent registrations.


make avar puts block

If an AVar is full, putting should block until it's empty.

This is mostly to match Haskell MVar semantics and avoid treating AVars as a queue.

i try to implements it, but look like Haskell does not specify an order in which it executes threads. My haskell code like this:

import Control.Concurrent
import Control.Monad

roundRobin :: IO ()
roundRobin = do
  x <- newEmptyMVar :: IO (MVar Int)
  void $ forkIO $ forever $ do
    val <- takeMVar x
    putStrLn $ "Received value (Thread #1): " ++ show val
  void $ forkIO $ forever $ do
    val <- takeMVar x
    putStrLn $ "Received value (Thread #2): " ++ show val
  mapM_ (putMVar x) [0..10]
  threadDelay 100000

roundRobinPut :: IO ()
roundRobinPut = do
  x <- newEmptyMVar :: IO (MVar Int)
  void $ forkIO $ mapM_ (putMVar x) [0..10]
  void $ forkIO $ mapM_ (putMVar x) [11..20]
  replicateM_ 20 $ do
    val <- takeMVar x
    putStrLn $ "Received value: " ++ show val
  threadDelay 100000

main :: IO ()
main = roundRobin >> roundRobinPut

execute on 4 core yield:
Received value (Thread #1): 0
Received value (Thread #2): 1
Received value (Thread #2): 3
Received value (Thread #2): 4
Received value (Thread #1): 2
Received value (Thread #2): 5
Received value (Thread #2): 7
Received value (Thread #2): 8
Received value (Thread #1): 6
Received value (Thread #2): 9
Received value (Thread #1): 10
Received value: 0
Received value: 1
Received value: 2
Received value: 11
Received value: 12
Received value: 13
Received value: 3
Received value: 14
Received value: 15
Received value: 16
Received value: 4
Received value: 17
Received value: 18
Received value: 19
Received value: 5
Received value: 20
Received value: 6
Received value: 7
Received value: 8
Received value: 9

similliar code on Purescript which block when putting full AVar yield different results, it just switching. I guess it because Javascript is single threaded.
received value (Thread #1): 0
received value (Thread #2): 1
received value (Thread #1): 2
received value (Thread #2): 3
received value (Thread #1): 4
received value (Thread #2): 5
received value (Thread #1): 6
received value (Thread #2): 7
received value (Thread #1): 8
received value (Thread #2): 9
received value (Thread #1): 10
received: 0
received: 1
received: 11
received: 2
received: 12
received: 3
received: 13
received: 4
received: 14
received: 5
received: 15
received: 6
received: 16
received: 7
received: 17
received: 8
received: 18
received: 9
received: 19
received: 10
received: 20


[SPARK-21268][MLLIB] Move center calculations to a distributed map in KMeans

What changes were proposed in this pull request?
The scal() and creation of newCenter vector is done in the driver, after a collectAsMap operation while it could be done in the distributed RDD.
This PR moves this code before the collectAsMap for more efficiency
How was this patch tested?
This was tested manually by running the KMeansExample and verifying that the new code ran without error and gave same output as before.

Can one of the admins verify this patch?

Test build #3826 has started for PR 18491 at commit 9e7ef53.

Test build #3826 has finished for PR 18491 at commit 9e7ef53.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Is there any real performance benefit here? Doesn't seem like it unless you have really large k?

I doubt it makes much difference, yeah. It pushes a little computation off the driver, and computation that happens on each iteration. It seemed like a tiny win that didn't complicate things.

Fair enough. LGTM with small comment about possibly changing the variable name?

@MLnick Great ! Updated variable name

Test build #3834 has started for PR 18491 at commit 3aa072d.

Test build #3834 has finished for PR 18491 at commit 3aa072d.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds no public classes.

Test build #3835 has started for PR 18491 at commit 3aa072d.

Test build #3835 has finished for PR 18491 at commit 3aa072d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged to master


Helm cannot find service during upgrade.

If I install a chart (with subs) and subsequently upgrade the chart helm reports;
UPGRADE FAILED
ROLLING BACK
Error: no Service with the name "edge-objectstore" found
Error: UPGRADE FAILED: no Service with the name "edge-objectstore" found
Yet if I query the cluster it is clearly there!
kubectl -n edge get service edge-objectstore
NAME               TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)   AGE
edge-objectstore   ClusterIP   <IP_ADDRESS>           80/TCP    11m
Having been created by the install.
For whatever reason helm or whatever it is calling is unable to determine that the service exists even though it is unmodified since the install.
Is helm losing state because some of the pods are failing to start? That is the issue isn't with the service but somewhere else and helm is reporting incorrectly?

It's hard to tell why you are facing this without knowing the contents of the ./upgrade-edge.sh script. Can you verify that the right namespace is being passed in to the helm upgrade call in there? Sharing the script contents could also be useful. Thanks!

Hi, it's a one line wrapper as follows;
helm upgrade edge ./ -f ./edge-values.yaml "${@}"
We words around install and upgrade so that the releases are handled properly.
The nodes file is passed in because our charts need to know what nodes they are being deployed onto.

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!

I think this is related to https://github.com/helm/helm/issues/6031 where one can find workaround ideas.


Database initialization

I am reading your README and I see an example about testing.
But is it possible to run your script for database data initialization ?

Hi @jbouzekri, if you take a look at @elvis-macak's pull request #16, you'll see the way you can use the library to initialize a database. Hope that helps.


Add x86 packaging and manifest.json

[X] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Adds behavior outlined in
https://github.com/ni/niveristand-custom-device-build-tools/pull/158
and
https://github.com/ni/niveristand-custom-device-build-tools/pull/157
Why should this Pull Request be merged?
Scan engine has custom packaging needs and so it currently has a separate packaging that requires these edits to be made to it specifically.
What testing has been done?
Output of PR build.

The x86 packaging change looks good from the dev build. However, the manifest file logic will need to be called for each lvVersion to match the logic in ni/niveristand-custom-device-build-tools#157. There it is called in the post-build step for each version, so a manifest.json file is placed in each installer directory.
I think it would be as straightforward as indenting to include the new task in this larger loop?
https://github.com/ni/niveristand-scan-engine-ethercat-custom-device/blob/1653bdf23de071ddabc40006c83a162c436d2aa0/custom-packaging.yml#L15

Ah you're right, I just put it in the wrong level of the packaging.  Good catch!


catch more exceptions & make a couple tests more robust

[ ] Is it mergeable?
[ ] Did it pass the tests?
[ ] If it introduces new functionality in scripts/ is it tested?
Check for code coverage with make clean diff-cover
[ ] Is it well formatted? Look at make pep8, make diff_pylint_report,
make cppcheck, and make doc output. Use make format and manual
fixing as needed.
[ ] Did it change the command-line interface? Only additions are allowed
without a major version increment. Changing file formats also requires a
major version number increment.
[ ] Is it documented in the ChangeLog?
http://en.wikipedia.org/wiki/Changelog#Format
[ ] Was a spellchecker run on the source code and documentation after
changes were made?

@ctb @camillescott @luizirber Ready for review & merge

LGTM


MediaQuery.of(context).padding always returns with EdgeInsets.zero in Flutter Android

Hi,
I'm totaly new in Flutter.
I've tried to get the resolution of the current device in flutter, however the MediaQuery.of(context).size returns with the size of the widget without other elements, like the bottom system ui elem in Android.
As far as i know, when we use
SystemChrome.setEnabledSystemUIOverlays([SystemUiOverlay.bottom]);

, then we should be able to get the bottom navigation bar's height with MediaQuery.of(context).padding, but it always returns with EdgeInsets.zero.
Same happens with MediaQuery.of(context).viewInsect and ui.window elems.
Please advise.
Steps to Reproduce
  @override
  Widget build(BuildContext context) {
    SystemChrome.setEnabledSystemUIOverlays([SystemUiOverlay.bottom]);
    print(MediaQuery.of(context).padding);
  }

Logs
EdgeInsets.zero
Doctor
[flutter] flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, v0.9.4, on Microsoft Windows [Version 10.0.17134.345], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK 28.0.2)
[√] Android Studio (version 3.1)
[√] VS Code (version 1.28.2)
[√] Connected devices (1 available)

• No issues found!
exit code 0

Do you see this on a real device or an emulator?
iOS or Android?
Can you perhaps create a screenshot that demonstrates the discrepancy?

Sorry that I haven't stated these.
Android, Nexus 5X emulator.
Unfortunately I'm not allowed to share screenshots, but this issue is really confusing.. I've heard a solution that i should create a stream for both android and ios, but I believe there should be an easy way to get these values with Flutter too.

You don't need a screenshot from a real-world project.
I'd expect more bug reports if this wouldn't work.

Well, okay, but what should I do to have the resolution?

@zoechi I believe this is related to a question asked on SO which gives a bit more detail - https://stackoverflow.com/questions/52954549/get-width-and-height-of-the-device-in-flutter. The issue isn't that flutter is returning the dimensions of the area it's drawing to improperly, it's that @Tremx389 was expecting that there would be a way to get the resolution of the entire phone's screen from within flutter using something like ui.window.physicalSize or MediaQuery(context).size. Instead, it's just returning the size of the area being drawn to (which is what it says it does if you read the docs closely)
This part of the docs is probably the most confusing, as it makes it seem like all "system UI" might include the navigation buttons:

This value does not take into account any on-screen keyboards or other system UI. The padding and viewInsets properties provide a view into how much of each side of the application may be obscured by system UI.

@rmtmckenzie thanks for the clarification!

In my case,
  SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle.dark);   double width = MediaQuery.of(context).size.width;
When I run in iOS simulator or Android device (debug), it is fine, but with real device (release apk or Testflight) that width will be 0 at first build then it has value later.

@amen71311 I think this is a different issue.
The size is provided by the native platform with a delay. When run in debug mode and perhaps also on a low emulator, then startup is usually slower and on first build() the size is already available.
In release mode startup is much faster and the first build often happens before the size is available and therefore MediaQuery.of(context).size returns 0x0.
When the actual size becomes available the view is rebuilt again and the proper size will be reported.
You just need to ensure that a size of 0x0 does not cause an exception.

I run into this with bottombar.   below line in 1.12.15 return 0
final double additionalBottomPadding = math.max(MediaQuery.of(context).padding.bottom - widget.selectedFontSize / 2.0, 0.0);

On a physical Pixel 3 XL the padding returned is always ZERO in Flutter 1.12.13+Hotfix5. So:
MediaQuery.of(context).padding.top

Returns zero.

As described in the StackOverflow question above, this behavior is correct from Flutter's point of view.  Closing.

@tvolkert I think it's not true, since flutter consider android bottom navigation bar as part of the app's
height but when calling:
MediaQuery.of(context).padding.top
It returns Zero.


Getting min max values of axis when legend clicked

Hi!
Thanks for your hard work with angular-nvd3!
I'm using NVD3 to display of a number of line charts and would like to be able to get the min and max values of the xAxis currently displayed. In this case the xAxis shows time values.
When toggling the visibility of lines, by clicking the legend label, the min and max values in the chart are changed. I've added an event to listen to legendClicks but I'm not sure how to get my hands on the newly rendered date range inside the event handler.
Great if anyone could shed some light on this.
// John

Ok, solved. Added to SO:
http://stackoverflow.com/questions/32327890/getting-rendered-date-range-from-axis-with-nvd3/32350461#32350461


CNCF Newsletter subscriber list

CNCF's newsletter subscriber numbers are wildly inconsistent week over week. We need to identify why. SFDC report is here.

Identify if Cvent auto-populates into SFDC?

Marijana to have auto-refresh added to the dashboard on the back end.

waiting on Marijana to update on the back end. Taylor has been following up

@taylorwaggoner Do you have a JIRA ticket open for the newsletter backend update issue with Marijana? If not, let me know and I'll open one. If so, can you paste it here? We want the report she built for you to also show: sources of subscription - webinars, via website, event booth scans, kubecon registration event. Dashboard for Taylor - what was the number last week, what’s the number this week and where did the increases come from for newsletter subscribers?

@kwussow, here's the Jira ticket: https://jira.linuxfoundation.org/browse/LP-5862


Common Template No Editable

🎥 Demo

@upalatucci hi, the design is to:
a - have a top warning that common templates are not aditable
b - disable all edit options, [ except "edit boot source" that actually edit a dataVolume and not the template ]

/lgtm


Player Error Protocol Not Found

Hello I get an error when trying to play a hls video.

What is the video format?
On Fri, Apr 17, 2020 at 3:05 PM Serkan-Karacelikler <
<EMAIL_ADDRESS>wrote:

Hello I get an error when trying to play a hls video.
[image: WhatsApp Image 2020-04-17 at 17 02 40]
https://user-images.githubusercontent.com/54067269/79577531-879bb100-80cd-11ea-8d8f-3f2b61e7486c.jpeg
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pejonic/cordova-pejonic-plugin-exoplayer/issues/10,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABPF6EVSZ7T5WHL4X6K2NHTRNBO2BANCNFSM4MKYX5GQ
.

M3u8

Video link: https://srv87.filmmodu.xyz/tt6515910/480p.en.m3u8

Try this stream
http://dwstream72-lh.akamaihd.net/i/dwstream72_live@123556/index_1_av-p.m3u8?sd=10&rebase=on
Send back to me the error response

I will update this version soon, try using this version https://github.com/frontyard/cordova-plugin-exoplayer

OK thanks.


v4 Custom forms focus box-shadow

Custom form controls should only get a box-shadow if $enable-shadows is true.
This issue only concerns .c-select and .file-custom.

Fix in 7d194a9.


[Q] I have many wandb files in my local server, Can I see values and graphs on the web?

I have many local run files, directory structure is like
── run-YYYYMMDD_??????-
├── files
│   ├── conda-environment.yaml
│   ├── config.yaml
│   ├── output.log
│   ├── requirements.txt
│   ├── wandb-metadata.json
│   └── wandb-summary.json
├── logs
│   ├── debug-internal.log
│   └── debug.log
├── run-.wandb
I think I deleted my wandb project in the online app, but I have local files.
can I see the result using this run.wandb files?

Thanks for your help.
I tried to re-upload the run.
So i commanded 'wandb sync ./run-YYYYMMDD_?????-'
I came across error, which says
Find logs at: ~~path~~/wandb/wandb/debug-cli.team_ai.log
wandb: ERROR Error while calling W&B API: sweep 6obsuqtj not found (<Response [404]>)
Syncing: https://wandb.ai/userid/projectname/runs/i97cv2jc ... done

And when I check the online wandb project, no runs are added to default project.
I think the error line cause some problem,
wandb: ERROR Error while calling W&B API: sweep 6obsuqtj not found (<Response [404]>)

Can I get more information?
And when I commanded 'wandb offline' also nothing happens.

and this is what i get when i entered on run folder

I tried but I cannot
since I deleted the project which sweep is in.
is there any method I can access to sweep without project name in the url? with only sweep id?

Thanks for your help!
Since I came to know I cannot do anything in this situation, I just accepeted!


Problem to install the package

I'm having trouble installing the package correctly.
Following your instructions, when I run
pip install datacamp-downloader
I get this warning message
WARNING: Ignoring invalid distribution -ip (c:\python310\lib\site-packages)
The same thing happens when I try to install it by cloning the repository:
pip install git+https://github.com/TRoboto/datacamp-downloader.git
In fact, when I check if the installation has happened
datacamp --install-autocompletion
or
datacamp login
I get
"datacamp" is not recognized as an internal or external command,   an executable program or batch file
How can I solve?
Sorry if my question is trivial but I couldn't find answers elsewhere. Thanks.
Python version: 3.10.2
Pip version: 22.1.2

i have the same problem


feat(crdUpgrades): enable CRD converter in OSM

Description:
Enable CRD converter by default as a part of OSM control plane
Part of #3396
Signed-off-by: Sneha Chhabria<EMAIL_ADDRESS>Please mark with X for applicable areas.
Affected area:

Functional Area

New Functionality
[ ]

Documentation
[ ]

Install
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[X]

Ingress
[ ]

Egress
[ ]

Networking
[ ]

Observability
[ ]

SMI Policy
[ ]

Sidecar Injection
[ ]

Security
[ ]

Upgrade
[ ]

Tests
[ ]

CI System
[ ]

Demo
[ ]

Performance
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Did you notify the maintainers and provide attribution?

Is this a breaking change? no

Codecov Report

Merging #3838 (4606230) into main (d1584f1) will increase coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3838      +/-   ##
==========================================
+ Coverage   67.71%   67.80%   +0.09%
==========================================
  Files         198      198
  Lines       11034    11034
==========================================
+ Hits         7472     7482      +10
+ Misses       3513     3503      -10
  Partials       49       49

Flag
Coverage Δ

unittests
67.80% <ø> (+0.09%)
:arrow_up:

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

Impacted Files
Coverage Δ

...icate/providers/certmanager/certificate_manager.go
86.61% <0.00%> (+0.70%)
:arrow_up:

pkg/crdconversion/crdconversion.go
69.56% <0.00%> (+9.78%)
:arrow_up:

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 d1584f1...4606230. Read the comment docs.


Flutter Icon in Drawer header does not respect Dark theme

I don't think this is a huge deal, but probably the text should turn white?

Doesn't look good; will fix.

done


vscode-extensions.{language-haskell,hie-server}: init

Motivation for this change
These weren't available via Nixpkgs yet for declarative addition to VSCode.
I'm also packaging these extensions for home-manager, https://github.com/rycee/home-manager/pull/619
I've tried more than once to get something resembling a Haskell IDE, which usually ends with a half-broken setup. I'm happy to make the next Nix user's setup a bit easier.
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Looks fine, verified hashes match.


v0.0.36

Hopefully resolving the issue described in: https://github.com/integrated-reasoning/KeyHippo/pull/20 now that https://github.com/supabase/postgres/pull/1193 has merged.

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

Project coverage is 83.20%. Comparing base (4bde7d1) to head (99a9e1e).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #29   +/-   ##
=======================================
  Coverage   83.20%   83.20%
=======================================
  Files           3        3
  Lines         399      399
  Branches       70       70
=======================================
  Hits          332      332
  Misses         67       67

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


type annotations

add type annotations throughout

Yep, that would indeed be great! In #43 I have added two very simple types for Qrels and Run:
https://github.com/terrierteam/ir_measures/blob/27088215789dad877f75b9aba4d456e1894556ec/ir_measures/util.py#L31
https://github.com/terrierteam/ir_measures/blob/27088215789dad877f75b9aba4d456e1894556ec/ir_measures/util.py#L40
We could probably also include Iterable[NamedTuple] in the union so that it matches the documented behavior of QrelsConverter and RunConverter.


Upgrade to latest LTS, clean up stack.yaml

This brings up a more modern compiler, which will be useful for next PR changes I am planning.

Cool, do you think it would be possible to setup automated testing of PRs with Travis CI or AppVeyor or something like that in the future? I tried Travis on my own fork and it worked fine.

I could try sending a PR to set-up GitHub actions to set CI up.

CI done in #18

Hey great effort with the CI, been meaning to do it but never got around to it. Much appreciated!


x32dbg version causes crash on breakpoint

hello, much gratitude for this course.
a small issue - when i try to apply a breakpoint using x32dbg, as in https://gamehacking.academy/lesson/2/4, and the execution reaches it; my VM crashes totally?
is this a problem with the x32dbg version included?
thanks!

Interesting, I've never run into this issue. Does it crash when you attach to other processes as well? Also, how much RAM do you have available for your VM?


File compare to find duplicates

Description of the new feature / enhancement
There are so many times that I need to de-duplicate files on my system. The most hungry files are photo's. But there are worksheets, downloads etc. that also end up being duplicated across multiple folders.
I need a tool (and I think everyone could use it) to search all directories and files and list identical files (based on binary level).
Scenario when this would be used?
Primarily when trying to free up disk space.
Supporting information
No response

/dup #8100


Can't access correct map property in where condition for a map type property

Hi there! Sorry being the one annoying at the end of the year.
Let's say I have these nodes in my database:
CREATE (p1:Person {location: point({x: 12, y: 12})}),
       (p2:Person {location: point({x: -6, y: -6})}),
       (p3:Person {location: point({x: 8, y: -2})}),
       ...

I'm create a query like:
MATCH (p:Person) WHERE p.location.x > 6 RETURN p

My test case is
Node node = Cypher.node("Person").named("p");

Statement statement = Cypher.match(node)
		.where(node.property("location.x").gt(Cypher.literalOf(6)))
		.returning(node).build();

assertThat(cypherRenderer.render(statement))
		.isEqualTo("MATCH (p:`Person`) WHERE p.`location`.`x` > 6 RETURN p");

But what I get is
Expecting:
 <"MATCH (p:`Person`) WHERE p.`location.x` > 6 RETURN p">
to be equal to:
 <"MATCH (p:`Person`) WHERE p.`location`.`x` > 6 RETURN p">
but was not.

I know the property method doesn't not know whether I'm trying to access a map or I'm just giving my property a fancy name. Am I missing something?
Thank you for the great work on this library! Have a great holidays everyone and stay safe!

That's a tricky one… What if you actually do have a property named foo.bar?
neo4j@neo4j> create (n:Node {`foo.bar`: 'x'});
0 rows available after 113 ms, consumed after another 0 ms
Added 1 nodes, Set 1 properties, Added 1 labels
neo4j@neo4j> match (n) return n;
+--------------------------+
| n                        |
+--------------------------+
| (:Node {`foo.bar`: "x"}) |
+--------------------------+

1 row available after 176 ms, consumed after another 2 ms
neo4j@neo4j> match (n) return n.`foo.bar`;
+-------------+
| n.`foo.bar` |
+-------------+
| "x"         |
+-------------+

1 row available after 76 ms, consumed after another 1 ms

neo4j@neo4j> match (n) return n.`foo`.`bar`;
+---------------+
| n.`foo`.`bar` |
+---------------+
| NULL          |
+---------------+

1 row available after 45 ms, consumed after another 2 ms

That's a tricky one… What if you actually do have a property named foo.bar?
neo4j@neo4j> create (n:Node {`foo.bar`: 'x'});
0 rows available after 113 ms, consumed after another 0 ms
Added 1 nodes, Set 1 properties, Added 1 labels
neo4j@neo4j> match (n) return n;
+--------------------------+
| n                        |
+--------------------------+
| (:Node {`foo.bar`: "x"}) |
+--------------------------+

1 row available after 176 ms, consumed after another 2 ms
neo4j@neo4j> match (n) return n.`foo.bar`;
+-------------+
| n.`foo.bar` |
+-------------+
| "x"         |
+-------------+

1 row available after 76 ms, consumed after another 1 ms

neo4j@neo4j> match (n) return n.`foo`.`bar`;
+---------------+
| n.`foo`.`bar` |
+---------------+
| NULL          |
+---------------+

1 row available after 45 ms, consumed after another 2 ms

Here's a possible workaround:
@Test
		void gh114() {
			Node node = Cypher.node("Person").named("p");

			Property locationProperty = node.property("location");
			Statement statement = Cypher.match(node)
				.where(Cypher.property(locationProperty, "x").gt(Cypher.literalOf(6)))
				.returning(node).build();

			assertThat(cypherRenderer.render(statement))
				.isEqualTo("MATCH (p:`Person`) WHERE p.location.x > 6 RETURN p");
		}

I'm gonna change the property methods to take in a varg of Strings, so that you can spare yourself the extra steps.
Looking forward to work with you again this year. Cheers and all the best, Michael

Here's a possible workaround:
@Test
		void gh114() {
			Node node = Cypher.node("Person").named("p");

			Property locationProperty = node.property("location");
			Statement statement = Cypher.match(node)
				.where(Cypher.property(locationProperty, "x").gt(Cypher.literalOf(6)))
				.returning(node).build();

			assertThat(cypherRenderer.render(statement))
				.isEqualTo("MATCH (p:`Person`) WHERE p.location.x > 6 RETURN p");
		}

I'm gonna change the property methods to take in a varg of Strings, so that you can spare yourself the extra steps.
Looking forward to work with you again this year. Cheers and all the best, Michael

Hi @michael-simons,
Sorry for the late reply. Workaround works like a charm, thank you very much for the answer and for the super quick fix!
Can't wait to work with you too, thank you for your time, have a great week!

Hi @michael-simons,
Sorry for the late reply. Workaround works like a charm, thank you very much for the answer and for the super quick fix!
Can't wait to work with you too, thank you for your time, have a great week!


Validation not Working

When running bubblewrap validate url=https://www.google.com, following error occurs:
cli ERROR Error calling the PageSpeed Insights API: Error: Failed to run the PageSpeed Insight report for https://www.googleapis.com/pagespeedonline/v5/runPagespeed?url=https%3A%2F%2Fhome.ameer.io%2F&category=performance&category=pwa&category=accessibility&strategy=mobile. Server responded with status 429

Same happens when trying to access a server protected by a VPN.

Validation uses a PageSpeed Insights endpoint. If the endpoint is unreacheable, the call will fail. It doesn't stop the project from building though.
@nohe427 I' wondering about just removing validation from the build step. Thoughts?

#675 removes the validation from the build step. We should also consider deprecating the validate command and package.

Closed via #675


migrate auth slice to ts, RTK & RTK-Query

WIP for authSlice

Error handling is broken. If you enter the wrong credentials, you get four errors instead of one 😄

First time setup is broken, it doesn't redirect to /login again to reload the page 😅

Looks good to me!


Add simple https-client example charm

Can you split things a bit?
I think we should at least split this in two with one commit which introduces the test charm and supporting changes and another for the changes to the Github workflow.


Add mime.types to prevent potential classpath conflicts

Travis tests began failing due to 'json' and 'csv' filetypes not being supported.  The issue was traced to the mime.types file in https://github.com/cndreisbach/ring.middleware.mime-extensions not being present in the classpath.
It appears that some lein environments load the mime.types from another qu dependency first:
user=> (clojure.java.io/resource "mime.types")
#<URL jar:file:/home/vagrant/.m2/repository/com/amazonaws/aws-java-sdk/1.3.18/aws-java-sdk-1.3.18.jar!/mime.types>

Rather than messing with how lein orders its dependencies, we can work around this by adding a mime.types to qu's own resources directory.  The file in this PR is a copy of the file in the ring.middleware.mime-extentions repo.

Some more context: For the past few years, this has worked because ring.middleware.mime-extensions was ordered earlier than the aws-java-sdk in lein's classpath. As of 2.5, lein's classpath ordering appears to have changed, and aws-java-sdk is now ordered earlier. Because  ring-middleware.mime-extensions by default looks for mime.types on the entire classpath, it's now picking up the mime.types file in the aws-java-sdk jar before the one in ring.middleware.mime-extensions' own jar.
Basically, we've just been getting lucky for a few years. By putting the file in qu's resources directory, we guarantee that it'll get picked up first because lein orders project resources before dependencies. Yes, we're relying on that behavior not changing, but that seems like a reasonable bet.
We can also talk to @cndreisbach about a suitable fix for ring.middleware.mime-extensions itself so that it does a targeted search for the mime.types file by default


Slack Interactive Message Menus with DeliveryMode set to Ephemeral generate error

Add a Slack legacy interactive message menu to your activity:
IMessageActivity respActivity = Activity.CreateMessageActivity();
List<object> attachments = new List<object>();
List<object> actions = new List<object>();

// Build select menu
actions.Add(new
{
    name = "MENU_SELECT",
    text = "Select an account...",
    type = "select",
    options = new object[]
    {
        new
        {
            "text": "Account 1",
            "value": "account1"
        },
       new
       {
           "text": "Account 2",
           "value": "Account2"
       }
   }
});

attachments.Add(new
{
    fallback = "You are unable to choose an account",
    callback_id = "NA",
    color = "#3AA3E3",
    attachment_type = "default",
    actions = actions
});

respActivity.ChannelData = new
{
    text = "*Account Selection*",
    attachments = attachments
};

If you send the activity as defined above, no issue, it will show up on Slack, you can select an entry in the menu and an event is sent without any error.
If you inspect the network traffic you'll see Slack sending this POST request:
POST https://my-bot-tests-xxx.slack.com/api/chat.attachmentAction?_x_id=61a2bc38-1719047525.780&_x_csid=nAVN34hw6lE&slack_route=XXX&_x_version_ts=1719018759&_x_frontend_build_type=current&_x_desktop_ia=4&_x_gantry=true&fp=8f&_x_num_retries=0

With payload:
{"actions":[{"id":"1","name":"MENU_SELECT","text":"Select an account...","type":"select","data_source":"static","options":[{"text":"Account 1","value":"Account1"},{"text":"Account 2","value":"Account2"}],"selected_options":[{"value":"Account1"}]}],"attachment_id":"1","callback_id":"NA","channel_id":"XXX","message_ts":"1718978926.546839","prompt_app_install":false,"team_id":"XXX"}

And response: {"ok":true}
Now, I want to display the interactive menu using an ephemeral message as I don't want other Slack users to see what account I selected. I just add following line to my code:
respActivity.DeliveryMode = DeliveryModes.Ephemeral;

Now, going back to Slack, my interactive menu is displayed properly with the Only visible to you mention, meaning it has been sent as an ephemeral message just like I wanted to. But when I select an entry in the menu, now Slack returns an error.
The POST request is almost identical as the previous one except for the "is_ephemeral":true payload field that is new. See:
POST https://my-bot-tests-xxx.slack.com/api/chat.attachmentAction?_x_id=61a2bc38-1719048029.946&_x_csid=pdl0xabFIAY&slack_route=XXX&_x_version_ts=1719018759&_x_frontend_build_type=current&_x_desktop_ia=4&_x_gantry=true&fp=8f&_x_num_retries=0

Payload with new "is_ephemeral":true field:
{"actions":[{"id":"1","name":"MENU_SELECT","text":"Select an account...","type":"select","data_source":"static","options":[{"text":"Account 1","value":"Account1"},{"text":"Account 2","value":"Account2"}],"selected_options":[{"value":"Account1"}]}],"attachment_id":"1","callback_id":"NA","channel_id":"XXX","is_ephemeral":true,"message_ts":"1719048014.000500","prompt_app_install":false,"team_id":"XXX"}

And response now contains an error: {"ok":false,"error":"dispatch_failed"}
Looking at the network traffic, Slack emits another call just after with additional error info:
POST https://slack.com/beacon/error

{"subtype":"missing_subtype","message":"hashed timestamp chat.attachmentAction not ok: dispatch_failed","stack":"dispatch_failed: 61a2bc38-1719048029.946 chat.attachmentAction not ok: dispatch_failed\n    at Hn._onNotOk (https://a.slack-edge.com/bv1-13-br/gantry-v2-shared.ea6777821724f355444d.min.js?cacheKey=gantry-1719018759:5:377182)\n    at Hn.resolve (https://a.slack-edge.com/bv1-13-br/gantry-v2-shared.ea6777821724f355444d.min.js?cacheKey=gantry-1719018759:5:376785)\n    at Hn._onLoad (https://a.slack-edge.com/bv1-13-br/gantry-v2-shared.ea6777821724f355444d.min.js?cacheKey=gantry-1719018759:5:361634)"}

If I remove the respActivity.DeliveryMode = DeliveryModes.Ephemeral; from my code, it works ok again.
So it seems there's an issue when Slack adds the "is_ephemeral":true field in the payload request and the Slack Bot Adaptor fails to handle it properly.
Note that my bot does not receive any event from Slack/Bot Adapter when the error occurs: the error sent as response to the Slack event comes from a higher layer (MS Slack Adapter maybe, to be confirmed).

Hi,
I raised this issue in the Bot builder SDK repo
https://github.com/microsoft/botframework-sdk/issues/6657


Adds the get contents operation

Fixes #79 by adding the getContents operation
Please @juanpedromoreno could you review? Thanks

Codecov Report

Merging #80 into master will increase coverage by 0.28%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #80      +/-   ##
==========================================
+ Coverage   79.47%   79.75%   +0.28%
==========================================
  Files          35       35
  Lines         492      499       +7
  Branches        2        2
==========================================
+ Hits          391      398       +7
  Misses        101      101

Impacted Files
Coverage Δ

...c/main/scala/github4s/free/domain/Repository.scala
100% <ø> (ø)
:arrow_up:

...in/scala/github4s/free/algebra/RepositoryOps.scala
75% <100%> (+3.57%)
:arrow_up:

...4s/shared/src/main/scala/github4s/GithubAPIs.scala
96.15% <100%> (+0.32%)
:arrow_up:

...cala/github4s/free/interpreters/Interpreters.scala
83.01% <100%> (+0.32%)
:arrow_up:

...b4s/shared/src/main/scala/github4s/api/Repos.scala
95.65% <100%> (+0.65%)
:arrow_up:

...ub4s/shared/src/main/scala/github4s/Decoders.scala
95.83% <100%> (ø)
:arrow_up:

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 911188f...4cc2153. Read the comment docs.


Add npm run watch command to watch /src

@rhamilto @sg00dwin this should fix the lack of a npm run watch for the catalog.

This does not appear to result in new dist.  I ran npm run watch and changed src/styles/services-view.less and src/components/services-view/services-view.html, but no new dist was generated.

I tested with .just, .html, .less. it took a while at times and the cli was
not immediately responsive. Also, I deleted /dist often to be sure it was
working.
I can give it another look
On Dec 19, 2017 2:04 PM, "Robb Hamilton"<EMAIL_ADDRESS>wrote:

This does not appear to result in new dist. I ran npm run watch and
changed src/styles/services-view.less and src/components/services-view/services-view.html,
but no new dist was generated.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openshift/origin-web-catalog/pull/607#issuecomment-352855495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARHwNw4er0DiqmjP6Xc9Mgf_0saCnWyks5tCAiugaJpZM4RHZkB
.

@rhamilto tested changing src/styles/services-view.less, which did cause a /dist change, however, it takes about 20+ seconds to show.
Per this issue with webpack 2, messing around with setting resolve.unsafeCache: true in the config to speed things up a bit.  So far, doesn't seem to help.
Tried recording a .gif to show you, but recording fails due to the stated slowness.

Guessing there are other optimizations that need to happen to our webpack.conf.js in order to speed this up.  Will take some research.


Use BOOST_ATTRIBUTE_UNUSED from boost/config

This macro does the proper version check and works against clang 3.6 (which otherwise spits warnings).
I haven't tested this patch. If someone could do it, I would be grateful, otherwise I'll do it myself later.

Solves the same problem as https://github.com/boostorg/concept_check/pull/6 but seems to do it better.  I recommend ignoring my patch and using this one. :-)
You can probably just get rid of the whole #define statement and simply use BOOST_ATTRIBUTE_UNUSED in its place, especially since BOOST_CONCEPT_UNUSED_TYPEDEF is only used in one single location.

Im confused about the "does the proper version check" comment by @blastrock.
I see a version check being removed, not added.

Removed the macro as @Houndie suggested, I didn't know if it was the best way to go.
@mclow What I meant is that the version check is already done in boost/config/suffix.hpp, and it is done a better way than it was done here (it actually supports clang 3.6), so this patch just make use of that

I don't think the version check is removed, it's simply reusing the version check work done in boost/config instead of repeating ourselves here.
The "proper" version check has to do with clang.  Clang defines its GCC variables to replicate GCC 4.2, which, with the existing code in concept_check, doesn't add attribute unused (this is because GCC 4.2 was what clang aimed for feature equality with once upon a time).  However, clang does currently do an unused typedef check, so simply checking the GCC version numbers is not enough to satisfy clang.  Presumably the code in boost/config takes that into account.
I tried to do the same sort of fix in my patch, but reusing existing code is the way to go, so I'll probably close mine in favor of this one.

Oh and @blastrock I implemented your fix in my code base, and it seems to work for what its worth.  Not really and "official" test but its better than nothing.

Exactly as @Houndie said :)
And thanks for testing!

bump
Any chance that this gets merged before the next boost release ?

Committed with a small change and on develop, in https://github.com/boostorg/concept_check/commit/bdcd06c4cc1971d763e528b8cb1d0f16fcc5ecf4


glTF 2.0 Model Test Failure

Both @mramato and I are seeing this test failure.  Can someone please look at this before the release on Monday? @lilleyse @likangning93 @ggetz @OmarShehata
loads a glTF 2.0 model
     Scene/Model
     Expected to render [169,3,3,255], but actually rendered [170,6,6,255].
    at <Jasmine>
    at Specs/Scene/ModelSpec.js:948:27
    at Promise.then (Source/ThirdParty/when.js:196:34)
    at Source/ThirdParty/when.js:297:13
    at processQueue (Source/ThirdParty/when.js:647:4)

@OmarShehata could you please take a first look?  Potentially a trivial change to adjust the test if we deem nothing broke.

Wonder if this is related to https://github.com/AnalyticalGraphicsInc/cesium/pull/6877

Yeah I think it's just a matter of the test being too strict, since the rendered color may vary between systems. Usually checking that the color is within some loose range is good enough.


Support versions: 3.8.0, 3.8.1, 3.9.0

LP#2035139
Adds support for 3 new versions of ceph-csi RBD and CephFS

adds support for auto-updating

No need for https://github.com/ceph/ceph-csi/issues/4213 once i found the foul storageclass option


ci: remove unused workflow and config

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

Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


Invite a guest user to an app from the Access Panel (myapps mirosoft com )- not accurate - not possible

Issue 1:
Invite guest users, step 4.
The picture is not accurate. myapps.microsoft.com -  After clicking "manage your application," you will be redirected to"App assignment - Manage user - link). After clicking "manage users" you will be redirected to standard Entra portal for Group management.  So, you never get the prompt shown on the picture under step 4.

Issue 2.
Invite guest users, step 4.
You cannot directly invite guest accounts like it is explained and shown in the picture under step 4.
Additional Explanation:
Because you are redirected to the standard Entra page for Group management, from there, it is not possible to invite Guests. You can just add existing accounts - Direct Members (Guests or members). Guest need to be pre-invited through Azure.
(all permissions have been checked, - owners allowed to invite, self-service application access enabled)

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 7fbdc24d-5bc5-54f0-208c-8574b19dfb40
Version Independent ID: 09e9c6cf-8626-532b-8a59-640ec5d4798a
Content: Add B2B collaboration users as an information worker - Microsoft Entra External ID
Content Source: docs/external-id/add-users-information-worker.md
Service: entra-external-id
GitHub Login: @csmulligan
Microsoft Alias: cmulligan

@mlinar79
Thanks for your feedback! We will investigate and update as appropriate.

@mlinar79
I'm going to assign this to the document author so they can take a look at it accordingly.
@csmulligan
Can you please check and add your comments on this doc update request as applicable.

Hi @mlinar79 thank you so much for pointing this out. I've moved this issue to an internal work item for tracking. I'll go ahead and close this issue, but changes to the article are in the works. Thanks again for helping us improve the documentation!
#please-close


Update docs and tests for ibm DB2

What does this PR do?

Motivation

Install instructions for unix should mention the latest version that we support
We can simplify the test setup now that we no longer test python 2

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
[ ] If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

Users also might also have to install build-essential with apt, I'll update you!

Users also might also have to install build-essential with apt, I'll update you!

@nubtron good point! we can do that as a separate PR when we know for sure! Shouldn't block the current update to the docs.

Oh yeah, sure!


Streaming from icecast on iOS

I tried to create a basic stream player on iOS, loading media from:
http://mp3.hitradiort1.c.nmdn.net/rt1nordschwabenwl/livestream.mp3
Content-Type: audio/mpeg

The player is setup correctly, after calling TrackPlayer.play() it returns the following:
YellowBox.js:
playback-state` with no listeners registered.
playback-track-changed` with no listeners registered.
playback-state` with no listeners registered.
playback-error` with no listeners registered.

Does the player supports live streaming  out-of-the-box ?
Thanks for answering 💚

react-native-track-player works great with Icecast. Your problem is most probably for http audio source instead of https.
Try whitelisting your domain for http request in NSAppTransportSecurity

For some reason I'm having the same issue. It used to be not a problem at all, but I cannot play IceCast only in iOS. Tried both simulator and a device; tried 0.2.2, 0.2.3, 0.2.4; tried HTTPS...but seems that the issue is present. I see the following log in my XCode:

nw_connection_get_connected_socket 17 Connection has no connected handler

Any suggestion would be appreciated.

Oh interesting, this is the same issue I'm having. My issue may provide more clarity on why you're running into this problem on iOS only: https://github.com/react-native-kit/react-native-track-player/issues/207

It's working 🎉
Thanks @moonstruck
Setting NSAppTransportSecurity for the domain mp3.hitradiort1.c.nmdn.net in info.plist was the missing piece.
(There is ability to setup a domain including subdomains via NSIncludesSubdomains)
Before adding NSAppTransportSecurity to info.plist, debug console from xCode was the following the time of the issue:
2018-05-31 00:29:33.059129+0200 AppName[19385:2813239] [] nw_connection_get_connected_socket 83 Connection has no connected handler
Starting/Resuming playback
2018-05-31 00:29:34.418 [warn][tid:com.facebook.react.TrackPlayerModuleQueue][RCTEventEmitter.m:53] Sending `playback-state` with no listeners registered.
2018-05-31 00:29:34.419129+0200 AppName[19385:2813239] Sending `playback-state` with no listeners registered.
2018-05-31 00:29:34.431712+0200 AppName[19385:2813731] Cannot start load of Task <F93A994D-7DF9-48DC-9BB5-BBF1C93BB1CB>.<1> since it does not conform to ATS policy
2018-05-31 00:29:34.435294+0200 AppName[19385:2813731] Task <F93A994D-7DF9-48DC-9BB5-BBF1C93BB1CB>.<1> finished with error - code: -1022
2018-05-31 00:29:34.443 [warn][tid:main][RCTEventEmitter.m:53] Sending `playback-state` with no listeners registered.
2018-05-31 00:29:34.444674+0200 AppName[19385:2811754] Sending `playback-state` with no listeners registered.
2018-05-31 00:29:34.447 [warn][tid:main][RCTEventEmitter.m:53] Sending `playback-error` with no listeners registered.
2018-05-31 00:29:34.449995+0200 AppName[19385:2811754] Sending `playback-error` with no listeners registered.
2018-05-31 00:29:34.629132+0200 AppName[19385:2813239] [] nw_connection_get_connected_socket 84 Connection has no connected handler
2018-05-31 00:29:35.255739+0200 AppName[19385:2812182] [] nw_connection_get_connected_socket 85 Connection has no connected handler

When added, the warning remains.
<key>NSAppTransportSecurity</key>
  <dict>
      <key>NSExceptionDomains</key>
      <dict>
          <key>wasabisys.com</key>
          <dict>
              <!--Include to allow subdomains-->
              <key>NSIncludesSubdomains</key>
              <true/>
              <!--Include to allow HTTP requests-->
              <key>NSTemporaryExceptionAllowsInsecureHTTPLoads</key>
              <true/>
              <!--Include to specify minimum TLS version-->
              <key>NSTemporaryExceptionMinimumTLSVersion</key>
              <string>TLSv1.1</string>
          </dict>
      </dict>
  </dict>

How to fix?


add artifact & static code analysis badges

Add badges to artifact and static code analysis (code climate).

:+1:

CLA is valid!


Failing tests

PHPstan is complaining about several unused private methods:
 ------ ----------------------------------------------------------------------
  Line   CorsMiddleware.php
 ------ ----------------------------------------------------------------------
  295    Method Tuupola\Middleware\CorsMiddleware::origin() is unused.
  304    Method Tuupola\Middleware\CorsMiddleware::methods() is unused.
  320    Method Tuupola\Middleware\CorsMiddleware::headersAllow() is unused.
  328    Method Tuupola\Middleware\CorsMiddleware::headersExpose() is unused.
  336    Method Tuupola\Middleware\CorsMiddleware::credentials() is unused.
  344    Method Tuupola\Middleware\CorsMiddleware::originServer() is unused.
  352    Method Tuupola\Middleware\CorsMiddleware::cache() is unused.
  360    Method Tuupola\Middleware\CorsMiddleware::error() is unused.
  372    Method Tuupola\Middleware\CorsMiddleware::logger() is unused.
 ------ ----------------------------------------------------------------------

Should the methods be removed or made public?

They are being used. PHPStan just don't see it. Private methods on purpose.
https://github.com/tuupola/cors-middleware/blob/master/src/CorsMiddleware.php#L178


Library throws an internal error when attempting to use createLogger

Please tell us about your environment:

winston version?

[ ] winston@2
[x] winston@3

node -v outputs: v14.5.0
Operating System? Windows 10
Language? ES6/7

What is the problem?

Upon trying to create a logger with transports, the library will throw an internal unhandled promise rejecton.
It doesn't matter if: I keep only one transport; remove the file transports; remove the console transports.
Sample code:
const winston = require("winston");
const transports = [
  new winston.transports.File({ filename: "logs/debug.log", level: "debug" }),
  new winston.transports.File({ filename: "logs/info.log", level: "info" })
];
if (environment !== "production") transports.push(new winston.transports.Console({
  format: winston.format.simple()
}));
this._winston = winston.createLogger({
  levels: {
     emerg: 0,
     alert: 1,
     crit: 2,
     error: 3,
     warning: 4,
     notice: 5,
     info: 6,
     debug: 7
   },
   format: winston.format.json,
   exitOnError: false,
   silent: false,
   transports
});

What do you expect to happen instead?

I would expect this to work without throwing any errors.
Other information

Error Stack:
(node:14172) UnhandledPromiseRejectionWarning: TypeError: this.format.transform is not a function
    at DerivedLogger._transform (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\lib\winston\logger.js:305:29)
    at DerivedLogger.Transform._read (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\node_modules\readable-stream\lib\_stream_transform.js:177:10)
    at DerivedLogger.Transform._write (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\node_modules\readable-stream\lib\_stream_transform.js:164:83)
    at doWrite (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\node_modules\readable-stream\lib\_stream_writable.js:409:139)
    at writeOrBuffer (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\node_modules\readable-stream\lib\_stream_writable.js:398:5)
    at DerivedLogger.Writable.write (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\node_modules\readable-stream\lib\_stream_writable.js:307:11)
    at DerivedLogger.log (C:\Users\prolp\Programing\NodeJS\learn\node_modules\winston\lib\winston\logger.js:223:12)
    at Logger.emergency (C:\Users\prolp\Programing\NodeJS\learn\lib\structures\Logger.js:53:26)
    at C:\Users\prolp\Programing\NodeJS\learn\src\app.js:30:12
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:14172) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:14172) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

More information:
After installing it from npm, the issue was present therefore I used npm i winston@next --save in hopes it would fix this - but it did not.

In your config, shouldn't format be winston.format.json() instead of winston.format.json
it has to be a function call as per the examples given in the documentation.

In your config, shouldn't format be winston.format.json() instead of winston.format.json
it has to be a function call as per the examples given in the documentation.

That was it! Thank you.