Malformed response after 1.3.0 update

After updating from 1.2.3 to 1.3.0 I started getting a quite peculiar and difficult to debug bug.  Before diving into the description, I do have a working solution, but the cause of the issue and even the reason for the solution working are both a bit perplexing right now...
I have an API Client class with a URLRequestConvertible Router enum that handles routing and I'm also using ObjectMapper (and AlamofireObjectMapper) to serialize response objects.
After updating to 1.3.0, suddenly my response blocks weren't being called at all.  I threw in some chained responseString() and responseJSON() handlers before the custom object handler, the responseJSON handler was rightfully throwing an error at serializing the response to JSON, as the responseString handler showed me the response payload was a whopping 10mB (expected maybe ~10kB).  This was happening for 2 specific endpoints, now is only happening with 1, all other API endpoints work fine.
Upon close inspection of the response string, I noticed that it was truncating the "normal" response a good way through (part way through a keyname in the JSON, or other places), and then repeating it, truncating/repeating a good number of times until it finally finishes properly.
I completely removed the custom object serialization and still saw the issue.  I even removed AlamofireObjectMapper entirely from the project and still saw the issue.
The API I'm using is internal so I can't divulge much details on the specs, but its also HTTPS making it slightly more difficult to debug.
I loaded up some certs and a mitm proxy HTTP(s) request sniffer (Cellist, and Charles, the latter worked better) and was able to watch the requests going out, After performing 1 request with Alamofire (and debugging when I perform that 1 request), println() shows 1 request fireoff in Xcode debug, but Charles (and Cellist) show a barrage of repeated requests, 245 to be exact in one measurement.  These requests/responses themselves looked quite innocent, all in tact for each one, but what was interesting is Charles was reporting "Client closed connection before receiving entire response"
Incidentally the "working" solution I found was to chain .resume() to all of my requests;  this is interesting because in the Docs it says one specifically needs to call .resume() if startRequestsImmediately is set to false.  I confirmed it indeed does "solve" the problem by also inspecting requests sent, and with .resume() chained to the end of the request it only ever sends 1 request.  When comparing requests/responses to the single/barrage cases, they're identical with the exception of the "Client closed connection before receiving entire response", leading me to believe Alamofire is somehow the culprit
Any guidance into further debugging how/where/why this is happening would be greatly appreciated.  I wish I could give some logs/screencaps but unfortunately its all confidential.

This is most likely a bug in AlamofireObjectMapper, not in Alamofire itself.
The Alamofire 1.3.0 release had a non-backwards compatible change that went into it which was documented in the release notes. You are most likely running into a problem with the AlamofireObjectMapper that hasn't yet updated to match the response serialization changes in PR #593.
It was decided that it would be better to get the serialization change in before going to Alamofire 2.0. You are most likely one of the unfortunate ones that had their logic break due to this change.
What I would do if I were you would be to debug the simple case first. Try to pull out AlamofireObjectMapper and just make the requests with Alamofire 1.3.0 and print out the results. Make sure things are working properly. Then you'll know you've narrowed down the issue to AlamofireObjectMapper. Then you can dig into the response serializers in AlamofireObjectMapper and find the issue and submit a PR.
I'm going to close this issue out for now since we've never seen this type of behavior and I'm almost positive it's not an issue with Alamofire itself. If you do end up seeing the issue without AlamofireObjectMapper, please provide some more details (code samples) and I'll gladly re-open this issue and help you dig into it.
Best of luck! 🍻

Incidentally, AlamofireObjectMapper did indeed update for 1.3.0 which is actually what forced me to update Alamofire as well (I'm using Carthage, and AlamofireObjectMapper with carthage built nested frameworks which cant be submitted, so I had to add AlamofireObjectMapper as a submodule, when updating my git it broke causing me to need to update Alamofire as well).
As I mentioned, I removed all object serialization and even removed AlamofireObjectMapper enitrely from the project and was still seeing the issue.
I'm going to attempt to make a test project to see if I can externally reproduce the issue itself, Is there an email with which I can correspond with you on this issue?  The API i'm using is a company internal API which I don't wish to publicly post snippets to ;).
Thanks!

Solved!  It was actually BigBrother
I took over /just/ my Router enum to a new standalone project and duplicated the request flow (auth first then broken endpoint).  With /just/ Alamofire, it worked perfectly fine.  After bringing in BigBrother and setting it up the same as in my main project:
# Shouldn't do either of these!
BigBrother.addToSharedSession()
BigBrother.addToSessionConfiguration(manager.session.configuration)

The issue was reproduced.  I commented out the .addToSharedSession() call and all of a sudden things worked great (Incidentally the setup there worked fine for 1.2.3 of Alamofire)
Changed fully now to use
public class var manager: Alamofire.Manager {
    struct Singleton {
      static var configuration: NSURLSessionConfiguration = {
        var configuration = NSURLSessionConfiguration.defaultSessionConfiguration()

        BigBrother.addToSessionConfiguration(configuration)

        return configuration
        }()

      static let instance = Alamofire.Manager(configuration: configuration)
    }

    return Singleton.instance
  }

(A more appropriate way to set it up according to https://github.com/marcelofabri/BigBrother/issues/2 which says the 2nd way I was setting it up was wrong)
Incidentally I'm still unsure as to what's going on in the background that's causing requests to be closed prematurely and their responses to be compounded, but am glad to know the culprit at least :) (Not Alamofire /or/ AlamofireObjectMapper !)
Also incidentally, this would be a +1 to an Alamofire solution to the network activity indicator akin to AFNetworking ;) (I believe there's already some feature requests for this?)
Thanks for help and reading and hopefully this info can be useful to someone else in the future.

Ah...that makes sense. I'm glad you solved it, and thanks for posting the solution here. Hopefully it helps someone else out that encounters a similar issue.
With that said, I would really like to discourage using any libraries in production facing apps that mess with Alamofire's underlying NSURLSession by using an NSURLProtocol. This is fine for things like stubbing, but the protocol usage does not respect all the properties of the session configuration which can cause some bad issues and strange behaviors. What you are reporting here is WAY worse than what I've seen so far. Therefore, I would keep your distance from something like BigBrother to ensure your networking logic doesn't go haywire.
Unfortunately I don't have an alternative at the moment to solve your activity indicator need. I can say though that we will be adding more support around UIKit most likely after the Alamofire 2.0 release. Until then, I'm afraid you're on your own in this area unless you want to roll the dice with something like BigBrother.

I tend to agree with you there, no need wasting effort debugging things like this!  Especially with such a strange edge case (believe me, I always find those extreme edge cases -_-)
Since I encapsulate most of my networking logic in my apiclient class, and given your input, I may go with a different solution and use BigBrother.Manager and manually inc/dec activity count around each request/response, (also making it easier to do dependency injection to perhaps swap out another "activity indicator manager" like PKNetworkActivityController, something custom, or even mocked objects for testing)
This would be a bit more manual and will require conscious effort when adding in new requests (even outside the apiclient), but its worth the trade off of flexibility (and my sanity).
For anyone else wondering what this looks like, something simple like this
import AlamoFire
public class ApiClient {
  static let sharedInstance = ApiClient()
  var incrementActivity: () -> Void = {}
  var decrementActivity: () -> Void = {}

  func search(query: String, callback: (data: AnyObject?) -> Void) {
    incrementActivity()
    Alamofire.request(Router.Search(query: query))
      .responseString { (_,_,data,error) in
          if error != nil {
              callback(data: nil)
              return
         }
        callback(data: data)
      }
      .response { (_,_,_,_) in self.decrementActivity() }  // Call after other response handlers
  }
}

import BigBrother
let apiClient = ApiClient.sharedInstance
apiClient.incrementActivity  = { BigBrother.Manager.sharedInstance.incrementActivityCount()  }
apiClient.decrementActivity = { BigBrother.Manager.sharedInstance.decrementActivityCount() }

// alternatively
import PKNetworkActivityController
apiClient.incrementActivity  = { NetworkActivityController.sharedController.registerActivity()  }
apiClient.decrementActivity = { NetworkActivityController.sharedController.deregisterActivity() }

// finally
apiClient.search("lorem") { data in
    println(data)
}


The icons lib doesn't work.

Seems like it is not installed in dependencies... and... npm install should install it. What is wrong?

Pleas can you specify a) what you want to do and b) what happens? Icon libs have to be activated in the "app/config.json" files.

Please read the documentation and let me know any open issue: https://github.com/scriptPilot/app-framework/blob/master/docs/icon-fonts.md


fabs, fabsf and fabsl are missing

It would be great to have the full abs function set for compatibility purpose with standard library.
https://en.cppreference.com/w/cpp/numeric/math/fabs

Added, though I learned in the process that there are issues calling std::fabsf and std::fabsl...


Can tutel support Pipeline Parallel?

Hi all. After walking through the examples, I suppose that tutel currently support data / tensor parallel for its moe layer module. Is it correct? Then, what can I do if I want the entire model training support Pipeline parallel?
Or, can tutel be used concurrently with Megatron or Deepspeed? If so, then I can configure the hybrid parallel following the other two's configuration manner.

I believe this answer exactly works for me. I'll close this issue now.


More of a request than an issue

Dear Paulo, greetings!
We have been using dss-extensions in Costa Rica for a while now, for teaching purposes in the university and for consultancy work.
We have detected an oportunity of improvement in terms of the harmonic mode.
Could you be so kind to add an input parameter to the loads, where you are able to include an Rprofile and an Xprofile as an array for each harmonic, instead of using the built in harmonic profile calculator (which is limited to an RL branch)? This would help a lot to model the equivalent impedance of power electronics devices, which depend on the control parameters and topology.
Thanks!
Andres

Hi, @AndresAG91,

We have been using dss-extensions in Costa Rica for a while now, for teaching purposes in the university and for consultancy work.

Nice, happy to see that!

Could you be so kind to add an input parameter to the loads, where you are able to include an Rprofile and an Xprofile as an array for each harmonic

From a glance, that seems like a good idea, but so far we have avoided adding any DSS property to the objects. Right now DSS scripts are nearly fully compatible with the official OpenDSS, except for the OpenDSS features that are closed-source (and not available publicly).
Since we're working on some big changes right now though, your suggestion might be feasible in a couple of months. We can probably add it easier after the changes land, and add a compatibility flag to hide the properties for a compatibility mode.
We could add some API functions to achieve something like that earlier, without changing the properties, but I imagine you may already be doing something like that, is that right?
(I'll move this to the main dss-extensions repo as a feature request since it's an engine-level feature.)


