lr_finder fails when called after training for 1 or more epochs

🐛 Bug

Calling lr_finder on the model after trainer.fit() has been called will fail with:
LR finder stopped early due to diverging loss.
Failed to compute suggesting for `lr`. There might not be enough points.

, even when the default value of min_lr=1e-08 has been changed to 1e-30.
Please reproduce using the BoringModel and post here

Reproduced using a callback: https://colab.research.google.com/drive/1sbOPs8edyFi_idJNnd6gr3etyv7V57YU?usp=sharing
Reproduced with calling Trainer twice: https://colab.research.google.com/drive/1WxUvayBBg_163nu8fjv-jsvPk6pUrSrK?usp=sharing

To Reproduce

Add the following callback (as demonstrated with the BoringModel):
# Call Learning Rate finder after X epochs
class LRFinderXEpoch(Callback):
    def __init__(self, epoch=1):
        super().__init__()

        self.epoch = epoch

    def on_train_epoch_start(self, trainer, pl_module):
        if trainer.current_epoch == self.epoch:
            print("Calling learning rate finder!")
            trainer.tune(pl_module)
            # trainer.tuner.lr_find(pl_module, min_lr=1e-30)

Expected behavior

Find the best learning rate after a few epochs of training (e.g. when doing Transfer Learning).
Environment
* CUDA:
	- GPU:
		- Tesla T4
	- available:         True
	- version:           10.1
* Packages:
	- numpy:             1.18.5
	- pyTorch_debug:     True
	- pyTorch_version:   1.7.0+cu101
	- pytorch-lightning: 1.0.8
	- tqdm:              4.41.1
* System:
	- OS:                Linux
	- architecture:
		- 64bit
		-
	- processor:         x86_64
	- python:            3.6.9
	- version:           #1 SMP Thu Jul 23 08:00:38 PDT 2020

Additional context

Issue came from the following discussion: https://forums.pytorchlightning.ai/t/train-2-epochs-head-unfreeze-learning-rate-finder-continue-training-fit-one-cycle/366/4
Potentially related issues:

https://github.com/PyTorchLightning/pytorch-lightning/issues/4784
https://github.com/PyTorchLightning/pytorch-lightning/issues/4616

The early stopping threshold of the Learning Rate finder seems to be too small. I regularly observe https://github.com/PyTorchLightning/pytorch-lightning/blob/f2fa3c82567ade0aa5bea3aa298542fc5040e4b7/pytorch_lightning/tuner/lr_finder.py#L419 to trigger on a standard image classification task after the first batch.

@maxjeblick Thank you for the suggestion.
I tried in both the BoringModel and my own ML experiment the following code in the lr-finder callback:
lr_finder = trainer.tuner.lr_find(pl_module, early_stop_threshold=1000.0, min_lr=1e-30)
lr = lr_finder.suggestion()
print(f"Found lr: {lr}")
pl_module.hparams["learning_rate"] = lr

I tried both early_stop_threshold=1000.0 and 0.001 (default: 20.0) in combination with min_lr=1e-30 (default: 1e-8).
Which both print: Found lr: None.
So the issue seems to be somewhere else :/

As by @maxjeblick 's suggestion on SLACK, I tried with early_stop_threshold=None. This throws a RuntimeError: value cannot be converted to type float without overflow: -5.01187e+38 for the BoringModel, but it works for my own ML project.
However, the next issue I face is: RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!
Even if I try the following modifications to my snippet above:
from pytorch_lightning.utilities.apply_func import move_data_to_device

lr = lr_finder.suggestion()

torch.tensor(lr).cuda()
torch.tensor(lr, device=pl_module.device)
torch.tensor(lr).to(pl_module.device)
move_data_to_device(lr, device=pl_module.device)

All fail to get rid of the device issue.

@SkafteNicki any idea?

For my project, I found that setting early_stop_threshold=None fixes this. Thanks to @NumesSanguis for the suggestion.

Preventing stale bot from closing the issue, as it is still a problem.

Preventing stale bot from closing the issue, as it is still a problem.

For my project, I found that setting early_stop_threshold=None fixes this. Thanks to @NumesSanguis for the suggestion.

Same here. Thanks guys! And let's hope that this is eventually properly solved,

early_stop_threshold=None

It makes it work!

@NumesSanguis and @jbohnslav, would you agree that this is not an ideal solution?  I would think we would want early stopping enabled.  I'm finding that I get RuntimeError: value cannot be converted to type float without overflow when early stopping is not set to None.

@jstremme This is indeed not a solution, just a workaround that works in some cases. I'm not working anymore on the project I encountered that problem though (never solved it).

Neither setting it to None or 100000.0 works.

early_stop_threshold=None works perfectly for me. this issue should be resolved I believe.


chore: Make wasmcloud host heartbeat interval configurable

Feature or Problem
For integration testing, it would be nice to make the host heartbeat interval configurable so that it can be shortened in an integration testing that uses the heartbeats.
In the future being able to adjust the host heartbeat frequency would also open up the possibility of reducing the amount of heartbeat traffic in a large deployment, though this likely necessitates changes in other components of the system as well (i.e. adjusting the wadm side of things appropriately as well).
Related Issues

Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

oh also you might want chore(host): ...


IT'S VIRUS ( Trojan:Script/Wacatac.H!ml )

Yes dude are you saying the open source project is a virus?

No description provided.

lmao


Bump MSRV in README

Since this was increased in https://github.com/raphlinus/pulldown-cmark/commit/f30d7201ce078af72d305c3e6287215ae2679154.

Thank you!


Add support for Chinese character sets GB2312 and GB18030.

These character sets are mostly used in mainland China.

This is not needed. This problem is already fixed. And there is a test for that.

Great! 👍


Added three algo to recursion

📥 Pull Request
Description
Added 3 algo to Recursion

Generate Paranthesis
Count all subsequences with sum K
Letter Combinations of a Phone number

Fixes #340
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@ajay-dhangar check this out

update code based on suggestions

@ajay-dhangar Updated changes . Please check once


Translation to Chinese Simplified

{
  "entity.minecraft.villager.dynamicvillage.mechanical_engineer": "机械工程师",
  "entity.minecraft.villager.dynamicvillage.hydraulic_engineer": "液压工程师",
  "entity.minecraft.villager.dynamicvillage.train_mechanic": "列车机械师",
  "entity.minecraft.villager.dynamicvillage.miner": "矿工",
  "itemGroup.village_life_tab": "Create: Dynamic Villages"
}

Thank you so much for the localization! I've updated the mod with the mainland Chinese translation included, and also added a credit to you in the Github readme, on Curseforge and on Modrinth.
Again, thank you so very much--I really do appreciate it!


Set page opacity in PageRouteBuilder

Hi, I want to set page opacity before I push It. With Navigator 2.0 Ir could be done like this
Navigator.push(
                            context,
                            PageRouteBuilder(
                              opaque: false, //Setting opacity here
                              pageBuilder: (ctx, _, __) {
                                return DropDownMenuItems(
                                  topMargin: 87,
                                  alignment: Alignment.topRight,
                                );
                              },
                            ),
                        );

I want to set opaque value to false when I push a new page using Routemaster. Any idea how that could be done?

Hi, you need to define a custom page, this should work:
class DropDownMenuPage extends Page<void> {
  @override
  Route<void> createRoute(BuildContext context) {
    return PageRouteBuilder(
      settings: this, // Make sure to provide the Page as settings
      opaque: false,
      pageBuilder: (ctx, _, __) {
        return DropDownMenuItems(
          topMargin: 87,
          alignment: Alignment.topRight,
        );
      },
    );
  }
}

Then return DropDownMenuPage from your route builder.


Add missing link to Handlebars.java

It was present in the graph but not listed with the other template engines. This will improve SEO.

Maybe we could also specify "Handlebars.java" and "Mustache.java" somewhere in the README, to improve SEO, and to make it clear that we are benchmarking those specific implementations?
Either:

Handlebars (Handlebars.java implementation)
Mustache (Mustache.java implementation)

Or:

Handlebars.java
Mustache.java


(ASC-704) added Nova molecule utils methods to pytest-prc

@phongdly It seems like there is quite a bit of sleeping going on here.  For a future improvement should we build a command retry mechanism with an exponential backoff?  @johnduarte What do you think?

@zreichert I am unclear as to what the sleep statements are for. They don't seem to have any impact internally to the methods. I am waiting for @phongdly to clarify the intent of these sleep statements.

@johnduarte @zreichert I have experienced some failure when verifying if an object is completely removed right after deleting it, so I added sleep() after and before some actions like create or delete. I can update the PR and use the existing method of get_expected_value for creating, but probably need to add another function to make sure an object is completely deleted

@zreichert @johnduarte : updated based on your comments. Thanks


added load tests for vpa

adds load test by making post requests for the vpa
output of  kyverno-vpa.js-25vu-500it-logs.txt
  execution: local
     script: /script/script.js
     output: -

  scenarios: (100.00%) 1 scenario, 25 max VUs, 10m30s max duration (incl. graceful stop):
           * default: 500 iterations shared among 25 VUs (maxDuration: 10m0s, gracefulStop: 30s)

     ✓ verify response code of POST VPA is 201

     █ teardown

     checks.........................: 100.00% ✓ 500        ✗ 0
     data_received..................: 437 kB  659 kB/s
     data_sent......................: 188 kB  284 kB/s
     http_req_blocked...............: avg=1.18ms   min=327ns   med=647ns   max=59.76ms p(90)=999ns    p(95)=370.63µs
     http_req_connecting............: avg=51.2µs   min=0s      med=0s      max=1.95ms  p(90)=0s       p(95)=7.14µs
     http_req_duration..............: avg=30.47ms  min=7.72ms  med=28.92ms max=72.06ms p(90)=38ms     p(95)=44.23ms
       { expected_response:true }...: avg=30.47ms  min=7.72ms  med=28.92ms max=72.06ms p(90)=38ms     p(95)=44.23ms
     http_req_failed................: 0.00%   ✓ 0          ✗ 500
     http_req_receiving.............: avg=439.68µs min=26.3µs  med=76.77µs max=16.69ms p(90)=920.41µs p(95)=2.41ms
     http_req_sending...............: avg=312.2µs  min=45.37µs med=80.72µs max=28.97ms p(90)=544.57µs p(95)=1.07ms
     http_req_tls_handshaking.......: avg=1.07ms   min=0s      med=0s      max=53.6ms  p(90)=0s       p(95)=292.54µs
     http_req_waiting...............: avg=29.72ms  min=7.6ms   med=28.53ms max=67.71ms p(90)=36.72ms  p(95)=43.3ms
     http_reqs......................:<PHONE_NUMBER>04/s
     iteration_duration.............: avg=32.3ms   min=2.08µs  med=29.66ms max=91.97ms p(90)=43.73ms  p(95)=54.22ms
     iterations.....................:<PHONE_NUMBER>04/s
