Fails to connect to ESP32

I know there was a similar issue #67 but this is lightly different.
I can connect to the esp32 directly after flashing it but after that, I'm unable to do so.
Putty connects fine every time so I'm sure there isn't a hardware problem.
I'm using a "doit esp32 devkit v1" (at least that's what's written on its back) with the mainline esp32 daily firmware.
I'm not running a main.py and I use the precompiled firmware available for download.
This is my error trace:
 rshell -p /dev/ttyS5 -a --buffer-size=30
Using buffer-size of 30
Connecting to /dev/ttyS5 (buffer-size 30)...
Testing if ubinascii.unhexlify exists ... b'\x1b[0;32mI (426) cpu_start: Pro cpu up.\x1b[0m\r\n\x1b[0;32mI (427) cpu_start: Application information:\x1b[0m\r\n\x1b[0;32mI (427) cpu_start: Compile time:     00:43:59\x1b[0m\r\n\x1b[0;32mI (429) cpu_start: Compile date:     Mar 25 2019\x1b[0m\r\n\x1b[0;32mI (434) cpu_start: ESP-IDF:          v3.3-beta1-268-g5c88c5996\x1b[0m\r\n\x1b[0;32mI (441) cpu_start: Single core mode\x1b[0m\r\n\x1b[0;32mI (445) heap_init: Initializing. RAM available for dynamic allocation:\x1b[0m\r\n\x1b[0;32mI (452) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM\x1b[0m\r\n\x1b[0;32mI (458) heap_init: At 3FFB92B0 len 00026D50 (155 KiB): DRAM\x1b[0m\r\n\x1b[0;32mI (464) heap_init: At 3FFE0440 len 0001FBC0 (126 KiB): D/IRAM\x1b[0m\r\n\x1b[0;32mI (471) heap_init: At 40078000 len 00008000 (32 KiB): IRAM\x1b[0m\r\n\x1b[0;32mI (477) heap_init: At 400928AC len 0000D754 (53 KiB): IRAM\x1b[0m\r\n\x1b[0;32mI (483) cpu_start: Pro cpu start user code\x1b[0m\r\n\x1b[0;32mI (54) cpu_start: Starting scheduler on PRO CPU.\x1b[0m\r\n\x1b[0;32mI (98) modsocket: Initializing\x1b[0m\r\nI (118) wifi: wifi driver task: 3ffe44ec, prio:23, stack:3584, core=0\r\nI (118) wifi: wifi firmware version: 38e2484\r\nI (118) wifi: config NVS flash: enabled\r\nI (118) wifi: config nano formating: disabled\r\n\x1b[0;32mI (128) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE\x1b[0m\r\n\x1b[0;32mI (138) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE\x1b[0m\r\nI (148) wifi: Init dynamic tx buffer num: 32\r\nI (148) wifi: Init data frame dynamic rx buffer num: 32\r\nI (148) wifi: Init management frame dynamic rx buffer num: 32\r\nI (158) wifi: Init static rx buffer size: 1600\r\nI (158) wifi: Init static rx buffer num: 10\r\nI (168) wifi: Init dynamic rx buffer num: 32\r\nStarted webrepl in normal mode\r\nOSError: [Errno 2] ENOENT\r\nMicroPython v1.10-230-ge0c6dfe90 on 2019-03-25; ESP32 module with ESP32\r\nType "help()" for more information.\r\n>>> '
Traceback (most recent call last):
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1255, in connect
    ip_address = socket.gethostbyname(port)
socket.gaierror: [Errno -5] No address associated with hostname

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/kevin/.local/bin/rshell", line 11, in <module>
    sys.exit(main())
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/command_line.py", line 4, in main
    rshell.main.main()
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 2827, in main
    real_main()
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 2789, in real_main
    connect(args.port, baud=args.baud, wait=args.wait, user=args.user, password=args.password)
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1261, in connect
    connect_serial(port, baud=baud, wait=wait)
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1285, in connect_serial
    dev = DeviceSerial(port, baud, wait)
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1549, in __init__
    Device.__init__(self, pyb)
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1339, in __init__
    unhexlify_exists = self.remote_eval(test_unhexlify)
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1459, in remote_eval
    return eval(self.remote(func, *args, **kwargs))
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/main.py", line 1437, in remote
    self.pyb.enter_raw_repl()
  File "/home/kevin/.local/lib/python3.6/site-packages/rshell/pyboard.py", line 187, in enter_raw_repl
    raise PyboardError('could not enter raw repl')
rshell.pyboard.PyboardError: could not enter raw repl

To me it looks like the board doesn't start in raw_repl after a reset but goes straight to repl. As rshell expects it to start in raw_repl, it fails.

rshell doesn't need it to start in the raw reply. It sends  Control-C to interrupt any running program and a Control-D to enter the raw reply. So far I've only tested with the loboris port. I'll try with the official port and see what happens.

Oh then I did not look at the code carefully enough and I'm out of ideas.
Thanks for taking a look.

Some people have added a small sleep to pyboard.py, which I think works around the problem that you're running into. I haven't been able to reproduce the problem locally. In particular, see this comment: https://github.com/dhylands/rshell/issues/27#issuecomment-424726695
I have some ideas on how to improve this so that it will work without an explicit delay but rather waits for the REPL, and I just need to find some time to implement and test it.

I'm wondering if the esp32's that people are having issues with are being reset when the serial port is opened. I've been unable to reproduce this on my SparkFun Esp32 Thing.
I've ordered a DOIT ESP32 DEVKIT V1 so that I can see if I can reproduce this.

This workaround indeed works! Thanks a lot.
My ESP certainly resets everytime I connect to the repl, doesn't matter if I use rshell or putty.

ok - that helps to understand what's going on then. My SparkFun ESP32 Thing doesn't reset when I connect to it.

I (442) cpu_start: Application information:
I (442) cpu_start: Compile time:     13:09:22
I (445) cpu_start: Compile date:     Mar 27 2019
I (450) cpu_start: ESP-IDF:          v3.3-beta1-268-g5c88c5996
I (456) cpu_start: Starting app cpu, entry point is 0x40082950
I (0) cpu_start: App cpu up.
I (467) heap_init: Initializing. RAM available for dynamic allocation:
I (474) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (480) heap_init: At 3FFB9B90 len 00026470 (153 KiB): DRAM
I (486) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (492) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (499) heap_init: At 40093434 len 0000CBCC (50 KiB): IRAM
I (505) cpu_start: Pro cpu start user code
I (76) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.

and rshell is trying to talk to the board while that reset process is happening. Since the way rshell works is essentially to type some commands and parse the response, it gets confused by this initial burst of text. And this is evident in your very first comment:
Testing if ubinascii.unhexlify exists ... b'\x1b[0;32mI (426) cpu_start: Pro cpu up.\x1b[0m\r\n\x1b[0;32mI (427) cpu_start: Application information:\x1b[0m\r\n\x1b[0;32mI (427) cpu_start: Compile time:     00:43:59\x1b[0m\r\n\x1b[0;32mI (429) cpu_start: Compile date:     Mar 25 2019\x1b[0m\r\n\x1b[0;32mI (434) cpu_start: ESP-IDF:          v3.3-beta1-268-g5c88c5996\x1b[0m\r\n\x1b[0;32mI (441) cpu_start: Single core mode\x1b[0m\r\n\x1b[0;32mI (445) heap_init: Initializing. RAM available for dynamic allocation:\x1b[0m\r\n\x1b[0;32mI (452) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM\x1b[0m\r\n\x1b[0;32mI (458) heap_init: At 3FFB92B0 len 00026D50 (155 KiB): DRAM\x1b[0m\r\n\x1b[0;32mI (464) heap_init: At 3FFE0440 len 0001FBC0 (126 KiB): D/IRAM\x1b[0m\r\n\x1b[0;32mI (471) heap_init: At 40078000 len 00008000 (32 KiB): IRAM\x1b[0m\r\n\x1b[0;32mI (477) heap_init: At 400928AC len 0000D754 (53 KiB): IRAM\x1b[0m\r\n\x1b[0;32mI (483) cpu_start: Pro cpu start user code\x1b[0m\r\n\x1b[0;32mI (54) cpu_start: Starting scheduler on PRO CPU.\x1b[0m\r\n\x1b[0;32mI (98) modsocket: Initializing\x1b[0m\r\nI (118) wifi: wifi driver task: 3ffe44ec, prio:23, stack:3584, core=0\r\nI (118) wifi: wifi firmware version: 38e2484\r\nI (118) wifi: config NVS flash: enabled\r\nI (118) wifi: config nano formating: disabled\r\n\x1b[0;32mI (128) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE\x1b[0m\r\n\x1b[0;32mI (138) system_api: Base MAC address is not set, read default base MAC address from BLK0 of EFUSE\x1b[0m\r\nI (148) wifi: Init dynamic tx buffer num: 32\r\nI (148) wifi: Init data frame dynamic rx buffer num: 32\r\nI (148) wifi: Init management frame dynamic rx buffer num: 32\r\nI (158) wifi: Init static rx buffer size: 1600\r\nI (158) wifi: Init static rx buffer num: 10\r\nI (168) wifi: Init dynamic rx buffer num: 32\r\nStarted webrepl in normal mode\r\nOSError: [Errno 2] ENOENT\r\nMicroPython v1.10-230-ge0c6dfe90 on 2019-03-25; ESP32 module with ESP32\r\nType "help()" for more information.\r\n>>> '

