The HOI loss is NaN for rank 0

Dir sir,
I followed with readme to build this UPT network,but when i use the instruction
python main.py --world-size 1 --dataset vcoco --data-root ./v-coco --partitions trainval test --pretrained ../detr-r50-vcoco.pth --output-dir ./upt-r50-vcoco.pt
i got an error
`Traceback (most recent call last):
File "main.py", line 208, in
mp.spawn(main, nprocs=args.world_size, args=(args,))
File "/root/miniconda3/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 230, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/root/miniconda3/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 188, in start_processes
while not context.join():
File "/root/miniconda3/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 150, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 0 terminated with the following error:
Traceback (most recent call last):
File "/root/miniconda3/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 59, in _wrap
fn(i, *args)
File "/root/autodl-tmp/upload/main.py", line 125, in main
engine(args.epochs)
File "/root/pocket/pocket/pocket/core/distributed.py", line 139, in call
self._on_each_iteration()
File "/root/autodl-tmp/upload/utils.py", line 138, in _on_each_iteration
raise ValueError(f"The HOI loss is NaN for rank {self._rank}")
ValueError: The HOI loss is NaN for rank 0`
I tried to train without pretrain model it works the same error.I tried to print the loss but it shown an empty tensor.As a beginner , i have no idea what it happened.If you could give me any help,i would be appreciated.
I look forward to receiving your reply.Thank you for a lot.

I use TITAN Xp with torch1.9.1 to train this model,and i installed the packing and testing it works,the dataset is vcoco and download with the script.Thank you for a lot.

Hi @OBVIOUSDAWN,
Thanks for taking an interest in our work.
The Nan loss problem was quite a pain. I ran into the issue a long time ago and managed to resolve it by using larger batch sizes. The problem was that the spatial encodings have bad scales, which made the training very unstable. I see that you are using only one GPU to train. So the batch size is most likely insufficient.
Here are a few things you can try

For the log terms in the pairwise positional encodings, use log(1+x) instead of log(x+epsilon).
Add batch norm in the spatial head that computes the pairwise positional encodings.
Increase batch size (probably the easiest option).

Hope that resolves the issue.
Cheers,
Fred.

Dir sir，
I tried this model on new server with 3090*4 ,bacisize =4,which shows the same error on rank3.In your second suggestion,do you mean the "Pairwise Box Positional Encodings" in the paper?I find a "PositionEmbeddingSine" in /detr/model/position_encoding.py to change its eps show the same error ,and i also tired change "binary_focal_loss_with_logits" /"compute_spatial_encodings"'s eps in /ops.py.I print out the whole network but i dont know which part belongs to Pairwise Box Positional Encodings model.I look forward to receiving your reply.Thank you for a lot.

...do you mean the "Pairwise Box Positional Encodings" in the paper

Yes, it is implemented in ops.py. If you are running on 4 GPUs with batch size as 4, you should have an effective batch size of 16. I think that's sufficiently large. Are you still getting the error?
Fred.

yes,the effective bachsize is 16,it shows the same error.And i tride to change
features.append(             torch.cat([f, torch.log(f + eps)], 1)         )
by using log(1+x) instead of log(x+epsilon) in "compute_spatial_encodings",i got the same error.I look forward to receiving your reply.Thank you for a lot.

That's odd. If the batch size is 16, it should work now. Can you try some different seeds?
Fred.

Hi, @fredzzhang .
Thank you for your contribution. I am very interested in your work. Therefore, I want to deepen my understanding of your paper with running code. But I can't run the code.
I encountered the same error using the same command on 3090.
python main.py --world-size 1 --dataset vcoco --data-root vcoco/ --partitions trainval test --pretrained checkpoints/detr-r50-vcoco.pth --output-dir checkpoints/upt-r50-vcoco2
I haven't changed any code, just download good code and checkpoint model according to Readme.
Then I want to run the training command, but failed with this error.
Could you give me some help to solve it?

Hi @leijue222,
That should an issue related to the batch size. I trained the model on 8 GPUs with a batch size of 2 per GPU—effectively a batch size of 16. So since you are training with one GPU, you need to set the batch size to 16.
Let me know if that works.
Fred.

Towards end of the Model Zoo section, I added some stats for 8 TITAN X GPUs, which in the case of VCOCO, would be 40 minutes. I don't know how long it will take one 3090 to train it. It shouldn't be too long.
Fred.

Thanks again, I love this job.

I meet the same error using the command on 3090.
python main.py --world-size 1 --batch-size 16 --dataset vcoco --data-root vcoco/ --partitions trainval test --pretrained checkpoints/detr-r50-vcoco.pth --output-dir checkpoints/upt-r50-vcoco
Could you help me to solve the problem? Thanks.

Hi @yuchen2199,
Sometimes the training could be unstable even with batch size of 16. If possible, further increasing the batch size should make it happen less often.
Fred.

Thanks for your early reply. I solved the problem after increasing batch-size. This is really an interesting work.

Hi,
I am getting the HOI loss is NaN issue when training on a different dataset. The code used to work fine earlier. But, when I tried to train on images where there is only one human bbox and one object bbox, I started facing this issue.
I have tried:

Setting batch size to 16 and 32
features.append(
torch.cat([f, torch.log(f + 1)], 1)
)
adding Batch Norm
self.spatial_head = nn.Sequential(
nn.Linear(36, 128),
nn.BatchNorm1d(128),  # Batch normalization after the first linear layer
nn.ReLU(),
nn.Linear(128, 256),
nn.BatchNorm1d(256),  # Batch normalization after the second linear layer
nn.ReLU(),
nn.Linear(256, representation_size),
nn.BatchNorm1d(representation_size),  # Batch normalization after the third linear layer
nn.ReLU(),
)

But, I am still getting this issue.
Do you have any suggestions on how I can solve this issue?

To show an example,
I modified the output of compute_spatial_encodings() function like this:

So, the input to spatial head is

output is

spatial head is


Feature/set up redux store

In this project

React-Redux was setUp
Redux store was setUp
States were declared
Application files were structured following the use of ducks pattern

Kindly help me review
Thanks

Thanks you @roshan-bajgain


pydantic validation with response_model have bug

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
#CLIENTS = [
#        {"id": "1a", "short_name_loc": "Demo", "long_name_loc": "DEMO"}]
# special error in ID field
from fastapi import APIRouter
from typing import Union
from pydantic import BaseModel, Field, root_validator, parse_obj_as, ValidationError
router = APIRouter(tags=["http"])
from ..test.data import CLIENTS, TARGETS
class Client(BaseModel):
    id: int = Field(..., title="Id")
    short_name_loc: Union[str, None] = Field(..., title="short_name_loc")
    long_name_loc: Union[str, None] = Field(..., title="long_name_loc")

@router.get("/api/clients/", response_model=list[Client])
def clients(item: list[Client]):
    c = CLIENTS
    b:list[Client] = parse_obj_as(list[Client], c)
    return b

Description
Why if I delete arg in clients func item: list[Client] and pydantic model invalid it got500 error
If i added item: list[Client] then I got pretty json error in browser?
I think args dont related with output response.
I think best way use like
@router.get("/api/clients/")
def clients() -> list[Client]:
...
instead
@router.get("/api/clients/", response_model=list[Client])
def clients(item: list[Client]):
because its very strange
Operating System
Windows
Operating System Details
No response
FastAPI Version
0.68.2
Python Version
3.9.1
Additional Context
No response

Also analogy
@router.get("/api/backend/version", response_model=str)
def version(item: str = ''):
return {}
give 500 error BUT expected beauty pydantic error

@shurshilov
Few comments on your code:

HTTP GET is not suppose to have request body - your method should be like this:

@router.get("/api/clients/", response_model=list[Client])
def clients():
    return CLIENTS

when response_model is defined it always tries to instantiate it - in this case it will take CLIENTS and make object of type List[Client] even if returned CLIENTS is dict it will work

You get 500 because your Client takes ID as INTEGER - your data says

CLIENTS = [{"id": "1a", "short_name_loc": "Demo", "long_name_loc": "DEMO"}]

here in dict, ID is String and can not map it to Client - ValidationError is raised

In second comment you returned DICTIONARY and response_model is STRING - you have to have good types declared and returned

If I understood correctly you have issue with 500 error instead you expect 400 to be returned.
I would say that 500 is valid in this case, it is internal error, code has bugs.
400 is used if user called your API with wrong parameters/data - 400 Bad Request

Thank you for answer
May be best way send not 500 internal server with tracelog from pydantic? its way use in different frameworks...

You can always catch it with exception handler
@app.exception_handler(Exception)
async def http_exception_handler(request, exc):
    return PlainTextResponse(traceback.format_exc())

@zoliknemet

Yes, just strange that for 400 response set Text error but for 500 not =\


[NIFI-2027] Responsive Tables

Reviewing...

@scottyaslan Looks like this PR addresses the size issues we were seeing with the exception of the queue listing table and the comopnent state table. The only additional comment I have is around the use of Javascript to re-calculate the table height. Where possible we should allow CSS to do this to avoid the extra cycles keep it simple.
I'll wait for an update that includes the few extra tables before I merge. Thanks!

Thanks @scottyaslan! This has been merged into master.


Cocoapods repo update?

Hi Cocoapods repo version is still 2.1, would you like to update the version to Cocoapods?
Cocoapods JSONCodable

[!] Unable to satisfy the following requirements:

JSONCodable (~> 3.0) required by Podfile

None of your spec sources contain a spec satisfying the dependency: JSONCodable (~> 3.0).
You have either:

out-of-date source repos which you can update with pod repo update.
mistyped the name or version.
not added the source repo that hosts the Podspec to your Podfile.

I also think version is not updated

ok I attempted to update the podspec on release, but clearly its not working, will look into this today


Switch to cloud-init for host initialization

Did you revert only to use a PR instead? I am using this repo as a base for my own setup using Kubic and k8s+calico so wonder why it was done.

The installation of k3s-selinux is not working in the cloud-init setup. Will investigate this evening why.

Folks, did some testing, and found out a few things:

The kured reboot method, does not work, is not taken into consideration, with our without snapshots, it has nothing to do with it. But it's not needed, we can use the just rely on the reboot-needed flag file and it works as well.

On MicroOS, whatever you do, if you install k3s-linux with the k3s installer, the selinux never gets applied, this command never gets executed successfully chcon -u system_u -r object_r -t container_runtime_exec_t ${BIN_DIR}/k3s, see https://get.k3s.io/.