Input Field issue

Hey, I have updated my Unity project to 2019.2.9f1 and I am getting this error when I type something in a RTLTMP_Inputfield :
IndexOutOfRangeException: Index was outside the bounds of the array. TMPro.TMP_InputField.GenerateCaret (UnityEngine.UI.VertexHelper vbo, UnityEngine.Vector2 roundingOffset) (at Packages/com.unity.textmeshpro@2.0.1/Scripts/Runtime/TMP_InputField.cs:3307) TMPro.TMP_InputField.OnFillVBO (UnityEngine.Mesh vbo) (at Packages/com.unity.textmeshpro@2.0.1/Scripts/Runtime/TMP_InputField.cs:3275) TMPro.TMP_InputField.UpdateGeometry () (at Packages/com.unity.textmeshpro@2.0.1/Scripts/Runtime/TMP_InputField.cs:3213) TMPro.TMP_InputField.Rebuild (UnityEngine.UI.CanvasUpdate update) (at Packages/com.unity.textmeshpro@2.0.1/Scripts/Runtime/TMP_InputField.cs:3188) UnityEngine.UI.CanvasUpdateRegistry.PerformUpdate () (at C:/Program Files/Unity/Hub/Editor/2019.2.9f1/Editor/Data/Resources/PackageManager/BuiltInPackages/com.unity.ugui/Runtime/UI/Core/CanvasUpdateRegistry.cs:198) UnityEngine.Canvas:SendWillRenderCanvases() (at C:/buildslave/unity/build/Modules/UI/ScriptBindings/UICanvas.bindings.cs:72)
It appears every second or when I type a new letter.
I have tried using a normal TMP input field and it works fine. I have also downgraded to 2019.6 (same error). The latest working version is 2018.3.
Quick help would be very very much appreciated,
Thanks.

This fixed it.
It is weird because I am 100% sure I did it already and as you can from the error, the package is already in the folder (Packages/com.unity.textmeshpro@2.0.1/...).
Anyway, thanks for the assist :)


Remove console.log calls

This super-tiny change just removes console.log calls from around the codebase.
My philosophy is that console.log should only be used for temporary debugging statements. That way, when you're ready to open a PR, you can do a project-wide search for console.log and quickly clean them up.
console.info works just the same way as console.log, and feels a bit more semantic to boot. I also switched to console.error when the output was an error.

LGTM!
Somewhat related... ish, has anyone run into this before on Mac?

It seems to be this issue. I can log it in a new issue if needed.

Thanks for the review y'all!
@melanieseltzer I've seen that before. It seems like a safe warning to ignore. Ideally we'd disable it, but AFAIK there isn't a way.
I'm not sure if it makes sense as an issue... maybe we can create one, and then close it with this info? That way it'll be archived for people searching for it, so it might stop this from coming up more in the future?

How do you feel about using something like debug to allow adding debug logs without them polluting the console?

How do you feel about using something like debug to allow adding debug logs without them polluting the console?

Hm, I don't know... I feel like with debugging, you really want to add specific info to specific places. If I'm reading this correctly, the idea is you'd add a bunch of these debug calls around permanently and then enable them with an env variable? It seems less-than-ideal, since at least for my workflow, I just want to be able to quickly add some log statements, and I'll likely never need them again.
Maybe it's just for general console logging / error-catching? In that case I can see the appeal of being able to toggle them on/off with an env variable. Not sure that it's a huge benefit though.
I'm open to it if you want to try using it! But I don't feel particularly motivated to try it myself.


[EN DateTimeV2] "12-month period" cannot be recognized as a whole, and only "12-month" is recognized.

As the title.

"period" is not part of the duration, so by design it is not tagged.


Replaced Wieldable DoAfter with UseDelay

About the PR

Weapons using WieldableComponent can be wielded immediately with a brief cooldown before unwielding based on UseDelay component.
I added 1 second usedelay components to the 12 or so items I saw were wieldable.
Media

Here's some footage of me picking up wieldable items and spamming the z key.
https://youtu.be/_rHzTw3K1Vo
Seems to work as expected.

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Added brief delay after wielding two handed weapons.

the unwield/need a freehand popup seems to be getting spammed? is that a result of you spamming the z key or does it do that normally?


JS dates use a different format to the PHP dates

http://cloud.rebelinblue.com/image/3n3E0D01333s

Edited issue as I managed to commit with the wrong ID in the message


Flank snapshot smart result report differs from current 8.1.0 breaking backward compatibilty

The current 8.1.0 smart result reports stores information in the gcs bucket with a hostname.
Example:
<testsuite name="Nexus5X-23-en-portrait#" tests="50" failures="0" errors="0" skipped="0" time="1388.586" timestamp="2020-04-28T20:38:55" hostname="localhost">
The current flank snapshot release does not store information in the gcs bucket with a hostname.
Example:
<testsuite name="Nexus5X-23-en-portrait" tests="8" failures="0" flakes="0" errors="0" skipped="0" time="150.446" timestamp="2020-04-25T09:45:37">
When testing the snapshot and then reverting back to flank 8.1.0, flank 8.1.0 will fail to parse the information stored in the bucket:
com.fasterxml.jackson.module.kotlin.MissingKotlinParameterException: Instantiation of [simple type, class ftl.reports.xml.model.JUnitTestSuite] value failed for JSON property hostname due to missing (therefore NULL) value for creator parameter hostname which is a non-nullable type
 at [Source: (byte[])"<?xml version='1.0' encoding='UTF-8' ?>
<testsuites>
  <testsuite name="Nexus5-23-en-portrait" tests="1" failures="0" flakes="0" errors="0" skipped="0" time="2.594" timestamp="2020-04-23T20:40:34">
    <testcase name="seeView" classname="com.osacky.flank.gradle.sample.ExampleInstrumentedTest" time="2.594"/>
  </testsuite>
  <testsuite name="Pixel2-26-en-portrait" tests="1" failures="0" flakes="0" errors="0" skipped="0" time="2.712" timestamp="2020-04-23T20:40:14">
    <testcase name="seeView" cl"[truncated 109 bytes]; line: 5, column: 3] (through reference chain: ftl.reports.xml.model.JUnitTestResult["testsuite"]->java.util.ArrayList[0]->ftl.reports.xml.model.JUnitTestSuite["hostname"])

This poses a problem for teams testing out the snapshot because it will break all the other builds in the pipeline which are using the same gcs results bucket.

Hi @runningcode, thanks for reporting issue. You are right, API parsing is not backward compatible with legacy XML parsing and was not designed to be. That's because of structural changes and some improvements. But also is not possible to obtain hostname value from api, which in XML parsing probably always is localhost.
In description of legacy-junit-result option is mentioned that can generate slightly different output format but maybe it's insufficient info.

Flank provides two ways for parsing junit xml results.
New way uses google api instead of merging xml files, but can generate slightly different output format.
This flag allows fallback for legacy xml junit results parsing
Currently available for android, iOS still uses only legacy way.
legacy-junit-result: false

Simply, is not possible to use API generated report to calculate shards for XML generated report. It possible to adjust implementation to be backward compatibile, but I'm not sure If we really want that. @bootstraponline?

I think if we always add hostname="localhost" to match FTL, then the old Flank will be happy because it sees the hostname. The incompatibility is due to us removing hostname which is a required field in the old Flank.
This seems like a small change to make. The hostname value is static and not fetched from FTL.

OK, I can fix that tomorrow easly it but I can't promise that there will be no other problems. We will see.


Icon compute capability

Apparently -ta=nvidia:cc80 as a Fortran flag is deprecated. -gpu=cc80 replaces it.
This makes +ecrad work.

launch jenkins icon

tsa
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec

timed out after 5 seconds:red_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle:daint_cpu_gcc :green_circle:daint_cpu_nvhpc :red_circle:daint_gpu_nvhpc
WARNING: Serial tests did not run for system tests

launch jenkins icon

tsa
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec

daint
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle:daint_cpu_gcc :green_circle:daint_cpu_nvhpc :green_circle:daint_gpu_nvhpc

timed out after 5 seconds:red_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :lock:Icon-install_nwp_cpu spack locking problem:red_circle:Icon-install_nwp_gpu
WARNING: Serial tests did not run for system tests

timed out after 5 seconds:red_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :red_circle:Icon-install_nwp_cpu :red_circle:Icon-install_nwp_gpu
WARNING: Serial tests did not run for system tests

launch jenkins icon

balfrin
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle:Icon-install_nwp_cpu :green_circle:Icon-install_nwp_gpu

daint
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle:daint_cpu_gcc :green_circle:daint_cpu_nvhpc :green_circle:daint_gpu_nvhpc

launch jenkins icon

tsa
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec

daint
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle:daint_cpu_gcc :green_circle:daint_cpu_nvhpc :green_circle:daint_gpu_nvhpc

launch jenkins icon balfrin

tsa
:green_circle: unit testTest:green_circle:summary

daint
:green_circle: unit testTest:green_circle:summary

balfrin
:green_circle: unit testTest:green_circle:summary :green_circle: integration testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle: system testTest:green_circle:summary :green_circle:icon-spack_info :green_circle:icon-spack_spec :green_circle:icon_serialization=create_claw=std-spack_spec :green_circle:Icon-install_nwp_cpu :green_circle:Icon-install_nwp_gpu


Add icons8 connector

Purpose
$subject
Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? no
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment
JDK 11
Ballerina SL Beta3
Ubuntu 20.04

@Chuhaa This PR has icons8 and goto.meeting packages. Could you please check and send only icons8 package?

@Chuhaa This PR has icons8 and goto.meeting packages. Could you please check and send only icons8 package?

Resolved. Thanks.


affected/package: net/http

What version of Go are you using (go version)?

$ go version
latest

Does this issue reproduce with the latest release?
not
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
darwin arm64

What did you do?
There might be a potential safety problem about the variable of err in :https://github.com/golang/go/blob/master/src/net/http/transport.go#L1707
I am not sure if it's right. This variable can be accessed in two different goroutines, but without any mutex or other concurrent protection.
I have seen the didReadResponse chan so that it can guarantee the sequence of two goroutines. But I am not sure that if err variable will and must get the result of another goroutine. Will CPU cache affect it?
According to the The Go Memory Model: https://go.dev/ref/mem. Thanks.

What did you expect to see?
Confirmed that if it has a potential safety problem.
What did you see instead?

Not an issue. There's explicit synchronization in the form of channels.


Base snapshot images are duplicated with timestamped filenames

Hi,
The 'original' plugin suffers from duplicate base images being created during initial base snapshot generation, and so does this one.
[A number of fixes were proposed[(https://github.com/cypress-visual-regression/cypress-visual-regression/pull/135), but the maintainer closed the fix PRs, leaving the only solution to be a post npm script to remove the duplicated timestamp images
Can you fix for this plugin?

@stephenwil  could you provide us with a sample project/branch/pull request that reproduces the problem?


Const property in a blueprint causing error

Hello,
I think you've wrapped this private variable in a const method and decorated it as a read-only UPROPERTY, but it should be a UFUNCTION instead.
StableDiffusionTools/Source/StableDiffusionSequencer/Public/StableDiffusionPromptMovieSceneSection.h(22): Error: Const properties are not supported.

Oops, I think that was supposed to be a UFUNCTION. Thanks!

Closed in https://github.com/Mystfit/Unreal-StableDiffusionTools/commit/158bc13a8a91ebb0cb8c7645bca9453843609e59


angular 1.6.1

Issues with angular 1.6.1 .. autoPlay: true was the main one I found in my very limited testing.
Quick test:  Use your simple demo @ http://codepen.io/anon/pen/RoXaXy and change the 1.5 cdn out with 1.6.1 link below. You’ll notice  the slides will no longer play. This is a great wrapper though! Thanks!
https://cdnjs.cloudflare.com/ajax/libs/angular.js/1.6.1/angular.min.js

Thanks for pointing this out. I've been deep in the Angular2 world for work so haven't even tried this library with 1.6.x.  I'll try to find some time to look into this. 👍

Have you had a chance to use something like this in angular2.   A rewrite in typescript might be a fun side project ;)

I've actually just finished my second Angular2 project, so I finally feel competent enough to start rebuilding this for ng2. Finishing up a hard deadline this week, so I'm hoping I can put some time into this bug and the ng2 version in the coming weeks 👍

Ok, it seems a bug popped up around the merging of user options with default options. Fixed in 3.1.1. https://github.com/benjamincharity/angular-flickity/releases/tag/v3.1.1
Thanks for reporting! 🙇


Created Index page

Created index page
I would love to work on these project
Thank you!!

Created index page
I would love to work on these project
Thank you!!

First of all, thank you for try to help with this project and sorry for keep you waiting, I've only seen this now.
I'm sure you've done a great job, but I'm going to use React as the framework for building this application, and I already have the design made on Figma.  I want to make it alone for learn Node and React, but thank you again for your time and dedication.
Note: I'm brasilian, sorry if I wrote something wrong.

Hey, thanx for responding.
no problem....go ahead with your plan.. surely !! :) :) ;)


