Swift closure context memory leak when binding collection view items from BehaviorSubject

When the view controller below is loaded, xcode memory graph shows that a swift closure context is leaking.
import UIKit
import RxSwift
import RxCocoa

class Test4ViewController: UIViewController {
    private let collectionViewLayout: UICollectionViewLayout = {
        let layout = UICollectionViewFlowLayout()
        layout.scrollDirection = .horizontal
        return layout
    }()

    private lazy var collectionView: UICollectionView = {
        let collectionView = UICollectionView(frame: .zero, collectionViewLayout: collectionViewLayout)
        collectionView.register(UICollectionViewCell.self, forCellWithReuseIdentifier: "test")
        return collectionView
    }()

    private let subject = BehaviorSubject<[Int]>(value: [1, 2, 3])

    private let disposeBag = DisposeBag()

    override func viewDidLoad() {
        super.viewDidLoad()
        view.addSubview(collectionView)

        subject.asObservable()
            .bind(to: collectionView.rx.items(cellIdentifier: "test", cellType: UICollectionViewCell.self)) { item, feedId, cell in
                cell.backgroundColor = .black
            }
            .disposed(by: disposeBag)
    }
}

It's not.

@kzaher care to elaborate on this? I have several similar leaks but not on binding.


Bug: Cannot delete demo account in Dashboard

What are you trying to achieve?
I'm trying to delete a superuser account with email<EMAIL_ADDRESS>Nor can I disable this account.
Steps to reproduce the problem
Go to Staff Members and delete unnecessary accounts.

What did you expect to happen?
As other demo accounts, I can delete this account.
Logs
No response
Environment
dashboard v3.20.20
core v3.20.56
OS and version: debian 12

Thank you for the report @charleslcso, I have created a bug report for our team.
Internal link to track: MERX-1323: Apps Marketplace not loading in Dashboard 3.20.20

Hey, you essentially cannot delete the user that is a "superuser" - when you bootstrap the Saleor with an empty DB there is no possibility to create accounts except you superuser account thus, it's being populated.
We are going to change it a bit in the future and probably stop creating these accounts without demand.

Thank you for your info.
Now I have created extra users, and I’m not sure if they are superusers or just some other users. How can I check this?
Also I’m leaving my Saleor instance open to the net. Even though I can’t remove it, is there a way to:

disable it
Change password

Leaving it there exposed should create security risks.


No tasks detected when not use wrapper

Extension Name: vscode-gradle
Extension Version:  3.5.2
OS Version: Windows 10 20H2 19042.685
VSCode version: 1.52.2
My project does not use gradle wrapper.
Try use
"gradle.autoDetect": "off"

but not show gradle task view
When i use vscode-gradle and Java Extension Pack together with specified local installation directory,
    "java.import.gradle.wrapper.enabled": false,
    "java.import.gradle.home": "D:\\Bin\\gradle-5.6",

"Gradle Tasks" output this
Output from "Gradle Tasks"
[debug] [gradle-server] Starting server
[debug] [gradle-server] Gradle Server cmd: c:\Users\xiaoyao9184\.vscode\extensions\richardwillis.vscode-gradle-3.5.2\lib\gradle-server.bat 6044
[info] [gradle-server] Server started, listening on 6044
[info] Gradle client connected to server
[debug] Refreshing tasks
[error] [gradle-server] Gradle version is required
[error] Error getting build for e:\WorkProjects\********************: Gradle version is required
[info] Found 0 tasks

Does the bug still exist if you disable all other extensions?
Yes without wrapper
No with Java Extension Pack and disable it wrapper mode.

I am having similar issues where i do not have a wrapper and need to work in offline mode.
I have the below settings enabled
"java.import.gradle.offline.enabled": true,
"java.import.gradle.wrapper.enabled": false,
"java.import.gradle.version": "6.0.1",
"java.import.gradle.home": "/opt/gradle/gradle-6.0.1/bin/gradle",
"gradle.nestedProjects": true,
"gradle.debug": true,
"gradle.autoDetect": "off",

No tasks appear in the 'gradle tasks' window, and upon clicking 'run a gradle build' and inputting
build -x test
I get the following error:
org.gradle.tooling.GradleConnectionException: Could not install Gradle distribution from 'https://services.gradle.org/distributions/gradle-6.0.1-bin-zip.
I can run the same gradle command on the command line fine, as well as if i configure it as a vscode task

fixed in #922. Close it now. If you find this issue still exists, feel free to reopen it.


Add "k4run" gaudirun script (formerly fccrun)

This exposes almost all job options as command line parameters.

Hi Valentin,
Is it possible to specify the evtmax in this script?
Thank you.
Tao

Hi Tao, yes it is, with k4run -n EVTMAX, see https://github.com/key4hep/K4FWCore/blob/a933d5bce483b25cb3795ee1512c287820e15cff/FWCore/scripts/k4run#L46

Hi Tao, yes it is, with k4run -n EVTMAX, see
https://github.com/key4hep/K4FWCore/blob/a933d5bce483b25cb3795ee1512c287820e15cff/FWCore/scripts/k4run#L46

Hi Valentin, it is great. Thanks.


allow configuring opa plugin to provide custom path to opa

configuration of the opa plugin
we made an extension to opa to support an extra function in the policy file. Right now, opa plugin executes the default opa installed (not the customized one), so we cannot use the plugin with our "customized" opa to execute policy files
Implementation details

Hi @gusega, the opa.path configuration setting lets you specify what executable of OPA the extension should use.

Sorry man, mixed up the plugins 😅. Reopening.


add reset signal to display

define reset signal
add connection to a dummy in Wokwi model, as the model's display board is missing a reset pin
configure display library to use reset pin (it does configure the pin to output mode)

Although I can see in the simulation that the reset signal is used by the library, it has no effect on the outcome (neither in the simulation, nor in the actual device).

The addition does not have the intended effect.


bbb 2.2 version

Hi, how can i get code for BBB 2.2?

@kenan52
The source code for BBB 2.2 is not available.
You will have to modify  the below files
bbb-mp4.js
bbb-mp4.sh
Once you are done with changes, create a new image and use that image in bbb-mp4.sh

Closing this due to inactivity


Clarity change viewbox of svg's when using clarity-icons

Select one ...  (check one with "x")
[x] bug
[ ] feature request
[ ] enhancement

Expected behavior
I have a dashboard application which is using clarity as the ui-framework.
In this application a use a custom angular2-module which includes svg charts.
I wanted to add the clarity icons which needed the two entries in the angular-cli.json

<EMAIL_ADDRESS>"../node_modules/clarity-icons/clarity-icons.min.js"

My charts should work fine and i should see the clarity icons.
Actual behavior
When i add the "custom-elements.min.js" clarity seems to change the viewBoxes of my svg's in my custom charts module.
Reproduction of behavior
https://plnkr.co/edit/MNh01jN1hgos3aTliGto
if you remove:
<script<EMAIL_ADDRESS>from index.html the svg is displayed correctly!
Environment details

Angular version: 2.0.X
2.4.3
Clarity version:
0.8.8
OS and version:
Windows 10
Browser: [Firefox 51.0.1 | Edge 38.14393.0.0 ]

Works fine in Chrome 56.0.2924.87