running (00m00.7s), 00/25 VUs, 500 complete and 0 interrupted iterations
default ✓ [ 100% ] 25 VUs  00m00.7s/10m0s  500/500 shared iters

Good job writing your first test.
I think we want to test how the VPA would respond to another Pod being load tested. We don;t want to load test the API Server by creating a lot of VPAs.

@KhaledEmaraDev please review it now if this is what you actually wanted ?


AttributeError: 'Namespace' object has no attribute 'label_list'

Training with a custom dataset I get the following error.  Changing ' cat_list=args.label_list' to  'cat_list=[list, of, my, classes]' seems to work.
Traceback (most recent call last):
File "/content/Open-GroundingDino/main.py", line 372, in
main(args)
File "/content/Open-GroundingDino/main.py", line 144, in main
model, criterion, postprocessors = build_model_main(args)
File "/content/Open-GroundingDino/main.py", line 81, in build_model_main
model, criterion, postprocessors = build_func(args)
File "/content/Open-GroundingDino/models/GroundingDINO/groundingdino.py", line 802, in build_groundingdino
postprocessors = {'bbox': PostProcess(num_select=args.num_select  , text_encoder_type=args.text_encoder_type,nms_iou_threshold=args.nms_iou_threshold,args=args)}
File "/content/Open-GroundingDino/models/GroundingDINO/groundingdino.py", line 652, in init
cat_list=args.label_list
AttributeError: 'Namespace' object has no attribute 'label_list'

Thank you for pointing that out， I've updated it in the README

Thank you for pointing that out. I've updated it in the README

Do you mean add a new line with label_list=[list, of, my, classes] to the cfg.py file ? That did not work on my end. I needed to add it directly to the groundingdino.py and engine.py files in order for training to work.

It looks like there is a problem with your config. When setting your label list, each class needs to be enclosed in quotation marks.
e.g. label_list = [
'aeroplane', 'bicycle', 'bird', 'boat', 'bottle', 'bus', 'car', 'cat',
'chair', 'cow', 'diningtable', 'dog', 'horse', 'motorbike', 'person',
'pottedplant', 'sheep', 'sofa', 'train', 'tvmonitor'
]

Here's my whole config file. I think it matches your instructions.
data_aug_scales = [480, 512, 544, 576, 608, 640, 672, 704, 736, 768, 800]
data_aug_max_size = 1333
data_aug_scales2_resize = [400, 500, 600]
data_aug_scales2_crop = [384, 600]
data_aug_scale_overlap = None
batch_size = 4
modelname = 'groundingdino'
backbone = 'swin_T_224_1k'
position_embedding = 'sine'
pe_temperatureH = 20
pe_temperatureW = 20
return_interm_indices = [1, 2, 3]
enc_layers = 6
dec_layers = 6
pre_norm = False
dim_feedforward = 2048
hidden_dim = 256
dropout = 0.0
nheads = 8
num_queries = 900
query_dim = 4
num_patterns = 0
num_feature_levels = 4
enc_n_points = 4
dec_n_points = 4
two_stage_type = 'standard'
two_stage_bbox_embed_share = False
two_stage_class_embed_share = False
transformer_activation = 'relu'
dec_pred_bbox_embed_share = True
dn_box_noise_scale = 1.0
dn_label_noise_ratio = 0.5
dn_label_coef = 1.0
dn_bbox_coef = 1.0
embed_init_tgt = True
dn_labelbook_size = 91
max_text_len = 256
text_encoder_type = "bert-base-uncased"
use_text_enhancer = True
use_fusion_layer = True
use_checkpoint = True
use_transformer_ckpt = True
use_text_cross_attention = True
text_dropout = 0.0
fusion_dropout = 0.0
fusion_droppath = 0.1
sub_sentence_present = True
max_labels = 80                               # pos + neg
lr = 0.0001                                   # base learning rate
backbone_freeze_keywords = None               # only for gdino backbone
freeze_keywords = None                        # for whole model, e.g. ['backbone.0', 'bert'] for freeze visual encoder and text encoder
lr_backbone = 1e-05                           # specific learning rate
lr_backbone_names = ['backbone.0', 'bert']
lr_linear_proj_mult = 1e-05
lr_linear_proj_names = ['ref_point_head', 'sampling_offsets']
weight_decay = 0.0001
param_dict_type = 'ddetr_in_mmdet'
ddetr_lr_param = False
epochs = 3
lr_drop = 10
save_checkpoint_interval = 10
clip_max_norm = 0.1
onecyclelr = False
multi_step_lr = False
lr_drop_list = [10, 20]
frozen_weights = None
dilation = False
pdetr3_bbox_embed_diff_each_layer = False
pdetr3_refHW = -1
random_refpoints_xy = False
fix_refpoints_hw = -1
dabdetr_yolo_like_anchor_update = False
dabdetr_deformable_encoder = False
dabdetr_deformable_decoder = False
use_deformable_box_attn = False
box_attn_type = 'roi_align'
dec_layer_number = None
decoder_layer_noise = False
dln_xy_noise = 0.2
dln_hw_noise = 0.2
add_channel_attention = False
add_pos_value = False
two_stage_pat_embed = 0
two_stage_add_query_num = 0
two_stage_learn_wh = False
two_stage_default_hw = 0.05
two_stage_keep_all_tokens = False
num_select = 300
batch_norm_type = 'FrozenBatchNorm2d'
masks = False
aux_loss = True
set_cost_class = 1.0
set_cost_bbox = 5.0
set_cost_giou = 2.0
cls_loss_coef = 2.0
bbox_loss_coef = 5.0
giou_loss_coef = 2.0
enc_loss_coef = 1.0
interm_loss_coef = 1.0
no_interm_box_loss = False
mask_loss_coef = 1.0
dice_loss_coef = 1.0
focal_alpha = 0.25
focal_gamma = 2.0
decoder_sa_type = 'sa'
matcher_type = 'HungarianMatcher'
decoder_module_seq = ['sa', 'ca', 'ffn']
nms_iou_threshold = -1
dec_pred_class_embed_share = True
match_unstable_error = True
use_detached_boxes_dec_out = False
dn_scalar = 100
use_coco_eval = False
label_list = ['none', 'avr', 'mvp', 'defib', 'lvad']

I tested it using your config and it seems to be right.Perhaps you can refer to this https://github.com/open-mmlab/mmdetection/issues/2698 and check if your config and path are set correctly.

Unfortunately, I'm still having the same issue with the config file. The work around is easy enough. I suspect it has something to do with a package version/the colab environment.  Maybe keep this issue open for a bit to if others encounter it.

I think this might be related to the colab environment


The state of the "Deployments" node doesn't refresh automatically after configuring deployment source

OS: All
Build Version: 20201110.1
Repro Steps:

Create a new function app and expand it.
Right click the function app -> Click "Configure Deployment Source..." -> Select "LocalGit".
Check the result.

Expect:
The "Git" icon appears automatically next to "Deployments" node.

Actual:
The "Git" icon doesn't appear automatically next to "Deployments" node.

More Info:

This issue also reproduces when selecting "None/GitHub".
This issue will be fixed after refreshing.

:slightly_frowning_face: In the last 60 days, this issue has received less than 5 community upvotes and we closed it. Still a big Thank You to you for taking the time to create it! To learn more about how we handle issues, please see our documentation.
Happy Coding!


Property 'token' is missing in type 'PropsWithChildren' but required in type '{ token: string; }'.

Hey, I'm kinda stuck at 5:09:03 and im getting the following error:

Argument of type 'FunctionComponent<{ token: string; }> & { getInitialProps?(context: NextPageContext): { token: string; } | Promise<{ token: string; }>; }' is not assignable to parameter of type 'NextComponentType<PartialNextContext, {}, {}>'.
Type 'FunctionComponent<{ token: string; }> & { getInitialProps?(context: NextPageContext): { token: string; } | Promise<{ token: string; }>; }' is not assignable to type 'FunctionComponent & { getInitialProps?(context: PartialNextContext): {} | Promise<{}>; }'.
Type 'FunctionComponent<{ token: string; }> & { getInitialProps?(context: NextPageContext): { token: string; } | Promise<{ token: string; }>; }' is not assignable to type 'FunctionComponent'.
Types of parameters 'props' and 'props' are incompatible.
Type 'PropsWithChildren' is not assignable to type 'PropsWithChildren<{ token: string; }>'.
Property 'token' is missing in type 'PropsWithChildren' but required in type '{ token: string; }'.

With this piece of code:
import { Box, Button, Text } from '@chakra-ui/react';
import { Formik, Form } from 'formik';
import { NextPage } from 'next';
import { withUrqlClient } from 'next-urql';
import Image from 'next/image';
import { useRouter } from 'next/router';
import React from 'react';
import { useState } from 'react';
import { InputField } from '../../components/InputField';
import { Wrapper } from '../../components/Wrapper';
import { useChangePasswordMutation } from '../../generated/graphql';
import { createUrqlClient } from '../../utils/createUrqlClient';
import { toErrorMap } from '../../utils/toErrorMap';

const ChangePassword: NextPage<{ token: string }> = ({ token }) => {
  const router = useRouter();
  const [, changePassword] = useChangePasswordMutation();
  const [tokenErr, setTokenErr] = useState('');
  return (
    <Wrapper variant='small'>
        {/* you know that stuff */}
    </Wrapper>
  );
};

ChangePassword.getInitialProps = ({ query }) => {
  return {
    token: query.token as string,
  };
};

export default withUrqlClient(createUrqlClient)(ChangePassword);

And I really can't go further, it made so frustrated I'm out of ideas how to fix this. I know, Ben's switching to Apollo Client, but I want to follow the whole tutorial.
Thanks in advance