More detail in instructions for running Cockpit

Thanks for the PR! Why do you stated that no password is required? The standard distributions do use the password "sa" for user "sa".

I ran flowing-retail first, then tried to run cockpit from the standard distribution. Cockpit didn't work for me with the password but worked without. I assumed flowing-retail had set a different (blank) password for some reason than the standard distribution does.

Ah sorry - now I got it. you are right - in the flowing-retail there is no password set. It might be easier to adjust it there to avoid confusion - I will double check

Cheers :-)

I changed the codebase to also use sa/sa when starting the flowing retail, then the distro doesn't have to be touched for this. I liked it more this way round. Thanks a lot for raising the issue!

Makes sense, thank you.


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/109 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit f0d6878:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit f0d6878:
:clock10: Pending: waiting for processors (6 builds ahead of you)
:warning: Docs Build is busy, currently there are 6 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit f0d6878:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit f0d6878:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
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.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Missing PRs (merged PR lost) in changelog

I noticed some merged PRs which are also shown in the commits log in the main branch are not shown in the changelog.
The following info indicated fewer PRs fetched than the release commits. This rise my confusion because we are using the squash and merge model for our PRs, which means PRs fetched should always be equal to the commits.
ℹ Retrieved 287 merged PRs for xxx/xxxx
ℹ Retrieved 350 release commits for xxx/xxxx

@yuhao-su I see you opened a PR with the improvement to the sort order. Will look into it


Extending some number of Vecs

In order to gain a performance boost from auto-vectorization, I'm storing data in multiple Vecs. Think struct of arrays instead of array of structs. In the array of structs case, it's very easy to simply extend the vector with whatever the parallel iterator provides. However, in the other case I don't have this luxury.
I came up with two solutions:
vec1.reserve(...);
...
vecN.reserve(...);

/// initialize Vec with Default data; but this is visibly expensive in my benchmarks

vec1.par_iter_mut().zip(...).map(|(val, ...)| *val = ...)

Or use uninitialized memory:
let vec1: Vec<MaybeUninit<...>> = ...;
...
let vecN: Vec<MaybeUninit<...>> = ...;

vec1.reserve(...);
...
vecN.reserve(...);

vec1.par_iter_mut().zip(...).map(|(val, ...)| unsafe { ... })

The problem I have with the second solution is that it's very painful to use and it litters all my code with a lot of unsafe. However, I don't see any solution to remove it since initializing the vector with default values is visibly costly. Not zipping the iterators together also doesn't work because it will have to iterate through the data multiple times.
Any ideas on how to improve this?

Recent version of Rayon provides us with parallel iteration over a tuple.
(a, b, c).into_par_iter()
            .for_each(|(a_i, b_i, c_i)| {
     ;; write your code here
}

I think this make code look better, even though this doesn't help answering the current question.

Rayon also has two implementations of ParallelExtend that work like unzip and partition_map:
impl<A, B, FromA, FromB> ParallelExtend<(A, B)> for (FromA, FromB)
where
    A: Send,
    B: Send,
    FromA: Send + ParallelExtend<A>,
    FromB: Send + ParallelExtend<B>,
{...}

impl<L, R, A, B> ParallelExtend<Either<L, R>> for (A, B)
where
    L: Send,
    R: Send,
    A: Send + ParallelExtend<L>,
    B: Send + ParallelExtend<R>,
{...}

These were added in #604, and as noted there, you can also nest this in pairs.
let (vec1, (vec2, (vec3, vec4))) = par_iter.map(|(a, b, c, d)| (a, (b, (c, d)))).unzip();

That should also work for calling par_extend directly. Perhaps we could add flattened expansions of this similar to the MultiZip that @dineshadepu mentioned. I'd probably only want that for the unzip-like fashion, as trying to mix Either in there for partition_map will get really complex.
I kind of wish we had impl<C: ParallelExtend> ParallelExtend for &mut C too, as it would make this automatically able to extend a tuple of mutable references, instead of just by value. That would be a breaking change to add such a blanket impl now. Even testing just for (&mut A, &mut B) now complains:
error[E0119]: conflicting implementations of trait `iter::ParallelExtend<(_, _)>` for type `(&mut _, &mut _)`:
   --> src/iter/unzip.rs:428:1
    |
413 | / impl<A, B, FromA, FromB> ParallelExtend<(A, B)> for (FromA, FromB)
414 | | where
415 | |     A: Send,
416 | |     B: Send,
...   |
425 | |     }
426 | | }
    | |_- first implementation here
427 |
428 | / impl<A, B, FromA, FromB> ParallelExtend<(A, B)> for (&'_ mut FromA, &'_ mut FromB)
429 | | where
430 | |     A: Send,
431 | |     B: Send,
...   |
440 | |     }
441 | | }
    | |_^ conflicting implementation for `(&mut _, &mut _)`
    |
    = note: downstream crates may implement trait `iter::ParallelExtend<_>` for type `&mut _`
    = note: downstream crates may implement trait `iter::ParallelExtend<_>` for type `&mut _`

That should also work for calling par_extend directly.

This doesn't seem to work because of the lack of impl<C: ParallelExtend> ParallelExtend for &mut C.

I'm using this as a temporary fix:

Ah, a newtype wrapper does the trick -- we could even make something generic like that.

This seems to cause rustc to compile very slowly.

I'm not surprised. While the generic unzip.rs is not a lot of code, it's a fairly complex composition, and it will compound quickly when those are nested together. It might fare a little better if you nest that like a binary tree, (((vec0, vec1), (vec2, vec3)), ((vec4, vec5), (vec6, vec7))), but it will still be a challenge.
In a presentation I gave last year, I demonstrated a giant symbol -- 132967 characters! -- from just a 3-way collect. See also #671, which was alleviated some by #673 and rust-lang/rust#62429, but the overall structure still remains the same.
If we had something more directly designed for just unzipping vectors, it wouldn't need to be nearly so complicated.

I guess it's probably because of the giant symbols and the monomorphization that this is so slow. The tree arrangement doesn't help much, unfortunately.
I was thinking whether there would be a way to maybe implement par_extend for each type of tuple that I'm using separately, or maybe have an 8-way unzip implementation.
Even if the solution is dirty, it would still be much better than my current unsafe-riddled approach. Do you think any of these ideas could help reduce the costs?

I was taking a look at the unzip implementation and it seems like even a manual 8-way tuple would probably produce the same result.
@cuviper, it seems like finding a way to force the Cosumer to be the same in all generic types would be a way to solve this, but this would mean that any UnzipN type would not be able to implement the ParallelIterator trait. Do you think there's any way around this? Maybe the consumers can be made into trait objects and saved in a Vec, but I'm not completely sure they are convertable to trait objects and whether or not this would impact performance in any way.

I'm at a loss for designing a better generic unzip. It's really difficult that we can only get each ParallelExtend consumer through a callback into drive_unindexed.
Maybe we need a new trait with a method that directly creates a consumer -- although that also means each collection's consumer type would become part of the public API.

So, my current approach is basically extending CollectConsumer to CollectTupleConsumer. I've got a working MVP, but I don't know if this is the best approach. However, it does compile fast enough.

@cuviper, would it desirable to have the ParallelExtend trait be defined for larger tuples, e.g. up to 16? While my workaround works well, I'd rather not have to use unsafe for this.

@cuviper, would it desirable to have the ParallelExtend trait be defined for larger tuples, e.g. up to 16? While my workaround works well, I'd rather not have to use unsafe for this.


Containerd monitoring script - remove/fix?  Bad side effects + no clear advantage

How to categorize this issue?