Turns out, k3s runs perfectly fine without SELinux, this is just an added security measure, that for us, never got used, because even when we installed k3s-selinux via combustion, we never manually applied the above command.

For the above command to work however, the package has to be installed, and we need to have rebooted into the new snapshot. So if we want SELinux, rebooting is not an option. Turns, out even with reboot, it's under 5 min install time, so all good there.

So we install k3s-selinux, reboot, then install k3s with skip rpm, and then apply the above command. And that command executes succesfully, meaning SELinux has been applied.

Turns out it's not about k3s-selinux being installed, it should be installed, after k3s, because the postinstall script in the RPM applies SELinux policies to all the k3s files and directories! So the order counts.
And according, to the folks over at k3s, see the issue mentioned above, it doesn't matter if it reboots in between to apply the policy (pending final confirmation).
Basically, the install.sh gets the order right! So that's the only way to go, with an immediate reboot after starting k3s.

Just pushed the final tweaks, now the order is correct and we reboot to apply the snapshot.

As for the reboot method, it just works, no need to worry about that too much. Probably the "kured" method was added before the kured reboot-sentinel config flag came about, that's a pretty recent addition.
But definitely good for MicroOS if we open a ticket and let them know, will let you do this @mnencia, as you dove deep down that issue.

Added a small fix to make filesystem resizing work (with /var/, not /)

Ok, folks, so it the last commits:

The hostname was fixed for good, it was indeed DHCP @mnencia, all that was needed was to set a flag.
DNS was set to use Google DNS servers primarily, and Hetzner comes up third as a backup.
Snapshot numbers are reduced.
Finally, k3s-selinux gets installed with a compatible version, and gets applied post k3s installation successfully, before starting k3s.

Finally, there is no need even to define any reboot method. For one, it does not seem to be read, for two, relying on the default works great. Here's why:

The default reboot method is "auto", and it tests for the presence of the rebootmgrctl binary. In our case it is present, so rebootctlmgr is selected automatically. By default. And we set its strategy to off explicitly.

The default path to the reboot flag file is /var/run/reboot-needed.

Those are deterministic conditions. It just works and with fewer lines of code for us!

All seems clear on my end. Now I just need to give it 24h to 48h to confirm automatic reboot upon upgrade. I want to see the real deal, not just the simulation.

Thanks, @mnencia and @phaer, great collab as always!


Is it possible to handle permissions in a unified way for iOS & Android?

Again, thank you for this plugin, it's really helpful.
Right now, for iOS, the notification permission seems to be requested internally by the plugin. However, for Android, the example you provide uses the Permission Handler plugin to check and request for permissions. This makes a bit inconvenient and asymmetric.
Since the permission check/request is done after Alarm.init, I think it would be better to just pass the desired config to it, and have it do the permission checks/requests, or, if it's too much, just let the devs handle permissions themselves.
Thank you

Hi @younesouhbi
Thanks for your feedback ! You're right, notification permission handling is currently asymmetric. I think I will ask for Android notification when Alarm.set is called, like in iOS. In my opinion it's better this way because:

you won't have to add a permission check in your code
I think it's a better practice to ask the permission only when the alarm feature is requested (on Alarm.set). If the permission is asked on app init, the users won't necessarily understand why it's needed. But if it's asked at the moment the user schedules his alarm then he will know the purpose and how important it is.

What do you think ?

Thank you for your reply!
I would say that there are advantages & disadvantages to this. Let me start with the advantages:
Advantage: making it the permission request contextual can help the user better understand why the permission is being requested, however, I also think that users are now accustomed to permissions being requested from apps, especially an app that schedules alarms. What is suspicious and confusing is when a calculator app wants to access the user's contact. This means that while contextualizing the permission request is good, I don't think it's needed in this case.
Disadvantage 1: when the permission is requested at the moment of doing something, it interrupts the flow of what is being done. Personally, when I set out to perform an action, I don't like being distracted by "side quests". Imagine the user is trying to set an alarm for a specific date & time, then the permission request pops up, now they forgot which date or time they were trying to set an alarm for.
Disadvantage 2: if there are multiple permissions and the plugin asks for each permission as it is requested, it might become annoying as it interrupts the user multiple times.
Disadvantage 3: what if the app that uses the plugin cannot function without the permission? In this case, it might be better that the user doesn't waste their time if they are not willing to grant the permission. If said permission is only requested as it is needed, then the user might invest time in an app only to stumble upon the permission request and realizing the app doesn't work for them. It might be frustrating and it happened to me several times with various apps.
Disadvantage 4: from a developer standpoint, it might be better to let devs manage the permissions themselves and concentrate on what the purpose of the plugin is, which if I understand correctly is: setting alarms and letting you know when one is triggered. In my app for example, I would prefer to have an initial screen that informs the user of all the required permissions and why they are needed before they even get to the home screen. Other developers might prefer to do it as needed. Still, removing permission handling from the plugin might be better. This way, all devs know that before calling the plugin APIs, they need to have the needed permissions granted. How they manage that is their problem.
So, all in all, I kind of lean towards another direction ;)

@younesouhbi Thank you so much for your feedback !
It looks like it would be better to remove the permission request from the plugin then !
There is one thing you might need to know about notifications: for the android part I use a foreground service for triggering the alarm. It allows me to make the alarm work even if the app is in the background or killed. In order to use this foreground service, it is required by android to have a notification linked to it. Documentation here.
It's not an issue but I just thought you may want to know that.
I'll add to my to do list: remove iOS notification permission request.
Sounds good ?

@younesouhbi I realized I will have to wait to release version 4.0.0 to remove the iOS notification permission prompt because it's a breaking change because the apps in production that use the plugin may rely on this.

@gdelataillade Thank you for your reply.
I have managed to remove the notification permission request from iOS. Here is the modified code for the NotificationManager.swift, in case someone wants to do the same thing:
import Foundation
import UserNotifications

/// Custom error to handle notification scheduling
enum NotificationSchedulingError: LocalizedError {
    case permissionNotGranted

    public var errorDescription: String? {
        switch self {
            case .permissionNotGranted:
                return "Could not schedule a notification because the notification permission is not currently granted!"
        }
    }
}

/// Singleton used to manage OS notifications
class NotificationManager {

    static let shared = NotificationManager()

    private init() {} // Private initializer to ensure singleton usage

    /// Schedules a notification
    /// - Parameters:
    ///   - id: the id of the alarm, which is a unique integer passed in as a string
    ///   - delayInSeconds: the number of seconds until the datetime of the notification
    ///   - title: the title of the notification
    ///   - body: the body of the notification
    ///   - completion: the callback to call once the scheduling attempt completes, either succeeds or fails
    func scheduleNotification(id: String, delayInSeconds: Int, title: String, body: String, completion: @escaping (Error?) -> Void) {

        // Check whether the notification permission is granted before going ahead and attempting to schedule one
        isNotificationPermissionGranted(completion: { granted in

            // Make sure the permission is currently granted
            guard granted else {
                // The notification permission is not currently granted so call the completion callback with an error
               completion(NotificationSchedulingError.permissionNotGranted)
               return
            }

            // Construct the content of the notification
            let content = UNMutableNotificationContent()
            content.title = title
            content.body = body
            content.sound = nil

            // Make the trigger of the notification (time based)
            let trigger = UNTimeIntervalNotificationTrigger(timeInterval: TimeInterval(delayInSeconds), repeats: false)

            // Make and attempt to schedule the notification request
            let request = UNNotificationRequest(identifier: buildNotificationIdentifier(id: id), content: content, trigger: trigger)
            UNUserNotificationCenter.current().add(request, withCompletionHandler: completion)
        })

    }

    /// Cancels a previously scheduled but not yet delivered notification
    /// - Parameter id: the id of the notification to cancel
    func cancelNotification(id: String) {
        UNUserNotificationCenter.current().removePendingNotificationRequests(withIdentifiers: [buildNotificationIdentifier(id: id)])
    }

    /// Checks whether the notification permission is currently granted or not
    /// - Parameter completion: the callback to call with the result after the check
    func isNotificationPermissionGranted(completion: @escaping (Bool) -> Void) {
        UNUserNotificationCenter.current().getNotificationSettings(completionHandler: { settings in
            switch settings.authorizationStatus {
                case .authorized, .provisional:
                    completion(true)

                default:
                    completion(false)
            }
        })
    }

    /// Builds the notification identifier
    /// - Parameter id: the base id from which to build a unique notification id
    /// - Returns: the final id to use for the notification that is guaranteed not to collide with other apps
    func buildNotificationIdentifier(id: String) -> String {
        return "<APP_BUNDLE_ID>.notifications.\(id)"
    }
}

This works in iOS. For Android, I haven't modified anything yet, and to be honest, I don't know where to begin. I read some of the Android native code, but couldn't yet identify the code that asks for the notification permission. I thought it was handled by Permission Handler, like it is in your example, but somehow, maybe it's the foreground service you mentioned in your comment above. I'll investigate more, but if you can point me in the right direction, that would be great!
Thank you


Improve license text so data is CC0

-> text CCBY40, data like coordinates, rooms/size, type of amenities, measures, routes close by is CC0

även här

-> CC0 för data

Fixed, changed all text to CC0. deleted all non supporting text and replaced it


Kivy does not support RTL

Kivy does not support RTL
Thanks for your great library
But there are several big problems
kivy fonts do not support many languages, such as Chinese Persian Arabic and ...
And your library does not support RTL (only LTR supports) In the event that must be a capability in your components to adjust RTL or LTR (as a parameter)
For example some methods or components cannot be adjusted RTL to display text and this is a problem
Please check and solve these elementary problems

Thank you for not simply rejecting these problems
Versions

OS: windows 10
Python: 3.10.5
Kivy: 2.1.0

It's my problem
Kivy is really great
I wish this problem would be solved soon
Because of this problem, I have to go to Kotlin and Android Studio

Duplicate of https://github.com/kivy/kivy/issues/7704


Use --base-dir instead of --host-*dir on newer oc tools

Options --host-*dir are missing in oc version 3.10 and higher.

lgtm

Thanks!


Npm publish

This is a large change but main goal is to make sure we can publish on npm.
I had to make the following changes:

@hyperion scope requires paid account on npm and we cannot use that to publish, so I removed that from all of the packages
hyperion-global is already registered on npm and I had to rename it to hyperion-globals to work.
had to update all package.json files to prepare for npm publish.