Maybe it's a new requirement in the library since he did the video. I took the error message as a type issue with the NextComponentProps once you hook up the urql client. I changed the component declaration from this:
const ChangePassword: NextPage<{ token: string }> = ({ token }) => {
to this:
const ChangePassword: NextPage<WithUrqlProps, { token: string }> = ({ token }) => {
and the code works. I'm newer to typescript so somebody could correct me if there is a better way of doing it.

try:
export default withUrqlClient(createUrqlClient) (ChangePassword as any);
That worked for me

You can also use this one as TS frowns on any and if you still want the autocomplete functionality
export default withUrqlClient(createUrqlClient)(
  ChangePassword as FunctionComponent<
    PropsWithChildren<WithUrqlProps | { token: string }>
  >
);


The Song loader is only active for a min

https://github.com/Umbranoxio/BeatSaberModInstaller/issues/15

I start the game and I can play one map before beat saber reverts to an un-modded capacity. The little in game beat-saver button that took you to the map download screen is not there anymore. Basically, i can only play maps I have already downloaded and I can only play one unless i want to restart the game over and over again to load the list again.

Same problem, there are only default tracks in the menu after any played game.
No stats are saved too, because game thinks you played "100 dollar bills" - the first default track.

Same deal here

This only happens if you have outdated plugin installed as well. You'll want to do a clean install, delete the Plugins folder to get rid of all your old plugins (you won't need those old versions again) and run the mod installer again or install manually if you want.


Question: return type of cancelOnGracefulShutdown

https://github.com/swift-server/swift-service-lifecycle/blob/f5780d95bdf3f740f14decb6c8c823815c883451/Sources/ServiceLifecycle/GracefulShutdown.swift#L118
I noticed that the cancelOnGracefulShutdown method signature suggests it returns an optional (T?).
However, I couldn’t find any code paths in the implementation where a nil value would be returned.
Could you please clarify why the return type is optional in this context?
Is there a scenario where this method might return nil that I might have missed?

You are right this seems like a mistake we should fix. This method indeed shouldn't return an optional. Would you be open to file a PR for this?


Error: looks like we got no XML document

I've got this error message ("looks like we got no XML document") because the content of one email had the character "&#x19 ;".
I've fixed this problem by adding the folowing line:
$response = preg_replace('/&#x[0-1]?[0-9A-E];/', ' ', $response);
to NTLMSoapClient.php after line 82 (after the line with curl_exec)

So the regex pasted here doesn't cover all possibilities since hex goes to F.
Proper:
$response = preg_replace('/&#x[0-1]?[0-9A-E];/', ' ', $response);

Symbols 0x9, 0xA and 0xD are allowed, so here is my version of the quick fix:
$response = preg_replace('/(?!&#x0?(9|A|D))(&#x[0-1]?[0-9A-F];)/', ' ', $response);

I'd love to resolve this issue, and I've left a comment on the pull request as well, but I need to be able to reproduce to ensure this works. If somebody could provide me with an example request I could look into this further.
It sounds like this issue would require an email one of these characters to already exist in the user's mailbox. If somebody has an email they could forward to me, please reach out to me and I will give you an email address you could forward it to.

I found a way to reproduce this issue and have committed this fix in jamesiarmes/php-ntlm@b6c3ee5ba980fdf5e8e7803a8b8a6f0de874b731. Running composer update should get you the latest code from this repo.


Hello,

Hello, I converted yolov3 learned with pytorch to onnx
I converted yolov3 learned with pytorch to onnx. Is it possible to use the converted onnx file with GitHub project?
Thank you.

I have not tried it by myself.  Some other people have had success doing so: https://github.com/jkjung-avt/tensorrt_demos/issues/267#issuecomment-721714934

I have not tried it by myself.  Some other people have had success doing so: https://github.com/jkjung-avt/tensorrt_demos/issues/267#issuecomment-721714934


Default add triage to issues

Summary of the Pull Request
What is this about:
adding in Triage Needed by default
What is include in the PR:
the templates with the tag
How does someone test / validate:
We merge in and check.  Does not touch any source
Quality Checklist

[x] Linked issue: #9136
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[ ] Binaries: Any new files are added to WXS / YML

[ ] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

Contributor License Agreement (CLA)
A CLA must be signed. If not, go over here and sign the CLA.

@crutkas
when the author of an issue replies, the Needs-Author-Feedback label is removed and both Needs-Team-Response and Triage-Needed are added.
Do we need the Triage-Needed label in that case?

@crutkas
when the author of an issue replies, the Needs-Author-Feedback label is removed and both Needs-Team-Response and Triage-Needed are added.
Do we need the Triage-Needed label in that case?

My thought was the needs response helps us directly know it is a follow up but they still need triage. This makes it one vs 2 queries

My thought was the needs response helps us directly know it is a follow up but they still need triage. This makes it one vs 2 queries


Implement the Download File Information Sections To Dataset Page

Context
The type of Implementation: New feature (non-breaking change which adds functionality)
On the dataset page (which is accessible using urls that are emailed to users), there are three sections called "Download File Summary", "Dataset Summary", and "Samples" that need to be implemented.
Problem or idea
Mockup: Dataset Pages

Based on the mockup, we'll add these three sections to the dataset page.
Solution or next step

Use the existing download page's sub-component in the dataset page
Create any sub-components or helpers as necessary

Completed:

Download File Summary #57
Dataset Summary #58
Samples: Species View #69, Experiment View #70


Extract and reuse networkCompletion closure in requestNormal body

Saw the opportunity to reuse this code while playing with the project. Just a bit of gardening.

          2 Warnings

:warning:
Any changes to library code should be reflected in the Changelog. Please consider adding a note there.

:warning:
The library files were changed, but the tests remained unmodified. Consider updating or adding to the tests to match the library changes.

SwiftLint found issues
Warnings

File
Line
Reason

MoyaProvider+Internal.swift
74
Lines should not have trailing whitespace.

  Generated by :no_entry_sign: Danger

Codecov Report

Merging #1149 into master will increase coverage by 0.57%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #1149      +/-   ##
==========================================
+ Coverage   76.51%   77.08%   +0.57%
==========================================
  Files          22       22
  Lines         728      720       -8
==========================================
- Hits          557      555       -2
+ Misses        171      165       -6

Impacted Files
Coverage Δ

Sources/Moya/MoyaProvider+Internal.swift
63.67% <100%> (+1.4%)
: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 722eee8...66ae6a9. Read the comment docs.

Thanks a lot for contributing to Moya! I've invited you to join the Moya GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.


How to fix sub process /usr/libexec/cydia/cydo returned an error code (2) no pc no tweaks no iFile?

Ask on /r/jailbreak.

ugh another one of these, read #64

#162


lambda expression sometimes not correctly parsed

Consider this lambda expression:
let Lambda = {-> 1
\                ? '│' : ''}

If I write it in a temporary file, then source it, the following error is raised:
Error detected while processing /tmp/v8xVFPH/7:
line    1:
E15: Invalid expression: {-> 1

It seems Vim ignores everything after the line continuation.
If I remove the line continuation:
let Lambda = {-> 1 ? '│' : ''}

Or if I replace the multibyte character with its unicode point:
let Lambda = {-> 1
\                ? "\u2502" : ''}

Then the lambda is correctly defined.

Here's another similar issue.
Consider this lambda:
echo map(['foo'], {i,v -> 1
\?                            v
\:                            v
\ })

If I write it in a temporary file, then source it, the following error is raised:
Error detected while processing /tmp/v8xVFPH/30:
line    4:
E109: Missing ':' after '?'

If I separate the backslash from the colon with a space, the code works fine:
echo map(['foo'], {i,v -> 1
\?                            v
\ :                           v
\ })

The first issue was wrong sorry. The second one is probably not important.


Following the license change, modify the boilerplates in crypto/evp/

Merged.
4a8b0c55c0b149a266b4ec61ef643027b6408475 Following the license change, modify the boilerplates in crypto/evp/


Restructure start-mailserver.sh

From my point of view it would be a big benefit, to structure the script with functions, for better maintenance.
With this improvement it would be possible to better control the bootstrap of the container:
I would appreciate this step and would spent my time working on this.
Best Regards

You're right, this file is a bit messy.
Can you work on that?
All, any thoughts?

Functions for start-mailserver.sh would be great!

If you merge my pull request #335 then we would have the logic for container setup with postfix as central part as mta of a full featured mailserver:
Postfix <-> User database and User Validation <-> imap/pop server
Postfix <-> dovecot/postfix local <-> dovecot
Postfix <-> dovecot/postfix ldap (dovecot ldap needs to be implemented)<-> dovecot
Postfix <-> dovecot local/postfix ldap <-> dovecot
Postfix <-> dovecot ldap (needs to be implemented)/postfix local <-> dovecot
Postfix <-> ldap/activedirectory <-> kopano/zarafa
With this in mind, i would suggest the following structure:
do_setup_postfix ()
do_setup_opendkim()
do_setup_opendmarc()
do_setup_ssl()
do_setup_permit_docker_options()
do_setup_saslauthd()
do_setup_spamassassin()
do_setup_amavis()
do_setup_fail2ban()
do_setup_dovecot ()
do_setup_kopano ()
do_setup_container() [e.g. persistent state)
do_init ()
At the end of the script:
control section --> validating ENV-VARS with possible setup, if not valid, then throw error on container startup e.g. user dovecot and kopano or cyrus sasl and dovecot sasl. After validating and settngs configuration state of the container do_init()

@tomav i will start here, when we can be sure, whether the pull request of @dennis95stumm, and also my new rebased pull request for the kopano feature has been successfully merged or not. Because both will result in some alterations of the start-mailserver script.

Roger that!

@alinmear tests are missing in your other PR by the way.

We should have a pre function that checks configuration to stop the execution if a configuration is provided:

if FQDN is not provided (already exists at the top of the script)
display configuration (alias provided or not, regexp-alias provided or not, dkim provided or not...) before it starts the server

Let me know if I can help or if you want to talk about that.

Sorry for my absence, but i was very busy.
Today i start working on this. Your last post seems to be clear and a good idea. If i can think of something else i will post here again!

Thank you @alinmear!

@tomav i just made a pull request for this. The next step would be implementing the checks logics you mentioned. I haven't done this in this scope, otherwise the batch size of changes would be that big, that if we mess something else up here it's hard to find where we did so.
I marked some sections with @TODO where we could start with several checks.

@tomav: there is also one very important thing i recognized while working on this. We have to different kinds of environment variables: task based, e.g. should we make a config step like ENABLE/DISABLE_SOMETHING and section based, e.g. what should we do withing that task.
Dealing with this we should make a convention that task based vars should always have the same form and is a conditional for registering a function/task or not (so we don't need to make several unneeded function calls here).
@Josef-Friedrich maybe you could think about this with your suggestions in #297.


libclc: 2015-03-27 -> 0.2.0

Things done:

[x] Tested using sandboxing (nix-build --option build-use-chroot true or nix.useChroot on NixOS)
[x] Built on platform(s): NixOS
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

More
cc @wkennington
Disclaimer: I have no clue about libclc, I just picked the next stable release with almost only contains the llvm bugfix. If this looks good please cherry-pick into release-16.03 as well.

#13559

Any reason to not merge this?


Update network-watcher-visualize-nsg-flow-logs-open-source-tools.md

@mattreatMSFT : Thanks for your contribution to the Azure documentation! The author, @jimdial, has been notified to review your proposed change.

#sign-off


typo: coexsiting -> coexisting

@nschonni : Thanks for your contribution! The author, @charwen, has been notified to review your proposed change.

Merging typo fixes


Upgrade to 1.0.5 (and up) results in ReflectionException on symfony cache:clear

I've git bisected it to this commit.
git bisect bad
339a910a6314ff633ced79e96301fd650d68f18e is the first bad commit
commit 339a910a6314ff633ced79e96301fd650d68f18e
Author: Maciej Malarz<EMAIL_ADDRESS>Date:   Sun Dec 20 23:45:39 2015 +0100
Let ReflectionException bubble

:040000 040000 ef8770485fc75a13bee684e4c010de7fd3fcf1cb 8af82efd24a5b5e445e5c60dcdd1e6615007ba5e M      lib
:040000 040000 72d8c87f84327d1a0401456eaf710ac8862fb2db 78cb287b70fa9a12b979234b628a47c4b8bb8c1a M      tests
It results in the symfony cache:clear not correctly running, so breaking the app.
I'm running with the samlSpBundle, which uses a doctrine mongodb mapping in a seperate file, which works fine in 1.0.4 but breaks in 1.0.5 and up due to this reflectionException bubble.
SSOState mapping:
https://github.com/aerialship/SamlSPBundle/blob/master/src/AerialShip/SamlSPBundle/Resources/config/doctrine-mapping/SSOState.mongodb.xml
I'm not sure if the issue is with the mapping or the way this changed in the bundle.
Let me know what I can do to help resolve this. i want to upgrade to 1.1 for the php7 support, but this is blocking it.

It seems that the mapping in bundle you're linking is not correct, field's fieldName is supposed to point to class' property name while name can be used to change field's name in the database (and linked mapping does opposite).
Disclaimer: we have mistakenly used field/fieldName wrongly ourselves (or at least I did few times) so please do take above statement with grain of salt and if I'm wrong please tell :)

@malarzm That does indeed seem to solve the cache clear issue.
I guess the mapping was wrong all the time, but it was working as i mapped it myself in my child object. When fixing this in the bundle, that is no longer necessary.
So the exception is showing that something was wrong all along.
I'll create a pull request against the Saml bundle, thanks for the fast feedback.
I'm not sure if the mapping is correctly described in the documenntation, but a note about this might be a good idea if it's not :)

Glad it works, I'm marking this as documentation issue then ;)

Fixed in #1447.

Thanks :)


add missing import for signal package

Can one of the admins verify this patch?

Jenkins, test this please.

I guess we need to change the cmakelists as well.

@drpngx Unfortunately, I don't know enough about CMake or the code organization to do this myself yet.

You'll need to add the module:
https://www.github.com/tensorflow/tensorflow/blob/master/tensorflow/contrib/cmake/tf_python.cmake#L467

@drpngx Thanks for the hint!
I noticed that the entry session_bundle/testdata/saved_model_half_plus_two doesn't exist (anymore).
This seems to be some kind of a consistency issue.
Do you think these entries could be put into a file separate from the code?
And would it be possible to programatically check them for validity?

Could you send a separate PR fixing the saved model issue?
/CC: @gunan @yifeif for the consistency check. @Androbin I don't have a great answer to that, but if you can propose something I'm all ears. Right now, CMake has been adding support piecemeal as needed, but as you found out, it's not perfect.

Thanks for the PR!

I agree with @drpngx 's comment that cmake support is not perfect. It is also under contrib, so any contributions there are welcome.
Im not sure about separating those out of the cmake file though. Especially contrib is under very heavy development, and as things are added and removed people just forget updating all references to those.

@drpngx @gunan I get your point. It's just that even documentation changes are run through the entire build process and all the unit tests. And although it's contrib, installing proper testing allows for a more dynamic work flow. And since they are there to grow in a safe environment and maybe eventually get upgraded to core they should not be treated as second-class citizens when it comes to testing.

For multiple reasons, bazel will stay as our primary build system.
cmake wont be promoted to core.
I am not against the test itself, I am happy to accept a test as you described as a contribution, and run it under contrib/cmake. But it is not a priority for us.


Improve Homebrew installation support

Make this repository work with brew tap and use SHA-256 for checksum,
as SHA-1 support is deprecated.

Fantastic! I had no idea that this was all that was required. Thank you.

By adding this repo as a tap, multiple versions could be supported by brew as well?


Upgrade QoL isssues/enhancements

Hi @jasonacox,
I've just upgraded to the latest version via upgrade.sh and hit a couple of issues on the way - most of them due to modifying my own configuration and doing some preliminary development work. Based on these, I came up with a couple of very low priority quality of life enhancements. If you see value in any of these, I'm happy to work up PRs to gradually address them after I've got the first pass cumulative kWh PR in place.

Could we add implement a user generated addition to telegraf.conf in setup and upgrade scripts - maybe appended from a telegraf.local file, which I think works OK with TOML? This would be helpful for maintaining personal customisations such as my three phase power monitoring, and may also be useful when developing new functionality.

I was hit with my rootless docker and grafana permissions issue again. It would be good if we could customise powerwall.yml to replace the default user for grafana (maybe by environment variable)? This problem is mostly mine at the moment, but it'd be nice to sort out.

Minor issue: Should there be a dashboard.json in both the root folder and in the dashboards folder? The one in the root folder has the latest changes to the temperature and sun and moon.

Very minor issue: I got hit with a repeated UID error when importing the new dashboard json while keep the old dashboard in place for comparison. It may be worth changing the UID when new versions of the dashboard are released.

On dashboards, I haven't had time to figure out library panels in detail, but maybe they can be used to provide clean up updates to small elements of the dashboard, and maybe allow easier roll-your-own dashboards?

I think the influxdb.sql import raised errors about CQs already existing - maybe these should be dropped before re-adding?

On dashboards, I haven't had time to figure out library panels in detail, but maybe they can be used to provide clean up updates to small elements of the dashboard, and maybe allow easier roll-your-own dashboards?

I had typed up a reply and then closed the tab... argh! Hopefully typing this again will let me be more organized and concise, but don't bet on it :-)
Library panels could be useful for people making their own dashboards, plus they make it easier for the official project to have multiple dashboard variations for people to choose from. For example we could have a version for Powerwall+ and a version for regular Powerwall... (Powerwall Classic? OG Powerwall? Powerwall-?) and the user could set either one as their home dashboard based on their setup. I also think others might find some value in having multiple dashboards with different levels of detail, or ones showing current data vs. older data. That is how I've set up my Grafana.
I've been using library panels because I run a simplified dashboard set as my 'home' dashboard. I have the big dashboard that is closer to the project's default dashboard linked from there, but I don't really look at the big dashboard that often. More on that below.
On a mostly unrelated tangent... are the electricity prices for the 'savings' panel still hard-coded into the queries? Because that seems like a good place to use Grafana variables. If the variable is coded into the query, it is easy for the user to set and update the variable as needed (although since we're not storing the data, any changes to pricing will change the totals when looking at older data). This is another thing I might find easier to do using my mySQL copy of the data. It should be relatively easy to set up a table there with electricity costs for different time periods, and then get mySQL to return a cost per kWh that is accurate for each date. I may play with that if I get a chance. My panel is set up as a 'cost' panel  instead of a 'savings' panel, since I have a solar PPA and pay for all of my power with one rate for grid and another for solar.
Back from the tangent... here are some screenshots of how I've set up my multiple dashboards:
The dashboard that I have set as 'home' in Grafana fits on one screen without scrolling on most of my devices. It just runs off the bottom of the screen on the laptop I'm using now:

At the top you can see a row with dashboard links. I have that set to show all starred dashboards. When expanded, it looks like this:

So current status is my simplified dashboard, Powerwall Monitor is a customized version of the project's standard dashboard, and I have two others that show data from other sources (one monitors my home automation and raspberry Pi hardware and one monitors my HVAC and some weather data).
The power flow and energy usage are displayed on both my simplified "Current Status" dashboard and on my Powerwall-Monitor dashboard, so they are library panels.
I was thinking of making a new home dashboard with just the data I look at the most from all 4 current dashboards. Any panels that I pull from the other dashboards that aren't already in the library will get put there first, and then added to the new home dashboard.
Just found out that deprecated visualization types don't work as library panels. Time to go down the rabbit hole of trying to convert the Savings (or in my case, cost) graph from Graph(old) to a Bar Chart.

Hopefully typing this again will let me be more organized and concise

I'm familiar with the problem myself :-).

Library panels could be useful for people making their own dashboards, plus they make it easier for the official project to have multiple dashboard variations for people to choose from.

This and your experience lines up with what I was hoping/expecting from the library panels. From what you are saying, it should be possible to transition (slowly) from custom dashboards to a modular panel library with dashboard templates that share elements from this library?
I like your summary dashboard layout! Could it be used as a test bed for this enhancement?

On a mostly unrelated tangent... are the electricity prices for the 'savings' panel still hard-coded into the queries?

Into the grafana queries? Yup. I'm mildly interested in this as well because I'm on a time of use tariff, so my value depends when usage occurs in the day. I'm pretty sure the way to hit this is with influx queries. The price data could be stored as an infrequently updated table in influx, or in grafana if the variables support array forms. This one is down the list of my priorities.

Answering one of my own questions: docker compose provides support for environment variable expansion in the .yml, and allows
specification of these variables in a .env file. https://docs.docker.com/compose/environment-variables/

This and your experience lines up with what I was hoping/expecting from the library panels. From what you are saying, it should be possible to transition (slowly) from custom dashboards to a modular panel library with dashboard templates that share elements from this library?

Yes. I think we could pack all the options for panels into the library and then set up one or more standard dashboards, but users could easily build their own dashboards by adding panels from the library in any combination or arrangement they want. When something needs to be updated, only one edit needs to be made and it affects all the linked panels.

I like your summary dashboard layout! Could it be used as a test bed for this enhancement?

I just did some major revisions to it so it no longer looks like the screenshot above :-) But if @jasonacox is interested in going down this route, I'd be happy to work on it. I can start with the current standard dashboard, update any panels that are listed as "(old)" to current ones, and add them all to the library. I can make two dashboards... one as close as possible to the current standard one, and one 'lite' one, and then see if all of those changes can be merged into the project.

On a mostly unrelated tangent... are the electricity prices for the 'savings' panel still hard-coded into the queries?

Into the grafana queries? Yup. I'm mildly interested in this as well because I'm on a time of use tariff, so my value depends when usage occurs in the day. I'm pretty sure the way to hit this is with influx queries. The price data could be stored as an infrequently updated table in influx, or in grafana if the variables support array forms. This one is down the list of my priorities.

My experience over the last few days says this may be challenging in InfluxDB. The InfluxQL language doesn't allow for 'datepart' type queries. The Flux language reportedly does, but despite working with databases every day, trying to do anything in Flux gives me acid reFlux. I can handle syntax variations between InfluxQL, mySQL, and MSSQL, but Flux is a completely different thing.
If I were going to try what you're describing, I'd definitely copy the data from InfluxDB to mySQL and then handle the cost calculations in mySQL queries. It should be relatively easy to join a table with dates and times defining different rates to the kwh totals (or even the detailed autogen data) using mySQL. I expect it would be much more difficult if not impossible in InfluxDB.

There is a lot here. 😄
BuongiornoTexas - quality of life enhancements?
First, @BuongiornoTexas - these are gold and so are you!  I love your writing...  let me take some of these:

Could we add implement a user generated addition to telegraf.conf in setup and upgrade scripts

Great point - Taking the pypowerwall.env.sample approach we could have telegraf.conf.sample that setup copies to telegraf.conf which is not impacted in the upgrade process.  And YES to submitting a PR if you want to take this on.

It would be good if we could customise powerwall.yml to replace the default user for grafana (maybe by environment variable)?

I know about the env files, but can you give details on how we could do that?  Yes on PR here too if you can figure it out.

I got hit with a repeated UID error when importing the new dashboard json

Great point, we need to change the UID.  Anyone here, feel free to update the UIDs and submit a PR.  But also see my "Dashboard Market" note below...

influxdb.sql import raised errors about CQs already existing

I have that on my list.  If/when we actually change he CQs that will be an issue.  I'm hesitant to do anything much to InfluxDB because @youzer-name loves it so much. 😆  In all seriouness, the urgency on this is low and I prefer to do a > /dev/null 2>&1 on those commands to mute the non-impacting error noise for now.

panels

@youzer-name I'm interested but how do we make it easy and not create a lot of bloat?  I have a slightly different proposal, but still open to the panel idea.  So....
Dashboard Market?
I must say, I love your dashboard @youzer-name !  It occurs to me that we have no good way to share and store community built dashboards.  I suspect the list and creativity will just keep going.  I want to keep the project basic and simple on initial install with a default starter panel like we have.  BUT, I would like to make it easy for the community to browse and shop for other dashboard (panels too?) that are easily imported.
What if we converted the /dashboards folder into a "User Contributed" store of dashboards (and panels)?  It would have a README with screenshots and details for each, including "how to import" instructions.  Community members here could submit their dashboard layouts via pull request.  Thoughts?

Dashboard Market?
I must say, I love your dashboard @youzer-name ! It occurs to me that we have no good way to share and store community built dashboards. I suspect the list and creativity will just keep going. I want to keep the project basic and simple on initial install with a default starter panel like we have. BUT, I would like to make it easy for the community to browse and shop for other dashboard (panels too?) that are easily imported.
What if we converted the /dashboards folder into a "User Contributed" store of dashboards (and panels)? It would have a README with screenshots and details for each, including "how to import" instructions. Community members here could submit their dashboard layouts via pull request. Thoughts?

@jasonacox Having complete dashboards available for download could be very useful, but I still think having at least some of the panels in the library might help keep things easier to maintain. That way if someone wants to let others do updates and improvements, or put another way, if one wants to stay with the 'official' panels,  they can use the library. Then when there is a change made, like adding weather to the energy use panel, any user-created dashboards that include that panel will also be updated. The library would also allow some built-in customization by having interchangeable panels, like a 'cost' panel for people who pay per kWh for their solar with a PPA and a 'savings' panel for people who don't pay for the solar per kWh.
As I see it the Market idea has (at least) two benefits. First, people can share those user-created dashboards, and other people might download them or use them for inspiration for their own custom dashboards. Second, people would be free to go off-script with the dashboards they upload to the Market, and if people (mainly you as the maintainer) like the ideas enough, they could flow back into the main project.
While trying to work around some issues caused by migrating the Graph (old) panels to the new Bar Graph panel... it seems to be impossible to get the numeric values to appear in the legend... I ended up with this (for now).

I'm still not sure if I like the horizontal bars on use and cost, but the one stat panel with house/solar/grid/battery totals replaced 4 different panels that were showing those numbers before.
And related to the topic about possibly changing the CQs or data sampling: I currently have the stat panel pulling the numbers from the integral and the bar graph using the sum of the kwh.http bucket. While they're close, they are not identical. Looking at yesterday's numbers, the integral matches what I get from the Tesla app for solar and grid. The kwh.http exactly matches for Home use, and neither matches the app-reported battery use.
The 'solar' page in my Tesla app shows 7.5 kWh produced yesterday but the page with the Powerwall info shows 6.8 kWh. I assume one number is being reported by the inverter and the other by the Powerwall Gateway. I looked at more days and the Powerwall page was higher on every one I looked at except yesterday. When looking at week or month intervals, the Powerwall page was always higher. Considering that I pay for what is reported on the Solar page, I'm surprised it's showing the lower number. My September bill shows 552.38 kWh, the solar part of the app shows 552.4 kWh, but the Powerwall part of the app shows 575.7 kWh! So in my case it's not worth chasing the difference between the integral and the kwh bucket, because my 'cost' in Grafana will always be off by 5% or so due to the difference between the gateway and the inverter.

From this discussion, it looks as though there are legs to do some form of modularising dashboards - can I suggest we raise it as a separate enhancement discussion and/or issue, rather than keeping it in this grab bag set?
I'll get on to the minor PRs when I get a break from other things in my life.
| I assume one number is being reported by the inverter and the other by the Powerwall Gateway
You'll need to dig into the specifics of your setup to see what is being monitored (system_status, site meters, vitals, and any separate neurio meter if you have one). For instance, I know I have CTs on supply phase, on each solar system and by the looks of things, one on the battery. Site load looks like it is being done by difference. But from my reading, other line ups can be quite different.
I've got a separate issue on reconciling the app and the local powerwall data #87 - adding information on these type of issues is likely to help me with progressing it. But I'm with you on mismatches between the app and the data we can pull directly - losing hair on trying to figure out what is happening there. To date, I have one (1) piece of good news: over a 9 day period, the powerwall CTs pretty well match the solar output my inverters say they are generating - but the app thinks I'm generating much less. Still waiting for my current meters to changed over into a single three phase meter, at which point I'm hoping to do a similar reconciliation on the grid import/export. Unfortunately, I think I'm going to need to spend some time with Teslapy to get to grips with this.

@jasonacox - I'm still learning my way around this - let me know if I have made any mistakes in making the PR.
PR closing this issue implements the following:

Key change: adds compose-dash.sh utility script to run docker compose, as the command line becomes unwieldy with the updates implemented in this PR.
Adds local compose.env file which provides environment variables to docker compose. Currently only implements GRAFANAUSER.
Overrides the telegraf command in docker compose to read additional telegraf .conf files from a config directory. Maps telegraf.local to local.conf in this directory. Can add non-standard measurements and/or use for local development testing without modifying the main development tree.

Setup/upgrade will silently create the local  compose.env or telegraf.local files if they do not exist, and these default settings should work for most users. Setup/upgrade will not modify these files if they exist already.
CAUTION! I've tested setup.sh, but don't know how to test upgrade.sh due to #104.


run var tests

@ivy-lgi please fix the commented checks. I would recommend designing the test so that it is repeatable on failure.


No static ip address after upgrading to 24.01.1

Version
24.01.1
Explain the issue and what happens
I upgraded from GhostBSD 23.10.1 to 24.01.1 via update-station.
On GhostBSD 23.10.1 I configured a static ip address on my lan card (re0) and it always worked very well.
After the upgrade to 24.01.1 i can't assign a static ip address anymore. The lan card only works when I use
DHCP. The rc.conf file wasn't changed from 23.10.1 to 24.01.1
After booting GhostBSD 24.01.1 ifconfig looks like this:
re0: flags=1008802<BROADCAST,SIMPLEX,MULTICAST,LOWER_UP> metric 0 mtu 1500
options=8209b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM,WOL_MAGIC,LINKSTATE>
ether 70:85:c2:ff:71:69
media: Ethernet autoselect (1000baseT )
status: active
nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>
Explain how to reproduce the bug?
Just boot GhostBSD 24.01.1 - no network available - no static ip address assigned to the network interface re0
What is the Expected behavior (if applicable)
after booting GhostBSD 24.01.1 network with a static ip address should be available on interface re0.
Additional context (if applicable)
It seams that networkmanager causes this issue. I was able to reproduce the issue on a machine with FreeBSD 14.0 RELEASE.
Uninstalling or deactivating networkmanager solved the issue. After deactivating networkmanager GhostBSD 24.01.1 provides network services with a static ip address. This is ifconfig after booting into GhostBSD 24.01.1 without networkmanager:
re0: flags=1008843<UP,BROADCAST,RUNNING,SIMPLEX,MULTICAST,LOWER_UP> metric 0 mtu 1500
options=8209b<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,VLAN_HWCSUM,WOL_MAGIC,LINKSTATE>
ether 70:85:c2:ff:71:69
inet 19x.xxx.x.xxx netmask 0xffffff00 broadcast 19x.xxx.x.xxx
media: Ethernet autoselect (1000baseT )
status: active
nd6 options=29<PERFORMNUD,IFDISABLED,AUTO_LINKLOCAL>

I tested that on my desktop, and the static IP works fine.
Was the static IP setup with NetworkMgr?

On the second machine, I still had the same results. I set the static IP with NetworkMgr and rebooted, and everything is working.

That is what NetworkMgr sets in rc.conf.
ifconfig_igb0="inet 19x.xxx.xxx.xxx netmask <IP_ADDRESS>"
defaultrouter="19x.xxx.xxx.xxx"

I normally set it directly in /etc/rc.conf. I also tried to set it in networkmanager but after reboot the static ip-address was not shown anymore.
I have this issue on 2 desktop pc's. On my laptop everything works fine.

I have reproduced the issue after resuming from Suspend.

I might have found the issue. It looks related to auto-switch.py. It is run by devd and was done when GhostBSD was using OpenRC. I do not think we need it with FreeBSD intit. I will do some more testing and ensure the rest works fine.

I have been testing NetworkMgr without auto-switch.py, and the problem I see is when I unplug the wire connection, it does not auto-switch to the WiFi or the other nic. I am still working on a solution.

I experienced a similar problem. After upgrading to 24.01.1 I don't get a IPV4 address via DHCP.
Executing "service dhclient start alc0" solves this problem for me - until reboot.
IMHO networkmgr seems to be the culprit.
In FreeBSD 14.0 I solved the issue by simply deinstalling networkmgr (only wired lan needed).
If necessary, I can provide more details.

The package is being built and will be added tomorrow.

Im having this issue trying to run GhostBSD 24.01.1 in a TrueNAS CORE VM.

I experienced a similar problem. After upgrading to 24.01.1 I don't get a IPV4 address via DHCP. Executing "service dhclient start alc0" solves this problem for me - until reboot. IMHO networkmgr seems to be the culprit. In FreeBSD 14.0 I solved the issue by simply deinstalling networkmgr (only wired lan needed). If necessary, I can provide more details.

This command service dhclient start em0 seems to resolve the issue as well on boot.


Homebrew Installation

I rather prefer Homebrew to install all my packages instead over Mint and of course over manual installation for every update. Could there be a Homebrew option for installation?

Generally speaking, this would be a good idea especially because it would help improve adoption of the tool.

Out of curiosity for your specific use case though, is a Swift Package Plugin something that might work out well for you? You can check out one implementation here: AvdLee/appstoreconnect-swift-sdk#191
I'm also actively looking at other plugin-based workflows that might be useful for CreateAPI users over in #48.

I personally haven't used a Swift Package Plugin yet but definitely take a look at it!

Additionally, I think a --version flag needs to be added.

Additionally, I think a --version flag needs to be added.

👍 Yep that would be useful! I think swift-argument-parser has support built in, we just need to set it on the CommandConfiguration iirc

0.0.5 is now available via Homebrew! Thanks for the suggestion
https://formulae.brew.sh/formula/create-api
➜ brew install create-api
==> Downloading https://ghcr.io/v2/homebrew/core/create-api/manifests/0.0.5
######################################################################## 100.0%
==> Downloading https://ghcr.io/v2/homebrew/core/create-api/blobs/sha256:99d2635
==> Downloading from https://pkg-containers.githubusercontent.com/ghcr1/blobs/sh
######################################################################## 100.0%
==> Pouring create-api--0.0.5.arm64_monterey.bottle.tar.gz
🍺  /opt/homebrew/Cellar/create-api/0.0.5: 6 files, 9.1MB
==> Running `brew cleanup create-api`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).
➜ create-api generate --help
USAGE: create-api generate [<options>] <input>