/area os
/area robustness
/kind bug
What happened:
The containerd monitoring script sends a SIGTERM to a restarting containerd daemon process triggering a potential containerd bug leading to loss of shim process and user processes. The containerd daemon can only be recovered after manual intervention.
This was reported by a Stakeholder.
Problem: the command timeout 60 ctr c list returns as soon as ctr returns with exit code 1. This depends on a ctr internal timeout which is much smaller than 60 seconds.
This in turn can lead to the monitoring script failing on a containerd restart (e.g when a stakeholder re-configures the containerd config) and sending a SIGTERM to containerd. Containerd's signal handler catches the signal.
Hypothesis: Containerd is terminated due to handling a SIGTERM during startup (systemd status activating) reconnection to shim processes.
It is unclear why the shim processes are terminated (that should never happen). I will link the containerd issue here once the Stakeholder has created it.
What you expected to happen:
Containerd monitoring script retries ctr c list for 60s until it sends a signal to containerd. This avoids sending SIGTERMS too early.

also relevant for CPU/memory contention scenarios when containerd might be slow to react to RPC calls. We do not want to run into a potential restart loop.

Additionally, we should not send SIGTERMS during startup i.e before the systemd unit active status.
I could also see the monitoring script getting removed

the containerd systemd service is configured with auto-restart in case the containerd process terminates
no clear evidence that a process restart brings any benefit when it is unable to respond to RPC calls in time.
I would rather argue the containerd monitor is actively harmful

restarting a process under resource contention causes all it's memory pages to be flushed. Demand-paging large amounts of pages during memory contention can be a performance problem. Having to fill the page cache again can be a performance problem (containerd page cache size can be in the tens of Gi)
possible exposure to bugs - such as we see with this issue.

How to reproduce it (as minimally and precisely as possible):

run the monitoring service i.e command watch timeout 60 ctr list
restart containerd
ctr list will fail once the containerd socket is gone, returns exit code 1.

ctr: failed to dial "/run/containerd/containerd.sock": context deadline exceeded

the containerd monitor script sends a SIGTERM to containerd
containerd restarts and hangs during reconnection to the (gone) containerd shims

Note that containerd-shim processes have been terminated + the respective user processes are either gone (probably SIGPIPE) or re-parented to init.
65535       4579  0.0  0.0    996     4 ?        Ss   06:04   0:00 /pause
65535       4698  0.0  0.0    996     4 ?        Ss   06:04   0:00 /pause
65535       4714  0.0  0.0    996     4 ?        Ss   06:04   0:00 /pause
65535       4726  0.0  0.0    996     4 ?        Ss   06:04   0:00 /pause
65535       4728  0.0  0.0    996     4 ?        Ss   06:04   0:00 /pause
nobody      4753  0.0  0.0    996     4 ?        Ss   06:04   0:00 /pause
root        4859  0.0  0.2 1178356 32068 ?       Ssl  06:04   0:04 /gce-pd-csi-driver --endpoint=unix:/csi/csi.sock --run-controller-service=false --logtostderr --v=5
root        5035  0.0  0.1<PHONE_NUMBER>8 ?        Ssl  06:04   0:01 /apiserver-proxy-sidecar --ip-address=<IP_ADDRESS> --setup-iptables=false --interface=lo
root        5073  0.5  0.5<PHONE_NUMBER>0 ?        Ssl  06:04   2:11 cilium-operator --config-dir=/tmp/cilium/config-map --debug=false
nobody      5111  0.0  0.1<PHONE_NUMBER>6 ?        Ssl  06:04   0:13 /bin/node_exporter --web.listen-address=:16909 --path.procfs=/host/proc --path.sysfs=/host/sys --path.rootfs=/host --log.level=error --collector.disable-defaults --collector.conn
root

To be more thoroughly tested: can this inconsistent state of containerd only be reproduced when the containerd-monitor sends a SIGTERM during startup? In essence, are manual restarts with inactive monitoring script fine?
Anything else we need to know?:
I will link the containerd bug issue once it got created.
Environment:

GL 576.11.0 running containerd 1.5.11

containerd bug issue.


refactor: nuking L2Block.fromFields

Fixes https://github.com/AztecProtocol/aztec-packages/issues/8340
It's an unnecessary syntactic sugar which is causing issues. fromFields name is also used to deserialize from an array of Frs in other parts of codebase so it's quite confusing.

#8882  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @benesjan and the rest of your teammates on  Graphite

Docs Preview
Hey there! 👋 You can check your preview at https://66fa962a7684121270947ad3--aztec-docs-dev.netlify.app


IsAutoCloseEnabled="True" for SimpleChildWindow - possible?

Hi,
I would like to ask if it's possible to add AutoClose to the SimpleChildWindow, just like Flyouts has in MahApps.Metro project? IsAutoCloseEnabled="True" with proper interval.
Thank you :)

@RiderOfTheli This will be available in 1.4.0


fix: allow storage implementation to determine writability

Description
This allows the storage driver or plugin to determine whether or not a file or folder is writable. This is achieved by extending the stat interface to include isWritable as a field, and to expect that field to be present when calculating this value for use in the client.
This will allow third party storage plugins to determine who and where bot projects can be created using the appropriate internal mechanism. For example, using a logged in user identity to determine if a bot project is accessible.
Task Item
Fixes #3319
Screenshots
This controls where a bot project can be created. It appears in the "new bot" flow.

Coverage increased (+0.01%) to 53.221% when pulling 361d7d986671544f048685aef4a9bb11bcc97ee0 on benbrown/3319 into a124273f8d46e53b8a3b96094c8f9b65a0165aa9 on main.


forAll :: ∀ e a f. Foldable f => (a -> String) -> String -> f a -> (a -> Aff e Unit) -> Spec e Unit

What are thoughts on adding something like this this?
forAll :: ∀ e a f. Foldable f => (a -> String) -> String -> f a -> (a -> Aff e Unit) -> Spec e Unit
forAll itTitle title arb f = describe title do
  for_ arb \a -> it (itTitle a) (f a)

...
  forAll _.str "format (unformat a) = a" \ ({ str }) -> do
    (format $ unformat str) `shouldEqual` (Right str)
...

I have been using for_ in it block but if some test case fails then other items are not tested and you can't see them in log. using this function you can see each item in log and all of them will be executed.

I would welcome utility functions like these as I have been re-inventing that wheel many times over for neodoc

I was looking for something like this to write a tests that takes a record
of functions that work against a database, and a record of the same type
with fakes, making sure both behave the same in the test.
On Tue, Apr 25, 2017 at 10:24 PM, Felix Schlitter<EMAIL_ADDRESS>wrote:

I would welcome utility functions like these as I have been re-inventing
that wheel many
https://github.com/felixSchl/neodoc/blob/development/test/Test/Spec/SolveSpec.purs#L240
times
https://github.com/felixSchl/neodoc/blob/development/test/Test/Spec/DescParserSpec.purs#L58
over
https://github.com/felixSchl/neodoc/blob/development/test/Test/Spec/ArgParserSpec.purs#L974
for neodoc
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/owickstrom/purescript-spec/issues/49#issuecomment-297169561,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAK7uOLsTKOtE7xPrUJuIcrwvaoq_hWDks5rzmSigaJpZM4NH69o
.

Yeah, this does seem useful. I have some questions/considerations:

The name forAll feels like it's too close to forall. Would prefer something closer to it, as it's basically a "multi-it". Perhaps they?
I'd prefer to drop the outer describe in this definition, keeping it a bit simpler, and not making an assumption that you want a separate level for the foldable. I'm not sure, but you might want to do something like:describe "many things" do
  they show [2, 4] \n -> (n `mod` 2) `shouldEqual` 0
  they show [4, 8] \n -> (n `mod` 4) `shouldEqual` 0

And as a last thing, is the argument order the way we want? Not saying it isn't, but you might have feedback on that.

Thanks for the suggestion!

dropping out describe is good idea.
they seems good to me.

so at this point it looks like:
they :: ∀ e a f. Foldable f => (a -> String) -> f a -> (a -> Aff e Unit) -> Aff e Unit
they itTitle arb f = for_ arb \a -> it (itTitle a) (f a)

...
  describe "format (unformat a) = a" do
    they show arb \(str) -> (format $ unformat str) `shouldEqual` (Right str)
...

About order:
I don't have strong opinion on it, but If we leave f as last argument, then we have two options:

NAME itTitle arb f, infix usage: itTitle `NAME` arb f
NAME arb itTitle f, infix usage: arb `NAME` itTitle f
I can't think of a good name (english is not my native language) but there might be some name which will best fit this usage. maybe that? (show that validDates parseAndUnparseComutes)

Yeah, I could not come with any specific order either, as it doesn't read naturally anyway. I think we should go with what you already have:
they :: ∀ e a f. Foldable f => (a -> String) -> f a -> (a -> Aff e Unit) -> Aff e Unit

Would you like to submit this as a PR?

Would you like to submit this as a PR?
Yes!


Our package name is scooped!

It appears someone has registered the balm package name on pypi but not released anything. We can only see the package on test.pypi, which we did not check (the github repo is private): https://test.pypi.org/project/balm/
It looks like it was last updated last August. We can try to contact the author, or we can change the name. Suggestions?

P.S. as far as I can tell, there is no good way to check name availability in advance, because that would run counter to PyPI's goals of giving me a migraine.

 I'll write him an email to ask if he's going to use it. Maybe we can have it if it isn't some secret cool project that will be released soon.

In the worst case, we can do some simple renaming, e.g., map->diagram = bald

Yes. While waiting for the response, I will think about a new name for our project.

In the worst case, we can do some simple renaming, e.g., map->diagram = bald

The name "bald" recalled me this article https://neurosciencenews.com/bladness-protein-21115/. :)

So, we've got a response :)
Unfortunately, we'll have to find another name :/ The balm name will be used and the project is now live https://github.com/itrummer/balm
I'm fine with BoALM, but we can try to compile a list of options for the next meeting. We could also consider balm-bn (biological attractor landscape mapper for boolean networks).

We're biobalm now. I'll change the repo name soon.

done!


program crashes when both grpc++ and openssl are linked