For now, I published a version of these modules to make sure we have registered the names to ourselves. I'll make sure gradually each module is fully workable on its own.
Fun fact: I pushed the docs on the website last night and it had a reference to hyperionj module. Some mofo immediately registered that package https://www.npmjs.com/package/hyperionj.
Luckily I had already registered https://www.npmjs.com/package/hyperionjs before which we can use to host all of the packages at once.
I'll continue to test and update, but want to get early feedback from the team.

Build failed, does it compile locally and something we need to change in the build step as well?

Found all the issues and sent the fix. Now tests also pass.


On login sends OPTIONS method instead of POST

Hi,
I'm using http://root.system.local url for APP and http://api.system.local for API.
With settings like this on  $auth.login() it sends data in OPTIONS method - not in POST.
When I put both platforms on localhost http://localhost/system_root and http://localhost/system_api all is ok - send POST request.
Why is that?

I have the same problem sahat, When I check request it shows the request method "options", but I am using Angularjs, above code for php.

+1 I have same Issue with Yii2
Could someone point out the solution? Why would it send Options instead of POST?

Same here.
only happens to me when I set the $authProvider.baseUrl = API_URL;

In fact, seems not to be a Satellizer bug as explained in this link
http://stackoverflow.com/questions/12111936/angularjs-performs-an-options-http-request-for-a-cross-origin-resource?answertab=active#tab-top


Publish to Github nuget feed is broken

The publish of nightly packages to the Github nuget feed is broken right now.
Error reported:

{"error":"Your request could not be authenticated by the GitHub Packages service. Please ensure your access token is valid and has the appropriate scopes configured."}

This is unfortunately still broken

Can't find a way to fix it -.-


DCOS-52782: add Typeahead component

Looks good. Some questions:

Should the multi-select show the selected values in the input? Having an empty input seems useless.
I need to un-focus the input to be able to select again without changing the value. Is this intended?

I can't select the text values in order to copy them. Should I be able to? The cursor when hovering them is the one that shows that they can be selected.
Should we make the input have customizable width? Sometimes we need a smaller or a bigger input.

Should the multi-select show the selected values in the input? Having an empty input seems useless.

That is intentional. The idea is that you could use onSelect to do something with those values. Here's an example from the design team's Sketch symbols for how those might be used: https://cl.ly/45baadd535ff
Implementing this input is out of scope for this ticket

I need to un-focus the input to be able to select again without changing the value. Is this intended?

I'll update the dropdown to open on click as well as on focus.

I can't select the text values in order to copy them. Should I be able to? The cursor when hovering them is the one that shows that they can be selected.

Can't select them from the list or from the text input?

Should we make the input have customizable width? Sometimes we need a smaller or a bigger input.

The width of the parent element decides the width of the input

Can't select them from the list or from the text input?

From the list.

@GeorgiSTodorov - I don't think that's a problem


[BUG] Spotify not showing up for asignment

When I try to assign Spotify to a slider it won't let me because it is not shown as an option.
I deinstalled the windows store application and installed it from the website.
Spotify is still not showing up.
I am using the
Korg NanoControl on windows10

Resolved!
Even though Spotify is open but playing on another device it wont be an Option to asign.
Just play some musik on the device and it will be asignable

Resolved!
Even though Spotify is open but playing on another device it wont be an Option to asign.
Just play some musik on the device and it will be asignable


klarna paylater payments do not get status "completed"

Expected behavior
If we set the Prestashop "Sent" state on a klarna paylater order, it should mark all products as sent and mollie should set the state to "completed".
Actual behavior
This actually happens only in very rare occassions. See description below.
Steps to reproduce the behavior
klarnapaylater is set up with OrdersApi and "Automatischer Versand bei markiertem Status" = "automatic shipping with selected state" is activated on the extended configuration tab. 2 states in the next configuration are are selected, while one of them is "sent".
Customer buys a product with klarnapaylater and everything is fine so far. An xx_mollie_payments table entry is created:
transaction_id (ord_xxxxx), cart_id(123456), order_reference(mol_12312332132),method(klarnapaylater, bank_status(open), and created_at  are filled. The fields order_id, updated_at are "NULL".
In very rare cases (6 out of 200), after updating the state (via webservice), to "sent" the order_id will be filled up with the prestashop id_order and the order_reference will be replaced with the actual prestashop order_reference and the bank_status switches to "complete". in most other cases a log message in the prestashop  with "Shipment information cannot be sent. Order reference (XXX) has no payment information." is written. As far, as I can understand the source code, this happens, when no order_id can be found in above table ( see CanSendShipment") class. I guess the webhook communication backwards somehow fails most of the time?
Environment
Prestrashop <IP_ADDRESS>

Mollie module version: 4.5.3
PHP version: 7.0

Hey there,
We will be glad to help you with this! Our team will check your case, and get back to you as soon as possible with an update.
--
Best Regards,
Invertus Support team.

@Shoprunners,
Could you please send us your Back-Office, FTP, and SSH access to<EMAIL_ADDRESS>so that we could debug the issue?
--
Best Regards,
Invertus Support team.

Hi,
did you receive our mail?

Could you please send us your Back-Office, FTP, and SSH access to<EMAIL_ADDRESS>so that we could debug the issue?
-- Best Regards, Invertus Support team.

Hi there @Shoprunners,
We have not received an email from you, please try to resend it.
--
Best Regards,
Invertus Support team.

Done

We have not received an email from you, please try to resend it.
-- Best Regards, Invertus Support team.

@Shoprunners,
It seems like we are having some kind of issues with the<EMAIL_ADDRESS>mailbox.
Until we sort that issue out, could you please forward that email to me directly at<EMAIL_ADDRESS>Thank you for understanding!
--
Best Regards,
Invertus Support team.

Done

hi,
btw. another customer faces the same problem with the latest 1.7 based version.

Hi @Shoprunners,
We are currently under investigation. Mollie's team is communicating with Klarna to make sure this issue is solved.
As soon as we have some updates, we will let you know.
Thank you for your patience and understanding!
--
Best Regards,
Invertus Support team.

Hi,
any news here? Problem still occurs.

Hi @Shoprunners,
We are currently under investigation. Mollie's team is communicating with Klarna to make sure this issue is solved. As soon as we have some updates, we will let you know.
Thank you for your patience and understanding!
-- Best Regards, Invertus Support team.


Parallel testing using the new globalSetup feature

Do you want to request a feature or report a bug?
A bug.
What is the current behavior?
If there are more than 1 tests, running Jest with--runInBand will succeed, but without will fail.
▶ yarn jest
yarn run v1.5.1
$ /Users/ojongerius/repos/jest-bug/node_modules/.bin/jest

 RUNS  ./mongo-insert.test.js
 FAIL  ./mongo-aggregate.test.js
  ● Test suite failed to run

    TypeError: Cannot read property 'getConnectionString' of undefined

       9 |     console.log('Setup MongoDB Test Environment');
      10 |
    > 11 |     this.global.__MONGO_URI__ = await global.__MONGOD__.getConnectionString();
      12 |     this.global.__MONGO_DB_NAME__ = global.__MONGO_DB_NAME__;
      13 |
      14 |     await super.setup();

      at MongoEnvironment.setup (mongo-environment.js:11:57)

 FAIL  ./mongo-insert.test.js
  ● Test suite failed to run

    TypeError: Cannot read property 'getConnectionString' of undefined

       9 |     console.log('Setup MongoDB Test Environment');
      10 |
    > 11 |     this.global.__MONGO_URI__ = await global.__MONGOD__.getConnectionString();
      12 |     this.global.__MONGO_DB_NAME__ = global.__MONGO_DB_NAME__;
      13 |
      14 |     await super.setup();

      at MongoEnvironment.setup (mongo-environment.js:11:57)

Test Suites: 2 failed, 2 total
Tests:       0 total
Snapshots:   0 total
Time:        0.454s
Ran all test suites.
Teardown mongod
error An unexpected error occurred: "Command failed.
Exit code: 1
Command: sh
Arguments: -c /Users/ojongerius/repos/jest-bug/node_modules/.bin/jest
Directory: /Users/ojongerius/repos/jest-bug
Output:
".
info If you think this is a bug, please open a bug report with the information provided in "/Users/ojongerius/repos/jest-bug/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.
Pull down https://github.com/ojongerius/jest-bug, running yarn test will fail and yarn test-inband will succeed.
What is the expected behavior?
Running more n+1 tests will succeed
Please provide your exact Jest configuration
module.exports = {
  globalSetup: './setup.js',
  globalTeardown: './teardown.js',
  testEnvironment: './mongo-environment.js'
};

Run npx envinfo --preset jest in your project directory and paste the
results here
▶ npx envinfo --preset jest
npx: installed 1 in 3.967s

  System:
    OS: macOS High Sierra 10.13.4
    CPU: x64 Intel(R) Core(TM) i7-3615QM CPU @ 2.30GHz
  Binaries:
    Node: 8.10.0 - ~/.nvm/versions/node/v8.10.0/bin/node
    Yarn: 1.5.1 - /usr/local/bin/yarn
    npm: 5.8.0 - ~/.nvm/versions/node/v8.10.0/bin/npm
  npmPackages:
    jest: ^22.4.3 => 22.4.3

This is a duplicate of #5731 but I thought I'd use the issue template and create a small repo for you folks to replicate this issue more easily.

Can you repost this message in mentioned thread? I appreciate the effort, but let's keep this issue tracker clean :).
Closing as duplicate of #5731


labels in overview for logged cubes

Would be nice is we was able to see which cubes in a cell has a active status in log (every status except good). @nik had a good layout for it with clickable labels showing the status, if this is not possible a special colour for it would also work.
Have to be something you can turn of and on since not everyone would like it. I think this would be a great help when completing a branch because you would see all logged cubes and be able to good them as part of checking the rest of the branch.

Nik approves this request lol jk, just like we discussed in chat works fine for me @crazyman4865 :)


Add logging

✨ Features
Add a logging module and some logs for the package operations.
🔗 Linked Issue/s
#36
🧪 Tests

[x] Did you implement unit tests if required?

Fixed tests and added some.

Codecov Report

Merging #40 (44905dc) into main (399cbd5) will increase coverage by 0.08%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   98.90%   98.98%   +0.08%
==========================================
  Files          13       14       +1
  Lines         274      296      +22
==========================================
+ Hits          271      293      +22
  Misses          3        3

Files Changed
Coverage Δ

src/tensorshare/client.py
100.00% <100.00%> (ø)

src/tensorshare/logging.py
100.00% <100.00%> (ø)

src/tensorshare/serialization/processor.py
100.00% <100.00%> (ø)


[Request For External] CU13 Gen Jnl.-Post Batch - UpdateDialog

Hello,
can we make this public please:
local procedure UpdateDialog(PostingState: Integer; LineNo: Integer; TotalLinesQty: Integer)
this will allow updating already opened dialog, i.e. in custom UpdateAndDeleteLines procedure.
Thanks,
Bartek

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[docs] Improve explanation of company_name in create_app_online

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Motivation and Context
Explain in more detail the meaning of company_name parameter in create_app_online docs

Description
company_name parameter is required when creating first app on App Store Connect team and as it can be set only once without contacting Apple support its description could be more detailed.

Testing Steps

Hi, I've applied the change.


refactor: less information sent per SessionUpdate (Backend)

Description

Reduced the amount of information sent when a participant either changes his appearance or status on a board – as mention in #4217.
Before a PARTICIPANT_UPDATED event was published containing all information (account and board session). This is now split into two:

Account (ID, name, avatar, account type) – for now remained PARTICIPANT_UPDATED
Board Session (connected, banned, etc.) – named PARTICIPANT_SESSION_UPDATED

Changelog

Simplified DB call, since we just need the session IDs and not all data per entry
Updated the UpdatedUser function to publish only a dto.User to realtime
Updated the dto.BoardSession – removing the whole account data and just leaving the id for identification
Simplified the UpdatedSession function, since we only need to publish a dto.BoardSession to realtime for one board going foreward
Adapted tests to use new dto.BoardSession

Next Steps
Since this is a breaking change, we need to modify the frontend to accommodate the backend changes. From my perspective either use this branch directly to make changes to the frontend or create a branch based on this one.
After the frontend changes are done, all tests should work again. For now, since I did not change the frontend, those tests obviously fail.
Checklist

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] The light- and dark-theme are both supported and tested
[ ] The design was implemented and is responsive for all devices and screen sizes
[ ] The application was tested in the most commonly used browsers (e.g. Chrome, Firefox, Safari)