@rubiktubik : One issue I see is that you have an unnecessary space in the shape attribute of your icon. So if you change <clr-icon shape=" dashboard"> to <clr-icon shape="dashboard">, the info icon goes away.
You should also specify a color in your CSS (#fff?) so that the icon doesn't appear blue. It's picking up the blue color because it's wrapped inside a link.

@rubiktubik : This looks to me like a browser issue with SVGs in general.  But the rectangle is positioned oddly inside the viewbox and the svg is left to its own devices to determine its width. So the svg element is doing double the effort of trying to figure out its width and height as well as applying viewbox. From my experience, it's easier to put the svg inside a containing div so it only has to worry about sizing to its container.
I'm not sure what custom-elements is doing that is interfering here. But if using a container doesn't suffice, it may be worth it to file a bug on their project.

I found this issue on the custom-elements project:
https://github.com/webcomponents/custom-elements/issues/14
Using alpha4 and rc1 version solves the problem!
This versions are not on npm yet. So i needed to download it from github manually.

Awesome. Thanks for following up on that, @rubiktubik.
We will need to make sure we update when these packages are available through NPM.

Looks like 1.0.0-rc.3 has been published: https://www.npmjs.com/package/@webcomponents/custom-elements

We should make sure this gets into planning this week or next. Thanks for the heads up.
On Apr 4, 2017, at 5:54 PM, Jehad Affoneh <notifications<EMAIL_ADDRESS>wrote:
Looks like 1.0.0-rc.3 has been published: https://www.npmjs.com/package/@webcomponents/custom-elementshttps://urldefense.proofpoint.com/v2/url?u=https-3A__www.npmjs.com_package_-40webcomponents_custom-2Delements&d=DwMCaQ&c=uilaK90D4TOVoH58JNXRgQ&r=fTP1U0GY81PhDMWNG5UbxoWqeCcP89bIAA4eQJ2VcGk&m=-Zq5HncaxY18FHei46utb7x-eLevYvdR87m4wAsQs60&s=jXDowItpw9H1YGFuRqe_O37iUQLJ-HMbUbeGFgjIuUk&e=
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_vmware_clarity_issues_546-23issuecomment-2D291663361&d=DwMCaQ&c=uilaK90D4TOVoH58JNXRgQ&r=fTP1U0GY81PhDMWNG5UbxoWqeCcP89bIAA4eQJ2VcGk&m=-Zq5HncaxY18FHei46utb7x-eLevYvdR87m4wAsQs60&s=cnmPlxCCB6klZjqj1Dgzza0T4qaEMvxTu1qa9JjkKAc&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ACmhp19R5SOivXibxyM3bxpSzELoRf2wks5rssoBgaJpZM4MRHdB&d=DwMCaQ&c=uilaK90D4TOVoH58JNXRgQ&r=fTP1U0GY81PhDMWNG5UbxoWqeCcP89bIAA4eQJ2VcGk&m=-Zq5HncaxY18FHei46utb7x-eLevYvdR87m4wAsQs60&s=LTjKlG3plcG159WY3AIPND_C4VB3QCya1UHiPD-KUAE&e=.


Controlling stepper driver boards with multiple stepping options

HI,
On stepper driver boards with multiple stepping options (1/8th stepping, 1/16 stepping .....) and multiple pins to set those options, e.g the big easy stepper driver board, do the stepping option pins need to be set specifically by the controller (e.g. an Arduino) or is this completely handled internally by the continuous stepper library.
Regards,
Richard.

Hi Richard,
As far as I know, libraries don't deal with the controller's settings.
You just need to adapt the speed in your application.
Best regards,
Benoit


[Event Request] Table 5406 "Prod. Order Line" - OnAfterGetItem, OnAfterGetUpdateFromSKU

Please add two new Publishers in Table 5406 "Prod. Order Line":
    [IntegrationEvent(false, false)]
    local procedure OnAfterGetItem(var Item: Record Item; var Rec: Record "Prod. Order Line")
    begin
    end;

    [IntegrationEvent(false, false)]
    local procedure OnAfterGetUpdateFromSKU(var Rec: Record "Prod. Order Line")
    begin
    end;

Change Function "GetItem":
    local procedure GetItem()
    begin
*        if Item."No." <> "Item No." then begin      //add "begin"
            Item.Get("Item No.");
*           OnAfterGetItem(Item. Rec);               //new
*       end;                                         //new
    end;

Change Function "GetUpdateFromSKU":
    local procedure GetUpdateFromSKU()
    var
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeGetUpdateFromSKU(Rec, SKU, Item, IsHandled);
        if IsHandled then
            exit;

        GetItem();
        if GetSKU then begin
            "Unit Cost" := SKU."Unit Cost";
            if (SKU."Routing No." <> "Routing No.") and (SKU."Routing No." <> '') then
                Validate("Routing No.", SKU."Routing No.");
            if (SKU."Production BOM No." <> "Production BOM No.") and (SKU."Production BOM No." <> '') then
                Validate("Production BOM No.", SKU."Production BOM No.");
        end else begin
            "Unit Cost" := Item."Unit Cost";
            "Routing No." := Item."Routing No.";
            "Production BOM No." := Item."Production BOM No.";
        end;

*       OnAfterGetUpdateFromSKU(Rec);              //new

    end;

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. 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.
Build ID: 42495.


Regression Test Failures

I am getting 16 passed test and 13 failed tests when trying to run the regression testing method using Visual studio outlined here ... https://openfast.readthedocs.io/en/master/source/testing/regression_test_windows.html.

I also tried to inspect the plots as suggested on issue #260 and have matplotlib installed. However I can't seem to produce the plots using -p. Opening the regression test summary gives me a page of the tests, inspecting an individual test gives me the screenshot below and if I select go to plots I get a file no found error.

How do I go about producing the regression test plots and fixing the failed tests?
Thanks
George

Hi @GeorgeElderfield,
Given that all of the 5MW cases failed, it seems likely that the DISCON controllers are not compiled correctly or not in the correct location. You should check the log file for those cases for more info. And also check out the docs for ctest configuration.
On the plotting, only the failing test cases will generate plots so since AWT_YFix_WSt passed, the plots are not generated. If the 5MW cases also do not have plots, they very likely didn't run. And finally, the linearized cases don't produce a time series so there's nothing to plot there. In fact, the manual regression test script doesn't run the linearization cases. To run all of the tests, you should use the ctest driver.


Angular Wrapper in sync with core MSAL 1.0.x

This is a placeholder to track all the "Angular" specific issues which simply need the wrapper to be upgraded to the latest msal js core version.

supports latest Angular versions with rxjs support
is Msal JS 1.0.X compatible - with access to the new API Surfaced

All tickets raised asking for this support will be tagged with this issue going forward. This is to simplify the traction from our end.
In case if anyone has specific Angular related requirement that you believe is not covered with the plan above, please feel free to drop a comment on this issue.

From #385: Make protectedresourcemaps flexible post-initialization of MSAL.

From #421: Angular 6/7/8 support with rxjs-compat

From #475: Typescript typings support in Angular Wrapper

From #786: redirectURI as a function -> This should happen when the wrapper is compatible with msal 1.0.x

From #526: Separate the ATS redirect handling from the main application code. Write a small html/js service that processes the token extraction and passes the control to the parent (Angular app) to avoid reload every time a token is acquired.

From #539: Routes without router guard would need full path match before they are pushed to unprotected resources. Add this as a test case for the new Angular Wrapper.

From #507: Error handling for Angular Wrapper.

From #497: Introduce a flag in the angular wrapper for the interceptor to fallback to interactive calls when acquireTokenSilent fails. This is needed for the wrapper as the angular developers has no control of the interceptor code to write the fall back code themselves.

From #733: Angular wrapper for error handling

From #629: prompt=select_account removal from msal js. This is already addressed in msal 0.2.4 but since angular is stuck on an older version, this is an issue for some users. Feedback is to not hardcode msal versions to be used with angular, #785 partially addresses this - revisit if more checks are needed.

From #614: state parameter set by the user in request to be sent in the response. Addressed in latest msal js 1.0.x.

From #481: Make msal configuration in Angular dynamic

From #598: Route guard does not have the ability to handle acquireTokenSilent() failure. See how to better design this in the new angular wrapper.

From #788: Same as #526. Redirect routed to a different HTML than the main angular app.

From #585: Redirect sample and documentation for Angular

From #729: Broadcast does not work when the token is picked up from cache in Angular (using sessionStorage).

From #703: Msal Guard to support redirect to "Access denied page" as CanActivate guards support URLTree as a return type now in Angular. Feature-ask for the angular wrapper.

From https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/552: Support glob patterns for protected resources

From https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/729: No event when token is retrieved from cache


Fix broken Response file documentation link

Hi there, changing ConfigurationOption/ResponseFiles.md -> ConfigurationOptions/ResponseFiles.md.
This was jarring while I was browsing the docs, happy to find the real page there.

Nice catch! Thanks a lot! :)


Make as method work on GodotScript objects

This was causing a compiler error due to the fact that the conversion to an inout type wasn't an explicit cast. This makes it an explicit cast.

Not sure why the constructor doesn't work, but I'm also not surprised. The const system has caused a lot of issues like this, especially in templates. Thanks for the fix.


[Feature Request] Add ContextEntries to requests that can be retrieved in custom gql Link

In order to create advanced Links that can execute custom logic per request, we need a way to extend request with ContextEntries.
Currently, generated code for a request is:
  @override
  Request get execRequest => Request(
    operation: operation,
    variables: vars.toJson(),
    // no passed context. Thus, the default is `context = const Context()`
  );

gql_exec has a method .withContextEntry on Operation that we could use.
I'd be happy to contribute if you think it is a good issue 🙂
(😅 Hope I did not miss something in the API that already allows us to create context entries)
Reference: #556

Hello @knaeckeKami !
Alright, here is my plan. What do you think ?

Extend OperationRequest with a withContextEntry<T extends ContextEntry>(T entry) method
Modify codegen to implement withContextEntry and pass it to execRequest