What version of gRPC and what language are you using?
0.3.6 / c++
What operating system (Linux, Windows,...) and version?
Linux Rethat 7
What runtime / compiler are you using (e.g. python version or version of gcc)
gcc 5.2.1
What did you do?
I'm trying to build a program using Bazel. which depends on "@com_github_grpc_grpc//:grpc++"
and restclient-cpp, which in turn depends on a system installed openssl .so files, version 1.0.1
grpc++  built from src and statically linked into the program, but restclient-cpp is prebuilt and distributed as .so files.
What did you expect to see?
Program should not crash.
What did you see instead?
When I remove the grpc++ dependency, program does not crash. When I remove the restclient-cpp dependency, program does not crash.
When both grpc++ and restclient-cpp are linked(verified by examining ldd and binary sizes),program crashes at run time:
@     0x7fb3dc6e05e0 (unknown)
@     0x7fb3db05599c SSL_ctrl
@     0x7fb3e11464f3 ossl_connect_common
@     0x7fb3e1147410 Curl_ssl_connect_nonblocking
@     0x7fb3e1100652 https_connecting
@     0x7fb3e1111e91 Curl_protocol_connect
@     0x7fb3e1125a0e multi_runsingle
@     0x7fb3e11267b1 curl_multi_perform
@     0x7fb3e111d12c curl_easy_perform
@     0x7fb3e0ee26b6 RestClient::Connection::performCurlRequest()
@     0x7fb3e0ee2b02 RestClient::Connection::post()

Anything else we should know about your project / environment?
From searching Google, some users have suggested to point grpc to build against openssl instead of boringssl to fix this issue. I'm just wondering how I can go about doing this and whether this will have implications down the line when future grpc versions are released.

If you are building a software using Bazel, use grpc's Bazel files themselves. You will then avoid the problem altogether, since Bazel will enforce a single openssl library being linked in.
In general with Bazel, do NOT use .a files in there. Always compile everything.

Hi Nicolas, thanks for the reply. To clarify:

I am building an application using bazel
the dependency "@com_github_grpc_grpc//:grpc++" of my application is pointing to the grpc source(i.e., all of grpc and its own dependencies are compiled and statically linked into my binary)
In addition to grpc++, my application is also pulling in a precompiled restclient-cpp .so via cc_import, (note we don't want to compile this particular dependency from src as this is already packaged and vetted prior). The restclient-cpp links in curl, which in turn links a system provided openssl.

Only when both of these are linked into the application, the program crashes with the above stacktrace. it 's happy when only one of these two dependencies exist.

There's no right way to solve this. That's kind of on bazel, not grpc. You simply can't do this properly with bazel. You need to build ALL of your dependencies with bazel for it to properly work.
One ugly workaround would be to provide grpc with a different library of OpenSSL that coincides with the one your .so uses. But again, that's not the intended usage of bazel.


lidarr: <IP_ADDRESS>39 -> <IP_ADDRESS>49

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for lidarr is: A Usenet/BitTorrent music downloader
meta.homepage for lidarr is: https://lidarr.audio/
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found <IP_ADDRESS>49 with grep in /nix/store/gkfcqf6xl0rnd93zyql8f1xip7b8wh98-lidarr-<IP_ADDRESS>49
found <IP_ADDRESS>49 in filename of file in /nix/store/gkfcqf6xl0rnd93zyql8f1xip7b8wh98-lidarr-<IP_ADDRESS>49

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
lidarr

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gkfcqf6xl0rnd93zyql8f1xip7b8wh98-lidarr-<IP_ADDRESS>49 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A lidarr https://github.com/r-ryantm/nixpkgs/archive/ae8175d702227ef0c8cdc73467ece2690f1d42f3.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ae8175d702227ef0c8cdc73467ece2690f1d42f3#lidarr

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gkfcqf6xl0rnd93zyql8f1xip7b8wh98-lidarr-<IP_ADDRESS>49
ls -la /nix/store/gkfcqf6xl0rnd93zyql8f1xip7b8wh98-lidarr-<IP_ADDRESS>49/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

lidarr

Maintainer pings
cc @etu for testing.

Result of nixpkgs-review pr 274708 run on x86_64-darwin 1

1 package built:

lidarr


🔐 ITHC Prep

User Story
As a AP engineer, I need to prepare for the ITHC that will be carried out on the platform on from March 4th to March 11th.
This includes completing the project planner, onboarding users to our AWS environment and Dev/Prod control Panel. Add them to the moj-analytical-services GH org and make sure their access is functiional.
Value / Purpose
Our ability to carry out this activity successfully will reflect in the quality of the outcomes we get,
Useful Contacts
@julialawrence
Definition of Done

[x] Add PenTester Roles to AP Dev Cluster
[x] Testers added to moj-analytical-services Org
[x] IAM users created in the landing account
[x] Roles created in the analytical-platform-development account with Admin access
[x] Roles created in the analytical-platform-data-production with ReadOnly and SecurityAudit access
[ ] Roles exclude PNC data from visibility
[ ] Dev Control Panel users created/auth confirmed
[ ] Prod Control Panel users created/auth confirmed
[x] User level in prod CP: User
[x] User level in dev CP: Admin
[x] Comms channel agreed
[x] Standup scheduled
[x] Leadership briefed
[x] Team briefed
[x] Story to cover support raised
[x] Project Planner confirmed

Waiting on Github usernames.

Four github users added to prod (2 promoted to superadmin)
Four github usernames added to dev (2 promoted to superadmin)


Can you add an iOS Demo ?

Hi, is possible to add an iOS Demo target in this project ?
I tried to use your framework to create a simple iOS project,
but always got "EXC_BAD_ACCESS" when glDrawElements called

problem solved, I should new VAO, VBO after setup EaglContext

I'm glad you figured it out. But as per your suggestion, I created a demo application anyway, so that it might help anyone else trying to use the project.


Remove re-declarations of MTRDeviceController public API on MTRDeviceController_Concrete.

We should just override whichever selectors we want to override here, instead of re-declaring them all.

Fast-tracking platform-specific fix with platform owner review.


FIXED: Search View on Orientation Change #122

App On Orientation Change,Search View and Search Text not Retained #122 ,FIXED

@opticod resolve conflicts

@mananwason i have resolved the conflict :smiley: .is there any other issue with the code ?
Thank You.

@opticod There is a common problem with all the activities. Lets get it sorted out :)

@opticod are you working on this?

@mananwason ,yes i am.

:+1:

@creativepsyco done with some earlier  corrections..
Currently search is not incorporated in bookmark fragment...should i add it or its not needed and remove the icon ?

yeah better to remove it.
We can merge it once you are done :+1: Thanks

@creativepsyco done...removed search from menu...and corrected the share functionality(like..track fragment)

Awesome @opticod. Merging this then :grinning:


cmd/compile/internal/ssa: incorrect type width when folding widening ops

Go version
tip
Output of go env in your module/workspace:
GOARCH=ppc64le

What did you do?
Further exploration of lower rule improvements has uncovered problems when using a Value Type's size to determine whether to apply a rule.
E.g the rule in PPC64.rules:
(MOVWZreg y:(AND (MOVDconst [c]) _)) && uint64(c) <= 0xFFFFFFFF => y

And later in PPC64latelower.rules:
(AND <t> x:(MOVDconst [m]) n) && t.Size() == 4 && isPPC64WordRotateMask(m) => (RLWINM [encodePPC64RotateMask(0,m,32)] n)
If m is the value 0xF000000F or a similar wrapping mask on PPC64, the upper 32 bits of the value are not always 0. I think this only affects rules which use RLWINM and RLWNM. I believe they are the only GPR opcodes on PPC64 which have this behavior.
What did you see happen?
The latelower rule is triggered in cases where the result is expected to be wider than the assigned type
What did you expect to see?
The Type of the result in lower should reflected the widened type.

Similar Issues

https://github.com/golang/go/issues/67844

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I suspect this is not causing problems today. 32 bit constants are sign extended, and thus any wrapping mask would produce a negative int32 constant, which prevents the AND rule above from running for such constants.


Merging a pull request creates a negative issue count

We DO NOT take questions or config/deploy problems on GitHub, please use our forum: https://discuss.gogs.io
Please take a moment to search that an issue doesn't already exist.
For bug reports, please give the relevant info:

Gogs version (or commit ref): <IP_ADDRESS>7
Git version:
Operating system:
Database:

[ ] PostgreSQL
[X] MySQL
[ ] SQLite

Can you reproduce the bug at http://try.gogs.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Whenever I merge a pull request the issues from this repository becomes negative. (Note there where no issues before).
Basically 8 commits and 3 where merged pull requests but now the issues looks like this:

...

+1
Exact same version, also MySQL.

Duplicate of #3159

it also happens with postgres

I can not reproduce this on develop-branch

MariaDB [gogs]> SELECT num_issues, num_closed_issues, num_pulls, num_closed_pulls FROM repository WHERE id=25;
+------------+-------------------+-----------+------------------+
| num_issues | num_closed_issues | num_pulls | num_closed_pulls |
+------------+-------------------+-----------+------------------+
|         51 |                70 |        67 |               67 |
+------------+-------------------+-----------+------------------+

https://github.com/gogits/gogs/blob/e9ae926e040c94ce66666ae6c330c93918fa321c/models/repo.go#L204
NumOpenIssue (used to calculate the issue count) = num_issues - num_closed_issues
num_issues contains the list of issues that ARE NOT pull request. The problem is, here, we set num_closed_issues as all the closed Issues (including PR-issues), which leads to NumOpenIssue being miscalculated.

#3207

This is fixed in https://github.com/gogits/gogs/pull/3207


Does not work with linter any more

I noticed that my parsed errors were not showing up in the linter any more. I believe this is due to changes in the linter API.

I know that the linter API changed, but it's still working on my machine, so I didn't update my code.
Which profile do you use ?
Atom version, Linter version, build-tools version ?

My mistake. In the linter settings it was configure to ignore VCS ignored files and the particular file that i was using to test was VCS ignored! Now that I changed that setting all is well. Thanks.


Writing to usb controller does nothing until manually echoing