[Feature Request]Support for LG Ovens?

Any plans for adding oven support to this integration?  I can assist if necessary.  I have a Signature Duel Oven (Gas, Elec).

Does wideq support this?

Potentially it is possible, wideq is the protocol so is just a matter to manage the property that comes from the payload. Real complexity is to implement without having an oven device.
I put this in the todo list, but my priority here is implement controls so to be able to add climate entity and control existing sensors.

Oven now is supported, I close this request


New standardised test results file format?

There's support for the trx file format in the new test runner, but that's been around for quite some time now.
I'd argue it's got some flaws such as:

Lack of extensibility such as adding custom data.
Exception details lack the actual exception type? Providing only message and stacktrace. The type itself seems pretty important?
It's XML - I know it's not necessarily a file designed to be human readable, but with how easy JSON is to use, it could be a better choice for a new test file format.

If you're implementing a new testing platform so to speak, with new runners, and more (extensions etc.), would designing and standardising on a new test report format make sense too?

YES!!! Hum sorry ;) It's on our plan but we had to adjust some priorities and got some pushback. I was planning on looking at other languages to see what they are doing and if their format could either be reused or could be used as inspiration.


Is it worth it to have keyvault common just for a small subset of types like KeyVaultPipeline?

Right now KeyVault Common in C++ has:

SHA Hashing APIs (probably should be moved to core)
Base64 Url (should be moved to core)
KeyVaultException (should be removed)
Unix Time helper (should be moved to core)
Single Page (should use PagedResponse<T> and removed)

If the only piece leftover in KeyVault common is KeyVaultPipeline, do we need that package at all or can we move it to azure-security-keyvault-keys as a private implementation detail/shared header in /src?
https://github.com/Azure/azure-sdk-for-cpp/blob/0874e1c1c9047b9ccf6caac5057f67c9b92144a7/sdk/keyvault/azure-security-keyvault-common/inc/azure/keyvault/common/internal/keyvault_pipeline.hpp#L24-L30
.NET has it in Azure.Security.KeyVault.Keys:
https://github.com/Azure/azure-sdk-for-net/blob/4162f6fa2445b2127468b9cfd080f01c9da88eba/sdk/keyvault/Azure.Security.KeyVault.Keys/src/KeyVaultPipeline.cs#L8-L14
But it also has some source sharing (though still in the Azure.Security.KeyVault namespace):
https://github.com/Azure/azure-sdk-for-net/blob/4162f6fa2445b2127468b9cfd080f01c9da88eba/sdk/keyvault/Azure.Security.KeyVault.Shared/src/KeyVaultPipeline.cs#L10-L12
cc @heaths, @tg-msft, @JeffreyRichter, @RickWinter, @vhvb1989