One way would be to add a Context? getter to the abstract OperationRequest class and adapt ferry_codegen to add this to the generated Request classes.
This would already allow users to add the context and modify Contexts via the built_value builders;
withContextEntry() is probably not strictly necessary since the built value classes already support that in their rebuild() methods, but it would allow to work with the context in TypedLinks where the exact type is not known.
A challenge is the serialization of the OperationRequest; Since the context is not serializable (it's a map of <Type, ContextEntry>, we would have to exclude it like updateResult.

Ok got it. So:

Context? getter
codegen & exclude from serialization

sounds good to me!

@tpucci This is a life saver, thank you! Would it be possible to also have a withContextEntry and updateContextEntry in the OperationRequest so we can also modify the context globally, but before any link? My use case is I need to add a unique transaction id for each request header (not the same as the requestId), and also want to pass the original stack trace of the caller to simplify debugging (stack trace in chains loses the original caller info it seems).

Hey @bawahakim
I am not sure I understand.
withEntry and updateEntry exist on Context, you could use those methods 🙂
If you want to set your context per request:
client.request(GSomeOperation((b) {
        b
          ..context = const Context().withEntry(YourEntry());
      })

If you want to set your context before all your existing links, create a link and put it first in your client link chain:
class YourLink extends Link {
  @override
  Stream<Response> request(Request request, [NextLink? forward]) {
    return forward(request.withContextEntry(YourEntry()));
  }
}

...

Client(
    link: Link.from([
          YourLink(),
          OtherLink(),
          HttpLink(
            const String.fromEnvironment('GRAPHQL_ENDPOINT'),
            httpClient: httpClient,
          )
    ]),
)

What do you think?

@tpucci Thank you! Let me explain a bit more my use case. The main issue I have is logging. Handling any error within a chain does not give me a clear stack track, only the trace of the links, so I lose information about the original caller. What I do is I inject the current stack trace in a wrapper method to ensure I can always know who the original calls was. I then use an extension method to handle any errors and use the original stack trace for my logger
class MyGqlClient {
  final Client client;

    Stream<OperationResponse<TData, TVars>> request<TData, TVars>(
    OperationRequest<TData, TVars> request,
  ) =>
      client.request(request).catchErrors(StackTrace.current);
}

extension RequestExtension<TData, TVars>
    on Stream<OperationResponse<TData, TVars>> {
  Stream<OperationResponse<TData, TVars>> catchErrors(
    StackTrace stackTrace,
  ) =>
      map((response) {
        // Error handling
      }
}

class SomeConsumerClass {
  void someMethod() {
    MyGqlClient().request(GMyRequest());
  }
}

This way, my stack trace bypasses anything related to the links and I immediately get the stack from the original caller, making debugging easier. This works so far.
Now I also need to send a unique x-transaction-id header with every request. That's easy enough to do, but I also need to pass it down my catchErrors method so I can attach it to the logger (so I can match the client request error to the backend error). Note that I cannot use requestId because it serves a different purpose.
I can see 3 solutions, for which I have not found a way to implement:

Continue with my approach, and inject the x-transaction-id header in a custom Link. However, I have no idea how to retrieve it later when I get an error in the catchErrors method
Inject all I need in the request.context object wrapper method (stack trace, transaction id, etc). This way I can handle all error handling within the link without catchErrors
Find another way to get the proper stack trace within the links without injecting it

If the abstract OperationRequest had a withContextEntry method, then I could create a new request with my context entry, enabling solution 2.
Appreciate your help!

Ha, got it 👍
Yeah you are right, there is no API to implement this behavior.
Maybe we could create a transformContext like the existing transformOperation on OperationRequest.
What do you think @knaeckeKami ?


OwnCloud RainLoop App Permission Issue

RainLoop version, browser, OS:
Version 4.2.7, Firefox Developer Edition, Arch Linux (client), Debian/Apache2.2 (production)
Expected behavior and actual behavior:
Except the webmail / admin interface to load, but it doesn't due to a configuration error.
I have added a specific directory rule for '/apps/rainloop' with an allow all directive to try and circumvent the problem. It is unclear to me which apache rule is causing this issue.
Steps to reproduce the problem:
Install the OwnCloud RainLoop App. OwnCloud installation trough OwnCloud stable repositories.
https://apps.owncloud.com/content/show.php?action=content&content=165254
Logs or screenshots:
Apache error.log
[error] [client <ip>] client denied by server configuration: /var/www/owncloud/index.php, referer: https://example.com/owncloud/index.php/apps/rainloop/app/?admin

This appears to be an extention of issue 1163. It appears that certain requests (especially in the admin panel) hit default-OWASP mod-security rules.


Why Raw returns string type, not []byte?

This is because the base type for gjson.Get(json, path) is the string. The result.Raw field is just a substring of the originally passed in json parameter. From my testing, having everything as strings seems to keep the total allocations down and allows for better overall performance
If you are using the gjson.GetBytes(json, path) function and you want to avoid the converting result.Raw to a []byte, then you can use this pattern:
var json []byte  // using json byte slice
result := gjson.GetBytes(json, path)
var raw []byte
if result.Index > 0 {
        raw = json[result.Index:result.Index+len(result.Raw)]
} else {
        raw = []byte(result.Raw)
}

Which is a best-effort no allocation sub slice of the original json. This method utilizes the result.Index field, which is the position of the raw data in the original json.
It's possible that this value might equal zero, in which case we'll just convert the result.Raw to a []byte.
I hope this information helps. Let me know if this does not answer you question.

Thank you for the detailed answer, this method is very useful, can it be included in the documentation or realized in the code?

Good idea. I think I'll add it to the README file.


Full uaa.yml config file reference

For those of us deploying this project as an end-user application, compete documentation on the yaml config file would be wonderful!

@dfraser We will get the docs updated.  In the meantime please refer to the spec file here:  https://github.com/cloudfoundry/uaa-release/blob/develop/jobs/uaa/spec which contains explanation of the properties.

Thank you!  that's very helpful.


Test arming detector before grid detection

If we can arm before grid detection we can produce some speed up. Currently we need to do grid detection for working out the number of images but after discussion with @GDYendell it should be possible to set the number of images to 0 and arm before/whilst working out grids.
Acceptance Criteria

Run with 0 frames and see if we still get good data
We may need to be careful in the disarming to watch to make sure the number of written frames matches the expected number before disarming.

Ah - I may have misunderstood. The file writer accepts 0 to run forever, but the eiger does not. It also does not implement Continuous mode, despite being able to set that in the UI.

Hmm... What would happen if we just set an arbitrarily high number of frames then disarm part way through?

That could work, but you would get extra frames in the file in the delay between seeing it has the correct number and sending the stop. Is that a problem?

We're still using an external trigger, which will stop on it's own at the correct number of frames so I don't think there would be any additional?

Oh ideal then that will be fine.

Do you think we should be setting both Eiger and Odin to this arbitrary number or just the Eiger then Odin to 0? Probably best to go with whatever minimises potentially scary log messages

It is probably fine either way. Setting to 0 would be slightly neater because if you set the writer to an arbitrary number, then the that number will appear as the max size in the file and it won't be full. If you set zero it will just resize up to the correct size. I don't know if that makes any difference to you.

Attached commit tested on beamline and works, needs tidying


Env2Comp Helm Chart

ortelius/ortelius#164
Addition of helm chart. Will need to update values file when env2comp image is available.

Merging. CloudBuild check can be ignored for now since there is no image to build


Possible typo

https://github.com/cytoscape/cytoscape.js-cose-bilkent/blob/master/cytoscape-cose-bilkent.js#L5121
I think it should be "this.cy" instead of "cy".
var cyNode = this.cy.getElementById(lNode.id).parent()[0];
It was throwing an error, which I corrected in the source file and now all's ok for me.

@danpaulsmith
You are right. It should be 'this.cy' instead of 'cy'. In the application where I am usually testing cose-bilkent there is a global variable 'cy', so I could not find out this error before. I have committed the changes after replacing 'cy' with 'this.cy' as you described. Thanks for informing us.

Published as 1.2.2
@metincansiper In general, each bugfix should get a new  patch release once it's closed.


Continue after exception + show exception log

Hello,
Today abapGit aborts after the first exception. Besides, it is very hard to figure out what's wrong. Right now I'm getting "Error from RPY_TRANSACTION_INSERT" but I don't know which object is causing the problem. So I'll have to debug.
It would be great if, instead of aborting it would:

Skip that object and continue
Provide a log of all the objects with error + exception text

when does it happen? when pulling abapGit?
Most of the error texts are unique, so it raises the error in:
https://github.com/larshp/abapGit/blob/14fc9a2413c0a0d9c11b548bfd41b93f768bfe28/src/zabapgit_object_tran.prog.abap#L348

It happens when importing in one system a ZIP file which I exported from another system. I am using ZIPs while I cannot make abapGitServer to work ;)

first I was getting an exception due to that problem with the function group
so I removed the function group from the ZIP
then I started getting this error and couldn't understand why
that's when I realized that there was indeed a transaction object in the ZIP which was related to the removed function group. so I removed it and now the import is successful.

But my point is: I had to debug in order to find out the name of the object. It would be more convenient to get a simple ALV with the list of objects with exceptions.

yes, good idea

I now realize that I posted this issue in abapGitServer when it really applies to abapGit. Sorry. But now I don't know how to move it to the right place...

just close this one and create a new
you can post the url in the text and Github will show a link between the issues

It hasn't happened in a while so I don't have a test case. I'll create a new one in abapGit the next time it happens ;)


Update K8s version in the CI pipeline

Is your feature request related to a problem? Please describe.
Due to a recent problem with Activity deployment on activity.hikaya.app being down, @Darius-Ndubi has resolved the issue and upgraded the K8s version of the cluster.
The CI needs to be brought up to this manual upgrade by setting the K8s version in its respective Github secret.
Acceptance Criteria
A clear and concise description of how to ensure the feature request is met.
GIVEN I am a developer
AND I trigger a CI pipeline
THEN I expect the K8s Activity deployment to be of the new version

This will be abandoned soon since FluxCD will be responsible of the infrastructure.


How is the GPU vs CPU tensor management done?

Hi. I have trouble understanding how and when do you send tensors to the GPU. I have a custom training routine where I use your dataloader and some modules using the build_X methods. When the data comes from your dataloaders, neither the data nor the model is required to be in the GPU. The tensors must be in the CPU for the modules to handle, which otherwise will raise an error of mismatched memory allocation for input and model. For example, for an Anchor3dHead, the passed tensor is sent through a Conv2d layer (https://github.com/open-mmlab/mmdetection3d/blob/78f456216f101b5b684db754176e0def17ab9d49/mmdet3d/models/dense_heads/anchor3d_head.py#L150), and I cannot find anywhere along the way any .to(device) or .cuda(). Yet, the processing seems really fast. Can you give me some insights on how is this done?

All the fundamental operations, e.g. training, data loading, model construction is handled via MMCV. Let's take a Detector trained for several epochs using multi-GPUs distributed training for example:

We construct a epoch_based_runner for model training, each train/val step is performed inside it.
In a epoch_based_runner, we train each step via this line.
Move one step further, it leads to here and here.
Note that the Detector is wrapped inside a MMDistributedDataParallel (which is similar to torch's DistributedDataParallel). The model is wrapped here so that it's put to GPU.
Finally we got this line, which put data to GPUs as well.

I think this answers your question. Because our model and data are not explicitly converted to GPU, but they are done in some basic inner modules. Other conditions (iter_based_runner, single GPU training) is similar, where MMCV is used to perform these fundamental operations.

Thanks for the prompt response. I am not using your training routine at tools/train.py, but my custom training file. I build your modules with the corresponding build_X functions. Namely, I use an image backbone and a point cloud backbone (with build_backbone(my_custom_config)) and a dense head (build_head()). These should be standalone modules on the CPU, or am I mistaken?

Yes, if you only build them, they should only be on CPU. (You can simply check them using pdb)


Lazy validation is missing errors

Describe the bug
When validating with the lazy=True flag Pandera does not seem to be catching all failure_cases the first time around.

I run schema.validate(df, lazy=True) on my DataFrame
I inspect the failure cases on the SchemaErrors failure_cases property to see where I have validation errors
I remove all the problematic rows (from index) from my DataFrame via df = df.drop([<indexes>])
I re-run validate(df, lazy=True) on the DataFrame (from where I removed the problematic rows), expecting no errors to appear, but now I see new failure cases on the Exception object. OBS: All of the first reported failure cases are removed, but new ones has appeared now. Here my expection would have been that all errors were reported intitially to be removed, but this is not the case

Do I have a wrong understanding of the failure cases? Should'nt all cases be reported in the first place?
I've tried to run the steps over and over again, and eventually all failure cases will be removed and no new cases are found. But I have to re-run the steps 10-15 times. This behavior appears odd to me.
If I simple go look for the data with Pandas, I am able to successfully retrieve all problematic rows in the first try.

[ x ] I have checked that this issue has not already been reported.
[ x ] I have confirmed this bug exists on the latest version of pandera.
[ ] (optional) I have confirmed this bug exists on the master branch of pandera.

Code Sample, a copy-pastable example
# Schema defintion
schema = pa.DataFrameSchema({
    "mileage": pa.Column(int, pa.Check.in_range(0, 400000, include_min=True, include_max=True))
})

# Apply validation
try:
    df = schema.validate(df, lazy=True)
except SchemaErrors as ex:
    failure_cases_df = ex.failure_cases

# Drop failure cases
df = df.drop(failure_cases_df["index"].values)

# Re-apply validation
try:
    df = schema.validate(df, lazy=True)
except SchemaErrors as ex:
    failure_cases_df = ex.failure_cases # <- New unseen failure cases appear here

# Get problematic cases with Pandas
test = df[
    (df["mileage"] <= 0) |
    (df["mileage"] >= 400000)
]

# "test" now has all the problematic cases

Expected behavior
I would expect ALL failure cases to be reported the first i run validate() on my DataFrame, so I can remove the problematic rows afterwards.
Desktop (please complete the following information):

OS: [macOS Big Sur 11.2.3]
Browser [Firefox]
Pandas Version [1.2.4]
Pandera Version [0.6.3]

Well, 5 min after posting this issue I found n_failure_cases on the Check class which by default is set to 10. If set to None ALL the failure_cases are now displaying.
For reference:

# This will only return 10 failure cases
schema = pa.DataFrameSchema({
    "mileage": pa.Column(int, pa.Check.in_range(0, 400000, include_min=True, include_max=True))
})

# This returns ALL of them
schema = pa.DataFrameSchema({
    "mileage": pa.Column(int, pa.Check.in_range(0, 400000, include_min=True, include_max=True, n_failure_cases=None))
})

@jesperbruunhansen yes this behavior is kind of unintuitive and one of those artifacts from the early days of the library.
Do you think n_failure_cases should be something configurable at the global level?
A couple of non-mutually exclusive options:

have a mechanism to configure these options like pandas.set_option or pandas.option_context
configure as an environment variable like PANDERA_N_FAILURE_CASES
establish a config file that looks at various default places like ./pandera.yml, ~/.pandera.yml with an n_failure_cases option
... something else?

we can preserve the pa.Check(n_failure_cases=...) kwarg to override the configured value in case the user wants to set a different value for a particular check. Thoughts?

Hi @cosmicBboy and thanks for looking into this.
Personally, I can say that I intuitively expected the framework to return ALL failure cases as default. For me it would make sense to set n_failure_cases  as None as pr. default, and then if people wanted less output they could set a limit?
All of the options you mention could be a potential solution, but for me it would OK to just invert the logic from what it is now.

@cosmicBboy and one more thing - Are n_failure_cases also applicable for check like nullable which is not on the Check object?

Hi @cosmicBboy first off, awesome library, thanks for the hard work!
I agree with @jesperbruunhansen that inverting the logic from what it is now i.e. all failure cases are returned but we can specify a maximum number for failure cases through n_failure_cases, would be a nice solution.


Hedwig 2.0 - Robot Behaviour

This PR starts the process to move away from issues brought up in #60 and #61.
Hedwig 2.0 will aim to provide a very ergonomic API for folks who just need a basic bot:
defmodule MyApp.Robot do
  use Hedwig.Robot, otp_app: :my_app
end

As well as more advanced OTP use-cases:
defmodule MyApp.Robot do
  use Hedwig.Robot, otp_app: :my_app

  def start_link(opts \\ []) do
    Hedwig.Robot.start_link(__MODULE__, nil, name: {:global, "my_app_robot"})
  end

  # ...
end

The API is still a work in progress...
TBD

Should :otp_app and/or :adapter options be required at compile-time?

TODO

[ ] Get test coverage to > 95%
[ ] Documentation

Coverage decreased (-0.6%) to 91.534% when pulling 274b7775f5ba35790d26f9d7f5e2e4370e4babe0 on robot_behaviour into bc8dfab94b95b14d56a9dcf133b5a19ace49c911 on master.

Coverage decreased (-22.7%) to 69.373% when pulling dcb25caf0c06cf7e07cfb9da80ca502dfaa3ac20 on robot_behaviour into bc8dfab94b95b14d56a9dcf133b5a19ace49c911 on master.

Coverage decreased (-22.7%) to 69.373% when pulling dcb25caf0c06cf7e07cfb9da80ca502dfaa3ac20 on robot_behaviour into bc8dfab94b95b14d56a9dcf133b5a19ace49c911 on master.

Coverage decreased (-17.7%) to 74.453% when pulling f05a9a4d0b3c1c8b0b165ad11ae3acf4b7aac96c on robot_behaviour into bc8dfab94b95b14d56a9dcf133b5a19ace49c911 on master.

Coverage decreased (-17.7%) to 74.453% when pulling a8266945ad6484c84fad4a960511d61122705fd5 on robot_behaviour into 3817edb3d4513068da178794717a6a8085c72b87 on master.

Coverage decreased (-17.7%) to 74.453% when pulling 6b3ae95bc05429373f968788c212cbdf2b2ea245 on robot_behaviour into 3817edb3d4513068da178794717a6a8085c72b87 on master.

I just got around to see this PR. I'd like to ask you to consider a slightly different approach to injecting default callback implementations in __using__. In particular it is to use optional callbacks and handle defaults appropriately in the behaviour module when the callback is not defined. This also probably means injecting almost no code in use besides the @behaviour declaration and the child_spec function. This is most probably how GenServer will work in Elixir 1.8 and how GenStage works already - it might be a good reference on how to go about doing this.

Is this something that's still being worked on ? If so wouldn't mind to lend a hand, just not sure how to get started.

@johanb thanks for your interest in helping!
I do intend to finish this PR someday soon 😄
I'll probably pick this back up next month.

This still seems useful to be worked on.

@scrogson what are you thinking on this PR and about Hedwig, generally, these days?
I'm considering putting in some grunt work in your lil' ecosystem (eg, wip, and potentially toss some PRs your way here in this repo); I don't want to be spinning wheels though if Hedwig itself is on its way to mothballed.

@defmhs hey there 👋
Unfortunately, my work in the last 5+ years has lead me away from working on chat related things, so I haven't had time/energy to maintain this project.
That said, if you're interested in helping push this project forward, feel free to send some PRs, I'll try my best to respond.


Contact Google and ask for a free domain

Request the vir-us.io domain from Google.
Let them know what we are doing and that we need the TLD.

@un7c0rn purchased the domain name.


Clarification of model

Hi Patrick,
Thanks for this code. The code itself is not 100% useable, for instance the paid version of timefold is used.
Further the student group entity is not entirely clear to me. What is the use of "year", "name", "student group" and "semi group" in this model?
Further there are a few code cleanups that could help, for instance removal of 'public' in interfaces, update of libraries and removal of deprecations, or introduction of a few enumerations.
I could take this up if you would like.

Hi Rutger,
Thank you for raising the issue.
This project is a prototype for generating a timetable for Polytechnic University of Bucharest. It was implemented for research/development reasons. Indeed, the paid version of Timefold is used, but you can use version 1.9.0 of Timefold to test its capabilities (multithreading solver included) and see how Timefold would behave in a complex scenario. Note that the paid version can be used only in development.
The model is designed at the moment only for the Faculty of Automatic Control and Computer Science which requires a complex model. The students are divided into smaller groups by year, series, groups and semigroups. For example, Student A. is in year 3 (THIRD), series AB, group 333 AB, semigroup B.
I will clarify all details about this project in the future. At the moment, this project is used for research only, so it doesn't need any modifications. I will let you know if it is necessary to improve/modify the codebase. Thank you for your understanding.
If you have any questions, please don't hesitate to ask away.


ActionBarActivity is deprecated

How can i replace it by AppCompatActivity?

Yeah!

The library will soon support AppCompatActivity


Lecture 13: Approximate Inference: Monte Carlo and Sequential Monte Carlo Methods

Wrapping up variational inference, and introduction to approximate inference.

@jeffreywpli @hlzhou The notes look great!
I pushed some changes: https://github.com/sailinglab/pgm-spring-2019/pull/15/commits/ba720c1dd7d43beee2a448490228bf5b1fd590df If the changes are OK with you, we can push the notes to master branch. Let me know.
Notes preview: http://www.helen-zhou.com/pgm-spring-2019/notes/lecture-13/

Looks good! Thank you Lisa! :)
---------- Forwarded message ---------
From: Lisa Lee<EMAIL_ADDRESS>Date: Sat, Mar 9, 2019 at 3:20 PM
Subject: Re: [sailinglab/pgm-spring-2019] Lecture 13: Approximate
Inference: Monte Carlo and Sequential Monte Carlo Methods (#15)
To: sailinglab/pgm-spring-2019<EMAIL_ADDRESS>Cc: Helen Zhou<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>@jeffreywpli https://github.com/jeffreywpli @hlzhou
https://github.com/hlzhou The notes look great!
I pushed some changes: ba720c1
https://github.com/sailinglab/pgm-spring-2019/commit/ba720c1dd7d43beee2a448490228bf5b1fd590df
If the changes are OK with you, we can push the notes to master branch. Let
me know.
Notes preview: http://www.helen-zhou.com/pgm-spring-2019/notes/lecture-13/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sailinglab/pgm-spring-2019/pull/15#issuecomment-471218490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGFYMN9BAmvVroLmOvOtBUFG6YTDC9Jgks5vVBc2gaJpZM4biXGo
.


Organizing fedora_to_rof translations

Adding a ROF::Translators module
@255a93ebdc046f3cfe1ff507571b10fe05699c49
In an effort to organize the junk drawer, I'm trying to name things and
find their place.
Moving CSV translator into Translators module
@3731d8f6bf36674ee69522ce18a6286a5c69b734
Renaming translator class to express intent
@427bbdf4409aeae982c513cbc8c767df6910f6b7
The class name now reflects that it is translating a CSV to ROF.
Extracting a convenience method
@ef60c61966905f4c29562f7ffeaa3aa5b333a7d6
Instead of deep reference to translator modules, I want to define an
interface in between.
Adding rudimentary FedoraToRof translator
@9709748e9a7a3e5817495fea275aa11791b3f719
Replacing puts with .puts
@d8e7ef352d5679aca679ca406a9dbbaa4537a0c1
Raw move of translation code into translator
@e3268fcd3e46a89626b627a89dc7ca888a3b714a
Adding delegation test to ensure wiring up
@9b3aec5d12920ae7702e1cc660e967d2eede6e40
Moving spec to more appropriate location
@a33c1dbbc012143464c3e8ff0e1029c59e300664
Refactoring to test outer looping function
@3fd5fc471e81809a2d1a9996eb5134b8d9731a7e
Extracting utility method for has_embargo_date?
@46ffa729ecb008177c37fc661468a76076741816
Extracting an instance method
@0113b78e9f100f754cedd7932e24524017cfc298
Prior to this change there could have been odd collisions on the module
instance variable for the @fedora_info. By extracting an instance, now
each call to convert Fedora objects to Pids will instantiate a new
object (and thus avoid the collision).
Extracting fedora connection
@38410f707c4ccaa90b1fbcdd2a4d03ec40c98125
Instead of connecting to Fedora once for each PID, allow for the
connection to persist during the lifecycle of the FedoraToRof.
Removing parameter in favor of instance variable
@3ea94962410290e5e1c3802d623676047224349d
Extracting class to process pid
@46954475d59f4d9b84d98711e67080cf8e47b1bf
With the reliance on an instance variable that changes as we loop
through the PID, I want to ensure that we don't have collisions. It is
also a code smell, indicating that there should probably be another
class.
Extracting default_datastream_conversion method
@79df8bceb745772e2b227df7335446aeeaa1ee2f
Pushing concepts and ideas to similar levels of abstraction
Extracting explicit method mapping for Fedora
@b122cb5f0700e2d7523f606d93cd50739d6255ed
Organizing spec
@d1739db65a0983a9d4b823628956be5e4981e1b3
Renaming variable for clarity
@71f172d48f7b56c3ecd6661e1dcd8886d9a25ac2
Adding expectation concerning failed connection
@3399621406f607ee0cf2c1b6a31c73518b60b761
Pushing exit up to CLI instead of within file
@ddfbcff3d8b10f1f01a36f485b99ef04ad687bc4
The exit! within a non-CLI context makes testing rather hard.
Removing unused method and bumping coverage :)
@9f912db79365c7e6b2b38debb677e3125b5b16d1

@jeremyf it looks like the .run methods in the test cases need to be changed as well.

Just out of curiosity, is
.fetch("something") { :token }

preferable to
.fetch("something", :token)

@dbrower It is a matter of speed. However, for non-expensive operations, there's minimal difference. When #fetch is passed a block, the block is only evaluated if there is a miss on the key.


"cupid-clear" Command Implementation Addressing Issue #85

This pull request addresses issue #85 and contains the following additions:

A new clear.py script located at "cupid/clear.py"
cupid-clear = "cupid.clear:clear" added to the last line of the pyproject.toml file

The clear.py script can be run with the following steps:

After adding these changes to your local environment, type "pip install -e ." in your terminal

Navigate to the directory where the config.yml file is located (this config.file contains the run_dir variable which is the path to the computed_notebooks folder you want to clear). For example: "cd examples/coupled_model"

Run the command "cupid-clear config.yml" and this should clear the computed_notebooks folder at the path specified by the run_dir variable in your config.yml file (if you're running cupid-clear from the examples/coupled_model directory and the config.yml located here has run_dir= . then the folder at the path examples/coupled_model/computed_notebooks will be cleared, if it exists)

Also, just to be clear, I don't think any of my aforementioned environment issues are related to this PR since there were no changes to the environment yaml files-- I just wanted to be sure to note that and check to make sure we don't have other issues going on :)

Thank you for your feedback. I've modified clear.py to use the pre-exisiting get_control_dict function, then to read in control['data_sources']['run_dir']. I also added instructions about clear.py to the README.md. Please let me know if this works correctly.

Great! Thanks @shivaniikum . I'll test that again once our machines are back up!

Ok, this is looking great! Thanks for making those changes.
The environment issues were specific to my machine and how I had set things up-- I've also tested on derecho (an NCAR machine) and everything looks good with the environment setup now.
The new method of finding the configuration file run_dir seems to be working smoothly.
I'll approve this PR, but I think @rmshkv wanted to test it out as well before we merge it in.

That sounds awesome! Thank you!

Thank you for reviewing my PR  @mnlevy1981. I've modified the clear.py script and README.md as you suggested (and also removed redundant exceptions for the ones I noticed python reports). Please let me know how this looks.

Thank you for reviewing my PR @mnlevy1981. I've modified the clear.py script and README.md as you suggested (and also removed redundant exceptions for the ones I noticed python reports). Please let me know how this looks.

Thank you for reviewing my PR @mnlevy1981. I've modified the clear.py script and README.md as you suggested (and also removed redundant exceptions for the ones I noticed python reports). Please let me know how this looks.

Thanks for making those changes! Could you add the try / except block that you removed from readConfigFile() to get_control_dict()?

Thank you for reviewing my PR @mnlevy1981. I've modified the clear.py script and README.md as you suggested (and also removed redundant exceptions for the ones I noticed python reports). Please let me know how this looks.

Thanks for making those changes! Could you add the try / except block that you removed from readConfigFile() to get_control_dict()?

Sure, I just made that change.

I do see the benefit of adding that try/except block to get_control_dict() to clarify the file not found error and avoid clutter, since this error may come up somewhat often.
Regarding the run_dir error checking, I don't have a strong preference between using a sys.exit and raising a ValueError. It would reduce clutter to use sys.exit, but I think it's still fairly clear that there's a file not found error. My only other concern with skipping the tracebacks would be that if errors occur for unexpected reasons, it could be harder to debug. So, either way is fine by me.

Thanks for clarifying. I just added the changes you suggested. I agree that these changes make the errors easier to understand.

I agree that these changes make the errors easier to understand. I've modified util.py and clear.py as suggested by @mnlevy1981.

I think I'd vote for the ValueError rather than sys.exit, looks a little less clean but might be better to keep things consistent.

That's awesome! Thank you so much for reviewing my PR and for all the suggestions @mnlevy1981 @TeaganKing @rmshkv!

Great work @shivaniikum ! Thank you for contributing this useful feature to the repository!


asyncio: implementation

[x] POC
[x] Non-blocking, remove chan and goroutine

[x] Parallel version

[x] Scheduling last frame instead of scheduling ROOT task for performance
[ ] Compiling async functions

[x] generator
[ ] promise
[ ] compitable recover block

[ ] Support event IO

Use async branch to polish it: https://github.com/goplus/llgo/tree/async


Allow Kitura-Net to only listen on one network address

Description
@billabt was nice enough to add support for this at the BlueSocket layer here. However, I really want to use that functionality to fix https://github.com/IBM-Swift/Kitura/issues/1450.
I also have code ready to go for Kitura-NIO and Kitura.
Motivation and Context
This is necessary to close https://github.com/IBM-Swift/Kitura/issues/145. Should I open another issue for Kitura-Net?
How Has This Been Tested?
Manual testing by myself. I also modified the test harness to use the new code path.
Checklist:

[x] I have submitted a CLA form
[x] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Tabor Kelly seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

As for naming, I am happy to change node to address. I used node to match BlueSocket which in turn matches getaddrinfo.

@TaborKelly I tried out @pushkarnk's suggestion with the protocol extension here, and it seems to work:
https://github.com/djones6/Kitura-net/commit/46a097c8ed8bca8f6bb63be5d419aba342e6c980
Here's a quick hack up of the corresponding Kitura change to surface the new option:
https://github.com/djones6/Kitura/commit/a5539cf103ea1e3047b23d64b633160b091b4c4f

@djones6, @pushkarnk,
Thanks of the help. djones, I used your code almost verbatim. I only fixed up some comments. I think the changed (stripped) whitespace was in your version, and I just copied it. I'm fine with it if you are, but I am also willing to re-add trailing whitespace depending on what you folks want.

I think that I changed all the documentation comments to match your suggested wording. If we wanted to we could change the word interfaces to addresses as it would be more correct. Well, at least for Kitura-Net. I suspect the Kitura-NIO is only listening on IPv4 addresses, but I haven't verified it.


About the experimental settings for ViT architecture.

Thanks for your nice work!
I am interested in the superior performance of CLIP-FSAR based on ViT network, but its script seems not to be given in paper or code. Would you please supplement the settings? BTW, I attempt to reproduce it by changing the BASE_LR to 1e-5 and keeping others as in R50, but obtained ~1% 1-shot accuracy gap (SSV2-Full: 60.0% reproduce vs 61.2% in paper).
Thanks.

fix the typo:
SSV2-Full:  paper 61.9 vs. my reproduce 60.0, with gap 1.9%


Potential security issue in src/protocol/survey0/survey.c: Unchecked return from initialization function

 What is a Conditionally Uninitialized Variable?  The return value of a function that is potentially used to initialize a local variable is not checked. Therefore, reading the local variable may result in undefined behavior.
1 instance of this defect were found in the following locations:
Instance 1
File : src/protocol/survey0/survey.c
Function: nni_list_remove
https://github.com/siva-msft/nng/blob/ac1323710b17fb4dfe353e53424cd4ddadbda4b6/src/protocol/survey0/survey.c#L485
Code extract:
	    (nni_lmq_full(&ctx->recv_lmq))) {
		nni_msg_free(msg);
	} else if ((aio = nni_list_first(&ctx->recv_queue)) != NULL) {
		nni_list_remove(&ctx->recv_queue, aio); <------ HERE
		nni_aio_finish_msg(aio, msg);
	} else {

Will be filtered out as return type name is void


How to handle WebSocket messages inside an User

Hi!
I am migrating my java socket server to a web socket server with your library.
Previously I had a class User that handled the user messages because it was a thread always waiting on inputStream.read() it processed the message there, and depending on the message type (a kind of protocol I created), it would perform some operations in the object, like a login (saving that info in the User object), setting a status, requesting database data, or dispatch the message to a main system which would deal with the that in case the message couldn't be handle by the user (like sending a private message to other User).
While implementing this library, I faced the problem that I can't have a User, because all I got on onOpen is a WebSocket connection, with no information from the user. If I create a user with that connection User(WebSocket connection), I would need to listen to an onMessage() event. To process the message there.
Is this possible?
Or should I opt for another approach?
I thought about creating an HashMap<WebSocket, User> conns, so when I receive an onMessage(WebSocket conn, String message) I could call and onMessage() inside my User like: conns.get(conn).onMessage(String message) and then handle the message inside that User object. But, isn't this HashMap a too heavy?
Can someone please guide me on the best approach?
Thank you!

Hello @promatik,
Right now it is not possible to assign a WebSocket additional values.
There is however a an open discussion on such a new feature in issue #485
So right now you have to use a map for assigning additional values to a websocket.
Gretings
marci4

Thank you @marci4, I'll close this issue and continue there.


Why is there a path dependency?

Why is there a dependency on the https://www.npmjs.com/package/path package? Normal require('path') will provide the internal module AFAICT, which means the package is installed but never used. It didn't seem to be a dependency in standard (given https://github.com/feross/standard/pull/187 where it switched to deglob) so perhaps this was by accident?
With npm@3  it's being installed as a top-level dependency, and another package which uses Node's private module loading methods ends up loading the fake path. The version requested doesn't work correctly under Node 5 (but it'd always be out of date anyway).

Good catch! This should be removed!

Fixed and released as 1.0.2.


rustup --help does not show "install" subcommand.

rustup --version
rustup 1.5.0 (92d0d1e9e 2017-06-24)

rustup --help

rustup 1.5.0 (92d0d1e9e 2017-06-24)
The Rust toolchain installer

USAGE:
    rustup [FLAGS] [SUBCOMMAND]

FLAGS:
    -v, --verbose    Enable verbose output
    -h, --help       Prints help information
    -V, --version    Prints version information

SUBCOMMANDS:
    show           Show the active and installed toolchains
    update         Update Rust toolchains and rustup
    default        Set the default toolchain
    toolchain      Modify or query the installed toolchains
    target         Modify a toolchain's supported targets
    component      Modify a toolchain's installed components
    override       Modify directory toolchain overrides
    run            Run a command with an environment configured for a given toolchain
    which          Display which binary will be run for a given command
    doc            Open the documentation for the current toolchain
    man            View the man page for a given command
    self           Modify the rustup installation
    set            Alter rustup settings
    completions    Generate completion scripts for your shell
    help           Prints this message or the help of the given subcommand(s)

DISCUSSION:
    rustup installs The Rust Programming Language from the official
    release channels, enabling you to easily switch between stable,
    beta, and nightly compilers and keep them updated. It makes
    cross-compiling simpler with binary builds of the standard library
    for common platforms.

    If you are new to Rust consider running `rustup doc --book` to
    learn Rust.

Hi @jerry-sl
install is an alias for toolchain install. That's why it's hidden.
https://github.com/rust-lang-nursery/rustup.rs/blob/ac5bffad7ba126b2313692c3dfce83ec91cb7fea/src/rustup-cli/rustup_mode.rs#L121

install is an alias for toolchain install. That's why it's hidden.

Would it be worth having it in the main menu anyway, given that it's (at least for me) the first command that gets run after installing (rustup install nightly)?
I was confused by this too.

It is confusing because the toolchain subcommand's description is

Modify or query the installed toolchains

I didn't notice that I can deal with not installed toolchains too.

Would be happy to accept a PR to improve the description of the toolchain subcommand.


Using server paging on a function causes problem

Hi,
It seems like when we use paging on something that is a function it stops working when it tries to run the tne next link. The code looks like this:
        public void GenerateFullFile()
        {
            var skuTypeIdToExclude = SkuStatusType.WaitingForErp.SkuTypeId;
            var skuStatusIdToExclude = SkuStatus.Inactive.StatusId;
            var annotations = new ODataFeedAnnotations();
            var items = oDataClient.For<ClientProductSku>()
                .Function(FunctionNameExportProductFull)
                .Set(new { clientId, skuTypeIdToExclude, skuStatusIdToExclude })
                .Expand(ExportProductExpandTables)
                .FindEntriesAsync(annotations)
                .Result;

            using (var generator = new ExportProductStreamingXmlFileGenerator(FileType.Full))
            {
                generator.GenerateStartXmlDocument();
                WriteItemsToFile(items, generator);
                while (annotations.NextPageLink != null)
                {
                    items = oDataClient.For<ClientProductSku>()
                        .Function(FunctionNameExportProductFull)
                        .Set(new { clientId, skuTypeIdToExclude, skuStatusIdToExclude })
                        .Expand(ExportProductExpandTables)
                        .FindEntriesAsync(annotations.NextPageLink, annotations)
                        .Result;
                    WriteItemsToFile(items, generator);
                }
                generator.GenerateEndXmlDocument();
            }

So the first query works but the second (nextlink) crashes. It says entity type could not be found. I tried to run the annotations.NextPageLink uri in a browser and that worked. The error message we get is
System.AggregateException occurred
  HResult=-2146233088
  Message=One or more errors occurred.
  Source=mscorlib
  StackTrace:
       at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
       at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
       at System.Threading.Tasks.Task`1.get_Result()
       at Enferno.Services.Adlibris.Export.ExportProduct.ExportProductService.GenerateFullFile() in d:\tfs\eCom\Services\Adlibris\Release\1.1\Enferno.Services.Adlibris\Export\ExportProduct\ExportProductService.cs:line 45
       at Enferno.Services.Adlibris.AdlibrisService.ExportProductFull() in d:\tfs\eCom\Services\Adlibris\Release\1.1\Enferno.Services.Adlibris\AdlibrisService.cs:line 63
  InnerException: System.AggregateException
       HResult=-2146233088
       Message=One or more errors occurred.
       Source=mscorlib
       StackTrace:
            at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
            at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
            at System.Threading.Tasks.Task`1.get_Result()
            at Simple.OData.Client.FluentClientBase`1.<RectifyColumnSelectionAsync>b__1(Task`1 y)
            at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
            at System.Threading.Tasks.Task.Execute()
         --- End of stack trace from previous location where exception was thrown ---
            at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
            at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
            at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
            at Simple.OData.Client.BoundClient`1.<FindEntriesAsync>d__6.MoveNext()
       InnerException: System.AggregateException
            HResult=-2146233088
            Message=One or more errors occurred.
            Source=mscorlib
            StackTrace:
                 at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
                 at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
                 at System.Threading.Tasks.Task`1.get_Result()
                 at Simple.OData.Client.FluentClientBase`1.<>c__DisplayClass4.<RectifyColumnSelectionAsync>b__3(Task`1 x)
                 at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
                 at System.Threading.Tasks.Task.Execute()
            InnerException: Simple.OData.Client.UnresolvableObjectException
                 HResult=-2146233088
                 Message=Entity type FunctionService.GetExportProductFull(clientId=35,skuTypeIdToExclude=14,skuStatusIdToExclude=5) not found
                 Source=Simple.OData.Client.V4.Adapter
                 ObjectName=FunctionService.GetExportProductFull(clientId=35,skuTypeIdToExclude=14,skuStatusIdToExclude=5)
                 StackTrace:
                      at Simple.OData.Client.V4.Adapter.Metadata.GetEntityType(String collectionName)
                      at Simple.OData.Client.V4.Adapter.Metadata.GetEntityCollectionTypeNamespace(String collectionName)
                      at Simple.OData.Client.MetadataBase.GetEntityCollectionQualifiedTypeName(String collectionName)
                      at Simple.OData.Client.V4.Adapter.RequestWriter.<WriteEntryContentAsync>d__8.MoveNext()
                   --- End of stack trace from previous location where exception was thrown ---
                      at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
                      at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
                      at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
                      at Simple.OData.Client.RequestWriterBase.<CreateGetRequestAsync>d__1.MoveNext()
                   --- End of stack trace from previous location where exception was thrown ---
                      at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
                      at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
                      at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
                      at Simple.OData.Client.ODataClient.<FindEntriesAsync>d__31.MoveNext()
                 InnerException:

Hi,
Can you try the following syntax within the while loop:
items = oDataClient.For≶ClientProductSku≫()
.FindEntriesAsync(annotations.NextPageLink, annotations)
.Result;
When using annotations it's sufficient to pass just NextPageLink information, so I wonder if the additional clauses may cause the problem (which is a bug anyway).

Hi Vagif,
Unfortunately it didn't work. We are experiencing the same issue.

OK, will investigate it. I suppose the service has the same metadata as you mailed me.

I've made some changes in 4.5.4 that probably didn't eliminate the problem but might change the call stack if the error occurs. Could you rerun the test in case of the error provide the following information:

Call stack, similar to what you sent earlier.
In case the error is sent after the HTTP request is issued, the HTTP request details (URI, content).

Thanks.

Hi Vagif,
I just tried the function calls and it seems to be working now!?

Good news, I did some changes but wasn't quite sure if it was sufficient. If I don't hear anything from within a few days I will close the issue.

OK. Cool. Thanks for your assistance


App templates problem

Bug description
From the portainer dashboard clicking on "App templates" leads to this error:

Failure
json: cannot unmarshal string into Go struct field Template.type of type portainer.TemplateType

Expected behavior
I would expect to see a list of app templates
Technical details:

Portainer version: 1.22.0
Docker version (managed by Portainer): 19.03.2
Platform (linux):
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): using my own docker-compose.yml file, available if really needed
Browser: Chrome