This is quite strange – I thought the code I was using to send a simple signal, any signal, to usb controller (whose function is to open a cash drawer) wasn't working.
This is on an ubuntu machine, and this is the code in question (in typescript):
openDrawer(d: usb.Device) {     d.open();     const iFace: usb.Interface = d.interface(0);     // detach kernal driver if necessary     if (iFace.isKernelDriverActive()) {         iFace.detachKernelDriver();     }     iFace.claim();     // I could write a loop to specifically find the OutEndpoint for this device, but I know it is at position 1     const outEndPoint: usb.OutEndpoint = iFace.endpoints[1] as usb.OutEndpoint;     outEndPoint.transfer(new Buffer(1), (err) => {         console.log('transfer finished');         if (err) {             console.warn('TRANSFER ERROR', err);         }         iFace.release((err) => {             if (err) {                 console.warn('RELEASE ERROR', err);             }             console.log('reattaching');             iFace.attachKernelDriver();             d.close();         });        }); }
No errors, but also no cash drawer opening.
I wanted to check that the usb controller was actually running, and after checking that the device could be found at /dev/ttyUSB0. So I tried the following just to check the drawer was working:
sudo chmod o+rw /dev/ttyUSB0 sudo echo '1' > /dev/ttyUSB0
And the drawer did indeed open.
But what's weird now is that the typescript code above was working now too! I could reliably open the drawer by running that function. Somehow echoing the '1' to /dev/ttyUSB0 "unlocked" that device for typescript code to work. Unplugging the device and plugging it back, however, returned it to its dormant state. Same for restarting the machine.
Any idea why this might be? If you want stackoverflow points, I've put the question there too:
https://stackoverflow.com/questions/58043799/cant-write-to-usb-device-using-node-usb-until-manually-doing-so-using-echo

If your device is a USB serial port, you should use node-serialport to use the kernel serial drivers instead. With node-usb, you'd have to implement the FTDI or CDC-ACM or whatever protocol to configure the serial port before writing to the device. When you are writing to the port through the kernel driver, it is performing the initialization for you. This library is intended for implementing drivers for USB devices in Node, but since the kernel already has a driver for your device, you should use a library that interfaces with that driver.


Offload tile fetching in websockets to separate worker processes for performance

Deploy using gunicorn
Planning to use zeromq for load balancing

Will fix with Girder.


AND-7419 - Improve permission error message

Hi @rgcorrea, thanks for the pull request. Before we can merge it, we need you to sign our Contributor License Agreement. You can do so electronically here: http://opensource.box.com/cla
Once you have signed, just add a comment to this pull request saying, "CLA signed". Thanks!

+1


Users with 'View Processors' permission only, see the error 'User does not have the required permission (Manage Processors)' when  trying to open a 'Processors' tab

'View Processors' is no longer a valid permission, see issue #235.


Enable Consistent Data Push for Standalone Segment Push Job Runners

Description:
This PR addresses https://github.com/apache/pinot/issues/9268 for the segment push job runners under the standalone execution framework: SegmentMetadataPushJobRunner, SegmentTarPushJobRunner, and SegmentUriPushJobRunner.
This is accomplished by introducing a new class ConsistentDataPushUtils which contains APIs and helpers for *PushJobRunner(s) to call to invoke the consistent push protocol.
Since there are large overlaps in the code for all of the *PushJobRunner(s), also took this opportunity to refactor and extract the common logic out to BaseSegmentPushJobRunner abstract class.
To enable consistent data push, this PR also introduces a new boolean config in table config under
TableConfig->IngestionConfig->BatchIngestionConfig->consistentDataPush.
Users can enable consistent data push by setting the consistentDataPush config to true as below before invoking ingestion jobs,
...
    "batchIngestionConfig": {
      "segmentIngestionType": "REFRESH",
      "segmentIngestionFrequency": "DAILY",
      "consistentDataPush": true
    },
...

which will

In the segment generation phase: inject timestamps to segment names (via segment postfix) in order to prevent segment name conflicts from overwriting existing segments directly for the REFRESH usecase and
In the segment push/upload phase: wrap the segments replacement protocol around segments upload which will achieve atomic switching between old and new segments data. In the case of failure, revert and abort the swap, which will ensure broker only routes to the old segments.

Testing Done:
Added new test testUploadAndQueryWithConsistentPush in SegmentUploadIntegrationTest, which

Runs SegmentMetadataPushJobRunner with consistent push enabled.
[] -> [v1 segments]

Checks that the segment lineage entry is in expected and completed state.

Checks that count stars return expected outputs.

Runs SegmentTarPushJobRunner with consistent push enabled.
[v1 segments] -> [v2 segments]

Checks again that the segment lineage entry is in expected and completed state.

Checks again that count stars return expected outputs (that we have successfully bulk replaced the original set of segments).

Codecov Report

Merging #9295 (27443ab) into master (5ecca80) will increase coverage by 13.10%.
The diff coverage is 11.24%.

@@              Coverage Diff              @@
##             master    #9295       +/-   ##
=============================================
+ Coverage     15.28%   28.38%   +13.10%
+ Complexity      168       53      -115
=============================================
  Files          1814     1863       +49
  Lines         97379    99392     +2013
  Branches      14893    15146      +253
=============================================
+ Hits          14880    28215    +13335
+ Misses        81367    68427    -12940
- Partials       1132     2750     +1618

Flag
Coverage Δ

integration1
26.22% <11.24%> (?)

integration2
24.77% <0.00%> (?)

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

 Learn more

This is a great feature! Can you please help also update the pinot documentation about this new feature? https://github.com/pinot-contrib/pinot-docs

+1 to @Jackie-Jiang, I think this commit breaks the quickstart for batch import: https://docs.pinot.apache.org/basics/getting-started/pushing-your-data-to-pinot


Update PythonMap transform to accept extra packages

This fixes #25221

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run XVR_JavaUsingPython_Dataflow PostCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

R: @ahmedabu98

Run XVR_JavaUsingPython_Dataflow PostCommit

Run Java PreCommit

Run XVR_JavaUsingPython_Dataflow PostCommit

Thanks!


Fixed subtitle positions to match FFmpeg

FFmpeg uses the composition position - a field I was currently ignoring. This lib used the window position to place the subtitle image. This PR changes the placement to match FFmpeg.

Fixes #14


Tweak design and technology subject name

Hint at presence of other DT types: food, product design, textiles, and systems and control
https://trello.com/c/uEIhWX8V/1171-tweak-design-and-technology-description-on-subjects-picker

I don't understand Razor.
Views/Filter/Subject.cshtml(59,52): error CS1012: Too many characters in character literal


[question] - Issue with language server

Hi Syntax Tree,
Heard about you on Remote Ruby, getting below after I installed the gem.
Ruby 2.5.0
Ruby Version: 2.5.0
VS Code: Version: 1.66.0 (Universal)
Starting language server...
No bundled syntax_tree, running global stree.
/Users/chriswade/.asdf/installs/ruby/2.5.0/lib/ruby/gems/2.5.0/gems/syntax_tree-2.0.1/lib/syntax_tree.rb:8:in `require_relative': /Users/chriswade/.asdf/installs/ruby/2.5.0/lib/ruby/gems/2.5.0/gems/syntax_tree-2.0.1/lib/syntax_tree/formatter.rb:12: syntax error, unexpected ... (SyntaxError)
    def initialize(source, ...)
                           ^~~

Ruby 2.7.4
Starting language server...
No bundled syntax_tree, running global stree.
/Users/chriswade/.asdf/installs/ruby/2.7.4/lib/ruby/gems/2.7.0/gems/syntax_tree-2.0.1/lib/syntax_tree/language_server.rb:28: warning: Pattern matching is experimental, and the behavior may change in future versions of Ruby!
/Users/chriswade/.asdf/installs/ruby/2.7.4/lib/ruby/gems/2.7.0/gems/syntax_tree-2.0.1/lib/syntax_tree/language_server/inlay_hints.rb:69: warning: Pattern matching is experimental, and the behavior may change in future versions of Ruby!
Requesting implicits
Requesting implicits
/Users/chriswade/.asdf/installs/ruby/2.7.4/lib/ruby/gems/2.7.0/gems/syntax_tree-2.0.1/lib/syntax_tree/language_server.rb:54:in `run': Unhandled: {:jsonrpc=>"2.0", :id=>1, :method=>"syntaxTree/implicits", :params=>{:textDocument=>{:uri=>"file:///Users/chriswade/Exercism/ruby/sieve/sieve.rb"}}} (RuntimeError)
	from /Users/chriswade/.asdf/installs/ruby/2.7.4/lib/ruby/gems/2.7.0/gems/syntax_tree-2.0.1/lib/syntax_tree/cli.rb:187:in `run'
	from /Users/chriswade/.asdf/installs/ruby/2.7.4/lib/ruby/gems/2.7.0/gems/syntax_tree-2.0.1/exe/stree:9:in `<top (required)>'
	from /Users/chriswade/.asdf/installs/ruby/2.7.4/bin/stree:25:in `load'
	from /Users/chriswade/.asdf/installs/ruby/2.7.4/bin/stree:25:in `<main>'
[Info  - 9:43:20 am] Connection to server got closed. Server will restart.

Ruby 3.0.0
Starting language server...
No bundled syntax_tree, running global stree.
Requesting implicits
/Users/chriswade/.asdf/installs/ruby/3.0.0/lib/ruby/gems/3.0.0/gems/syntax_tree-2.0.1/lib/syntax_tree/language_server.rb:54:in `run': Unhandled: {:jsonrpc=>"2.0", :id=>1, :method=>"syntaxTree/implicits", :params=>{:textDocument=>{:uri=>"file:///Users/chriswade/code/oe-qikpac-monitor/app/models/account_record.rb"}}} (RuntimeError)
	from /Users/chriswade/.asdf/installs/ruby/3.0.0/lib/ruby/gems/3.0.0/gems/syntax_tree-2.0.1/lib/syntax_tree/cli.rb:187:in `run'
	from /Users/chriswade/.asdf/installs/ruby/3.0.0/lib/ruby/gems/3.0.0/gems/syntax_tree-2.0.1/exe/stree:9:in `<top (required)>'
	from /Users/chriswade/.asdf/installs/ruby/3.0.0/bin/stree:23:in `load'
	from /Users/chriswade/.asdf/installs/ruby/3.0.0/bin/stree:23:in `<main>'
[Info  - 9:51:50 am] Connection to server got closed. Server will restart.

Thanks for putting this out :)
Regards,
Chris

Ahh sorry I updated the gem and forgot to update the vscode plugin. v0.2.1 of the plugin is out now and should have the fixes.

Thanks for that,
Still no go for repo's with Ruby 2.5.0 as version. ( image below ), other 2 are working
Also does it need any setup like below that I was using for rubocop ?
I don't seem to get any of the light blue squiggly lines suggesting things like I was with rubocop
Should I change ruby.format & ruby.intellisense to syntax.tree ( i'm guessing this )
{
  "editor.formatOnSave": true,
  "editor.formatOnType": true,
  //ruby start
  "[ruby]": {
    "editor.defaultFormatter": "misogi.ruby-rubocop"
  },
  "ruby.useLanguageServer": true, // use the internal language server (see below)
  "ruby.format": "rubocop", // use rubocop for formatting
  "ruby.intellisense": "rubyLocate",
  "ruby.useBundler": true,
  "ruby.lint": {
    "rubocop": {
      "useBundler": true
    }
  },
  "ruby.rubocop.suppressRubocopWarnings": false,
  "ruby.rubocop.onSave": true,
  "files.associations": {
    "*.erb": "erb"
  },
  "editor.codeActionsOnSave": {
    // For ESLint
    "source.fixAll.eslint": true,
    // For TSLint
    "source.fixAll.tslint": true,
    // For Stylelint
    "source.fixAll.stylelint": true
  },
  // RUBY END

}

Okay, a couple of things. Totally not your fault, the docs are definitely lacking.

First of all, this requires Ruby >= 2.7.3. So your EOL rubys aren't going to work. Sorry about that.
You don't need any setup, except you can set "ruby.format" to false. That configuration option is provided a different extension, so you should turn off its formatting so that syntax tree can take over.
There's no ruby.intellisense option for this plugin, since it's just for formatting.
So you don't get diagnostics with this extension. The function of this extension is just to format, not to lint.


German DBpedia not working

It's been few days. We might have to establish our own instance.

Works for now


React template IE11

React template doesn't work in IE11.
Seems no Promises in IE11.
So how to use Polyfill in this template?
and
Why Polyfill doesn't included into the template ?

We don't include polyfills for all browsers because it would make the applications much larger, and not everybody needs to support legacy browsers.
For an example of including polyfills, please see https://github.com/aspnet/JavaScriptServices/pull/953

Your forwarding link is for

Angular
Not answers the question how to setup it in react template :)

It will work the same in the React template.

Seems, no.
In reactredux template what I need is only
import 'core-js/es6';
This approach doesn't work in react.


Sequence checks missing residue threshold

Hi,
Thanks for making ImmuneBuilder available. For ABodyBuilder2 I tend to get failures for sequences not passing the number_single_sequence check. Specifically, too many missing residues.
Specific line for the error is within ImmuneBuilder.sequence_checks.number_single_sequence:
    # Check for missing residues assuming imgt numbering
    assert (max(numbers) > 120) and (min(numbers) < 8), f"Sequence missing too many residues to model correctly. Please give whole sequence:\n{sequence}"

Question: What is the motivation for this? That missing residues at the tails of the sequence have to be minimized? The assertion statement implies that there is a threshold to how many missing residues are allowed for this.
Code to reproduce
This is the case where both heavy and light chains fail the sequence check under the assertion  (max(numbers)>20)
from ImmuneBuilder import ABodyBuilder2
predictor = ABodyBuilder2()

output_file = "my_antibody.pdb"
sequences = {
  'H': 'QVQLVESGGGMVQPGEVPETLLCSLDSPSVAMAAGSARLQGKGLEWVSVISYEEGRKYYADSVKGRFTISRDNSKNTLYLQMSSLRAEDTAVYYCAEDSEQGWFGDPWGALTSGARDNGHRLS',
  'L': 'DIVLTQTPGTLSLSPGERATLSCRASQSVSSYLAWYQQKPGQAPRLLIYDASNRATGIPARFSGSGSGTDFTLTISRLEPEDFAVYYCQQYGSSPLTFGRRDQGWRS'}

antibody = predictor.predict(sequences)
antibody.save(output_file)

To access the specific error:
from ImmuneBuilder.sequence_checks import number_single_sequence
number_single_sequence(sequence='QVQLVESGGGMVQPGEVPETLLCSLDSPSVAMAAGSARLQGKGLEWVSVISYEEGRKYYADSVKGRFTISRDNSKNTLYLQMSSLRAEDTAVYYCAEDSEQGWFGDPWGALTSGARDNGHRLS',chain="H")
number_single_sequence(sequence='DIVLTQTPGTLSLSPGERATLSCRASQSVSSYLAWYQQKPGQAPRLLIYDASNRATGIPARFSGSGSGTDFTLTISRLEPEDFAVYYCQQYGSSPLTFGRRDQGWRS', chain="L")

Specifically max(numbers) == 120 for both the heavy and light chains. What are the consequences of adjusting the cutoff?

Hi Mary,
The sequence length checks were added because all structures seen by the Immunebuilder models during training were not missing more residues than that. You can try disabling them and the model should still run, but I can not guarantee that the predicted structure will be accurate.
An alternative to this, that is more likely to generate accurate structures, would be to fill in the missing residues in your Fv sequence. This can be done either by assuming they are germline or by using a language model such as Ablang https://github.com/oxpig/AbLang
You would only have to do this for the sequences were the assertion is raised.
I hope that helps!!
All the best,
Brennan


RShiny or Jupyter integration

This was actually requested by a user at one point https://github.com/GMOD/jbrowse-components/discussions/3125#discussioncomment-3366128

Can you be a bit clearer about what specific milestones would be for this integration

I don't have specific milestones set, but I do think incorporating react-msaview into the notebook environment would enable some interesting on the fly analysis workflows (example: on-the-fly alphafold, since alphafold is relatively easy to run on colab notebooks, or just phylogenetic calculations/loading dynamically created multiple sequence alignments)

sort of similar to https://github.com/GMOD/react-msaview/issues/30

Sure


[ZEPPELIN-4585] Add HTTP Cache-Control Header

What is this PR for?
A few sentences describing the overall goals of the pull request's commits.
First time? Check out the contributing guide - https://zeppelin.apache.org/contribution/contributions.html
What type of PR is it?
Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-4585
How should this be tested?

Travis-ci should pass
To test the change:

curl -i localhost:8080/api/version
HTTP/1.1 200 OK
Date: Fri, 31 Jan 2020 19:09:04 GMT
Access-Control-Allow-Credentials: true
Access-Control-Allow-Headers: authorization,Content-Type
Access-Control-Allow-Methods: POST, GET, OPTIONS, PUT, HEAD, DELETE
X-FRAME-OPTIONS: SAMEORIGIN
X-XSS-Protection: 1
X-Content-Type-Options: nosniff
Cache-Control: no-cache
Content-Type: text/plain
Content-Length: 144
Server: Jetty(9.4.18.v20190429)

{"status":"OK","message":"Zeppelin version","body":{"git-commit-id":"b4fc38a","git-timestamp":"2020-01-31 14:27:00","version":"0.9.0-SNAPSHOT"}}%

Questions:

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

I'm not 100% sure that Cache-Control belongs to the CORS filter. Maybe we need to check, how it will affect the serving of the static assets, for example, from Helium bundles. Maybe it makes sense to add cache control as a separate filter that would distinguish between "static" and "dynamic" assets?

@alexott makes sense to me. I will check that.


upgrade react-native-fast-image to 5.1.2, IOS build error

follow file cannot be found
node_modules/react-native-fast-image/ios/Vendor/SDWebImage/Vendors/libwebp/src/dec/idec.c
node_modules/react-native-fast-image/ios/Vendor/SDWebImage/Vendors/libwebp/src/dec/webp.c

I believe this is fixed in the latest version. Try it out and let me know how it goes!


[Question] Execute PipelineBehavior only to specific types of request

Can I execute pipeline matching a specific Interface to avoid the test like this?
https://gist.github.com/thiagoteles/4f6776e23660e5eba168fb9a1916683e

You could do
where TRequest : IRequest<TResponse>
and then inside Handle(...)
if (request is ITestReadContext) {.. do stuff ..} return next();
not as nice but should work?


Use of Stage.js

Hi,
I had a chance to play with Planck.js and Stage.js, but separately.
I'm not js guru. In the last few days I decided to use the library without the testbed.
My first choice was to replace it with Stage.js.
I follow box2d manual hello world example, nothing super ambitious.
I have a bouncing ball on a flat surface/edge. Now I want to use Stage.js to render that on the screen.
It seems like all examples are testbed examples. Even though it supposed to be only a case of calling the world.step(timestep) function, I'm still struggling as I assume there will be something similar needed on the Stage.js side.
Would it be possible to post an example that uses Stage.js? Something really simple.
Thank you.
Chris

Thank you for your suggestion. I tried few things and at one point I thought everything was working as expected but after adding the first texture I noticed that it was upside down. Of course I could use -y scale but there must be a better way.
Below is the code extracted from the asteroid example. I know I can flip everything just by -y scaling the stage node. How would you attach the world element to a new element? Do you mean to new node (node = Stage.create())? I'm sure I'm doing something wrong, I tried to -y scale the meta element just to see if this would turn the status text upside down but that didn't work.
Stage(function(stage) {
  var activeKeys = {};
  var KEY_NAMES = {
    32 : 'fire',
    37 : 'left',
    38 : 'up',
    39 : 'right',
    40 : 'down'
  };

  var physics = new Physics({
    updateStatus: updateStatus,
    activeKeys: activeKeys
  });

  var world, meta, status, box, gameView;

  stage.background('#222222');
  //stage.scale(1,-1); // this one is fine for turning the entire scene upside down
	stage.on('viewport', function(size) {
console.log("test");
    meta.pin({
      scaleMode : 'in-pad',
      scaleWidth : size.width,
      scaleHeight : size.height,
    });
   world.pin({
      scaleMode : 'in-pad',
       scaleWidth : size.width,
       scaleHeight : size.height,
    });
   });

	world = new Stage
		.planck(physics.world, { ratio: 80 });

    world.pin({
      handle : -0.5,
      width : physics.spaceWidth,
      height : physics.spaceHeight,
    });
    world.appendTo(stage);

	stage.tick(physics.tick);

	meta = Stage
    .create()
    .pin({ width : 1000, height : 1000 })
	//.scale({x : 1, y: -1})    // doesn't work
    .appendTo(stage);

   	var ball = Stage.image('ball').appendTo(meta);

	ball.pin('align', 0.5);

   status = Stage
   .string('text')
   .pin({ align : 0, offsetX: 0, offsetY : 10 })
   .appendTo(meta);

   function updateStatus() {
	status.value('TEXT: ' + 3 + ' TEXT: ' + 7);
   }

  document.onkeydown = function(evt) {
    activeKeys[KEY_NAMES[evt.keyCode]] = true;
  };

  document.onkeyup = function(evt) {
    activeKeys[KEY_NAMES[evt.keyCode]] = false;
  };

   physics.start();
});

My last suggestion was not good, now I recall for P2.js viewer I actually multiplied all y values from physics simulation by -1, for example see here.

I updated the viewer, please feel free to try it and let me.

Another solution is to just setup the game world with y pointing down.

If shakiba would just listen to what I have to say on my issue.

@prolightHub what are your thoughts?


How to add additional classpaths / libraries?

Is there a possibility to add additional classpaths, for example if I want to use external libraries? I now someone asked a similar question and the issue was closed, because we should ask the language server guys. But it should exist a possibility to forward arguments to the actual server from within the UI (not only JVM arguments). Also it would be awesome to set different classpaths for different projects :-)

Any ideas? :-)

Maybe adding in the .classpath file

Indeed it was able to do it by altering the .classpath file. However it would be still awesome if there was a way to do it inside the IDE (similar to Netbeans, etc...)


Subscriptions: RabbitMQ provider

First version of RabbitMQ PubSub. Implementation of the PubSub should be final. Still working on tests as RabbitMQ cannot run within .NET nor is there any premade mock (compared to Redis).
@michaelstaib How are you doing this kind of tests?

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@michaelstaib I have inspected the issue reported by code analyzer, it seemst that it is caused by tests in namespace StrawberryShake.CodeGeneration.CSharp.Analyzers.CustomScalar as there is
public static class DependencyInjection
{
    public static void Configure(IServiceCollection services)
    {
        services.AddCustomScalarClient();
    }
}

But method AddCustomScalarClient does not exist. Is si supposed to get generated? If so, it does not happen.

@Deathrage I am working on the code gen issue with strawberry shake ... so no worries. It started to happen when we moved to .net 7.

We have reworked the subscription system and implemented this driver now with a separate PR.


make _acme-challenge. subdomain configurable

I think this is the relevant place to post this...
Some DNS systems, such as the ones people would use acme-dns instead of, don't allow a leading underscore in CNAME records.
why? a leading underscore is prohibited in hostnames, but allowed in other dns records. Some systems are trying to safeguard users, others are confused in implementations, and others... who knows.
anways, _acme-challenge. is hardcoded here:
https://github.com/joohoi/acme-dns-certbot/blob/1873fd105a72544173a3461237dcc1974a06bbb5/acme-dns-auth.py#L25
I believe you can supply an alternate.  there's a bit of a discussion here https://community.letsencrypt.org/t/acme-sh-support-domain-alias-mode-now/52700/5

The hardcoded value is only used for printing the user instructions here.
The main issue is that in this case these DNS providers do not allow Let's Encrypt DNS challenges at all. That's because Let's encrypt will always ask for the _acme-challenge.domain-to-be-validated.tld record, and this cannot be changed on the client side.

Oh, to clarify: the _acme-challenge goes to the CNAME record name, as this is the record that Let's Encrypt is checking. Acme-dns generated subdomains do not have this prefix at all, so you won't be creating a CNAME to a hostname starting with an underscore.

Ok, then this shouldn't be an issue with most "bad" DNS hosts.
The issue was with other libraries that originally required the cname to be onto an "_acme-challenge" prefix.
I believe the underscore is usually valid for the key but not the value in the cname record on the providers I found.

Awesome! Hope you can work with this information. I'm going to close this issue for now, but please comment here if needed, and I'll reopen.

Thanks for the quick replies! I haven't even dug into using this package yet, a quick glance suggested it had the same limitation as another system (which required the CNAME to be set up a special way). Looking deeper at the acme-dns package, it doesn't have that limitation at all.


Change type returned by the backend service

Signed-off-by: Jakub Dzon<EMAIL_ADDRESS>

/approve

/lgtm

/approved


Fixed printf that starts with minus sign in clientStat.sh

At line 53 of clientStat.sh there is a printf that starts with minus sign
 53           printf "- %s\n" "${array[9]}"
When pivpn clients command is executed the minus sign is interpreted as an option and the script fails by not printing the connection time.
I fixed the issue adding -- before the format string to disarm the minus sign.
 53           printf -- "- %s\n" "${array[9]}"
Hope this helps.
Thanks
Bye.

On my system the output with my fix is the following:
Dec 1 2022 - 00:14:15
If I remove "${array[8]}" and "${array[9]}" the output is wrong.
This is my output with your fix:
Thu Dec
I am on stable now. I need to switch on testing to verify if there are differences.

hmm crap ... don't think changing from stable to test will change anything, it seems the openvpn-status.log saves data based on some regional settings which affects the position of the array where the data ends up =/  this kind of "parsing" is always messy =/
then I say it might be better to just remove the dash, and some day if i have the time I might try to find a better way to parse the log file.

Ok. If you agree I will adjust the fix removing the dash on line 53:
printf " %s\n" "${array[9]}"
without touching array order. If some are empty I don't think will be an issue and at the same time we will not lose information on other regional settings.
I will do it tomorrow as it is too late for me now.
Thanks

yeah its fine, just remove the dash.
On my case (using aws ec2 instnace), it was actually returning meaningless data.

Done, thanks a lot :D

:tada: This PR is included in version 4.1.6-test.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Configuration to increase session timeout for the web-app

It would be good to be able to increase the existing session timeout. With the current timeout there were a few occasions where I got logged out during documentation. This resulted in the written text being deleted.

Hi @bmy-ashampoo
Are you being timeout while working or in the ideal condition?
Also can you add more details here like: OMD version, Which SSO, How did you deployed OM currently

@chirag-madlani
It happened after I came back to the OM browser-tab, maybe after 60 minutes. After thinking about it, the problem might not be the timeout itself but that I was logged out after I clicked the save button for the documentation. If I was logged out when returning back to the browser-tab, there would be no problem.
OM version is 0.13.0 and we use basic authentication with mail and password. We deployed it in a AWS environment with ECS, RDS and OpenSearch.

Hi @bmy-ashampoo
With 0.13.2 we have added some improvements.
When you come back to use OMD application we will verify the token and if it's not valid anymore we will ask user to signin again. Let me know if this feature is helping you any way

Hi @chirag-madlani,
that sounds good! The remaining question would be what happens to the already written text when I am promted to sign in again? Will I at least be able to copy it to my clipboard?

Can you share the page where you are editing?
I will see the option to retain it

This would be an example. If I return to the OMD application it would be good to somehow copy this to my clipboard before I will get the promt to sign in again.

@harshach @mohityadav766 Can we Provide and option to configure longer session for Basic auth at least?

@chirag-madlani it's already there in config  , or you mean from the UI?

Hi @bmy-ashampoo
Can you check if this variable will work for you? https://github.com/open-metadata/OpenMetadata/blob/7eba3e6b4a047ed167a0b602d081caca7e6c68f7/conf/openmetadata.yaml#L313

Closing this. Let us know if you have any doubts here


In profile page, Top 5 Movies button/scrollbar does not show anything

The Top 5 Movies are already there in Preview section below
Maybe we can add a preview of available lists here

Hey @mehul-m-prajapati it will show scrolls, once you add more lists, also top5 movies is not clickable, that is a bug.
@daccotta @AshuKr22

@mehul-m-prajapati @shubhagarwal1  no clue about the top5 bug , would one of you fix it ?

@AshuKr22 : I think we should some pop up so that Users can see the list from scrollbar.
Currently, if you have only one item in list and you hover on it nothing happens.

@mehul-m-prajapati @shubhagarwal1 no clue about the top5 bug , would one of you fix it ?

Hey @AshuKr22 i can solve, the clickable nature of the top5 movies this please assign this to me.


unity3d 2017 support

Current version not work with unity3d 2017.

Use https://github.com/Perfare/UnityStudio


⚠ Home 主页 has degraded performance

In 7f2319b, Home 主页 (https://www.fangkehou.eu.org) experienced degraded performance:

HTTP code: 521
Response time: 20438 ms

Resolved: Home 主页 performance has improved in f228a5c after 46 minutes.


Support for Laravel 8

Is it possible to get support for Laravel 8?

We just released the version 1.11 that should be compatible with Laravel 8.
Could you please confirm that it works smoothly?
Thanks for the heads up!

A fresh installation of Laravel 8 require Guzzle v7
tio/laravel v1.11 requires guzzlehttp/guzzle ^6.3 -> found guzzlehttp/guzzle[6.3.0, ..., 6.5.x-dev] but it conflicts with your root composer.json require (^7.0.1).

Sorry about that!
We relaxed the Guzzle dependency and made sure that was not an issue anymore in Version 1.12

Thanks.
Works on a clean install of Laravel 8.
But if you install Laravel Jetstream, neither init nor sync works.
Error Message: InvalidArgumentException Unable to locate a class or view for component [jet -....].
None of the jetstream components need to be searched.

Works on a clean install of Laravel 8.

Great! Thank you for your useful feedback 😄

Error Message: InvalidArgumentException Unable to locate a class or view for component [jet -....].

It seems that the component called is not present in its correct directory (see here).
Are you sure that this error is related to this package? Does the app works correctly without triggering the same error?

None of the jetstream components need to be searched.

By default, the whole app and resources folders are searched for GetText strings: https://github.com/translation/laravel/blob/master/config/translation.php#L28
You can override this configuration to only allow some of your directories, but it should not trigger any error anyway.

Yes, the app works as intended.
It is a new laravel installation with the extension for jetstream. (laravel new [name] --jet).

I reopened this issue and we'll investigate on what went wrong. Thanks again for your useful feedback.
I'll keep you in touch soon.

We located the error and created a pull request to fix it: https://github.com/php-gettext/Gettext/pull/261
I hope that it will make sense be merged soon.
In the meantime, you can fix this by updating the gettext_parse_paths to avoid blade files with jet components.
Thanks for your help!

Works on a clean install of Laravel 8.

Great! Thank you for your useful feedback 😄

Error Message: InvalidArgumentException Unable to locate a class or view for component [jet -....].

It seems that the component called is not present in its correct directory (see here).
Are you sure that this error is related to this package? Does the app works correctly without triggering the same error?

None of the jetstream components need to be searched.

By default, the whole app and resources folders are searched for GetText strings: https://github.com/translation/laravel/blob/master/config/translation.php#L28
You can override this configuration to only allow some of your directories, but it should not trigger any error anyway.

I noticed that when i use components namespaces https://laravel.com/docs/8.x/blade#manually-registering-package-components
like here <x-client::page.title />  it gives me this error Unable to locate a class or view for component

This seems to be the same issue that we fixed in this pull request: https://github.com/php-gettext/Gettext/pull/261
I hope it will be merged soon.

Cool.. i'll watch it Michaël . Thanks!

php-gettext/Gettext#261 was merged and released.
I already bumped this package to force the new version. So if you update it, it should finally work without parsing errors for Component Tags 🎉