In the comment this prints as a very long single line. Adding the delay allows this text to go by, but I think that a better solution is to wait for the '>>>' prompt so that we don't delay when we don't need to.

I just released 0.0.21 which now tries to get a REPL prompt before continuing, I was able to simulate a device which resets (by twiddling the RTS & DTR lines just after opening the serial port) and it now skips that text looking for the >>> prompt.

Thanks for the quick fix! It works great now.
Interesting observation: I had some things connected and it didn't reset on serial connect, so there's some interference from pin usages/states.


Allow settings references in product and page mutations

Update AttributeValueInput with references field
Add reference field to AttributeValue type
Create reference value based on reference entity:

AttributeValue name = reference name
AttributeValue slug = instancePK_referencePK

Task linked: SALEOR-1426

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-1426 Allow setting references in product mutations


Implement Gitaly gRPC services for push and fetch

Choose required gRPC services to be implemented for MVP. The initial version should support fetch and push operations. Decompose this ticket to smaller after investigation.

@stepanvalyavskiy I think we can choose only method required to push and pull from this list: https://github.com/cqfn/degitx/blob/master/docs/API_GITALY.md

@stepanvalyavskiy we need to choose RPC methods to implement for the current milestone, but we'll start implementing this later.

@stepanvalyavskiy we need to choose RPC methods to implement for the current milestone, but we'll start implementing this later.

To fetch from and push to a single degitx node we have to implement:

InfoRefsUploadPack
InfoRefsReceivePack
PostUploadPack
PostReceivePack

To fetch from and push to a single degitx node we have to implement:

InfoRefsUploadPack
InfoRefsReceivePack
PostUploadPack
PostReceivePack

@stepanvalyavskiy thanks, created 4 tickets for them.

@stepanvalyavskiy thanks, created 4 tickets for them.


Run ember-cli-update to v3.28.4

There are a lot of files given the sub projects. Looked over it, didnt see anything to comment on.


Update backup-reports-email.md

Authorize connections to Azure Monitor Logs and Office 365 must be done using an account with en Exchange Online mailbox and at least Contributor privileges to the Azure subscription or Resource Group of the Logic App.

Docs Build status updates of commit ea87173:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/backup/backup-reports-email.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@v-amallick

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thanks for the update.


Introduce dan-level rank assessments (and Kata)

I noticed that this has been suggested multiple times before (and closed multiple times for various reasons) but I think it is finally time to consider this for real.
What I understood/considered from previous issues on this Topic
Previously, certain users advocated for opening up dan-level Kata due to the sheer difficulty of certain algorithmic Kata (e.g. toBrainfuck transpiler) which are believed to be more difficult than typical 1 kyu algorithmic Kata.  Such issues were closed since the previously active Codewars administrator Jake Hoffner wanted to make dan levels distinct from kyu levels by introducing project-level Kata among other things (please do correct me if I am wrong).
What prompted me to re-open this Feature Request?  How is the current situation different from previous situations?
With the swift (and sudden!) support of dedicated proof-assistant languages such as Idris and Agda, there has been a surge in theorem-proving Kata recently.  While it was possible to prove very simple and straightforward statements such as A + B = B + A in Haskell using an array of GHC extensions back when Idris/Agda were not supported, the introduction of the aforementioned languages made it feasible to author theorem-proving Kata of all levels of difficulty ranging from less trivial arithmetic proofs such as 1 + 2 + ... + n = n * (n + 1) / 2 to properties of lists and list operations and even research-level academic topics such as Cubical Type Theory (which I have absolutely no idea what it is about).
There has been a debate as to how such theorem proving Kata should be ranked.  Obviously, proving even the simplest of theorems such as "flipping a binary tree twice gives back the original tree" is in no way 7-8 kyu as some have suggested since a Codewarrior who has only began learning the syntax and semantics of Idris/Agda would seriously struggle to understand deep concepts such as Curry-Howard isomorphism, let alone apply said concepts in the form of machine-checked proofs.  This means that even the simplest of proofs (maybe except absurdly simple proofs such as n + 0 = n for all natural numbers n which is currently offered as an 8 kyu Agda Kata) are currently ranked starting from 4 kyu upwards.
What this means is that since even the simplest of proofs are ranked 4 kyu or above (and ought to be!), slightly less straightforward proofs requiring a bit more insight and intuition are easily ranked as 2 kyu or even 1 kyu.  However, it is totally possible to author rather complex proofs on obscure academic theories such as Cubical Type Theory (a few kata on the topic which can be found in @ice1000 's authored Kata list) or properties of mathematical entities involving extremely abstract concepts such as the injectivity of the type constructor Maybe (which is totally different from proving the injectivity of an ordinary function).  Such Kata are yet an entire level above those "less straightforward" proofs (e.g. List concatenation is injective? Prove it! currently ranked at 1 kyu) but under the current ranking scheme, they would all be ranked as 1 kyu at the very most.  This would lead to a whole bunch of 1 kyu theorem-proving Kata of vastly differing difficulties (ranging from "quite difficult" to "insanely difficult") which could discourage Codewarriors from authoring and/or solving the more difficult ones.
The proposed solution is to open up dan-level Kata and rank assessments, at least up to and including 2 dan.  Even higher dan-ranks may need to be opened up in the near future as ever more complex and abstract theorem-proving Kata are authored.
Supporting comments demonstrating the need to open up dan-level Kata ASAP
@ice1000 recently left a comment on the Maybe Injective Kata stating that it should honestly be ranked 1 dan:

Specifically, I would like to hear the opinions of the following people on this specific Issue:

@ice1000
@lwoo1999
@kazk

Others are free to chime in as well :smile:

Completely disagree. Let me quote my stance from gitter again:

> just because nobody on CW except a few understands proof assistant languages yet doesn't mean you're all gonna rank it blue or purple
> the same thing happened to Haskell katas too, all of them went purple when they really shouldn't
> so I'm gonna say the same old thing again and tell you all to git gud

There has been a debate as to how such theorem proving Kata should be ranked. Obviously, proving even the simplest of theorems such as "flipping a binary tree twice gives back the original tree" is in no way 7-8 kyu as some have suggested since a Codewarrior who has only began learning the syntax and semantics of Idris/Agda would seriously struggle to understand deep concepts such as Curry-Howard isomorphism, let alone apply said concepts in the form of machine-checked proofs. This means that even the simplest of proofs (maybe except absurdly simple proofs such as n + 0 = n for all natural numbers n which is currently offered as an 8 kyu Agda Kata) are currently ranked starting from 4 kyu upwards.

since even the simplest of proofs are ranked 4 kyu or above (and ought to be!),

That's just completely absurd. Just because someone can't handle proof assistant languages or that proof assistant languages are hard to get started (unlike, say, Python) doesn't mean it should be ranked blue. By the same logic everything should be ranked 8 dan because for complete newbies, they all look like that.
If you think about it, it has always been like this: back in 13-14 pretty much every power user is like a noob, and problems that would be yellow or blue nowadays would be blue or purple back then just because users aren't good enough. Did anyone argue otherwise that they're overranked? Anyone?

and even research-level academic topics such as Cubical Type Theory (which I have absolutely no idea what it is about)

Then... go read the papers? You're talking like you can write code in any language without reading any documentation about your language or the domain you're writing code for. That's called spoon-feeding. It's completely optional.
CW gets katas with topics of increasing span over time, and it will only get wider. It's up to you who should catch up to the meta, not instead sitting inside your comfort zone and thinking why isn't everything 8 dan already.

This would lead to a whole bunch of 1 kyu theorem-proving Kata of vastly differing difficulties (ranging from "quite difficult" to "insanely difficult") which could discourage Codewarriors from authoring and/or solving the more difficult ones.

Easy. Just deflate the ranking to make it more realistic. That's the thing we've been advocating for instead of opening dan ranking katas. I'm not sure where you get the latter from.
If you don't want ridiculous ranking on new katas, you gotta re-adjust the old ones, and it's not our fault that it's unavailable at the moment.

By the way, are you aware of a feature called "My Languages"? If you're only familiar with a few languages, there are no reasons to show katas of all languages, and then try tackling them anyway without avail. You're not supposed to be able to solve all katas (even if you want to ;-)). Perhaps you need to rethink about your abilities and stop thinking you're like a certain someone with name starting with a V? :wink:

What this means is that since even the simplest of proofs are ranked 4 kyu or above (and ought to be!)

