Can one ScrapySplash server handle multiple SpashRequests from multiple Spiders?

I have a Spider that crawls multiple URLs (100+) and I plan using both Requests and SplashRequests as the website needs JavaScript rendering. And, I am thinking of running Spiders simultaneously for the URLs.
Please note that my Spider does a lot of series of callbacks.
Is ScrapySplash capable of handling the simultaneous requests that can come from the multiple Spiders?
That is, even if there is only on Scrapy-Splash running (via Docker) on localhost:8050? If yes, how can I implement such or are there better alternatives?

This probably makes more sense in https://github.com/scrapinghub/splash, since it is about the server, and not the Scrapy plugin to use the server easily from Scrapy.

This probably makes more sense in https://github.com/scrapinghub/splash, since it is about the server, and not the Scrapy plugin to use the server easily from Scrapy.


Search/Filter Feature

Implement search (filters based on name, friendly_name & comment)
Found some buttons in dialogs didn't get the text style from the dark mode refactoring
Tweak the status logic so it has a offline status rather than no status at all
Change the footer so web.esphome.io does not have a strange vWEB Documentation link

Some screnshots of search bar:

Get stack trace when searching. Searching is also very slow.

There should be a message shown when the search didn't match any devices, now the page is just empty.

Can we move the search into the top toolbar, just icon only, and have the search overlay the toolbar when active. That way it doesn't take up a lot of space when not used.
Search icon could still be shown next to the overflow menu icon on mobile.

@balloob

Get stack trace when searching.

Can I add source maps so we get a more useful info... dunno where that is...

Searching is also very slow.

I'll see what I can do to improve it - I think Lit is not rendering immediately on the change of the search input... (I am used to Angular)

There should be a message shown when the search didn't match any devices

Will add message

Can we move the search into the top toolbar, just icon only

I can, but wouldn't that UX be potentially confusing if the input is hidden but there is a value that is filtering the results... (also being selfish that's more work than I bargained for :wink:)


"Location History" Link is Broken

When viewing an appeal, Clicking the "Location History" link brings you to an Access Denied XML page that's missing a stylesheet.
Example Appeal: https://www.phila.gov/li/appeals-calendar/#/appeals/36957
The Location History link points to: http://li.phila.gov/&_ga=2.214431711.935618296.1562084219-320138463.1551365272#summary?address=7828%20CHELWYNDE%20AVE
Which looks like:

Thanks for reporting!
This was an issue with our Google Analytics setup and has now been resolved.


🐛: PBI wrapper - Access denied message is wrong

Description
If user does not have access to a report, the wrapper generates this error message, asking the user to try again.

Reproduce
Open a PowerBI report you do not have access to.
Expected behavior
Error message saying you do not have access.
Fusion Core reference

https://statoil-proview.visualstudio.com/Fusion Core/_workitems/edit/47586

In addition to making the error message more accurate, we also want the ability to customize the error message to explain users how they should get access. We have this functionality in the old fusion power bi wrapper.

@Htonning  : That is covered by issue: #551

@atlewee The error message showing comes from the fusion core legacy powerbi wrapper not from workspace.

I have contacted Core Team Lead @EirikHovland.

Updated with reference to Fusion Core https://statoil-proview.visualstudio.com/Fusion Core/_workitems/edit/47586.

Keeping alive to track it.

@ken-mellem  - Hans has asked for more information in the DevOps case. Can you or someone from your team provide more details?

It should be mentioned in the OP that this is for the new fusion wrapper which we primarily use in the Project Portal - not the old Fusion PBI wrapper typically used in the Fusion portal. I suspect he is testing the wrong thing.


fix(hotfix): try to fix CI

Description
Proper metadata for packages to ensure a proper publication over crates.io
Notes
N/A
Links to any relevant issues or pull requests
N/A
Open Questions
N/A
Change checklist

[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works

No hotfix/bump needed, closing PR.


feat(mapzen map): basic map with center loader

basic map implementation.

No need to merge. All the final implementation of leaflet map is in PR #34.


Use a Railtie to extend Rails

Railties are the core of the Rails framework and provide several hooks to extend Rails, using Railties is the 'Rails-way' to introduce extensions/patches to Rails libraries/components.
Based on this I'm going to add hooks to introduce a new renderer for Action Controller as described in https://github.com/Netflix/fast_jsonapi/issues/175

@guilleiguaran curious if you're still following this up with those hooks? From your comments on #175 and #89, this sound promising for smooth integration in controllers.


Link concealing?

I've been using plasticboy/vim-markdown for markdown syntax highlighting and other stuff for years.
I really misses the link concealing feature of that plugin when I switched to tree-sitter-markdown today.
Link concealing shows [text](link) as text and make the whole document less cluttered, especially if the link is very long.
Not sure if link concealing is within the scope of tree-sitter.

You could try additional_vim_regex_highlighting = true as documented here. But that's just a workaround that enables the traditional vim syntax in the background, so you will get no performance benefit from using tree-sitter.

Ok, thanks.

so have you achieve concealing? I tried additional_vim_regex_hightlighting and it doesn't work for me

@Liverm0r
No, I went back to plasticboy/vim-markdown. Because I need link concealing, and I can live with the not-so-good performance.

By the way, before treesitter and lsp I was using vim-polyglot, and it was also concealing .md links.


UE edit of Razor file compilation doc

Fixes #5408
Changes

Make the doc unavailable for ASP.NET Core 1.0, where precompilation isn't supported
Replace tabs with versioning monikers
Move an unused .csproj file from the sample folder to a samples folder for the metapackage doc using it
Address Acrolinx suggestions, Gauntlet warnings, and Markdownlint warnings
Add headings to improve readability
Add an Additional resources section

Internal Review Pages

ASP.NET Core 1.1
ASP.NET Core 2.0
ASP.NET Core 2.1

@pranavkm  asks

Out of curiosity, do these appear like tabs?

See Internal Review Pages at the top of this issue for links you can see the build.


[feature] Complex filters for Consul discovery

Proposal
In current Consul SD configuration there ability to filter services by tags or node metadata. Fragment of docs:
# An optional list of tags used to filter nodes for a given service. Services must contain all tags in the list.
tags:
  [ - <string> ]

# Node metadata key/value pairs to filter nodes for a given service.
[ node_meta:
  [ <string>: <string> ... ] ]

I have some metadata keys on which I want to do discovery at the same time. For example, I have key env with values dev and stage. How I can filter this services in one Prometheus job? I am forced to use 2 jobs with different filters.
First job:
node_meta:
  env: stage

Second job:
node_meta:
  env: dev

It would be more convenient to add the ability to use more complex filters. Consul API already supports filters with multiple operators: https://www.consul.io/api-docs/features/filtering

Prometheus enables you to do this in multiple ways. Can you please provide me the relevant part of your config so I can help you and explain.

- job_name: stage-exporters
    consul_sd_configs:
      - server: localhost:8500
        node_meta:
          env: stage
        services:
          - haproxy_exporter
          - node_exporter
        refresh_interval: 15s
    relabel_configs:
       ...

- job_name: dev-exporters
    consul_sd_configs:
      - server: localhost:8500
        node_meta:
          env: dev
        services:
          - haproxy_exporter
          - node_exporter
        refresh_interval: 15s
    relabel_configs:
       ...

Option 1
- job_name: stage-exporters
    consul_sd_configs:
      - server: localhost:8500
        node_meta:
          env: stage
        services:
          - haproxy_exporter
          - node_exporter
        refresh_interval: 15s
      - server: localhost:8500
        node_meta:
          env: dev
        services:
          - haproxy_exporter
          - node_exporter
        refresh_interval: 15s
    relabel_configs:
       ...

Option 2
- job_name: stage-exporters
    consul_sd_configs:
      - server: localhost:8500
        services:
          - haproxy_exporter
          - node_exporter
        refresh_interval: 15s

    relabel_configs:
       - source_labels: [__meta_consul_node_metadata_env]
         regex: 'dev|prod'
         action: keep
       ....

Oooops
The prefered way would be option 2 but we do not expose the node metadata as labels. I think we should look into supporting that if that does not require new API calls.

So option 2 was what I was mainly thinking about but we miss some labels :)

@roidelapluie
Thank you very much! I will use option 2 :)

But it does need not work. I think we do not have the labels required yet.

It,s working :) You have a typo in the label name.
My config:
scrape_configs:
  - job_name: exporters
    consul_sd_configs:
      - server: localhost:8500
        services:
          - haproxy_exporter
          - node_exporter
    relabel_configs:
      - source_labels: [__meta_consul_metadata_env]
        regex: ^(stage|dev)$
        action: keep

__meta_consul_metadata_<key>: each node metadata key value of the target
Indeed you are correct.
I will close this issue then, as this is better than two separate scrape configs - because it lessens the load on consul.


MetaData Tags merging

Tags set was always empty if tags from md2 are absent (even if set of tags from md1 is not empty)


Step 1 has inconsistent Button imports

The code snippet in https://www.carbondesignsystem.com/tutorial/react-step-1 that says
import Button from 'carbon-components-react/lib/components/Button/Button';

needs to be updated to how we previously imported Button as
import { Button } from 'carbon-components-react';

I get this error when import the Button from carbon-components-react, but it pops up carbon-icons issue,   I'd like to know what's happening, thanks!

Failed to compile
./node_modules/carbon-components-react/es/components/Icon/Icon.js
Module not found: Can't resolve 'carbon-icons' in '/Users/chenzhengwei/Documents/CMDP/sourcecode/carbon-tutorial/node_modules/carbon-components-react/es/components/Icon'


Multiple beacon advertisement fix

Utilize arguments map for this. not perfect but it does the trick just great.
Deny advertisebeacon if one was sent less then 5 blocks away

This would close #265

yup was waiting on suggestions. but the general idea here does work (tested) 👍 will make changes later busy on windows dealing sb stuff


[BUG] Error thrown when user reviewer comments after a merge

Describe the bug

