Fix node tests across all providers

What happened?
A recent change https://github.com/pulumi/pulumi/pull/12759 makes Pulumi consult package.json when running Node acceptance tests for the providers, which now breaks because they reference non-existing TypeScript output files.  Pulumi CLI uses ts-node to compile the projects in-memory and does not generate these files:
"main": "bin/index.js"
"typings": "bin/index.d.ts"

Current hypothesis is that before the change these files were simply ignored in acceptance tests. We will verify this shortly before proceeding.
The suggested remediation is to drop "main" and "typings" from package.json under examples/ from every provider.
If it is true that package.json were not consulted by ProgramTest perhaps another workaround is to remove them from repos so we continue with prior behavior where they "didn't exist in practice."

[ ] Link to a writeup on the core issue
[ ] Automate remediation as a source migration in ci-mgmt
[ ] Roll out to ecosystem providers via update-workflows-ecosystem-providers
[ ] List out providers that are not covered by the automatic workflow and ensure they are manually updated

Expected Behavior
Acceptance tests work.
Steps to reproduce
TBD
Output of pulumi about
N/A
Additional context
N/A
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Platform is reverting the change but we should go ahead and fix this up as this removes warnings from the acceptance test process.

Do we still need to do this?

Nope. Node tests are working so we're good to close.


hotspots: Turn on hotspots for new users.

Merged, after writing a proper commit message, thanks @rishig!


"φανελακι" does not have the same stem as "φανελα"

@chief I think this is true for many (all?) diminutives. For example, «ΠΑΙΧΝΙΔΑΚΙ» is stemmed to «ΠΑΙΧΝΙΔΑΚ», while «ΠΑΙΧΝΙΔΙ» is stemmed to «ΠΑΙΧΝΙΔ». I will look into the handling of the suffix «ΑΚΙ» and try to fix it.

Diminutive forms are not stem by choice. I can't recollect the reason but if you check the stemming samples you can see which words will get stemmed wrongly.

what about 'κοκολαλάκι'? it only matches 'κοκολαράκι'

@astathopoulos hmm I was afraid of that - ok I'll look into it, thanks!

@greenonion Try on skroutz "φουρνος" and "φουρνακι". You don't want to get the same results on this searches.

@astathopoulos Yep, I see. So you think it will lead to overstemming.

Yep!

Mark it as wontfix then and revisit on a per-case basis if the need arises?


sending email with multiple content types (text+html)  java

Expected content
Actual content
Link to page:

Hello @VikasAmbare,
Seems like you're struggling, but I'm not really sure what your issue is. Since this doesn't seem like a docs-related issue, please contact our Support team directly and they can help you out. You can access support contact options by logging into https://support.sendgrid.com.
SendGrid Docs Team