Hi there,
The error you specify is related to the type parameter of the templates.json file not being defined as a integer, are you using external templates by chance?
FYI there are two exisitng issues around templates, I believe that some of the comments may be helpful. You can view these here: #2283 #2124

I don't use templates and never used them. This is a pretty new installation of portainer, I simply clicked on the "App templates" from the dashboard...
I was just reporting the error. If I can supply more info let me know, otherwise simply close this issue. I read the issues you linked and following this advice solved the issue:

Go to settings, and disable “use external templates”…


v4l2 output options

Is there any way to leverage the impressive performance benefits with a v4l2loopback output? I see it's not built in (yet?) like wf-recorder has, but even past that, any way to pipe to ffmpeg or such? Any plans?

Yeah, that's possible. It might work by using the file output and format options to feed raw video into v4l2 loopback. It would likely introduce a buffer copy which would negate some of the performance wins of this program. I'll give that a test sometime soon.
If v4l2loopback supports feeding h264 into it (I know v4l2 does, just unsure about v4l2loopback), then this could be quite efficient assuming the program that consumes it doesn't re encode it.
I assume you want this for zoom or something?

After 4b8fc4e59b93e434ec1442a43026cd5484138a92 and b3f9945805e9d8a3d9929eac3428810a30893cf8 this seems to be working well for me. Tested with
sudo modprobe v4l2loopback
wl-screenrec --muxer v4l2 -f /dev/video6