@BenedictHomuth I'll do the frontend part as soon as all/most of the other active PRs are merged into main – I'm worried that with such a change, there are just too many conflicts bound to happen 😅

@BenedictHomuth I'll do the frontend part as soon as all/most of the other active PRs are merged into main – I'm worried that with such a change, there are just too many conflicts bound to happen 😅

Definitely worth its own release 😅
Hopefully, TS will make your life a little easier there. Let me know if I can help in any way.

Update: I'm going to take a stab at this after the new store (related to https://github.com/inovex/scrumlr.io/pull/4423) is implemented.


Implement basic tensor pretty printing, based on NumPy printing.

Pretty-printing functionality:

Print element tensors on new lines.
Truncate long tensors.

Summarize overly long contents with ....

Provide configuration options.

lineWidth: max line width for printing.
edgeElementCount: max number of elements to print before and after
summarization via ....
summarizing: if true, summarize description when element count exceeds twice
edgeElementCount.

Todos:

Add scalar formatting.
For ShapedArray: test aesthetics for custom Scalar types.

Resolves TF-420.
TF-419 has more info, including a survey of tensor printing in existing frameworks.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

Thanks @rxwei! All feedback has been addressed, ready for re-review.

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow

@swift-ci please test tensorflow macOS

@swift-ci please test tensorflow macOS

@swift-ci please test tensorflow Linux gpu

@swift-ci Please test tensorflow linux gpu

Linux GPU tests are failing with a strange error:
-- Testing: 11646 tests, 16 threads --
Testing: 0 .. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90..Build timed out (after 60 minutes). Marking the build as aborted.
Build was aborted
Recording test results
ERROR: Step ‘Publish JUnit test result report’ failed: No test report files were found. Configuration error?
[BFA] Scanning build for known causes...
[BFA] No failure causes found
[BFA] Done. 0s
Adding one-line test results to commit status...
Setting status of c1ec93fb2b138d2bb96a076af43ed3c41205f835 to FAILURE with url https://ci-external.swift.org/job/swift-PR-TensorFlow-Linux-gpu/1214/ and message: 'Build finished. No test results found.'
Using context: Swift Test Linux Platform with GPU (TensorFlow)
Terminated
Finished: ABORTED

@shahmishal: do you have any clue what's going on?

@swift-ci Please test tensorflow linux gpu

@dan-zheng this is failing because Jenkins does not see anything logged to the console for 60mins.
Testing: 0 .. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90..Build timed out (after 60 minutes). Marking the build as aborted.

@dan-zheng this is failing because Jenkins does not see anything logged to the console for 60mins.
Testing: 0 .. 10.. 20.. 30.. 40.. 50.. 60.. 70.. 80.. 90..Build timed out (after 60 minutes). Marking the build as aborted.

I see - does that mean tests are timing out (lit is taking too long)?
An idea: I should run tests locally to try to repro the hang.

Jenkins job is configured to timeout if it does not see any out to console for 60mins.

Disabled timeout to debug the test hang.
@swift-ci Please test tensorflow linux gpu

I cannot replicate the GPU test failure on my Ubuntu 18.04 GPU machine.
I will restart the GPU CI machine and retrigger tests. If tests fail again, I will merge regardless.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow


add active flag

Added the active flag.

I don't have write access.  So you will need to merge or set me up to write.


How can I make a cell in the sheet dropdown list with accept multiple values? (multiple selection)?

RTFM.


RocksDB state backend causes segfault on Alpine images

Alpine Linux uses MUSL instead of glibc, but the version of RocksDB embedded in Flink apparently requires some absent glibc functionality.
java.lang.UnsatisfiedLinkError: /tmp/rocksdb-lib-33e471f8d228c175dfc6148869213083/librocksdbjni-linux64.so: Error loading shared library ld-linux-x86-64.so.2: No such file or directory (needed by /tmp/rocksdb-lib-33e471f8d228c175dfc6148869213083/librocksdbjni-linux64.so)

By installing the Alpine Linux package libc6-compat, this error was resolved but was replaced by this one:
java.lang.UnsatisfiedLinkError: /tmp/rocksdb-lib-fea5dea17151b5dfdc87d323576a04c3/librocksdbjni-linux64.so: Error relocating /tmp/rocksdb-lib-fea5dea17151b5dfdc87d323576a04c3/librocksdbjni-linux64.so: __strtod_internal: symbol not found

I didn't find an obvious solution to that one, but it's possible installing an additional package for compatibility would work.
We might want to consider dropping the official -alpine images until this is resolved.
cc @iemejia @StephanEwen

Additionally, some ldd output:
bash-4.3# ldd /tmp/rocksdb-lib-017af084e60916b3bffbf90d411ad14d/librocksdbjni-linux64.so
	ldd (0x56488fb47000)
	libpthread.so.0 => ldd (0x56488fb47000)
	librt.so.1 => ldd (0x56488fb47000)
	libstdc++.so.6 => /usr/lib/libstdc++.so.6 (0x7fc11320b000)
	libm.so.6 => ldd (0x56488fb47000)
	libgcc_s.so.1 => /usr/lib/libgcc_s.so.1 (0x7fc112ff9000)
	libc.so.6 => ldd (0x56488fb47000)
	ld-linux-x86-64.so.2 => /lib/ld-linux-x86-64.so.2 (0x7fc112d6c000)
Error relocating /tmp/rocksdb-lib-017af084e60916b3bffbf90d411ad14d/librocksdbjni-linux64.so: __strtod_internal: symbol not found
Error relocating /tmp/rocksdb-lib-017af084e60916b3bffbf90d411ad14d/librocksdbjni-linux64.so: backtrace: symbol not found
Error relocating /tmp/rocksdb-lib-017af084e60916b3bffbf90d411ad14d/librocksdbjni-linux64.so: backtrace_symbols: symbol not found

I am +1 for dropping the alpine images in favor of images which have the necessary libraries.
How much larger is the ubuntu image in comparison to the alpine?

The latest builds are 253MB for hadoop27/scala_2.11/debian and 185MB for hadoop27/scala_2.11/alpine.
Because of how large a Flink distribution is, those 68MB have less of an impact than they might for other projects, but a potential 27% decrease still isn't trivial.
Nevertheless, I'm also in favor of dropping the alpine images for now. @iemejia?

Good catch @patricklucas. I agree with the removal, is better to have a reliable image even at the cost of the size, I can take a look for a fix, but I am quite booked until tomorrow at 15h CEST so I will check it after that, so it is up to you if we can wait. What did you do exactly to reproduce the issue just checkpointed ?

@iemejia: yeah, pretty much. Steps to reproduce:

$ docker run --rm -t -i plucas/flink:alpine bash
# echo "state.backend: rocksdb" >> conf/flink-conf.yaml
# echo "state.backend.fs.checkpointdir: file:///tmp/checkpoints" >> conf/flink-conf.yaml
# jobmanager.sh start-foreground local &
# flink run examples/streaming/WindowJoin.jar

I think I found the fix for this. PTAL at https://github.com/docker-flink/docker-flink/pull/22

Fixed in #22


Develop managment system

The main idea is to create the management system server and client.
You may notice that the code of mgmt-server is very similar to that of the main file (proxy). The workflow is the following:

Develop a functional server dedicated solely to the protocol we talked about with Juan (env variable to login to the server and execute commands).
Create a functioning client.
Unify the mgmt-server with the main file.

I'll add buffer.c and buffer.h to work in the branch. Before merging, I'll remove them cause they will be on main probably.


Callback for displayIncomingCall method

Useful callback to know whether or not reportNewIncomingCall has succeeded and handle actions accordingly.

@ianlin Please, review :)


metsHdr/agent causes error "Main METS.xml file is not valid"

If metsHdr/agent is present, the validator gives an error:

ERROR: Main METS.xml file is not valid.

This happens with a valid minimal IP that has folders metadata, representations, representations/rep1 and representations/rep1/data and the following METS.xml in the root folder:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<mets xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://www.loc.gov/METS/" xmlns:csip="DILCIS" OBJID="ID0001" TYPE="SIP:X" csip:CONTENTTYPESPECIFICATION="MIXED" PROFILE="http://www.eark-project.com/METS/IP.xml">
  <metsHdr CREATEDATE="2018-10-12T14:20:00" csip:OAISPACKAGETYPE="SIP">
    <agent ROLE="CREATOR" TYPE="OTHER" OTHERTYPE="SOFTWARE">
      <name>E-ARK Corpus Team</name>
      <note csip:NOTETYPE="SOFTWARE VERSION">1.0</note>
    </agent>
  </metsHdr>
  <structMap TYPE="physical" LABEL="Common Specification structural map">

  </structMap>