Its the structure foundation for certificates and secrets packages.

For .NET, the pipeline had functionality that predates much of the shared code in Azure.Core. In fact, much of that code was copied and modified minimally. But I see no reason to remove it from .NET because the methods also give me consistent control over what is classified as errors and better support for our pseudo-LROs. If C++ wants to do it for the same reasons and uses source sharing, I don't see any harm. Is there some reason to otherwise duplicate code across the different packages here?

we can't do shared-code in cpp the way it works on .NET
We prefer shipping and maintaining a common package for all the packages from a service.
This common package should be transparent for consumers (similar to the shared-source from .NET).

Is there some reason to otherwise duplicate code across the different packages here?

Not cool.  That would bring pain and chaos to keep all copies in sync and avoid issues.

It was a pointed question in response to this from the OP:

If the only piece leftover in KeyVault common is KeyVaultPipeline, do we need that package at all or can we move it to azure-security-keyvault-keys as a private implementation detail/shared header in /src?

If you remove the shared pipeline, you'd have to duplicate code or - as you stated - have a shared package. I just know that for all other languages, we've explicitly avoided shared packages. IIRC, Python and JS effectively dup shared code during generation such that, while it's duped, it's done from a single source and the duped source is treated like generated source i.e., don't modify it (so like your _details namespace, I imagine).


WASM project fails to build inside workspace after add dependency

Problem
I am not completely sure if this is a cargo issue but the error only occurs inside a workspace so this was my best bet.
When adding tonic-web-wasm-client to a leptos project the project fails to build as if non WASM supporting dependencies have been included. Where as if I have an identical project outside of a workspace it builds just fine. On the original project where I observed this it only started happening after a recent cargo update if I use the old lockfile the project will build inside a workspace.
A snippet of the  compile errors:
error[E0432]: unresolved import `crate::sys::IoSourceState`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/io_source.rs:12:5
   |
12 | use crate::sys::IoSourceState;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^ no `IoSourceState` in `sys`

error[E0432]: unresolved import `crate::sys::tcp`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/net/tcp/listener.rs:15:17
   |
15 | use crate::sys::tcp::{bind, listen, new_for_addr};
   |                 ^^^ could not find `tcp` in `sys`

error[E0432]: unresolved import `crate::sys::tcp`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/net/tcp/stream.rs:13:17
   |
13 | use crate::sys::tcp::{connect, new_for_addr};
   |                 ^^^ could not find `tcp` in `sys`

error[E0433]: failed to resolve: could not find `Selector` in `sys`
   --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/poll.rs:301:18
    |