typo ("no"  isn't of "not")

@jgscherber : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off
Reviewed by Docs team
@jgscherber  Thanks for your feedback. We've submitted your documentation change, and the updated article should be live by tomorrow. Please let us know if there's anything more we can do for you. To follow up on this issue, leave us a comment with an @mention.

@shsagir: I'm sorry - only the author of this article, @orspod, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Adjusting code to parse the new challenge, adding test for it

CF have introduced the new challenge, taking this fix https://github.com/codemanki/cloudscraper/pull/287 and adapting cloudscraper to support both old and new challenges.
Fixes - https://github.com/codemanki/cloudscraper/issues/285 https://github.com/codemanki/cloudscraper/issues/286

Travis tests have failed
Hey @codemanki,
Please read the following log in order to understand the failure reason.
It'll be awesome if you fix what's wrong and commit the changes.
Node.js: 8
View build log

     npm test

><EMAIL_ADDRESS>test /home/travis/build/codemanki/cloudscraper
> npm run lint && npm run test:typescript && nyc --reporter=html --reporter=text mocha

><EMAIL_ADDRESS>lint /home/travis/build/codemanki/cloudscraper
> eslint --ext .json --ext .js --ext .ts .

/home/travis/build/codemanki/cloudscraper/index.d.ts
  80:40  warning  Unexpected any. Specify a different type  @typescript-eslint/no-explicit-any

/home/travis/build/codemanki/cloudscraper/index.js
  362:3   error  Expected space(s) after "if"                 keyword-spacing
  373:3   error  Expected space or tab after '//' in comment  spaced-comment
  373:49  error  Trailing spaces not allowed                  no-trailing-spaces

✖ 4 problems (3 errors, 1 warning)
  3 errors and 0 warnings potentially fixable with the `--fix` option.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>lint: `eslint --ext .json --ext .js --ext .ts .`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>lint script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!  Test failed.  See above for more details.

TravisBuddy Request Identifier: b5d09aa0-11c8-11ea-b428-cb0bccf60dd8

Hello, i need help with cloudscraper i have a script for cloudflare bypass uam and stoped work
Hwo can help me to ifx it will pay 50USD BTC /paysafecard
discord : marco#9606 add me

Error solved, pro-src is awesome! help me to fix it
Thank


Update deps

Update dependencies to latest stable version

thank you very much for the help, you could do another PR with the change or change this for yarn.lock

Do you want me to delete the .lock file?

if you can delete it, it would be great, for me to add it later and that it does not form incorrectly because there are some dependencies that I want to delete and add others like scss (sass)

Sure


Add clear screen functionality

Still under 1 kloc

it does not work in macOS , why not do :
#ifdef _WIN32
system("cls");
#else
system("clear);
#endif

system() is bad function because when it's executed, it does a lot of things and take more resources on your computer. First, it suspends your program and it calls OS to make shell as subprocess and then look for command specified in system function and allocate memory for the commannd and after command execution, deallocate memory, exit OS and finally, resume your program.

const char *CLEAR_SCREEN_ANSI = "\x1b[H\x1b[J";


README: reference correct package path

It seems #175 missed the READMEs. Note that the Linux Status, Window Status and Gitter badges were left unchanged, as the link they would point to would have to exist (or be in use, in the case of Gitter).

+1 for merging this!

Rebased and tweaked comment, but merged as:
commit 43402d9c8feb29ccf6aa4b72d2e890ca91eb79ea
Author: Alex Diaconu<EMAIL_ADDRESS>Date:   Wed Feb 24 19:31:35 2016 +0200
fixes #189 fix import paths and locations in README

Thanks.


Bug: QA Test RATA Data

Issue:
Received error msg:  [RATA-79-A] - You did not provide [pitchAngle], which is required for [RATA Traverse]
Steps to Recreate:

Check out Limestone (298) LIMI1
Select QA Cert Test
Select Relative Accuracy Test Type
Click on Import Data
Select 2022 Q2 Reporting Period
Click on Preview
Select RATA test number VFX-Q2-2022-001
Click Import button

Private Zenhub Image

Able to import the test number above without issue


what is this warning in the callbacks?

UserWarning: Method on_batch_end() is slow compared to the batch update (0.151899). Check your callbacks.
what is that warning? and why?
Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[ ] If running on TensorFlow, check that you are up-to-date with the latest version. The installation instructions can be found here.

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[ ] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Literally what it says, You are running something like saving the model or rendering images after each batch and it is taking longer than the batches themselves.
Are you doing something each batch that could be causing the issue? Or are you just asking how the warning can be disabled?
Cheers

yes i'm doing "reduce_lr" and "early_stopping" callbacks,
actually this message only appear if my learning rate is high (.1)
but if it is .001 it disappears.

Neither ReduceLROnPlateau nor EarlyStopping have batch end callbacks, only epoch end callbacks, at least in the current version. Does your version of Keras have batch_end callbacks in those 2 classes? Might be time to update. If that isn't the issue, maybe you just have really small batches and epochs so anything is slow compared to your batches.

@bstriner, how are you? Could you please help me?
I have the same problem.
UserWarning: Method on_batch_end() is slow compared to the batch update (0.151899). Check your callbacks.
What should I modify in my algorithm?
I have the following settings:
nb_train_samples=len(os.listdir(train_Class1_dir))+len(os.listdir(train_Class2_dir))
epochs=30
nb_validation_samples=len(os.listdir(validation_Class1_dir))+len(os.listdir(validation_Class2_dir))
batch_size=100
from keras.callbacks import EarlyStopping, ModelCheckpoint, Callback
class LossHistory(Callback):
def on_train_begin(self, logs={}):
self.losses = []
def on_batch_end(self, batch, logs={}):
    self.losses.append(logs.get('loss'))

earlystopper = EarlyStopping(patience=10, verbose=1)
checkpointer = ModelCheckpoint('Class1_Class2_weights.h5', verbose=1, monitor='val_acc', mode='max', save_best_only=True, save_weights_only=False)
print("test_generator")
test_generator = test_datagen.flow_from_directory(
test_dir,
target_size=(227, 227),
batch_size=batch_size,
class_mode='categorical')
print("validation_generator")
validation_generator = validation_datagen.flow_from_directory(
validation_dir,
target_size=(227, 227),
batch_size=batch_size,
class_mode='categorical')
for data_batch, labels_batch in train_generator:
print('data batch shape:', data_batch.shape)
print('labels batch shape:', labels_batch.shape)
break
Fit model
from keras.optimizers import SGD, Adam
sgd = SGD(lr=0.001, decay=1e-6, momentum=0.9, nesterov=True)
model.compile(loss='categorical_crossentropy',
optimizer=sgd,
metrics=['accuracy'])
print("fit_generator")
History = LossHistory()
Results_Train = model.fit_generator(
train_generator,
steps_per_epoch=nb_train_samples // batch_size,
epochs=epochs,
validation_data=validation_generator,
validation_steps=nb_validation_samples // batch_size,
callbacks=[History, checkpointer, earlystopper])

Is it possible to get this warning without using any callbacks?
I am using a DataGenerator to feed data, which has an on_epoch_end method. I am also getting this warning:

UserWarning: Method on_batch_end() is slow compared to the batch update (0.586719). Check your callbacks.

@xiaohk I am having the same issue. I scripted my own data generator where I read data from an .h5 file and perform data augmentation. I suspect that the GPU processes the batch through the network faster than the time it takes to generate the next batch. I will keep you updated if I find out how to solve it...
Cheers!

Hi @xiaohk.
It seems like the issue was caused by my CPU that generates the training batch (consisting of 8 training examples) much slower than the GPU processes it. I have increased the number of training examples per batch to 32 and the warning disappears.
Another approach that might solve this problem is to set use_mutiprocessing = TRUE in the fit_generator and select the appropriate number of workers to generate your training batches more efficiently.  More information about this is described in: [https://keras.io/models/sequential/]
Hope it helps!

Somehow this thread did not really help me understanding what's wrong.
The warning means exactly what it's saying (duh!)

Method on_batch_end() is slow compared to the batch update (factor_of_slowness)

So if it says compared to the batch update  update (2.059934) it means your validation (or whatever you're doing there) takes twice the time of training (which given a small batch size and a serious dev set is a pretty realistic scenario).
Apparently it warns you if the factor is above 0.5, and does not warn otherwise.

@Demetrio92 the threshold seems to be different for  Method (on_train_batch_begin):
WARNING: tensorflow:Method (on_train_batch_begin) is slow compared to the batch update (0.121552).
Check your callbacks.

Does somebody know what the difference in the warning means?

@ghost  Here is my self-define weightsSaver, only every self.N iterations tf will show this warning.
class WeightsSaver(Callback):
    def __init__(self, N):
        self.N = N
        self.batch = 0

    def on_batch_end(self, batch, logs={}):
        if self.batch % self.N == 0:
            name = '.\\model\\synDS_weights%09d.h5' % self.batch
            self.model.save_weights(name)
        self.batch += 1

Does yours show every iteration or some iterations once?

This discussion is going waaay off-topic, guys. Consider StackOverflow for specific problems and solutions.

Otherwise, you pollute the thread and anyone trying to figure out what does the warning in the OP mean will be reading through the code snippets on how to save weights on every N iterations.
Which is discussed here thoroughly, including the built-in CheckPoint option.

Neither ReduceLROnPlateau nor EarlyStopping have batch end callbacks, only epoch end callbacks, at least in the current version. Does your version of Keras have batch_end callbacks in those 2 classes? Might be time to update. If that isn't the issue, maybe you just have really small batches and epochs so anything is slow compared to your batches.

@bstriner How do you eliminate this "UserWarning: Method on_batch_end() is slow compared to the batch update (0.151899). Check your callbacks"?
Best Regards

Neither ReduceLROnPlateau nor EarlyStopping have batch end callbacks, only epoch end callbacks, at least in the current version. Does your version of Keras have batch_end callbacks in those 2 classes? Might be time to update. If that isn't the issue, maybe you just have really small batches and epochs so anything is slow compared to your batches.

@bstriner How do you eliminate this "UserWarning: Method on_batch_end() is slow compared to the batch update (0.151899). Check your callbacks"?
Best Regards

Any solution?

Any solution for this warning message?

@left-brain Increasing batch_size did the trick in my case

Any solution for this warning message?

reading this thread is the solution.

In my case it was the Tensorboard callback https://github.com/tensorflow/tensorflow/issues/37876#issuecomment-674450985.

WARNING:tensorflow:Callbacks method on_train_batch_end is slow compared to the batch time (batch time: 0.0032s vs on_train_batch_end time: 0.1142s). Check your callbacks.

Hi!, Im facing this issue too. Not sure its time to update it as its the latest branch of keras.
Do you think guys Is it related to CPU? Appreciate

the level of idiocy in this thread is unbelievable.

In my case, increasing the batch size resulted in a memory error. Fortunately, the kwarg experimental_steps_per_execution can be supplied in the model.compile() call, reducing how often on_batch_end callbacks are evaluated. This removed the warning, and should be reducing my training time by ~50%.

In my case, increasing the batch size resulted in a memory error. Fortunately, the kwarg experimental_steps_per_execution can be supplied in the model.compile() call, reducing how often on_batch_end callbacks are evaluated. This removed the warning, and should be reducing my training time by ~50%.

May I ask to what did you set it?  thought it should be a number representing the number of steps per execution, but didn't work and cannot find references.

In my case, increasing the batch size resulted in a memory error. Fortunately, the kwarg experimental_steps_per_execution can be supplied in the model.compile() call, reducing how often on_batch_end callbacks are evaluated. This removed the warning, and should be reducing my training time by ~50%.

Thanks！Set steps_per_execution=1000 in compile is very helpful! And TPU trainning speed 120ms/step -> 5ms/step


Removing aws_iam_service_linked_role from module

PR o'clock
Description
Removing this resource. I think it's not required and seems to be the source of various errors anyway.
It was added here: https://github.com/terraform-aws-modules/terraform-aws-eks/pull/91
And issues here:
https://github.com/terraform-aws-modules/terraform-aws-eks/issues/132
https://github.com/terraform-aws-modules/terraform-aws-eks/issues/144
If anyone gets an error about removing the aws_iam_service_linked_role.elasticloadbalancing resource, I think you can just remove it from the state: terraform state rm aws_iam_service_linked_role.elasticloadbalancing
Checklist

[x] terraform fmt and terraform validate both work from the root and examples/eks_test_fixture directories (look in CI for an example)
[ ] Tests for the changes have been added and passing (for bug fixes/features)
[x] Test results are pasted in this PR (in lieu of CI)
[x] Docs have been updated using terraform-docs per README.md instructions
[x] I've added my change to CHANGELOG.md
[x] Any breaking changes are highlighted above

@max-rocket-Internet this used to be required to fix the issue of EKS not being able to create ELB. Is that no longer an issue? How do you know we didn’t introduce a regression here?

Is that no longer an issue? How do you know we didn’t introduce a regression here?

Because I tested it. Also, the default AWS IAM policy arn:aws:iam::aws:policy/AmazonEKSClusterPolicy includes this now:
        {
            "Effect": "Allow",
            "Action": "iam:CreateServiceLinkedRole",
            "Resource": "*",
            "Condition": {
                "StringLike": {
                    "iam:AWSServiceName": "elasticloadbalancing.amazonaws.com"
                }
            }
        }

I would be curious if you still have that error when your cluster is using the default AWS IAM policy.

Ah ok. Got it thank you.


Compress option is depricated in openvpn 2.5

The compress option in openvpn is a security risk and is deprecated in openvpn 2.5.
Info about security risk HERE

https://github.com/angristan/openvpn-install#compression :)

Yes, but should there not be a question about enabling compression when creating a new cert?
Compression:
1 Enable compresion (not supported in openvpn 2.5)
2 No compression

I guess we could remove compression entirely 🙂

OpenVPN intend to remove compression, eventually.
https://community.openvpn.net/openvpn/wiki/Compression


rancher/mirrored-library-busybox:1.34.1 has critical vulnerability

Environmental Info:
K3s Version:

v1.25.14-rc1+k3s1
Node(s) CPU architecture, OS, and Version:

Linux node-131 3.10.0-957.el7.x86_64 #1 SMP Thu Nov 8 23:39:32 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:

1 server
Describe the bug:

The image rancher/mirrored-library-busybox:1.34.1 is used by local-path-provisioner component，it has critical vulnerability
Steps To Reproduce:

Installed K3s with local-path-provisioner component
use local-path-provisioner storageclass create pvc，help pod with image rancher/mirrored-library-busybox:1.34.1 will be created

Expected behavior:

rancher/mirrored-library-busybox:1.34.1 has no critical vulnerability
Actual behavior:

rancher/mirrored-library-busybox:1.34.1 has critical vulnerability and can't be setted to other busybox  version image tag.
Additional context / logs:

update the image to busybox:1.35.0 can resolve the problem

We'll take a look at updating it. I don't see that this is much of a direct concern, as it is only used to create paths for the local storage volume, and uses inputs that are validated by the Kubernetes storage API.

We'll take a look at updating it. I don't see that this is much of a direct concern, as it is only used to run the script that creates paths for the local storage volume, and uses inputs that are validated by the Kubernetes storage API.

Thank you for looking into this issue. I appreciate your prompt response. Although the vulnerability may not pose a direct concern due to the script's limited usage and inputs validated by the Kubernetes storage API, it would still be beneficial to update the busybox image to a version without critical vulnerabilities. This will provide an additional layer of security for our environment and can pass an image security scan. If possible, kindly consider updating the image to busybox:1.35.0 or a version that resolves the vulnerability. Thank you for your attention to this matter!

Hello @brandond In which release can we expect this CVE to be fixed?
We are also facing the same issue. Though the attack surface is limited, having a 8.8 CVE does not bode well with our product. If possible i kindly request you to update to a version of busybox greater than 1.35.0. Thank you!

Validated on v1.28.3-rc1+k3s1
Environment Details

Infrastructure

[ ] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

Ubuntu

Cluster Configuration:

1 server

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Install k3s
Run k apply -f https://raw.githubusercontent.com/rancher/distros-test-framework/main/workloads/amd64/local-path-provisioner.yaml && k describe pods -n kube-system and immediately describe the pod to be able to see the busybox version.

Validation Results:

Name:             helper-pod-create-pvc-c097124d-d7b4-4327-b540-75ff35b40dd9
Namespace:        kube-system
Priority:         0
Service Account:  local-path-provisioner-service-account
Node:             ip-172-31-11-244.us-east-2.compute.internal/<IP_ADDRESS>
Start Time:       Tue, 24 Oct 2023 17:46:10 +0000
Labels:           <none>
Annotations:      <none>
Status:           Pending
IP:
IPs:              <none>
Containers:
  helper-pod:
    Container ID:
    Image:         rancher/mirrored-library-busybox:1.36.1
    Image ID:
    Port:          <none>
    Host Port:     <none>
    Command:
      /bin/sh
      /script/setup
    Args:
      -p
      /var/lib/rancher/k3s/storage/pvc-c097124d-d7b4-4327-b540-75ff35b40dd9_local-path-storage_local-path-pvc
      -s
      524288000
      -m
      Filesystem
    State:          Waiting
      Reason:       ContainerCreating
    Ready:          False
    Restart Count:  0
    Environment:
      VOL_DIR:         /var/lib/rancher/k3s/storage/pvc-c097124d-d7b4-4327-b540-75ff35b40dd9_local-path-storage_local-path-pvc
      VOL_MODE:        Filesystem
      VOL_SIZE_BYTES:  524288000
    Mounts:
      /script from script (rw)
      /var/lib/rancher/k3s/storage/ from data (rw)
      /var/run/secrets/kubernetes.io/serviceaccount from kube-api-access-p9pqw (ro)
Conditions:
  Type              Status
  Initialized       True
  Ready             False
  ContainersReady   False
  PodScheduled      True
Volumes:
  data:
    Type:          HostPath (bare host directory volume)
    Path:          /var/lib/rancher/k3s/storage/
    HostPathType:  DirectoryOrCreate
  script:
    Type:      ConfigMap (a volume populated by a ConfigMap)
    Name:      local-path-config
    Optional:  false
  kube-api-access-p9pqw:
    Type:                    Projected (a volume that contains injected data from multiple sources)
    TokenExpirationSeconds:  3607
    ConfigMapName:           kube-root-ca.crt
    ConfigMapOptional:       <nil>
    DownwardAPI:             true
QoS Class:                   BestEffort
Node-Selectors:              <none>
Tolerations:                 op=Exists
Events:                      <none>


How to set up a mobile device to access web pages

At present, it seems that the webpage is accessed by a pc device. How to set it up to use a mobile device (mobile phone) to access it?

@tlerbao Hi
Didn't understand what you are asking?

Sorry, my English is terrible. What I mean is, how do I simulate access using a mobile device? How should I describe it?

I am sorry still don't get it.
You want to use the gbrowser toolwindow from a mobile?
This is only for IDE - so again I don't understand what you are trying to do here exactly

@Jonatha1983
Sorry,
I don't want to use Gbrowser on my phone. I want the website to be displayed in the form of a mobile device.
It seems like the window is just resized, but it still shows the PC version of the website.
There are some differences between accessing a website on a PC and on a mobile device, such as dragging. I tried setting the UserAgent, but it didn't work.

Ah ok now o got you
But unfortunately I don't know of a way to do it.
How would you do it with a normal browser?

Chrome's built-in developer tools can switch by using command + shift + m, allowing you to select different devices like iPhone 12/11/6, etc. Can't Gbrowser do this?
I recorded a video.
https://github.com/user-attachments/assets/ed0e1bb3-5e7e-411c-85e8-18e815ba7f29

I don't see this feature in the plugin—need to investigate a bit more.
Will update if I find something or it is something that needs to be implemented.

Thank you, great plugin


KeyError: 'outputs' when module has no outputs defined

I've been exploring the package to test my terraform modules and got confronted to this error and it was a bit confusing to debug.
When no output blocks are defined in the Terraform configuration, setting up tests like this (see below) raises a KeyError: 'outputs'. Adding a output block in the module fixes the problem.
@pytest.fixture
def tfplan():
    tf_vars = {
        'my_var': 'value
    }
    TF.setup(
        extra_files=[
            os.path.join(ROOT, '../', 'vars.tf')])
    plan = TF.plan(tf_vars=tf_vars, output=True)
    return plan

However Terraform configurations without output blocks are valid so we should be able to test them in the same way.
Would it make sense for you to prevent this error when no output is configured? If yes, I'd love to contribute a PR.
Cheers

Thanks for deep diving on this. It does make absolute sense to tolerate configurations with no outputs defined. If you have the time for a PR, it's of course more than welcome. Otherwise I'll try to get around to it next week.

The fix might be trivial. Can you check that the code in the branch above works for you?

Works perfect! Thanks a lot for the reactivity :)

Ahah thanks for confirming! Will merge and push on pypi shortly. Thanks a lot for the detailed report!


Cleanup Connection headers before passing the middleware chain

What does this PR do?
This pull request cleans the Connection headers before passing the middleware chain. When a connection header needs to pass through the middleware chain, the forwardedHeaders.connection option has to be used.
More

[X] Added/updated tests
[X] Added/updated documentation

Additional Notes
Co-authored-by: Romain<EMAIL_ADDRESS>

Is this the relevant PR that fixes https://github.com/advisories/GHSA-62c8-mh53-4cqv?
We want to backport the fix to a version of Traefik, that does not need Go 1.23, as we don't have it in the stable branch of Nixpkgs yet.
See https://github.com/NixOS/nixpkgs/pull/344149

Hello @Scrumplex,
Yes, indeed, this is the PR that fixes https://github.com/advisories/GHSA-62c8-mh53-4cqv, and there are no additional ones strictly in this regard.


[1.11.1] Chart is not defined - In Stats view

Description
I have a error in JS Chart is not defined in the container stats view.
So the charts doesn't show up.
Steps to reproduce the issue:

docker run -d -p 9000:9000 -v /var/run/docker.sock:/var/run/docker.sock portainer/portainer
Go to a random container stats view

Technical details:

Portainer version: 1.11.1
Portainer Docker image tag (latest/arm/windows...): latest
Platform (windows/linux): linux & macOS
Browser: Chrome

Yes, seems that there is an issue with the Javascript concatened files introduced with #422

This was due to a build issue. 1.11.1 has been updated on Dockerhub please pull the image again.

Thank's, it works 👍


Add support for https for openbadgekit

Especially since images are loaded through openbadgekit, it creates a mixed content (http over https)

Won't be necessary, we're migrating out of bagekit as of https://github.com/CoderDojo/community-platform/issues/1092


Fix regressions and issues introduced in v0.51.0

🛠 Changelog

Improves null safety in generating reservations from allocations
Reverts some optimizations to FRT calculation causing issues that prevent release to prod. These will be fixed later and reapplied then.
Fixes an issues with unit reservation unit count counting archived reservation units
Adds some missing fields to generated reservations from applications

🧪 Test plan

Automated tests

🚧 Dependencies

None

🎫 Tickets

TILA-3437
TILA-3443

/fast-forward

/fast-forward


constructors.allPropeties seems to default to true

I just migrated from 4.3.1 to 6.0.0
Afterwards I compared the generated sources, and discovered that now there were new constructors generated. I checked the documentation and it suggested that the default for this was false:
https://github.com/jsonschema2dataclass/js2d-gradle/blob/main/docs/usage/parameters_6.adoc#constructor-parameters
I changed this explicitly to false now:
            constructors {
                allProperties = false
            }

which seems to have done the trick. So please either change the default back or reflect this in the documentation.

FYI this is the structure before:
jsonSchema2Pojo {
    targetPackage.set("mypackage")
    source.setFrom("${project.rootDir}/src/main/resources/schema")
    generateBuilders.set(true)
    dateTimeType.set("java.time.ZonedDateTime")
    dateType.set("java.time.LocalDate")
}

and after:
jsonSchema2Pojo {
    executions {
        create("main") {
            klass {
                targetPackage.set("mypackage")
            }
            io {
                source.setFrom("${project.rootDir}/src/main/resources/schema")
            }
            methods {
                builders.set(true)
            }
            dateTime {
                dateTimeType.set("java.time.ZonedDateTime")
                dateType.set("java.time.LocalDate")
            }
            constructors {
                allProperties.set(false)
            }
        }
    }
}

This seems to come from jsonschema2pojo:
org/jsonschema2pojo/DefaultGenerationConfig.java:319

Hi @StefanLobbenmeierObjego, thank you for an issue. I'll look into it.

Thank you for your time, I've looked into it.
Plugin doesn't provide any explicit defaults beside few options removed from the public interface and well documented in the migration document. If some defaults has been changed in underlying processor, a user would experience sudden behaviour change.
Behavior of many other similar plugins (for example, Spotless) is similar to what has been implemented in the this plugin. Nevertheless, I'll accept any help (ideas and/or PRs), If there would be any ideas how to mitigate such sudden changes for a user.
Please note, that a plugin user can control processor version in several ways ways

by using the same plugin version
by explicitly specifying the version of the processor, as it shown in demos.

Please mind, off-release upgrades might not work as the processor might be incompatible with the current plugin version due to interface changes as it was before with jsonschema2pojo processor.

I'm closing this issue as resolved. You're welcome to ask for more information.


Also build format 12 subtables in Cmap::from_mappings if any char > BMP

Fixes https://github.com/googlefonts/fontc/issues/451
This is intended to match fontmake/fonttools output.
If the input mappings contain any characters beyond the basic multilingual plane (> 0xFFFF) we also build two cmap12 subtables (one for Unicode platform, another identical one for WIndows platform, both pointing to the same format 12 subtable) which will be a superset of the BMP-only cmap4 subtables.

intended to match fontmake/fonttools output

in reality this PR differs slightly in that it will not produce empty cmap format 4 subtables in the unlikely scenario the input mappings only contain characters beyond the BMP, which is what fontmake would do in this case, but an empty cmap subtable should be as good as an altogether absent one...


Support background color for a range of text

Hi Tomas,
I am in the process of implementing a highlighting method.
I am doing this simple code. I verified all values, they are all fine.
tempStype = "-fx-background-color: yellow;" ;
indexRange =richText.getSelection();
richText.setStyle(indexRange.getStart(), indexRange.getEnd(), tempStyle);
I notice a change in the range style, but the background is not affected. Is it broken? Can you verify?

This is now resolved by #166 thanks to @MewesK. The name of the property is -fx-background-fill.


npm can't run because of the long path, in V2.5.3-RELEASE

Cause:
In some os, long path is sensitive, it also causes some software run to fail.
Environment:
os: Windows 10
npm version: 9.2.0
project version: V2.5.3-RELEASE and laster
Example:

Solve 1: copy build/spotless-node-modules-prettier-format from success to fail
Solve 2: update project module name to small or medium length
Solve 3: try to break the file path limit


Add information on how to use LA100-P

This PR is to add information on how to use LA100-P to AMDS documentation.
I added;

the description of the burden register and current sensor gains with LA100-P, which has +/- 150 Aamp
the LA100-P to the BOM CSV file
the datasheet of LA100-P

@ngadiyar93, please review these changes above.

Thanks @noguchi-takahiro, I'll have you do a round of revision with @ngadiyar93. Then I am realizing that we'll need you to make these same changes here as well --> source is here.

Thanks, @ngadiyar93
I changed the sentences that you mentioned. And please review it and let me know of further improvements and requests.

Thank you, @ngadiyar93. Please review it again.
We also need to address this below.

Thanks @noguchi-takahiro, I'll have you do a round of revision with @ngadiyar93. Then I am realizing that we'll need you to make these same changes here as well --> source is here.

@elsevers this PR closed due to deleting its target branch. Do you still want this content to be merged?

Thanks @npetersen2. Good catch. I did not mean to close this PR.


⚠ GitLab Server has degraded performance

In b705125, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11657 ms

Resolved: GitLab Server performance has improved in ca3549f.


colcon package versions assume package already installed

The way colcon package versions are specified relies on having the current module already installed.
https://github.com/colcon/colcon-core/blob/caee5e64608912f52e86f79de1cf53b8c34713df/setup.cfg#L1-L3
At build time, we have version = attr: colcon_core.__version__ but there is no guarantee that colcon_core is in the python path at that time,
When you install colcon_core with poetry, this shows up as a ParseVersionError:
C:\Users\dan\Documents\colcon-ws>poetry install
Updating dependencies
Resolving dependencies...

  ParseVersionError

  Unable to parse "attr: colcon_core.__version__".

  at ~\.poetry\lib\poetry\_vendor\py3.8\poetry\core\semver\version.py:206 in parse
      202│         except TypeError:
      203│             match = None
      204│
      205│         if match is None:
    → 206│             raise ParseVersionError('Unable to parse "{}".'.format(text))
      207│
      208│         text = text.rstrip(".")
      209│
      210│         major = int(match.group(1))

This setup is also known to cause trouble with setuptools:
https://github.com/pypa/setuptools/issues/1724

see https://packaging.python.org/guides/single-sourcing-package-version/ for some recommended ways to specify the package version which don't have this problem.

The way colcon package versions are specified relies on having the current module already installed.
...
At build time, we have version = attr: colcon_core.__version__ but there is no guarantee that colcon_core is in the python path at that time,
...
see https://packaging.python.org/guides/single-sourcing-package-version/ for some recommended ways to specify the package version which don't have this problem.

The colcon-core package follows the best practice from the documentation you referenced - described in the note of option 1: using attr: ... in the setup.cfg file. As that documentation mentions you need at least version 46.4.0 of setuptools to not require the imported modules to be available which is why the build-from-source instructions mention that: https://colcon.readthedocs.io/en/released/developer/bootstrap.html#virtual-environment
What version of setuptools do you have when running into this problem? Assuming it is older than the above version does uodating it resolve the problem for you?

You’re right - it should work for setuptools 46.4.0. I somehow totally missed that note!
But setuptools is an install_requires and not a setup_requires and has a minimum version of 30.3.0. https://github.com/colcon/colcon-core/blob/caee5e64608912f52e86f79de1cf53b8c34713df/setup.cfg#L39
I suspect the issue is either that poetry does not implement the special AST analysis or that a clean venv does not have a bleeding edge setuptools. Will investigate deeper.

It's a super slippery bug to reproduce, and I'm having trouble pinning it down fully.
I think this might be a bug in poetry in that it takes a shortcut when comparing the package version against that in the lockfile, which bypasses the special behavior of setuptools.
I'm not sure if #381 will resolve the issue fully with poetry but I'll revisit if it doesn't.


Run tests on push master

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (checklist)

[x] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[x] Did you read the contribution guide?
[x] Did you check the code style? make lint (poetry install -E style).
[x] Did you make sure to update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you check that your code passes the unit tests pytest tests/ ?
[x] Did you add your new functionality to the docs?
[ ] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes

Related Issue

PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Closing issues
Put closes #XXXX in your comment to auto-close the issue that your PR fixes (if such).

This should fixes "No coverage uploaded for pull request base" in pull requests

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@0a0c9f7). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #38   +/-   ##
=========================================
  Coverage          ?   85.09%
=========================================
  Files             ?       44
  Lines             ?     1919
  Branches          ?        0
=========================================
  Hits              ?     1633
  Misses            ?      286
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0a0c9f7...43ee8c4. Read the comment docs.


Need to convert react app to typescript

We need to slowly convert the whole app to typescript.

We could work out of a new folder (ie. react-admin-ts) to move things over that way.

Well done @smattera


Not possible to install / uninstall extension via keyboard

I like how the keydown starts going through the list and how enter opens the extension description.
I would exprect that an enter on already selected list item installs / uninstalls that extension.
The alternative is that you open extension description on each up / down and on enter install / uninstall. I think this might also work nicely

Tab now moves focus out of search box, while the click anywhere in the viewlet still focuses the search box. Unforunetetly I did not find a differentiator in the FocusEvent so I had to add another listener for a press of a tab key.

I find the current way very confusing if I compare it to the tree. This really needs some fixing in the list widget. I will open a separate issue.

Agree - create me an issue and I will look into it next week. Thanks!

#8621 and #8622

Is there a particular reason there's no command for removing an extension?

Not really, what is your use case?

I want to be able to remove extension using only the keyboard. I don't think it deserves its own keybinding, but rather a command that's accessible via the command palette.

You should be able to do that in the latest insiders:

Cmd Shift X
Tab Tab
You should now be focused on the first extension's uninstall button.

Not a command sure, but definitely keyboard accessible.
We can think about commands yeah... if you'd make a PR for it, I'd gladly review it! It would need some sort of interaction with the quick open pick API in order to have the user pick which extension he'd like to uninstall.

@joaomoreno what is the quick open pick API?

Here's a good example


Adds management of extra inputs and outputs

Addresses #60 and #62

@lisad not complete yet, but it is at a place where your input will be helpful

@jeffkole I added a few comments, seems like the right direction

Goal for the employee pipeline is to have it fit all on one page.
Add additional pipelines that can show the complexities.

File with multiple kinds of events that are emitted as separate streams that are pushed to different phases (employee events -- promotion, pay raise, job change)
Traffic incidents or even bicycle and pedestrian counts
Retail events -- purchasing services or consumable things


Decompile Pesudo Code No Longer Works

It keeps asking me to load a binary when using the IDA plugin on Version 7.7. What do I do to allow me to decompile the functions in static analysis?

Can you be more specific please? Your IDA is version 7.7, but what about these:

What is you RetDec IDA plugin version?
What exactly (screenshot) does it do?
Did it work before at any point? Did you update IDA/RetDec?


Decaffeinate spec/spec-helper (Merge first: it's a dependency)

Helper for unit test.
PR needed before other decaffeination in spec, because this file is included.
I don't remove spec/spec-helper.coffee actually, because it's useful during decaffeination.
⚠ Should be removed at this end! ⚠

You can remove the .coffee file in a commit but then restore it on your computer without committing it.

decaffeinate-project is used at first, then manual refactor/clean.

@confused-Techie Well well, it seems that the "small time" has passed. :)
Unfortunately, though, it cannot be merged immediately, since it needs to be consolidated with later changes... My idea would be to collect all changes to these coffee files, since the last commit of this branch, and integrate them into the js code as well. There cannot be that many changes. I can help if you wish.

@confused-Techie Well well, it seems that the "small time" has passed. :)
Unfortunately, though, it cannot be merged immediately, since it needs to be consolidated with later changes... My idea would be to collect all changes to these coffee files, since the last commit of this branch, and integrate them into the js code as well. There cannot be that many changes. I can help if you wish.

Yeah I'm sorry about that. If it's any consolation, your collection of PRs here helped us determine our proper workflow for decaf PRs, due to the difficulty in accurately reviewing them.
If willing any help would be awesome, but also if fine by you, I'd be willing to restructure these into our new workflow, so that they can finally be reviewed and merged.
Essentially what we have been doing going forward for any decaf work is this:
Firstly create a branch of only machine decaf files.
Then from there have a second branch that does the work of decafing the files manually.
Finally make a PR of your human-decaf branch against the machine-decaf branch.
That way we are able to review what you specifically have done, without caring at all about the actual decaf, since we know the machine decaf will work.
Finally from there, once properly reviewed we can then merge it, but at the last second change the target branch to be the main branch.
But again, sorry for how long these have sat here untouched, if you're willing seeing this as a PR would be amazing, or if it's fine by you, I could do this with your changes, and attribute you as the author on the PRs

This doesn't sound too bad. One can create a new machine-decaf branch from the current state of things, and then merge the updated version of this PR to that. What still needs to be kept in mind is that the main branch still needs to kept frozen, from the generation of the machine-decaf to the eventual merging of it.
I heard you had a discord server, let's talk about it there.

Without further objections, this PR is superseded by https://github.com/pulsar-edit/ppm/pull/63.

@2colours Has very helpfully refactored this PR into our modern Decaf process, and has kept your commits in their entirety, so this one has been merged, just on a different PR (#63 )
Thanks a ton to @2colours and @GuilleW for all the hard work here.


Add search interface to main sidebar

I was using Sparrow with a friend, and we were doing some digging into the wallet history which required searching for outputs, addresses, and transactions. This was a little painful, because looking for each requires going to a different panel in the sidebar. So I was thinking that it would be good to add a unified search interface, where you could search for anything. So I opened this issue, and was also fooling with Sparrow at the same time, and found that Sparrow does indeed have a unified search interface! I originally didn't find it, because it's currently accessible from the menu.
I think that the search interface is incredibly useful, and people probably don't know about it or find it because it's not in the main sidebar. I think it's useful enough that it should be promoted to the main sidebar. Search for anything is a very useful and general operation, and it's the main interface to services like, for example, mempool.space, so I think users are used to it.

Searching for things in a wallet is an important function - but not I think nearly as often used as the other tabs in the sidebar. So is it really worth it to decrease the height of the other tabs, and have a more complicated interface to do this? I'm not convinced it is.
That said, there may be a way to make this more accessible. It's worth thinking about more.


fix config merge issue which reported at #336

Check config if it is empty by using mout/lang/isEmpty instead of !config expression

thanks!


Update the one-liner for Workers Analytics Engine

I was trying to understand the analytics options within Cloudflare (there are many! https://developers.cloudflare.com/analytics/). I think the one-liner for Workers Analytics Engine today is confusing, and propose this change to improve it.
Today, the one-liner for Workers Analytics Engine is: "[Workers Analytics Engine] Provides analytics about anything using Cloudflare Workers."
This is confusing because it reads as follows: "[Workers Analytics Engine] (Provides analytics about (anything using Cloudflare Workers))."
Here is the issue with this one-liner:

Workers Analytics Engine itself does not provide built-in analytics. It is an engine to which events can be emitted, but does not provide analytics about something out of the box.
"anything using Cloudflare Workers" is confusing. What is using Cloudflare Workers? What is anything?

I propose that we update the one-liner for Workers Analytics Engine to: Provides a queryable analytics engine for custom events emitted from Cloudflare Workers.
Here are the advantages with this proposed one-liner:

Clearly indicates that this allows custom events emitted from Cloudflare Workers
Clearly indicates that the analytics engine can be a sink for events, and can be queried
Better differentiated against all other analytics options within Cloudflare

I won't be merging this, leaving it up to @rohinlohe

Closing, Rohin will open a new PR with content improvement


Dredge, resolution changes automatically, cutting off content at the bottom

FocusFrame build version
v0.1.1
Operating System
Windows
Steps to reproduce

Start Dredge
Configure Dredge
The resolution should change once the window was moved cutting off content at the bottom

Expected result
Windows is moved and retained at 2560x1440
Actual result
Game changes resolution to 5120x1440 after it was moved, cutting off content at the bottom
Logs
No response
Screenshots or videos
No response

Introduced a temporary fix in #49. It's not the ideal solution but it works for now. I could use some help here


All values are being converted to strings

I believe the Content-Type of the requests are converting my post parameters from a valid JSON to string values. For example, when I use the built-in HTTPRequest node and I include an integer or an array, this add-on is converting those values from JSON objects into String values.
The following is an example of the payload being received on my server with the HTTPRequest node:
{
"id": 1,
"marked": [
1,
2,
3,
4,
5,
56
]
}

And here is the format of the same parameters when using the HTTPManager class:
{
"id": "1",
"marked": "[1,2,3,4,5,56]"
}

I have tried changing the content-type in the HTTPPipe.gd file from "application/x-www-form-urlencoded; charset=utf-8" to "application/json" but it doesn't like that, I imagine because the UTF-8 conversions are causing an issue but I am not familiar enough with that stuff to determine if that's the issue or figure out how to fix it.
Any thoughts on how I could use standard json with this add-on?

Hi,
can you provide me the part of the code where you add the post-data to the request?
It must be something like ...
.add_post("data", JSON.stringify([1,2,3,4,54]))

func get_marked_spots() -> PackedInt32Array:
	var values:PackedInt32Array = []
	for grid in grids:
		if grid.is_marked:
			values.push_back(grid.value)
	return values
func get_free_spots() -> PackedInt32Array:
	var values:PackedInt32Array = []
	for grid in grids:
		if grid.is_marked and grid.is_free_daub:
			values.push_back(grid.value)
	return values

var params:Dictionary = {}
params.game = data.id
params.card = card.data.id
params.marks = card.get_marked_spots()
params.free = card.get_free_spots()
print(params)
**
## output:
## { "game": 20897, "card": 603749, "marks": [8, 5, 3, 12, 14, 21, 29, 28, 30, 25, 37, 39, 45, 34, 47, 54, 51, 52, 57, 67, 61, 68, 69, 72], "free": [3,12] }
**
_manager = $HTTPManager
_manager.display_progress = false
_manager.job(
	server_endpoint
).add_post(
	params
).on_success(
	_on_complete
).on_failure(
	_on_error
).get()

Ah, i see!
There is no automatic json encoding of post or get data, you have to do this on your own.
Basicly all request data in HTTP communication must be translated to text. HTTPManager only ensures that the string is then url encoded. If you feed the add_post function with any other then a string godot will do some sort of typecasting. This isnt a problem with integer or floats but complex objects like dictionarys or arrays will be broken.
If you:
_manager = $HTTPManager
_manager.display_progress = false
_manager.job(
	server_endpoint
).add_post(
	"myjson", JSON.stringify(params) #<---- here your data gets JSON encoded and send with the name myjson
).on_success(
	_on_complete
).on_failure(
	_on_error
).get()

if you have PHP working on the other side you could then ...
$params = json_decode($_POST['myjson']);
... to decode

Hmmm, is there a reason that every post value is getting encoded as a string? It should be able to send arrays, integers, floats, etc. as raw values, and just encode text as a string when it is already a string. Is there a way to prevent the string conversions and just submit the raw data?

I believe I was able to get this working properly by changing some lines in HTTPPipe.gd (lines 81-84):
	elif job.request_post.size() > 0:
		var raw_data = JSON.stringify(job.request_post)
		job.add_header("Content-Type", "application/json")
		job.add_header("Content-Length", str(len(raw_data)))
		body = raw_data.to_utf8_buffer()

Note: I'm using this class strictly for api requests, not for downloading any images or any of the other behaviors you have created. I have still tested some of the functionality and it appears to be working just fine.


Provide environment at init time

Description
As discussed here, it changes the container proxy to provide an environment at init time (/init).
But I couldn't move the entire context from /run to /init to maintain backward compatibility.
1. /init
Put environment variables in the env field. and runtimes can use this value.
{
  "value": {
    "name" : String,
    "main" : String,
    "code" : String,
    "binary": Boolean,
    "env": Map[String, String] = {
       "__OW_NAMESPACE",
       "__OW_ACTION_NAME",
       "__OW_ACTIVATION_ID",
       "__OW_TRANSACTION_ID",
       "__OW_DEADLINE"
    }
  }
}

2. /run
The namespace, action_name, activation_id, transaction_id, and deadline fields are uppercased and prefixed with "__OW_" [1].

namespace -> __OW_NAMESPACE
action_name -> __OW_ACTION_NAME

I think these fields cannot be removed due to backward compatibility.
This is a draft pull request, please let me know if there is a better way.
ref
[1] https://github.com/apache/openwhisk/blob/master/docs/actions-actionloop.md#what-the-launcher-needs-to-do
Related issue and scope
<EMAIL_ADDRESS>My changes affect the following components

[x] API
[x] Invoker
[x] Tests
[ ] Documentation

Types of changes

[x] Enhancement or new feature (adds new functionality).
[x] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[x] I added tests to cover my changes.
[x] My changes require further changes to the documentation.
[x] I updated the documentation where necessary.

Codecov Report

Merging #4722 into master will decrease coverage by 6.61%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #4722      +/-   ##
==========================================
- Coverage      85%   78.38%   -6.62%
==========================================
  Files         198      198
  Lines        8841     8846       +5
  Branches      614      617       +3
==========================================
- Hits         7515     6934     -581
- Misses       1326     1912     +586

Impacted Files
Coverage Δ

.../openwhisk/core/containerpool/ContainerProxy.scala
93.09% <100%> (+0.12%)
:arrow_up:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...enwhisk/connector/kafka/KamonMetricsReporter.scala
0% <0%> (-100%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedConsumer.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-96.23%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-93.8%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0% <0%> (-92.31%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0% <0%> (-77.78%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-74.08%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2929893...08cd266. Read the comment docs.

This LGTM.


Hide suggested actions not destinated to the user

Fix #1057

Background
Suggested actions (spec here) can specify to field to show the suggested actions only for a set of users.
Today, we did not filter suggested actions based on the to field.
This fix will not show suggested actions with a to field targeting another user. For backward compatibility reason, if the to field is empty or not defined, we will continue to show suggested actions.
Changelog
Fixed

Fix #1057. Fixed suggested actions destinated for other recipients should not show up, by @compulim in PR #1706

Pull Request Test Coverage Report for Build 984

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

Files with Coverage Reduction
New Missed Lines
%

packages/component/src/Utils/Timer.js
1
88.89%

Totals

Change from base Build 981:
0.6%

Covered Lines:
925

Relevant Lines:
1566

💛  - Coveralls


Voronoi Segmentation for different scientific disciplines (bioimaging & Earth Observation)

This tutorial is related to "imaging" topic and shows how to perform a Voronoi Segmentation on images in Galaxy. Different types of images are used and the end-user will be able to choose its "path" through a button e.g.

Bioimaging <-- Will use bioimages from the BIA to run the tutorial
Earth Observation <-- Will use aerial images from https://zenodo.org/records/5494629#.YWQQetnMKjA

Bioimaging <-- Will use bioimages from the BIA to run the tutorial
Earth Observation <-- Will use aerial images from https://zenodo.org/records/5494629#.YWQQetnMKjA

Suggestion: Use the names of the imaging modalities for the different paths (and don't use the term "path", this might be confusing for the end user, rather "imaging modality"). For the cellular image, the modality probably is "fluorescence microscopy". Duno about the earth observation.


Fix #87

Created a new webpack-plugin-replace in order to move forward & close #87.
I can add & update this plugin as needed 👍
As it is, I went through the reproduction steps & it has been fixed.

Hmm... lukeed dismissed rkostrzewski’s stale review that automatic wording seems a bit rough!
Updated with @developit's suggestion for source-mapping (via character length).

@jmadler 😆 I giggled. Glad to hear!
@developit Your lead, good sir -- I'd feel strange merging own PR.

@developit Can I see the PR is merged sometime back only 👍 Still asking anyways when can we expect new fixes for firebase in preact-cli ?. Working on an existing app porting from react to preact ;)

@gokulkrishh today!

@developit Cool. Thanks 😃


Allow kubectl commands to be copy and pasted

There are two kubectl commands that are buried behind a shell prompt and followed by the output. A straight copy and paste fails because of the leading $ sign, and adds the sample output as a new command.
This pull request splits the command in the actual command and the expected output.

@ronaldb : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @mlearned - Could you take a look at the proposed update to your article? Thanks!

#sign-off , should mark console or cli too

#sign-off , should mark console or cli too

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Neptune: Ability to create a replica instance with deploy command

Aws Connected Device Framework Affected Module(s):

assetlibrary
I'm submitting a ...

[ ] bug report
[x] feature request

Description:

cfn-neptune.yaml appears to have the ability to specify a replica instance, but there's no way, from the deployment script, to specify to turn that functionality on.
Current behavior:

No way, from the deployment script, to specify a Neptune replica instance
Expected behavior:

A flag in the deployment script that sets the replica instance flag to true
Steps to reproduce:

Additional Information:
I'm not sure if this is intended behavior. I'm happy to write the fix if this is something the team wants to have. Just wanted to start the conversation.

Hi Aaron. We have a new cli wizard based installer which we will be releasing soon once we have wrapped up reviews and testing. We implemented this to overcome limitations of the existing script based deployments such as being able to provide values for more Cloudformation templates such as Neptune replicas. This new cli wizard installer will replace the script method.

Saw this was implemented in the new cdf-cli. Looks good to me!


isArray doesn't check input is empty variant

Describe the bug
Need to check the Variant is empty before looping
Area Affected
[x] Collections code
[ ] Documentation
[ ] Samples
Paul knows
Screenshots

Platform (please complete the following information):

OS: Both
Runtime
Version 1.0.2

Additional context
Add any other context about the problem here.

Completed in 1.0.3


When both wifi mesh and ble are enabled, continuously print "wifi: m f null" (IDFGH-11619)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.2
Espressif SoC revision.
ESP32
Operating System used.
Windows
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
None
Development Kit.
ESP32_DEVKIT_V4
Power Supply used.
USB
What is the expected behavior?

When both wifi mesh and ble are enabled, continuously print "wifi: m f null"
Steps to reproduce.
internal_communication.zip

I use esp-idf\examples\mesh\internal_communication project.
Enable BT in menuconfig.
Init BLE.
Call esp_mesh_fix_root(true).
Call esp_mesh_set_type(MESH_NODE);(another development board call call esp_mesh_set_type(MESH_ROOT)).
Start wifi mesh.
Wait 70 seconds,you will see it.

Debug Logs.
No response
More Information.
No response

Duplicated with https://github.com/espressif/esp-idf/issues/12737
Will close this issue.
Thanks.


Make consistent use of mayBeReadWrite for deciding to clear the session or not.

This addresses the following SDN issue: https://jira.spring.io/browse/DATAGRAPH-1407
Basically: The read only indicator of the transaction would overwrite any hint or other means to decide whether it's a rw or ro query.  We cannot use it to decide to clear the session or not in a transaction spanning multiple queries.
@torstenkuhnhenne Would that still work with you use case?
Paging @Phoosha here as well.

That's perfect for me. All I needed is a way to prevent the Session clearing in this case, which this does.
Thanks, @michael-simons! I very much appreciate how you've been finding solutions for various use cases here. :+1:

@michael-simons 👍 it still works in my use case. Thanks for asking 😄 Is it possible to get a new release soon?

Release 3.2.18 went out on Monday.

In production since yesterday 😄


Adds AuthenticationFilter Support to IR.

Adds AuthenticationFilter support to IR:

api/v1alpha1/validation/authenticationfilter.go: Adds validation package for validating the AuthenticationFilter type.
internal/ir/xds.go: Adds internal Request Authentication types with initial support for JWT.

Requires #804
xref: #790

Codecov Report

Merging #813 (9e2b1f5) into main (22131aa) will decrease coverage by 0.60%.
The diff coverage is 46.33%.

@@            Coverage Diff             @@
##             main     #813      +/-   ##
==========================================
- Coverage   63.91%   63.30%   -0.61%
==========================================
  Files          50       51       +1
  Lines        6506     6674     +168
==========================================
+ Hits         4158     4225      +67
- Misses       2087     2186      +99
- Partials      261      263       +2

Impacted Files
Coverage Δ

internal/ir/zz_generated.deepcopy.go
14.22% <0.00%> (-3.08%)
:arrow_down:

internal/ir/xds.go
77.35% <52.17%> (-2.20%)
:arrow_down:

internal/provider/kubernetes/controller.go
46.79% <66.66%> (-0.74%)
:arrow_down:

internal/provider/kubernetes/routes.go
31.92% <73.68%> (+2.22%)
:arrow_up:

api/v1alpha1/validation/authenticationfilter.go
79.54% <79.54%> (ø)

internal/gatewayapi/filters.go
81.23% <100.00%> (+1.14%)
:arrow_up:

internal/gatewayapi/helpers.go
77.72% <100.00%> (ø)

internal/gatewayapi/route.go
89.00% <100.00%> (ø)

internal/provider/kubernetes/helpers.go
75.40% <0.00%> (-2.46%)
:arrow_down:

... and 2 more

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

@arkodg since this PR depends on and includes https://github.com/envoyproxy/gateway/pull/804, can you /approve that PR as well?


Resized images look weird

Ran in to some weirdness with resized images just now. They occurred after updating to 2.0.0 and when I jumped back to 1.2.4 the problem disappeared. Similar problems with a bunch of photos, but I've only included one of them here.
http://imageshack.us/photo/my-images/855/originalxh.jpg/
http://imageshack.us/photo/my-images/839/resize250x200.jpg/

Yikes, I'll look into it.

Please use v2.0.1 of the gem.


make header to avaiable on all pages

Closing in favor of https://github.com/code100x/daily-code/pull/23


Additional star tree feedback.

Adds additional feedback post merging #8131.
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@Naarcha-AWS @bharath-techie We need to wrap up and implement any remaining tech feedback ASAP so that I can read again and approve. Thank you.

@Naarcha-AWS @bharath-techie We need to wrap up and implement any remaining tech feedback ASAP so that I can read again and approve. Thank you.

No more comments from my end, just the last two.


Can't import

I get this:

Add "google()" inside buildscript { repositories { //here}  } and all projects { repositories{//here } }

Why doesn't the repository have what's needed?
Anyway, I tried, and it still fails...

Also change the it from alpha 4 to latest classpath

Why not update the project to something that we can use? Or at least to something the IDE can auto-fix?
Anyway, tried what you wrote, and still fails:

hover.zip

I think I know what the problem is,
Replace HoverViewStateExpanded with
https://pastebin.com/x333kdFs
yeah, this is an old library anyway that's not maintained anymore, even if you successfully import it and make an app out of it the Playstore will tell the user "It's coded for old devices and some features might not work".
I suggest you take a look at https://github.com/sentialx/discord-bubbles instead.

Just because it's not maintained doesn't mean Android Studio should fail so badly to open it. It's only 3 years old.
Eclipse did a very good job opening very old Java versions when I last used it (before Android Studio started being used).
I now tried to do as you wrote. Finally succeeded.
Just now noticed you are not working at Google. :)


[proposal] Extract DateTools into separate library.

We are using HypSystem in our projects for monitoring network status and device information.
It works very well, thanks! :)
But after long time app evolution on android and update to latest android-supported-libs we was faced with "64K Reference Limit" https://developer.android.com/studio/build/multidex.html?hl=ru#about
After investigating I was found that app has many unused joda-time classes.
We was cut libs with "jarjar" and fully remove Joda-time and DateTools.hx in "vendoring" version of HypSystem and it works now.
It would be great to be able option to turn off the Joda-time dependency at compile time or split lib.
I think splitting into two libs (device information and date tools) will be more practical, thanks!

Yes, I agree, in the lime build process all the resources always get copied over which makes pre-dexing of joda-time also impossible slowing down the android build process. Hypsystem is great for device information, +1 for moving DateTools into separate library


Implement Arbitrary for &[u8].

https://github.com/rust-fuzz/arbitrary/issues/62#issuecomment-736041629

@nagisa @fitzgen Thanks for your patience, this is ready for another review. Just added a couple more commits

@nagisa @fitzgen Thanks for your patience, this is ready for another review. Just added a couple more commits


2024 3 17 Deploy action

It deploys when you commit to a branch with the name "deploy".
Requires the secrets:
HOST
USERNAME
PORT
KEY

Can be defined in the repository settings.
Could be modified to deploy to specific servers with certain branches, or it can deploy to multiple servers at once.

Also requires a branch named deploy. Should probably be write-protected.


Getting errors init.py after a few minutes

Expected Behavior
Running without exiting.
Actual Behavior
Getting errors after a few minutes of running.
2016-08-08 20:37:36,320 [PokemonCatchWorker] [INFO] [pokemon_appeared] A wild Goldeen appeared! [CP 396] [Potential 0.53] [S/A/D 12/10/2]
2016-08-08 20:37:39,227 [PokemonCatchWorker] [INFO] [threw_pokeball] Used Pokeball, with chance 37.43 (40 left)
2016-08-08 20:37:40,226 [       cli] [INFO]
2016-08-08 20:37:40,227 [       cli] [INFO] Ran for 0:03:14
2016-08-08 20:37:40,227 [       cli] [INFO] Total XP Earned: 1860  Average: 34465.37/h
2016-08-08 20:37:40,227 [       cli] [INFO] Travelled 0.16km
2016-08-08 20:37:40,227 [       cli] [INFO] Visited 2 stops
2016-08-08 20:37:40,227 [       cli] [INFO] Encountered 6 pokemon, 6 caught, 0 released, 0 evolved, 0 never seen before
2016-08-08 20:37:40,227 [       cli] [INFO] Threw 7 pokeballs
2016-08-08 20:37:40,228 [       cli] [INFO] Earned 2200 Stardust
2016-08-08 20:37:40,228 [       cli] [INFO]
2016-08-08 20:37:40,228 [       cli] [INFO] Highest CP Pokemon: Goldeen [CP: 486] [IV: 12/0/6] Potential: 0.4
2016-08-08 20:37:40,228 [       cli] [INFO] Most Perfect Pokemon: Paras [CP: 277] [IV: 6/15/6] Potential: 0.6
Traceback (most recent call last):
  File "pokecli.py", line 503, in <module>
    main()
  File "pokecli.py", line 89, in main
    bot.tick()
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/pokemongo_bot/__init__.py", line 441, in tick
    if worker.work() == WorkerResult.RUNNING:
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_visible_pokemon.py", line 36, in work
    return self.catch_pokemon(self.bot.cell['catchable_pokemons'].pop(0))
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/pokemongo_bot/cell_workers/catch_visible_pokemon.py", line 48, in catch_pokemon
    return_value = worker.work()
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 329, in work
    data={'pokemon': pokemon_name}
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/pokemongo_bot/base_task.py", line 27, in emit_event
    data=data
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/pokemongo_bot/event_manager.py", line 52, in emit
    raise EventNotRegisteredException("Event %s not registered..." % event)
pokemongo_bot.event_manager.EventNotRegisteredException: Event pokemon_escaped not registered...
2016-08-08 20:37:40,250 [sentry.errors] [ERROR] Sentry responded with an error: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128) (url: https://app.getsentry.com/api/90254/store/)
Traceback (most recent call last):
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/raven/transport/threaded.py", line 174, in send_sync
    super(ThreadedHTTPTransport, self).send(data, headers)
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/raven/transport/http.py", line 47, in send
    ca_certs=self.ca_certs,
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/raven/utils/http.py", line 66, in urlopen
    return opener.open(url, data, timeout)
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 494, in open
    response = self._open(req, data)
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 512, in _open
    '_open', req)
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 466, in _call_chain
    result = func(*args)
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/raven/utils/http.py", line 46, in https_open
    return self.do_open(ValidHTTPSConnection, req)
  File "/Users/brunoramalho/Desktop/PokemonGo-Bot/lib/python2.7/site-packages/future/backports/urllib/request.py", line 1284, in do_open
    h.request(req.get_method(), req.selector, req.data, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1053, in request
    self._send_request(method, url, body, headers)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1093, in _send_request
    self.endheaders(body)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 1049, in endheaders
    self._send_output(message_body)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/httplib.py", line 891, in _send_output
    msg += message_body
UnicodeDecodeError: 'ascii' codec can't decode byte 0x9c in position 1: ordinal not in range(128)
2016-08-08 20:37:40,255 [sentry.errors.uncaught] [ERROR] [u'EventNotRegisteredException: Event pokemon_escaped not registered...', u'  File "pokecli.py", line 503, in <module>', u'  File "pokecli.py", line 89, in main', u'  File "pokemongo_bot/__init__.py", line 441, in tick', u'  File "pokemongo_bot/cell_workers/catch_visible_pokemon.py", line 36, in work', u'  File "pokemongo_bot/cell_workers/catch_visible_pokemon.py", line 48, in catch_pokemon', u'  File "pokemongo_bot/cell_workers/pokemon_catch_worker.py", line 329, in work', u'  File "pokemongo_bot/base_task.py", line 27, in emit_event', u'  File "pokemongo_bot/event_manager.py", line 52, in emit']

Other Information
OS: Mac OS
Git Commit: d0f60a221a8af7eaf7b41ffeeb756a636032534e
Python Version: 2.7.10

Update your Python to 2.7.12 and update your git commit version. You are WAY down there.

@GitPupper How to update Python?

@GitPupper yes you were right, I updated Python to 2.7.12.
When I cloned the repo, I switched to dev, but forgot to pull.
Now it's running for 10 min without any issue. Closing this issue...
@JaroSound here you can find this specific version, download the version Mac OS X 64-bit/32-bit installer, and reboot. Check the version running python -V
Then you need to pull origin dev and run again the installation virtualenv ., source bin/activate and pip install -r requirements.txt
You may need to recreate the config.json again with your credentials.

But i use Debian, so there is no linux version?


The path doesn't include the immediate steps for me

Hi, I'm doing a problem with the formulation below:
There exist $N$ passengers $1, 2, \dots, N$, in which passenger $i$ wants to travel from point $i$ to point $i + N$, $\forall i = 1, 2, \dots, N$. Besides, there exist $M$ buses starting from point $0$, in which bus $k$ can transport up to $q_k$ passengers at any instant. Given $d(i, j)$ to be the distance from point $i$ to point $j$ ( $i, j = 0, 1, \dots, 2N$ ). Schedule a plan so that the total distance of transport is minimized.
Input:

The first line is $N$ and $M$
The second line is $q_k, k=1,\dots,M$
The next lines are the $(2N+1)\times(2N+1)$ distance matrix
The content of my test file is as below:

4 2
3 2
0 3 1 3 1 1 2 2 3
3 0 1 1 2 4 1 3 1
1 1 0 4 2 3 3 2 1
3 1 4 0 4 4 3 1 3
1 2 2 4 0 3 4 4 4
1 4 3 4 3 0 1 4 1
2 1 3 3 4 1 0 4 4
2 3 2 1 4 4 4 0 2
3 1 1 3 4 1 4 2 0

Below is my code
f = open("./test/test_1_n_4_k_2.txt", "r")
line = f.readline().split()
n, m = int(line[0]), int(line[1])
q = f.readline().split()
q = [int(i) for i in q]
d = [ [0]*(2*n+2) for _ in range(2*n+2)]
for i in range(2*n+1):
    line = f.readline().split()
    for j in range(2*n+1):
        d[i][j] = int(line[j])
f.close()

from simpleai.search import SearchProblem, hill_climbing

class BusProblem(SearchProblem):
    def __init__(self, n, m, q, d):
        self.n = n # number of passengers
        self.m = m # number of buses
        self.q = q # capacity of each bus
        self.d = d # distance between each pair of stops
        self.available_stops = [i for i in range(1, n+1)] # stops that haven't been visited yet
        self.initial_state = [[0]*m, [0]*m, [[] for _ in range(m)]] # current stops of buses, cap, picked up stops (not dropped off yet)
        self.goal = [[2*n+1]*m, [0]*m, [[] for _ in range(m)]]

    def actions(self, state):
        stops, cap, picked_up = state
        actions = []
        for bus in range(self.m):
            for stop in range(1, 2*n+2):
                actions.append((bus, stop))
        return actions

    def result(self, state, action):
        stops, cap, picked_up = state
        bus, stop = action
        if (stop == 2*n+1) and (cap[bus] > 0):
            return state
        if (stop <= self.n) and ((not stop in self.available_stops) or (cap[bus] >= self.q[bus])):
            return state
        if (stop > self.n and stop != 2*n+1) and (not stop - self.n in picked_up[bus]):
            return state

        stops[bus] = stop
        if stop == 2*n+1:
            pass
        elif stop <= self.n:
            cap[bus] += 1
            picked_up[bus].append(stop)
            self.available_stops.remove(stop)
        else:
            cap[bus] -= 1
            picked_up[bus].remove(stop - self.n)
        return [stops, cap, picked_up]

    def cost(self, state, action, state2):
        stops, cap, picked_up = state
        bus, stop = action

        final_cost = self.d[stops[bus]][stop]
        return final_cost

    def value(self, state):
        stops, cap, picked_up = state
        return sum(1 if stop == 2*n+1 else 0 for stop in stops)

    def is_goal(self, state):
        return state == self.goal

problem = BusProblem(n, m, q, d)
result = hill_climbing(problem)
print(result.state)
print(result.path())

In the module result method, the action should move the bus to a stop, which changes the state, and hence the result should be in result.path(). However, I only observe the final action state when I print out the result.path(). Can you check for me if anything is not right?

Turn out I shouldn't have changed the state directly in the result method, instead I should make a deepcopy of the state and modify the new state as I want to

Sorry for the delay in answering, that was the case, yes. Thanks for sharing the solution :)


Submit Charlie Coach

Submit project Charlie Coach

Hi @celia11111, the community-plugins.json file is no longer used. Please install the Omi AI App and submit your app/plugin from within there

Hi,
Here are some questions and uncertainty when I try to submit this project:

I am not sure about the "trigger on" part and if I should fill in "memory creation".
I am not sure if I need to add anything special in "capabilities".

I submitted this project before, but I'm not sure if it is correct. Please let me know if any clarifications need to be made!
Thanks!

Hi @celia11111,

If you are creating an external integration app and it is triggered at the end of every conversation, then you should use memory_creation as the trigger. If you want the transcript of the conversation as it happens in real-time, then you should use transcript_processed

Based on the changes you made here in the json file, you'll have to select the memories and external integration capabilities


Patch set from Gentoo

This is the updated patch set we are using in Gentoo. I rebased the patches against current znc git master.
While I have noticed that you have recently added support for ENV var ZNC_NO_LAUNCH_AFTER_MAKECONF (https://github.com/znc/znc/pull/257) I'd like to take a different road:
Yes, wen can set an environment variable in our pkg_config phase (emerge --config znc) but isn't it cleaner to have a parameter which is also documented in help?
If you don't like a hardcoded pidfile I can provide a different path using configure.

 If the package is going to be up to date again, cool.

Yes, wen can set an environment variable in our pkg_config phase (emerge --config znc) but isn't it cleaner to have a parameter which is also documented in help?

I don't see how this feature can be useful outside of Gentoo...
Usual users don't need to create config and actually start it separately from each other.

I will be co-maintaining the package with @wired as he seems to be AWOL.
With the environment variable, we should be able to drop one of our patches but the PID one will remain.
It's probably best we rely on znc's ability to daemonise instead of using the --foreground option and --make-pidfile.
I think the best way to configure the PID file would be via a command-line option. This way, we could have our init script specify it. This is something several applications do and it sounds like a sensible choice.

It's probably best we rely on znc's ability to daemonise instead of using the --foreground option and --make-pidfile.

May you explain why? Start-stop-daemon is used in much more places than ZNC, and should be reliable enough...

@DarthGandalf I'm not really sure what all the downsides are but I think start-stop-daemon wasn't initially made to start programmes in the background, most daemonise on their own.
There is one major drawback to the s-s-d approach: when you rely on it to save logs, you can't rotate without a service restart. However, I don't think this applies to znc as it doesn't write any system logs.

Superceded by https://github.com/gentoo/gentoo/pull/3236


ARROW-15452: [FlightRPC][Java] JDBC driver for Flight SQL

This implements a JDBC driver able to communicate to Flight SQL sources.
So far this covers:
Metadata retrieval by DatabaseMetadata, ResultSetMetadata, etc.
Query execution by statements and prepared statements

Yet to be done:
Parameter binding on prepared statements

This PR is a duplicate of #12254 , except targeting the flight-sql-jdbc branch. It's intended to be a snapshot of the prior PR for the purpose of IP clearance.

Thanks @jduo!
I see several ICLAs have been filed already, I'll move the list here:
For the IP clearance: it looks like these are the current contributors. As far as I can see, nobody has an ICLA on file. Can each of you file an ICLA (instructions) and confirm here?

[ ] Abner Eduardo Ferreira
[x] Gabriel Escobar
[x] iurysalino
[ ] jayhomn-bitquill
[x] Jose Almeida
[ ] Juscelino Junior
[ ] Kyle Porter
[x] Rafael Telles
[ ] Ryan Nicholson
[ ] tifflhl
[ ] Vinicius Fraga

BTW, it looks like the first hundred or so commits can probably be rebased out without affecting the PR (there are some minor merge conflicts to fix after that), since they look to be old Flight SQL commits that were already merged in some form.

Oh, CI indicates there are files that need a license header:

Error:  Failed to execute goal org.apache.rat:apache-rat-plugin:0.13:check (rat-checks) on project arrow-java-root: Too many files with unapproved license: 1 See RAT report in: /arrow/java/target/rat.txt -> [Help 1]

@jduo would you like to rebase to try to reduce the committer list?
Also at some point, this needs to have a stable Git commit hash so we can actually fill out the paperwork and hold the vote

Was the merge commit intended? It's made this PR quite a bit messier

Was the merge commit intended? It's made this PR quite a bit messier

We're looking into creating a new branch with a single commit -- there are many rebase conflicts at the moment.

Hi, I've tried this code and found it supports auth token only when set the server auth handler as NoOpAuthHandler and add an auth header in middleware to pass the token at the same time, right?

Looks like everyone's filed ICLAs. I'll get the paperwork filed ASAP


Fix #101; heightForMessageLabel now uses messageLabel text to calculate height.

Closing this, since I want to create more smaller pull requests instead of a big one with unrelated changes.


there are 2 cobblestone stairs

One written "Cobblestone Stairs"
and one written "Cobblestone Stair"

"Cobblestone Stair" => Stairs mod
"Cobblestone Stair" => moreblocks mod, created with "circul_saw"
It's the same node, just come from different mods.
We will check if we can do something ;)

ok because i can't stack them :(

This is not a bug, these are two different stairs. Though, we can maybe add an alias.

Aliases to from circul_saw stairs to the stairs from the [stairs] mod would be indeed a great idea

@LeMagnesium @Ombridride Aliases are a solution, to the only condition that both stairs use exactly the same param1 and param2 meanings. Having different bindings for those attributes would result in a data clash, causing some kind of corruption (e.g. bad rotation).

I saw aliases for slabs in moreblocks but not for stairs, I think it's in fact because we cannot have aliases because of metadata. So, just get used to have multiple type of any stairs made of any material that can be sawn using the circular saw. There is nothing to be done here.


Update README.md with OpsWorks commands

Coverage remained the same at 97.329% when pulling fe769a8c92b9b1594934e543c1f9989856e77c94 on DJRH:update-readme into e049e0077d5671f4bcca1a5a1050b37422748eda on dtan4:master.

LGTM 👍 Thanks ❗


Show a warning notification when simple calendar has no seasons set

When i was trying to get the basic weather configuration to work, it would stall and not function properly when i pressed "Preview Weather". I thought this was a bug at first, but found the issue was i had not changed the icon settings to the right season under Simple Calendar's (SC) "Season Settings".
I am using one of SC's pre-made calendars and customized it and thought nothing of changing the season icon under the settings.

If you could update the "Read Me/ How To" to include this detail, have foundry throw a warning notification about not having the season icon set, or put a note in the basic weather config tool tip would help others that may not have their settings set correctly.
-Thanks

Thank you for the advice. I thought that the icons were standard. As soon as I can I am gonna update the readme, but I am thinking of a solution for this problem already. Thank you very much.


Generating report error - cannot convert undefined or null to object

I am also having issues with the latest v2.0.4 and I have a similar setup where I run jest and collect coverage in separate steps and then use ArtiomTr/jest-coverage-report-action for the annotations.
     - name: Jest coverage report
        uses<EMAIL_ADDRESS>        with:
          github-token: ${{ secrets.GITHUB_TOKEN }}
          coverage-file: ./temp-coverage-results/current/jest.results.json
          base-coverage-file: ./temp-coverage-results/target/jest.results.json
          threshold: 80
          skip-step: all
          annotations: failed-tests

With v2.0.4 it keeps failing with TypeError: Cannot convert undefined or null to object and I am not sure how to debug it.

Out of curiosity I tried to use ArtiomTr/jest-coverage-report-action@Debug-action instead (mentioned in another conversation) and with that particular version it works perfectly every time: branch in not switching, reports are being collected, annotation with coverage diffs is generated, etc..
I did some research but wasn't able to figure out where the Debug-action version comes from... is it based on an older version?! @ArtiomTr
Originally posted by @raspo in https://github.com/ArtiomTr/jest-coverage-report-action/issues/242#issuecomment-1059559731

Related: https://github.com/ArtiomTr/jest-coverage-report-action/issues/242#issuecomment-1059744737

@raspo,
The Debug-action is not a version, it is a branch. I created it from the main branch, so it should be the same as v2.0.4 version, with few console.log's added.
Looks like error occur on this line
https://github.com/ArtiomTr/jest-coverage-report-action/blob/9ce24ef8d37706be462dcd6128760ea7f1c1857b/src/format/summary/getSummary.ts#L11
The variable map is null or undefined. I don't know what could cause that issue, but it is definitely is not related to this one. I will create a separate issue for this.

I just faced this same (I believe) issue:

My first run was without any threshold, this was after adding it (not sure if it has relation, need further runs to check it)

@coelhucas what's your config? where you using version 2.0.5?
uses<EMAIL_ADDRESS>

I have also started to see this error on another repo I have been working on.
Like @lockettks I am also on v2.05 and I am manually generating both reports.
      - name: Jest coverage report
        uses<EMAIL_ADDRESS>        with:
          coverage-file: ./temp-coverage-results/current/coverage-final.json
          base-coverage-file: ./temp-coverage-results/target/coverage-final.json
          skip-step: all
          annotations: failed-tests

@ArtiomTr I'd be happy to help debug this, is there a branch or version of this action I can use that can help understand where exactly it is breaking? ideally one that isn't minimized? I think we should re-open this issue, btw.

Hello @raspo :wave:,
Flags, which are required for generating report are --json --testLocationInResults --outputFile=<filename>. Action automatically modifies your test script and adds necessary flags, so unless you're generating a report file by yourself, remove all additional flags https://github.com/ArtiomTr/jest-coverage-report-action/blob/c33e891a0f37cb87086633eb2058df7a2838621b/src/utils/getTestCommand.ts#L23-L31
As for the issue, yes, I think the action should throw errors when the report format doesn't match since the action will lose a lot of functionality if the rest of the fields are missing.
About the debugging, I will create a separate issue on this. Action should generate source maps, to make errors more readable.

@raspo,
Yes, you're right, renaming options will require a breaking change, but I understand your indignation at the incomprehensibility of the option name. I will consider renaming it.

I've been using this tool with jest reports but I'm converting to vitest. But I can't manage to get a json from vitest that contains the coverageMap property. Any ideas?

Hello @Crafoord :wave:,
I'm not familiar with vitest, so this requires some time for investigating.

This simple script does the job
./scripts/save-coverage-file.js
const fs = require("fs");

/*
  don't forget to update `coverage-file` input inside workflow when specifying path here
  `test.yml`
  with:
    coverage-file: ./coverage/report.json
*/
const testReportFilename = process.cwd() + "/coverage/report.json";
const coverageReportFilename = process.cwd() + "/coverage/coverage-final.json";

const testReport = require(testReportFilename);
const coverageReport = require(coverageReportFilename);

testReport.coverageMap = coverageReport;

fs.writeFile(testReportFilename, JSON.stringify(testReport), (err) => {
  if (err) {
    console.error(err);
    process.exit(1);
  }

  console.log("Coverage report appended to " + testReportFilename);
});

package.json
{
  "scripts": {
    "test:coverage": "vitest run --coverage",
    "test:coverage:ci": "npm run test:coverage -- --outputFile ./coverage/report.json && ./scripts/save-test-coverage.js",
  }
}

I used a very similar solution to https://github.com/ArtiomTr/jest-coverage-report-action/issues/244#issuecomment-1260555231. It's pretty much same solution but a bit more verbose and I actually had to add --reporter=json to the vitest command for it to generate the desired report.json 👍
ci/save-coverage-file.js
#!/usr/bin/env node
// eslint-disable-next-line @typescript-eslint/no-var-requires
const fs = require('fs');

const reportFilename = 'report.json';
const coverageFinalFilename = 'coverage-final.json';
const cwd = process.cwd();

const reportJsonFilepath = `${cwd}/coverage/${reportFilename}`;
const coverageFinalFilepath = `${cwd}/coverage/${coverageFinalFilename}`;

let reportJsonFile;
let coverageFinalJsonFile;

if (fs.existsSync(reportJsonFilepath)) {
    reportJsonFile = require(reportJsonFilepath);
}
if (fs.existsSync(coverageFinalFilepath)) {
    coverageFinalJsonFile = require(coverageFinalFilepath);
}

console.log('Files exists?', { reportFilename: !!reportJsonFile, coverageFinalFilename: !!coverageFinalJsonFile });
if (reportJsonFile && coverageFinalJsonFile) {
    if (!reportJsonFile.coverageMap) {
        console.log(`Adding coverageMap property to ${reportFilename} based on ${coverageFinalFilename}`);
        reportJsonFile.coverageMap = coverageFinalJsonFile;
        fs.writeFileSync(reportJsonFilepath, JSON.stringify(reportJsonFile), (err) => {
            if (err) {
                console.error(err);
                process.exit(1);
            }
        });
    } else {
        console.log(`coverageMap already exists in ${reportFilename}, not doing anything...`);
        process.exit(0);
    }
} else {
    console.log('Not doing anything...');
    process.exit(0);
}

package.json
"scripts": {
    "test:coverage": "vitest run --coverage",
    "test:coverage:ci": "vitest run --coverage --reporter=json --outputFile ./coverage/report.json && ./ci/save-coverage-file.js",
  },


Grid filter placeholder?

Summary
Would it be possible to add placeholder text to filter input like on the screenshot attached? I don't see any way to achieve this with data grid out of the box.

The default string/number filters do not have an API for this at this time. Given our future movement towards a different filtering pattern (#1632), I don't know if we'll make this change before that feature is implemented.
Either way, a workaround is to make a custom filter so you can control the contents of the popup to whatever your needs are.

The feature request here has been captured into our list and we’re going to take it into consideration as we develop Clarity Core capabilities. In an effort to clean up our backlog and focus our attention, I’m going to close this as captured in our feature requests. Please follow our development and releases to see when we release relevant components to make this possible. Future feature requests can be made in our GitHub Discussions.


rollback gradle version to 8.9

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

Project coverage is 87.01%. Comparing base (2533fae) to head (5fbf87d).

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #119      +/-   ##
============================================
- Coverage     87.30%   87.01%   -0.29%
  Complexity      680      680
============================================
  Files           133      133
  Lines          2095     2095
  Branches        135      141       +6
============================================
- Hits           1829     1823       -6
  Misses          191      191
- Partials         75       81       +6

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


Size for under 1MB shows as zero

https://github.com/import-pandas-as-numpy/safepull/blob/bda35cae53f6b56318f200e6116d0a5e9309cda6/src/safepull/models.py#L39
A file size of 108921 will display as 0MB
Other places like the downloads table display
Some ideas:

use a similar decimal value as the table, see https://github.com/import-pandas-as-numpy/safepull/blob/bda35cae53f6b56318f200e6116d0a5e9309cda6/src/safepull/models.py#L95
implement a custom formatter like this one https://stackoverflow.com/a/1094933/244037
use humanize or another library to make human-readable strings that adapt based on size

Thanks!