</mets>

Full IP as a zip file: valid_minimal_IP.zip
The error disappears when metsHdr/agent is removed:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<mets xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://www.loc.gov/METS/" xmlns:csip="DILCIS" OBJID="ID0001" TYPE="SIP:X" csip:CONTENTTYPESPECIFICATION="MIXED" PROFILE="http://www.eark-project.com/METS/IP.xml">
  <metsHdr CREATEDATE="2018-10-12T14:20:00" csip:OAISPACKAGETYPE="SIP">
  </metsHdr>
  <structMap TYPE="physical" LABEL="Common Specification structural map">

  </structMap>
</mets>

Full IP as a zip file: agent_removed.zip

Looking at the XML, I'm missing SchemaLocations in the mets-element. You need to have that for getting the extra attributes for the agent.
For the note to be able to have the extra attribute, make sure you are using version 1.12 of METS.

In addition to what @karinbredenberg said, one needs to keep in mind that commons-ip library being used by the rest-ip-validator is still an old version which does not support these new attributes (based on METS version 1.11 and common specification version 0.17).
I'm working on the library during the next weeks in order to turn it common specification version 2 compliant (will use METS version 1.12).


v1.2 backports: 18-10-16

Backported PRs:

#5908
#5909
#5911
#5912

Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

test-missed-k8s

test-upstream-k8s

k8s tests failed due to cilium not being deployed

test-missed-k8s

test-docs-please

Same failure as before. This was broken before this PR so I'm merging this PR to unblock further testing.


What are the units of 'window_size'?

Does it represent the gene number or MB?

Number of genes

Thanks!


Add additional headers when custom default backend is used

Additional available variables:

$namespace
$ingress_name
$service_name

This change is

Coverage decreased (-0.07%) to 43.818% when pulling a85cfd10b53868b35f9aa59737117b3c9d2f75fc on aledbf:additional-headers into b791460206f81ab5870d2f588e15fe1bcbdecb76 on kubernetes:master.


ND4S - tuples mapping

I was wondering why ND4S doesn't have mapping between org.nd4j.linalg.primitives.Pair and Triple and scala tuples. Something like
  implicit class PairOP[T, U](a: org.nd4j.linalg.primitives.Pair[T, U]) {
    def asScala: (T, U) = (a.getFirst, a.getSecond)
  }

  implicit class TripleOP[T, U, V](a: org.nd4j.linalg.primitives.Triple[T, U, V]) {
    def asScala: (T, U, V) = (a.getFirst, a.getSecond, a.getThird)
  }

  implicit class Tuple2OP[T, U](a: (T, U)) {
    def asND4J: org.nd4j.linalg.primitives.Pair[T, U] = new org.nd4j.linalg.primitives.Pair(a._1, a._2)
  }

  implicit class Tuple3OP[T, U, V](a: (T, U, V)) {
    def asND4J: org.nd4j.linalg.primitives.Triple[T, U, V] = new org.nd4j.linalg.primitives.Triple(a._1, a._2, a._3)
  }

Any reason for that? # #

good idea!
By the way, I wonder what's the usefulness of ND4S?
In fact, I can write/run my DL4J program by Scala, without ND4S, and without ScalNet.
val features = Nd4j.valueArrayOf(Array(100, 3), 1.5)
val labels = Nd4j.valueArrayOf(Array(10, 1), 10)
val ds = new DataSet(features, labels)
val conf = new NeuralNetConfiguration.Builder()
.updater(new Sgd(0.5))
.list()
.layer(0, new DenseLayer.Builder()
.nIn(inputOfLayer0)
.nOut(inputOfLayer1)
.activation(Activation.IDENTITY)
.weightInit(WeightInit.IDENTITY)
.hasBias(false)
.build())

@zorba128 No reason, just waiting for contributions :)
@Charele They are there to provide more idiomatic Scala APIs, but they obviously need much work...

https://github.com/deeplearning4j/deeplearning4j/pull/6323

Implemented quite some time ago.


logging-winston: metadata serialization

This is a semver-major change that modifies how we serialize winston
provided log metadata. Since util.inspect behaviour is quite variable
between node releases, this used to give us different behaviour
depending on node version.
The choice to go with util.inspect was motivated by trying to match
the document for the built-in winston transports. It turns out that the
docs weren't quite precise and inspect isn't the default behaviour
anyway.
Previously we attached metadata as string labels. We now attach
metadata as a metadata property on the payload. If the config flag
options.inspectMetadata is provided, we will use util.inspect to
stringify.
We no longer use labels in either case.
Fixes: https://github.com/GoogleCloudPlatform/google-cloud-node/issues/2218

Coverage increased (+0.0002%) to 99.513% when pulling 4fb6c8f3d4422fb537d86dc4e0d1f6780ef57233 on ofrobots:winston-inspect into 890c2fa5cc7439ee95a0e63fd840abab485c13be on GoogleCloudPlatform:master.

Thanks @ofrobots. I made a few style changes. If CI is happy, I'll merge after. Would you like a release to immediately follow that?

Changes Unknown when pulling 745318ce36d13c22553e3d2d59a8b256df2f7eca on ofrobots:winston-inspect into ** on GoogleCloudPlatform:master**.


Zabbix APP's Zabbix Server Dashboard

I'm submitting a ...

[ ] Bug report
[X] Feature requestdd
[ ] Question / Support request

On latest Grafana 5.1.2 and zabbix-grafana 3.9.1:
Datasource is set on variable which is fine. But then there is missing Variable for selecting server on default group "Zabbix servers". As if people are running different hostnames and master-proxy setup the default app-dashboard do not work.
Feature request is to implement standard Server dashboard which understands multiple servers at least, and perhaps selecting host group too if someone is using different group than "Zabbix servers".

Also dashboard's metric items has a wrong application for proxies.. /^Zabbix/ is better, maybe /^Zabbix (?:server|proxy)/ is the best..


Expose onError in runSaga

This allows onError to be used when using redux-saga outside of react.

Thanks! gonna merge right away
Side note - when i look on both exposed APIs (sagaMiddleware.run and runSaga) they are 90% the same and I'm gonna prepare a PR with their unification. That however will be probably a breaking change for runSaga (I'll try to avoid it, but it might be impossible).
Would you consider such unification beneficial, even if its gonna be a breaking change?

I think unification is a worthy goal. I actually had expected that and had to dig through the code to realize why onError was not working (I realize it's not in the docs...but it was worth a shot anyway, and off hand seemed reasonable).

The refactor is available here. If you have time please take a look. I've managed to avoid breaking change here, only mere deprecation warning for now.


Purl Generation name

Describe the bug
According to the purl spec, some package names should always be in lowercase : https://github.com/package-url/purl-spec/blob/master/PURL-TYPES.rst
One of them is for pypi languages, but at the moment the generated purl for Django is<EMAIL_ADDRESS>To Reproduce

Launch Tern on a docker with django with a cyclonedxjson report

I will work on the PR (if it's okay of course).

Hello @rnjudge !
I have a question: so there is the list of package recognized in the purl spec and the one in Tern : tern/tern/analyse/defaukt/command_lib/base.yml
Should I only focus on the twelve in base.yml and ignored the others ?

Hi @ThiefaineM, base.yml lists the package managers that Tern is able to collect metadata for (I imagine the purl spec has many more than 12). If you have other package mangers that you would like to see added to base.yml please let me know and we can look in to them. The scripts in base.yml are what Tern runs to collect package metadata for each package manager so any package manager we add will have to have a corresponding command or set of commands that can be run in a container from a command line to collect metdata.

OK, then for the pull request I only worked with the one present in base.yml.
I think base.yml has most of the interesting packages manager but maven could be a big plus :)


Add support for MSRV

Current MSRV is nightly-2021-09-08

Decline this since this is not needed.

Decline this since this is not needed.


Component Action Button

Would be great to have a component like https://github.com/mastermoo/react-native-action-button. I may work on this.

:+1:  :+1:

HI! Is anyone working on this? It may take some time but I think I can take a shot :]

Is anyone working on this?

@dltmd6262 Not I'm aware of. It's your 💃.
P.S: make sur you start from the 'next` branch

Great I'll start working on it!

Is this in the Material Design Docs?

@barbu110 https://material.google.com/components/buttons-floating-action-button.html#buttons-floating-action-button-transitions
Is this what you're looking for??

@dltmd6262 Yeah thank you! I will take a look at it, maybe give a hand!

@barbu110 Great :] I'm new to this repo so it'd be great to have someone help!

I never knew there was already a floating action button in MUI. Should we extend this class or make a completely new one??

Closing this as a dupe, as #2851 has the correct name  and labels already. :)


Latest Version

Hello! Any chance to get the latest version supported? Would love to try and train my eye using beatsaber:)

Latest version should work with the v1.1.0 update


Docker publish release stage failures

While releasing 2.4.0 we ran into failures of the docker publish stage: https://github.com/meltano/meltano/actions/runs/2806611004
Specifically, its complaining about:
https://github.com/meltano/meltano/blob/3bf8d54b69b2758fb2f3fa7fd6bd0abde156ff3c/.github/actions/docker-build-scan-push/action.yml#L72

/home/runner/work/meltano/meltano/./.github/actions/docker-build-scan-push/action.yml (Line: 68, Col: 12): Unrecognized named-value: 'secrets'. Located at position 1 within expression: secrets.GITHUB_TOKEN

Note for when we pick this up next week. We''ll need to make sure we re-run the publish to ship the images 2.4.0 once this has been resolved.

We are accessing the secret directly within an action, but actions must have secrets passed in as inputs, since they don't get access to the secrets context.

Note for when we pick this up next week. We''ll need to make sure we re-run the publish to ship the images 2.4.0 once this has been resolved.

Kicked this off: https://github.com/meltano/meltano/actions/runs/2819676858


[V3] Implement new Concept Exercise: Comparison and Conditionals

Getting started
Please please please read the docs before starting. Posting PRs without reading these docs will be a lot more frustrating for you during the review cycle, and exhaust Exercism's maintainers' time. So, before diving into the implementation, please read up on the following documents:

Rationale for v3.
The features of v3
What are concepts and how they are structured?
What are concept exercises and how they are structured?

Please also watch the following video:

The Anatomy of a Concept Exercise.

See the documentation above (general documentation), as well as How to implement a Concept Exercise in JavaScript.
Goal
The goal of this exercise is to teach the student how to compare values in Javascript and then use that in conditional statements.
Concepts
The following concept pages need to be created. You can combine the two introduction.md files from the concepts and use that as introduction.md for the concept exercise. No need to create different content at this point.

