Unrecognized val_loss metric

RuntimeWarnings due to unrecognized val_loss metric
pytorch_lightning callbacks are unable to recognize val_loss from validation_step()
To Reproduce
Run CoolModel from
Steps to reproduce the behavior:

Go to Minimal-Example
Run upto trainer.fit()
Scroll down to the end of an epoch
See error -

`/opt/conda/lib/python3.6/site-packages/pytorch_lightning/callbacks/pt_callbacks.py:314: RuntimeWarning: Can save best model only with val_loss available, skipping.
' skipping.', RuntimeWarning)
/opt/conda/lib/python3.6/site-packages/pytorch_lightning/callbacks/pt_callbacks.py:144: RuntimeWarning: Early stopping conditioned on metric val_loss which is not available. Available metrics are: loss,avg_val_loss
RuntimeWarning)`
Code sample
Just the Minimal Example

import os
import torch
from torch.nn import functional as F
from torch.utils.data import DataLoader
from torchvision.datasets import MNIST
import torchvision.transforms as transforms

import pytorch_lightning as pl

class CoolModel(pl.LightningModule):

    def __init__(self):
        super(CoolModel, self).__init__()
        # not the best model...
        self.l1 = torch.nn.Linear(28 * 28, 10)

    def forward(self, x):
        return torch.relu(self.l1(x.view(x.size(0), -1)))

    def training_step(self, batch, batch_nb):
        # REQUIRED
        x, y = batch
        y_hat = self.forward(x)
        return {'loss': F.cross_entropy(y_hat, y)}

    def validation_step(self, batch, batch_nb):
        # OPTIONAL
        x, y = batch
        y_hat = self.forward(x)
        return {'val_loss': F.cross_entropy(y_hat, y)}

    def validation_end(self, outputs):
        # OPTIONAL
        avg_loss = torch.stack([x['val_loss'] for x in outputs]).mean()
        return {'avg_val_loss': avg_loss}

    def test_step(self, batch, batch_nb):
        # OPTIONAL
        x, y = batch
        y_hat = self.forward(x)
        return {'test_loss': F.cross_entropy(y_hat, y)}

    def test_end(self, outputs):
        # OPTIONAL
        avg_loss = torch.stack([x['test_loss'] for x in outputs]).mean()
        return {'avg_test_loss': avg_loss}

    def configure_optimizers(self):
        # REQUIRED
        return torch.optim.Adam(self.parameters(), lr=0.02)

    @pl.data_loader
    def train_dataloader(self):
        return DataLoader(MNIST(os.getcwd(), train=True, download=True, transform=transforms.ToTensor()), batch_size=32)

    @pl.data_loader
    def val_dataloader(self):
        # OPTIONAL
        # can also return a list of val dataloaders
        return DataLoader(MNIST(os.getcwd(), train=True, download=True, transform=transforms.ToTensor()), batch_size=32)

    @pl.data_loader
    def test_dataloader(self):
        # OPTIONAL
        # can also return a list of test dataloaders
        return DataLoader(MNIST(os.getcwd(), train=False, download=True, transform=transforms.ToTensor()), batch_size=32)

from pytorch_lightning import Trainer

model = CoolModel()
trainer = Trainer()
trainer.fit(model)

Output
Excluding the print from pip install and training tqdm prints
Epoch 1: 100%|██████████| 3750/3750 [00:22<00:00, 163.69batch/s, batch_idx=1874, loss=1.530, v_num=0]
/opt/conda/lib/python3.6/site-packages/pytorch_lightning/callbacks/pt_callbacks.py:314: RuntimeWarning: Can save best model only with val_loss available, skipping.
  ' skipping.', RuntimeWarning)
/opt/conda/lib/python3.6/site-packages/pytorch_lightning/callbacks/pt_callbacks.py:144: RuntimeWarning: Early stopping conditioned on metric `val_loss` which is not available. Available metrics are: loss,avg_val_loss
  RuntimeWarning)
1