Sounds more like all the people who are into theorem-proving katas are overranking everything considerably as it has been done with some of the purple Haskell katas we have (there's an Idris kata currently in Beta which ZED.CWT completed by closing holes and ranked 7 kyu, and Donald solved by reimplementing the whole thing from scratch and ranked 1 kyu - wtf?). I'd asked about this on gitter some time ago, and I was told that these katas are blue/purple because it's indeed hard to programmatically prove mathematical theorems, but after seeing this post I'm only further convinced that this whole thing is a joke, and the ranking is 100% based on the fact that this whole theorem-proving topic is very niche, and very different from typical procedural tasks we deal with on CW everyday.

Codewarrior who has only began learning the syntax and semantics of Idris/Agda would seriously struggle to understand deep concepts such as Curry-Howard isomorphism, let alone apply said concepts in the form of machine-checked proofs

The fact that @DonaldKellett openly says this only proves the point that such katas are ranked not based on their difficulty in terms of language/concept but because "plebs are too stupid to solve it" (which is a bit of an exaggeration, but is it?).

slightly less straightforward proofs requiring a bit more insight and intuition are easily ranked as 2 kyu or even 1 kyu

Apparently, now, when somebody created an actually, truly, genuinely hard theorem-proving katas which does deserve to be ranked very high, it can only be approved at the same level as those "not so difficult things" because the very same theorem-proving community (including @DonaldKellett) has been misjudging the katas, and now their rankings are way off from reasonable.

But hey, this is just my opinion. I don't know Idris/Agda, I'm not into theorem-proving, and maybe I'm wrong. So feel free to

there's an Idris kata currently in Beta which ZED.CWT completed by closing holes and ranked 7 kyu, and Donald solved by reimplementing the whole thing from scratch and ranked 1 kyu

I don't know Idris/Agda

Are you sure you know what you are talking about?  "Reimplementing the whole thing from scratch" would imply that I reconstructed basic lemmas as n + m = m + n or n + 0 = n from basic principles before using them in my proof which I have not done.  I have made good use of the standard library - the only thing I didn't use was the interactive theorem-proving capabilities of Idris (which IMO obscures the very nature of the proof one is constructing, e.g. @lwoo1999 once said something to the effect to "sometimes I prove theorems in Idris/Agda without knowing what I am actually proving").
I do agree that certain Kata (such as Haskell's "A + B = B + A") are overhanded but surely not everything is overranked?  Also, as I mentioned, wouldn't a high barrier of entry automatically warrant a somewhat higher rank (if not that high)?

I don't think that's a problem for katas that's as difficult as n-dan will stay beta for a very long time.

By the way, except the univalence axiom, the kata "maybe injective" doesn't involve complicated mathematical concepts. I can explain the idea in mathematics easily. It's agda's fault that the kata is so difficult.

It's agda's fault that the kata is so difficult.

I remain neutral. I'm already used to rank those dan-like Katas as 1kyu, but I feel like it's good if we can have some extra difficulty.

as I mentioned, wouldn't a high barrier of entry automatically warrant a somewhat higher rank (if not that high)?

Have you realized that programming has a very high barrier of entry?

While I agree that ranks used on Codewars have various issues, I don't think adding few dan level will fix anything. I wish we had more flexible and systematic ways to describe the difficulty. Trying to fit challenges from different areas and different languages into a single scale doesn't work.
I also don't think "could discourage Codewarriors from authoring and/or solving the more difficult ones" will be an issue. I don't think most of us are doing for honor points. We simply like to solve/create interesting problems.
Like @Voileexperiments wrote, I'd also recommend deflating the ranks as we get more kata. This will at least provide meaningful scale within the category.
"The sum of an arithmetic progression" is approved with 2 kyu. My Idris experience is about few hours (mostly reading code and docs enough to add Idris support). I have no idea what I'm doing, but I can still get to the point where I can imagine solving it if I learned how to use the functions in the standard library. I can be obviously wrong, but it didn't feel like I needed advanced knowledge of the language.
Is it really 2 kyu? If it really is, I don't think adding 1 dan and 2 dan will be enough.
Would you consider yourself advanced in Idris? If not, then 2 kyu might be too high.

the only thing I didn't use was the interactive theorem-proving capabilities of Idris

Do people regularly write Idris code without it? Add definition, split cases, solve trivial holes, make lemma, inspect the type. Repeating these got me to the point where I need to rearrange the types. I couldn't do it because I don't know anything yet, but it seemed doable if I take the time.
I thought this was the intended workflow. I imagine Agda is similar (gave up for now since setting up Emacs was too time consuming :().

Also, as I mentioned, wouldn't a high barrier of entry automatically warrant a somewhat higher rank (if not that high)?

I don't think so.

The issue here in its general sense is that whenever something new and relatively niche is introduced, since almost nobody knows what they're doing, almost nobody knows what the difficulty scale should be either. The sample size is too small. Nowadays a BF interpreter kata would be at most 6 kyu. They were considered 2 kyu at the past because whenever you mention "language parser", people panic.
There are 2 things to address here:

We don't know what the actual difficulty is, yet. But that's expected, so it's better to be robust
If you don't know what you're doing (like I don't even write Idris/Agda without an IDE and it's hopeless), you tend to be inclined to overrank the kata. It's a poisonous line of thought. This is what the power users meant when they were saying "everything is 8 dan" and "you should git gud instead of overranking everything".

In a sense, I treat kata difficulty akin to junior math olympaid problems: what kind of knowledge do you need to know at the very least to tackle the kata? If you can only recognize a problem by its textbook form, you really need to git gud. Most solutions to actual problems come from novel use and combinations of many easy concepts together.

As a person who is not a mathematician, and knows nothing about theorem proving or AGDA, I feel fully qualified to voice my opinion :)
I am with Voile here - first thing we (CW community) need is possibility to re-adjust difficulty ratings. There were tons of scenarios when task is something like "fizzle a frobble" which is ranked as 2 kyu while it's terribly overranked and should be 4 kyu tops. Then someone creates a kata of "fuggle a fizz" which requires first step of fizzling a frobble, and it should be ranked as 2 or 3 kyu but noone will estimate it as such, because easier, introductory task is already 2 kyu, and this one is definitely harder. Now, it's ranked relatively higher as 1 kyu and instead of adjusting misjudged, easy kata, we get yet another overranked one. (Side note: I have an idea how to handle rank adjustment over time, but I am not sure whether it's the right place to share it).
Another thing is that I do not understand how "sum of arithmetic progression" kata is so difficult, while that's one of few proofs I know how to do (on paper) and I could not even finish 8 kyu a+0=0 kata. Granted, the statement of the problem shown to me by my highschool teacher might be dumbed down to class' level, but still - I cannot understand how you can do academic- or research- level stuff while at the same time I was told one cannot easily do really basic proofs: real numbers too hard, geometry not possible/difficult to represent, so, what's possible at all? Are we stuck with Z and Succ n forever? I know nothing about proof assistants, Idris, AGDA, or even Haskell, but it seems to me that these katas are in the same category as Brainfuck katas: they cannot be compared to any other and need its own hierarchy of difficulties. If it is so, then do not compare them to other 1 kyu katas, just decide that a+0=0 is 8 kyu, Fermat is 1 kyu, and just try to fit a new kata somewhere in between :)
But I know nothing about the topic, so I might be wrong.

Since the official Codewars stance regarding the ranking criteria for theorem-proving Kata has been made clear, I guess there isn't much more I have to say.  I will re-adjust my rank assessments to Kata in said area from this moment forwards.
Speaking of which, the existing Haskell Kata on this topic are heavily overranked since they aren't actually that difficult once you know what you are doing even without the interactive theorem-proving capabilities of Idris/Agda.  I will open a separate Issue addressing this.

Side note: I have an idea how to handle rank adjustment over time, but I am not sure whether it's the right place to share it

@hobovsky If you have an idea, feel free to share it (maybe on a separate Issue though) :smile:


feat(costmodels): migrate to pf4 datatoolbar in price list tab

related #1332

Codecov Report

Merging #1355 into master will increase coverage by 0.87%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1355      +/-   ##
==========================================
+ Coverage   76.99%   77.86%   +0.87%
==========================================
  Files         222      255      +33
  Lines        4347     5291     +944
  Branches      840      978     +138
==========================================
+ Hits         3347     4120     +773
- Misses        874     1015     +141
- Partials      126      156      +30

Impacted Files
Coverage Δ

src/store/rootReducer.ts
100% <0%> (ø)
:arrow_up:

...oard/awsCloudDashboard/awsCloudDashboardWidget.tsx
73.84% <0%> (ø)

...re/ocpUsageDashboard/ocpUsageDashboardSelectors.ts
87.5% <0%> (ø)

...zureCloudDashboard/azureCloudDashboardSelectors.ts
87.5% <0%> (ø)

...e/azureCloudDashboard/azureCloudDashboardCommon.ts
84.37% <0%> (ø)

...store/awsCloudDashboard/awsCloudDashboardCommon.ts
84.37% <0%> (ø)

.../azureCloudDashboard/azureCloudDashboardActions.ts
100% <0%> (ø)

src/store/ocpUsageDashboard/index.ts
100% <0%> (ø)

...ntaryDashboard/ocpSupplementaryDashboardActions.ts
100% <0%> (ø)

src/pages/createCostModelWizard/filterLogic.ts
81.81% <0%> (ø)

... and 27 more

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 f103e65...2ece9dd. Read the comment docs.


Couldn't get the App to work, the command line method will not go to set dates.

Started with the app v. 0.10.2
with a lot of trouble getting the search button to work after entering dates,
So I went to method 2 using the latest master download (~3 hours ago)
It refreshes as per the setting, but stays on the July calendar page, miq-assistance.js is set to go to specific dates in Oct.
The command window is showing:
... (\miq-nz-booking-tools-master\MIQ-Booking-ssistance\node_modules\puppeteer\lib\cjs\puppeteer\common\ExecutionContext.js:151:21)
at WaitTask.rerun (miq-nz-booking-tools-master\MIQ-Booking-Assistance\node_modules\puppeteer\lib\cjs\puppeteer\common\DOMWorld.js:528:37)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:95:5)
(Use node --trace-warnings ... to show where the warning was created)
(node:20256) 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:20256) [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.
Win10 Pro
v. 21H1
I've read, searched and gone through most of the issues.
Any help appreciated.

@Champenoislevrai The calendar staying on July is normal behavior. The script doesn't need to switch the month to find the dates. But if it does find a date it will switch to that date for you, and select the date.

@Champenoislevrai The calendar staying on July is normal behavior. The script doesn't need to switch the month to find the dates. But if it does find a date it will switch to that date for you, and select the date.

Hi Alex, it is not switching and selecting the date. I can confirm that.

Yes, I can confirm that is not happening
I've spent all day on it today, if a date becomes available as entered in miq-assistance.js
the screen has to be manually taken to the month and it's not obvious which the date is when you get there.

This has also been reported in #78 .

There is a long conversation about this topic in #86. I'm closing this in favour of this other issue.


Installation on windows 10 failed

I've downloaded AtomSetup-x64.exe current release is 1.33.1
sha1: bba1f5d10b2a5c494b59422cd617b21212bd4d3b
When I run it, it fails with this message:

There is no setup log available anywhere.
There is one trace on filesystem: %LocalSettings%\SquirrelTemp with files:
atom-1.33.1-full.nupkg
background.gif
RELEASES
Update.exe

sha1 sum of .nupkg matches with RELEASES file.
I've tried this solution: https://github.com/atom/atom/issues/5109#issuecomment-153311100
I've extracted files and tried to run Update.exe --install=. from c:\tmp folder within cmd shell with administrative rights - with no effect. Update.exe just exits silently
Probably related to #9515 #5109
My path is (406 bytes):
PATH=C:\Program Files\Far Manager\;C:\Program Files (x86)\Common Files\Oracle\Java\javapath;C:\Python37\Scripts\;C:\Python37\;C:\WINDOWS\system32;C:\WINDOWS;C:\WINDOWS\System32\Wbem;C:\WINDOWS\System32\WindowsPowerShell\v1.0\;C:\Program Files (x86)\Skype\Phone\;C:\Program Files\Git\cmd;C:\WINDOWS\System32\OpenSSH\;C:\Program Files (x86)\Pandoc\;c:\gow\bin;C:\Users\olekh\AppData\Local\Microsoft\WindowsApps;

I have no antivirus installed, Windows defender is disabled.
Are there any means to get more detailed logs from the installer?
I've even checked Windows Events with no success.
Same issue with AtomSetup.exe (for x86), sha1: df4a83b9743071ecb336424b9e081e045e4c7c29

Do you have a log file at %AppData%\local\atom\SquirrelSetup.log?  Also what version of Windows are you running?

Windows 10 Pro N 1803
Microsoft Windows [Version 10.0.17134.472]

%appdata% is C:\Users\olekh\AppData\Roaming, there is no %appdata%\local folder
I'm using portable variant of Atom now, probably it messed with folders already.
There is a folder %appdata%\Atom\logs which is empty.
C:\Users\olekh\AppData\Local does not contain Atom folder.

Well, I've run procmon (SysInternals Process Monitor) on Update.exe from x64 installation package. It seems to be a 32-bit executable, huh?
It depends on 32-bit msvcr120_clr0400.dll
I have x64 copy of this dll in system32 folder, but not in SysWOW64.
I have tried to install VS2013 redist both x86 and x64, this file is not from this redist probably.
My .NET framework is 4.7.2 and it says everything is already up to date.
Same applies to x86 installation package as it has 32-bit Update.exe too

Ok, I've run sfc /scannow and it restored missing dll.
Atom now installs successfully.
The cause of this error was probably that I've recently uninstalled Visual Studio 2017 from this machine.


name use in form is not the same as name in controller params

lib/puffer/resource.rb#128
 def attributes
   throw [params, name]
   params[name.to_s.singularize]
 end

and when I try to update:
ArgumentError (uncaught throw [{"utf8"=>"✓", "_method"=>"put", "authenticity_token"=>"gMkKL+AVwisos7CDHK88XD3/RAYmR7PgNiUuhOXSpK4=", "cms_menu_item"=>{"name"=>"O Nas i Nie Nas", "menu_id"=>"4ec79997c566db1a1300015d", "page_id"=>"4ec79997c566db1a1300015e"}, "commit"=>"Save", "puffer_scope"=>:admin, "puffer"=>menu_items, "action"=>"update", "controller"=>"admin/menu_items", "id"=>"4ec79997c566db1a1300015f"}, :menu_items]):

params have name ( from form) cms_menu_item, but name in resource is :menu_item

https://github.com/puffer/puffer/blob/master/lib/puffer/resource.rb#L132-138
Just update )

thanks


Split editor/dashboard bundles into chunks with webpack

Context
Split editor and dashboard bundles into chunks for shared code/modules. Related to #868.
Summary
Reduces total load of both editor and dashboard by 26% from 1.45MB to 1.07MB (gzip).

Dedupes common node_modules between editor/dashboard into vendors~edit-story~stories-dashboard (2.28MB / 258KB gzip)
Dedupes common JSON (fonts, templates) into shared chunks (2.4MB / 208KB gzip)

Before

After

Relevant Technical Choices

Combine editor/dashboard webpack configs and use splitChunks to analyze shared code and split into chunks
Use html-webpack-plugin to output chunk manifest of each bundle as a PHP snippet (example below)
Include the PHP snippet in Dashboard.php and Story_Post_Type.php to register/enqueue each chunk script into WP

assets/js/edit-story.chunks.php
<?php return array(
    'css' => ["vendors~edit-story"],
    'js' => ["vendors~edit-story~stories-dashboard-374bdd1db54db5dfc664","vendors~edit-story-981a678a2f0719a0921b"]);

To-do

[ ] Consider moving assets/js/*.[asset|chunks].php files into a new assets/php/ directory

User-facing changes
None, except faster loading going from dashboard -> editor.
Testing Instructions
QA
This PR can be tested by following these steps:

Sanity check that this doesn't accidentally break any dashboard or editor functionality.

UAT

[x] UAT should use the same steps as above.

Reviews
Does this PR have a security-related impact?
No.
Does this PR change what data or activity we track or use?
No.
Does this PR have a legal-related impact?
No.
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[ ] This PR contains automated tests (unit, integration, and/or e2e) to verify the code works as intended (docs)
[ ] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Partial for #868.

I am not sure I love having the hash in the vendor name or the use of Tilde in file names. This is not a conversion WordPress or I personally follow. I think we should stick to WordPress naming / versioning of files like this.
[name]-vendors.js?ver=[hash]

How was this change effect activation and block bundles?

Web stories with no poster images.

I am not sure I love having the hash in the vendor name or the use of Tilde in file names. This is not a convention WordPress or I personally follow. I think we should stick to WordPress naming / versioning of files like this.
[name]-vendors.js?ver=[hash]

I don't really mind that, tbh. It may not be a convention in WordPress core, but it's a legit and proven way for loading built JS files in other plugins, for example Site Kit.

How was this change effect activation and block bundles?

This PR doesn't change those, and I think that's OK. The Gutenberg block is different in a lot of ways (e.g. different version of React loaded by WP instead of us) and the activation notice is only shown once and doesn't need to be unnecessarily split up.
Focusing on editor & dashboard (at least for now) makes sense.

Web stories with no poster images.

What am I looking at here? Those circles are from the new blocks in #6543, from what I can see.

What am I looking at here? Those circles are from the new blocks in #6543, from what I can see.

Wrong PR 🤦


Advice

Pleas finish I would like to know who I should be friends with. Have spent 2 yrs at cmu and made no friends, gotten no jobs, and taken no classes bc I have been waiting for this advice.

Hey, thanks for reaching out. Didn't know anybody was going to look at this repo, so this gives me some motivation to get this to some sort of starting point. Was hoping to open-source this so more people can give their thoughts, but I can start some articles.
The friend one is difficult, I'm not even sure what I originally intend to write about there.
In the meanwhile, you can find some more academic and career-related advice here: https://github.com/mikinty/advice

Ask and you shall receive, happy reading the articles I put up

Ebic ty, some fine literature. I can finally continue my CMU career with this knowledge!.


java.net.UnknownHostException: Unable to resolve host (using mobile connection)

Expected behavior
Sending and fetching mail possible when using mobile connection and when using Wi-Fi connection.
Actual behavior
Sending mail and fetching mail fails when using mobile connection - everything works when using Wi-Fi. K9 is the only app that does not work when using mobile connection. The failure happens all the time so it is not dependent on a particular time or place or on the SMTP server. Background sync is "Never" so I manually fetch the mails from a SMTP server. As soon as I switch on Wi-Fi sending and fetching works normally.
K9mail-error:
K9-Mail version: 5.010
Device make: LGE
Device model: Nexus 5
Android version: 6.0.1
com.fsck.k9.mail.MessagingException: Unable to get IMAP prefix
at com.fsck.k9.mail.store.ImapStore$ImapFolder.getPrefixedName(ImapStore.java:844)
at com.fsck.k9.mail.store.ImapStore$ImapFolder.internalOpen(ImapStore.java:902)
at com.fsck.k9.mail.store.ImapStore$ImapFolder.open(ImapStore.java:867)
at com.fsck.k9.controller.MessagingController.synchronizeMailboxSynchronous(MessagingController.java:1041)
at com.fsck.k9.controller.MessagingController.access$400(MessagingController.java:111)
at com.fsck.k9.controller.MessagingController$8.run(MessagingController.java:934)
at com.fsck.k9.controller.MessagingController.run(MessagingController.java:435)
at java.lang.Thread.run(Thread.java:818)
Caused by: java.net.UnknownHostException: Unable to resolve host "imap.web.de": No address associated with hostname
at java.net.InetAddress.lookupHostByName(InetAddress.java:470)
at java.net.InetAddress.getAllByNameImpl(InetAddress.java:252)
at java.net.InetAddress.getAllByName(InetAddress.java:215)
at com.fsck.k9.mail.store.ImapStore$ImapConnection.open(ImapStore.java:2427)
at com.fsck.k9.mail.store.ImapStore$ImapFolder.getPrefixedName(ImapStore.java:842)
... 7 more
Caused by: android.system.GaiException: android_getaddrinfo failed: EAI_NODATA (No address associated with hostname)
at libcore.io.Posix.android_getaddrinfo(Native Method)
at libcore.io.ForwardingOs.android_getaddrinfo(ForwardingOs.java:55)
at java.net.InetAddress.lookupHostByName(InetAddress.java:451)
... 11 more
Steps to reproduce

Compose mail
Send it
OR
(Manual) Fetch mails

Environment
K-9 Mail version: 5.010
Android version: 6.0.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP

This is a network error. I don't see how this can be K-9 Mail's fault.
Please use the mailing list for support requests.


"Terminal process "bash "" failed to launch (exit code: 127)" debugging C++ over WSL with last Insiders build.

Issue Type: Bug
This issue occurs when starting a debug session even with the simplest hello world C++ program on WSL (did not try other languages). I noticed that on an existing project, then tried a brand new hello world project, obtaining the same issue.
It seems not to be related to the C/C++ extensions, since the extension was not recently updated and using a non-Insiders VSCode version works flawlessly.
Running any debug instance, will generate a "Terminal process "bash "" failed to launch (exit code: 127)", than the "debugging state" will hang until the stop button is pressed
To replicate the bug, just create a simple C++ project and try to debug the main file.

VS Code version: Code - Insiders 1.57.0-insider (45aafeb326d0d3d56cbc9e2932f87e368dbf652d, 2021-05-18T05:14:15.253Z)
OS version: Windows_NT x64 10.0.19041
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2
Remote OS version: Linux x64 5.4.72-microsoft-standard-WSL2

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 x 2592)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.78GB (16.54GB free)

Process Argv
--crash-reporter-id b1bdaaae-e0bb-4809-9b74-caea9dd5a156

Screen Reader
no

VM
0%

Item
Value

Remote
WSL: CentOS8

OS
Linux x64 5.4.72-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 x 2592)

Memory (System)
24.88GB (20.44GB free)

VM
0%

Item
Value

Remote
WSL: CentOS8

OS
Linux x64 5.4.72-microsoft-standard-WSL2

CPUs
Intel(R) Core(TM) i7-10750H CPU @ 2.60GHz (12 x 2592)

Memory (System)
24.88GB (20.44GB free)

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

Bookmarks
ale
13.0.4

remote-ssh
ms-
0.65.4

remote-ssh-edit
ms-
0.65.4

remote-wsl
ms-
0.56.2

gitlens
eam
11.4.1

vscode-reveal
evi
4.0.3

comment-anchors
Exo
1.9.3

cpptools
ms-
1.4.0-insiders2

asn1
wil
0.5.3

Do you set terminal.integrated.inheritEnv?

Yes, I've tried true, false and unsetted (which should default to true, I suppose...?). Setting terminal.integrated.inheritEnv or not does not change the behaviour. Also, setting it or not does not alter the (correct) behaviour with the stable version of VSCode.

These are reported elsewhere as well.
#124007 - This one was incorrectly closed
#123949 - This was closed as a duplicate of the previous one.

Also facing this issue

Also experiencing the exact same problem.
Version 1.57.0-insider

Same issue here using zsh and a python program to debug.
Code Version 1.57.0-insider (from 2021-05-20T05:14:43.087Z)
WSL: Debian Buster
WinVer: 1909 Build 18363.1500
I very briefly see the following text in the terminal before it switches back to my previous terminal:
This shell is running on your local machine, NOT on the connected remote machine
/usr/bin/zsh: can't open input file:

After that I get the same error
The terminal process "zsh ''" failed to launch (exit code: 127).

Solved with setting  "console": "internalConsole" in the configuration (from #123949)

"console" is not possible to set with a cppdbg launch configuration. Therefore it's not a solution.

Also have this issue in Java debugger. Since the terminal is the only way to accept program input, hope this issue can be fixed before new VS Code release out.

I also faced this problem, I have tried WSL (Ubuntu 20.04), Remote SSH(CentOS) and Windows, this problem will appear everytime when I was launching a debug task. I use default setting on anything about terminal.integrated.
Version: 1.57.0-insider (user setup)
Commit: 4fd4e10e99eff9eb77408b71c7f58b7ed7458ee5
Date: 2021-05-21T05:13:41.097Z
Electron: 12.0.7
Chrome: 89.0.4389.128
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19043

same issue. happens only in insiders version

same issue

Also having this issue only with code-insiders. Code is configured with the same exact settings as Code Insiders, which works.
I tried disabling extensions in case it might have an effect but the result was the same.

Also having this issue only with code-insiders. Code, which works, is configured with the same exact settings as Code Insiders. I tried disabling extensions in case it might have an effect but the result was the same.

I'm also getting issue trying to run Java apps / macOS 11.3.1. Fails with Insiders only.

Possible improvements I just made:

Force inheritEnv/useShellEnvironment for ext debug terminals https://github.com/microsoft/vscode/issues/124041
Add -l to macOS default profiles, was only in fallback profiles https://github.com/microsoft/vscode/issues/124409

Someone is pointing out the double single quote ('') here, but it hasn't changed recently: https://github.com/microsoft/vscode/issues/123949#issuecomment-841847897

A cpp duplicate: https://github.com/microsoft/vscode-cpptools/issues/7551

For anyone experiencing this issue can you enable trace logging and paste is logged when creating the terminal? This will give information about how the terminal launches

These logs contain a lot of personally identifying information, is there a particular log file you would like to look at, and how much of it? Is it "extension host"? "Window"? "Main"?
On another note, is it possible to downgrade to previous Insiders build, because this bug has thrown a giant wrench in my workflow?

@anatolyburakov this one:

Plus this one (which will be in the remote server log):

The env part is important but you may want to sanitize some of that.

I'm not sure how to get to the screens you have shown (please don't assume everyone is a proficient web developer!), could you please provide more details as to how to get there?
Looking in remote logs, i found the following in "Remote Server" log:
[2021-05-25 13:50:23.868] [remoteagent] [debug] Terminal process launching on remote agent {"shellLaunchConfig":{"name":"Build docs","executable":"bash","args":["-c","make docs"],"cwd":"/root/xxxx","useShellEnvironment":true},"initialCwd":"/root/xxxx","cols":195,"rows":15,"env":{"no_proxy":"localhost,<IP_ADDRESS>","USER":"root","SSH_CLIENT":"xxxx","XDG_SESSION_TYPE":"tty","SHLVL":"0","MOTD_SHOWN":"pam","HOME":"/root","OLDPWD":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46","DBUS_SESSION_BUS_ADDRESS":"unix:path=/run/user/0/bus","https_proxy":"xxxx","LOGNAME":"root","http_proxy":"xxxx","_":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/node","XDG_SESSION_CLASS":"user","XDG_SESSION_ID":"5296","PATH":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/bin:/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin","ftp_proxy":"xxxx","XDG_RUNTIME_DIR":"/run/user/0","LANG":"en_US.UTF-8","SHELL":"/bin/bash","PWD":"/root","SSH_CONNECTION":"xxxx","HISTTIMEFORMAT":"%d/%m/%y %T ","LESSCLOSE":"/usr/bin/lesspipe %s %s","LESSOPEN":"| /usr/bin/lesspipe %s","LIBVIRT_DEFAULT_URI":"qemu:///system","XDG_DATA_DIRS":"/usr/local/share:/usr/share:/var/lib/snapd/desktop","BROWSER":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/bin/helpers/browser.sh","TERM_PROGRAM":"vscode","TERM_PROGRAM_VERSION":"1.57.0-insider","COLORTERM":"truecolor","VSCODE_GIT_IPC_HANDLE":"/run/user/0/vscode-git-ac21d75fa8.sock","GIT_ASKPASS":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/extensions/git/dist/askpass.sh","VSCODE_GIT_ASKPASS_NODE":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/node","VSCODE_GIT_ASKPASS_MAIN":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/extensions/git/dist/askpass-main.js"}}

This looks vaguely relevant to what we're trying to achieve here...

@anatolyburakov and that's exiting with a code of 127? It looks fine based on the log 🤔
Shell and args look fine:

"executable":"bash","args":["-c","make docs"]

useShellEnvironment is set (forces inheritEnv = true):

"useShellEnvironment":true

PATH looks fine:

"PATH":"/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/bin:/root/.vscode-server-insiders/bin/286b643ecd231f66f3f2e67e1456f3c601838b46/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin"

@Tyriar is there anything else i can check and help debug this issue?

Same problem here on Linux debugging python code.

Will be fixed in Insider's tomorrow.


SUBMARINE-579. [WEB] Add duration information to experiment

What is this PR for?

Implement duration of experiment.
Remove progress bar.

What type of PR is it?
[Feature]
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-579
How should this be tested?
https://travis-ci.org/github/kobe860219/submarine/builds/713190854
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@pingsutw  @liuxunorg  Please help me review this pr. Thanks :))

@jiwq Sorry about that I forgot to update screenshots. I have update it :)

Thanks @kobe860219 for the contribution, @liuxunorg and @pingsutw help to review.


Postman integration

Automatically create a postman collection based on the schema.
Here is the Postman collection schema definition: https://schema.postman.com/json/collection/v2.1.0/collection.json.

I'd like to take a stab at this

Go for it!!!

I think this was a great idea that's not really needed anymore. Thanks!


Use repository URL provided as input for finding a commit

In cases that the repository URL for an artifact is not available, we should allow the user to provide the URL as input and let the commit finder find the commit if the input PURL has a version, e.g.,
macaron analyze -rp https://github.com/assertj/assertj -purl<EMAIL_ADDRESS>--skip-deps

Right now we require the user to provide the commit in addition to the URL even though the PURL contains a version:
[ERROR] Please provide the commit digest for the repo at https://github.com/assertj/assertj that matches to the PURL string<EMAIL_ADDRESS>

I had a typo in the artifact name: assertj.core should be assertj-core. So, assertj is not a good example, but I can imagine other libraries that might have this problem.

One example of a project that could benefit from this is Tiny Media Manager. This project stopped releasing on Maven after version 3.0.5 in 2019. Passing Macaron a PURL with a later version, e.g. 4.3.13, would lead to no repository being found.


Remove the need for publishing config using MergeConfig

Adding this prevents any errors when config isn't published.

Ah yes, I thought this package already had this. Thanks a lot! Will release it immediately.

Released in 2.5.2.


transition-property: transform is not idempotent

Prefixing transition properties is not idempotent -- if you pass in the same object to prefix multiple times, it will break the style object each time.
var assert = require('assert')
var prefix = require('inline-style-prefix-all')
var style = {
  transitionProperty: 'transform'
}

JSON.stringify(prefix(style)) // => yields correct output
JSON.stringify(prefix(style)) // => yields insane output

Example: http://requirebin.com/?gist=73acb9d22d0998fb299932282ae31358

Oh I see. I think we need to check if the value already includes prefixed versions and if so not do anything. Thanks for reporting! I am currently having exams until May, 16. If you want to fix this on your own and push a pull request, feel free to! Otherwise this will take some time until it gets fixed.

:+1: Opened #9, let's discuss there.


Dynamic options

Hi,
it is possible to change the options after the creation of the object?
Like the following:
var clusters = new L.MarkerClusterGroup();
clusters.setOption( 'showCoverageOnHover', false );

Until now i tried changing valued in clusters.options with any luck.

Not with the way this option is set up:
https://github.com/Leaflet/Leaflet.markercluster/blob/master/src/MarkerClusterGroup.js#L655-L660
You could add functions which replicate that to toggle it

Ok i try to write a pull request for that. What obout the other options?

I'm not sure. I think most of them will be hard to vhange
On Friday, 3 April 2015, Riccardo Volonterio<EMAIL_ADDRESS>wrote:

Ok i try to write a pull request for that. What obout the other options?
—
Reply to this email directly or view it on GitHub
https://github.com/Leaflet/Leaflet.markercluster/issues/478#issuecomment-89236627
.

Closing, PR will be accepted if provided, but not keeping an issue open for it.


Set up to record without stopping the pipeline

Required Info

Camera Model
D400

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Platform
PC

SDK Version
2.19.0

Language
C++

Segment
others

Issue Description
I am using RealSense depth data (D435) to detect objects in the scene. As soon as it is detected by my software, I stop the streaming RealSense pipeline and enable it to record video of this object. The problem is that to stop the streaming and start the recording pipeline takes approx. 1150 ms and 300 ms, so, in total, almost 1.5s meaning that my object has almost left the FOV.
I was wondering if there is a solution that would minimize (or even eliminate) this time taken to initialize the camera for recording.
Minimal example:
int main(int argc, char* argv[])
{
	auto pipe = std::make_shared<rs2::pipeline>();
	rs2::config cfg;
	cfg.disable_all_streams();
	cfg.enable_stream(RS2_STREAM_DEPTH, 640, 480, RS2_FORMAT_Z16, 30);
	cfg.enable_stream(RS2_STREAM_COLOR, 640, 480, RS2_FORMAT_RGB8, 30);

	pipe->start(cfg);
	auto start = std::chrono::high_resolution_clock::now();
	pipe->stop();
	auto finish = std::chrono::high_resolution_clock::now();
	std::chrono::duration<double, std::milli> elapsed = finish - start; // approx. 1.1s
	std::cout << elapsed.count() << std::endl;

	auto start2 = std::chrono::high_resolution_clock::now();
	cfg.enable_record_to_file("a.bag");
	pipe->start(cfg);
	auto finish2 = std::chrono::high_resolution_clock::now();
	std::chrono::duration<double, std::milli> elapsed2 = finish2 - start2;
	std::cout << elapsed2.count() << std::endl;
	pipe->stop();
	return 0;
}

RealSense SDK 2.0 has a multi-streaming model that enables you to do different processes at the same time.  A limitation of it though is that two processes cannot access the same type of stream.
So Process A of a camera could access the depth stream for example, whilst Process B on the same camera accesses a non-depth stream at the same time.  This implies that to detect and record at the same time, the recording and detection process must make use of different stream formats.
https://github.com/IntelRealSense/librealsense/blob/master/doc/rs400_support.md#multi-streaming-model

@MartyG-RealSense Thank you for your response. I may not have put all the details in my question. I am using only the depth data to detect objects, but I would like to record both colour and depth data for the 'video'.
I have now successfully incorporated recorder pause/resume functionality for a fast recording startup. To resume the pipeline (device.asrs2::recorder().pause();) takes approx. 0.03 ms. As the pipeline initialization is done when the objects are not present, I am not losing any data.

You are very welcome.  I'm very glad you created a solution that works for you.   :)


Add autoscan option

Hi there,
We use Veracode to scan our Java applications for vulnerabilities. This plugin has been absolutely fantastic in integrating into our CICD pipeline. What we were hoping for, however, was an autoscan option. This PR adds support for turning the "Autoscan after prescan" on.

@DavidGamba - thanks for the feedback! I definitely considered that, but I felt like adding a new overload would be safer from a backwards-compatibility point of view. What do you think?

The only consumers are the tasks within the plugin. The only API you need to maintain is the actual top level task.


Update ClinVar VCF

Looks like we should update the version of the ClinVar VCF being used for annotations. I noticed this because I am doing work on clinical specimens previously tested with other methods getting our clinical NGS pipeline operational and have come across variants annotated with a clinical significance of "other" in the 20150305 release that have been updated to pathogenic in newer versions. In this case the variant has always been pathogenic (since 2007 I believe) but ClinVar has had some issues with their VCFs and sorting things into the right categories.

Awesome, Brent. Just a reminder to set the correct S3 permissions for the .gz and .gz.tbi file as I always forget to do so.

yep. doing a full install to check everything.


Instruction for self-hosting

Structure the code in a way that makes self-hosting easy with a config page where people simply need to define a few variables (like discord bot token) to make it work.

Finally added some instructions on self-hosting: https://github.com/NotNorom/discord-banner-bot#hosting-the-bot-yourself
Please leave feedback when you encounter any issues. I hope these instructions are enough


Unable to import types

In my typescript project I am having issues importing types such as the BracketsViewer. It appears that this was completed according to https://github.com/Drarig29/brackets-viewer.js/issues/27 but I am using version 1.6.2 and I am unable to import the types.
I am not much of a typescript guru but I believe that either the tsconfig.json needs declaration set to true and/or the package.json needs to have the types property set. I am willing to look into this issue and make a fix if problem is not between chair and keyboard.
For more context I am able to import BracketsManager like so:
import { BracketsManager } from 'brackets-manager/dist'

But I cannot import BracketsViewer, I have tried the following:
import { BracketsViewer } from 'brackets-viewer/dist';

and the following as mentioned in https://github.com/Drarig29/brackets-viewer.js/issues/27
import { BracketsViewer } from 'brackets-viewer';

I get the following error: TS2307: (TS) Cannot find module 'brackets-viewer' or its corresponding type declarations.

Hello, have you find any solution?

Did you install brackets-viewer?

Did you install brackets-viewer?

Yes I'm still same...

What package manager are you using? And how are you importing the viewer?

What package manager are you using? And how are you importing the viewer?

Using yarn but I tried on npm too doesn't matter it's giving me to "that error.
import BracketsViewer from "brackets-viewer" //giving error

const BracketsViewer = require("brackets-viewer") // giving same error too


Services.d not reliably starting up on Kubernetes.

Hello,
Having a little trouble getting an NGINX container built on alpine-linux to start on Kubernetes. Dockerfile is at the URL below:
https://github.com/andrewhowdencom/docker-nginx/blob/ahowden_rebuild-on-alpine/Dockerfile
Used as the base for
https://github.com/andrewhowdencom/m2onk8s/blob/master/package/build/docker/nginx/Dockerfile
Kubernetes RC configuration is here: https://github.com/andrewhowdencom/m2onk8s/blob/master/package/build/kubernetes/magento-rc.json
Logs from
$ kubectl logs {pod} -c {container} -p
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01-resolver-logdir: executing...
[cont-init.d] 01-resolver-logdir: exited 0.
[cont-init.d] done.
[services.d] starting services
[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] syncing disks.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.

Importantly, the services.d call never finishes.
From local on the Kubernetes node (and invoked by Docker):
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 01-resolver-logdir: executing...
[cont-init.d] 01-resolver-logdir: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
^C[cont-finish.d] executing container finish scripts...
[cont-finish.d] done.
[s6-finish] syncing disks.
[s6-finish] sending all processes the TERM signal.
[s6-finish] sending all processes the KILL signal and exiting.

Still looking into why, but heads up!

The super weird part is one container in the pod always starts, and it's not always the same container. At the moment my thinking is maybe services.d is starting something that builds on the same port as services.d in another container, which being in the same pod, craps out -- I think they share the same address space within the pod.

@andrewhowdencom
Hi Andrew, the log indicate that the resolver service fails to start. I recently changed it so that the service logs to a file instead to stdout - probably not the best idea for debugging purposes...
Is there anyway you can get the logfile at  /var/log/go-dnsmasq from a failing container?
Something along:

docker export -o export.tar CONTAINER_ID
tar --extract --file=export.tar var/log/go-dnsmasq
cat ./var/log/go-dnsmasq

2016-02-05 09:24:41.354069500  level=info msg="Starting go-dnsmasq server 0.9.8 ..."
2016-02-05 09:24:41.354116500  level=info msg="Search domains in use: [default.svc.cluster.local. svc.cluster.local. cluster.local. c.m2onk8s.internal. google.internal.]"
2016-02-05 09:24:41.354275500  level=info msg="Ready for queries on tcp://<IP_ADDRESS>:53 - Nameservers: [<IP_ADDRESS>:53]"
2016-02-05 09:24:41.354301500  level=info msg="Ready for queries on udp://<IP_ADDRESS>:53 - Nameservers: [<IP_ADDRESS>:53]"
**2016-02-05 09:24:41.354367500  level=fatal msg="listen udp <IP_ADDRESS>:53: bind: address already in use"
**
https://github.com/kubernetes/kubernetes/blob/master/docs/design/networking.md#container-to-container

All containers within a pod behave as if they are on the same host with regard to networking. They can all reach each other’s ports on localhost.

I don't know well enough what go-dnsmasq does to say whether it's worth trying to resolve. It looks like you can only run one per pod?

FYI, with the latest version you can have one container in the Pod run go-dnsmasq as usual but disable the service for the other containers by passing the env var ALPINE_NO_RESOLVER with a value of True. The primary nameserver for these containers will still be set to localhost, hence they will use the DNS server of the sibling container.
Example:
apiVersion: v1
kind: Pod
metadata:
  name: redis_django
  labels:
    app: web
spec:
  containers:
    - name: key-value-store
      image: alpine-kubernetes-redis
      ports:
        - containerPort: 6379
    - name: frontend
      image: alpine-kubernetes-django
      ports:
        - containerPort: 8000
      env:
        - name: ALPINE_NO_RESOLVER
          value: True

Fixed in https://github.com/janeczku/docker-alpine-kubernetes/commit/0a4e5a1b8f38611761f513f614b2ea241764e986

Couldn't find a better way to thank you than to comment on this. Thanks!


[11.x] Add deepArrayify Method to Easily Convert Nested Objects into Arrays in Obj.php

This pull request adds a deepArrayify function to Obj.php, making it easier to convert all objects in a data structure into arrays, even in deeply nested setups. This function helps developers get consistent array outputs, making it simpler to work with complex structures.
Usage Example
$object = new StdClass();
      $object->name = 'Laravel';
      $object->features = [
          (object) ['feature' => 'Eloquent'],
          (object) ['feature' => 'Blade']
      ];

// Result dd((array) $object);

array:2 [▼
  "name" => "Laravel"
  "features" => array:2 [▼
    0 => {#257 ▼
      +"feature": "Eloquent"
    }
    1 => {#258 ▼
      +"feature": "Blade"
    }
  ]

// Result dd(Obj::deepArrayify($object));

array:2 [▼
  "name" => "Laravel"
  "features" => array:2 [▼
    0 => array:1 [▼
      "feature" => "Eloquent"
    ]
    1 => array:1 [▼
      "feature" => "Blade"
    ]
  ]
]

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If applicable, please consider releasing your code as a package so that the community can still take advantage of your contributions!


The UI does not look good.

Description
The user interface (UI) does not look good on the Courses page. I have tried the following steps to resolve the issue:

Checked Layout: Verified the HTML structure for proper layout but found no significant issues.
Reviewed CSS Styles: Examined the CSS for inconsistencies and made adjustments, but the UI still appears unappealing.
Tested Responsiveness: Viewed the page on different screen sizes to check for responsiveness issues, but the UI remains inadequate.
Checked Browser Compatibility: Tested the page on multiple browsers to rule out compatibility issues, yet the appearance did not improve.

Steps to Reproduce
Steps to Reproduce the UI Issue:

Open the web application and navigate to the Courses page.
Observe the overall layout and structure of the UI.
Inspect CSS styles of various elements for any issues.
Resize the browser window to test responsiveness.
View the Courses page in different browsers for consistency.
Check for missing assets like images, icons, and fonts.

Expected Behavior
No response
Actual Behavior
No response
Screenshots
You can Check out form this screenshot..

Additional Information
No response
What browser are you seeing the problem on?

[ ] Firefox
[ ] Chrome
[ ] Safari
[X] Microsoft Edge

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

yeah i can do it! please assign under it to me.

@RAMIREDDYGAYATHRI02 are you doing this on purpose
i gave you warning 2-3 times now
do you want to really get blocked ??

this issue has been solved, so we are closing this issue.


getProducts() and getSubscriptions() show duplicates

Version of react-native-iap
2.1.1
Platforms you faced the error (IOS or Android or both?)
IOS
Expected behavior
getProducts and getSubscriptions should return a singular list of products
Actual behavior
Each successive call to either of these functions returns a cumulative list, instead of the same list.
Tested environment (Emulator? Real Device?)
Both
Steps to reproduce the behavior
Call getSubscriptions() twice in a row
This seems to be down to two problems in the getProducts() function in Objective-C:
Firstly this line:
if (cur.productIdentifier == aProd.productIdentifier) {
...fails to compare the strings correctly, but it works with:
if ([cur.productIdentifier isEqualToString:aProd.productIdentifier]) {
Secondly this line:
if (delTar > 0) {
...doesn't account for a duplicate at position 0, so it should instead be:
if (delTar >= 0) {
These fixed work for me.
Cheers

@petetastic Yeah looks bug in this PR. Would you like to give us a [PR] to resolve this?

eek! I'd love to, but I'm afraid I don't know how to do that! Happy to if you show me how, otherwise equally I can wait until you fix it, that's not a problem. Cheers.

You can see guide here. Generally, you may fork our repo then update your fix and then create a PR.

@petetastic I've updated your fix in 2.1.2 please try again.


[feat] add argument to ignore devices

Implement #137
--ignore-device takes the same format as --device and ignores the devices listed, separated by semicolon. If a device is listed both in the --device argument and --ignore-device argument, it is not added.

Hi before I review this I notice that no documentation was added. Would you mind documenting the flags you're adding?

CC: @zubairmh can you review this PR?

I'll wait on zubairs review too, if he doesn't respond by today, I'll merge it.

Hi, I would like to merge this, can you kindly rebase this? My apologies for the delay.


Feature request: Labels for Wikidata items

Hi @tomayac,
I really like the toolkit!
It would be cool to have a function that fetches labels for QIDs. Do you think that is feasible? What would be the best way to implement this?
Andreas

@athalhammer A WIKIDATALABEL function has just been implemented:

Thanks!


AUR release

I'd like to use this app on Arch. I would appreciate it if you uploaded it to AUR 🙏

I just looked into what it would take to package it for the AUR and I don't have the time as of now. However, feel free to take it and package it for the AUR as a contribution.

Also, since applets will be distributed through Flatpak in the future, it might not be necessary to package it.

I actually can’t make this one into a Flatpak. It can’t access the Tailscale CLI; I’ve tried.

There seems to be a flatpak app that can interface with the CLI. https://flathub.org/apps/dev.deedles.Trayscale

That app is using the API calls, which I didn’t want to use. I wanted to just use the CLI commands.

If anyone would like to contribute by packaging this for the AUR, it would very much be appreciated.


Emails received from Jenkins job is not having test run details

Details of the requirements:
trainer: Installed using gem and not using fastlane.
CI: Jenkins
To convert xcodebuild output to xml I am using trainer tool.
I have a test suite of 20 test cases written in swift using XCTest UITesting and setup a jenkins job to run the suite. The job runs on scheduled time and send an Email to the recipients. But, when I received the Email, the email is empty.
I am using Email-plugin for jenkins that uses xml file generated by job and uses jelly script to send the Emails in html/text format.
When I am checking console output of test job, it is nicely formatted.
xcodebuild -workspace ../../NYTimes.xcworkspace -scheme "$scheme" -sdk "$1" -destination "$2" -derivedDataPath $3 test
run trainer --path ../NYTimesUITests/test-output/Logs/Test -e .xml --output ../NYTimesUITests/Scripts/build/reports
In first line above, using test-output for 3 and it is generating the test output in the workspace path.
In second line I am running trainer to specify the path where to find the derived data path and it should convert the output to xml at desired location.
Please help me in resolving this issue. I already had a Email discussion with Felix and as suggested i am creating this github issue.
I know I am missing something but not able to figure it out.
I am attaching screenshots here.

The first screenshot is from appium python test running on the same slave machine.
The second screenshot is the one I am facing issue.
The third one is from jenkins post build action.


increment gradle and log4j version

Codecov Report

Merging #9 (9469ff6) into develop (76b050e) will increase coverage by 0.23%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop       #9      +/-   ##
=============================================
+ Coverage      85.62%   85.85%   +0.23%
  Complexity       444      444
=============================================
  Files             67       67
  Lines           1092     1089       -3
  Branches          28       28
=============================================
  Hits             935      935
+ Misses           140      137       -3
  Partials          17       17

Impacted Files
Coverage Δ

...a4gh/registry/exception/CustomErrorAttributes.java
25.00% <ø> (+10.71%)
: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 76b050e...9469ff6. Read the comment docs.


feat(NumberTheory/Harmonic): asymptotics of Riemann zeta as s → 1

Show that the constant term of the Laurent series of Riemann zeta at s = 1 is the Euler-Mascheroni γ.

@sgouezel Thank you for the review! I have pushed a new version addressing your comments.

bors r-

bors r+


Make Client threadsafe

reqwest::Client "holds a connection pool internally, so it is advised that you create one and reuse it".
It would be nice if elastic::client::Client could also be re-used (for example, as a static ref with lazy_static). The headers field of elastic_reqwest::RequestParams prevents one from doing so:
`std::cell::UnsafeCell<std::option::Option<std::vec::Vec<std::vec::Vec<u8>>>>` cannot be shared between threads safely

It would be possible to work around this—I haven't thought about this very much, but the most naive approach would be to store raw header keys/values for RequestParams as owned strings or vecs, and then setting them back on the actual request.
Another workaround/compromise would be to provide an elastic::client::Client constructor that took both a &reqwest::Client and the RequestParams. That way the user could static-ify their client outside the elastic API.
However, that also would require some non-trivial changes (making the client some kind of Cow or Borrowed/Owned enum, since the current elastic::client::Client owns its reqwest client).

You're right, the Client definitely needs to be thread-safe. It's also an issue if you construct a request and want to send it from another thread. Right now you can't do that. I think being able to pass a reqwest::Client in too is also a useful thing to do.
I'll have a look at hypers raw header support, and if that removes the UnsafeCell then that'll be a good start.
We've got a story for adding a ClientBuilder, so one of the use cases I'll add for that is sharing a Client with lazy_static, and passing in your own reqwest client, probably as an Arc<Client>, since it doesn't need mutability to call.


Enable Sourcegraph

I want to use Sourcegraph for pythoscope code search, browsing, and usage examples. Can an admin enable Sourcegraph for this repository? Just go to https://sourcegraph.com/github.com/mkwiatkowski/pythoscope. (It should only take 30 seconds.)
Thank you!

Done, enjoy! :)

Thanks MK !


Change quantifier syntax from x**y to x*y

x*y is the correct syntax according to the spec.
x**y is still allowed for backwards compatibility.
Signed-off-by: Øyvind Rønningstad<EMAIL_ADDRESS>

should we keep some of them to ensure that backwards compatibility is kept?

Good idea, I changed one instance back.


Changes for ACS CallRecording BYOS with ManagedIdentity

StartRecordingOptions - new field added ExternalStorageLocation to specify blob container uri path

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Communication.CallAutomation


Upload File: Not clear that user must scroll to reach next steps

The progress bar is also not visible when the user clicks Submit, unless they scroll further down.
@sabahfromlondon What do you think a good solution is? Should the page auto-scroll after clicking Submit?
https://user-images.githubusercontent.com/26463/114888858-99df2680-9dd7-11eb-9af8-238562de5771.mov

Current solution: add scroll down after analysis is finished.

I just watched the video again and I think this could be a messy UX issue. I think there are two possible options:

Once the file has been analysed, remove the upload box and replace it with the feedback message (instead of the message being on the top right of the screen. Below that add a heading with the text: "Select how to access the flattened data"
The other option is for the user to be taken to a new page after the file has been analysed. The new page will have the heading "Select how to access the flattened data" and the two options only.

I think the second option will be the best one, but happy to hear other views.


responsive header, just need width at smallest

Header is now much more responsive. At smallest size, text still shows under header because it's not full width, but that might be the only fix left.

Not merging this becuase it didn't fix what I thought it fixed, and desktop is a tiny bit worse.  :P  So, leave it the way it is for now, since desktop already looks fine.

Alright, I don't know what is going on, because the small previews look perfect on my laptop, but then have horizontal scroll on the phone.... but this is pretty darn responsive otherwise.

It almost looks intentional... so I vote merge, and let the chips fall where they may!!


Feature Request: Ability to Snooze

First off great extension guys, really well done. I have a quick feature request if you don't mind.
Background
When a Pomodoro goes off, I often find myself in the middle of a task that I don't want to stop at that very moment so I end up closing the Pom window. The problem is that I end up forgetting that the Pom hit and I keep working for another 20 minutes.
Proposal
It would be REALLY nice to have a setting that lets you snooze a Pomodoro for 0-x times for y min each snooze (Personally I would set this to 1 snooze/5 min). Each time the snooze goes off the screen we have now would be displayed, only with a snooze button added. If someone snoozes too many times for a given Pom the snooze button would be disabled.
Thanks
Bob Riley

Hi Bob, thanks for the detailed request.
I fight hard against complexity and feature creep – just look at the issues page, there are another dozen features there – but I've had such a need myself before, and I know where you're coming from.
I think this will manifest as a default snooze time setting (e.g. 5 minutes) with a button on the expiration page that lets you snooze alongside a "snooze counter". I don't think I would add a snooze limit setting.

Very cool, I look forward to the feature :)
Bob
On Fri, Sep 1, 2017 at 2:45 PM, Chris Schmich<EMAIL_ADDRESS>wrote:

Hi Bob, thanks for the detailed request.
I fight hard against complexity and feature creep – just look at the
issues page, there are another dozen features there – but I've had such a
need myself before, and I know where you're coming from.
I think this will manifest as a default snooze time setting (e.g. 5
minutes) with a button on the expiration page that lets you snooze
alongside a "snooze counter". I don't think I would add a snooze limit
setting.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/schmich/marinara/issues/74#issuecomment-326690445,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABF7M3hdBJhSmcRRkvoc_5jBeUi8tYwSks5seHrjgaJpZM4PJGCh
.


*: update backup policy spec

@etcd-bot retest this please

@etcd-bot retest this please

@etcd-bot retest this please

Disregard previous test result.
 That might affect testing.

@etcd-bot retest this please

@etcd-bot retest this please

lgtm

fixes https://github.com/coreos/etcd-operator/issues/703