comparison
conditionals

Learning Objectives
In the concepts the student should learn about the following operators and statements and then practice them in the concept exercise.
Comparison

<, >, >=, <= for numbers and strings
strict equality (===)
not equal (!==)

Conditionals

if(){}
if(){} else{}
if(){} else if(){} else{}

Out of Scope

Loose equality == and truthy/falsy (will be taught once the student learned about type coercion)
Comparing values of different types
Shallow/deep comparison of objects
ternary operator x ? y : z
switch, for

About.md of the Concept
If there are no pitfalls, advanced usages etc. that you would like to point out, the about.md file can be a copy of the introduction.md of the concept.
For the conditionals concept here some ideas what to mention in the about.md file.

multiple else if
if statements without a block to execute just one thing (e.g., if(...) return ...)

Prerequisites

basics
booleans
numbers
strings

When the students start this exercise they only know some basics about the primitive types. Keep that in mind when designing the exercise. See this list for details on the learning curve we are aiming for.
Exercise Idea
Julia Conditionals Exercise could be used as template
Help
You can choose to do this solo-style, or collaborate with multiple people on this. The suggested approach is to

First accept this issue by saying "I'd like to work on this" (you will not get a response, just go with it) and optionally request that someone works with you (and wait for a second person to accept your request).
Use this issue to write down what you need (collect reference material) and discuss, if necessary, what to write and what not to write.
Draft a PR and request the feedback from at least one other JavaScript member (you can ping the #maintaining-javascript channel in Slack).

I will work on this.

FYI, I am expecting <repo-root>/concepts/comparison to exist after this PR as I am linking to it :)

FYI a quick status update: I am done, I posted the PR as draft, I still need to resolve some conflicts and give it a self-review

I'll review it once you undraft it :)


H3 or H2?

File: datacenter/dtr/2.3/guides/user/manage-images/scan-images-for-vulnerabilities.md, CC @joaofnfernandes
Should the last heading "Where do I do next?" be changed from H3 to H2?

Yes it should be an H2, and as a best practice headings should also not have question marks.
I've filed #4917 to fix that.


Use case: Individually importable generic methods

Please correct me if I am wrong, but my understanding of this proposal has me thinking that the real power of it is the extension function capability, as it would allow for libraries like rxjs and other functional libraries to provide free functions (that are tree shakeable) that operate on this. Additionally, it works well as a local builder pattern for a foreign object.
RxJS example:
Before
const searchResults$ = fromEvent(document.querySelector('input'), 'input').pipe(
  map(event => event.target.value),
  filter(searchText => searchText.length > 2),
  debounce(300),
  distinctUntilChanged(),
  switchMap(searchText => queryApi(searchText).pipe(retry(3))),
  share(),
)

After
const searchResults$ = fromEvent(document.querySelector('input'), 'input')
  -> map(event => event.target.value)
  -> filter(searchText => searchText.length > 2)
  -> debounce(300)
  -> distinctUntilChanged()
  -> switchMap(searchText => queryApi(searchText).pipe(retry(3)))
  -> share()

where a function like map has the signature

function map<T, R>(this: Observable<T>, project: (value: T, index: number) => R): Observable<R> {
  // ...
}

Builder example:

// imagine this class is external library code
class UrlBuilder {

   constructor(url) {
     this.url = new URL(url)
   }

   build(): string {
     return this.url.toString()
   }

	setHost(host: string): this {
       this.url.host = host;
       return this;
    }

}

// and our own local code provides extensions to it
function toUrlBuilder(this: string): UrlBuilder {
   return new UrlBuilder(this);
}

function addQueryParam(this: UrlBuilder, name: string, value: string): UrlBuilder {
   this.url.searchParams.append(name, value);
   return this
}

then we can use the proposed bind syntax to augment the existing library with our new function in a fluent chaining way

const url: string = "https://github.com/js-choi/proposal-bind-operator" -> toUrlBuilder() // local extension
.setHost("google.com") // external library member
->addQueryParam("s", "TC39 bind proposal") // local extension
.build() // external library member

I feel like this addresses a number of the concerned use cases around the Hack style pipeline operator

It’s true: this proposal would allow developers to use individually importable “generic” methods more ergonomically and fluently.
Generic methods are already an existing and common pattern in JavaScript, of course. The pattern is even built into the language itself: Array.from is a generic factory method that can be transplanted into other array-like classes.
The explainer currently focuses on the security use case because that use case’s benefits might be the clearest to TC39. But I have been planning on adding a section to the explainer about generic methods for a while.

With regards to Hack pipes, there is a lot of history behind its relationship with bind operators.
During 2015–2017, people debated a lot about how the [old proposed bind operator ::][bind] could serve as a “pipe” operator in JavaScript (e.g., a::f(b, c)::g(d, e) would be a “pipeline”, equivalent to g.call(f.call(a, b, c), d, e)).
However, parts of TC39 objected to promoting this as the primary way to create pipelines, saying that using this is strange outside of methods. This debate, both online in the :: repository and also offline, was bogged down in circles, and it was a major reason why that proposal stagnated.
So I’ve wanted to try to avoid that old debate. We have a clear-cut security use case for a bind operator, and we’ll focus TC39 on that security use case. Meanwhile, we’ll present the bind operator as independent of the pipe operator, whose fundamental function (as the pipe champion group is presenting it) is flattening and linearizing deeply nested expressions.

But we definitely can and should present individually importable generic methods as another use case: a pattern that is already common in JavaScript. I would welcome any existing examples from high-impact open-source libraries. (I know that @benlesh had been originally planning for RxJS to use this pattern while waiting for the bind operator that did not come. I would welcome his expertise and insight here, too.)

The original catalyst for RxJS moving to someObservable.pipe(map, filter, mergeMap) etc was because we needed a way to provide our "operators" in a left-to-right chainable way that was tree-shakable. Originally we had everything implemented as methods that could be patched onto the prototype by importing modules. This worked okay but it came with a LOT of problems. This was mostly done with the hopes that :: would be standardized (You can actually see it in the README of the 5.x branch). But when we realized that was never going to happen, we moved to functional pipes.

@benlesh: Thanks for your insight! Was that RxJS v5 system (its individually importable this-based functions) initially inspired by another existing library, or is this system unique as far as you know?
I’m trying to assess how generally common this pattern already is, given that this proposal would greatly lubricate it.

I'm not sure how often this technique is used, considering its drawbacks. However it was important enough that TypeScript added a feature around it to support modules augmenting interfaces in maybe TS 2.1 or so (I don't remember the exact version).


Mastersword can fire projectiles even when not at max health

Title. Seems to always work.

Was an oversight, fixed for the next update.


Kotlin 1.9-compatible release

Blocked by:

~https://github.com/cashapp/zipline/issues/1071~
https://github.com/JetBrains/compose-multiplatform/issues/3323
https://github.com/JetBrains/compose-multiplatform/issues/3418
https://github.com/JetBrains/compose-multiplatform/issues/3421

https://github.com/cashapp/redwood/releases/tag/0.6.0


Fixed meeting create button in safari

Closes #1842
For future reference, I have explained what was going wrong in the issue

@alveolus can you test it with safari?


Meta Issue: List of PeMS Data QC visulization Task

We have created several datasets along with QC data table. However, it is difficult to understand how this QC looks like. Therefore, it is necessary to monitor the data QC through visualization. I am proposing below list of the QC to ensure the high quality data production in PeMS.

A dashboard showing number of daily detectors side by side (may be bar chart) from the metadata, five minutes detector models with and without imputation. The bar value can be absolute number or percentage of difference between the number of detectors. This dashboard can be build dynamically based on last seven days data from current date which will provide temporal data quality assurance over time. This dashboard visualization can be further break down by district, route, county and others meta data catagory to bring spatial data quality understand.
Similar to Detector level QC dashboard, station level daily number of detectors for last seven dynamic days can be visualized to ensure data quality in station level.
Compare the Number of good and Bad (details Category) detectors reported in
INT_DIAGNOSTICS__REAL_DETECTOR_STATUS over consecutive time. For example, show the number of detectors for each error type over seven days to understand any unusual detector status.
Only comparing number of detectors or stations over spatial and temporal period does not provide 100% guaranty of good data quality. To ensure good data quality we need to have long term visualize plan of some of the metrics spatially and temporally to understand their pattern and outliers. The below is the subcategory for some of those visualization-
a. PeMS two weeks daily health which includes station health status by district, route, county, facility, and direction
b.  Detector/Station level imputed Vs. observed volume, occupancy, VMT and VHT line diagram for a week to understand any unusual observed or imputed value.
c.  Detector/Station Level AADT for last 2/3 years to check if there is abnormal drop of AADT value between two/three consecutive years for a same station or detector. This AADT check will cover the others micro level QC between annual and daily level. If the Bigger picture looks good, then other small pictures should be fine.

I will add more in later and feel free to add any other ideas of QC visualization if you can remember.

@mmmiah will create separate issues for these tasks for tracking purposes.

All of these listed QC and reporting dashboard was completed


feat: FileCollection: add function to retain multiple patterns

Description
Added a function to FileCollection that accepts an array of strings to retain multiple patterns in the file collection.
Checklist:

[X] Securely signed commits
[X] Component(s) with PHPDoc blocks, only if necessary or adds value
[X] Unit testing, with >80% coverage
[ ] User guide updated
[X] Conforms to style guide

This is my first time trying to contribute. I probably committed the wrong changelog. What file should I write my change to if the commits are going to the future 4.6 release? I have not found a v4.6 file or am looking in the wrong direction?

I have not found a v4.6 file or am looking in the wrong direction?

If you have not found v4.6.0.rst, you have created your PR branch from the wrong branch (develop).
You should have created your PR branch from 4.6 branch.
See https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/workflow.md#if-you-sent-to-the-wrong-branch

Understood. Is it better to amend this PR or start a new one based on 4.6?

If you are not familiar with git, starting a new one based on 4.6 would be easier.

Not yet familiar enough to fix it. Let me submit a new PR based on 4.6 branch. Thanks for the clarification.


[corda-ent] upgrade cenm version to 1.5

Signed-off-by: suvajit-sarkar<EMAIL_ADDRESS>Changelog

Add 1.5 CENM images
Fix base directory hard coding in charts
Updated the baseDir path to /opt/cenm
Update code to support CENM upgrade
Fixed sample network.yaml to showcase multiple notary setup (renamed notary -> notary1, notary2...)