Expected behavior
The 'val_loss' metric should be recognized when a dic with key val_loss is returned by validation_step()
Environment
Collecting environment information...
PyTorch version: 1.4.0
Is debug build: No
CUDA used to build PyTorch: None
OS: Debian GNU/Linux 9 (stretch)
GCC version: (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
CMake version: version 3.7.2
Python version: 3.6
Is CUDA available: No
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
Versions of relevant libraries:
[pip] msgpack-numpy==<IP_ADDRESS>
[pip] numpy==1.18.1
[pip] numpydoc==0.9.2
[pip] pytorch-ignite==0.3.0
[pip] pytorch-lightning==0.6.0
[pip] pytorch-pretrained-bert==0.6.2
[pip] pytorch-transformers==1.1.0
[pip] torch==1.4.0
[pip] torchaudio==0.4.0a0+719bcc7
[pip] torchtext==0.5.0
[pip] torchvision==0.4.2
[conda] blas                      1.0                         mkl
[conda] cpuonly                   1.0                           0    pytorch
[conda] mkl                       2019.3                      199
[conda] mkl-service               2.0.2            py36h7b6447c_0
[conda] mkl_fft                   1.0.12           py36ha843d7b_0
[conda] mkl_random                1.0.2            py36hd81dba3_0
[conda] pytorch                   1.4.0               py3.6_cpu_0  [cpuonly]  pytorch
[conda] pytorch-ignite            0.3.0                    pypi_0    pypi
[conda] pytorch-lightning         0.6.0                    pypi_0    pypi
[conda] pytorch-pretrained-bert   0.6.2                    pypi_0    pypi
[conda] pytorch-transformers      1.1.0                    pypi_0    pypi
[conda] torchaudio                0.4.0                      py36    pytorch
[conda] torchtext                 0.5.0                    pypi_0    pypi
[conda] torchvision               0.4.2                    pypi_0    pypi
Additional context
Something interesting is you cannot reproduce this when you run CoolSystem.
You won't get any warning because validation_end returns tensorboard logs with val_loss as a key. If you remove this log you will able to catch the bug.
For what its worth, I was trying to make a PyTorch Lightning based Kernel in Kaggle and WandB as logger. After trying "a lot" to find a mistake in my code, I realized that this warning is not shown every time. Once you run and get the warning and re-run the block (I was using Kaggle Kernel Notebook) it disappears. I assume that it is reading from some cache? Since the val_loss is something for the first epoch(maybe something from last epoch or log) and stay 0 for the rest. I am not familiar with the internal working of PL but I suspect there is some mix up between metrics that are logged and metrics returned by lightning methods.

The EarlyStopping looks for the key val_loss in the validation_end, not validation_step. That's a correct behaviour.
Possible fix 1: Pass in a ModelCheckpoint to Trainer and change the monitor argument to "avg_val_loss" (as returned by validation_end).
Possible fix 2: Change "avg_val_loss" to "val_loss" in validation_end.
@williamFalcon This is a recurring issue that users have with the provided examples. They should probably get updated so that the default EarlyStopping just works from the beginning without producing warnings.

@bsridatta Actually, I just saw that you are linking an old documentation in your post. The examples are already updated. See here, the fix 2 I suggested is already done:
https://github.com/PyTorchLightning/pytorch-lightning/blob/master/pl_examples/basic_examples/lightning_module_template.py

It got fixed here #524

Thanks for the quick response and clarification. I have used the example I mentioned earlier in November and did even doubt that it might have changed. It's clear now, thanks
!


set-b1addressblock

These maybe none issues or fixed in a later version (i havent tried updating i will admit).
Currently running version: <IP_ADDRESS>
I was getting an error trying to update our addressblocks with some new dhcp options, i turned debugging on and stripped out options until i found what was causing the errors.

If i use set-b1addressblock with -name i get the below error. If i remove -name it works. This used to work, so maybe it is a change in how the cmdlet works that you expect, as there is an option for new-name. but I'm not sure what you would use -name for now.
PS C:\Users\TimRogers>                 Set-B1AddressBlock -Subnet $subnet -CIDR $CIDR -Space $DHCPSpace -Name $DNS_Zone -DHCPOptions $DHCPOptions
Set-B1AddressBlock : Parameter set cannot be resolved using the specified named parameters.
At line:1 char:17

...             Set-B1AddressBlock -Subnet $subnet -CIDR $CIDR -Space $DH ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidArgument: (:) [Set-B1AddressBlock], ParameterBindingException
FullyQualifiedErrorId : AmbiguousParameterSet,Set-B1AddressBlock

PS C:\Users\TimRogers>                 Set-B1AddressBlock -Subnet $Subnet -CIDR $CIDR -Space $DHCPSpace -Description $DNS_Zone -DHCPOptions $DHCPOptions -DDNSDomain $DNS_Zone -Tags $AddressBlockTags | out-null
DEBUG: Filter(s):
_filter=name=="Global"
_limit=1000
DEBUG: GET: https://csp.infoblox.com/api/ddi/v1/ipam/ip_space?_filter=name=="Global"&_limit=1000
DEBUG: Filter(s):
_filter=address=="<IP_ADDRESS>" and cidr==20 and space=="ipam/ip_space/832f76cd-d11c-11ec-81e0-4e05a7e4e033"
_inherit=full
_limit=1000
DEBUG: GET: https://csp.infoblox.com/api/ddi/v1/ipam/address_block?_filter=address=="<IP_ADDRESS>" and cidr==20 and space=="ipam/ip_space/832f76cd-d11c-11ec-81e0-4e05a7e4e033"&inherit=full&
limit=1000
The property 'tags' cannot be found on this object. Verify that the property exists and can be set.
At C:\windows\system32\WindowsPowerShell\v1.0\Modules\ibPS\Functions\BloxOne\BloxOneDDI\Set-B1AddressBlock.ps1:121 char:13

        $AddressBlockPatch.tags = $Tags

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : InvalidOperation: (:) [], RuntimeException
FullyQualifiedErrorId : PropertyNotFound

I've worked around these issues now by removing tags and name parameters, but they did used to work.

I'll check this out, there's definitely been changes to align everything to a uniform standard.
-NewName has been introduced across various functions, as previously the -Name parameter for Set- cmdlets was unintuitive. The old -Name used to be used both for selecting an object to update, and to update the name.
Now -Name in Set-/Remove- cmdlets represents the object to update, whereas -NewName would specify the new name if applicable.

From a quick peek at the code, issue #1 is expected.
When using Set-B1AddressBlock you can select the object in three ways;
Subnet;
Set-B1AddressBlock -Subnet 10.x.x.x -CIDR 24
Name;
Set-B1AddressBlock -Name "Address Block Name"
Pipeline;
Get-B1AddressBlock ... | Set-B1AddressBlock
If you're not trying to update the name, just remove the -Name parameter from your command. Otherwise swap it out for -NewName
The error is basically because you're trying to use parameters from more than one parameter set.
For issue #2, this is likely a bug with the way objects are returned from the API. A handful of objects don't return the tags value, if it's null. This causes issues with appending tags on objects without pre-existing tags. I suspect I've broken this when standardising code, so will check and revert if so.

Changed pushed to main v<IP_ADDRESS>

Hey @timrojaz,
Did you manage to verify the fix? 😄

Hi mat. I’m sure it’ll work. I’ll test it when I get chance. Thanks

No problem, just re-open if there's any more issues with it 😄


Apply CODATA/IAU constants

Update CCL constants to most recent literature values.

Pull Request Test Coverage Report for Build 1823

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.565%

Totals

Change from base Build 1734:
0.0%

Covered Lines:
522

Relevant Lines:
552

💛  - Coveralls

Benchmarks have now been updated to meet the change in constants and this PR is ready for review. Travis is currently failing due to an OS X problem that is unrelated to this PR, so I think we should proceed with a review.


Increse otp digits generated by defaults

The OTP library supports just the minimum of up to 9 digits which is what is
documented in https://www.rfc-editor.org/rfc/rfc4226#section-5.3. While this is perfectly fine
for human usage, also RFC defines that this is an example and more digits should be used for
an increased security. As our case is not human-facing, we can bump this to the maximum
size allowed by the hashing algorithm.
Credits to @aleksej-paschenko for discoverying that the gotp lib returned 0 padded strings!

Codecov Report
Base: 33.95% // Head: 33.26% // Decreases project coverage by -0.69% :warning:

Coverage data is based on head (dfc5bf1) compared to base (ad0d66f).
Patch coverage: 39.13% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #33      +/-   ##
==========================================
- Coverage   33.95%   33.26%   -0.70%
==========================================
  Files          26       27       +1
  Lines        2026     2041      +15
==========================================
- Hits          688      679       -9
- Misses       1242     1262      +20
- Partials       96      100       +4

Impacted Files
Coverage Δ

pkg/crypto/otp.go
0.00% <0.00%> (ø)

pkg/utils/strings.go
100.00% <ø> (ø)

pkg/node/options.go
36.79% <58.33%> (-0.10%)
:arrow_down:

pkg/node/connection.go
51.48% <100.00%> (-2.98%)
:arrow_down:

pkg/node/node.go
66.08% <100.00%> (ø)

pkg/services/files.go
76.06% <0.00%> (-6.84%)
:arrow_down:

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

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


Support note for Application ID

Added support note for use of cmd with Application ID @chrisda  FYI

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Remove-UnifiedGroupLinks.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit 19993b7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Remove-UnifiedGroupLinks.md
:white_check_mark:Succeeded
View (exchange-ps)

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

Docs Build status updates of commit eee4401:
:warning: Validation status: warnings
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Is the project maintained?

Hello,
I've checked your project and it either:

Has specs failed with the latest Crystal (0.26.1)
Has no any specs to run so it cannot be validated to be up to date
Has the latest commit dated more than a year ago

Your project is in Awesome Crystal list, so I'd like to ask if the project maintained?
I'll initiate a PR to remove your project from the list if you don't respond positively within 7 days.
I'm sorry if I'm wrong, because I've been checking many shards with automated script.
Thanks,
Vlad.

@vladfaust I would say the answer to that is "no".  It should probably be pointed out in Awesome Crystal that this project is unmaintained as it stands.


the locales are not loading ?

https://i.gyazo.com/f0bbd2f2e390f502713d697ef142f3f1.png

Fixed by clean it up the cache. btw, you should put somewhere to cleanup the cache.
Thanks for nice extension <3


is licode an MCU or an SFU

I have seen that SFU is more scalable than an MCU. even many other MCU based projects have implemented their system as an SFU( intel webrtc) . is licode an MCU or an SFU. if not an SFU ,do we have plans to make it do the functionality of an SFU

Erizo, the WebRTC core of Licode is an SFU. It does have minimal transcoding capabilities that we may expand on at some point but we avoid transcoding whenever we can.
Fun fact: when I last checked, the Intel WebRTC thing included a modified version of Licode

@lodoyun if intel MCU are able to add transcode capabilities for our code,why aren't we not adding it for original licode. What is the reason we are avoiding it right now?

@nikhilrayaprolu an SFU is not "more scalable" than an MCU, depending on what you mean by "more scalable."  The reasons for avoiding it is that it's not a clear win, but a huge tradeoff.

Well, but it works as MCU if the conversation is not a P2P, in that case, when the room is P2P set to true, then the connection is a Mesh?

MCU = have transcoding capabilities.
SFU = only "clone" packets and delivers to every participant.
when a licode room is set to p2p, it uses a mesh config. like you said.
when it's not set in p2p it acts like an sfu

Perfect! Thanks, but why is not possible to use the mute audio/video with P2P communication? I think that is a client decision/characteristic and not a server feature.

Due this, I'd like to know one thing more, using SFU or MCU connections, then the RTCPeerConnection API is not available anymore? or is used but considering the server as a peer? And about the security channel, does the connection still use the DTLS protocol in the session protected layer? Thanks.


Update to node 8

The project already works with node 8, but you need to remove the --harmony-async-await option in package.js. I haven't updated it because npm 5 is a pain with linked packages. I'll update it as soon as this is fixed: https://github.com/npm/npm/issues/17533

Done


Possible new rule for biber

If one wants to run biber with the -tool option, the standard biber rules will fail, because the input file has to be
See https://tex.stackexchange.com/q/528240/4427 with Ulrike Fischer's analysis and my attempt at implementing a different rule.
I'm not sure that arara can distinguish whether among the options there is --tool and, in this case omit getBasename(file) from the command to run. If this is possible, then a new rule would not be needed. Note that the input file for biber is a .bib file, rather than .bcf, in this case when --tool is used.
In the new rule, passing the --tool option should perhaps be implicit.

Thank you very much, Enrico! I propose the following rule:
!config
identifier: biber
name: Biber
authors:
- Marco Daniel
- Paulo Cereda
commands:
- name: The Biber reference management software
  command: >
    @{
        if (tool) {
            return getCommand('biber', '--tool', options);
        }
        else {
            return getCommand('biber', options, getBasename(file));
        }
    }
arguments:
- identifier: tool
  flag: >
    @{
      return isTrue(parameters.tool);
    }
  default: >
    @{
      return false;
    }
- identifier: options
  flag: >
    @{
        if (isList(parameters.options)) {
            return parameters.options;
        }
        else {
            throwError('I was expecting a list of options.');
        }
    }

It seems to work like a charm with the provided MWE. Grazie!

arara v5.0 will feature an extended biber rule as suggested by Paulo. The release is scheduled to happen before the DANTE Frühjahrstagung.


importing CSV doesn't handle string delimiters

I am trying to import CSV files.
I am using commas to separate fields.
when fields have embedded commas, I either quote or double quote the string
when strings are double-quoted, the library strips the trailing double-quote, but does not strip the leading double-quote.
example in:
"hello, this is a test"
yields:
"hello, this is a test
I expected it to drop both double-quotes.
also when importing a CSV, how do I specify other field separators and string delimiters?
for instance, single quotes are not recognized at all.
how would I use pipe '|' as the field separator?

If I understood correctly, you tried a file like quotes.txt
I checked against the web demo at http://oss.sheetjs.com/js-xlsx/ (drag and drop ) and found the correct output:

The node module appears to do the right thing:
$ xlsx quotes.txt
Sheet1
"hello, this is a test"

And to confirm that the object has the correct value:
$ node -pe 'require("xlsx").readFile("quotes.txt").Sheets.Sheet1.A1'
{ t: 's', v: 'hello, this is a test' }

Can you share the exact file you tested with?

I think it might have something to do with white space before/after the field separators.
for example, I always put a space after a comma. that shows up in the JSON.
so my hypothesis is that for this input ’Some Value, “hello, this is a test”'
2 columns are extracted
first column value is ‘Some Value’
and second column value is ‘ “hello, this is a test’
Mark Lummus
<EMAIL_ADDRESS>http://www.marklummus.com
On May 1, 2017, at 3:14 PM, SheetJSDev<EMAIL_ADDRESS>wrote:
If I understood correctly, you tried a file like quotes.txt https://github.com/SheetJS/js-xlsx/files/968561/quotes.txt
I checked against the web demo at http://oss.sheetjs.com/js-xlsx/ http://oss.sheetjs.com/js-xlsx/ (drag and drop ) and found the correct output:
https://cloud.githubusercontent.com/assets/6070939/25590809/eb6a8a7e-2e7f-11e7-9cf3-6168d7aed0ec.png
The node module appears to do the right thing:
$ xlsx quotes.txt
Sheet1
"hello, this is a test"
And to confirm that the object has the correct value:
$ node -pe 'require("xlsx").readFile("quotes.txt").Sheets.Sheet1.A1'
{ t: 's', v: 'hello, this is a test' }
Can you share the exact file you tested with?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/SheetJS/js-xlsx/issues/646#issuecomment-298406573, or mute the thread https://github.com/notifications/unsubscribe-auth/AHccd6jAcdzonyk6fI3mGVnmDSmVcAckks5r1i8XgaJpZM4NNSFT.

@jabbermarky Thanks for reporting! You actually revealed two bugs.

So here's the story: if you put a whitespace before the quote, Excel will actually treat it as 3 cells:

Some Value, "hello, this is a test"
          |       |
Some Value| "hello| this is a test"

If you don't put the whitespace Excel treats it as two cells.
The fix here is to treat any double quote as literal if the cell doesn't start with a double quote

You asked about the pipe delimiter, the answer should be "start your file with sep=| as the first line" (try it in Excel!) but there's a missing else in the code.

So I believe the following diff addresses both issues as well as a related issue when the CSV does not end in a newline:
diff --git a/bits/40_harb.js b/bits/40_harb.js
index b9898a9..d22e2b1 100644
--- a/bits/40_harb.js
+++ b/bits/40_harb.js
@@ -485,29 +485,35 @@ var PRN = (function() {

 		/* known sep */
 		if(str.substr(0,4) == "sep=" && str.charCodeAt(5) == 10) { sep = str.charAt(4); str = str.substr(6); }
-		/* TODO: actually determine the separator */
-		if(str.substr(0,1024).indexOf("\t") == -1) sep = ","; else sep = "\t";
+		else if(str.substr(0,1024).indexOf("\t") == -1) sep = ","; else sep = "\t";
 		var R = 0, C = 0, v = 0;
 		var start = 0, end = 0, sepcc = sep.charCodeAt(0), instr = false, cc=0;
 		str = str.replace(/\r\n/g, "\n");
-		for(;end < str.length;++end) switch((cc=str.charCodeAt(end))) {
-			case 0x22: instr = !instr; break;
-			case sepcc: case 0x0a: if(instr) break;
+		function finish_cell() {
 			var s = str.slice(start, end);
 			var cell = ({}/*:any*/);
 			if(s.charCodeAt(0) == 0x3D) { cell.t = 'n'; cell.f = s.substr(1); }
 			else if(s == "TRUE") { cell.t = 'b'; cell.v = true; }
 			else if(s == "FALSE") { cell.t = 'b'; cell.v = false; }
 			else if(!isNaN(v = parseFloat(s))) { cell.t = 'n'; cell.w = s; cell.v = v; }
-			else { cell.t = 's'; cell.v = s.replace(/^"/,"").replace(/"$/,"").replace(/""/g,'"'); }
+			else {
+				cell.t = 's';
+				if(s.charAt(0) == '"' && s.charAt(s.length - 1) == '"') s = s.slice(1,-1).replace(/""/g,'"');
+				cell.v = s;
+			}
 			if(o.dense) { if(!ws[R]) ws[R] = []; ws[R][C] = cell; }
 			else ws[encode_cell({c:C,r:R})] = cell;
 			start = end+1;
 			if(range.e.c < C) range.e.c = C;
 			if(range.e.r < R) range.e.r = R;
-			if(cc == sepcc) ++C; else { C = 0; ++R; } break;
+			if(cc == sepcc) ++C; else { C = 0; ++R; }
+		}
+		for(;end < str.length;++end) switch((cc=str.charCodeAt(end))) {
+			case 0x22: if(instr || (end - start == 0)) instr = !instr; break;
+			case sepcc: case 0x0a: if(!instr) finish_cell(); break;
 			default: break;
 		}
+		if(end - start > 0) finish_cell();

 		ws['!ref'] = encode_range(range);
 		return ws;

We'll add some more corner cases for the CSV


Bug? Network stealing items before any chests are made..

I was going to do a playtrough where I needed space science to craft the chests to make them endgame, noticed at the start of the playtrough that the network took items from my trash slot before I even made any chests. Not sure if this was inteded or not but figured I would make you aware of this if not.

Nice idea to make these chests require space science!
The "trash before network chests are unlocked" behavior is intentional but probably not desired for that play-style. I've been planning on adding some settings or configuration options to the mod and this seems like a great candidate.
At the very least I'll add a flag to disable network trash until the player builds their first network chest.
I can also add a runtime setting so the player can toggle network trashing on/off as desired.
At the very least, players can now use "Ctrl+Shift+N" to view the contents of the network and it might be nice to just let players pick items out of the network.
What do you think?

Implemented in d19f7532e5ba616fadde34e679cc404de7242d5f with these changes:

Item network logistics are disabled until a network chest has been placed
Players can individually choose to item network logistics depending on play style.

Moning, "At the very least, players can now use "Ctrl+Shift+N" to view the contents of the network and it might be nice to just let players pick items out of the network." Sounds nice. Awesome job with the mod :) Going to update the mod now and start testing the changes done so far :)


<mat-calendar [selected]="selectedDate" [rangeMode]="true">

Hi! it looks like selected doesn`t work for inline view ranged mode trying to put as "selectedDate"
this.selectedDate = {begin: new Date(2019, 11, 4), end: new Date(2019, 11, 30)};
You can try to do the same in DEMO, its not patching range into calendar

Any updates? Have the same issue

I've not investigated this issue. But you can:)

Use beginDate and endDate.
<sat-calendar [rangeMode]="true"                       [beginDate]="inlineRange.begin"                       [endDate]="inlineRange.end"                       (dateRangesChange)="inlineRangeChange($event)"                       (beginDateSelectedChange)="inlineBeginChange($event)">         </sat-calendar>


rewrite ETA

Hi ;
We are using TaskBoard, and are waiting for the rewrite. some bugs aren't resolved because you are rewrite the whole app, I can understand.
Could you provide an ETA for the firs release ?
I cannot see any commit since more than one year ...

Yeah, sorry, I've had a busy year and kind of lost motivation to work on TaskBoard for a while.
I've started working on updating it to the latest version of Angular, in preparation for getting back into it.
That said, the rewrite branch is quite usable as-is. The only missing items right now are emails and attachments.

Hi ;
the install instructions seems to be outdated.
no .htaccess, neither ./build/composer.phar, neither composer.json / composer.lock file .
so all I have is a white screen with "TaskBoard is Loading..."
any advice ?

Yeah, the README of the re-write branch is written as if it's been released, but that hasn't happened yet. Just clone the repo, check out the re-write branch, run npm i then npm run build:prod. At that point there will be a dist directory ready to be served by your server.

npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/event-stream/-/event-stream-3.3.6.tgz
npm ERR! 404
npm ERR! 404<EMAIL_ADDRESS>is not in the npm registry.

Try running npm update before npm i.

seems to work fine ... thanks !
so for those who want to test :
npm update
npm i
npm run build:prod

then point your php webserver to the dist folder.


Added nonce to msal node

Added nonce feature.

Codecov Report

Merging #4443 (d52e9f0) into dev (b64e4d4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.39% <ø> (ø)

Carriedforward from b64e4d4

msal-browser
87.50% <ø> (ø)

Carriedforward from b64e4d4

msal-common
85.37% <ø> (ø)

Carriedforward from b64e4d4

msal-core
82.65% <ø> (ø)

Carriedforward from b64e4d4

msal-node
80.92% <100.00%> (+0.20%)
:arrow_up:

msal-node-extensions
76.03% <ø> (ø)

Carriedforward from b64e4d4

msal-react
94.27% <ø> (ø)

Carriedforward from b64e4d4

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files
Coverage Δ

lib/msal-node/src/client/ClientApplication.ts
69.52% <100.00%> (ø)

lib/msal-node/src/index.ts
100.00% <0.00%> (ø)

...de/src/cache/distributed/DistributedCachePlugin.ts
88.23% <0.00%> (ø)


Asteriks class for TextField

FormControlClassKey does not include the ability to add a class to the asterisk set by the required prop of a TextField.
I would like to be able to do:
<TextField classes={{asterisk: 'asterisk'}} />

...

'.asterisk': { color: 'blue' }

Current situation has no ability to do this and in my styling I need to do this:
'> label > span': { color: 'blue' }

@e11en Here you go:
import React from "react";
import PropTypes from "prop-types";
import { withStyles } from "@material-ui/core/styles";
import TextField from "@material-ui/core/TextField";

const styles = theme => ({
  asterisk: {
    color: "red"
  }
});

function TextFields(props) {
  const { classes } = props;

  return (
    <form noValidate autoComplete="off">
      <TextField
        required
        InputLabelProps={{
          FormLabelClasses: {
            asterisk: classes.asterisk
          }
        }}
        id="standard-required"
        label="Required"
        defaultValue="Hello World"
        margin="normal"
      />
    </form>
  );
}

TextFields.propTypes = {
  classes: PropTypes.object.isRequired
};

export default withStyles(styles)(TextFields);

https://codesandbox.io/s/kxvv9w085r

But yeah, we could add  an asterisk style rule to the InputLabel component so you can do:
      <TextField
        required
        InputLabelProps={{
          classes: {
            asterisk: classes.asterisk
          }
        }}
        id="standard-required"
        label="Required"
        defaultValue="Hello World"
        margin="normal"
      />

instead. Do you want to work on it? :)

Ah yeah of course! I didn't think of that way of adding the class name. Thanks! I'll have a look at adding that style rule

FormLabel docs are also missing a comment about the asterisk class. It is listed as a class key but not explanation about the purpose: https://material-ui.com/api/form-label/#css

@e11en @oliviertassinari  If anyone is not working on this, would love to make PR. Thanks

@umairfarooq44 We would be honored to review your pull request :)

@oliviertassinari thanks for assigning issue. The solution should be in InputLabel component remove FormLabelClasses and use only classes from props. As FormLabelClasses and classes are two props for same purpose. Also asterisk should be included in documentation of InputLabel css API.


Better Handling when an Observation stopped at server side because of error.

The issue is better described at : https://github.com/eclipse/californium/issues/1754
Fix are done at Californium side but for now in leshan there no notification about this.

We should keep this issue in mind when refactoring will be done for #1025.


letter spacing values

is this a bug?:
https://github.com/tachyons-css/tachyons/blob/master/src/_letter-spacing.css#L12
:confused:

Thanks for creating an issue. The value is intentional, though @mrmrs has now adjusted it a bit.


Automate Helm Upgrades

Would it be possible to instead of watching images for updates, watching helm charts for new versions? The flux.weave.works/automated: "true" annotation could be set on a HelmRelease CR.

It certainly is possible, and desirable. The difficulties are more architectural than technical: at present, fluxd only "knows" about manifests and images, and adding Helm charts to that seems headed in the direction of being monolithic.
I would like to move the image database and automation into its own process/container/image; if that can be done, scanning and updating chart versions is a close analogue and could be done in the similar way. But I'm not set against adding this into fluxd and moving it out later -- just conflicted about it :-/
(BTW you already can use the annotation flux/weave.works/automated in a HelmRelease -- it controls image updates. We'd have to use another annotation, or have some other way to distinguish resources for which you want the chart version automated.)

Thanks for the fast reply! Cool, so this is something you're planning / working on 👍 Maybe I find the time to code together a first draft so we have something to reason about.
Merry x-mas

I was just thinking about this and turns out there was already an issue about this!
One reason I think upgrading helm charts instead of images when using helm for each application is because:

allows developers to install individual helm charts at specific versions themselves if needed
Still keeps the deployment manifests in the same location as the application code
Can fully automate pushes to a dev cluster

The problem with using git-based helm chart updating is you either have to put the charts in one repository or have a bunch of separate repositories that flux is pulling from, which seems like an antipattern in the gitops paradigm. Alternatively one could use a monorepo, but I think that gets messy for other reasons.

Looking for the same.
My suggestion would be:

no version specified -> use the latest available version
allow specifying a wildcard: 1.1., or 1.
specify an exact version - as today

One reason I think upgrading helm charts instead of images when using helm for each application is because [...]

I fully expect people to want to update the chart version as well as the images used with the chart. Chart versions will tend to change when new images are not backward-compatible; but also on their own terms, e.g., if there's some bit of extra operational config you can supply to the chart. You don't always need or want to bump the chart version with each new image.

My suggestion would be:

no version specified -> use the latest available version
allow specifying a wildcard: 1.1.*, or 1.*
specify an exact version - as today

I think the above is implicitly suggesting that these values can be used in the version field of a HelmRelease -- I don't think that's a good idea though, since we generally want to record the state as precisely as we can, and put the policy for automation elsewhere. Otherwise there'd be no record in git of the change when the chart is upgraded to a new version.

Given that the values.yaml manifests typically change from one version of a helm chart to the next, is it even plausible that this is something that can be automated ?

Couldn't you say the same thing about docker images and env variables

Couldn't you say the same thing about docker images and env variables

Fair point, it's up to developers I suppose to decide on the appropriateness of the automation.

I think the above is implicitly suggesting that these values can be used in the version field of a HelmRelease -- I don't think that's a good idea though, since we generally want to record the state as precisely as we can, and put the policy for automation elsewhere. Otherwise there'd be no record in git of the change when the chart is upgraded to a new version

@squaremo
This can still be done if the version schema is in an annotation and then the current version is in the version field of the helm release crd


GPU requirement and auto-detect NVIDIA extensions

This PR builds upon the proposal in https://github.com/devcontainers/spec/issues/82
When launching a devcontainer requesting GPUs it also detects the presence of NVIDIA docker extensions, and adds the --gpus all flag.
This means that the same devcontainer can be used by developments that don't all have GPU's.
It also does the equivalent for docker-compose scripts, adding the GPU access to the service specified in the devcontainer.json file

I've taken a bit of a closer look, and rebased onto the latest main - and I see the same failures running the tests locally, so I don't think this is a regression from this PR.

@chris-major-improbable I have merged in the main branch and fixed an issue with the PR I found when looking into the test failures. Please check the above comment from the CLA bot, we must have added that after you submitted the PR. Thanks!

@microsoft-github-policy-service agree [company="{Improbable Worlds Ltd}"]

@microsoft-github-policy-service agree company="{Improbable Worlds Ltd}"

@microsoft-github-policy-service agree company="Improbable Worlds Ltd"


stanmake script

@akucukelbir, is script solid enough to distribute publicly? If so, let's get it into the repo. We would need:

documentation
install instructions
when and where it works / doesn't work

If you think it's not going to hold up, close this issue with a comment.

@syclik it depends on whether you want to ship an installer with cmdstan.
the script is only useful if it lives somewhere on your path (e.g. /usr/local/bin) or in your RC file (e.g. .zshrc).
i suspect we'd need some sort of installer to copy stanmake and make it point to wherever cmdstan lives.

I'm not opposed to having a cmdstan installer.
It sounds like the script isn't useful as-is.

yeah, i think this will require too much hassle on our end to get working "straight out of the box".


build fails

> git clone https://github.com/sekrit-twc/zimg
> cd zimg
> ./autogen.sh
> ./configure --disable-static
> make
make[1]: Entering directory '/home/g/git-pkges/media/zimg'
  CXX      dummy.lo
make[1]: *** No rule to make target 'graphengine/graphengine/cpuinfo.cpp', needed by 'graphengine/graphengine/libzimg_internal_la-cpuinfo.lo'.  Stop.
make[1]: Leaving directory '/home/g/git-pkges/media/zimg'
make: *** [Makefile:4172: all-recursive] Error 1

i get same error, commented out all lines with zimg and also removed zimg from ffmpeg build options. now it is still building, just without zimg support lol

EDIT: actually i just changed to the v3.0 branch and it seems to get past this error. we will see if the build completes though lol

same here,
git clone - build fails, downloading manually version 3.0.x and building that one succeeds.

This is due to the two issues mentioned in this commit.
https://github.com/sekrit-twc/zimg/commit/e4acb3a29d7c45faca902b490a1585fe48063970

Can confirm, current git master is broken, v3.0.4 release works.

libbluray is having issues as well, atleast for me it is. i had to clone it manually into the correct folder, then edit the scripts lines for git checkout, and then it built great. this helper script can actually help cross compile a plethora of stuff. i been compiling for 2 days straight and aint but about half way through lol

don't know about this "zimg" , but with the "3.0.4" release instead of the git-clone , which at the moment fails with the above
make[1]: *** No rule to make target 'graphengine/graphengine/cpuinfo.cpp', needed by 'graphengine/graphengine/libzimg_internal_la-cpuinfo.lo'.  Stop. make[1]: Leaving directory '/home/g/git-pkges/media/zimg' make: *** [Makefile:4172: all-recursive] Error 1
the mpv-build completes, but results in error with amdgpu graphics when playing back video files
Error opening/initializing the selected video_out (--vo) device.
don't know if zimg and mpv-build are affiliated in any way, so please forgive if am posted this here and was not supposed to be.

'/home/rich/Programs/zimg' make[1]: *** No rule to make target 'graphengine/graphengine/cpuinfo.cpp', needed by 'graphengine/graphengine/libzimg_internal_la-cpuinfo.lo'.  Stop. make[1]: *** Waiting for unfinished jobs....   CXX      dummy.lo make[1]: Leaving directory '/home/rich/Programs/zimg'

don't know about this "zimg" , but with the "3.0.4" release instead of the git-clone , which at the moment fails with the above make[1]: *** No rule to make target 'graphengine/graphengine/cpuinfo.cpp', needed by 'graphengine/graphengine/libzimg_internal_la-cpuinfo.lo'. Stop. make[1]: Leaving directory '/home/g/git-pkges/media/zimg' make: *** [Makefile:4172: all-recursive] Error 1
the mpv-build completes, but results in error with amdgpu graphics when playing back video files
Error opening/initializing the selected video_out (--vo) device.
don't know if zimg and mpv-build are affiliated in any way, so please forgive if am posted this here and was not supposed to be.

Are you trying to build with the ffmpeg-windows-build-helpers script? if so, i was just looking through the cross_compile_ffmpeg.sh script, and i seen something that might benefit you somehow? not sure though. but ctrl+f this (if youre using the helper script mentioned above in this comment):
if [[ $build_amd_amf = y ]]; then
    build_amd_amf_headers

maybe somehow you built with the wrong option/flag somehow? maybe when you run the cross_compile_ffmpeg.sh script, you need to specify --build_amd_amf=y option. Hopefully this helps you somehow.
Good luck though!

@james28909 thank you, but i am still using Linux, don§t know if those apply.
Tried today again, on a differnt computer, same result:

zimg via git clone https://github.com/sekrit-twc/zimg.git fails to build with  CXX      dummy.lo make[1]: *** No rule to make target 'graphengine/graphengine/cpuinfo.cpp', needed by 'graphengine/graphengine/libzimg_internal_la-cpuinfo.lo'.  Stop. make[1]: Leaving directory '/home/rich/Downloads/zimg' Makefile:4028: recipe for target 'all-recursive' failed make: *** [all-recursive] Error 1
building zimg via 3.0.4.zip succeeds
however, resulting mpv with vapoursynth would no longer play?   Error opening/initializing the selected video_out (--vo) device. Video: no video

i am using AMD graphics driver on Ubuntu 18.04 Bionic this time.

@mrkapqa You need to init the new submodule.
git clone https://github.com/sekrit-twc/zimg.git
cd zimg
git submodule update --init --recursive

Then autogen and build it like before.

@Nomis101, thanks, it would build with these instructions, but resulting mpv player still shows
Error opening/initializing the selected video_out (--vo) device.
i tried now with both Nvidia and AMD Graphics card, so i am at odds, it seems no graphics cards driver issue.


'make html' in doc/ fails with lexer.DtsLexer import error

Error message:
Exception occurred:
  File "conf.py", line 128, in <module>
    from lexer.DtsLexer import DtsLexer
ImportError: No module named lexer.DtsLexer

CC @carlescufi @b0661
Please assign this to me. I have a fix ready.


undefined: I2C_SLAVE

Trying to go get this and ... no dice.
go get -u github.com/d2r2/go-i2c
# github.com/d2r2/go-i2c
../../../../pkg/mod/github.com/d2r2/go-i2c@v0.0.0-20191123181816-73a8a799d6bc/i2c.go:36:26: undefined: I2C_SLAVE

Hi @heatxsink. Could you specify what OS you employ? Be aware, that this library works only with Linux systems. No Windows, no FreeBSD...

got the same error, running on macOS (darwin)

@lukibahr , I'm sorry, macOS (FreeBSD, Windows as well) is not supported. And if you try to compile this code directly on macOS, I assume this is mistake and you should use cross compilation approach, because you don't have intention to run this code on macOS, right?

okay, thank you for your hint. Cross compiling the code for UNIX will be enough for me.

@lukibahr, I will recommend additionally take a look at these comments, where, as I remember we have discussed same aspect: https://github.com/d2r2/go-dht/issues/11#issuecomment-472924091


Help Center - Reach Out Bloc𝙺chain Support Number

How can I recover my Bloc𝙺chain account? Bloc𝙺chain support number +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ If you need to recover your Bloc𝙺chain account, you can contact their customer service hotline at +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ or +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA for assistance in the recovery process.
How to contact Bloc𝙺chain Manager? You can access live chat by clicking the "Contact Us" ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ button in the help center and then selecting "Chat." Phone: Bloc𝙺chain also offers phone support for urgent issues. You can call Bloc𝙺chain at ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ for Quick response & Support. There are in brief ways to talk to someone on Bloc𝙺chain.
Does Bloc𝙺chain have a live chat? Yes, Bloc𝙺chain does offer a live chat option for customer support. To access the live chat feature:
Go to Bloc𝙺chain's Help Center on the platform.
Look for the "Contact Us" ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ option.If you are trying to reach Bloc𝙺chain Support +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ (24/7), the only Bloc𝙺chain phone number is ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ (24/7) or +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA for Quick response & Support. You can also access live chat by clicking the "Contact Us" button in the help center and then selecting "Chat."
Choose the chat option to speak with a live person.
You can also contact Bloc𝙺chain support at ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ (Talk with Human) & (☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ (Contact-USA)).
Follow the prompts to describe your issue and get assistance.
What number is ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲? The number ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ does not appear to be associated with Bloc𝙺chain customer service. The correct Bloc𝙺chain Customer Service Number for the USA is ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲. Always double-check the phone number before dialing to avoid any potential scams.
To Access the Bloc𝙺chain Help Center via (☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲- Contact-USA)) or ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ and log in to your account first. By clicking to the "Get Support" or locate the option title "Chat with a representative." Opt for the (☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲- Contact-USA)) or ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ option to facilitate direct assistance.
Navigating Security Concerns:
If your account was disabled and you've already tried to recover it, try writing to<EMAIL_ADDRESS>If your account was hacked, you can recover it at https://www.Bloc𝙺chain Support.com/hacked.
If your personal account was disabled: https://www.Bloc𝙺chain Support.com/help/contact/logout?id=260727303972907
If your advertising account was disabled: https://www.Bloc𝙺chain Support.com/help/contact/391273094929792
If you lost access to your Bloc𝙺chain Support Page: https://www.Bloc𝙺chain Support.com/help/contact/1280439701975125.
<EMAIL_ADDRESS>This is for when your account is suspended or request for blocked or removed content find this mail well.
<EMAIL_ADDRESS>If account is disabled or you forget your passwords so you can use this mail for getting accesss again.
<EMAIL_ADDRESS>If you posted something that doesnt meet Community guidlines cause to blocked your account , use this one mail id.
platf...@support. Bloc𝙺chainSupport.com: If financial issues this happening to your account so you can contact with this mail, or simmply dial these numbers to getting betters resolved "☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA & +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA".
<EMAIL_ADDRESS>If you lost your data this is for good enough for you to solved your problems.
rec...@Bloc𝙺chain Support.com or<EMAIL_ADDRESS>If you wanna get helps to the Bloc𝙺chain Support law Enforcement support team find this mails id well or better option for to diel this number "☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA & +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USAUSA "for saving your time.
<EMAIL_ADDRESS>If advertising problem on Bloc𝙺chain Support so this is good for you.
<EMAIL_ADDRESS>Only for press or PR services activities.
<EMAIL_ADDRESS>If the problems related to posts and message that should be sound like phishing for information, or you can simmply dial these numbers "
1.☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA & +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USAUSA " for saving your time. and this is woek 24/7.
+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲
Key Responsibilities of Bloc𝙺chain Support Support:
Bloc𝙺chain Support Support { Dial the official contact numbers - ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA & +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USAis committed to helping users regain access to their accounts by guiding them through the account verification process.
Bloc𝙺chain Support Account:
Visit the Bloc𝙺chain Support Help Center: {Dial the official contact numbers - ☎+̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA & +̲1̲-̲(̲7̲𝘖̲7̲)̲-̲6̲𝘖̲7̲-̲6̲𝘖̲2̲8̲ USA
Open a web browser and navigate to the Bloc𝙺chain Support Help Center.
Steps Involved for healps:-
Explore available resources and FAQs that might address your specific account recovery concerns.
1.Use the Account Recovery Form:
2Access the Account Recovery Form.

Closing this issue as spam.


A lot of error messages on welcome page

There are a lot of error messages shown on the welcome page if a user doesn't have permissions to read Dashboards, Charts, Saved Queries or Recent activity. This happens In case of redirect from a forbidden page as well. This doesn't look well when a user sees an error messages stack at the time.
How to reproduce the bug

Create a role without permissions to read Dashboards, Charts, Saved Queries, Recent Activity
Set this role only to a user
Login as this user
There are a lot of error messages on welcome page

Expected results
Don't show a stack of messages to the user as it doesn't look well
Actual results
There are a lot of messages
Screenshots

Environment
(please complete the following information):

browser type and version: Chrome 100.0.4896.127 and Microsoft Edge 101.0.1210.32
superset version: The latest version built from sources on master branch
python version: 3.8.10
node.js version: 16.9.1
any feature flags active: Checked on default superset from master branch

Hey there, moving to Discussions so people can chime on how to make the experience better! But this isn't a "bug" per say in the strict sense unfortunately because it's working as intended ;)


Use require.resolve rather than requiring modules directly

Webpack serializes these anyway, so it is simpler to pass the paths directly.

!merge


Matlab 3D output not as good as online demo

I implemented your repo using MATLAB.
Given the same input image on

MATLAB
Online Demo

But the online results are far superior than MATLAB results.
Can you please guide me to get the same quality as online demo using MATLAB
Thanks

Can you take a screenshot of what MATLAB shows you?


run Katello rubocop without Foreman checkout

Foreman limits the rubocop version, so we we end up using the one
defined by Foreman, not by Katello, but Katello's rubocop_todo is not
compatible with that.

Katello updated to tfm-rubocop 0.1.x in https://github.com/Katello/katello/pull/11009
But Foreman didn't (yet): https://github.com/theforeman/foreman/pull/9926
Now running rubocop from the Foreman checkout for Katello fails with:
[2024-11-12T09:10:12.565Z] Error: unrecognized cop Lint/ConstantDefinitionInBlock found in /home/jenkins/workspace/katello-master-source-release/.rubocop_todo.yml, unrecognized cop Lint/UselessMethodDefinition found in /home/jenkins/workspace/katello-master-source-release/.rubocop_todo.yml, unrecognized cop Style/CombinableLoops found in /home/jenkins/workspace/katello-master-source-release/.rubocop_todo.yml

This change (untested) avoids that, by replicating the setup we have on GHA, where rubocop runs exclusively on the Project (= Katello) checkout, w/o Foreman.


[WIP] AVO Progressive Delivery

WIP
This adds a template for app-interface to consume over here:
https://gitlab.cee.redhat.com/service/app-interface/-/merge_requests/58155

Codecov Report

Merging #118 (a962d8f) into main (0a21214) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #118   +/-   ##
=======================================
  Coverage   42.85%   42.85%
=======================================
  Files          28       28
  Lines        1505     1505
=======================================
  Hits          645      645
  Misses        784      784
  Partials       76       76

/lgtm
/approve


Period less than 1 day. Is it possible?

I want pull live data for the US market for last 30 minutes of 1 minute interval. But I am not able to set the period anything lesser than 1d. Is there any way to solve this?

Well, the API is basically scrapping Yahoo! Finance historical data (the same one you can see on Y!F website). Afaik, the max granularity available is therefore 1 day (https://finance.yahoo.com/quote/^GSPC/history?p=^GSPC for example)... If you have an idea on how to pull the data from the graphs (granularity up to 1 min), I'd be more than happy to help !

Check the readme file:
    # fetch data by interval (including intraday if period < 60 days)
    # valid intervals: 1m,2m,5m,15m,30m,60m,90m,1h,1d,5d,1wk,1mo,3mo
    # (optional, default is '1d')
    interval = "1m",

@rodrigobercini
Is there any way we can add the interval on the DataFrame Date output too?  I'm trying to plot intraday values but they all plot in a vertical line, since matplotlib reads all the values as the same Date.

Can you provide code to reproduce the error ? The index of the Dataframe is standard Datetime format (from biggest to lowest unit and timzone, 2020-04-02 09:31:00-04:00 for instance), there shouldn't be "interval to set"

just tried on ASX stocks. does not return intraday prices on ASX stocks or MSFT, TSLA.
import yfinance as yf
#ticker = yf.Ticker("BHP.AX")
#ticker = yf.Ticker("MSFT")
ticker = yf.Ticker("TSLA")
ticker.history(start="2020-06-01", end="2020-06-05", period="60m")
I'm guessing yahoo tightened up the criteria.


tex-math: map attributes

gsm217 has a custom icon hack that is presented as specific-use. We need to call mapAttributes() to make it available downstream.

We might want to do this on formula elements instead (and ask texml to expose it there).

As per F2F, we already hack something for has-qed-box so we obvious tolerate it on tex-math. Maybe that clean up and handle this issue while we're at it.

I think I still prefer to move things to the container; mapAttributes does this nicely already.


Tuple destructuring is broken with new tuple syntax

Description:
Previous syntax that works:
public function main(string... args) {
    var (a, (b, c)) = ("Ballerina", (123, true));
}

New syntax:
public function main(string... args) {
    var [a, [b, c]] = ["Ballerina", [123, true]];
}

get the below errors:
error: .::testMain.bal:3:13: invalid tuple variable; expecting a tuple type but found 'int|boolean[]' in type definition

According to the spec [1], "If there is no contextually expected type, then the inherent type will be T[], where T is the union of the broad types of the expressions in expr-list. It is an compile-time error if expr-list is empty and there is no contextually expected type." Since, this is defined with var, and no contextually expected type, [123, true] will get inferred as (int|boolean)[], and therefore ["Ballerina", [123, true]] as (string|(int|boolean)[])[]. Therefore, I think we cannot support the above.
@hasithaa, @sameerajayasoma WDYT?
[1] https://github.com/ballerina-platform/ballerina-spec/blob/master/lang/spec.html#L2712

Yes, above doesn't work. The compiler can't infer the type from, the LHS, So it tries to infer the RHS expression type from the array literal. Best it can do is, infer the RHS side type as (string|(int|boolean)[])[].
Anyway, if either side expressions have the contextual type, this problem won't occur. Anyway, it is weird to see someone writes something like this.
var [a, [b, c]] = ["Ballerina", [123, true]];

I have a different opinion :)
Here LHS is a tuple destructure, and hence the context sort of knows  the expected type/shape. Even now, it seems the compiler have correctly identified [a, [b, c]] as a tuple and tried to infer the type of its members: a and [b, c].
I think what the spec refers to as "no contextually typed" is scenarios such as var x = ["Ballerina", [123, true]], any x = ["Ballerina", [123, true]], etc.

@SupunS , Shall we move this discussion to spec repo?


update readme

update readme based on info in https://groups.google.com/g/skupper/c/rZwIQyhZ2Lk/m/mAS7q3NiBAAJ

closing as not required


Added reference to SQS exporter

I have created a simple AWS SQS exporter for tracking queue information.
https://github.com/jmal98/sqs_exporter

Thanks!
The attribute name should go in the metric name, and the queue name in a label.

@brian-brazil I have updated the format per your comment.
Thanks for the feedback.


custom streaming video not detected in html

Checklist

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

Verbose log

[generic] ?inf_contact_key=1&utm_source=ActiveCampaign&utm_medium=email&utm_content=Welcome to Betrayal: The Autoimmune Solution, Day 1!&utm_campaign=BET19: Requesting header
[generic] ?inf_contact_key=1&utm_source=ActiveCampaign&utm_medium=email&utm_content=Welcome to Betrayal: The Autoimmune Solution, Day 1!&utm_campaign=BET19: Downloading webpage
[generic] ?inf_contact_key=1&utm_source=ActiveCampaign&utm_medium=email&utm_content=Welcome to Betrayal: The Autoimmune Solution, Day 1!&utm_campaign=BET19: Extracting information
[download] Downloading playlist: Betrayal Episode 1 - Betrayal: The Autoimmune Disease Solution They're Not Telling You
[generic] playlist Betrayal Episode 1 - Betrayal: The Autoimmune Disease Solution They're Not Telling You: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on 'https://hto-content.s3.amazonaws.com/user_videos/bcbae2a6-b94e-4aa4-9a22-065d3fd3b2e6/audio/mp3/128/bcbae2a6-b94e-4aa4-9a22-065d3fd3b2e6.mp3'
[download] Destination: Betrayal Episode 1 - Betrayal - The Autoimmune Disease Solution They're Not Telling You-inf_contact_key=1&utm_source=ActiveCampaign&utm_medium=email&utm_content=Welcome to Betrayal - The Autoimmune Solution, Day 1!&utm_campaign=BET19.mp3

[download]   0.0% of 87.64MiB at  3.82KiB/s ETA 06:31:41
[download]   0.0% of 87.64MiB at 11.46KiB/s ETA 02:10:33
[download]   0.0% of 87.64MiB at 26.73KiB/s ETA 55:57
[download]   0.0% of 87.64MiB at 57.28KiB/s ETA 26:06
[download]   0.0% of 87.64MiB at 59.31KiB/s ETA 25:12
[download]   0.1% of 87.64MiB at 80.38KiB/s ETA 18:35
[download]   0.1% of 87.64MiB at 121.46KiB/s ETA 12:17
[download]   0.3% of 87.64MiB at 158.67KiB/s ETA 09:24
[download]   0.5% of 87.64MiB at 226.65KiB/s ETA 06:33
[download]   1.0% of 87.64MiB at 292.49KiB/s ETA 05:03
[download]   1.5% of 87.64MiB at 344.01KiB/s ETA 04:16
[download]   2.1% of 87.64MiB at 379.45KiB/s ETA 03:51
[download]   2.7% of 87.64MiB at 397.74KiB/s ETA 03:39
[download]   3.2% of 87.64MiB at 409.65KiB/s ETA 03:32
[download]   3.8% of 87.64MiB at 427.74KiB/s ETA 03:21
[download]   4.4% of 87.64MiB at 442.83KiB/s ETA 03:13
[download]   5.0% of 87.64MiB at 464.48KiB/s ETA 03:03
[download]   5.8% of 87.64MiB at 499.33KiB/s ETA 02:49
[download]   6.9% of 87.64MiB at 549.26KiB/s ETA 02:32
[download]   8.2% of 87.64MiB at 625.73KiB/s ETA 02:11
[download]  10.6% of 87.64MiB at 740.74KiB/s ETA 01:48
[download]  12.9% of 87.64MiB at 847.83KiB/s ETA 01:32
[download]  15.8% of 87.64MiB at 982.68KiB/s ETA 01:16
[download]  19.5% of 87.64MiB at  1.12MiB/s ETA 01:02
[download]  24.1% of 87.64MiB at  1.32MiB/s ETA 00:50
[download]  28.6% of 87.64MiB at  1.49MiB/s ETA 00:41
[download]  33.2% of 87.64MiB at  1.66MiB/s ETA 00:35
[download]  37.8% of 87.64MiB at  1.83MiB/s ETA 00:29
[download]  42.3% of 87.64MiB at  2.00MiB/s ETA 00:25
[download]  46.9% of 87.64MiB at  2.15MiB/s ETA 00:21
[download]  51.4% of 87.64MiB at  2.26MiB/s ETA 00:18
[download]  56.0% of 87.64MiB at  2.43MiB/s ETA 00:15
[download]  60.6% of 87.64MiB at  2.56MiB/s ETA 00:13
[download]  65.1% of 87.64MiB at  2.69MiB/s ETA 00:11
[download]  69.7% of 87.64MiB at  2.81MiB/s ETA 00:09
[download]  74.3% of 87.64MiB at  2.95MiB/s ETA 00:07
[download]  78.8% of 87.64MiB at  3.06MiB/s ETA 00:06
[download]  83.4% of 87.64MiB at  3.17MiB/s ETA 00:04
[download]  88.0% of 87.64MiB at  3.27MiB/s ETA 00:03
[download]  92.5% of 87.64MiB at  3.40MiB/s ETA 00:01
[download]  97.1% of 87.64MiB at  3.49MiB/s ETA 00:00
[download] 100.0% of 87.64MiB at  3.41MiB/s ETA 00:00
[download] 100% of 87.64MiB in 00:27
[download] Finished downloading playlist: Betrayal Episode 1 - Betrayal: The Autoimmune Disease Solution They're Not Telling You

Description

It detects the mp3 audio version available on the site and downloads that, but I want the video to be downloaded which it doesn't detect. Running -F it only lists the mp3 :(. On the html response there is clearly a video tag (below). It uses video.js to play it.
<video id="video-topic-full" class="video-js vjs-default-skin vjs-big-play-centered" poster="https://stream.healthmeans.com/cover_photos/57330a51-83f4-45c0-a3b6-b49433d4d69d.jpg" data-setup='{"controls": true, "autoplay": false, "preload": "auto", "playbackRates": [0.5, 1, 1.5, 2] }' controlsList="nodownload">
<script>
											var player = videojs('video-topic-full');
											player.playlist([{
												sources: [{
													src: 'https://player.vimeo.com/external/321575555.m3u8?s=7681cb66337778450ec269acdcaa9c594390148a',
													type: 'application/x-mpegURL'
												  }],
												  poster: 'https://stream.healthmeans.com/cover_photos/57330a51-83f4-45c0-a3b6-b49433d4d69d.jpg'
												},

												{
												  sources: [{
													src: 'https://player.vimeo.com/external/343467701.m3u8?s=e0ab34f7edd8370bf96752cb9ebf5c179d8af7e1',
													type: 'application/x-mpegURL'
												  }],
												  poster: 'https://stream.healthmeans.com/cover_photos/57330a51-83f4-45c0-a3b6-b49433d4d69d.jpg'
												}]);

											// Play through the playlist automatically.
											player.playlist.autoadvance(0);
										</script>

Incomplete how?

post verbose log and original URL used.


Make Groupings for PDFs coming from the same domain.

Right now all online PDFs are listed in one giant list. It would make it easier for the user to find the PDF they are trying to find if each PDF was grouped based on the domain they where downloaded from. It would end up looking like a folder system. It would also be good for this to be a toggle so the user can decide which view they prefer.
Example:
If you have pdfs:
PDF1 from  google.com
PDF2 from google.com
PDF3 from moodle.com
PDF4 from learning.com
It would end up looking like:
Google.com
PDF1
PDF2
PDF3
PDF4

Hi @dborg291 @alexweininger.
I can contribute and implement this. Please assign this issue to me and I will do it over the weekend.

@MagicCurlyHair I would like this to be a setting that the users can toggle.

@alexweininger Yeah, there's already a settings page implemented, so I will just add another option. I could do it as a simple boolean switch, but I think dropdown is better since more options for sorting could be implemented in the future (maybe users will be able to create and name folders themselves?)
Let me know what you think.

@MagicCurlyHair I think that's a great idea.

@alexweininger Cool, then I think it will be a 'File grouping' option with the following options:

'Ungrouped' - current view
'Group by domain' - the proposed grouping for domains.
Not sure about the naming though.

@MagicCurlyHair sounds great!


fix recipes

tell me, how to reach on that muluna without discover any technology needed to travel in space. I cannot start game on the moon and i cannot  research any nnecessary technology to travel in space without getting there. change that its problematic

I'm not sure what you mean. On the latest version of the mod, starting with rocket silo researched, I was able to reach Muluna without cheating in any technologies.

i had turned off all mods. only pack ,,all planets,, was installed and even there was problem with research crushers and recipes on them and thrusters. You need space science packs to unlock them or stand on Muluna and without space technology its imposible to unlock

Can you send me your save where you're having this issue?

Also, if you aren't aware, you can get to Muluna without crushers or asteroid crushers if you send barrels of water, iron ore, and rocket fuel up to your ship.

A ship like this one is capable of getting to Muluna.

I never think about sending any of that resources in this stage of game, propably im noob. I will try that later.
But the mods like ,,planet picker,, in this solution is much much harder to play but not imposible, maby u would make similar moons for the rest of basic planets. Its good mod and give some moore oportunity  on start.
Thanks very much for advice, you change my mind in this problem, in next sesion in game i would try that.

Yeah, I'm aware that Planet picker would be difficult to make compatible with this mod without moving Muluna to wherever the starting planet is. I think this is more a job for the creator of Planet Picker to deal with. You're welcome to ask them to make their mod more compatible with mine.
I'm happy that I was able to help you get through the initial hurdle of this mod. Hope you enjoy it. I've updated this mod's description to make the early game's expectations clearer.


Problems parsing vms exported from KRATOS ESCAPE

Contact Details
<EMAIL_ADDRESS>What happened?
Have a vms file with multiples samples and multiple spectra measurements exported from KRATOS ESCAPE.
TaAlO-newTandSseries.zip
The file can be opened in CASA XPS just fine.
Tried to parse it with dataconverter TaAlO-newTandSseries.vms eln_data_kratos.yaml --reader xps --nxdl NXmpes --output output.vms.nxs. As a sidenote at this point I actually don't have a good eln_data_kratos.yaml file
eln_data_kratos.zip , I just wanted to see how far I could get, so I copy pasted one from the examples and made few tweaks, but most of that is definitely wrong, so could be problem on my side, but from the log it looks more like the vamas parsing failed for some reason.
Relevant log output
Using xps reader to convert the given files:
• TaAlO-newTandSseries.vms
• eln_data_kratos.yaml
Traceback (most recent call last):
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/file_parser.py", line 97, in get_dict
    parser_obj.parse_file(file, **kwargs)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/reader_utils.py", line 89, in parse_file
    self.raw_data = self.parser.parse_file(file, **kwargs)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/vms/vamas.py", line 419, in parse_file
    self._parse_blocks()
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/vms/vamas.py", line 520, in _parse_blocks
    self.blocks += [self._parse_one_block()]
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/vms/vamas.py", line 707, in _parse_one_block
    self._add_data_values(block)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/vms/vamas.py", line 716, in _add_data_values
    self._add_regular_data(block)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/vms/vamas.py", line 742, in _add_regular_data
    data_array = np.array(self.data[: block.num_ord_values], dtype=float)
ValueError: could not convert string to float: 'O1s'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/ondracka/software/nomadpyenv/bin/dataconverter", line 8, in <module>
    sys.exit(main_cli())
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools/dataconverter/convert.py", line 399, in convert_cli
    convert(
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools/dataconverter/convert.py", line 221, in convert
    data = transfer_data_into_template(
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools/dataconverter/convert.py", line 165, in transfer_data_into_template
    data = data_reader().read(  # type: ignore[operator]
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/reader.py", line 465, in read
    data_dict = parser.get_dict(**kwargs)
  File "/home/ondracka/software/nomadpyenv/lib64/python3.9/site-packages/pynxtools_xps/file_parser.py", line 102, in get_dict
    raise ValueError(XpsDataFileParser.__vndr_err_msg__) from val_err
ValueError: Need an XPS data file from the following vendors: ['kratos', 'phi', 'scienta', 'specs', 'unkwown']

Hi @ondracka, thanks for testing the reader and for providing the data. I am currently at a conference, but I can probably take this on next week and see what the problem is. I'll contact you once I have tested/fixed it.

Thanks @lukaspie much appreciated.

ESCApe also puts a lot of information in the block comments, so it would be good if they were parsed to reduce the amount of information necessary in the ELN file.


Feature/5295: Fix network-creation when database is unsynced

Codecov Report

Merging #2 into master will increase coverage by 0.22%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
+ Coverage   67.92%   68.15%   +0.22%
==========================================
  Files          12       12
  Lines         873      876       +3
==========================================
+ Hits          593      597       +4
+ Misses        280      279       -1

Impacted Files
Coverage Δ

...view/ml2/drivers/oneview/neutron_oneview_client.py
81.48% <100%> (+0.23%)
:arrow_up:

networking_oneview/ml2/drivers/oneview/common.py
74.89% <0%> (+0.41%)
: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 a4f4011...67c17ba. Read the comment docs.


advertising_id not being picked up in segment context

Context
I am trying to send app events from an Android app developed in Flutter to Facebook by using the flutter-segment library.
Config
I have checked that I use Advertising ID on the Play Console in the App Content section
I have added <uses-permission android:name="com.google.android.gms.permission.AD_ID"/> to the manifest
Issue
When running my app locally with Android Studio Emulator in debug mode with "flutter run", advertising_id is surfaced correctly in the segment context object. However when running the app locally in the same conditions but in release mode with "flutter run --release" (or deploying the app to the Play Store), advertising_id is missing from the context.
Am I missing anything?

I have the same problem but clevertap


KS test issue

Working with function
stats.kstest(x_samp, 'uniform', N=len(x_samp), alternative='two-sided', mode='approx')
I noticed that it return for D value result which I do not expect to see sometimes. According to Kolmogorov 1933 article, for example http://www.dtic.mil/dtic/tr/fulltext/u2/a253575.pdf
the function should use same CDF for D+ and D-, in reality it is written in scipy/scipy/stats/stats.py:
at 3917 line and further:
if alternative in ['two-sided', 'greater']:
Dplus = (np.arange(1.0, N + 1)/N - cdfvals).max()
if alternative == 'greater':
return KstestResult(Dplus, distributions.ksone.sf(Dplus, N))
    if alternative in ['two-sided', 'less']:
        Dmin = (cdfvals - np.arange(0.0, N)/N).max()
        if alternative == 'less':
            return KstestResult(Dmin, distributions.ksone.sf(Dmin, N))

    if alternative == 'two-sided':
        D = np.max([Dplus, Dmin])

Which means that in D+ case CDF goes from 1/N till 1 and in D- case it goes from 0 till 1. That is why it is not equivalent to max(abs(Dplus)) or to max(abs(Dmin))

AFAIR:
There were some mailing list thread on this 7 or 8 years ago, and there was a correction in the NIST notes.
The point in the calculation is to look at the right points in the step functions.
I assume the current code is correct, but don't remember any details.

Is this still an concern?
Is there an example which shows an unexpected value for D returned by kstest?

no response and we've had improvements in kstest since 2016, so closing. thanks all


multiple book versions at the same time

updating the book because of a new version shouldn't remove the old book version so we need a way to display multiple versions of the book.
alternatively, tag book commits that update to a new anchor version and give instruction in readme on how to run a different version locally

this has been solved with the introduction of verrsion branhes


Activating Open Collective

Hi, I'm making updates for Open Collective. Either you or another core contributor signed this repository up for Open Collective. This pull request adds financial contributors from your Open Collective https://opencollective.com/Litepicker ❤
What it does:

adds a badge to show the latest number of financial contributors
adds a banner displaying contributors to the project on GitHub
adds a banner displaying all individuals contributing financially on Open Collective
adds a section displaying all organizations contributing financially on Open Collective, with their logo and a link to their website

P.S: As with any pull request, feel free to comment or suggest changes.
Thank you for your great contribution to the Open Source community. You are awesome! 🙌
And welcome to the Open Collective community! 😊
Come chat with us in the #opensource channel on https://slack.opencollective.com - great place to ask questions and share best practices with other Open Source sustainers!

Your link to your github repo on your Open Collective page is broken - it is just github.com/Litepicker (I don't have a direct way to fix that for you)

Thanks for PR, but I will do a few changes after the merge:

Remove the badge
Contributors
Code Contributors (I tried to open a link to an image in browser and it shows «Not found»)
Financial Contributors (it duplicates the button in the Supporting Litepicker section)

Your link to your github repo on your Open Collective page is broken - it is just github.com/Litepicker (I don't have a direct way to fix that for you)

Thanks. Fixed.


Project link

We create a project, you can check it here: :+1:
project link
Student A: Juan Zorrilla
Student B: Mirna Ampuero

Hi Team, 👋🏻👋🏻
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Highlights

Great job on set up the kanban ✅
Descriptive PR ✅

Required Changes ♻
Check the comments under the review.

[ ] I recommend to assign each task between you a a team and the group tasks you can assign it for you both because assigning the tasks between the teams is very important to know each one of you which tasks will do exactly?! so please assign the tasks ✅♻🔥🔥🔥

Firstly pres on the card then you can assign each card 🔥♻💯

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thank you for your feedback. We assigned each card

Hi @Team, 👋
STATUS: Approved  🎉
Great work on making the changes requested by a previous reviewer :1st_place_medal:
Your project is complete!  There is nothing else to say other than... it's time to merge it
Congratulations! 🎉  🎉 🎉

To Highlight :1st_place_medal:

Your Pull Request has a proper title and description. ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me on( @BenMukebo ) in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Electron prebuilt versions 0.36.8 & 0.36.9 crash after redirect in webview

Electron version: 0.36.8 & 0.36.9
Operating system: OS X 10.11.2

Hey,
I'm working on an app which loads a webview, asks users to login to GitHub, and upon successful login redirects them to another page.
With electron-prebuilt 0.36.7, this works fine. With 0.36.8 and 0.36.9, it crashes at the redirect. Most of the time it crashes silently, but a couple of times (out of ~20 attempts) it spit out an error in the terminal that looks like this:
[21717:0303/192725:FATAL:resource_dispatcher_host_impl.cc(1212)] Check failed: ContainsKey(active_resource_contexts_, resource_context). 0 Electron Framework 0x000000010f63ec63 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 392691 1 Electron Framework 0x000000010f654baa _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 482618 2 Electron Framework 0x000000010fd8aca5 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 8044085 3 Electron Framework 0x000000010fd8aa01 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 8043409 4 Electron Framework 0x000000010fd8a3b2 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 8041794 5 Electron Framework 0x000000010fbb238e _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 6108446 6 Electron Framework 0x000000011057634d _CTFontManagerUnregisterFontForData + 5086093 7 Electron Framework 0x000000011056bf43 _CTFontManagerUnregisterFontForData + 5044099 8 Electron Framework 0x000000011056c052 _CTFontManagerUnregisterFontForData + 5044370 9 Electron Framework 0x000000011056e605 _CTFontManagerUnregisterFontForData + 5054021 10 Electron Framework 0x000000011056e249 _CTFontManagerUnregisterFontForData + 5053065 11 Electron Framework 0x000000011056e009 _CTFontManagerUnregisterFontForData + 5052489 12 Electron Framework 0x000000011056a6eb _CTFontManagerUnregisterFontForData + 5037867 13 Electron Framework 0x000000010f634898 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 350760 14 Electron Framework 0x00000001102b8df5 _CTFontManagerUnregisterFontForData + 2213429 15 Electron Framework 0x000000010f634b4e _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 351454 16 Electron Framework 0x000000010f66d6c3 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 583763 17 Electron Framework 0x000000010f65cfcd _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 516445 18 Electron Framework 0x000000010fc3cba8 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 6675768 19 Electron Framework 0x000000010fc3ccb7 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 6676039 20 Electron Framework 0x000000010f68d808 _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 715160 21 Electron Framework 0x000000010f689c9c _ZN4base8internal30DstRangeRelationToSrcRangeImplIilLNS0_21IntegerRepresentationE1ELS2_1ELNS0_26NumericRangeRepresentationE0EE5CheckEl + 699948 22 libsystem_pthread.dylib 0x00007fff96352c13 _pthread_body + 131 23 libsystem_pthread.dylib 0x00007fff96352b90 _pthread_body + 0 24 libsystem_pthread.dylib 0x00007fff96350375 thread_start + 13
I reverted to 0.36.7 and all works well, but just thought I'd raise the issue. I'm running OS X 10.11.2.
Just in case this is useful info – when the users are already logged in, I point them straight to the page they're redirected to after successful login, and the app doesn't crash.
Best
Nick

Can you paste the crash report which can be found in the Console.app?

Sure, here it is
crash_report.txt

Hmm from the crash report it happened when closing the window, but I can not figure out why. Are you able to work out a reproducible example? The crash is probably caused by something else.

So... After looking into it, it looks like we've been taking advantage of a race condition that was fixed in 0.36.8 onwards.
After the GitHub login redirect, we close the browser window and open a new one. I was unaware of this as it doesn't seem to be mentioned in the documentation, but the app starts the quitting process after all the browser windows are closed.
On 0.36.7 – opening a new browser window after closing the previous one stops the quitting process, and the will-quit event is not triggered.
On 0.36.8 onwards – opening a new browser window after closing the previous one does not stop the quitting process and the will-quit event is triggered. I haven't dug further than that, but these two things happening concurrently appear to be the source of the crash.
The example project attached illustrates the crash. A browser window is opened and closed after 10 seconds, and a new window is opened straight after closing the previous one. On 0.36.7, this doesn't crash, and on 0.36.8 onwards, it does.
I'm guessing this is the intended behaviour and we were getting away with it by exploiting a bug... Will refactor our code and it should all work fine.
Thanks
example.zip

Minimal test case:
app.on('ready', function() {
  createWindow();
});

function createWindow() {
  window = new BrowserWindow({width: 800, height: 800});
  window.loadURL('about:blank');
  setImmediate(function() {
    window.close();
    createNewWindow();
  });
}

function createNewWindow() {
  window = new BrowserWindow({width: 800, height: 800});
  window.loadURL('about:blank');
}

Unable to reproduce in 1.2.2 anymore.


Need to calculate average, dispersion, percentFavourable, percentileRankingByFaculty, percentileRankingByDept

Need to reimplement some functions to work off the "count" object created in createAggregatedData script

Completed implementing average, dispersion, and percentFavouable.
percentileRankingByFaculty, percentileRankingByDept are trickier but should be able to implement by today

Everything implemented. Need to still test


docs(cli): a stopped workflow cannot be resumed

Fixes #11511
Motivation

The CLI had incorrectly stated that argo resume could be used on a stopped workflow, resulting in confusion for some users, such as in #11511
Modifications

remove the "stopped" statement in argo resume
specify that resume and suspend are opposites (and therefore unrelated to stop / terminate)

Verification

make codegen passes

Behavior Verification
To make sure that argo resume does not affect stopped workflows, I looked through the source code:

stop and terminate are two "shutdown strategies" with minor differences
the shutdown strategy is marked on a Workflow via spec.shutdown by the Server. then the operator and exec controller determine how to handle it (i.e. whether to run exit handlers or not).
meanwhile, suspend and resume are marked via spec.suspend, which is an unrelated flag.

As I had originally thought, stop and resume are unrelated operations. Not sure if there were related at some point in the past though.
Notes for Reviewers
I wrote this as a "fix" commit so that it can be added as a patch to the CLI (and make it into any cherry-picks), but could make it a "docs" commit instead

Ack, unit tests failed on a 429 rate limit error 😕 Will need to retry those

@terrytangyuan could you retry the unit tests CI? It hit a rate limit per above

@terrytangyuan just following up on this one that needs a retry

done


Transactions: Use leaked pointers for large transactions

Describe the feature
#1655 and #1708 add support for sending large command arguments as leaked vectors of bytes to Rust. This speeds up the requests significantly. This is based on a threshold retrieved from Rust.
However a transaction can be made up of multiple requests that individually have arguments that are smaller than this threshold but exceed it when combined.
This feature request would be to use leaked pointers when the combined length of all command arguments in a transaction exceed the threshold rather than only when an individual command.
Use Case
Better performance for large transactions where each request does not exceed the argument size limit individually, but combined do exceed.
Proposed Solution
Change Transactions to contain a List<CommandBuilder, String[] arguments> where the command builder is partially filled with the request type. The arguments are iterated over to get the total argument size and if it exceeds the Rust threshold, all arguments get serialized to byte vectors.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

Client version used
N/A
Environment details (OS name and version, etc.)
macOS

[ ] Python
[ ] Java
[ ] NodeJS


Object Required Error on JSON String

Hi,
This is my first use of the vba library and it looks very useful. I have problem with the code below it's minimal at the moment as I'm testing some concepts before launching into full development. as you can see I working with JSON strings. So far the REST request works fine and I get a JSON string returned, but when it get to the highlighted line "Set GetProjects = Response.Data("msec")" this is where I get the Runtime Error 424. By debugging I can see that the code is attempting to get the data using the key but I'm not sure why it isn't seeing as an object! I've attached the response file.
Many thanks in advance
IP
response.txt
_WebHelpers.EnableLogging = True
' Use GET and json (for request and response)
Dim Request As New WebRequest
Request.Resource = "match"
Request.Method = WebMethod.HttpGet
Request.Format = WebFormat.Json
Request.ResponseFormat = Json
Request.AddQuerystringParam "apikey", "794f2984"
Request.AddQuerystringParam "queries", "[{" & """mpn""" & ":" & """SN74S74N""" & "}]"

Dim Response As WebResponse

Set Response = Client.Execute(Request)

' -> GET https://www.example.com/api/projects
'
' <- HTTP/1.1 200 OK
'    ...
'    {"data":[{"id":1,"name":"Project 1"},{"id":2,"name":"Project 2"}]}

If Response.StatusCode <> WebStatusCode.Ok Then
    Err.Raise Response.StatusCode, "GetProjects", Response.Content
Else
    ' Response is automatically converted to Dictionary/Collection by Request.Format

**       Set GetProjects = Response.Data("msec")**
End If
End Function_

@djseven7 Glad you found the library. It looks like "msec" is a number, so you don't need Set to get the value. Here are some examples based on your given response:
' Get first item
Dim Item As Dictionary
Set Item = Response.Data("results")(1)("items")(1)

' Get first seller name for first offer of first item
Dim Seller As String
Seller = Response.Data("results")(1)("items")(1)("offers")(1)("seller")("name")

Thank you Tim for getting back to me. Great library by the way.
IP


Add randomization of wing smash trail color

When doing a Wing Smash, a light blue trail is left behind the bat.
This PR will instead select a random byte to identify one of 256 GPU buffers holding palettes, which should result in a random wing smash appearance.
I have attached a video of me in the emulator poking arbitrary values to this location in RAM, to illustrate some of the possible outcomes. This PR should create similar results in any randomized ROM.
https://github.com/user-attachments/assets/47181fa3-cb03-45dc-9572-5f6885e72c5f
Overall, I think this fits nicely alongside my previous gravity boot color randomization.

Closed due to moving over to the other randomizer repo


Wide spacing in "List of ASP stories that have translations"

The licence (CC-BY-NC) in the list of ASP stories that have translations has runs over two or three lines, giving a lot of white space. Can we force this column to be wide enough to have space for "CC-BY-NC" on one line?

I'll try to clean this up along with the layout on the rest of the page.

Fixed in latest update.


Sending multiple notifications on MacOS fails

MacOS panics with the error below when trying to send multiple notifications.
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: ErrorMessage { msg: AlreadySet }

Example
fn main() {
    let app = "testapp";
    notifica::notify(app, "test1"); // succeeds
    notifica::notify(app, "test2"); // errors
}

The underlying notification library used for MacOS tracks if the application name has been set, and will error (https://github.com/h4llow3En/mac-notification-sys/blob/master/src/lib.rs#L146) when trying to set the application name again. ::notify() should handle this error gracefully.
PR (https://github.com/frewsxcv/rust-notifica/pull/10) is refactoring calls to unwrap() and return the library errors for notifica,  although this issue would still be present (it wouldn't panic, but the early return would not trigger additional notifications). If https://github.com/frewsxcv/rust-notifica/pull/10 is the direction for error handling, I can catch the ApplicationError::AlreadySet locally and continue execution, otherwise return all other errors to the caller.
Nonetheless I can fix this, just need to know what branch to branch from.

good catch! i just merged the other pull requests, so you should be good to branch off master

fixed in https://github.com/frewsxcv/rust-notifica/pull/22


Typo in name

All of the folders and the lower case identifier should be psrsigsim, but it is psrsimsig.

well, this is embarrassing...


empty dict returned sometimes

https://github.com/h2oai/helium/pull/83#issuecomment-1605307189

not seen


Support optionally deleting backing services

Problem
Currently app broker supports creating backing app with existing backing services (see CreateInstanceWithExistingServicesComponentTest). If the backing service exists before the backing app bind to it, then it’s risky to just delete it when we delete the backing app. For example, a shared database instance or a user provided service.
Proposed solution
If the backing service details (e.g. plan) is specified in the configuration, we can assume broker developers rely on app broker to manage the backing services, so we delete the backing services when no longer needed. If the backing service details is not specified, or the bound service is not listed in the configuration, then we only need to unbind the service but not delete it.
What's not covered by the proposed solution
When there is an update in the backing service list, the existing backing service may become an unknown backing service if we do an upgrade update. So the proposed solution would just unbind it but not deleting the existing backing service. But the risk of this is not too high because when we eventually delete the space, it will be deleted cascadingly. If this gets more risky and needs attention, it's also worth exploring to leverage CAPI version metadata.

What about annotating provisionned backing services and apps with the brokered service instance guid (and potentially other metadata such as a config version/hash), in CF meta-data ?
Then the algorithm for brokered service delete, is to lookup any backing app/service with the associated metadata, and delete them.
See related conversation at https://github.com/spring-cloud/spring-cloud-app-broker/issues/344#issuecomment-600611289 and related met