with https://mozilla.github.io/webrtc-landing/gum_test.html

You may also want to pass --encode-pixfmt=bgra if you want it to not be chroma subsampled

Close. so close... so I am trying to replicate "wf-recorder -g 5135,55 1920x1080 -f /dev/video10 -m v4l2 -c mjpeg -x yuv420p" I think that mpjeg is mandatory for zoom (despite my actual camera support h264 and getting my hopes up...) but I can't get mjpeg enabled in wl-screenrec.
$ wl-screenrec --ffmpeg-muxer v4l2 -f /dev/video10 -g "0,55 1920x1080" --encode-pixfmt yuv420p --ffmpeg-encoder mjpeg
Using output DP-1
failed to create encoder: Format video4linux2,v4l2 does not support MJPEG codec

or
$ wl-screenrec --ffmpeg-muxer v4l2 -f /dev/video10 -g "5135,55 1920x1080" --codec mjpeg
error: invalid value 'mjpeg' for '--codec <CODEC>'
  [possible values: auto, avc, hevc, vp8, vp9]

I also have this running when the screen recorder is not, and it dishes out the mjpeg source just lovely.
ffmpeg -stream_loop -1 -re -i testcard.mp4 -f v4l2 -vcodec copy /dev/video10```

Please disregard queries here, everything seems great now, potentially as your app is SO much more efficient than others that distortions happening previously with rawvideo aren't happening with wl-screenrec at all.

@acidkewpie Could you please share what was the issue and the final command you used?

I added an example that works for me with Zoom to the README

Yes, it was that simple for me in the end, bar a -g option. Wrapped it up in a systemd service and it's doing great now.

So I have problems running version from git installed using cargo:
wl-screenrec --ffmpeg-muxer v4l2 -f /dev/video8
Using output DP-3
hw flag is specified, but there's no known vaapi codec for RAWVIDEO. Using a generic encoder...
codec "rawvideo" does not advertize supported pixel formats, just using NV12. Pass --encode-pixfmt to suppress this warning
Opening libva device from DRM device /dev/dri/renderD128
[video4linux2,v4l2 @ 0x558ad6fd9a00] ioctl(VIDIOC_G_FMT): Invalid argument
thread 'main' panicked at src/main.rs:1352:29:
called `Result::unwrap()` on an `Err` value: ffmpeg::Error(22: Invalid argument)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I noticed the error about pixel format so added it:
wl-screenrec --ffmpeg-muxer v4l2 -f /dev/video8 --encode-pixfmt yuv420p
sing output DP-3
hw flag is specified, but there's no known vaapi codec for RAWVIDEO. Using a generic encoder...
Opening libva device from DRM device /dev/dri/renderD128
[Parsed_scale_vaapi_1 @ 0x55a4bd42afc0] Hardware does not support output format yuv420p.
[Parsed_scale_vaapi_1 @ 0x55a4bd42afc0] Failed to configure output pad on Parsed_scale_vaapi_1
thread 'main' panicked at src/main.rs:1648:18:
called `Result::unwrap()` on an `Err` value: ffmpeg::Error(22: Invalid argument)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Now there is message about no support from hardware so I also disabled it:
wl-screenrec --ffmpeg-muxer v4l2 -f /dev/video8 --encode-pixfmt yuv420p
Using output DP-3
Opening libva device from DRM device /dev/dri/renderD128
[Parsed_scale_vaapi_1 @ 0x55d27da9ec80] Hardware does not support output format yuv420p.
[Parsed_scale_vaapi_1 @ 0x55d27da9ec80] Failed to configure output pad on Parsed_scale_vaapi_1
thread 'main' panicked at src/main.rs:1648:18:
called `Result::unwrap()` on an `Err` value: ffmpeg::Error(22: Invalid argument)
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Thank you for your time and help getting this far, I really appreciate it.


Allow to skip docker_py and to modify PIP packages

Allow to skip docker_py
allow to modify PIP packages
updating cache is always reported as a change

Hi @maqdev. I really like the skip_docker_py idea.
But for which use-cases do you want to modify the installed PIP packages?

Hi! Actually I don't want to install python-pip on target machine. In my case it depends on many other packages. So I just put python_pip_packages: [] into into my playbook.
It seems that pip is only needed for docker-py?

Hi! pip is needed also needed by the Install the Python SNI support packages. task in main.yml to make the apt_key module work.
Btw, I like your idea of making pip (and docker-py) optional.
Would you mind if i merge your PR in another branch (not in master) to resolve the conflicts and expand a little bit your idea?
In case, feel free to open an issue so that I can ask you to review my work before merging it into master.

Sure! You're free to make whatever you want with this PR :-)

Ehi @maqdev, your PR is now merged into master (see 38739a046861e1a83458a8f9150b209a1f119a35).
Thanks for your contribution!


Support external OCR sources

Use https://github.com/hoover/snoop/blob/master/snoop/ocr.py for reference.
Also implement the management commands needed to manage external OCR sources

closed by https://github.com/hoover/snoop2/pull/36


can not install 5.0.1 specifically

Steps to Reproduce:

$ logstash-plugin install --version 5.0.1 logstash-output-influxdb
Validating logstash-output-influxdb-5.0.1
Plugin logstash-output-influxdb version 5.0.1 does not exist
ERROR: Installation aborted, verification failed for logstash-output-influxdb 5.0.1

Works for me:
~/Downloads/logstash-5.4.3
❯ bin/logstash-plugin install --version 5.0.1 logstash-output-influxdb
Validating logstash-output-influxdb-5.0.1
Installing logstash-output-influxdb
Installation successful

Can you try installing without the version?

@inetfuture which logstash version are you using?

5.5.1

Hey, I can not install influxdb 5.0.1.
I am using logstash version 5.5.0 running on windows 7 64-bit.
I tried a lot of installation methods, but I can not get it working.
I tried:

Using prepare-offline-pack
Installing with ".zip" file
Installing normally
Installing with ".gem" file

Using prepare-offline-pack
C:\ELK-Stack\dev\logstash-5.5.0>bin\logstash-plugin.bat prepare-offline-pack logstash-output-influxdb
RETURNS
Bundler::VersionConflict: Bundler could not find compatible versions for gem "influxdb":
  In Gemfile:
    logstash-output-influxdb (= 5.0.1) java depends on influxdb (<= 0.3.99, >= 0.3) java
Could not find gem 'influxdb (<= 0.3.99, >= 0.3) java', which is required by gem 'logstash-output-influxdb (= 5.0.1) java', in any of the sources.
            start at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/resolver.rb:203
          resolve at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/resolver.rb:182
          resolve at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/definition.rb:192
            specs at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/definition.rb:132
        specs_for at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/definition.rb:177
  requested_specs at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/definition.rb:166
  requested_specs at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/environment.rb:18
            setup at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler/runtime.rb:13
            setup at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/bundler-1.9.10/lib/bundler.rb:122
           setup! at C:/ELK-Stack/dev/logstash-5.5.0/lib/bootstrap/bundler.rb:67
          execute at C:/ELK-Stack/dev/logstash-5.5.0/lib/pluginmanager/prepare_offline_pack.rb:14
              run at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/clamp-0.6.5/lib/clamp/command.rb:67
          execute at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/clamp-0.6.5/lib/clamp/subcommand/execution.rb:11
              run at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/clamp-0.6.5/lib/clamp/command.rb:67
              run at C:/ELK-Stack/dev/logstash-5.5.0/vendor/bundle/jruby/1.9/gems/clamp-0.6.5/lib/clamp/command.rb:132
           (root) at C:\ELK-Stack\dev\logstash-5.5.0\lib\pluginmanager\main.rb:48

Installing with ".zip" file
C:\ELK-Stack\dev\logstash-5.5.0>bin\logstash-plugin.bat install file:///C:/ELK-Stack/dev/data/plugins/logstash-output-influxdb-5.0.1.zip
RETURNS
ERROR: An error occured when installing the: file:///C:/ELK-Stack/dev/data/plugins/logstash-output-influxdb-5.0.1.zip, to have more information about the error add a DEBUG=1 before running the command., message: Bundler could not find compatible versions for gem "influxdb":
  In Gemfile:
    logstash-output-influxdb (= 5.0.1) java depends on
      influxdb (<= 0.3.99, >= 0.3) java
Could not find gem 'influxdb (<= 0.3.99, >= 0.3) java', which is required by gem 'logstash-output-influxdb (= 5.0.1) java', in any of the sources.

Installing normally
C:\ELK-Stack\dev\logstash-5.5.0>bin\logstash-plugin.bat install logstash-output-
influxdb

RETURNS
Validating logstash-output-influxdb
Unable to download data from https://rubygems.org - SocketError: initialize: nam
e or service not known (https://rubygems.org/latest_specs.4.8.gz)
ERROR: Installation aborted, verification failed for logstash-output-influxdb

Installing with ".gem" file
C:\ELK-Stack\dev\logstash-5.5.0>bin\logstash-plugin.bat install C:/ELK-Stack/dev
/data/plugins/logstash-output-influxdb-5.0.1.gem

RETURNS
Validating C:/ELK-Stack/dev/data/plugins/logstash-output-influxdb-5.0.1.gem
Installing logstash-output-influxdb
Error Bundler::HTTPError, retrying 1/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 2/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 3/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 4/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 5/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 6/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 7/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 8/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 9/10
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 10/10
Could not fetch specs from https://rubygems.org/
Too many retries, aborting, caused by Bundler::HTTPError
ERROR: Installation Aborted, message: Could not fetch specs from https://rubygems.org/

Hi, I have the same problem, I can't install logstash-output-plugin 5.0.4.
I'm runing logstash version 6.3.1 on CentOS 7 64-bit
Java version (java -version):
openjdk version "1.8.0_161"
OpenJDK Runtime Environment (build 1.8.0_161-b14)
OpenJDK 64-Bit Server VM (build 25.161-b14, mixed mode)

i'm behind a proxy so I did:
export http_proxy=http://myproxy:myport
export https_proxy=http://myproxy:myport
export HTTP_PROXY=http://myproxy:myport
export HTTPS_PROXY=http://myproxy:myport

From gem:

$ sudo bin/logstash-plugin install --local --no-verify  /path/to/gem/logstash-output-influxdb-5.0.4.gem

Installing logstash-output-influxdb
Plugin version conflict, aborting
ERROR: Installation Aborted, message: Bundler could not find compatible versions for gem "influxdb":
  In Gemfile:
    logstash-output-influxdb (= 5.0.4) java depends on
      influxdb (<= 0.3.99, >= 0.3) java
Could not find gem 'influxdb (<= 0.3.99, >= 0.3) java', which is required by gem 'logstash-output-influxdb (= 5.0.4) java', in any of the sources.

Normal way:

$ sudo bin/logstash-plugin install logstash-output-influxdb

Validating logstash-output-influxdb
Unable to download data from https://rubygems.org - Errno::ECONNREFUSED: Connection refused - Failed to open TCP connection to api.rubygems.org:443 (Connection refused - connect(2) for "api.rubygems.org" port 443) (https://api.rubygems.org/latest_specs.4.8.gz)
ERROR: Installation aborted, verification failed for logstash-output-influxdb

With no-verify flag:

$ sudo bin/logstash-plugin install --no-verify logstash-output-influxdb

Installing logstash-output-influxdb
Error Bundler::HTTPError, retrying 1/10
Could not fetch specs from https://rubygems.org/
.
.
.
Could not fetch specs from https://rubygems.org/
Error Bundler::HTTPError, retrying 10/10
Could not fetch specs from https://rubygems.org/
Too many retries, aborting, caused by Bundler::HTTPError
ERROR: Installation Aborted, message: Could not fetch specs from https://rubygems.org/

I've found a workaround! :)
First, I've created a "handmade" logstash-offline-plugin.zip with the dependencies:
cd /my/path
mkdir logstash
cd logstash
wget https://rubygems.org/downloads/logstash-output-influxdb-5.0.4.gem
mkdir dependencies
cd /my/path/logstash/dependencies
wget https://rubygems.org/downloads/influxdb-0.3.17.gem
wget https://rubygems.org/downloads/json-1.8.6-java.gem
wget https://rubygems.org/downloads/stud-0.0.23.gem
cd /my/path
zip -r logstash-offline-plugins-handmade.zip logstash

Then, according to Logstash plugins documentation:
cd /path/to/logstash
sudo bin/logstash-plugin install file:///my/path/logstash-offline-plugins-handmade.zip


S2U-21 Tests & Quizzes: Add seb resource collection and set display name property

Add new resource collection under /public for published seb configs
Add display name property to resources, both uploaded seb config (draft, in /attachment/site/Assessment/) and copied published version (/public/seb/)

https://sakaiproject.atlassian.net/browse/S2U-21

Great job Markus!

Thanks :)


Continue?

This repo will be continued? Because I see many issues, PR, missing icons and the last commit has in January. I would like to know it

Joining the question.

@adamdbradley

bump bump!

Hey all,
We are 100% continuing support for ionicons. We've just been really really busy working on the next version of the Framework. Once that is in Final, we plan to spend time updating and going through Ionicons issues as well as updating the site.
Thanks for your patience and stay tuned!

+1

@John-Luke changes are continuing on the 3.0 branch.

Wow! Thanks! :D

Ionicons are great.  It's somewhat of a shame that they need to wait for the new framework. I'm sure many people are using them detached from the framework. FontAwesome just landed a spectacular crowd funding campaign, would that perhaps be an option for Ionicons as well?

@mickeyl There are actually a lot of icons available right now. Ionic will probably not be taking the route of a Kickstarter since they are venture-backed. Since Ben is the primary designer (CDO), he has been super busy working on designing Ionic Framework 2. As he said earlier, the team will be working on Ionicons pretty soon now that 2.0.0 final is out.

I love ionicons. It's great!
I use it almost on every projects. Really wish it's continue developed @bensperry

Please update 😀

Sorry this repo hasn't received a lot of attention, we've been focused on Ionic itself and our latest project Stencil. The great news is that ionic will be ported to become a stencil component, and all of the icons will be svgs rather than font icons. Additionally, with the font icon the user needed to download all of them, as a stencil component and refactoring, users will only have to download the exact icons which the page is using. With this new structure, it'll allow us to add even more icons without worrying about bloating filesize further, and slowing down apps. We've got a great working example already and hope to add this to Ionicons 4.0 soon. Please stay tuned! This repo hasn't been forgotten about.

Sounds good, but please add the SVGs in addition, not as a replacement for the fonts. Icon fonts are very useful in particular with native apps on mobile platforms.

@mickeyl Yup agreed. We're not removing the icon fonts at all, and the web components are an addition like you said. More info is here: https://github.com/ionic-team/ionicons/issues/499


Fixed #30040 -- Used default permission name in docs examples to avoid confusion.

Fixes ticket 30040.
Continue to PR.
Changed previous pr polls app related permission 'polls.add_vote' to 'polls.add_choice' based on @claudep comment.
The remain add_vote permissions don't related to polls app and I didn't change them.

Hi @carltongibson.
Thanks for the review.
TBH, I am not very good at documenting :).
Just want to complete the PR.
Feel free to request for changes.

Hey @hramezani. Thanks again for effort!


Initial 16.3 release notes

Signed-off-by: Tim Smith<EMAIL_ADDRESS>

#10185 was done to fix flaky CI failures, but I believe it was a thread safety bug and if users have used that code like how we did in the tests someone has likely run into it.

@phiggins I added a vague reference to that fix.


Re-Work Member Listings

Rework the public Members page to use the newly structured Users Database

Rework the EBoard Members panel to include a section for Non-Member users management.

Done


Issues with screen orientation

OS target (Android/iOS): Android
Godot version: 2.1.4 / 2.1.5
Issue description: Is there a way to rotate the way ads are displayed? My app was build for landscape, which Godot defaults to, however it's actually meant to be played as portrait, which means that banners are displayed at the right corner, which is the screen's landscape "top", interstitials are further cropped, pixelated and off screen boundaries. This is also how the provided sample projects are set up.
Is there a way to fake the screen orientation, as in having an API function that does so? Or having an API function that fakes where the top or bottom of the screen is?

Why can't you set your app as portrait?

It's a board game, and makes heavy use of screen dimensions to calculate positions and scales. Simply setting it to portrait deranges all the icons in the screen.
I've been trying to port it however, rotating the menus was easy, everything else that is programmatically and mathematically generated, positioned and manipulated is a headache.
This is very much my first app, and in all honesty i thought Godot's default landscape was the right way to go for Android. Never came to my mind 6 months later at the final step of adding ads would prove such decision to have been such a colossal mistake.

I'm sorry but I'm afraid I'm not understanding. I have published games using both orientations and I had no issue with this module.

I understand, i did not know of these concepts prior to making the game, this is indeed a beginner's mistake.
When the game is exported landscaped the x axis is the top of the screen, so a banner set with isTop shows on the right side of the screen. When the game is exported as portrait banners are displayed correctly, intertitials and videos so too, however menus are pinched and squeezed (scaled) to the screen as they suffer a 90 degrees rotation.
If you don't mind me kindly asking this here if there's any acceptable workaround this (e.g. an option to rotate the viewport), since this is no longer an issue related to this admob library but rather a mistake of mine.

It's really sad :(
I don't know if there is a easy workaround, but your best shot is to ask in one of the official Godot Engine communication channels, maybe someone with more knowledge about the subject can help you.

Alrighty! Thank you and also to kodler-games for releasing this addon to the public !


Lift common armv7 instructions vcvt.f64.s32 and vmov.f64

What is the feature you'd like to have?
Lift the following instructions vcvt.f64.s32 and vmov.f64
struct_array_test.zip

Could we also include vcmp.f64.F64, vcmpe.f64.F64, vdiv.f64.F64, vmul.f64.F64, vneg.f64.F64, vcvt.f64.F64.U32, vldr, vmrs, vpop, vpush, and vstr in the list? It's bothersome to see "unimplemented" in the linear view.

I will review those additional instructions when adding the others. Note that only scalar operations will be lifted to LLIL instructions. Any vector operations will probably be lifted as intrinsics, if at all. (It looks at a glance like most of the ones you are requesting are, in fact, scalar.)


Examples. Describe the Tangnano4k clock pin.

The point is that the external crystal on this board is soldered to a pin, which is a PLL input, not a clock pin. Therefore, common routing is used.
By specifying that this is a buffered network, we are forcing the router to use global clock wires.

The fact that the network has become buffered may have a very minor effect in cases where you really need to use this pin as an input for a PLL.
For now we can simply remove CLOCK_LOC "clk" BUFG, but in the future we will need to make a small correction to our global router to catch PLL input pins.

Also runber, let's wait https://github.com/YosysHQ/nextpnr/pull/1286


[Messeging] Fixing issues with Controls on initialization and shutdown

Ensuring the proper initialization of statics in each transition unit on Thunder launch
Calling Destroy() on each announced Control of every message type to ensure a proper destruction of singletons on shutdown

@pwielders @sebaszm - my unique way to add you as reviewers until I have "great power"

@pwielders I had a chat with @sebaszm, and he agrees on these changes (to unify the registration here with the one in ProxyStubs), so I think we can merge it


Pathtosharepoint   social media

Category

[ ] Content fix
[x] New article

What's in this Pull Request?
New community page:
microsoft-365-on-social-media.md

Docs Build status updates of commit 4a4a8a8:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Community/microsoft-365-on-social-media.md
:warning:Warning
View
Details

Community/microsoft-365-on-social-media.md

Line 4, Column 9: [Warning-author-not-found] Invalid value for author: 'Christophe Humbert' is not a valid GitHub ID.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Thanks, @PathToSharePoint!


Update the version of images

There's a lot of commits that shouldn't be here - can you try and isolate your changes and submit a clean PR? Thanks :)

Sure >.^


[url_launcher] Add option to open url in same window

Use case
Looks like when you use url_launcher to open an external URL in web, it defaults to opening it in a new tab. It would be great to have an option to choose to open it in the same tab, replacing the current content.
Proposal
Add an option on launch to open in same tab instead of new tab when on Web.

We have plans to work on a Link widget that will have better support for the web. @ferhatb do you know if there's already an issue for that (I couldn't find it by searching)? If not, I will create one.

Awesome, looking forward to it!

I think we can add a web only target param on the interface and use it here to address this behaviour.
I would be happy to take a look if we decide to add this to url_launcher

Yep, @tugorez please create a new issue for the changes to url_launcher that is tagged with plugins, p:url_launcher and the correct tags. It might be a good interim solution until this Link widget exists, but let's not override this thread :)
(Link has other requirements, particularly around Semantics that aren't a thing in the url_launcher plugin).


Refactor partitions list

Signed-off-by: David Cassany<EMAIL_ADDRESS>

This refactor is based on a review https://github.com/rancher-sandbox/elemental-cli/pull/32#discussion_r776958537
@mudler finally I did not use a map because I needed the partitions be in a list of a known order, this way it is possible to loop them  from first to last when partitioning the disk.


fix uint8 type in histogram

fix clesperanto/pyclesperanto#150

Codecov Report
Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 50.19%. Comparing base (bc11103) to head (9dd0743).

Files
Patch %
Lines

clic/src/tier3.cpp
50.00%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #272   +/-   ##
=======================================
  Coverage   50.19%   50.19%
=======================================
  Files          15       15
  Lines        3263     3263
  Branches     1383     1383
=======================================
  Hits         1638     1638
  Misses        524      524
  Partials     1101     1101

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


Increment version for communication releases

Increment package version after release of azure-communication-common

@0rland0Wats0n, @DominikMe, This is an automated PR that simply updates the package version once a package is released. I pushed a commit with changes from running rush update --recheck because rush kept thinking that it needs to get the beta packages locally instead of from npm.
The next step here is to update the dependencies list in the package.json file where a communication beta package is pinned. Please create a follow up PR for that once this PR is merged

@ramya-rao-a Thanks for merging this.
The automatic increment PR failing for us has been a recurring experience for us and I thought I once had filed an issue but can't find it anymore.
With the automatic PR our common package gets bumped, but the other packages that depend on common don't get their dependency updated. I'll send a follow up PR to bump the dependencies to match common's new version.
The build actually used to fail when a dependency on a local package didn't match the latest version. Not sure if this has changed recently.

With the automatic PR our common package gets bumped, but the other packages that depend on common don't get their dependency updated.

This was intentional in the automation, because while it is pretty straight forward that we should update a package version once it is released, updating the other packages that depend on it should actually be the decision of the owners of such other packages.
For packages that are out of beta, this is never a problem because we use the ^ notation in the dependency list where as beta packages are pinned as they can have breaking changes between versions.

The build actually used to fail when a dependency on a local package didn't match the latest version. Not sure if this has changed recently.

What changed was that I ran rush update --recheck and pushed the changes :)
What we can do is have the automation run this command when it updates the beta version of a package and that package is a dependency of another. cc @xirzec
@praveenkuttappan, Can we have an issue logged for this?

This was intentional in the automation, because while it is pretty straight forward that we should update a package version once it is released, updating the other packages that depend on it should actually be the decision of the owners of such other packages.

I agree with this. And I didn't know about rush update --recheck, thanks again :-)
I've sent a follow up PR to bump the dependency and fix some of the other places where we use the version numbers.