Reviewed by
@pppazos
@jagpreetsinghsasan
Linked issue
#1289

Docs related changes to show that 1.5 is a supported CENM version now?

Docs related changes to show that 1.5 is a supported CENM version now?

added 1.5 in docs


2024-10-31_01-35-45_master-next_aws-sdk-cpp

Automatically created.

Successfully created backport PR for kirkstone-next:

#10236

Backport failed for scarthgap-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin scarthgap-next
git worktree add -d .worktree/backport-10227-to-scarthgap-next origin/scarthgap-next
cd .worktree/backport-10227-to-scarthgap-next
git switch --create backport-10227-to-scarthgap-next
git cherry-pick -x 42078c0209f2e8e19a69f80ed5ed3081c0a068da

Backport failed for styhead-next, because it was unable to cherry-pick the commit(s).
Please cherry-pick the changes locally and resolve any conflicts.
git fetch origin styhead-next
git worktree add -d .worktree/backport-10227-to-styhead-next origin/styhead-next
cd .worktree/backport-10227-to-styhead-next
git switch --create backport-10227-to-styhead-next
git cherry-pick -x 42078c0209f2e8e19a69f80ed5ed3081c0a068da


Page not found during universal package promote

I get a "Page not found" error during a universal package promotion.
Attached is the log from the task. I replaced the package / feed guids.
tasklog_5.log.txt

I can confirm getting the exact same issue on v1.3.1

And previous versions?
Verstuurd vanaf mijn iPhone
Op 3 jun. 2019 om 15:45 heeft Brainy Devs <notifications<EMAIL_ADDRESS>het volgende geschreven:
I can confirm getting the exact same issue on v1.3.1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frenevanosnabrugge%2Fvsts-promotepackage-task%2Fissues%2F24%3Femail_source%3Dnotifications%26email_token%3DABOH2WUC2ETQQLKDEH72GELPYUN7PA5CNFSM4HSGQG32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWZOIHQ%23issuecomment-498263070&data=02|01||9160bf39301d41fb348708d6e829bd33|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636951663289395614&sdata=QoHTlB%2FhHjaizGcL29ENOVjT%2F%2By4mImPeCuywuSLfjU%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABOH2WQ2NR7VRAA3OJUJTLLPYUN7PANCNFSM4HSGQG3Q&data=02|01||9160bf39301d41fb348708d6e829bd33|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636951663289405625&sdata=eND7qz0Wk0LgcXtSiHGLhMOW%2BSs1tzXtEhiSmxXrPw0%3D&reserved=0.

Hi,
we started to use it yesterday, so unfortunately I can't comment on any previews versions, and we didn't find a way to specify an older version.

Version: 1.2.12 was OK for me on 5/29/2019, 9:38:46 AM
Version: 1.3.1 was Failing to promote on 5/30/2019, 12:17:26 PM

I’m short on time. Can you revert to older version ?
Verstuurd vanaf mijn iPhone
Op 4 jun. 2019 om 17:09 heeft Brainy Devs <notifications<EMAIL_ADDRESS>het volgende geschreven:
Version: 1.2.12 was OK for me on 5/29/2019, 9:38:46 AM
Version: 1.3.1 was Failing to promote on 5/30/2019, 12:17:26 PM
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frenevanosnabrugge%2Fvsts-promotepackage-task%2Fissues%2F24%3Femail_source%3Dnotifications%26email_token%3DABOH2WRDTVTQ4ZZ7T2ZGSR3PY2AQ7A5CNFSM4HSGQG32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW44FGQ%23issuecomment-498713242&data=02|01||95b425b13eae4f38364c08d6e8fe956c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636952577453419558&sdata=LczVB%2BiG5zfi%2BvCPpMnQk9pNOpUtJ3wpHJ%2BG6jnJRjQ%3D&reserved=0, or mute the threadhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABOH2WUUKW5VLL3LOUYTKYTPY2AQ7ANCNFSM4HSGQG3Q&data=02|01||95b425b13eae4f38364c08d6e8fe956c|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636952577453429569&sdata=qUgF%2BwJ4osYybqTmyS%2F0NYCcxJ1XnoAA21tFgZ6DbTs%3D&reserved=0.

The problem is still there:

How can I use any previous version?

Everything is working just fine now (on Version: 1.3.3).
Thanks for reverting back until a fix is provided.

Cool thanks for the feedback
Verstuurd vanaf mijn iPhone
Op 11 jun. 2019 om 14:36 heeft Brainy Devs <notifications<EMAIL_ADDRESS>het volgende geschreven:
Everything is working just fine now (on Version: 1.3.3).
Thanks for reverting back until a fix is provided.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Frenevanosnabrugge%2Fvsts-promotepackage-task%2Fissues%2F24%3Femail_source%3Dnotifications%26email_token%3DABOH2WQMJDRDZCO2TNEHWHTPZ6L4ZA5CNFSM4HSGQG32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXM66II%23issuecomment-500821793&data=02|01||21ce751e3f8c4036c03508d6ee696ceb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636958533896235099&sdata=tP5XKXOnVClb8w1ZtNEnbEJQfs4TnQfURYHvLlkH2BI%3D&reserved=0, or mute the threadhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABOH2WW6QEQZBCADP3346A3PZ6L4ZANCNFSM4HSGQG3Q&data=02|01||21ce751e3f8c4036c03508d6ee696ceb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636958533896245110&sdata=ySLxh1IG3SKq1V0Wqb7VZhPF2lNcClCC6egYpMXyZ3U%3D&reserved=0.

new release works!

fixed


Send globals along with run_function step definition

Allows us to fix the issue where changing a global variable referenced in a function doesn't rebuild the function:
BASE_MODEL = "foo"

def download_models():
    from transformers import LlamaForCausalLM, LlamaTokenizer

    LlamaForCausalLM.from_pretrained(BASE_MODEL)
    LlamaTokenizer.from_pretrained(BASE_MODEL)

We're filtering out any functions referenced here because they can't be serialized in a "stable" way, but there might be other types we should filter out.
An alternative would be letting users specify parameters explicitly:
def download_models(model):
    from transformers import LlamaForCausalLM, LlamaTokenizer

    LlamaForCausalLM.from_pretrained(model)
    LlamaTokenizer.from_pretrained(model)

image = modal.Image.debian_slim().run_function(download_models, "foo")

This isn't great because it's easy for users to miss, and then leads to the same confusion as now when changing a variable doesn't change the image.

@prbot approve

Can look on Monday?

Not sure if you want another pair of eyes or just want to merge immediately

No rush. I decided to hold off to think about this a bit more.


Keep pdo mysql adapter fetch digit field type BC with php <= 8.0

Close #323
Task done

Specification PDO fetch mysql int type will return php 'int' string instead of php int type (test case)
https://github.com/Shardj/zf1-future/blob/048b066903190d22c1bef8c3b62b12943f8ca3b9/tests/Zend/Db/Adapter/Pdo/MysqlTest.php#L335-L352
Fail since php 8.1 https://github.com/hungtrinh/zf1-future/actions/runs/4171264561/jobs/7221025095#step:7:436
Specification PDO mysql adapter will init connection with driver_options PDO::ATTR_STRINGIFY_FETCHES is on (test case)
https://github.com/Shardj/zf1-future/blob/7c3911ca2933c8b57ea0a54d04eca4bfe1d73320/tests/Zend/Db/Adapter/Pdo/MysqlTest.php#L354-L374
Fail since php 8.1 https://github.com/hungtrinh/zf1-future/actions/runs/4171266288/jobs/7221028465#step:7:445
Production code fulfil 2 specifications above
https://github.com/Shardj/zf1-future/blob/7c3911ca2933c8b57ea0a54d04eca4bfe1d73320/library/Zend/Db/Adapter/Pdo/Mysql.php#L113-L119
2 Spec done https://github.com/hungtrinh/zf1-future/actions/runs/4171334806/jobs/7221167701#step:7:248

Mr @glensc could you please review this PR, thanks

this looks similar to https://github.com/Shardj/zf1-future/pull/320


10gen is now MongoDB

woo, we should start updating the references...

@jamesonjlee can you move this to https://github.com/chef-brigade/mongodb-cookbook/issues and close?


Customizable Run Message

Feature Request
Allow a run message to be passed as an argument to tfx run create, and I suppose tfx plan create?  Credit where credit is due, but TFx was here is not all that helpful 😉   Perhaps -m, --message string?

Will be in the next release


fix local variable name

What does this PR do?
Fix terraform error:
➜  bootstrap/terraform git:(main) ✗ terraform plan
│ Error: Reference to undeclared input variable
│
│   on main.tf line 89, in locals:
│   89:   kubernetes_version = var.kubernetes_version
│
│ An input variable with the name "kubernetes_version" has not been declared. This variable can be declared with a variable "kubernetes_version" {} block.
╵
╷
│ Error: Reference to undeclared local value
│
│   on main.tf line 148, in module "aws-eks-accelerator-for-terraform":
│  148:   cluster_version = local.cluster_version
│
│ A local value with the name "cluster_version" has not been declared.
╵

Motivation
Terraform bootstrap should be usable

@nabuskey Thanks for pointing this issue and raising the PR. However, this example needed more changes so i have added all the updated to this new PR #29
We can close this PR once you are happy with #29.


Release 2.0.0