301 |             sys::Selector::new().map(|selector| Poll {
    |                  ^^^^^^^^ could not find `Selector` in `sys`

error[E0433]: failed to resolve: could not find `event` in `sys`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/event/event.rs:24:14
   |
24 |         sys::event::token(&self.inner)
   |              ^^^^^ could not find `event` in `sys`

error[E0433]: failed to resolve: could not find `event` in `sys`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/event/event.rs:38:14
   |
38 |         sys::event::is_readable(&self.inner)
   |              ^^^^^ could not find `event` in `sys`

error[E0433]: failed to resolve: could not find `event` in `sys`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/event/event.rs:43:14
   |
43 |         sys::event::is_writable(&self.inner)
   |              ^^^^^ could not find `event` in `sys`

error[E0433]: failed to resolve: could not find `event` in `sys`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/event/event.rs:68:14
   |
68 |         sys::event::is_error(&self.inner)
   |              ^^^^^ could not find `event` in `sys`

error[E0433]: failed to resolve: could not find `event` in `sys`
  --> /home/jmintb/.cargo/registry/src/github.com-1ecc6299db9ec823/mio-0.8.6/src/event/event.rs:99:14
   |
99 |         sys::event::is_read_closed(&self.inner)
   |              ^^^^^ could not find `event` in `sys`

Steps
This example works best on nightly as leptos requires that by default, how error you should end up with the same error output as me in step five on stable either way.

create an empty directory.
Add a Cargo.toml file with a single workspace member:

[workspace]
members = ["testproject"]

Use cargo-leptos to create a new project inside the workspace.
cargo leptos new --git leptos-rs/start-axum
At this point you should be able to run cargo build --target wasm32-unknown-unknown and everything builds correctly(or see a nightly related error on stable)
Add tonic-web-wasm-client to the testproject's dependencies:
cd testproject && cargo add tonic-web-wasm-client
Run cargo build --target wasm32-unknown-unknown again and now the build should fail with error output resembling what I shared in the problem description.

Possible Solution(s)
No response
Notes
No response
Version
cargo 1.68.0 (115f34552 2023-02-26)
release: 1.68.0
commit-hash: 115f34552518a2f9b96d740192addbac1271e7e6
commit-date: 2023-02-26
host: x86_64-unknown-linux-gnu
libgit2: 1.5.0 (sys:0.16.0 vendored)
libcurl: 7.86.0-DEV (sys:0.4.59+curl-7.86.0 vendored ssl:OpenSSL/1.1.1q)
os: NixOS 22.5.0 [64-bit]

After further investigation I have found the commit in leptos that caused this issue to start occurring: https://github.com/leptos-rs/leptos/commit/29fb1842a5ad62e61913317ed5c53a25de7ab9c3 . From my local experimentation it looks like it is the changes to server_fn/Cargo.toml which cause the error to occur. Again only inside a workspace.

It turned this was related to features working differently in workspaces. Setting resolver="2" allows the build to succeed. my bad :facepalm:


ci: pin third party actions

Description of changes
Improves reproducibility of CI tasks, especially since actions often move their tags, resulting in foo@vN being equivalent to<EMAIL_ADDRESS>Dependabot knows how to update pinned dependencies and their comments, so there should be no overhead from this change.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

Previous PR: https://github.com/NixOS/nixpkgs/pull/140301

If we're going to merge this we may as well pin all of the actions so the dependabot updates are consistent.

If we're going to merge this we may as well pin all of the actions so the dependabot updates are consistent.

Whoops, turns out npx pin-github-action --comment=' {ref}' .github/workflows/*.y*ml failed, but for file in .github/workflows/*.y*ml; do npx pin-github-action --comment=' {ref}' "$file" || echo "$file"; done seems to have processed all of them.

Ok. Let's try this out.


pls make it for 1.21.1

Dude, just check pull requests. They're probably working on this just right now.


Data Explorer Discussion

Description
Tool for exploring MAPS food and micronutrient data more freely.
Screenshot

Data Views

Links
Miro board
Figma prototype
User stories

I would like to see in which nations a specific micronutrient is under the recommended intake level.

Questions

Should users be able to select multiple nations at once?
Should it be possible to select multiple food items or micronutrients at once?

Thank you so much!
These tools are perfect to prioritise our needs here - the table is especially useful as a means to capture that.


Add component type attribute so that renderqueue can choose which com…

…ponent type needs optimisation
PLEASE MAKE SURE YOU READ CONTRIBUTING GUIDELINES and CODE OF CONDUCT.
UNLESS IT IS A VERY SMALL CHANGE, CREATE AN ISSUE FIRST FOR DISCUSSION.
** DON'T FORGET TO ADD YOUR CHANGES TO CHANGELOG.md under
proper category **
i.e:
## master

### Features

- [#ISSUEID](https://github.com/trivago/melody/issues/000) Added coffee brewing feature

REMOVE THIS AND EDIT THE TEMPLATE BELOW

Reference issue: #110
What changed in this PR:
Write the description and changes here...

Pull Request Test Coverage Report for Build 625

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
8 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.005%) to 89.527%

Files with Coverage Reduction
New Missed Lines
%

packages/melody-idom/src/core.ts
8
92.0%

Totals

Change from base Build 481:
0.005%

Covered Lines:
2159

Relevant Lines:
2288

💛  - Coveralls


Docker status showing "health: starting" continuously

Hello Team,
I have launched asia docker instance using below commanda then docker started successfully.
sudo docker run
-e OVERPASS_META=yes
-e OVERPASS_MODE=init
-e OVERPASS_PLANET_URL=file:///db/asia-latest.osm.bz2
-e OVERPASS_RULES_LOAD=0
-e OVERPASS_UPDATE_SLEEP=36000000
-v /osm-services/asia/overpass_db/:/db
-p 10001:80
-i -t -d
--name overpass_asia wiktorn/overpass-api
But when I issued a command  #cat db/nodes.bin in shell terminal to see the content of "nodes.bin" under db then the docker status showing the "health: starting" continuously.
Please let me know what went wrong? what cases "health: starting" will come? how to fix it?y
when I see the docker logs, it shows "No database directory. Initializing"

The message No database directory. Initializing indicates that the container just started initializing the database. For whole Asia contintent, it may take a few days, depending on the hardware you're using for this.


gh: 2.37.0 -> 2.38.0

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Hey,
this is my first contribution. I hope I get it right. I saw that you build it under aarch64-darwin and since I have a x86_64-linux machine Im posting my results. Is this how it works?
Result of nixpkgs-review pr 264821 run on x86_64-linux 1

1 package built:

gh (github-cli)

@giomf, Welcome and thanks for your contribution🙂 Yes, what you posted—apart from the initial question—is what other contributors have posted on other PRs.
Thanks for merging, @zowoq, much appreciated 🙏


[QUESTION] Not sure how to overwrite Accept header for health checks?

Question
Right now health checks do not pass for Vaultwarden

Tried to debug the health check via:
curl 'https://dashy.local/status-check/?&url=https%3A%2F%2Fvaultwarden.local' -H 'User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:109.0) Gecko/20100101 Firefox/113.0' -H 'Accept: application/json, text/plain, */*' -H 'Accept-Language: en-US,en;q=0.5' -H 'Accept-Encoding: gzip, deflate, br' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Referer: https://dashy.local/' -H 'Cookie: dashyAuthToken=xxx' -H 'Sec-Fetch-Dest: empty' -H 'Sec-Fetch-Mode: cors' -H 'Sec-Fetch-Site: same-origin' -H 'Sec-GPC: 1' -H 'TE: trailers'
This resulted in the same error that was in the browser.
{"successStatus":false,"message":"❌ Service Unavailable: vaultwarden.local resulted in ENOTFOUND (-3008)"}
Then what I did was trying to reduce the number of headers that are sent and found out that if I change the Accept header from application/json, text/plain, */* to just */* I would get to the Dashy HTML page instead of status.
Anyone used healtchchecks on Vaultwarden?
Category
App Usage
Please tick the boxes

[X] You are using a supported version of Dashy (check the first two digits of the version number)
[X] You've checked that this question hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

same issue here, I set Access-Control-Allow-Origin * problem still there


Let Category support an additional description

As of 2.0.2 the Category class holds a simple description and a value. It would be great if an additional description could be added to a particular Category, see attached screenshot from IntelliJ

Here I have the same answer as I wrote in #23, feel free to implement if you want and submit a PR!

Here also my question like in #23 why the misleading naming betwenn the parameter and JavaDoc.


Store artifacts to existing bucket

Hello @DougTrajano,
I want to use the existing bucket to store the artifact. But when I run:
terraform apply -var mlflow_username="admin" -var mlflow_password="12345" -var artifact_bucket_id="mlflow-artifact-feeder"

I got below error:

Would you have any idea of why this happens ?
Thank you.

Hey buddy! It's an issue on the Terraform stack, I'll fix it ASAP.


BF: mv overwrote target

Moving/renaming assets/directories did not always check for the target to exist and could silently overwrite.
Won't address coverage herein. That's part of #487.
Closes #459

LGTM; I will merge it.


feat: clear all tags in one click button added in search page

Fixes Issue
Closes #9839

Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots
clearfilter.webm

Hello! It looks like the only file with relevant changes for this issue is search.js
Can you please remove tags and your package lock file from this PR?

done

Hello! During testing the clear function worked well if I used tags only. If I type in my search it would remove all values but the first word. I have created a video to illustrate what I mean.

https://github.com/EddieHubCommunity/BioDrop/assets/97615019/6ff30429-56de-437e-8099-c63d07c1cdfb

Hello! During testing the clear function worked well if I used tags only. If I type in my search it would remove all values but the first word. I have created a video to illustrate what I mean.

I've fixed this issue


Add missing sessions test

@kevin-bates and @echarles Here are the missing sessions tests left out of #152.

Test on my mac and tests passed. We have 1 more warning.
LGTM - I guess we can merge when Zach will have pushed the change he was not able during his fly. We can also come back on the async fixture later.

Thanks, @kevin-bates!


Add data-sharing with security incompatiblity warning

Description

Add missing data-sharing with security incompatiblity warning, both in security and data sharing sections.

@Mergifyio backport 3.1.x 3.0.x 2.14.x 2.10.x 2.6.x

Contributor Checklist

[ ] Commit messages follow the project guidelines.
N/A Code snippets related to the added documentation have been provided.
[x] Documentation tests pass locally.
[x] Applicable backports have been included in the description.

Reviewer Checklist

[x] The PR has a milestone assigned.
[x] The title and description correctly express the PR's purpose.
[x] Check contributor checklist is correct.
[x] CI passes without warnings or errors.

@Mergifyio backport 3.1.x 3.0.x 2.14.x 2.10.x 2.6.x


Move catalogs out of this template?

It might not be wise to have the catalog live in this repo. I propose we add them to https://github.com/pangeo-data/pangeo-datastore and then reference that from here.

We currently have the collections living in https://github.com/NCAR/intake-esm-datastore and duplicated here for (1) purposes of reproducibility and (2) because Cheyenne batch nodes do not have Internet access—so users there will need a local copy.

I see your point. But I'm worried that having a relative path in the notebook with hinder rather than help reproducibility (also related to #8).
Could we do the following?

pick a path on glade for the official catalog location and use an absolute path in cell 5 for the glade catalog
reference https://raw.githubusercontent.com/NCAR/intake-esm-datastore/master/catalogs/pangeo-cmip6.json in cell 5 for the cloud catalog

I'm willing to drop this if we can add some text and links to the hello_world notebooks that explain where to find the official versions of these catalogs ( https://raw.githubusercontent.com/NCAR/intake-esm-datastore).

I am happy to make these changes.
We have the catalogs on glade and could use this path:
/glade/collections/cmip/catalog/intake-esm-datastore/catalogs
but that's outside the project's direct control (of course the actual data is too), so it's possible to change files at that location without those changes being reflected in the project—and thus breaking reproducibility.
I was thinking about using git subtree to manage the catalogs subdirectory in the project. That doesn't solve the relative path issue, but would preserve the link to the primary source. Could be support by an update_catalogs script, for instance.
What do you think?

No to git subtree. Let's keep things as simple as possible.

Yeah...that's why I haven't pursued it yet.


[ZEPPELIN-4759] Paragraph refreshing make the other paragraph shaking

What is this PR for?
This PR just resolve the paragraph shaking issue. Before this PR, I will refresh paragraph output by 2 steps: step 1. clear previous output, step 2. update paragraph with new output.
After this PR, I will just make it as just one step: update paragraph with new output. The main change is o method  InterpreterOutput#clear. I add one flag to indicate whether it should send the update to frontend to refresh the output.
What type of PR is it?
[ Improvement ]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-4759

How should this be tested?

CI pass

Screenshots (if appropriate)

Before

After

Questions:

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

This is a great fix, I find it very disjointing when I edit a notebook with someone else, and the repainting of the screen when they make a change on their end is very jarring!

@epugh This is only to fix the issue happens in flink interpreter where I refresh paragraph when new data is coming from flink stream. I don't think it would fix the collaboration scenario you mentioned. Maybe you can create a ticket, we can fix it in other PR.


Powr2 stats

Hi,
Can you maybe add stats support for Powr2?
https://developers.home-assistant.io/docs/core/entity/sensor/#long-term-statistics
Best regards
Hyper

I wanted to add Powr2 to the energy dashboard as supply device but it is not available there. The app tells to send the above link as information.

I have same issue. As i understood, integration is not giving information properly to energy dashboard. My sensor is this :
https://itead.cc/product/sonoff-pow-elite-smart-power-meter-switch/
What information do you need ?

Actually this worked out with Riemann sum integral helper :)


ibeo_core: 2.0.0-0 in 'lunar/distribution.yaml' [bloom]

Increasing version of package(s) in repository ibeo_core to 2.0.0-0:

upstream repository: https://github.com/astuff/ibeo_core.git
release repository: https://github.com/astuff/ibeo_core-release.git
distro file: lunar/distribution.yaml
bloom version: 0.6.6
previous version for package: null

Holding for Lunar sync


See if we can use any new features from Kubernetes 1.4

Relevant release post: http://blog.kubernetes.io/2016/09/kubernetes-1.4-making-it-easy-to-run-on-kuberentes-anywhere.html
For starters, seems like it's now very easy to deploy Kubernetes on a cloud provider.  We can potentially "bundle" pachyderm with kubernetes such that a user can just deploy pachyderm on any cloud, without knowing kubernetes.

We're using k8s 1.4 at this point. I didn't see anything specific in that link about making deployment easier.
We use kops + pachctl deploy now which gets us pretty far. Is there something more specific you had in mind @derekchiang ?

I'm going to close this since it's pretty old and we have a number of specific issues about new k8s features we want to use.


Add apt-get packaging and upload to repo

The action will now create the .deb file and upload it to the apt repo that is hosted on the pyrsia/website repo as GitHub Pages. The package creation/repo upload only happens when the Action is running on the pyrsia/pyrsia repo and not a fork.

I also discovered https://crates.io/crates/cargo-deb

This is a Cargo helper command which automatically creates binary Debian packages (.deb) from Cargo projects.

Did you see this when you looked into this?

I did not see the cargo-deb.  Let me take a look at that to see if it simplifies some of the steps.

I did not see the cargo-deb. Let me take a look at that to see if it simplifies some of the steps.

Let's make it a new issue? a follow up to this? I'd love to have this in main so we can workout the kinks

https://crates.io/crates/cargo-deb

See issue #129

One last comment (and the PR CI will pass) https://github.com/pyrsia/pyrsia/pull/116#discussion_r770913928

Does this also close #59 ?

Yes

https://github.com/pyrsia/pyrsia/runs/4609127711?check_suite_focus=true#step:3:26
Failed check is fixed in #140


Update LICENSE

Update the year
Hello! I am Maxim Zavalniuk and I am a participant in Hacktoberfest 2021!
You can do everything you want with this pr, but please don't close it and don't report on me as spamer

If you accept this pr, please approve it, or merge, or give the 'hacktoberfest-accepted' label


Moving everything to typescript

What does this PR do?
Move all the files to typescript
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you successfully ran tests with your changes locally? Including container validation.
[x] Have you lint your code locally prior to submission?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you labelled this PR correctly? E.G. major, milestone, breaking, breaking-change, minor, feature, enhancement, patch, fix, chore, refactor etc

Ah just realized I should have waited to make the detector -> plugin changes until after this merged...   Maybe we do these filename changes at the end of a sprint once folks have their stuff merged in?  Incl mine and ranis PR.

could also run prettier on all tsx files at that point


Add Faraday::VERSION and add @yykamei to CODEOWNERS

I got hit with an error of Faraday::VERSION is missing when I run steep check on my gem.
Faraday::VERSION is defined here.
https://github.com/lostisland/faraday/blob/4024f4d4029e090e533c68c3f933c7509280592d/lib/faraday/version.rb#L4
By the way, I'm interested in maintaining Faraday signatures, so I also updated CODEOWNERS.

Thank you!


iris load documentation - clarify constraint as string

Hello, this is a small documentation modification.  I think if a constraint is simply a string it matches against cube.name() rather that the standard_name (is that right?)
I'm not completely sure the documentation is the best as I have it - feel free to reject if you think it could be clearer.
Sorry for these small pull requests - but I figure they are more likely to get done if I just do them as I come across them, rather than saving them up.

Sorry for these small pull requests - but I figure they are more likely to get done if I just do them as I come across them, rather than saving them up.

Small PRs are far easier to merge than larger ones. :+1:

Small PRs are far easier to merge than larger ones.

:+1: :+1:


feat(tutorial): complete step 2 endicott

Closes #
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Looks good!


13914 use read/write locking on Windows

Fixes #13914
The "SRWLock" synchronization primitive is available in Windows Vista
and later.  CRYPTO_THREAD functions now use SRWLock functions when the
target operating system supports them.
CRYPTO_win_rwlock::mode keeps track of the last lock mode acquired by
any thread. It is used again in CRYPTO_THREAD_unlock to call the correct
ReleaseSRWLock function. This should be thread-safe because multiple
reader threads will all write the same value, SRWLOCK_MODE_SHARED, and
only a single writer thread will ever set this to SRWLOCK_MODE_EXCLUSIVE
Checklist

[ ] documentation is added or updated
[ ] test_encoder_decoder_legacy.t currently hangs. Either #14319 or #14171 will fix.

Should this have a CHANGES entry?

Once #14319 is merged (in a day or so), you should be able to rebase this and remove the Merge branch..
Rewording the commit message of the first commit to just say...
Use read/write locking on Windows

Fixes #13914

The "SRWLock" ....

Should this have a CHANGES entry?

Yes - I think a CHANGES.md entry is warranted for this.

Once #14319 is merged (in a day or so), you should be able to rebase this and remove the Merge branch..

Yes... the merge commit was a failed attempt to replace the first commit, which was misattributed to my work email address. I'm not quite sure how to fix this. Re-pushing the remote branch comes to mind, but I thought that might lose review history. I do intend to clean this up before merge; I'll need to learn a bit more git before I can, though.

Once #14319 is merged (in a day or so), you should be able to rebase this and remove the Merge branch..

Yes... the merge commit was a failed attempt to replace the first commit, which was misattributed to my work email address. I'm not quite sure how to fix this. Re-pushing the remote branch comes to mind, but I thought that might lose review history. I do intend to clean this up before merge; I'll need to learn a bit more git before I can, though.

Do not worry. Just force push with clean commits.

#14319 just got merged...

#14319 just got merged...

Thanks, I'll rebase, have another go at deleting the first commit (causing the CLA violation) and force-push the branch.

I guess this PR can be taken out of "draft" now.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Pushed! Thank you for your contribution!!


Refactor StatusCodeTraits and HTTPMethodTraits

In my opinion, a single Traits should be enough which would check both http methond and response status code.

Migrated to apiaryio/drafter#643


5.45x39mm BP gs ammo pack (30 & 120pcs) should be blacklisted from flea

General information:
Project issue is for:
Server
Expected result:
BP ammo boxes should not be flea accessible (just like rounds)
bandit checked live (as I don't have a live account this wipe) and verified no BP boxes, wiki & eft-ammo have them as blacklisted as well
Actual result:
Both 30 pcs and 120 pcs boxes are being listed on flea by player offer
How to reproduce:

Search 5.45x39mm BP on flea
See & buy boxes from player offers

Attachments

server-2024-02-08.log (168 KiB)
LogOutput.log (75 KiB)
2024.02.09_0-45-52_<IP_ADDRESS>.28476 traces.log (157 KiB)
65bd5a3e0005975885125892.zip (92 KiB)
2024-02-08[19-39] (0).png (2497 KiB)

Originally written by acidphantasm

The 120 pack can be sold on flea, its just very rare:

https://tarkov.dev/item/545x39mm-bp-gs-ammo-pack-120-pcs

Attachments

image.png (20 KiB)

Originally written by chomp

Made some changes that should help: https://dev.sp-tarkov.com/SPT-AKI/Server/commit/b7b08f99f26334808eb1f4c6da01c84e8a9edaf0
This system can be used for other items. Currently only used for the 2 mentioned ammo boxes.
I chose the multipliers to get the prices close to what was shown in tarkov.dev data.

Originally written by chomp

5.45x39mm BP gs ammo packs look much more expensive when I check compared to OP's screenshot!

Attachments

2024-02-09[14-13] (0).png (3450 KiB)
2024-02-09[14-12] (0).png (3426 KiB)

Originally written by Reanimate


[GSSOC'23] Add Network Packet Sniffing using Python

Aim
This script aims to sniff a network packet to get to know the conversation happening or the messages being communicated.
Details

What the features will your script have
1.) Performs Network Packet sniffiing
2.) The Process will be explained in detail.

Do I want to work on this:

[✔]  Yes

Kindly Assign me this issue @avinashkranjan

@avinashkranjan kindly assign me this issue

Go Ahead @killerraj369


tls.PrivateKey not encrypted

I know this is the same with Terraform, but having Pulumi use the stack's encryption would provide some extra security, even if it a shared passphrase.
In other words, I would suggest the effect of this be the default.
import * as tls from '@pulumi/tls'

export const cert = new tls.PrivateKey(
  'cert',
  {
    algorithm: 'ECDSA',
    ecdsaCurve: 'P256',
  },
  {
    additionalSecretOutputs: ['privateKeyPem'],
  }
)

I would like to use something like Bitnami Sealed Secrets for this so that the encryption is effectively one-way on the developer machine. That would require somehow using just the provider and passing that to a command before saving it.

In other words, I would suggest the effect of this be the default.

This is tracked by https://github.com/pulumi/pulumi-terraform-bridge/issues/10.

I would like to use something like Bitnami Sealed Secrets for this so that the encryption is effectively one-way on the developer machine. That would require somehow using just the provider and passing that to a command before saving it.

Can you elaborate on this a bit more?

Sure. I would like the option to use public/private crypto rather than effectively a shared passphrase. So a developer could encrypt a secret but then could not decrypt it.
Like how one cannot retrieve an app specific password from an email client after entering it.
Sealed Secrets handle this for k8s. But to make it seemless, pulumi would provide an API to detect and then use the generated secret created with the same name when a sealed secret is created (and thus decrypted by k8s).

Going to close this out, we do encrypt this by default now
     Type                     Name                     Status
 +   pulumi:pulumi:Stack      test-tls-privatekey-dev  created
 +   └─ tls:index:PrivateKey  cert                     created

Outputs:
    cert: {
        algorithm                 : "ECDSA"
        ecdsaCurve                : "P256"
        id                        : "37babe450b9038dd3722cae6d7528ff0e0133285"
        privateKeyOpenssh         : "[secret]"
        privateKeyPem             : "[secret]"
        publicKeyFingerprintMd5   : "72:68:86:5d:10:eb:32:a3:c6:04:45:fc:b3:de:e3:19"
        publicKeyFingerprintSha256: "SHA256:LN1Go/MedU0JbbSjCXynR70inRgDanV3gl8M83O1e5c"
        publicKeyOpenssh          : "ecdsa-sha2-nistp256 AAAAE2VjZHNhLXNoYTItbmlzdHAyNTYAAAAIbmlzdHAyNTYAAABBBDADfkEIqFDLilF/oOcbY2zNV/uiFrLd24UXhRBXt3yhHkEqgdUHEaNq3asLqDpuzYRxlDop5ZiZ+pIWdBaI9OA=\n"
        publicKeyPem              : "-----BEGIN PUBLIC KEY-----\nMFkwEwYHKoZIzj0CAQYIKoZIzj0DAQcDQgAEMAN+QQioUMuKUX+g5xtjbM1X+6IW\nst3bhReFEFe3fKEeQSqB1QcRo2rdqwuoOm7NhHGUOinlmJn6khZ0Foj04A==\n-----END PUBLIC KEY-----\n"
        rsaBits                   : 2048
        urn                       : "urn:pulumi:dev::test-tls-privatekey::tls:index/privateKey:PrivateKey::cert"
    }

Resources:
    + 2 created

Duration: 6s