PR author signs CLA (https://github.com/Expensify/expensify-common/pull/327#issuecomment-769344096)
Reviewer approves PR (https://github.com/Expensify/expensify-common/pull/327#pullrequestreview-581568256)
Reviewer merges PR (https://github.com/Expensify/expensify-common/pull/327#event-4280480581)
Reviewer comments on PR (https://github.com/Expensify/expensify-common/pull/327#issuecomment-771792976)
Error is thrown on that comment (https://github.com/Expensify/expensify-common/actions/runs/531402880)

Could not update the JSON file: Error occured when creating or editing the comments of the pull request: Error occured when getting all the comments of the pull request: Unable to create comment because issue is locked.

To Reproduce
I believe you just have to comment on a PR after it's merged, however, the above should help reproduce.
Expected behavior
Ideally no error is thrown, it's not clear to the PR reviewer why an action should fail after the PR is merged and causes confusion. If you could check for the locked PR case, maybe a warning would cause less confusion.
Screenshots

Hi @andrei-panov, Ideally CLA step shouldn't even be triggered when there is a PR comment other than I have read the CLA Document and I hereby sign the CLA or recheck.
Could you please check the regex action which you are using? . I feel there is a bug in their side. I might be wrong though.
- name: CLA Assistant
    if: ${{ steps.recheck.outputs.match != '' || steps.sign.outputs.match != '' }} || github.event_name == 'pull_request_target'

I have checked without regex validation and the CLA step is not triggered.
- name: CLA Assistant
   if: (github.event.comment.body == 'recheck' || github.event.comment.body == 'I have read the CLA Document and I hereby sign the CLA') || github.event_name == 'pull_request_target'

Test repository -> https://github.com/ibakshay/test-repo-2/pull/2
Action logs -> https://github.com/ibakshay/test-repo-2/runs/1837047127?check_suite_focus=true

We haven't seen this anymore, thank you for your reply!


Add accessibility statement page.

I've added the PDF with the original name so if anyone downloads it, it
will keep the original name.

The unit tests are OK now, They were checking for changes based on my other PR which hadn't been merged yet, it's in now so they're ok.


Dataset shard index automatic change in Estimator DistributionStrategy

System information

TensorFlow version (you are using): 1.10
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
According to the cluster role defined in Estimator, the task index of chief and worker both start from 0. This brings trouble usage for reading data shard under distributed environment. Users should either change code to distinguish the task index between chief and worker_0 or modify the TF_CONFIG environment to read different data shards.
d = tf.data.TFRecordDataset(FLAGS.input_file)
d = d.shard(FLAGS.num_workers, FLAGS.worker_index)

The propose of DistributionStrategy is to make distribution easy writing with minimal changes to model function and input function. However, we currently cannot avoid the problems described above.
Is there any solutions for this problem? I am very willing to contribute for this but want to have more discussion for the design.
Will this change the current api? How?
Maybe yes.
Who will benefit with this feature?
DistributionStrategy users.
Any Other info.
Any  discussion is appreciated.

Hi @yuefengz @anj-s ,
This is an issue related to DistributionStrategy. Please take a look.
Thanks.

We will add an InputContext object which can be accessible in input_fn. This object should give an unique id and/or the number of workers. Would that work for you cases?

@yuefengz To add an InputContext seems reasonable. And I am very interested in the details for usage of this interface. Could you give an example for pseudo code? Is the input_fn introduced by Estimator or DistributionStrategy?

@wangsiyu Search for InputContexthere: https://github.com/tensorflow/community/pull/25/files. We will make that changes soon.

Nagging Assignee @yuefengz: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @yuefengz: It has been 29 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

@yuefengz I did some work on compatibility with input_context and Estimator. Please give some comments. Thanks  https://github.com/tensorflow/estimator/pull/15

@wangsiyu,
Sorry for the late response. The PR is reviewed and merged. Please feel free to close this issue. Thank you.


CB-88-Additional-cookie-for-GA

Replaced with a new PR as pull/47 wasn't building on TeamCity.
New PR - https://github.com/nice-digital/cookie-banner/pull/48


Add Ironclad.sln.DotSettings to match stylecop rules

Is this done? I see there is now a Ironclad.sln.DotSettings file.

No, I think we need to copy Apollo's, the one already here is pretty empty
On Thu, Dec 13, 2018 at 7:38 AM Cameron Fletcher<EMAIL_ADDRESS>wrote:

Is this done? I see there is now a Ironclad.sln.DotSettings file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LykkeCorp/ironclad/issues/34#issuecomment-446871192,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGapuUF2bhTsKkrEM6m_psp5s-RKXXnks5u4gQKgaJpZM4Y5m1t
.


html tag don't work in custom edge

Describe the Bug
we can't use react component or div within a custom edge component.
is custom edge need to be an svg object to be able to be displayed ?
Your Example Website or App
https://reactflow.dev/docs/examples/edges/custom-edge/
Steps to Reproduce the Bug or Issue
adding a div in my custom edge but does't seem to be displayed
Expected behavior
I expected to be able to use  or any other component to build my custom edge component
Screenshots or Videos
No response
Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Edges are svgs so in order to embed a div/button or similar you'll have to use a foreign object.
There's an example for that in the docs right here.

thanks :)


training viton hd dataset using test_tryon_imagen_trainer.py

i am using viton hd dataset as a sythentic dataset to train using test_tryon_imagen_trainer.py pyton file. After training it is giving noise images as output images. Can anyone suggest why this is happening and how to train the diffusion model with vitonhd dataset.

Hi @Teja414 if you share more about your code maybe I can help
First, right off the bat - Did you change the number of training iterations? number of sampling steps? The parameters in the example scripts are just for quick testing, not full-fledged training.
Another thing to mention is that this architecture is not optimal for VITON-HD, it is meant to take garment from another person (thus there is a garment_pose input), you will need to encode something in-place of the garment pose (e.g. all 0.5) or remove that part of the network.

Is the garment from person is extracted inside the code itself. In viton we
have garment image and keypoints so I have used them.
On Wed, 3 Apr 2024, 3:26 pm Dan Bochman, @.***> wrote:

Hi @Teja414 https://github.com/Teja414 if you share more about your
code maybe I can help
First, right off the bat - Did you change the number of training
iterations? number of sampling steps? The parameters in the example scripts
are just for quick testing, not full-fledged training.
Another thing to mention is that this architecture is not optimal for
VITON-HD, it is meant to take garment from another person (thus there is a
garment_pose input), you will need to encode something in-place of the
garment pose (e.g. all 0.5) or remove that part of the network.
—
Reply to this email directly, view it on GitHub
https://github.com/fashn-AI/tryondiffusion/issues/2#issuecomment-2034115042,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXIDCS5RP7EYDE32PI4JFYLY3PG4JAVCNFSM6AAAAABFU654YCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZUGEYTKMBUGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

this the test_tryon_imagen_trainer.py code that i have updated and used for training jst sample of 10. and for sample of 10 i have attaching the output.

import torch
from torch.utils.data import DataLoader, Dataset
import os
import json
from torchvision import transforms
from PIL import Image
import matplotlib.pyplot as plt
from tryondiffusion import TryOnImagen, get_unet_by_name
from tryon_imagen_trainer import TryOnImagenTrainer
TRAIN_UNET_NUMBER = 1
BASE_UNET_IMAGE_SIZE = (128, 128)
SR_UNET_IMAGE_SIZE = (256, 256)
BATCH_SIZE = 10
GRADIENT_ACCUMULATION_STEPS = 2
NUM_ITERATIONS = 20
TIMESTEPS = (2, 2)
Define the transformation to resize images to 128x128
image_transform = transforms.Compose([
transforms.Resize((128, 128)),
transforms.ToTensor()
])
Define your dataset class
class SyntheticTryonDataset(Dataset):
def init(self, data_dir, num_samples, image_size, pose_size=(18, 2)):
self.data_dir = data_dir
self.num_samples = num_samples
self.image_size = image_size
self.pose_size = pose_size
    # Get the list of image filenames
    self.person_image_files = sorted(os.listdir(os.path.join(data_dir, 'person_image')))
    self.ca_image_files = sorted(os.listdir(os.path.join(data_dir, 'ca_image')))
    self.garment_image_files = sorted(os.listdir(os.path.join(data_dir, 'garment_image')))
    self.person_pose_files = sorted(os.listdir(os.path.join(data_dir, 'person_pose')))
    self.garment_pose_files = sorted(os.listdir(os.path.join(data_dir, 'garment_pose')))

def __len__(self):
    return self.num_samples

def __getitem__(self, idx):
    person_image_filename = self.person_image_files[idx]
    ca_image_filename = self.ca_image_files[idx]
    garment_image_filename = self.garment_image_files[idx]
    person_pose_filename = self.person_pose_files[idx]
    garment_pose_filename = self.garment_pose_files[idx]

    person_image_path = os.path.join(self.data_dir, 'person_image', person_image_filename)
    ca_image_path = os.path.join(self.data_dir, 'ca_image', ca_image_filename)
    garment_image_path = os.path.join(self.data_dir, 'garment_image', garment_image_filename)
    person_pose_path = os.path.join(self.data_dir, 'person_pose', person_pose_filename)
    garment_pose_path = os.path.join(self.data_dir, 'garment_pose', garment_pose_filename)

    person_image = Image.open(person_image_path)  # Load person image from the path
    ca_image = Image.open(ca_image_path)  # Load ca image from the path
    garment_image = Image.open(garment_image_path)  # Load garment image from the path

    # Apply image transformation to resize to 128x128
    person_image = image_transform(person_image)
    ca_image = image_transform(ca_image)
    garment_image = image_transform(garment_image)

    with open(person_pose_path, 'r') as f:
        pose_data = json.load(f)
        long_pose = pose_data.get("long")[:self.pose_size[0]]  # Extract first 18 points from "long" key
        person_pose = torch.tensor(long_pose).view(self.pose_size[0], self.pose_size[1])

    garment_pose = torch.randn(*self.pose_size)  # Placeholder for garment pose data

    sample = {
        "person_images": person_image,
        "ca_images": ca_image,
        "garment_images": garment_image,
        "person_poses": person_pose,
        "garment_poses": garment_pose,
    }

    return sample

def tryondiffusion_collate_fn(batch):
return {
"person_images": torch.stack([item["person_images"] for item in batch]),
"ca_images": torch.stack([item["ca_images"] for item in batch]),
"garment_images": torch.stack([item["garment_images"] for item in batch]),
"person_poses": torch.stack([item["person_poses"] for item in batch]),
"garment_poses": torch.stack([item["garment_poses"] for item in batch]),
}
def main():
print("Instantiating the dataset and dataloader...")
dataset = SyntheticTryonDataset(
data_dir="/home/azureuser/try_on_diffusion/tryondiffusion/data/syn_data", #path to the data folder
num_samples=10,  # Adjust this according to your dataset size
image_size=SR_UNET_IMAGE_SIZE if TRAIN_UNET_NUMBER == 2 else BASE_UNET_IMAGE_SIZE
)
train_dataloader = DataLoader(
dataset,
batch_size=BATCH_SIZE,
shuffle=True,
collate_fn=tryondiffusion_collate_fn,
)
validation_dataloader = DataLoader(
dataset,
batch_size=BATCH_SIZE,
shuffle=False,
collate_fn=tryondiffusion_collate_fn,
)
print("Checking the dataset and dataloader...")
for i, batch in enumerate(train_dataloader):
print(f"Batch {i+1}:")
# Print the shapes of each tensor in the batch
for k, v in batch.items():
if isinstance(v, torch.Tensor):
print(f"{k}: {v.shape}")
else:
print(f"{k}: List of {len(v)} tensors with shapes: {v[0].shape}")
    # Print and display the images (assuming they are PIL Images)
    for j in range(len(batch["person_images"])):
        person_image = batch["person_images"][j].permute(1, 2, 0)  # Transpose the image data
        ca_image = batch["ca_images"][j].permute(1, 2, 0)  # Transpose the image data
        garment_image = batch["garment_images"][j].permute(1, 2, 0)  # Transpose the image data

        # Display the images using Matplotlib
        plt.figure(figsize=(10, 5))
        plt.subplot(1, 3, 1)
        plt.imshow(person_image)
        plt.title("Person Image")
        plt.axis("off")

        plt.subplot(1, 3, 2)
        plt.imshow(ca_image)
        plt.title("CA Image")
        plt.axis("off")

        plt.subplot(1, 3, 3)
        plt.imshow(garment_image)
        plt.title("Garment Image")
        plt.axis("off")

        plt.show()

    # Break after printing the first few batches
    if i >= 2:
        break

# Instantiate the unets
print("Instantiating U-Nets...")
base_unet = get_unet_by_name("base")
sr_unet = get_unet_by_name("sr")

# Instantiate the Imagen model
imagen = TryOnImagen(
    unets=(base_unet, sr_unet),
    image_sizes=(BASE_UNET_IMAGE_SIZE, SR_UNET_IMAGE_SIZE),
    timesteps=TIMESTEPS,
)

print("Instantiating the trainer...")
trainer = TryOnImagenTrainer(
    imagen=imagen,
    max_grad_norm=1.0,
    accelerate_cpu=True,
    accelerate_gradient_accumulation_steps=GRADIENT_ACCUMULATION_STEPS,
)

trainer.add_train_dataloader(train_dataloader)
trainer.add_valid_dataloader(validation_dataloader)

print("Starting training loop...")
# training loop
for i in range(NUM_ITERATIONS):
    # TRAINING
    loss = trainer.train_step(unet_number=TRAIN_UNET_NUMBER)
    print(f"loss: {loss}")
    valid_loss = trainer.valid_step(unet_number=TRAIN_UNET_NUMBER)
    print(f"valid loss: {valid_loss}")

# SAMPLING
print("Starting sampling loop...")
validation_sample = next(trainer.valid_dl_iter)
_ = validation_sample.pop("person_images")
imagen_sample_kwargs = dict(
    **validation_sample,
    batch_size=BATCH_SIZE,
    cond_scale=2.0,
    start_at_unet_number=1,
    return_all_unet_outputs=True,
    return_pil_images=True,
    use_tqdm=True,
    use_one_unet_in_gpu=True,
)
import os
from PIL import Image

images = trainer.sample(**imagen_sample_kwargs)  # returns List[Image]
assert len(images) == 2
assert len(images[0]) == BATCH_SIZE and len(images[1]) == BATCH_SIZE

# Create the output folder if it doesn't exist
output_folder = 'output/'
os.makedirs(output_folder, exist_ok=True)

for i, unet_output in enumerate(images):
    for j, image in enumerate(unet_output):
    # Save the image with a unique filename
        image.save(os.path.join(output_folder, f'image_{i * BATCH_SIZE + j}.jpg'))

print("Images saved to 'output/' folder.")

for unet_output in images:
    for image in unet_output:
        image.show()

if name == "main":
# python ./examples/test_tryon_imagen_trainer.py
main()

It takes a lot more time to train to stop seeing noise

Sample for 256 timesteps in the 1st UNet, 128 timesteps if you also use the 2nd UNet
You need much more data samples (minimum 10k)
Train for at least 10k iterations (to stop seeing noise) with batch size 256
Use GPU (CPU is used there just for testing)

Thank you for that but for this sample aftr the training, is it a not
expected thing to get unet 2 not trained msg in cmd.
On Wed, 3 Apr 2024, 6:16 pm Dan Bochman, @.***> wrote:

It takes a lot more time to train to stop seeing noise

Sample for 256 timesteps in the 1st UNet, 128 timesteps if you also
use the 2nd UNet
You need much more data samples (minimum 10k)
Train for at least 10k iterations (to stop seeing noise) with batch
size 256
Use GPU (CPU is used there just for testing)

—
Reply to this email directly, view it on GitHub
https://github.com/fashn-AI/tryondiffusion/issues/2#issuecomment-2034511370,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXIDCSZTZCSLW2FHLBEIKVLY3P2ZLAVCNFSM6AAAAABFU654YCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZUGUYTCMZXGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

can you please let me know how you have got the cloth landmark keypoints. can you guide me to do that task?

The pose keypoints are taken from the person wearing the cloth. This architecture is not suited as-is to work with VITON-HD where the garment images are flat clothing-only photos.

@Teja414 closing this issue as you have opened a new issue.


WebGL 2 enabled by default in Chrome 56 and Firefox 51

https://www.chromestatus.com/feature/6694359164518400
https://www.mozilla.org/en-US/firefox/51.0beta/releasenotes/

Thanks, updated accordingly!


Unable to create DynamoFormat when using scalaz tagged types

Here's a minimal example that seems like it should compile, but does not:
import com.gu.scanamo.DynamoFormat
import com.gu.scanamo.DynamoFormat._
import scalaz.{@@, Tag}
import scalaz.syntax.tag._

sealed trait MyTag

case class TestClass(s: String @@ MyTag)

object DynamoFormatters {

  implicit val tagFormatter: DynamoFormat[String @@ MyTag] =
    DynamoFormat.iso[String @@ MyTag, String](Tag[String, MyTag])(_.unwrap)

  implicit val testClassFormatter = DynamoFormat[TestClass]
}

It fails with could not find implicit value for parameter instance: com.gu.scanamo.DynamoFormat[TestClass]. This is with scala 2.12.4, scanamo 1.0.0-M2, and scalaz-core 7.2.16.
On the other hand, I did notice that when using shapeless's version of @@, it does compile. The most obvious difference I see is that shapeless @@ is covariant on the base type, while scalaz @@ is invariant -- I'm not sure if this is relevant, though.

As an update, I've switched to explicitly defining my DynamoFormat's read and write instead of relying on the automatically derived versions to have a bit more control over how the data is written out. When doing so, the following works without problem:
implicit def taggedFormatter[BASE: DynamoFormat, TAG]: DynamoFormat[BASE @@ TAG] =
    DynamoFormat.iso[BASE @@ TAG, BASE](Tag[BASE, TAG])(_.unwrap)

So I suppose it's not accurate to say that Scalaz tagged types don't work with DynamoFormat as they work fine here. Either something with the auto-derivation was getting tripped up, or perhaps I was misusing it my first example -- feel free to close if this is actually a non-issue.


Screenshots?

Pleaseee!

As in the in-game screenshots? they're raw rgb files
Battlespire:File Area on UESP has a tool for converting them to PCX (batraw.zip), which you can then open with gimp or similar

No no, I meant if you had any screenshots of the tools output, exported assets, levels re-imported into blender etc. :)

Not really, the blender import was a horrible mess and i didn't get textures working before moving on to something else
Only one i can find is a view of the first room

Amazing!
It's always fun when you finally get the format right and reveal the geometry in its naked beauty, isn't it? :)


Why Microsoft.Data.SqlClient has dependency to Microsoft.Identity.Client?

Why Microsoft.Data.SqlClient has dependency to Microsoft.Identity.Client?

Hi @dehghani-mehdi
We use Microsoft.Identity.Client (MSAL for .NET) to support Azure Active Directory authentication modes:

Active Directory Password
Active Directory Integrated
Active Directory Interactive
Active Directory Service Principal

May I ask if you are facing any issues? Could you elaborate?

Hi @cheenamalhotra
Yes, the description of Microsoft.Identity.Client package is as follow:

This package contains the binaries of the Microsoft Authentication Library for .NET (MSAL.NET).
MSAL.NET makes it easy to obtain tokens from the Microsoft identity platform for developers (formally Azure AD v2.0) signing-in users with work & school accounts, Microsoft personal accounts and social identities Azure AD B2C. These tokens again access to Microsoft Cloud API and any other API secured by the Microsoft identity platform. This version supports adding authentication functionality to your .NET based client on Windows desktop (.NET 4.5+), UWP, .NET Core, Xamarin iOS and Xamarin Android.

But I'm wondering how identitcation is relevent here? Microsoft.Data.SqlClient is just a data provider, again here is description of this package:

Provides the data provider for SQL Server. These classes provide access to versions of SQL Server and encapsulate database-specific protocols, including tabular data stream (TDS)
Commonly Used Types:
Microsoft.Data.SqlClient.SqlConnection
Microsoft.Data.SqlClient.SqlException
Microsoft.Data.SqlClient.SqlParameter
Microsoft.Data.SqlClient.SqlDataReader
Microsoft.Data.SqlClient.SqlCommand
Microsoft.Data.SqlClient.SqlTransaction
Microsoft.Data.SqlClient.SqlParameterCollection
Microsoft.Data.SqlClient.SqlClientFactory

Why this low level package should worry about Active Directory, Azure and ...?

May I ask if you are facing any issues? Could you elaborate?

Sure, no, I didn't, but I think having unnecessary dll/dependency should consider as issue.

Hi @dehghani-mehdi
Driver's support for Active Directory authentication modes is built-in, i.e. another dependency is not needed in order to authenticate as: Active Directory Password/Integrated/Interactive/Service principal, authentication modes and more are to come. MSAL is the new library and has replaced ADAL library that was used by System.Data.SqlClient.
The federated authentication design has always been built-in and moving away from it would have caused a lot of disturbance in community for existing SqlClient customers hence we chose to keep it backwards compatible. It's not impossible to do but Azure SQL is as much important to us as SQL Server and is heavily used, thus AD authentication is available in the driver as-is.
Also NuGet does not have a concept of Optional dependencies like Maven (https://github.com/NuGet/Home/issues/686), otherwise it would have been a perfect candidate.

Closing as by design.

All issues could be closed as by desgin.

Just as a note for any future, fellow troubleshooters:
Due to this dependency, the dependency chain for EF Core looks like this:

Entity Framework Core depends on:

Microsoft.Data.SqlClient, which in turn depends on:

Microsoft.Identity.Client, which depends on:

System.Windows.Forms

This is ultimately why upgrading to EF Core in an e.g. ASP.NET application or Console application pulls in Windows Forms. As far as I can tell it's completely harmless, though it does seem to trigger confusing static analysis warnings, like suggesting STAThread attributes in non-windows forms applications (e.g. RSPEC-4210).

Entity Framework Core depends on:
Microsoft.Data.SqlClient, which in turn depends on:
Microsoft.Identity.Client, which depends on:
System.Windows.Forms

Whaaat?!? Have we learnt nothing about inversion of control?


Collapse incident-handling as much as possible into single, linear doc

Our documentation of incident-handling procedures and tools have proliferated and it's been difficult to keep them up-to-date with evolving tools and conventions. This [work-in-progress] update attempts to consolidate them into a single doc. It doesn't yet remove the pre-existing docs.
For instructions that are occasionally relevant, but shouldn't get in the way of an on-call engineer who may be under pressure to respond to an incident, I employ the <details><summary>... syntax to keep that content collapsed by default.
(Some of the unrelated edits and questionable whitespace choices are required by the pre-commit hook, not me.)

Thanks for all the feedback. I'll merge, noting these areas of possible improvement:

clarify how to identify additional responders, especially once projects' point people are updated and spread around geographies
use incident priorities more strategically, perhaps by encouraging tracking of "low"/"informational" incidents that might not trigger on-call notifications


Slow React Tests

Thanks for React and Jest. Loving the combo. Anyhow, I am used to running tests in livereload mode. So anytime a test file is saved, my tests are automatically run. I got this working correctly, but my tests are taking almost 3 seconds to run. That is just with one test file. I am running the files through the preprocessor, so I suspect that is where the problem lies. Do you have any suggestions on how to get the tests to run quickly or any advice on how to speed up the TDD / BDD workflow?

This worked for me using React v0.12+:
var ReactTools = require('react-tools');

module.exports = {
  process: function(src, file) {
    if(!file.match(/\.react\.js$/)) return src;

    return ReactTools.transform(src);
  }
};

Just started using Jest as well - just testing regular JavaScript modules (no JSX / React) and Jest is nasty slow. I love the idea of inline tests and built-in mocking, but it's so painfully slow it makes me miss Mocha. I'm not sure what the culprit is... it's not searching the directory tree that's causing the slow speed. If I specify the file directly, it's still super slow.
Any ideas on at least the cause of the slowness? (I'm not using JSX / CoffeeScript)

The original example never worked for me as the first argument always returned true.
var ReactTools = require('react-tools');
var MAGIC = "/** @jsx ";
module.exports = {
  process: function(src, file) {
    if (src.slice(0, MAGIC.length) != MAGIC) return src;
    return ReactTools.transform(src);
  }
};

I noticed in https://github.com/facebook/jest/blob/master/src/TestWorker.js that it's using optimist - it's a useful module but I recall it affecting one of my own cli tools' startup time drastically.

@culshaw nice workarounds, shorten the 10s+ tests to 2+. this workarounds made me think that maybe we should use the extension *.jsx for our react files, i think it might help when using reactify too.

To be honest going through Grunt I was still getting times of +20s, got
React working with Karma and total time is about +4/5 secs.
All inside a VM though
On Wednesday, January 21, 2015, alvinsj<EMAIL_ADDRESS>wrote:

@culshaw https://github.com/culshaw nice workarounds, shorten the 10s+
tests to 2+. this workarounds made me think that maybe we should use the
extension *.jsx for our react files, i think it might help when using
reactify too.
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/jest/issues/116#issuecomment-70826072.

--
Ian Culshaw
www.ianculshaw.co.uk
@culshaw
+447792148454

Just to report back on this. We have been testing react with Karma/Mocha and we are close to 700 tests and it takes about 4 seconds to run all of the tests.  We do have to mange mocks, but it's worth it.  React has been AMAZING. Flawless and refreshingly stable. Game changer!  Our team couldn't imaging using anything else.

@iamrandys would you mind explaining in more detail how you managed to make jest so fast?

I didn't ever get Jest to be fast.  I am using mocha.  If Jest was fast, I
would have used it instead.
On Thu, Feb 5, 2015 at 12:17 PM, Gil Birman<EMAIL_ADDRESS>wrote:

@iamrandys https://github.com/iamrandys would you mind explaining in
more detail how you managed to make jest so fast?
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/jest/issues/116#issuecomment-73097182.

Thanks @iamrandys I had deleted my "how did you get jest to be fast?" question after realizing I had misread your post (but you responded at the same time)... anyway, it's a shame, I guess jest should still be considered not so ready for production use.

the workarounds/changes by @haihappen and @darcyadams helped me get the test time down from 3,5 sec to about 1 sec for a single test. That does not take into account the apparent startup time, which seems to be about 1 sec also. This seems much too slow to be efficient in daily work, so I guess I'll have a look at the mocha/karma combo. I like Jest's philosophy of mocking everything by default, but the benefit just is not there atm.

I have the same problem: I got just 3 tests in a small project and they take 3 seconds.
Is improving the performance anywhere near on the project roadmap?

Strange Jest was not mentioned at the React.js 2015 conf.

testPathDirs appears to be the culprit.  Specify like this in package.json:
  "jest": {
    "unmockedModulePathPatterns": [
      "./node_modules"
    ],
    "scriptPreprocessor": "./preprocessor.js",
    "testDirectoryName": "tests",
    "testPathDirs": ["tests"]
  }

@adjavaherian be careful with that if you're using automatic mocking: https://github.com/facebook/jest/issues/176
In fact, expect auto-mocking to break in subtle and frustrating ways if your testPathDirs doesn't cover your dependencies.

These concerns makes me think we might be doing something wrong with our test set-up. I might be wrong here, but as far as I know, we're all using stuff like: var component = React.createFactory(require("component/path")) along with other required modules within beforeEach test. Surely this mustn't be necessary for all tests. I mean a factory should produce a fresh component every time. If I move the require outside of the beforeEach block test speed increases 10x. Unfortunately, in that case, some tests oddly fail and I don't know why.
Not sure if this helps. Thoughts?

would be nice if preprocessing time was left out of test execution time. Kind of unfair to see red when the bulk of that work comes from a transformation step.

Hi guys,
I´m having the same problem. I even have some test cases which take around 1min to run :(

If speed is still an issue, I suggest moving to Mocha.  We have had a lot of success with this fork, which explains how to setup tests for simple to complex React deployments.  https://github.com/adjavaherian/mocha-react  We regularly run over a 100 tests in about 3 seconds.

I agree Mocha is the way to go. Up to almost 900 tests and it takes about 4 seconds.

On Apr 23, 2015, at 4:53 PM, Amir Djavaherian<EMAIL_ADDRESS>wrote:
If speed is still an issue, I suggest moving to Mocha. We have had a lot of success with this fork, which explains how to setup tests for simple to complex React deployments. https://github.com/adjavaherian/mocha-react We regularly run over a 100 tests in about 3 seconds.
—
Reply to this email directly or view it on GitHub.

Same experience, I just set up Jest with only one test (using JSX) and it takes about 3 seconds, this is a lot.

@iamrandys do you mind showing an example of your setup? Seems like the perfect combo.

@amasad Is there an option with Jest to build up a cache for compiled files similar to what the babel-loader allows in the cacheDirectory option? - https://github.com/babel/babel-loader#options

What makes jest slow for me isn't the compilation but rather the startup of the worker pool processes. All my test pass in under 0.05 seconds except the first ones, which take around 5 seconds.
https://github.com/jeffmo/node-worker-pool
https://github.com/facebook/jest/blob/master/src/TestRunner.js#L376

@songawee It doesn't but want to send a PR to have it as an option? I don't think we should have it on by default because it's sometimes impossible to know when to break the cache. For example, if you change your compiler options, the cache should reset. Another option is to have a reset-cache option in addition to caching.
@doodzik are you sure it's the worker-pool and not the node-haste stating and reading of modules?

@amasad What I did was to measured the time each step in running jest took for completion.
And the  node-worker-pool was the last instance where the tests were slow.
It could be that my findings are just the symptom and not the root of the problem.
But I hadn't had time to give it a proper analysis.
My tests currently look like this:

+1
Same here. Tests are very very slow :disappointed:

I thought It was only me that faced the issue. It is my first time using Jest and I don't get fast test result, either.

My question about Jest to the React guys at React Europe conference Q&A session - https://youtu.be/CRJZBZ_-6hQ?t=363

Switched to Mocha + Sinon. Never been happier.
On 31 August 2015 at 17:45, Alan Rubin<EMAIL_ADDRESS>wrote:

My question about Jest to the React guys at React Europe conference Q&A
session - https://youtu.be/CRJZBZ_-6hQ?t=363
—
Reply to this email directly or view it on GitHub
https://github.com/facebook/jest/issues/116#issuecomment-136394910.

I have the same problem. Jest tests just take a lot of time and the execution time actually varies. Whether to run them in parallel or in just one process (--runInBand) did not matter. It appears to not be a resource contention between the worker processes.
I created some cpu dumps with the v8 profiler (https://github.com/node-inspector/v8-profiler) and found that most time appears to be spent mocking modules. I.e. 25% of the execution time of my unit test is spent in jest-cli/src/lib/utils.js#runContentWithLocalBindings.

any performance updates? just picked up jest with es6 and babel-jest, but running 2 simple tests in > 10 seconds :-(
tried a lot of ideas from this thread to speed up, but nothing worked...

We'll be focusing on this soon. We are a little swamped with work on jest right now but we are committed to make it more awesome.

Are there any tasks the community could help with?

+1

The biggest help right now would actually be improving the documentation, website and going through issues and helping out people in open source.

One thing we have done to speed our JEST tests up in the build pipeline was to replace our single core machine with a multi core one. By default jest spaws as many workers as hardware threads are available. If this is not available to you you can manually play with '-w' (maxWorkers). You may gain a speedup also on a single core.
Ultimately we found that mocking modules is very costly (see my comment above) and causes a majority of the execution time.

Jest with es6 for me is completely unusable. it takes 10+ seconds just to start up, and then it takes 2s to run the only single test I have at the moment. I was expecting a lot more, switching back to karma :(

We are currently working on replacing node-haste with a new module resolver, which should fix this issue.

Hi all. Is there any news on this issue ?

Hi, is Jest suitable for non React testing? Would like to have a common standard for both react and non-react apps.

 :) Just take a look at one of the examples!

Hi all, some real interesting info on here. I'm also experiencing issues with tests running slow too. I've currently got 13 tests taking ~15secs to run.
I did find adding "testPathDirs": ["<rootDir>/path/to/tests/"] to our packages.json file helped to improve the startup time considerably.
@cpojer Have you got an update for us regarding the new and improved module resolver? I'm really hoping that this is going to be the key to getting tests running much quicker

This work is happening in #599.

Thanks @cpojer 😀
I look forward to seeing the finished Haste2

The same tests using mocha run in 44ms for me where jest took full 6 seconds.
It took me about 15 minutes to switch my initial 6 test files using jest to using Mocha, jsdom and sinon.

Good news everyone, I'm merging #599 today and it should do away with the slow startup, finally.

Ok, this should finally be fixed in Jest 0.9. Sorry that this took so long but there was some clowniness in Jest :)
See https://github.com/facebook/react/pull/6052 on how the React tests themselves were sped up. If you'd like to try this improvement, check out the comments in #599. It is currently tagged as jest@next until to see if there are any bugs that people in open source might encounter. I'll close this issue as resolved.

npm install jest-cli@next if you want to run this new version (rather than jest@next @cpojer)

oh yeah, I always make this mistake :) I edited my original comment.

@cpojer after upgrading using npm install jest-cli@next I'm having problems specifying dontMock. By which I mean, before the update (using<EMAIL_ADDRESS>this line works properly:
jest.dontMock('../../../../fixtures');
then after the update to 0.9.0, the same call results in the module being mocked

@steinbachr that should probably go into a separate issue. Would be great if you could provide a repro, haven't seen this issue come up at FB!

thanks @cpojer, issue created here


Transient virtualmachineinstances-create-validator "x509: certificate signed by unknown authority" error, that cause test failure.

/kind bug

What happened:
Running test_id 3187 in loops locally,
sometimes the test would fail, because starting VM fails, with the error
Message: "Internal error occurred: failed calling webhook \"virtualmachineinstances-create-validator.kubevirt.io\": Post https://virt-api.kubevirt.svc:443/virtualmachineinstances-validate-create?timeout=30s: x509: certificate signed by unknown authority (possibly because of \"crypto/rsa: verification error\" while trying to verify candidate authority certificate \"kubevirt.io\"

Simply sleep and retry solves it, (i slept 3 minutes, its too much)
the problem might affect other tests, didn't see it yet on prow, it happens 1 out of 200 runs locally
Error comes from:
tests/vmi_cloudinit_test.go
	LaunchVMI := func(vmi *v1.VirtualMachineInstance) {
		By("Starting a VirtualMachineInstance")
		obj, err := virtClient.RestClient().Post().Resource("virtualmachineinstances").Namespace(tests.NamespaceTestDefault).Body(vmi).Do().Get()
		Expect(err).To(BeNil())

hack/dockerized "hack/build-func-tests.sh"
go version go1.12.8 linux/amd64

go version go1.12.8 linux/amd64
hack/functests.sh
Running Suite: Tests Suite
==========================
Random Seed:<PHONE_NUMBER>
Will run 1 of 561 specs

SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
------------------------------
[rfe_id:151][crit:high][vendor:cnv-qe@redhat.com][level:component]CloudInit UserData [rfe_id:151][crit:medium][vendor:cnv-qe@redhat.com][level:component]A new VirtualMachineInstance with cloudInitConfigDrive networkData
  [test_id:3187]should have cloud-init userdata and network-config from separate k8s secrets
  /root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:544
STEP: Creating a secret with userdata
STEP: Creating a secret with network data
STEP: Starting a VirtualMachineInstance

• Failure [4.604 seconds]
[rfe_id:151][crit:high][vendor:cnv-qe@redhat.com][level:component]CloudInit UserData
/root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:52
  [rfe_id:151][crit:medium][vendor:cnv-qe@redhat.com][level:component]A new VirtualMachineInstance
  /root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:133
    with cloudInitConfigDrive networkData
    /root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:404
      [test_id:3187]should have cloud-init userdata and network-config from separate k8s secrets [It]
      /root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:544

      Expected
          <*errors.StatusError | 0xc0028fe8c0>: {
              ErrStatus: {
                  TypeMeta: {Kind: "", APIVersion: ""},
                  ListMeta: {
                      SelfLink: "",
                      ResourceVersion: "",
                      Continue: "",
                      RemainingItemCount: nil,
                  },
                  Status: "Failure",
                  Message: "Internal error occurred: failed calling webhook \"virtualmachineinstances-create-validator.kubevirt.io\": Post https://virt-api.kubevirt.svc:443/virtualmachineinstances-validate-create?timeout=30s: x509: certificate signed by unknown authority (possibly because of \"crypto/rsa: verification error\" while trying to verify candidate authority certificate \"kubevirt.io\")",
                  Reason: "InternalError",
                  Details: {
                      Name: "",
                      Group: "",
                      Kind: "",
                      UID: "",
                      Causes: [
                          {
                              Type: "",
                              Message: "failed calling webhook \"virtualmachineinstances-create-validator.kubevirt.io\": Post https://virt-api.kubevirt.svc:443/virtualmachineinstances-validate-create?timeout=30s: x509: certificate signed by unknown authority (possibly because of \"crypto/rsa: verification error\" while trying to verify candidate authority certificate \"kubevirt.io\")",
                              Field: "",
                          },
                      ],
                      RetryAfterSeconds: 0,
                  },
                  Code: 500,
              },
          }
      to be nil

      /root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:62
------------------------------
failed to dump entity named [sriovnetworknodestates]: the server could not find the requested resource
failed to dump entity named [sriovnetworknodepolicies]: the server could not find the requested resource
failed to dump entity named [sriovnetworks]: the server could not find the requested resource
failed to dump entity named [sriovoperatorconfigs]: the server could not find the requested resource
SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS
Waiting for namespace kubevirt-test-default to be removed, this can take a while ...
Waiting for namespace kubevirt-test-alternative to be removed, this can take a while ...
Waiting for namespace kubevirt-test-operator to be removed, this can take a while ...
{"command":"gocli scp /var/log/k8s-audit/k8s-audit.log -","component":"tests","level":"error","msg":"command failed","pos":"utils.go:3296","reason":"no gocli binary specified","timestamp":"2020-06-09T11:28:29.624431Z"}
{"component":"tests","level":"error","msg":"Could not generate REST coverage report command failed: no gocli binary specified","pos":"utils.go:597","timestamp":"2020-06-09T11:28:29.624515Z"}

Summarizing 1 Failure:

[Fail] [rfe_id:151][crit:high][vendor:cnv-qe@redhat.com][level:component]CloudInit UserData [rfe_id:151][crit:medium][vendor:cnv-qe@redhat.com][level:component]A new VirtualMachineInstance with cloudInitConfigDrive networkData [It] [test_id:3187]should have cloud-init userdata and network-config from separate k8s secrets
/root/go/src/kubevirt.io/kubevirt/tests/vmi_cloudinit_test.go:62

Ran 1 of 561 Specs in 31.365 seconds
FAIL! -- 0 Passed | 1 Failed | 0 Pending | 560 Skipped
--- FAIL: TestTests (31.37s)
FAIL
make: *** [Makefile:54: functest] Error 1
m1 kubevirt (master) $

What you expected to happen:
VM should be created, robust to transient errors, and test should pass.
How to reproduce it (as minimally and precisely as possible):
run 3187 in loops until it happens
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

seen as well on loops of test 1770


x/net/http2: invalid Connection request header

I meet the error and found the issue here
https://github.com/golang/go/issues/15422
and the source code
https://github.com/golang/net/blob/6972930/http2/transport.go#L626-L634
as it said the keep-alive is valid
but why it is case-sensitive ,
many request failed due to Keep-Alive header
does HTTP/2 RFC say it  must be case-sensitive ?

What is your issue?
How did you get this error?
HTTP/2 doesn't have connection-level headers, like the other bug says.

@bradfitz   I made a proxy for some urls
example code  https://gist.github.com/suconghou/50dd30df10bd6faa2abe1805afe3216f
notice it works well then you use chrome , because chrome send keep-alive header
but some other software and services may send Keep-Alive header  , eg: cloudflare , IE ( https://stackoverflow.com/questions/9650236/how-to-send-lower-case-keep-alive-header-through-httpwebrequest/30071784  the answers mentioned IE  )
use curl -H "Connection:Keep-Alive" http://<IP_ADDRESS>:8080/ simulate this behavior

@bradfitz
please replace this
(v != "" && v != "close" && v != "keep-alive")

with
(v != "" && v != "close" && v != "keep-alive" && v != "Keep-Alive")

in checkConnHeaders function

Doesn't it makes sense to handle this transparently?
I think Go should just drop the Connection header if H2 does not support Connection headers.
I see the same problem if I use plain http.Transport.RoundTrip(req) in case Connection: Keep-Alive is set and the backend called specified in the http.Request supports H2.
For me it sounds really weird to argue you should use httputil.ReverseProxy, instead of fixing this.


feat: Send submission email to CH staff

JIRA
Post-submission email to staff
Change description
Partially implements a PATCH endpoint to receive data from the CH Payment Service upon successful payment, triggering the sending of an email to CH staff.
Related PRs

Docker Env: companieshouse/docker-chs-development#58
Notification API: companieshouse/chs-notification-api#222

Email Screenshots (Approximation Only - In Context screenshots not available)

Work Checklist

[ ] Tests added where applicable
[ ] Test coverage of new code meets target of 80%

Email sending infrastructure such as models, serialisers and producers should be separate repository and imported as a dependency to increase reusability.
Without it we will be just like another project that reinvented the same thing again.

I agree but I don't have the necessary Java expertise to implement this. Another thing is that I don't want to further solidify the current email-sending approach. I'd be very interested to see the justification for the current workflow over using something as simple as GovNotify.


install correct locale en_US.utf8

ask user to install correct locale to fulfill OpenAPS prerequisites. Better solution would be installation without user intervention. Follows if there is more time.

I don't think we want to force newbies into dpkg-reconfigure locale - it's extremely non-intuitive.  I would rather leave things as-is until we figure out the best method to install the necessary locales without user intervention.
I'm also not sure why exactly this is necessary to add to the default setup, as the jubilinux and raspbian images we recommend come pre-installed with an English locale that supports the proper numeric separators for autotune.  Obviously anyone who doesn't want to interface with their rig in English would need to add their own language settings, but I don't think that needs to be in the install script.

Closing for now.  Please re-open if you'd like to continue work on it.


Move image loading logic to _load_image in VOC datasets (similar to COCO)

🚀 The feature
This would allow simple stubbing out image loading if needed for fast dataset transformed metadata access (when image reading is not needed) by dataset._load_image = lambda _: None and also make API more consistent across datasets.
It's also impossible to monkey-patch __getitem__ at instance level: https://stackoverflow.com/questions/57413453/is-it-possible-to-override-getitem-at-instance-level-in-python :(
Motivation, pitch
N/A
Alternatives
No response
Additional context
No response

I would recommend to inherit from the class, and override __getitem__ instead then, it should only be a few more lines of code.
If you submit a PR for this I think we would merge it, but keep in mind the _load_image function is private API, so while I know you know what you're doing, we don't offer any guarantee w.r.t. BC or consistency across datasets here.

Here's what I do now: dataset.__getitem__ = lambda index: dataset.transforms(None, dataset.parse_voc_xml(torchvision.datasets.voc.ET_parse(dataset.annotations[index]).getroot()))
Overriding getitem would entail to the same thing - having to copy-paste the even less standard / error-prone annotation parsing line.
Not wanting the dataset to load images for metadata purposes is a valid and frequent usecase. At least simplifying hacking around would be helpful and minimize the copy-pasting. Accepting a method for image loading by a given path in __init__ is another way (probably for pickle purposes, there would need to be a torchvision method function None)

As discussed in #4991, I'm ok with moving the functionality into a private method on the dataset as long as it is a 1-to-1 copy of the current behavior. Still, the BC part of https://github.com/pytorch/vision/issues/4975#issuecomment-976365004 applies.


11065 ess start downloading enc file from fss as soon as possible

Coding changes.
Updated Test Case

Trying to merge this and there appear to be 4 failing unit tests.  Could you have a look please?
https://dev.azure.com/ukhydro/Exchange Set Service/_build/results?buildId=24596&view=logs&j=30945a94-8ee8-5fb4-183e-8cd5348ef8d1&t=584f202c-0fab-5bcf-25b6-8c9e2a6cdbdb
@Siddartha-tech @janvi13603


Alternate job titles

About the PR

Media

(will change a bit)

Changelog

:cl: DEATHB4DEFEAT

add: Added the ability to choose alternate job titles for specified jobs.

Needs review 🙃


[Feature] support dayofweek_iso function

Fixes #20511
What type of PR is this:

[ ] BugFix
[x] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

cc @silverbullet233

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergify backport branch-3.1

@Mergify backport branch-3.0


Protect the users from using Sobol and GaussianPolynomial ROMs inside the HybridModel and CrossValidation

Issue Description
Currently, the Sobol and GaussianPolynomial ROMs should not be used inside the HybridModel and CrossValidation post-processor. This is because this type ROMs are constructed using the stochastic collocation method, and can not be retrained using the current structure of HybridModel and CrossValidation post-processor.
What did you expect to see happen?
What did you see instead?
Do you have a suggested fix for the development team?
An error should be raised if this type of ROMs are used in the HybridModel and CrossValidation post-processor.
Please attach the input file(s) that generate this error.  The simpler the input, the faster we can find the issue.

For Change Control Board: Issue Review
This review should occur before any development is performed as a response to this issue.

[x] 1. Is it tagged with a type: defect or improvement? improvement
[x] 2. Is it tagged with a priority: critical, normal or minor? minor
[x] 3. If it will impact requirements or requirements tests, is it tagged with requirements? N/A
[x] 4. If it is a defect, can it cause wrong results for users? If so an email needs to be sent to the users. N/A
[x] 5. Is a rationale provided? (Such as explaining why the improvement is needed or why current code is wrong.) Yes

For Change Control Board: Issue Closure
This review should occur when the issue is imminently going to be closed.

[x] 1. If the issue is a defect, is the defect fixed?
[x] 2. If the issue is a defect, is the defect tested for in the regression test system?  (If not explain why not.)
[x] 3. If the issue can impact users, has an email to the users group been written (the email should specify if the defect impacts stable or master)?
[x] 4. If the issue is a defect, does it impact the latest stable branch? If yes, is there any issue tagged with stable (create if needed)?
[x] 5. If the issue is being closed without a merge request, has an explanation of why it is being closed been provided?

Approved to close via merge.


Sorted imports

See https://github.com/wjakob/nanobind/issues/420#issuecomment-1986957935

Regarding the PackagesDict data structure. Can this not be done in an easier way all the way at the end when the import list is being generated? Instead of traversing sorted(), you could use sorted(.., key=) where a key looks at the module name that distinguishes between typing + collections, anything starting with self.module.__name__.split('.')[0], and everything else

Instead of traversing sorted(), you could use sorted(.., key=)

This would be the best solution, but I want to add extra line break between imports from different party, e.g.:
from typing import Sequence

from numpy.typing import ArrayLike

instead of:
from typing import Sequence
from numpy.typing import ArrayLike

It would not be possible to tell where does stdlib imports end and 3rd party imports start -> Unable to add line break
However, I did simplify things a bit in latest commit, please check

Alternative solution:
    def get(self) -> str:
        """Generate the final stub output"""
        s = ""
        last_party = 0

        for module in sorted(self.imports, key=lambda module: str(self.check_party(module)) + module):
            imports = self.imports[module]
            items: List[str] = []
            party = self.check_party(module)

            for (k, v1), v2 in imports.items():
                if k is None:
                    if v1 and v1 != module:
                        s += f"import {module} as {v1}\n"
                    else:
                        s += f"import {module}\n"
                else:
                    if k != v2 or v1:
                        items.append(f"{k} as {v2}")
                    else:
                        items.append(k)

            items = sorted(items)
            if items:
                items_v0 = ", ".join(items)
                items_v0 = f"from {module} import {items_v0}\n"
                items_v1 = "(\n    " + ",\n    ".join(items) + "\n)"
                items_v1 = f"from {module} import {items_v1}\n"
                s += items_v0 if len(items_v0) <= 70 else items_v1

            if party != last_party:
                s += "\n\n"
                last_party = party

        s += "\n"
        s += self.put_abstract_enum_class()

        # Append the main generated stub
        s += self.output

        return s.rstrip() + "\n"

I am not sure if this is cleaner though


Example bind does not work

{
    // Initialize a colorpicker like this.
    //
    $('#box').tinycolorpicker();

    // Try this to get access to the actual colorpicker instance.
    //
    var box = $('#box').data("plugin_tinycolorpicker");

    // Now you have access to all the methods and properties.
    //
    // box.setColor("#cccccc");
    // console.log(box.colorRGB);
    //
    // etc..

    // You can bind to the change event like this.
    //
    $box.bind("change", function()
    {
        console.log("do something whhen a new color is set");
    });
});

the $box should either be declared in the var or it should be just box.bind(...)
but the bind doesn't work in general; throws an error - not defined or not a function.

You are right i changed the docs. Now it should work :)


chromecast no cast destination found

Hi,
I am on Mac Sierra and am able to run chrome in docker with XQuartz using
docker run -it         --net host         --cpuset-cpus 0         --memory 512mb         -v /tmp/.X11-unix:/tmp/.X11-unix         -e DISPLAY=unix$DISPLAY         -v $HOME/Downloads:/root/Downloads         -v $HOME/.config/google-chrome/:/data               -v /dev/shm:/dev/shm         --name chrome     --security-opt seccomp:unconfined   -e DISPLAY=<IP_ADDRESS>:0  jess/chrome:beta
However chromecast is not found from the docker chrome (I have added the extension).
No Cast destinations found. Need help?
The only error I see in the docker running terminal after I search for chromecast is:
[1:1:1224/115700.713192:ERROR:CONSOLE(438)] "Uncaught TypeError: Cannot read property 'updateMaxDialogHeight' of null", source: chrome://media-router/media_router.js (438)
which looks like just an UI issue.
Ideas?

you arent in the same network as the chromecast. I would suggest looking into how network namespaces work. 1. if you are in a mac you are running in a virtual machine, then on the VM you are running in a linux network namespace. So yes, this means it cannot find the chromecast on your network because there are multiple levels of network you are in.


Make it harder to overwrite published, Production apps

This pull request proposes checking the Extension Registry API to see if an App Builder application is in a published state before attempting to deploy action code or web assets. It also proposes adding a flag which will bypass this check and allow users to deploy code for published Production applications.
Description
These changes implement the new Console API and Extension Registry API changes introduced here so that we can determine if an App Builder application is in a published state or not.
The getProject function was needed from the Console API so that we can fetch the appId for the project that is stored in the Extension Registry.
A new flag force-deploy has been added as a part of this pull request for the deploy command that bypasses this check.
Related Issue
Motivation and Context
To make it more difficult to overwrite published Production App Builder applications.
How Has This Been Tested?
Tested locally with npm run test and linked version of aio-cli-lib-console and aio-lib-console.
Screenshots (if appropriate):
Types of changes

[ ] 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 change)

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

@MichaelGoberling code conflict reported, can you resolve so the tests run?

I can't think of a better syntax for triggering this functionality so I will lets move forward with this.

This caused a bug: #587 (am fixing)


XeLaTeX won't render some special unicode chars

The "Right Single Quotation Mark" see https://unicode-table.com/en/2019/, is not present in the rendered PDF although it is in the latex output. This char is part of the name "Côte d’Ivoire" and has to be displayed correctly.

The problem might be that in the template.latex file no input encryption was defined. We can simply add  \usepackage[utf8]{inputenc} perhaps this helps. This package is also included in the booklet template so we should also have it here.
Another possible reason might be, that the font we use does not support this character. If solution 1 doesn't work, I can have a deeper look into the font and what characters it contains.


Added helpful tips about comment usage

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@kcrabtree 	Well done! Thank you!!


Raspbee on Pi3

Just a quick note to say many thanks for your work on this.
I was really struggling to get the Raspbee working on my Pi3 until I discovered a couple of config.txt lines from gjong at https://community.home-assistant.io/t/deconz-for-hass-io/41165/3 about disabling Bluetooth to allow the GPIO serial lines to work properly.
Might be a good addition to your README.

Thanks for the suggestion! Since it’s not a piece of info unique to running deCONZ in Docker, I didn’t think to include it, but I will add it to the README. Thanks again!


fix unpickling

Some pickle files for ConfidenceReports are broken because OrderedDict wants to use this constructor argument to unpickle itself. I actually haven't figured out what causes it to use this path so I haven't been able to make a unit test for it.

OK, it looks like I actually do have a unit test.
I submitted another PR with a unit test that I thought did not catch this problem. It turns out it does catch the problem, but only for a subset of the Travis builds:
https://github.com/tensorflow/cleverhans/pull/890


Installation error: python2 not found

Error line: gyp verb which failed Error: not found: python2
...
gyp verb check python checking for Python executable "python2" in the PATH
gyp verb which failed Error: not found: python2
gyp verb which failed     at getNotFoundError (D:_Template try\StarAdmin\node_modules\which\which.js:13:12)
gyp verb which failed     at F (D:_Template try\StarAdmin\node_modules\which\which.js:68:19)
gyp verb which failed     at E (D:_Template try\StarAdmin\node_modules\which\which.js:80:29)
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\which\which.js:89:16
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\index.js:42:5
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\windows.js:36:5
gyp verb which failed     at FSReqWrap.oncomplete (fs.js:152:21)
gyp verb which failed  python2 { Error: not found: python2
gyp verb which failed     at getNotFoundError (D:_Template try\StarAdmin\node_modules\which\which.js:13:12)
gyp verb which failed     at F (D:_Template try\StarAdmin\node_modules\which\which.js:68:19)
gyp verb which failed     at E (D:_Template try\StarAdmin\node_modules\which\which.js:80:29)
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\which\which.js:89:16
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\index.js:42:5
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\windows.js:36:5
gyp verb which failed     at FSReqWrap.oncomplete (fs.js:152:21)
gyp verb which failed   stack: 'Error: not found: python2\n    at getNotFoundError (D:\_Template try\StarAdmin\node_modules\which\which.js:13:12)\n    at F (D:\_Template try\StarAdmin\node_modules\which\which.js:68:19)\n    at E (D:\_Template try\StarAdmin\node_modules\which\which.js:80:29)\n    at D:\_Template try\StarAdmin\node_modules\which\which.js:89:16\n    at D:\_Template try\StarAdmin\node_modules\isexe\index.js:42:5\n    at D:\_Template try\StarAdmin\node_modules\isexe\windows.js:36:5\n    at FSReqWrap.oncomplete (fs.js:152:21)',
gyp verb which failed   code: 'ENOENT' }
gyp verb check python checking for Python executable "python" in the PATH
gyp verb which failed Error: not found: python
gyp verb which failed     at getNotFoundError (D:_Template try\StarAdmin\node_modules\which\which.js:13:12)
gyp verb which failed     at F (D:_Template try\StarAdmin\node_modules\which\which.js:68:19)
gyp verb which failed     at E (D:_Template try\StarAdmin\node_modules\which\which.js:80:29)
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\which\which.js:89:16
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\index.js:42:5
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\windows.js:36:5
gyp verb which failed     at FSReqWrap.oncomplete (fs.js:152:21)
gyp verb which failed  python { Error: not found: python
gyp verb which failed     at getNotFoundError (D:_Template try\StarAdmin\node_modules\which\which.js:13:12)
gyp verb which failed     at F (D:_Template try\StarAdmin\node_modules\which\which.js:68:19)
gyp verb which failed     at E (D:_Template try\StarAdmin\node_modules\which\which.js:80:29)
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\which\which.js:89:16
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\index.js:42:5
gyp verb which failed     at D:_Template try\StarAdmin\node_modules\isexe\windows.js:36:5
gyp verb which failed     at FSReqWrap.oncomplete (fs.js:152:21)
gyp verb which failed   stack: 'Error: not found: python\n    at getNotFoundError (D:\_Template try\StarAdmin\node_modules\which\which.js:13:12)\n    at F (D:\_Template try\StarAdmin\node_modules\which\which.js:68:19)\n    at E (D:\_Template try\StarAdmin\node_modules\which\which.js:80:29)\n    at D:\_Template try\StarAdmin\node_modules\which\which.js:89:16\n    at D:\_Template try\StarAdmin\node_modules\isexe\index.js:42:5\n    at D:\_Template try\StarAdmin\node_modules\isexe\windows.js:36:5\n    at FSReqWrap.oncomplete (fs.js:152:21)',
gyp verb which failed   code: 'ENOENT' }
gyp verb could not find "python". checking python launcher
gyp verb could not find "python". guessing location
gyp verb ensuring that file exists: C:\Python27\python.exe
gyp ERR! configure error
gyp ERR! stack Error: Can't find Python executable "python", you can set the PYTHON env variable.
gyp ERR! stack     at PythonFinder.failNoPython (D:_Template try\StarAdmin\node_modules\node-gyp\lib\configure.js:483:19)
gyp ERR! stack     at PythonFinder. (D:_Template try\StarAdmin\node_modules\node-gyp\lib\configure.js:508:16)
gyp ERR! stack     at D:_Template try\StarAdmin\node_modules\graceful-fs\polyfills.js:284:29
gyp ERR! stack     at FSReqWrap.oncomplete (fs.js:152:21)
gyp ERR!  Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Andrew\AppData\Roaming\npm-cache_logs\2019-04-20T11_07_58_096Z-debug.log
PS D:_Template try\StarAdmin>

Hi @andrewdaylondon , Please specify the command, running which this error occurs.

closing since the issue is inactive.

same issue when running npm install

running this command from powershell may resolve your issue : npm install -g --production windows-build-tools


Azure EA Content Pack reference

I belief the EA Content Pack is deprecated. There was a new Consumption Insight connector (https://docs.microsoft.com/en-us/power-bi/desktop-connect-azure-consumption-insights), but has meanwhile also been update to Azure Costmangement PowerBi Connector (https://docs.microsoft.com/en-us/power-bi/desktop-connect-azure-cost-management).
Please update the table above.

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

ID: 0725b78b-10ee-3868-12be-d9b6c7a2740b
Version Independent ID: fa71f1c3-7dc5-ac48-6eaf-3d880b84cf38
Content: Cost Management tools in Azure - Cloud Adoption Framework
Content Source: docs/govern/cost-management/toolchain.md
Service: cloud-adoption-framework
Sub-service: govern
GitHub Login: @BrianBlanchard
Microsoft Alias: brblanch

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#206123

This issue has been resolved. Thanks!


No works on grafana 4.6.3

Hi,
I upgrade discrete panel in version 0.0.8-pre on my grafana 4.6.3 install and now all my existant Discrete panels and new discrete panel not works.
I already have an error 404.
After research it seems same bug as #46
Edit:
I try on fresh grafana 4.6.3 and it works fine.
My current install (with the bug) is launched over docker

Ok it's my bad, after research the problem is the CloudFront cache in our AWS infra.
I close that.
Sorry


Suggestion: Update badges to reflect full FPC and Lazarus versions

Hey Chris,
If you look at the combinations of Lazarus and FPC versions running under the Actions, you can put labels like this:
https://img.shields.io/badge/Free Pascal-3.0.0~3..0.4-blue.svg :
https://img.shields.io/badge/Lazarus-1.6.2~2.0.8-blue.svg :
Cheers,
Gus

Done. I've updated the badges (linked to the .svg images as shields.io servers are not always reliable)

Hey Chris,
Yeah, I noticed you did that for the previous one. Good call.
Cheers,
Gus


Fix for no ipv6

Coverage decreased (-0.06%) to 78.813% when pulling ae052fcc5cdd612ca8bd30d7e52e12241c23865c on eatonphil:fix-for-no-ipv6 into 3923df24cde5ee6b43d700ddf0d4ed7aa60090bb on linode:master.

Coverage decreased (-0.04%) to 78.84% when pulling ae052fcc5cdd612ca8bd30d7e52e12241c23865c on eatonphil:fix-for-no-ipv6 into 3923df24cde5ee6b43d700ddf0d4ed7aa60090bb on linode:master.


The sign light is off when starting a new game (Part 1)

Describe the bug
The sign light is off when starting a new game (Part 1)
This happens from the commit:
Fix the hotel rooms have incorrect name (https://github.com/scemino/engge/commit/549aa6a2cb4952d715d71505297d3dd9bdfbe5ae)
until the last commit, to date:
Fix regression with object icons (https://github.com/scemino/engge/commit/c8bd7a9b758c267a4eed33ab66f72ea3238847a0)
To Reproduce
When starting a new game
Expected behavior
The light must be on, in order to break it with the stone
Screenshots
none
Desktop (please complete the following information):

OS: Windows x64
Version 10

Additional context
none

Now it works perfect :)


Seems like MerchantDelegate Regression

After updated to 0.14.0, MerchantDelegate don't calls didChangeStatesFor method in Sandbox when subscription expired. For example, I bought weekly subscription = 3 min period in Sandbox, but delegate method don't calls.

The validators now have an inbuilt leeway. The reason for this is that the iTunes Store server does not send renewal receipts in a timely fashion. It can sometimes take many days past the renewal date for the device to receive a fresh receipt. You can customize the configuration if you need to use a different leeway period.


Finer time resolution

I initially used your library to do multitasking on Arduino for my robot and 3D printer. I found that the time resolution (in milli-seconds) is not fine enough for my applications. I ended up copying your library and replacing all the millis() with micros().
It would be nice if TaskScheduler class could be updated to support finer time resolution.

Funny you say that - please check out the "testing" branch - I am testing exactly that!
#define _TASK_MICRO_RES			// Support for microsecond resolution
before include and recompile.

:)
Couldn't wait to see it released to the main branch.

Working on it. Would greatly appreciate if you could run some tests...
Do you have a website for your project? Very curious...

Sure, I will test it out tonight or tomorrow.
Well... regarding my robot project... you can check out my website: http://www.hackarobot.com
However, I didn't publish anything related to multi-tasking because I want to get the official support from your library first.
Regarding my 3D printer project. I am completely re-writing Marlin from scratch using C++. I am moving a few motors at the same time and millisecond resolution is simply not good enough.  With the modified TaskScheduler class, the motors moves much more smoothly.

I tried it.
I included it in a simple program. It seems to work if the #include <TaskScheduler.h> is added in the main ".ino" file.
However, when I try to include it in the header file in a Object class (e.g. Motor.h), it doesn't find the library. Any idea how I can update the search path? (note, TaskScheduler.h is already placed at
Arduino --> Libraries --> TaskScheduler --> TaskScheduler.h

What if you place #include <TaskScheduler.h> before #include <Motor.h> in the .ino?  Should have everything defined for the Motor.h to work. No?

Well... it doesn't work even if I include TaskScheduler in .ino before anything else.
Here is how my code look...
In Motor class, I need to pass the pointer of Scheduler class so that all motors refer to the same scheduler. Therefore, I need to include TaskScheduler in Motor.h
class Motor {
private:
Scheduler *sch;
...
It seems to be just a problem of the search path. I don't know how to get around this.
On the other hand, if I copy your .h file, rename everything to something like "MyScheduler.h" and place the .h in the same directory where .ino file sits, it compiles just fine.

Just to confirm:
You have two libraries: Motor and TaskScheduler, both in the 'libraries' folder of the Arduino sketch root folder; you have #include <TaskScheduler.h> in the Motor.h file, and it does not find it? Am I getting it correctly?
Can I look at the code (if you don't mind?)

I just ran a test:
In the TaskScheder .h file:
#include <Arduino.h>
#include <I2C.h>
I2C a;

#ifndef _TASKSCHEDULER_H_
#define _TASKSCHEDULER_H_

and it worked fine... Found I2C.h and compiled the sketch... Which version of IDE are you on?

No, Motor class only exists in the folder where the main .ino is located. Perhaps, I should move it to the libraries folder.

I am using 1.6.5 IDE

I am on 1.6.7. I believe there was some significant rework on the pre-processor front.
And yes, I would recommend moving the Motor library to 'libraries' folder. That should work.
To access current scheduler (if you compile with _TASK_PRIORITY option) you have a static method in the Scheduler class:
static Scheduler& currentScheduler();

would that be an alternative?

By the way, I can only imagine how much fun it is to work with your robot. I built something similar, and it was a lot of fun:
http://www.instructables.com/id/Arduino-Nano-based-Hexbug-Scarab-Robotic-Spider

Thanks. Your robot looks awesome.
By the way, I think your library looks fine. I will move the motor class to the libraries. It would work. Closing this issue. Thanks for helping

Thomas,
One question: can I reference your work in the ReadMe file for TaskScheduler?I want people to see what the library can do!
Thanks in advance.Anatoli
Date: Tue, 2 Feb 2016 20:56:58 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [TaskScheduler] Finer time resolution (#12)
Thanks. Your robot looks awesome.
By the way, I think your library looks fine. I will move the motor class to the libraries. It would work. Closing this issue. Thanks for helping
—
Reply to this email directly or view it on GitHub.

You are more than welcome to do so.

Thank you! Would you mind mentioning TaskScheduler somewhere there? (so people believe me.. :)
Date: Wed, 3 Feb 2016 21:19:41 -0800
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>CC<EMAIL_ADDRESS>Subject: Re: [TaskScheduler] Finer time resolution (#12)
You are more than welcome to do so.
—
Reply to this email directly or view it on GitHub.

let me polish my sample code a little bit. I will let you know once I uploaded it. Of course, I will mention TaskScheduler because it is such a great library. It is essential to my application.

Well, thank you. It is very important for me to know my work is essential to other projects.

Sent from a mobile device. Apologies for accidental typos.
From: Thomas Lee
Sent: Thursday, February 4, 11:25 AM
Subject: Re: [TaskScheduler] Finer time resolution (#12)
To: arkhipenko/TaskScheduler
Cc: Anatoli Arkhipenko
let me polish my sample code a little bit. I will let you know once I uploaded it. Of course, I will mention TaskScheduler because it is such a great library. It is essential to my application.
—
Reply to this email directly or view it on GitHub.


MAISTRA-2637 Disable ingress controller in istiod

I'm in the process of making sure 2.0 works on OpenShift 4.9+ and I found that even 2.1 doesn't work there. This needs to go into 2.0, too. Not sure if anyone needs this. If they do, maybe we need to only disable it in 4.9+ clusters.


Refactor Point

Closes https://github.com/ONSAS/ONSAS.jl/issues/173.

Codecov Report

Merging #179 (e3728d6) into main (ce58d48) will decrease coverage by 0.18%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #179      +/-   ##
==========================================
- Coverage   90.19%   90.01%   -0.18%
==========================================
  Files          40       40
  Lines        1061     1062       +1
==========================================
- Hits          957      956       -1
- Misses        104      106       +2

Impacted Files
Coverage Δ

src/Elements/Elements.jl
68.51% <ø> (ø)

src/Elements/Node.jl
60.00% <0.00%> (-6.67%)
:arrow_down:

src/Elements/Tetrahedron.jl
100.00% <ø> (ø)

src/Meshes/PointEvalHandler.jl
100.00% <ø> (ø)

src/StructuralModel/Structure.jl
96.96% <ø> (ø)

src/Utils.jl
83.33% <ø> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This issue:
ONSAS.StructuralAnalyses.StaticAnalyses.StaticState: Test Failed at /home/runner/work/ONSAS.jl/ONSAS.jl/test/test_StaticAnalyses.jl:140
  Expression: ≈((internal_forces(default_s))[1:6], fᵢₙₜ_e_1, rtol = RTOL)
   Evaluated: [0.7474039428181393, 0.3023418392007722, 0.8429027607606735, NaN, 0.36602645255495136, 0.7011507941876242] ≈ [0.7474039428181393, 0.3023418392007722, 0.8429027607606735, 0.7811307013064757, 0.36602645255495136, 0.7011507941876242] (rtol=0.05)

is a bug that occurs frequently in tests, maybe is related with the random StaticState defined. By the moment I think this is not relevant (since the end to end examples are passing the tests 👍🏼)
If we update the branch I think this is ready to be merged ?


Allow filesystem registry from git repository

Just set "kind" to "filesystem-from-git" and use the same parameters of "git".
            "kind": "filesystem-from-git",
            "baseline": "1603f26e9c52e8fe9dcee9266d939aa14e010221",
            "reference": "main",
            "repository": "https://git.example.com/repository.git",

What do you think about this feature ?

Sure.
I have a project. It has a few external dependencies (vtk, hdf5, Qt, etc...) and internal ones (requires our own registries, one is public for our open source libraries, the other is private for our proprietary libraries).
I'm trying to convince my colleagues to migrate from an old-fashioned CMake (an ExternalProject_Add for each external dependency and a submodule for each internal dependency) to vcpkg.
Gradually, I migrated from ExternalProject_Add to vcpkg ports and created ports for our internal dependencies. I centralized everything in a vcpkg-configuration.json file for ease of use of vcpkg.
I like the “git” registry because there's no need to clone manually and you can specify the commit precisely (although I don't like the fact that you have to specify a branch, which seems to me to be a duplication and causes problems cf. https://github.com/microsoft/vcpkg/discussions/42300). On the other hand, the procedure for modifying commits is a bit laborious (“git add files”, “git commit”, “vcpkg format-manifest”, “vcpkg x-add-version”, “git add versions”, “git commit”, “git push”).
I like filesystem registries because modifying a port consists in duplicating a folder, incrementing the version and updating the version/baseline.json / package.json files. No need for complicated git commands. But it does require manual cloning of our two registries. My colleagues don't understand the point of deleting submodules (our internal dependencies) and replacing them with others (our registries).
My “filesystem-from-git” has the advantage of both: automatic cloning and easy updating.
Yes, it may add confusion (a better name perhaps?). But I think that “git” for "registry from a git repository" and “filesystem” for "registry from a folder" are already confusing. It doesn't indicate that the architecture of the “ports” folders is not the same without having read the documentation.

You can do that now without needing a new feature.
Add your filesystem registry's repository as a submodule on your project and then have the vcpkg-configuration.json point to the submodule directory using a relative path.

But as I said, I'd rather not add submodule in project.
I have about ten repositories. One is the main projet, the other are librairies. CI is configured for each repo. I would like not to add a submodule in each repository but a single vcpkg-configuration.json is fine.

What do you think about this feature ?

Paraphrasing @ras0219-msft , it's difficult to evaluate how we feel about this feature because we don't have a great understanding of the problems the feature is intended to solve. Adding an entirely new registry format is not a small undertaking in terms of product complexity, documentation that would need to be written, and potentially tooling added to support that (e.g. does x-add-version need to be taught about this new thing?) so the problems it solves would need to be big.

I like the “git” registry because there's no need to clone manually and you can specify the commit precisely (although I don't like the fact that you have to specify a branch, which seems to me to be a duplication and causes problems cf. microsoft/vcpkg#42300). On the other hand, the procedure for modifying commits is a bit laborious (“git add files”, “git commit”, “vcpkg format-manifest”, “vcpkg x-add-version”, “git add versions”, “git commit”, “git push”).

It seems like "editing git registries needs to be made easier" is the real feedback here, and we need to fix that by, for example, making x-add-version format the manifest for you, and not require committing first, would solve the problem without introducing a new on-disk format?

I like filesystem registries because modifying a port consists in duplicating a folder, incrementing the version and updating the version/baseline.json / package.json files. No need for complicated git commands. But it does require manual cloning of our two registries. My colleagues don't understand the point of deleting submodules (our internal dependencies) and replacing them with others (our registries).

If it's a filesystem registry you would also need to duplicate an entire baseline key which seems more difficult to do than needing to commit first...


Protolock plugin for PGV annotations

Protolock is a tool for protecting against breaking changes to proto files. By default, Protolock only validates changes to proto messages and RPCs. It does not understand proto options like PGV.
While Protolock does not natively understand PGV, it does support plugins. We can write a plugin that teaches Protolock how to distinguish breaking changes in PGV annotations.

This is an excellent idea. First order of business would be to define what it means for a change to an annotation to be considered breaking.

A degenerate compatibility check would say that any change to any validation rule is a breaking change.
A more nuanced compatibility check would say that removing a rule is OK and broadening an existing rule is OK, but adding new rules is a breaking change.


fix async cbs error

the latest code in the master branch would result in the following error:
Traceback (most recent call last):
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\azure\eventhub\aio\_client_base_async.py", line 487, in _do_retryable_operation
    return await operation()
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\azure\eventhub\aio\_client_base_async.py", line 438, in _open
    self._client._address.hostname, auth
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\uamqp-1.4.2-py3.7-win-amd64.egg\uamqp\async_ops\client_async.py", line 241, in open_async
    await self._build_session_async()
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\uamqp-1.4.2-py3.7-win-amd64.egg\uamqp\async_ops\client_async.py", line 192, in _build_session_async
    **self._internal_kwargs)
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\uamqp-1.4.2-py3.7-win-amd64.egg\uamqp\authentication\cbs_auth_async.py", line 283, in create_authenticator_async
    kwargs.update(self._internal_kwargs)
AttributeError: 'JWTTokenAsync' object has no attribute '_internal_kwargs'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "D:/Projects/azure-sdk-for-python/sdk/eventhub/azure-eventhub/samples/async_samples/send_async.py", line 107, in <module>
    loop.run_until_complete(run())
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\asyncio\base_events.py", line 587, in run_until_complete
    return future.result()
  File "D:/Projects/azure-sdk-for-python/sdk/eventhub/azure-eventhub/samples/async_samples/send_async.py", line 97, in run
    await send_event_data_batch(producer)
  File "D:/Projects/azure-sdk-for-python/sdk/eventhub/azure-eventhub/samples/async_samples/send_async.py", line 29, in send_event_data_batch
    event_data_batch = await producer.create_batch()
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\azure\eventhub\aio\_producer_client_async.py", line 355, in create_batch
    await self._get_max_mesage_size()
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\azure\eventhub\aio\_producer_client_async.py", line 119, in _get_max_mesage_size
    EventHubProducer, self._producers[ALL_PARTITIONS]
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\azure\eventhub\aio\_producer_async.py", line 119, in _open_with_retry
    self._open, operation_need_param=False
  File "C:\Users\yuling\Anaconda3\envs\env37\lib\site-packages\azure\eventhub\aio\_client_base_async.py", line 505, in _do_retryable_operation
    raise last_exception
azure.eventhub.exceptions.EventHubError: 'JWTTokenAsync' object has no attribute '_internal_kwargs'
'JWTTokenAsync' object has no attribute '_internal_kwargs'

Process finished with exit code 1

Root cause:
self._internal_kwargs is not available on the JWTTokenAsync before calling create_authenticator_async which leads to the error. The fix should be simple, let the parent call handle to kwargs/loop.
However, I'm curious why it is not detected by the integrated livetest of EH/SB within in the uamqp livetest pipeline at the first place. Maybe our livetest pipeline has a bug.

@swathipil , I'll merge my PRs, generate wheels and use my old way (upload to a blob, and specify the uamqp wheel url to install in EH/SB livetest pipeline) to verify the uamqp changes first before we verify/fix the uamqp livetest pipeline. 😉
I have created a separate issue: https://github.com/Azure/azure-uamqp-python/issues/273