We should probably write a bit more in the changelog before merging! Feel free to create a PR to this branch, release-v1.7.0, or push your commits if you are a part of jazzband, or just comment.. :smile:
Commits since last release:
* 455a027 (HEAD -> release-v1.7.0, upstream/release-v1.7.0, v1.7.0) Release 1.7.0
* 2ce2837 (upstream/master) Add support for python 3.7 (#492)
* a73253a Fix problems with CI (#501)
* c4a0d71 README: Update APNs link
* 9182958 flake8: Ignore E503 (use Black style)
* 4962944  Add FCM channels support for custom notification sound on Android Oreo
* 45f0e96 Fix APN bulk messages
* 6f7b80d Avoid mutating arguments in `gcm._cm_send_request`
* e335ac6 Fix error when send a message and the device is not active
* 600495b gcm: Simplify some code
* 8c73a77 Fix README missing paragraph break confusion
* 1be2f2a Add Jazzband contributing guidelines
* 2ae27f9 Update repository URLs (again)
* 5954578 Cleanup overzealous tests
* 9f71624 Move Python 2 compatibility imports to push_notifications.compat
* 3052927 Update setup.cfg and DRF support notice
* c821ec9 Convert changelog to Markdown
* 86409f6 Move AUTHORS to ACKNOWLEDGEMENTS.md
* 803caa9 Clean up code style issues
* f625921 Move base NotificationError to push_notifications.exceptions
* c4785a3 Drop apns_errors, use exception class name instead
* a3bda0a Clean up imports and switch to isort+flake8-isort
* cee4160 (tag: 1.6.0) Release 1.6.0

Because of the breaking changes, we will bump to version 2.0.0.

Hmm. Since https://github.com/jazzband/django-push-notifications/commit/f625921d0e086fdb572a0890396c4cc3e7719576 is breaking, we should most likely bump to 2.0. I guess a lot of users use the exception and don't have tests verifying that their code works, so it might break some production systems if people update without reading the changelog.
Any thoughts?

yes right

Hi, could you please see https://github.com/jazzband/django-push-notifications/pull/507 as it is a very minor change that solve a really big bug in WebPushDevice.send_message() and include it in this release?

Hi @wtayyeb! Yes, that is no problem, as the tag is still not set, and the PR just got merged. 😄 Thank you so much for the contribution!

Can you try to formulate an entry for the changelog for your patch @wtayyeb? This is the rest of the current changelog: https://github.com/jazzband/django-push-notifications/pull/502/files#diff-4ac32a78649ca5bdd8e0ba38b7006a1e

Hi @odinuge, I create new PR https://github.com/jazzband/django-push-notifications/pull/508, for this. its your turn to merge it.

Thanks! Hopefully we can release v2.0.0 soon 😄


MD5 check failure while 206 partial content download

ATS Version: 4.2.3
Description:
(1) Single thread complete download the specify url , the MD5 is correct
(2) 206 partial content download the specify url, the range content MD5 is error

curl -x<IP_ADDRESS>:80 -so /tmp/part1 --range 488205666-732308498  http://a.foo.com/bigfile

Debug Log:

why the cache_seek is a negative number ?
Has anyone encountered a similar situation or explained it ?
Thanks a lot .

Sorry, we didn't support 4.2.x anymore.
Please feel free to reopen it if this is in our supported version.


Failure to return reply "dns: bad rdata"

Hello. I am trying to run skydns on the top of CoreOS.
To do so, I have downloaded docker image skynetservices/skydns and run it with following parameters:
docker run --rm -p 53:53/tcp -p 53:53/udp -p 8080:8080 --name skydns skynetservices/skydns -machines=<IP_ADDRESS>:4001 -addr=<IP_ADDRESS>:53 -domain=.

After that I have added record to etcd:
etcdctl --no-sync set /skydns/local/private_registry {"host":"<IP_ADDRESS>"}

and reverse PTR record for this domain:
etcdctl --no-sync set /skydns/arpa/in-addr/20/122/168/192 {"host":"private_registry"}

When I am trying to ping private_registry:
ping private_registry

all seems to work correctly, but when I am trying to curl docker private registry service, running on this host, I got five second delay and such messages in skydns logs:
[skydns] Jan 20 17:28:34.158 ERROR     | failure to return reply "dns: bad rdata"
[skydns] Jan 20 17:28:39.163 ERROR     | failure to return reply "dns: bad rdata"
[skydns] Jan 20 17:28:44.167 ERROR     | failure to return reply "dns: bad rdata"

I think that problem is in incorrect setup of reverse  PTR records but I don't know what exactly problem is.

[ Quoting<EMAIL_ADDRESS>in "[skydns] Failure to return reply "d..." ]

Hello. I am trying to run skydns on the top of CoreOS.
To do so, I have downloaded docker image skynetservices/skydns and run it with following parameters:
docker run --rm -p 53:53/tcp -p 53:53/udp -p 8080:8080 --name skydns skynetservices/skydns -machines=<IP_ADDRESS>:4001 -addr=<IP_ADDRESS>:53 -domain=.

After that I have added record to etcd:
etcdctl --no-sync set /skydns/local/private_registry {"host":"<IP_ADDRESS>"}

and reverse PTR record for this domain:
etcdctl --no-sync set /skydns/arpa/in-addr/20/122/168/192 {"host":"private_registry"}

When I am trying to ping private_registry:
ping private_registry

all seems to work correctly, but when I am trying to curl docker private registry service, running on this host, I got five second delay and such messages in skydns logs:
[skydns] Jan 20 17:28:34.158 ERROR     | failure to return reply "dns: bad rdata"
[skydns] Jan 20 17:28:39.163 ERROR     | failure to return reply "dns: bad rdata"
[skydns] Jan 20 17:28:44.167 ERROR     | failure to return reply "dns: bad rdata"

I think that problem is in incorrect setup of reverse  PTR records but I don't know what exactly problem is.

I think your problems is fix by:
$etcdctl --no-sync set /skydns/arpa/in-addr/20/122/168/192 {"host":"private_registry.skydns.local."}

I.e. make the name fully qualified.
I think not doing this by default is correct, but I could be convinced otherwise
:-)
/Miek
--
Miek Gieben

Actually the README doesn't use the closing dot either. Should fix that as well.
Can you confirm adding a dot makes it work?

Hmmm, looking at the code I do already make the names fully qualified.
Let's see if I can replicate  this.

Only now I see your -domain=., that might makes things more interesting.. Is there any way you could tcp dump those replies?

Ah replicated.

Ah, yes.
;; QUESTION SECTION:
;private_registry.skydns.local.	IN	 A
;; AUTHORITY SECTION:
.	60	IN	SOA	ns.dns.. hostmaster.skydns.local.<PHONE_NUMBER> 28800 7200 604800 60
Adding the domain twice, because you're using domain=.

It also occours when a domain-part is too long


updated deconz version to 0.2.0

Why you removed the "Publish date"?

Readme.MD says: Delete the versionTime or update it to the current time

I also learn new things each day :-))


[Cloud Deployment IIa] Rclone docker image extension for config file

Similar to the YAML based dockerfile, it's smoothest for AWS integration to just pass the file stream via an environment variable, and since the config file will never be larger than the size limit on environment variables this is fine
Any alternative approach would need to deal with file transfer of the configuration via separate operations, which adds more complexity

OK, I finally got this working!
It's probably best if you review this @h-mayorquin so you can learn how the Docker stuff works for maintenance going forward. I mean, hopefully it would never break, but maybe updating various tokens will be required at some point, or updated documentation, etc.


Add old versions back

Proposed changes

Jira ticket: CLOUDP-261305

Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works

Further comments

PR #100!!! 💯  😮 🙀


simple_test fails on Ubuntu 18.04

I'm new to gRPC, so this may be a "user error" and not a real issue.
On a freshly installed Ubuntu 18.04.5 LTS, I followed the instructions in:
https://github.com/google/distbench/blob/main/docs/getting-started.md
I started a test_sequencer and a node_manager on localhost manually.
bazel run :distbench -- test_sequencer &
bazel run :distbench -- node_manager --test_sequencer=localhost:10000 --port=9999 &

I checked that they are running.
$ sudo netstat -ltnp | grep distbench
tcp6       0      0 :::9999                 :::*                    LISTEN      45149/distbench
tcp6       0      0 :::10000                :::*                    LISTEN      44538/distbench

However, I noticed the following error when I run ./simple_test.sh
Failed! ((grpc::status: failed to connect to all addresses)

I also ran start_distbench_localhost.sh after killing the processes that I had started earlier manually.
It ends with console output:
I20210731 14:06:31.600041 48153 distbench_test_sequencer.cc:125] node node0 was already idle
DistBench is running.
Try running simple_test.sh to give it a test to execute.

Now there is more output at the end of ./simple_test.sh
I20210731 14:07:26.748265 48153 distbench_test_sequencer.cc:125] node node0 was already idle
I20210731 14:07:26.748328 48153 distbench_test_sequencer.cc:233] All services placed manually
I20210731 14:07:26.748340 48153 distbench_test_sequencer.cc:278] Service Placement:
I20210731 14:07:26.748351 48153 distbench_test_sequencer.cc:280] node0:
I20210731 14:07:26.748361 48153 distbench_test_sequencer.cc:282]   leaf/0
I20210731 14:07:26.748371 48153 distbench_test_sequencer.cc:282]   leaf/1
I20210731 14:07:26.748380 48153 distbench_test_sequencer.cc:282]   leaf/2
I20210731 14:07:26.748390 48153 distbench_test_sequencer.cc:282]   load_balancer/0
I20210731 14:07:26.748399 48153 distbench_test_sequencer.cc:282]   root/0
I20210731 14:07:26.757014 48153 distbench_test_sequencer.cc:361] Finished AsyncConfigureNode status:UNAVAILABLE: failed to connect to all addresses
Failed! (grpc::status: failed to connect to all addresses)

Hope this helps in debugging!

I tried to test gRPC helloworld and got the "same" error except that I also saw the
error code (14). I fixed the issue by adding localhost to the no_proxy env variable.
However, the distbench errors didn't go away, so I cloned the distbench repo again,
tried the build and ran the unit tests. I found that test_sequencer test failed:
//:distbench_engine_test                                                 PASSED in 0.3s
//:protocol_driver_test                                                  PASSED in 0.3s
//:distbench_test_sequencer_test                                         FAILED in 71.2s

In particular, the clique test had failed. Output of grep RUN test.log shows:
[ RUN      ] DistBenchTestSequencer.ctor
[ RUN      ] DistBenchTestSequencer.init
[ RUN      ] DistBenchTestSequencer.empty_group
[ RUN      ] DistBenchTestSequencer.nonempty_group
[ RUN      ] DistBenchTestSequencer.100k_grpc
[ RUN      ] DistBenchTestSequencer.100k_grpc_async_callback
[ RUN      ] DistBenchTestSequencer.clique_test
[ RUN      ] DistBenchTestSequencer.stochastic_test
[ RUN      ] DistBenchTestSequencer.protocol_driver_options_test

The output for grep FAIL test.log is:
[  FAILED  ] DistBenchTestSequencer.clique_test (15037 ms)
[  FAILED  ] 1 test, listed below:
[  FAILED  ] DistBenchTestSequencer.clique_test
 1 FAILED TEST

Hi Ajit,
Thanks for reporting the bug and the no_proxy fix. I will add a FAQ with that workaround.
For the clique test failing; could you save the output of make testlog and send it to me by email ?

Thanks, Olivier! I've sent you an email.
BTW, the no_proxy fix didn't really work for distbench (it only worked for the gRPC helloworld).

I added a FAQ in #21 which recommend un-setting the proxies; I think that should be the workaround for your issue.

Closing as FAQ has been merged.