Restructuring documentation

As per feedback session:

Moving CNI documentation to docs/ folder
Flattening Plugins directory, setting menu architecture as a tree
Copy edits throughout
Edit on Github and breadcrumbs bug fixed

fixes #30
fixes #35
fixes #36
fixes #37
fixes #37
Deploy preview: https://deploy-preview-43--cni.netlify.app/

Deploy preview: https://deploy-preview-43--cni.netlify.app/

@celestehorgan, @zacharysarah 👋 Restructuring this site into a simpler, flatter structure has sorted the breadcrumbs issue #30. (We may still want to make some edits to allow the breadcrumbs to show up on top level pages, but I'm not super concerned about that right now as those pages tend to be about adding structure to the content below)


feat: handling DTOs in generic controller

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

Close Issue(s)

So, looking at the code there are a few problems I see.

The idea of a repository is to be the layer where data persistence is handled. The pattern you are suggesting basically overrides the separation.

The data persistence layer should handle id creation, and ensure there is no collision. The handling in code doesn't ensure there are no collisions in the persistence layer, and it's additionally way less performant.

The update operation logic is inherently not performant. A database allows you to update partial values without previously retrieving the value.

The problem we have is, IMO, a problematic design of the repository Interface.

The problem we have is, IMO, a problematic design of the repository Interface.

The points you raise are valid but, IMO, repository design varies, repositories as defined in DDD get a domain model, not a dictionary-like data structure.
Since the repositories in Litestar are more aligned towards CRUD, maybe the interface doesn't suit having a domain model passed to the methods.

The points you raise are valid but, IMO, repository design varies, repositories as defined in DDD get a domain model, not a dictionary-like data structure.

On is this based? DDD as far as I am familiar with it, is not a specification. The interface for repository methods is arbitrary.
In either case - I don't think this PR here can proceed. We have to address the repository interface we have, since its not workable as such.

Yes it's not a specification and of course there is no one correct way.
The idea is that a domain model is either created or loaded from the db then modified. This domain model should in theory be rich in behaviour (or else it's just CRUD), after that, it's passed to the repository that persists the model in an abstracted way.
Your design implies that primitive validated data is passed to the repository and the repository will create/update. I'm with you that this is the way to go for CRUD.
I will close this and will try to finish reviewing your PR asap.


Add field "firstReported" in response structure of API.

Regarding issue covid19india/covid19india-react#337. I added one more field which helps to populate firstReported field in the district_wise data.

@strikertanmay Can you help me curating an object (which will be constant) to display first reported for every state?

@strikertanmay Can you help me curating an object (which will be constant) to display first reported for every state?

Sorry but I do not understand very clearly. Can u please elaborate some more. Do you need an extra object for this or I have to modify some?

Sorry i commented on wrong Issue/PR

Sorry i commented on wrong Issue/PR

Okay but regarding this issue, this PR will help to populate firstReported for each district. For state, you need a constant object. I can help to make a constant object as well. Please tell what type of object structure u want !!

@strikertanmay  Hi Tanmay, I have developed this entire page using Bootstrap and React.js also added components to check live data. Can we connect once?

@strikertanmay Hi Tanmay, I have developed this entire page using Bootstrap and React.js also added components to check live data. Can we connect once?

Yeah sure we can connect.

Can we connect 10 pm IST ?
On Tue, Mar 31, 2020 at 12:13 AM Tanmay Mittal<EMAIL_ADDRESS>wrote:

@strikertanmay https://github.com/strikertanmay Hi Tanmay, I have
developed this entire page using Bootstrap and React.js also added
components to check live data. Can we connect once?
Yeah sure we can connect.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/covid19india/api/pull/38#issuecomment-606404287, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AG2P64ADD6I4W3N2ZC6DVGDRKF3YXANCNFSM4LWOH7VA
.

@strikertanmay I am new to backend stuff but how do you run API in local server?

@strikertanmay I am new to backend stuff but how do you run API in local server?

For fetching data through JSON files u can directly fetch it without running anything. And for running js scripts, you just have to do node filename.js.

Thanks

There might be date mismatch in the initial reported cases. It would be wrong to add them as first reported case to the api.


Improve DecommitFreePages

Reduce number of kernel breaks on main thread.

@jianchun @leirocks anything else?

:shipit:

retargeted to 1.6

@dotnet-bot test OSX static_osx_osx_test please

@dotnet-bot test OSX static_osx_osx_test please

@jianchun Thanks for the review


BNode in the Ontop native mapping language

Currently, one cannot specify BNode in the .obda files. This is a problem for  bootstrapped mappings generated by ontop bootstrapper.
Actually ontop is already able to handle BNode internally. Most of the RDB2RDF compliance tests with BNode in R2RML and DM scenarios passed.
So what's missing is just a syntax for BNode in the .obda syntax and of course the parser for it.

There are two types of BNode syntax in Turtle

(Labeled) RDF Blank Nodes: http://www.w3.org/TR/turtle/#unlabeled-bnodes:

_:alice foaf:knows _:bob .

Unlabeled Blank Nodes http://www.w3.org/TR/turtle/#unlabeled-bnodes

[] foaf:knows [ foaf:name "Bob" ] .

Following Turtle, we will also have two syntax for BNode templates:

Labeled RDF Blank Node Templates:

_:{id}_{age}_{name}

Unlabeled Blank Node Template (actually not a template since no placeholder is involved):

[]

Class needed to be changed:

TargetQueryRenderer: for rendering Ontop Native syntax mappings
R2RMLWriter: for rendering R2RML Mappings
TurtleOBDA.g: for parsing Ontop Native syntax mappings

We are planning to use ontop within our organisation and run into this issue when attempting to bootstrap and convert to r2rml certain SQL databases (I believe tables with composite keys). Will this be included in the beta-3 release? Thanks in advance.

@ghxiao : was it already included in 3.0.1? Shall we close it?

Yes, this is already implemented.


.fla instead of .mp4

After updating youtube-dl downloads .fla files instead of .mp4 from public German TV:
Example:
Source:
http://www.zdf.de/ZDFmediathek/beitrag/video/2666762/Operation-Naked
MP4: http://nrodl.zdf.de/none/zdf/16/02/160209_operation_naked_ps_opn_1456k_p13v12.mp4
Output:
youtube-dl http://www.zdf.de/ZDFmediathek/beitrag/video/2666762/Operation-Naked
[ZDF] 2666762: Downloading video info
[ZDF] 2666762: Downloading SMIL file
[ZDF] 2666762: Downloading f4m manifest
[ZDF] 2666762: Downloading m3u8 information
[download] Destination: Operation Naked-2666762.flv

rtmp has higher bitrate according to metadata. youtube-dl downloads best available quality by default. You have to explicitly tell youtube-dl to download mp4 if you want mp4.


A class for a PHYX study

Proposed term: PHYXStudy (class)
Subclass of: document
Definition: A study consists of a set of phyloreferences as well as a set of phylogenies that can be used to test those phyloreferences. A PHYX file consists of a single PHYX study and are usually derived from a single publication or a small number of publications (e.g. if the phyloreferences are published in one paper based on phylogenies published in another).
Competency questions:

Which phylogenies can be used to test a particular set of phyloreferences?

Currently referred to as a PhyloreferenceTestCase, but I think "PHYX Study" is a better term.

Is this different from an attribute connecting a phyloreference instance to a phylogeny?

I see this as a representation of the PHYX study itself in RDF, though perhaps it's not necessary to define it in OWL. Instances of this class would assert that a group of phylogenies and a group of phyloreferences were grouped together by an author, and would have annotations reflecting authorship, publication status, and so on. In which case, a better competency question might be, "Which phylogenies and phyloreferences are included in a particular study?". This isn't very useful for the Ontology of Phyloreferences, though, so in retrospect it probably isn't worth including in the Phyloref Ontology either.
We do need a term for asserting that a curator expects a Phyloreference to resolve to a particular Node (#26). If we had such a term, then the Tree within which that Node is located must be a reference phylogeny for the phyloreference, and so would act as an attribute connecting a phyloreference instance to a phylogeny. I think that would be useful to include in the Ontology of Phyloreferences.

You are using Ontology of Phyloreferences in two places above, but we have obsoleted this term because it is confusing. To the point, I think in the first instance you mean Clade Ontology, but it seems in the second instance you may actually mean Phyloreferencing Ontology?

And why can't we simply use IAO:document? I think to justify being defined as a subclass of that, the definition will have to follow the genus-differentia pattern. I.e., An instance of PHYXStudy is an IAO:document that ..., followed by qualifying how PHYXStudy documents can be distinguished from all other IAO:document instances. Then this still needs to be followed by an argument as to how that differentia is valuable to and matters for our goals.

I'm unclear what the open issue here is. It seems that the relationship between a phyloreference and a study from which it was curated is one of provenance (such as prov:wasDerivedFrom) (see #16), and that between a phyloreference and a reference phylogeny is one of supporting evidence (see #15).

The original idea here was to model the PHYX file itself -- as a collection of related phyloreferences and phylogenies -- but linking phyloreferences to the document they were published in with prov:wasDerivedFrom and to their reference phylogenies with has evidence should provide all the information we need. Closing this issue.


Hot reloading support

Sorry to ask question as an issue. Please close after answering.
Does this template has hot reloading support?

Nevermind, I should have read the docs before. Its written that hot relaod is supported.


Using scene.turn_on from API doesn't apply scene correctly

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
latest dev (0.24)
Python release (python3 --version):
3.4.5
Component/platform:
scene
Description of problem:
When I execute the following api call, the scene gets applied, but the lights fade back to the previous state. This doesn't not happen when I do this from the webinterface.
curl -X POST -H "x-ha-access: xxx" http://localhost:8123/api/services/scene/turn_on -d '{"entity_id": "scene.evening"}'

When the scene is activated trough the webinterface, the api/services/homeassistant/turn_on call seems to be used instead and that works correctly. But that doesn't give the entity_id as a param, but the complete scene details.
Expected:
Problem-relevant configuration.yaml entries and steps to reproduce:
- name: Evening
  entities:
    light.links:
      state: 'on'
      xy_color: [0.5071, 0.415]
      brightness: 58
    light.midden:
      state: 'on'
      xy_color: [0.5071, 0.415]
      brightness: 58
    light.rechts:
      state: 'on'
      xy_color: [0.5071, 0.415]
      brightness: 58
    light.sfeerlamp:
      state: 'on'
      effect: 'colorloop'
    light.booglamp:
      xy_color: [0.5071, 0.415]
      brightness: 58
      state: 'on'

Traceback (if applicable):
home-assistant_1  | 16-07-14 00:14:27 INFO (CP Server Thread-13) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_on, service_data=None, service_call_id=140626125915192-71, domain=scene>
home-assistant_1  | 16-07-14 00:14:27 INFO (ThreadPool Worker 5) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_on, service_data=brightness=254, entity_id=['light.rechts', 'light.midden', 'light.links'], xy_color=[0.4595, 0.4105], service_call_id=140626125915192-72, domain=light>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 6) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.rechts, old_state=<state light.rechts=on; rgb_color=(167, 118, 50), friendly_name=Rechts, brightness=58, xy_color=[0.5071, 0.415], color_temp=455 @ 2016-07-14T00:05:55.191012+02:00>, new_state=<state light.rechts=on; rgb_color=(255, 199, 116), friendly_name=Rechts, brightness=254, xy_color=[0.4595, 0.4105], color_temp=455 @ 2016-07-14T00:05:55.191012+02:00>>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 6) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.midden, old_state=<state light.midden=on; rgb_color=(167, 118, 50), friendly_name=Midden, brightness=58, xy_color=[0.5071, 0.415], color_temp=455 @ 2016-07-14T00:05:55.184686+02:00>, new_state=<state light.midden=on; rgb_color=(255, 199, 116), friendly_name=Midden, brightness=254, xy_color=[0.4595, 0.4105], color_temp=455 @ 2016-07-14T00:05:55.184686+02:00>>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 6) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.links, old_state=<state light.links=on; rgb_color=(167, 118, 50), friendly_name=Links, brightness=58, xy_color=[0.5071, 0.415], color_temp=455 @ 2016-07-14T00:05:55.188710+02:00>, new_state=<state light.links=on; rgb_color=(255, 199, 116), friendly_name=Links, brightness=254, xy_color=[0.4595, 0.4105], color_temp=455 @ 2016-07-14T00:05:55.188710+02:00>>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 6) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140626125915192-72>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 5) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_on, service_data=brightness=187, entity_id=['light.booglamp'], xy_color=[0.4186, 0.4182], service_call_id=140626125915192-73, domain=light>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 20) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140626125915192-73>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 5) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_on, service_data=entity_id=['light.sfeerlamp'], effect=colorloop, service_call_id=140626125915192-74, domain=light>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 17) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140626125915192-74>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 5) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_on, service_data=brightness=58, entity_id=['light.rechts', 'light.midden', 'light.booglamp', 'light.links'], xy_color=[0.5071, 0.415], service_call_id=140626125915192-75, domain=light>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 11) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.rechts, old_state=<state light.rechts=on; rgb_color=(255, 199, 116), friendly_name=Rechts, brightness=254, xy_color=[0.4595, 0.4105], color_temp=455 @ 2016-07-14T00:05:55.191012+02:00>, new_state=<state light.rechts=on; rgb_color=(167, 118, 50), friendly_name=Rechts, brightness=58, xy_color=[0.5071, 0.415], color_temp=455 @ 2016-07-14T00:05:55.191012+02:00>>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 11) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.midden, old_state=<state light.midden=on; rgb_color=(255, 199, 116), friendly_name=Midden, brightness=254, xy_color=[0.4595, 0.4105], color_temp=455 @ 2016-07-14T00:05:55.184686+02:00>, new_state=<state light.midden=on; rgb_color=(167, 118, 50), friendly_name=Midden, brightness=58, xy_color=[0.5071, 0.415], color_temp=455 @ 2016-07-14T00:05:55.184686+02:00>>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 11) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=light.links, old_state=<state light.links=on; rgb_color=(255, 199, 116), friendly_name=Links, brightness=254, xy_color=[0.4595, 0.4105], color_temp=455 @ 2016-07-14T00:05:55.188710+02:00>, new_state=<state light.links=on; rgb_color=(167, 118, 50), friendly_name=Links, brightness=58, xy_color=[0.5071, 0.415], color_temp=455 @ 2016-07-14T00:05:55.188710+02:00>>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 11) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140626125915192-75>
home-assistant_1  | 16-07-14 00:14:28 INFO (ThreadPool Worker 5) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140626125915192-71>
home-assistant_1  | 16-07-14 00:14:30 INFO (ThreadPool Worker 3) [homeassistant.components.switch.command_line] Running state command: ping -q -W 1 -c 1 tv.local > /dev/null
home-assistant_1  | 16-07-14 00:14:30 INFO (ThreadPool Worker 14) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=sun.sun, old_state=<state sun.sun=below_horizon; friendly_name=Sun, next_setting=2016-07-14T19:49:27+00:00, azimuth=337.61, icon=mdi:weather-sunny, next_rising=2016-07-14T03:44:35+00:00, elevation=-14.59 @ 2016-07-14T00:05:52.803316+02:00>, new_state=<state sun.sun=below_horizon; friendly_name=Sun, next_setting=2016-07-14T19:49:27+00:00, azimuth=337.85, icon=mdi:weather-sunny, next_rising=2016-07-14T03:44:35+00:00, elevation=-14.65 @ 2016-07-14T00:05:52.803316+02:00>>
home-assistant_1  | 16-07-14 00:14:32 INFO (Thread-3) [homeassistant.core] Bus:Handling <Event mqtt_message_received[L]: qos=0, topic=sensor/humidity, payload=46.00>
home-assistant_1  | 16-07-14 00:14:32 INFO (ThreadPool Worker 9) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=sensor.living_humidity, old_state=<state sensor.living_humidity=45.80; friendly_name=Luchtvochtigheid, icon=mdi:water-percent, unit_of_measurement=% @ 2016-07-14T00:13:32.681686+02:00>, new_state=<state sensor.living_humidity=46.00; friendly_name=Luchtvochtigheid, icon=mdi:water-percent, unit_of_measurement=% @ 2016-07-14T00:14:32.711532+02:00>>

Additional info:

You are not setting the correct content type header like the example in the documentation. Curl will then auto set one which is wrong.

@balloob Thanks! Setting the Content-Type: application/json-header seems to fix it. Although it used to work fine in earlier versions, since scripts that used to work fine for over a year now gave this issue, and they never had the Content-Type: application/json-header.

When we switched to a new webstack we got a more strict request parser.
On Wed, Jul 13, 2016 at 10:15 PM, Michaël Arnauts<EMAIL_ADDRESS>wrote:

@balloob https://github.com/balloob Thanks! Setting the Content-Type:
application/json-header seems to fix it. Although it used to work fine in
earlier versions, since scripts that used to work fine for over a year now
gave this issue, and they never had the Content-Type: application/json
-header.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/2522#issuecomment-232558819,
or mute the thread
https://github.com/notifications/unsubscribe/ABYJ2s0dncrlkWiHUdb0UrpGblJ4Kmchks5qVcX7gaJpZM4JL56J
.

--
PaulusSchoutsen.nl
It's nice to be important but it's more important to be nice.


docs: fix spelling and grammar

SUMMARY
Correct spelling of "your" in docs/docs/installation/event-logging.mdx
Add closing bracket in RELEASING/release-notes-2-0/README.md
Fix grammar in docs/docs/frequently-asked-questions.mdx
Other sections were not relevant. Hence, they were removed.

Thanks @fenilgmehta for cleaning up the documentation.


[ZEPPELIN-2928] Hotfix on maven set version functionality

What is this PR for?
Hotfix on a recent commit that removed the version soft-coded variable and introduced hard-coded references to version tag.
What type of PR is it?
[Bug Fix | Hot Fix ]
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2928

How should this be tested?
Run "mvn versions:set -DprocessDependencies=false -DnewVersion=0.8.0-SNAPSHOT-123"
Questions:

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

Hi @zjffdu , @Leemoonsoo
Should be an easy one to get merged. Let me know if anything does not make sense. Cheers.

LGTM, wait for CI pass

Hi @zjffdu , are there known issues on the CI? I'm a bit puzzled.
 Any suggestions?
I have cleaned the cache already, did not work... Thanks

Don't worry. This is known issue tracked in ZEPPELIN-2927.

Missed that bug report. Thanks.
Ok, feel free to merge change or wait for more feedback.

Actually there is one more override missing on zeppelin-jupyter. I'll confirm tomorrow I got them all fixed.

That's correct, please go ahead @necosta

@zjffdu
Done. Ready for final review. Thanks.

LGTM, will merge it if no more comment


Android - systemNavigationBarColor is taken from the incorrect place of the screen

Steps to Reproduce
The example app is available here: https://github.com/jacek-marchwicki/flutter-insets-bug2
Changes to the example project that are necessary to reproduce the issue: https://github.com/jacek-marchwicki/flutter-insets-bug2/commit/74e17edc48c4045e29cfee6524e4f86f979be9f0

  Diff

Change in main.dart:
  @override
  Widget build(BuildContext context) {
    final style = SystemUiOverlayStyle.light;

    return Column(
      children: [
        Flexible(
          child: AnnotatedRegion<SystemUiOverlayStyle>(
            value: style.copyWith(
              statusBarColor: Colors.redAccent,
              systemNavigationBarColor: Colors.redAccent,
            ),
            child: Container(color: Colors.red),
          ),
        ),
        Flexible(
          child: AnnotatedRegion<SystemUiOverlayStyle>(
            value: SystemUiOverlayStyle.light.copyWith(
              statusBarColor: Colors.greenAccent,
              systemNavigationBarColor: Colors.greenAccent,
            ),
            child: Container(color: Colors.green),
          ),
        ),
        Flexible(
          child: AnnotatedRegion<SystemUiOverlayStyle>(
            value: style.copyWith(
              statusBarColor: Colors.blueAccent,
              systemNavigationBarColor: Colors.blueAccent,
            ),
            child: Container(color: Colors.blue),
          ),
        ),
      ],
    );
  }

Index: android/app/src/main/kotlin/com/example/insetstest/MainActivity.kt
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
--- android/app/src/main/kotlin/com/example/insetstest/MainActivity.kt	(revision d3362aabdefa74cc479fc50b8994f00b57b230a8)
+++ android/app/src/main/kotlin/com/example/insetstest/MainActivity.kt	(date<PHONE_NUMBER>000)
@@ -1,7 +1,17 @@
 package com.example.insetstest

+import android.graphics.Color
+import android.os.Build
+import android.view.View
 import io.flutter.embedding.android.FlutterActivity

-class MainActivity : FlutterActivity() {
-
+class MainActivity: FlutterActivity() {
+    override fun onWindowFocusChanged(hasFocus: Boolean) {
+        super.onWindowFocusChanged(hasFocus)
+        if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.LOLLIPOP) {
+            window.decorView.systemUiVisibility = View.SYSTEM_UI_FLAG_LAYOUT_HIDE_NAVIGATION or View.SYSTEM_UI_FLAG_LAYOUT_STABLE
+            window.navigationBarColor = Color.TRANSPARENT
+            window.statusBarColor = Color.TRANSPARENT
+        }
+    }
 }

We need to enable transparent navigation bar. According to (the Android documentation: https://developer.android.com/training/gestures/gesturenav) we need to set:
     window.decorView.systemUiVisibility = View.SYSTEM_UI_FLAG_LAYOUT_HIDE_NAVIGATION or View.SYSTEM_UI_FLAG_LAYOUT_STABLE
     window.navigationBarColor = Color.TRANSPARENT
     window.statusBarColor = Color.TRANSPARENT

Let's split the screen with Column widget with three even containers with different background colors and all wrapped with UI AnnotatedRegion widget with systemNavigationBarColor. Every looks like this with different colors:
  Flexible(
    child: AnnotatedRegion<SystemUiOverlayStyle>(
      value: SystemUiOverlayStyle.light.copyWith(
        statusBarColor: Colors.greenAccent,
        systemNavigationBarColor: Colors.greenAccent,
      ),
      child: Container(color: Colors.green),
    ),
  ),
```

Run the code on an Android device with API level >= 29

Expected results:
OS Navigation Bar background color should change to the color according to the lowest AnnotatedRegion.
Actual results:
OS Navigation Bar background color changes to the color according to the middleAnnotatedRegion.

Actual result
Expected result

Findings
Looks like I found the origin of the issue.
In flutter view.dart, lines 259-286 I see something like this:
    final Offset bottom = Offset(bounds.center.dx, bounds.center.dy - _window.padding.bottom / _window.devicePixelRatio);
    // ...
    SystemUiOverlayStyle lowerOverlayStyle;
    // ...
    lowerOverlayStyle = layer.find<SystemUiOverlayStyle>(bottom);
    // ...
    final SystemUiOverlayStyle overlayStyle = SystemUiOverlayStyle(
    // ...
      systemNavigationBarColor: lowerOverlayStyle?.systemNavigationBarColor,
    // ...
    );
    SystemChrome.setSystemUIOverlayStyle(overlayStyle);

So looks like the
Looks like bounds.center.dy - _window.padding.bottom / _window.devicePixelRatio is incorrect, because instead of bounds.center we should use bounds.bottom.
When I change the code to bounds.bottom - 1.0 - _window.padding.bottom / _window.devicePixelRatio the app behaves correctly.
Btw: I also think / _window.devicePixelRatio isn't necessary here because everything is calculated in physical device pixels, so we don't need to adjust this value.
Logs

Logs
Probably useless, but flutter run --verbose:
out.txt
Flutter Analyze:
Analyzing flutter-insets-bug2...
No issues found! (ran in 7.4s)

Flutter doctor:
flutter doctor -v
[✓] Flutter (Channel beta, 1.20.0, on Mac OS X 10.15.6 19G2021, locale en-GB)
    • Flutter version 1.20.0 at /Users/jacek/bin/flutter
    • Framework revision 916c3ac648 (8 weeks ago), 2020-08-01 09:01:12 -0700
    • Engine revision d6ee1499c2
    • Dart version 2.9.0 (build 2.9.0-21.10.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/jacek/Library/Android/sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /Users/jacek/Library/Android/sdk
    • Java binary at: /Users/jacek/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/201.6823847/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio 3.5.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[!] Android Studio (version 4.1)
    • Android Studio at /Users/jacek/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-1/201.6823847/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Android Studio (version 4.0)
    • Android Studio at /Users/jacek/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/193.6626763/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] Connected device (3 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 10 (API 29) (emulator)
    • Web Server (web)                   • web-server    • web-javascript • Flutter Tools
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 85.0.4183.102

! Doctor found issues in 1 category.

Related issues:

https://github.com/flutter/flutter/issues/63761
https://github.com/flutter/flutter/issues/59130
https://github.com/flutter/flutter/issues/65626
https://github.com/flutter/flutter/issues/61149
https://github.com/flutter/flutter/issues/40974
https://github.com/flutter/flutter/issues/21265

I was able to reproduce this issue on latest stable channel.
I modified the Flutter code sample to better reproduce the issue, so when we scroll the color of the navigation bar is always the one from the center of the screen instead of the one at the bottom.

I can confirm that modifying the line 249 bellow by replacing bounds.center to bounds.bottomCenter is a work around for this issue and may be a possible fix for this.
https://github.com/flutter/flutter/blob/10febea8777fca41779802086a431313827a2104/packages/flutter/lib/src/rendering/view.dart#L246-L250
Modified Flutter code sample
import 'package:flutter/material.dart';
import 'package:flutter/services.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        // This is the theme of your application.
        //
        // Try running your application with "flutter run". You'll see the
        // application has a blue toolbar. Then, without quitting the app, try
        // changing the primarySwatch below to Colors.green and then invoke
        // "hot reload" (press "r" in the console where you ran "flutter run",
        // or simply save your changes to "hot reload" in a Flutter IDE).
        // Notice that the counter didn't reset back to zero; the application
        // is not restarted.
        primarySwatch: Colors.blue,
        // This makes the visual density adapt to the platform that you run
        // the app on. For desktop platforms, the controls will be smaller and
        // closer together (more dense) than on mobile platforms.
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key, this.title}) : super(key: key);

  // This widget is the home page of your application. It is stateful, meaning
  // that it has a State object (defined below) that contains fields that affect
  // how it looks.

  // This class is the configuration for the state. It holds the values (in this
  // case the title) provided by the parent (in this case the App widget) and
  // used by the build method of the State. Fields in a Widget subclass are
  // always marked "final".

  final String title;

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      // This call to setState tells the Flutter framework that something has
      // changed in this State, which causes it to rerun the build method below
      // so that the display can reflect the updated values. If we changed
      // _counter without calling setState(), then the build method would not be
      // called again, and so nothing would appear to happen.
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    final style = SystemUiOverlayStyle.light;

    return SizedBox.expand(
        child: ListView(
          children: [
            AnnotatedRegion<SystemUiOverlayStyle>(
              value: style.copyWith(
                statusBarColor: Colors.redAccent,
                systemNavigationBarColor: Colors.redAccent,
              ),
              child: Container(color: Colors.red, height: 500,),
            ),
            AnnotatedRegion<SystemUiOverlayStyle>(
              value: style.copyWith(
                statusBarColor: Colors.greenAccent,
                systemNavigationBarColor: Colors.greenAccent,
              ),
              child: Container(color: Colors.green, height: 500,),
            ),
            AnnotatedRegion<SystemUiOverlayStyle>(
              value: style.copyWith(
                statusBarColor: Colors.blueAccent,
                systemNavigationBarColor: Colors.blueAccent,
              ),
              child: Container(color: Colors.blue, height: 500,),
            ),
          ],
        ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.4, on Mac OS X 10.15.6 19G2021, locale en)
    • Flutter version 1.20.4 at /Users/pedromassango/dev/SDKs/flutter_stable
    • Framework revision fba99f6cf9 (8 days ago), 2020-09-14 15:32:52 -0700
    • Engine revision d1bc06f032
    • Dart version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0, Build version 12A7209
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] IntelliJ IDEA Community Edition (version 2020.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 49.0.4
    • Dart plugin version 202.7319.5

[✓] VS Code (version 1.48.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.14.1

[✓] Connected device (2 available)
    • sdk gphone x86 arm (mobile) • emulator-5554                        • android-x86 • Android 11 (API 30) (emulator)
    • iPhone 11 Pro Max (mobile)  • 7A82F29E-E49E-4B3C-8609-1570361DBC17 • ios         • com.apple.CoreSimulator.SimRuntime.iOS-14-0 (simulator)

! Doctor found issues in 1 category.
Process finished with exit code 0

Btw. I think about providing a fix for this. This should be just:
diff --git a/packages/flutter/lib/src/rendering/view.dart b/packages/flutter/lib/src/rendering/view.dart
index 464e4b560c..0f7ce5a79a 100644
--- a/packages/flutter/lib/src/rendering/view.dart
+++ b/packages/flutter/lib/src/rendering/view.dart
@@ -244,8 +244,16 @@ class RenderView extends RenderObject with RenderObjectWithChildMixin<RenderBox>

   void _updateSystemChrome() {
     final Rect bounds = paintBounds;
-    final Offset top = Offset(bounds.center.dx, _window.padding.top / _window.devicePixelRatio);
-    final Offset bottom = Offset(bounds.center.dx, bounds.center.dy - _window.padding.bottom / _window.devicePixelRatio);
+    // center of the status bar
+    final Offset top = Offset(
+      bounds.center.dx,
+      _window.padding.top / 2.0,
+    );
+    // center of the navigation bar
+    final Offset bottom = Offset(
+      bounds.center.dx,
+      bounds.bottom - 1.0 - _window.padding.bottom / 2.0,
+    );
     final SystemUiOverlayStyle upperOverlayStyle = layer.find<SystemUiOverlayStyle>(top);
     // Only android has a customizable system navigation bar.
     SystemUiOverlayStyle lowerOverlayStyle;

But I tried to write a test for this, and failed :/
I tried using renderer.window.physicalSizeTestValue but unlike TestWidgetsFlutterBinding, the TestRenderingFlutterBinding doesn't support providing size. Do you have some suggestion how I can test this logic? Maybe you'd have some code that I can read as an example? Unfortunately, view_test.dart is almost empty.

Workaround
As a workeround you can try setting:
WidgetsBinding.instance.renderView.automaticSystemUiAdjustment=false;

And whenever you scroll content you can manually set SystemChrome.setSystemUIOverlayStyle() with:
SystemChrome.setSystemUIOverlayStyle(
      SystemUiOverlayStyle(
        statusBarColor: Colors.xyz,
        systemNavigationBarColor: Colors.xyz,
      ),
    );

Even tho this is possible, but its a rather complex solution that I personally don't recommend.

Workaround
As a workeround you can try setting:
WidgetsBinding.instance.renderView.automaticSystemUiAdjustment=false;

And whenever you scroll content you can manually set SystemChrome.setSystemUIOverlayStyle() with:
SystemChrome.setSystemUIOverlayStyle(
      SystemUiOverlayStyle(
        statusBarColor: Colors.xyz,
        systemNavigationBarColor: Colors.xyz,
      ),
    );

Even tho this is possible, but its a rather complex solution that I personally don't recommend.

Hi @jacek-marchwicki
Sorry, I don't havy much experience in that field.
I just checked the test on view_test.dart and it was written by @tvolkert maybe he can help you create a test case for this.

@pedromassango , @tvolkert I finally found a way to unit test the fix.
If you have time to take a look and give some suggestions there is my PR: https://github.com/flutter/flutter/pull/68102/files#diff-8a50c47a41346299da95ca934acd1da0fe122271bb0e5a8a4fe2eb1311ba5d8d


fix: remove unnecessary transform

Removes unnecessary transform. The transform hooks is not modifying anything so it can simply return undefined.

@AriPerkkio Thanks for your contribution, it works perfectly.


Reset sticker if dragged off page

Fixes #3

Should reset if any part of the sticker is outside of the frame
Browser metadata
Path:      /posts/day1/
Browser:   Chrome <IP_ADDRESS> on Android 10
Viewport:  424 x 810 @2.625x
Language:  en-GB
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved


Error dialog has no window/taskbar icon (Java default icon is used)

Will be obsoleted by #880

Does #880 create a new window for the dialog? Currently the errror dialog opens in full screen due to not telling my tiling window manager it want's to float.

It doesn't create a new window, it's kind of an inline node


CLI utils class cases seem to be incorrect

The cases for the DPOScriptArguments and SFTScriptArguments (as seen here have the correctly capitalized DPO and SFT names, but in the downloaded codebase, it seems to be in camel-case, i.e.
@dataclass
class SftScriptArguments:
    ...

@dataclass
class DpoScriptArguments:
    ...

I'm currently running version 0.8.6, installed from pip, but you can also see this in the downloaded zip release of v0.8.6.
Does anybody else see this or is this something just affecting me?

Hi @busycalibrating
Thanks for the issue, when searching into the codebase, only the capitalized DPO and SFT names are present:
https://github.com/search?q=repo%3Ahuggingface%2Ftrl+SftScriptArguments&type=code
https://github.com/search?q=repo%3Ahuggingface%2Ftrl+DpoScriptArguments&type=code

Can you check that this is true when you download the zip release I linked?

Apologies for the delay @busycalibrating - we just made a new release, can you double check it is still the case for trl==0.9.4 ?

Yes this seems to be resolved:

What a bizarre issue :S Thanks for following up on it!


Updated feature management release notes.

Updated feature management release notes.

@yegu-ms @lisaguthrie, FYI


Fix spacing between two sections, also update spacing of tab nav

The combined effect of padding: 3rem and margin: 3remcaused the spacing forwrapper--bottom-space` to have a total of 6rem on tablet+ screen size. It made the spacing look weird and wasted a lot of vertical spacing.
Dashboard:
OLD

NEW

Submission page:
OLD

NEW

Update column width for Rounds and Labs Overview
OLD:

NEW:

@theskumar  Merged in the other small fixes, nice improvements! Now this one has a minor conflict.

@frjo Merge conflict resolved.


Create cla.yml

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


The caffe2 was edited?

If we want to use the official caffe2, the two files mentioned in the readme should be copyed into corresponding path, then rebuild the caffe2?

This is an old version of Caffe2. If you want to use it in the new version, you have to copy the file to the corresponding path as mentioned in the readme file.


Additive Gaussian Noise

How do I add additive Gaussian Noise to the output of a sonnet Sequential?
Example:

snt.Sequential([snt.nets.MLP([n] * j, activate_final=True), snt.LayerNorm())

Thank you

Closing out some old issues, please reopen if this is still a blocker.


Uar 1614 ole bo new nocs

JIRA link
https://companieshouse.atlassian.net/browse/UAR-1614
Change description
Added the new noc fields:
"non_legal_firm_control_nature_of_control_types";
"trust_control_nature_of_control_types";
"owner_of_land_person_nature_of_control_jurisdictions";
"owner_of_land_other_entity_nature_of_control_jurisdictions";
along with validation and schema updates.
Work checklist

[x] Tests added where applicable

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

The DtoModelChange test fails with this message:
Unexpected NULL value for field 'nonLegalFirmControlNatureOfControlTypes'. This indicates that the DTO/DAO structure has been updated without the new fields being added to the latest version of the test JSON file or DTO/DAO generated mapper classes are not up to date
The schema, dto, dao, generated mappings and test data are all up to date so whatever the test believes to be missing is very well hidden.

The DtoModelChange test fails with this message:
Unexpected NULL value for field 'nonLegalFirmControlNatureOfControlTypes'. This indicates that the DTO/DAO structure has been updated without the new fields being added to the latest version of the test JSON file or DTO/DAO generated mapper classes are not up to date
The schema, dto, dao, generated mappings and test data are all up to date so whatever the test believes to be missing is very well hidden.

Turns out there was a second corporate bo in the test data, updating that seems to have fixed the test. I guess one corporate bo is never enough!

CI: No security warnings found

CI: No security warnings found

LGTM

Also tested locally and input for the new nocs are present in the mongo database.


Create and assess indicator of natural space

Description
Create a new indicator for natural space (exact name TBD) that is calculated as:
1 - (% impervious) - (% crop-land)
to get the percent of each tract that is natural space.
% imperviousness comes from NLCD, sourced intermediate from CDC. We've already loaded this data source into Python notebooks.
% crop-land comes from USDA Cropland Data Layer (CDL). We have not loaded this data yet.
Assess top 10 percentile of tracts meeting socioeconomic criteria.

Creating Impervious plus Cropland Attributes for Census Tracts
Crops

Add an attribute called TractAcres (or similar) to the census tracts to hold a value representing acres covered by the census tract.
Calculate the TractAcres field for each census tract by using the Calculate Geometry tool (set the Property to Area (geodesic), and the Units to Acres).
From the Cropland Data Layer (CDL), extract only the pixels representing crops, using the Extract by Attributes tool in ArcGIS Spatial Analyst toolbox.
a.	The attribute table tells you the names of each type of land cover.  Since the CDL also contains NLCD classes and empty classes, the actual crop classes must be extracted.
From the crops-only raster extracted from the CDL, run the Reclassify tool to create a binary layer where all crops = 1, and everything else is Null.
Run the Tabulate Area tool:
a.	Zone data = census tracts
b.	Input raster data = the binary crops layer
c.	This will produce a table with the square meters of crops in each census tract contained in an attribute called VALUE_1
Run the Join Field tool to join the table to the census tracts, with the VALUE_1 field as the Transfer Field, to transfer the VALUE_1 field (square meters of crops) to the census tracts.
Add a field to the census tracts called AcresCrops (or similar) to hold the acreage of crops in each census tract.
Calculate the AcresCrops field by multiplying the VALUE_1 field by 0.000247105 to produce acres of crops in each census tracts.
a.	You can delete the VALUE_1 field.
Add a field called PctCrops (or similar) to hold the percent of each census tract occupied by crops.
Calculate the PctCrops field by dividing the AcresCrops field by the TractAcres field, and multiply by 100 to get the percent.

Impervious

Run the Zonal Statistics as Table tool:
a.	Zone data = census tracts
b.	Input raster data = impervious data raster layer
c.	Statistics type = Mean
d.	This will produce a table with the percent of each census tract occupied by impervious surfaces, contained in an attribute called MEAN

Run the Join Field tool to join the table to the census tracts, with the MEAN field as the Transfer Field, to transfer the MEAN field (percent impervious) to the census tracts.

Add a field called PctImperv (or similar) to hold the percent impervious value.

Calculate the PctImperv field by setting it equal to the MEAN field.
a.	You can delete the MEAN field.
Combine the Crops and Impervious Data

Open the census tracts attribute table and add a field called PctNatural (or similar).  Calculate this field using this equation:  100 – PctCrops – PctImperv .  This produces a value that tells you the percent of each census tract covered in natural land cover.

Define the census tracts that fall in the 90th percentile of non-natural land cover:
a.	Add a field called PctNat90 (or similar)
b.	Right-click on the PctNatural field, and click Sort Ascending (lowest PctNatural values on top)
c.	Select the top 10 percent of rows after the sort
d.	Click on Show Selected Records in the attribute table
e.	Calculate the PctNat90 field for the selected records = 1
f.	Clear the selection
g.	The rows that now have a value of 1 for PctNat90 are the most lacking for natural land cover, and can be symbolized accordingly in a map

Gotta do this by Thursday

Email sent to team for raster data 8/10

Data received. Preliminary information reported in drive.


Provide compression options

{
  mtime,
  mode,
  compress,
  forceZip64Format
}

Could you provide these options?

Yep.
To clarify: you just want to specify it once for all files? (i.e. not a (filePath: string) => CompressionOptions callback)

To clarify: you just want to specify it once for all files?

Yep, thanks!


Redis based AuthorizationCodeServices

In a project on that I currently working we use the Redis based TokenStore.
To be able to completely scale out the Authorization Server with Redis it would be great to have a Redis based AuthorizationCodeServices as well.

Is there any update regarding Redis support for AuthorizationCodeServices?

I think currently the manpower goes into Spring Security 5.0 changes to consolidate the OAuth stuff. See https://github.com/spring-projects/spring-security/issues/3907

Is there any update regarding Redis support?

Is there any update regarding Redis support for AuthorizationCodeServices?

Anyone have any ideas on the best way to implement one?


Update RNA-based tutorial tool

trimgalore (particularly from iuc) does not exist in the Tool Shed so update to an existing tool.

thanks @afgane :)


TypeError: Cannot read property 'role' of undefined

Seen in the logs over a long time span.  Doesn't happen often.
Trace: TypeError: Cannot read property 'role' of undefined
    at app.post (/app/backend/routes/auth.js:159:30)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:95:5)
    at next (/app/node_modules/express/lib/router/route.js:137:13)
    at Shop.findOne.then (/app/backend/routes/_auth.js:54:7)
    at tryCatcher (/app/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/app/node_modules/bluebird/js/release/promise.js:547:31)
    at Promise._settlePromise (/app/node_modules/bluebird/js/release/promise.js:604:18)
    at Promise._settlePromise0 (/app/node_modules/bluebird/js/release/promise.js:649:10)
    at Promise._settlePromises (/app/node_modules/bluebird/js/release/promise.js:729:18)
    at _drainQueueStep (/app/node_modules/bluebird/js/release/async.js:93:12)
    at _drainQueue (/app/node_modules/bluebird/js/release/async.js:86:9)
    at Async._drainQueues (/app/node_modules/bluebird/js/release/async.js:102:5)
    at Immediate.Async.drainQueues [as _onImmediate] (/app/node_modules/bluebird/js/release/async.js:15:14)
    at runCallback (timers.js:705:18)
    at tryOnImmediate (timers.js:676:5)
    at processImmediate (timers.js:658:5)
    at process.topLevelDomainCallback (domain.js:126:23)
    at app.use (/app/backend/app.js:151:11)
    at Layer.handle_error (/app/node_modules/express/lib/router/layer.js:71:5)
    at trim_prefix (/app/node_modules/express/lib/router/index.js:315:13)
    at /app/node_modules/express/lib/router/index.js:284:7
    at Function.process_params (/app/node_modules/express/lib/router/index.js:335:12)
    at next (/app/node_modules/express/lib/router/index.js:275:10)
    at Layer.handle_error (/app/node_modules/express/lib/router/layer.js:67:12)
    at trim_prefix (/app/node_modules/express/lib/router/index.js:315:13)
    at /app/node_modules/express/lib/router/index.js:284:7
    at Function.process_params (/app/node_modules/express/lib/router/index.js:335:12)
    at next (/app/node_modules/express/lib/router/index.js:275:10)
    at next (/app/node_modules/express/lib/router/route.js:127:14)
    at Layer.handle [as handle_request] (/app/node_modules/express/lib/router/layer.js:97:5)
    at next (/app/node_modules/express/lib/router/route.js:137:13)
    at Shop.findOne.then (/app/backend/routes/_auth.js:54:7)
    at tryCatcher (/app/node_modules/bluebird/js/release/util.js:16:23)
    at Promise._settlePromiseFromHandler (/app/node_modules/bluebird/js/release/promise.js:547:31)
    at Promise._settlePromise (/app/node_modules/bluebird/js/release/promise.js:604:18)
    at Promise._settlePromise0 (/app/node_modules/bluebird/js/release/promise.js:649:10)
    at Promise._settlePromises (/app/node_modules/bluebird/js/release/promise.js:729:18)
    at _drainQueueStep (/app/node_modules/bluebird/js/release/async.js:93:12)
    at _drainQueue (/app/node_modules/bluebird/js/release/async.js:86:9)

Should be fixed in latest prod release


Add languages to slider and clean up a few things.

The left arrow works for me. I'm not sure why it doesn't work for you.


timeout on exec_command

get_default_encoding method can't get the encoding If ssh session doesn't give a prompt (i.e. showing a whiptail menu after login)
Adding timeout to exec_command method fix it.

Some ssh servers do not support exec_command command, so there are other exceptions may be thrown.
What exception is thrown when timeout is not provided in your example?

Some ssh servers do not support exec_command command, so there are other possible exceptions may be thrown. What exception is thrown when timeout is not provided in your example?

No exception thrown, just keep waiting


config/spring.rb watched files not reloading app

I am working on this rails issue and the expected behavior of Spring.watch in config/spring.rb doesn't seem to be working for me. In a new rails app (using 4.2.0 on OS X 10.10.2) with spring-watcher-listen added to the Gemfile and config/spring.rb copied directly from the other issue, the config file is loaded and the files are added to the watcher BEFORE the default files are in https://github.com/rails/spring/blob/master/lib/spring/application.rb#L105-L112 at which time printing watcher.files shows all the files, including the ones added in config/spring.rb.
However, spring only reloads the app when the files added to the watcher in application.rb are changed. Changing the other files seems to not do anything at all. Using Spring.watch_method = :poller, the expected behavior happens so this seems to me to be an issue with this gem. Any ideas why this is happening? I am relatively new to ruby/rails so I am not sure if this is a bug or operator error.

I never got to implementing support for Listen 2.0 (see: #1), so probably only polling works.
I'd probably need to find some spare time to do this (won't happen soon unless a lot of people need this -
personally I use Zeus instead of Spring).
Newer versions of Listen have detailed debugging facilities, so you can check if it's not a Listen related issue (see https://github.com/guard/listen/wiki#issues-and-troubleshooting).

Got it. However, is there a reason that :listen works for the default files added in application.rb? I'll just use polling for now but I am interested to figure out why it works in some cases but not others.

@ajgrover - check the troubleshooting section I mentioned above - the LISTEN_GEM_DEBUGGING will help you see which events Listen is watching or ignoring.
Also, Listen watches directories, not files - so you'll want to check which directories are being watched, which files are being ignored, and which ones Spring is treating as relevant.

I figured it out - the Listen gem ignores tmp/ by default (which isn't made clear in the docs). Thanks for your help!


Convert vmfloaty to use latest pooler API

Requests with token don't seem to fully work yet, but otherwise this library seems pretty much stable.


cryptic error with inherited classes?

Could you make this error a bit more explanatory? :-)
   Execution failed for task ':app:compileDebugJava'.
   > org.androidtransfuse.TransfuseAnalysisException: @Parcel code generation did not complete successfully.

I got it trying to use it with an abstract class hierarchy...something like:
@Parcel
abstract class A {
  int field1;
  int field2;
}
@Parcel
class B extends A {
  String fields
  //etc.
}

And marked class B as @Parcel only initially since I wouldn't try parceling class A.  I tried adding @Parcel to class A w/o luck.
It'd help if it printed out how far it got parceling so we could get a bit more of a clue as to what field or class it's having a hard time with :-P
It's probably also a good idea to add a hierarchy class as an example on the site too..

I am getting this error. Please help fix it asap.
warning: Ignoring InnerClasses attribute for an anonymous inner class
(org.parceler.apache.commons.collections.BeanMap$10) that doesn't come with an
associated EnclosingMethod attribute. This class was probably produced by a
compiler that did not target the modern .class file format. The recommended
solution is to recompile the class from source, using an up-to-date compiler
and without specifying any "-target" type options. The consequence of ignoring
this warning is that reflective operations on this class will incorrectly
indicate that it is not an inner class.
:app:dexMemberDebug
UNEXPECTED TOP-LEVEL EXCEPTION:
com.android.dex.DexIndexOverflowException: method ID not in [0, 0xffff]: 65536
at com.android.dx.merge.DexMerger$6.updateIndex(DexMerger.java:502)
at com.android.dx.merge.DexMerger$IdMerger.mergeSorted(DexMerger.java:283)
at com.android.dx.merge.DexMerger.mergeMethodIds(DexMerger.java:491)
at com.android.dx.merge.DexMerger.mergeDexes(DexMerger.java:168)
at com.android.dx.merge.DexMerger.merge(DexMerger.java:189)
at com.android.dx.command.dexer.Main.mergeLibraryDexBuffers(Main.java:502)
at com.android.dx.command.dexer.Main.runMonoDex(Main.java:334)
at com.android.dx.command.dexer.Main.run(Main.java:277)
at com.android.dx.command.dexer.Main.main(Main.java:245)
at com.android.dx.command.Main.main(Main.java:106)
Error:Execution failed for task ':app:dexMemberDebug'.

com.android.ide.common.process.ProcessException: org.gradle.process.internal.ExecException: Process 'command '/usr/lib/jvm/jdk1.8.0_45/bin/java'' finished with non-zero exit value 2

@shilu-stha you're running into the dex method limit. Unrelated to this library.


SPI Ports not working on Intel Joule

SPI ports not working, please see comment on Intel Joule Communities.
Here is the os-release info:
root@intel-5xx-64:~# cat /etc/os-release
ID="ref-os-iot"
NAME="Ref-Os-Iot"
VERSION="1.0+snapshot-20170419"
VERSION_ID="1.0-snapshot-20170419"
PRETTY_NAME="Ref-Os-Iot 1.0+snapshot-20170419"
BUILD_ID="20170421113711"

Continuing our discussion from the mraa's repo issue.
TL;DR version
Ref-OS-IoT doesn't by default contain  any configuration for "devices" on the SPI bus, which would bind the spidev driver to them. This is generally not a bug and actually a correct behavior from the kernel SPI bus philosophy standpoint (see longer version below for details). Though many boards in this segment neglect that and implement fake configs similar to the one below to activate it.
A quick immediate solution would be to remove this line and rebuild the image (beware - it needs around 135GB of disk space). The patch included on that line removes an existing init hack, which creates a fake configuration of three buses with two devices at each (CS0 and 1).
Long version
In general, AFAIU the kernel devs' philosophy is that SPI bus topology is considered to be static and devices attached should be configured by means of either board config or Device Tree, or ACPI and communication should be done by means of dedicated drivers. spidev is rather intended for prototyping and makes this more dynamic, so it needs some fake device configuration to be activated.
So from the Linux standpoint the right approach for SPI enablement is to create a full, setup-specific config using one of the abovementioned ways, which would specify the exact SPI topogy your system has: which device is at which CS, what frequency and other SPI parameters does it use, which driver to use for communicating with it etc.
For spidev to get into the picture the same set of data is necessary - but as long as there may be different devices connected to the bus when using this driver, it should be sufficiently generic to cover them all (i.e. SPI modes, frequencies, other params should take that into account). More details on this can be found in kernel docs, but that patch mentioned above gives a basic example as well.
I'm not sure what would be the best way to properly implement this in ref-os-iot in the longer run, but hopefully this analysis would help it. Maybe something like Soletta's dedicated platform device module would be a nice, modular, way to do that. When/if someone needs it, they can load it, when "proper" approach is used, it won't get in the way.

@alext-mkrs wow thanks! you just taught me a lot in one post! I will try to build an image without the that line.
Moving forward, and after reading those kernel docs, seems like a pretty clear cut answer to just make the SPI device I want to talk to a full configuration and leave it alone. But as you can assume, I have a lot more dev to do before I'm ready to starting messing with the kernel..

My pleasure :)

Hi! Just a question, so did it finally work? I've been fighting with Ostro and the SPI interface for quite long, now I was wondering if this workaround did work and if you were able to use SPI on Joule with Ref-OS-Iot.. ???

The answer is yes to all sub-questions :)

Oh great! thanks for the answer!  so is it still necessary to build the latest release (1706) without the indicated line or does it comes fixed by default?
Thanks again!

@Plafo, sorry I missed your follow-up question. Yes, you still need to remove it by yourself if you want that behavior - as I, hopefully, explained earlier, having this config is a hack and that's why it's not present in ref-os-iot. Just double-checked to make sure and the line is still there, so nothing has changed since then.


LongVILA - Base LLM not declared in the Paper?

I'd like to ask the base LLM of the following LongVILA checkpoint:

Efficient-Large-Model/Llama-3-LongVILA-8B-128Frames
Efficient-Large-Model/Llama-3-LongVILA-8B-256Frames
Efficient-Large-Model/Llama-3-LongVILA-8B-512Frames

This was named with Llama-3, however, as quote from the paper:

It is unclear what's the exact base LLM of LongVILA.

LongVILA was originally trained with LLama3 (so as the releaesd models) and recently updated to Qwen2 backbone. @yukang2017 can help confirm

Thanks for your interests in our work. We are waiting for legal permission from NVIDIA to release qwen2 models. It should be approved this week. I will let you know here when it is release on hf. Thanks for your patience.

@liyucheng09
Hi, our code and models have been released. You can find it below.
The benchmark comparison uses Efficient-Large-Model/qwen2-7b-longvila-256f.
Updated results is in the paper.
Paper: https://arxiv.org/pdf/2408.10188
Code: https://github.com/NVlabs/VILA/tree/main/longvila
Model: https://huggingface.co/collections/Efficient-Large-Model/longvila-66c3fce79284c8209f119b32

@yukang2017 Hi Yukang, thanks for updating the latest model. Will try it soon. Close as solved.

@yukang2017 Hi Yukang, the repo seems to be largely updated since the NVILA release.
the old command to reproduce the results in the paper is not valid anymore.
Any ideas how to run the experiments in the LongVILA paper?

@yukang2017 @zhijian-liu   Hi, it seems this repo is largely broken after the NVILA relase.
The vila-run and vila-eval all based on scripts under scripts/v1_5/, but these are removed in @zhijian-liu's recent NVILA release. Is this a merging issue or there will be a new version of vila-run?
Error see below.
(nvila) (base) v-yuchengli@microsoft.com@GCRAZGDL1694:~/vlm/nvila$ vila-run -m eval -J longvila-7b-256f/lmms-nextqa_mc_test-32 scripts/v1_5/eval/lmms.sh nextqa_mc_test-32 Efficient-Large-Model/LongVILA-7B-256f auto 12
No slurm installed. Ignore slurm-related args.
scripts/v1_5/eval/lmms.sh nextqa_mc_test-32 Efficient-Large-Model/LongVILA-7B-256f auto 12
/bin/sh: 1: scripts/v1_5/eval/lmms.sh: not found
returncode: 127
Job finished with exit code 127

(nvila) (base) v-yuchengli@microsoft.com@GCRAZGDL1694:~/vlm/nvila$ vila-eval -m Efficient-Large-Model/LongVILA-7B-256f -c auto -nf 128 -t vnbench_val
2024-12-20 19:55:00.621 | INFO     | llava.cli.eval:main:70 - Running evaluation for 'longvila-7b-256f' on 1 tasks: ['vnbench_val-128']
2024-12-20 19:55:00.621 | INFO     | llava.cli.eval:main:124 - Running 'vila-run -m eval -J longvila-7b-256f/vnbench_val-128 scripts/v1_5/eval/vnbench.sh val-128 Efficient-Large-Model/LongVILA-7B-256f auto 128 12'
2024-12-20 19:55:02.624 | ERROR    | llava.cli.eval:main:151 - Error running 'vnbench_val-128' evaluation (return code: 127)
2024-12-20 19:55:02.625 | INFO     | llava.cli.eval:main:188 - Saved all metrics to 'runs/eval/longvila-7b-256f/metrics.json'
2024-12-20 19:55:02.626 | INFO     | llava.cli.eval:main:194 - Results:
┌──────────┬─────────┐
│ Metric   │ Value   │
├──────────┼─────────┤
└──────────┴─────────┘

Hi @liyucheng09 ,
Sorry for the bug. I fix it in this pr. You can try this repo, https://github.com/yukang2017/VILA/tree/main, before it was merged.
https://github.com/NVlabs/VILA/pull/170
Regards,
Yukang Chen

@yukang2017 many thanks, will try it very soon.

@yukang2017 Hi Yukang, can you help to confirm the conv_template used for Efficient-Large-Model/qwen2-7b-longvila-256f?
conv_templates = {
    "auto": conv_auto,
    "hermes-2": hermes_2,
    "llama_3": llama_3_chat,
    "v1": conv_vicuna_v1,
    "vicuna_v1": conv_vicuna_v1,
    "plain": conv_llava_plain,
}

I cannot reproduce the results in the paper, and I suspect it's due to a wrong conv template.

@Lyken17 Hi Yukang, one more small question on reproduction. Should I specify add_newline_token in v-niah test?

Hi @liyucheng09 , I think we do not need to set conv_template when use vila-run evaluation. It can find the conv template from the tokenizer config json.
I did not set add_newline_token before.
In addition, sorry for the late reply. I think you can find me via WeChat,<PHONE_NUMBER>6, for more frequent discussion. Thanks.


Set SECRET_KEY_BASE length to 64 bytes

Hello @acco
Similar to #572 suggesting a change in the doc.
Thanks.

@mdminhazulhaque Thanks again! 🙏


How to use only .pcd format point cloud for segmentation

Our forestry scans rarely have .dat data, most of which are common data formats such as pcd. How to use your treeseg to directly split the data collected by pcd format forest radar? Thank you！

I initially had some confusion about the input files.
Each .dat file is is just a text file containing ascii 4x4 transformation matrix's.
This is used to register several rxp scans(indivual tls scans in reigl's format).
I will try to summarize the behavior below
plotcoords creates a text file as such:
-13.3903 45.2606 0 32.1628
With the xmin,xmax, ymin,ymax limits of your plot.
rxp2pcd:
Generates N pcd tiles over the plot area.
Tiles are non overlapping and extend 10m beyond the plot limits
filenames in this format:
FGC01.tile.0.pcd FGC01.tile.1.pcd
FGC01.tile.2.pcd`
Then you should be able to skip to this command:
downsample 0.04 FGC01.tile.*.pcd
Regards
Tony

Should I edit the file FGC01.dat? I rename my PCD files FGC01.tile.2.pcd and downsample 0.04 FGC01.tile..pcd,and getdemslice 2 3 6 FGC01.tile.downsample..pcd,when findstems 15 0.2 2 ../FGC01_coords.dat ../FGC01.slice.pcd,it not work and show：
Reading slice: [pcl::PCDReader::read] Could not find file '../FGC01.slice.pcd'.
complete
Cluster extraction: [pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud!
[pcl::KdTreeFLANN::setInputCloud] Cannot create a KDTree with an empty input cloud!
terminate called after throwing an instance of 'pcl::IOException'
what():  : [pcl::PCDWriter::writeBinary] Input point cloud has no data!
Regards
Jiang Li

I edit the file FGC01.dat and  run ./findstems 15 0.2 2 FGC01_coords.dat FGC01.slice.pcd,the shell show wrong with:
eading slice: complete
Cluster extraction: FGC01.intermediate.slice.clusters.pcd | 43
Region-based segmentation: FGC01.intermediate.slice.clusters.regions.pcd | 35
RANSAC cylinder fits: terminate called after throwing an instance of 'pcl::IOException'
what():  : [pcl::PCDWriter::writeBinary] Input point cloud has no data!

Yes so the documentation here is lacking (it is still on the to-do list!). See #4 for a bit more detail, but if you are not using riegl data and rxp2pcd, then yes you will need to put your lidar data into pcd format tiles (it can be just the 1 tile). The .dat and coord files are all specific to rxp2pcd, and not needed from downsample (or thin) onwards.

@jiang-li1 your last error is occuring because the line 95:
writeClouds(cyls,ss.str(),false);

Is trying to write a point-cloud of cylinders when no valid cylinders have been detected.
I suspect this is because your coordinates are not defined correctly or missing.
your best bet would be to add some print statements around this region.
float xmin = coords[0];
	float xmax = coords[1];
	float ymin = coords[2];
	float ymax = coords[3];
	float lmin = 2.25; //assuming 3m slice
	float stepcovmax = 0.2;
	float radratiomin = 0.8;
	for(int i=0;i<regions.size();i++)
	{
		cylinder cyl;
		fitCylinder(regions[i],nnearest,true,true,cyl);
		if(cyl.ismodel == true)
		{
			if(cyl.rad*2 >= dmin && cyl.rad*2 <= dmax && cyl.len >= lmin)
			{
				if(cyl.stepcov <= stepcovmax && cyl.radratio > radratiomin)
				{
					if(cyl.x >= xmin && cyl.x <= xmax)
					{
						if(cyl.y >= ymin && cyl.y <= ymax)
						{
							cyls.push_back(cyl.inliers);
						}
					}
				}
			}
		}
	}

to determine what expected coordinates you are getting.
Alternatively if you build the source form my fork there is already some extra debugging prints, and an early implementation of multi threading.
Regards Tony


cant upload asset

Hello,
When I try to upload assets on the player I have an error.
it works well with URL but not files
I don't understand why.
is there a way to get detailed logs?
sorry for my bad english, i hope it's understandable

Tom

Hello @Tom2446
no problem - I'm also not a native speaker ;)
You can get a detailed log if you activate the debug mode (Settings) and activate the developer console in your browser (F12) before you upload a file.
Check the network tab and what happened here --> Normally the upload response will show you a detailed error.
PS: Did you change the upload size in the settings?
Best
didiatworkz

Hello. I have exactly the same difficulty as my colleague and I reproduce the situation below.
Error displayed at the time of transfer. I think it might be the upload_max_filesize limit in php.ini.
I tried to change it but as it is a docker, the file seems to be reset every time I restart:

Error displayed in DEBUG MODE

Hey @jpdenss
Can you confirm the same issue? - Check Network -> _functions.php -> Preview:

Memo to myself:
Change in API -> New tmp path for screenly_assets

Do not. Here it just logs that it can't upload to host jpares01, in all red occurrences of _functions.php

Hi. I have the same problem as @Tom2446 and @jpdenss . Has anyone found a solution? Everything works fine using screenly interface. I can edit or delete images in SOMO.

When I connect over a domain name, I can't upload assets and gives me an error. uploading assests works over local IP

Same problem here

Hi @didiatworkz
I've installed Somo on a raspberry pi4  (using Anthias) and work well, but have the same Issue to upload somes files.
Thanks

any updates on this topic ?


[WIP] Allow the user to update avatar from profile view

@RocketChat/android
Closes #752
DESCRIPTION
The user will be able to choose avatar either by clicking an image from camera or by choosing an image from storage.
This is a WIP
This is blocked by #762

@aniketsingh03 will take a look later today, thanks for the PR.

@aniketsingh03 Is this PR ready to be reviewed already?

Yeah @rafaelks , you may review this, however this depends on #77 and #71 of the sdk as it is blocked by #762. You may merge this once you are sure to have resolved those.

@aniketsingh03 Looks like the payload we're sending is wrong. We should send data.email, data.name and data.password.

@leonardoaramaki are you talking about the payload for updateProfile ?

@aniketsingh03 :+1:

You might want to have a look at https://github.com/RocketChat/Rocket.Chat.Android/issues/762 @leonardoaramaki .

@filipedelimabrito this pr is ready to be merged. I have resolved the conflicts too.

@rafaelks can you have a look at this one?

@filipedelimabrito @leonardoaramaki @rafaelks ^

@filipedelimabrito I have made the suggested changes

@filipedelimabrito @luciofm please tell me if any more changes are required.

@filipedelimabrito @rafaelks ^^

@leonardoaramaki can you have a look at this ?

@filipedelimabrito ^^

@leonardoaramaki can you provide a complete log so that I may have a look as to why the app is crashing?

@leonardoaramaki ^

@leonardoaramaki @filipedelimabrito I have fixed the keyboard issue, please have a look and suggest any changes to be made. Also @leonardoaramaki can you provide me the logs for the crash so that I may try to fix the other issue too?

@leonardoaramaki I have made the requested changes. Can you provide the translations for the newly added strings? Some tests are failing because of lint errors which are probably occurring in all pull requests, so I guess we can have a separate pr which fixes all the lint errors.

@leonardoaramaki @filipedelimabrito can this be merged?

@aniketsingh03 I'm sorry for this, but your PR has some new conflicts. Can you solve them please?

@rafaelks @filipedelimabrito all changes have been, you may merge this.

@aniketsingh03 App is crashing for me when opening the Profile view. I'm on Android 6.0, can you check please?

@rafaelks can you please provide a log of the error obtained when the app crashed?

@rafaelks resolved some conflicts, have a look.

@aniketsingh03 Could you resolve the conflicts more one time? Thanks.

@filipedelimabrito I have the suggested changes.

@filipedelimabrito I feel that I have dealt with all the edge cases but I might have missed one, have a look and suggest changes. As far as the 4th point is concerned, I feel that this is an issue with SimpleDraweeView, can't we have a separate issue for that?

@filipedelimabrito @rafaelks can this be merged?

@aniketsingh03 Can you please fix the conflicts on this one?

@filipedelimabrito please have a look at this one.

This one is going to be resolved by #1527.


Ability to set custom attributes for auth

Describe the solution you'd like
Need to add custom attributes that go under the "Do you want to add custom attributes" in the cognito console.
Describe alternatives you've considered
Currently I am just modifying the cloud formation template that amplify produces. Is this ok practice - to modify the cloud formation template that amplify provides? I understand I can't run amplify configure on the auth service going forward as that will likely remove my template modifications.
Additional context
See this for context and how amplify library supports custom attributes: https://aws.amazon.com/blogs/mobile/aws-amplify-adds-support-for-custom-attributes-in-amazon-cognito-user-pools/. I don't believe amplify cli auth module supports that out of the box.

Also when doing an amplify push, where auth is updating, I notice that if cloudformation detects a new physical resource needs to be created it will actually delete your whole cognito userpool without notice and create a new one. This is horrible and can make someone accidentally delete all their app users. See https://forums.aws.amazon.com/thread.jspa?messageID=793367 for more info. Amplify should warn before cloudformation tries to delete your whole userpool...

We confirmed this behavior with ththe cognito service team last week. This behavior is detected when changing certain cognito policy attributes like the password policy and we have a PR out for same to not allow updating those scheme atteibutes using the CLI -
https://github.com/aws-amplify/amplify-cli/pull/184

We’ve also engaged with the Cognito service team to not allow the deletion of the user pool when users are present in the user pool. They’re working on changing this behavior as well.

Perfect, thank you!

Currently I am just modifying the cloud formation template that amplify produces. Is this ok

Can you share how you do this? I also need the functionality.

@ianpogi5 just go to backend/auth/<your_cognito_project_name>/ and modify the yml and json accordingly. Add your attributes to 'userpoolClientWriteAttributes' in parameters.json and go from there.

This issue seems to be fixed. Feel free to re-open this issue if the problem still exists.

Hey guys, with our new version of the CLI (>1.4.0), we've introduced the ability for users to select how they want users (including an option for email uniqueness) to sign in and select their sign up attributes upfront as a part of the default flow instead of setting the username attribute as a default in the default flow (which is immutable unfortunately after the creation of the userpool since that is a restriction from the Cognito service). Please feel free to check it out and re-open/comment on this issue if you see a problem with it.

Maybe an example of the updated cloud formation file with custom attributes added would help? Thanks.

I can't add custom attributes to the user pool i only get a list of predefined ones. And even still none of them work with federated sign in. It seems that the attribute selection is only for Required ones. I still have no idea how to proceed here.

@pr1ntr Check https://github.com/aws-amplify/amplify-cli/pull/1289/files the cf-templates/auth/README.md file. I've added an example of what works for me. Hope that helps.
Regards.

Excellent, thank you!


Add Hyrax multilinear PCS

Description
This PR implements the Hyrax polynomial commitment scheme: a multilinear PCS based on the hardness of the discrete logarithm problem introduced as part of the Hyrax zkSNARK in this 2017 article.
The PCS described therein is interactive. When implementing the Fiat-Shamir transform, this paper was consulted.
Modification note
In the PCS contained in the cited article, the verifier never learns the actual evaluation of the polynomial at the requested point, but is instead convinced that a previously received Pedersen commitment is indeed a commitment to said evaluation - this is what the SNARK proposed therein necessitates. However, the Arkworks framework requies the verifier to actually learn that value, which is why we have added the opening of the commitment at the end of the protocol. This likely does not result in an optimal non-hiding PCS, but we feel it is the most faithful adaptation of the original PCS that can be implemented with the current restrictions.
Future optimisations
Some natural optimisations to the scheme which are not part of the current PR, but would make sensible follow-up work, are the following:

Deal with the modification described above: either modify the PCS trait to encompass hiding PCSs (in terms of the actual   evaluation, not only the polynomial), or turn this scheme into a non-hiding one by removing unnecessary work (which would   probably involve non-trivial theoretical work).
Add parallelisation. There is at least one natural place where parallelisation could bring performance gains: in essence, the prover commits to the polynomial by expressing it as an evaluation matrix and Pederson-multi-committing to each row. Each of this commitments can be computed independently from the rest, and therefore, in parallel. It is still to be seen how much of an improvement this would entail, since each Pederson multi-commitment boils down to a multi-exponentiation and this operation is itself parallelised.
Due to the homomorphic nature of Pedersen commitments, it is likely some of the following methods can be designed more efficiently than their default implementations: batch_open, batch_check, open_combinations, check_combinations. This is not discussed in the reference article, but the IPA and KZG modules might be a good starting point.
On a related note to the previous point, there might be a more efficient way to open several polynomials at a single point (this is the functionality of the open method) than the currently implemented technique, where only the computation of the vectors L and R is shared across polynomials.
The cited article proposes an optimisation in the section Reducing the cost of proof-of-dot-prod. It allows for non-square matrices (and hence removes the requirement for the number of variables to be even) and introduces a tradeoff between proof size and verifier time. It is probably worth pursuing.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[x] Targeted PR against correct branch (master)
[x] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[x] Wrote unit tests
[x] Updated relevant documentation in the code
[x] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[x] Re-reviewed Files changed in the Github PR explorer

This PR relies on https://github.com/arkworks-rs/algebra/pull/691, so we temporarily expect CI to fail until that's merged.

Sorry for the late update on this, but happy to merge this as-is, once it's updated wrt master.

It is ready for review and to be merged.
@Pratyush If you are merging these three PRs, please do this PR first. Then, I will resolve the conflicts of the other two and let you know when they are ready.

I am happy with the current state of things! Let us see if @Pratyush would like any further tweaks regarding the open threads  of this discussion.

A gentle reminder @Pratyush :)

Gentle reminder @Pratyush. (It would be much simpler if we go ahead and merge, and then, we can always improve the code.)

Do you know what blocks this PR from getting merged? @mmagician

Seems like the same build failure here

This PR is ready now. After this getting merged, I will update #131 and #132.


cmd/kpod/ps.go: Use getCommand for JSON

The getCommand func strips out unwanted characters around the
command of the container.  The JSON output should use this func
like the regular ps output for both consistency and because
Python does a literal interpretation of the bracket [] characters
when consuming as JSON.
Signed-off-by: baude<EMAIL_ADDRESS>

@rhatdan per your comment in an Atomic CLI review.

@umohnani8 ptal

LGTM

LGTM


SSL handshake error with libmproxy

Hi,
I've been using libmproxy in conjunction with Python's requests module, and I've run into a problem with certain websites.
One of these websites is https://www.401save.com. Trying to access it through libmproxy results in:

SSL handshake error: The client may not trust the proxy's certificate.

Here's a short script that demonstrates my problem:
import threading
from libmproxy import controller, proxy
from libmproxy.proxy.server import ProxyServer
import requests

config = proxy.ProxyConfig(port=8888)
server = ProxyServer(config)
mitm = controller.Master(server)
t = threading.Thread(target=mitm.run)
t.daemon = True
t.start()

requests.get('https://www.401save.com', proxies={'https': 'http://localhost:8888'}, verify=False)

mitm.shutdown()

Printing the server's logs, this is what comes up:
<IP_ADDRESS>:58270: clientconnect
<IP_ADDRESS>:58270: request
  -> CONNECT www.401save.com:443 HTTP/1.0
<IP_ADDRESS>:58270: Set new server address: www.401save.com:443
<IP_ADDRESS>:58270: serverconnect
  -> www.401save.com:443
<IP_ADDRESS>:58270: Received CONNECT request to SSL port. Upgrading to SSL...
<IP_ADDRESS>:58270: Establish SSL
  -> with client
  -> with server (sni: None)
<IP_ADDRESS>:58270: SSL handshake error: The client may not trust the proxy's certificate.
<IP_ADDRESS>:58270: ProxyError('NetLibError("SSL handshake error: SysCallError(-1, \'Unexpected EOF\')",)',)
<IP_ADDRESS>:58270: clientdisconnect
<IP_ADDRESS>:58270: serverdisconnect

Could this be related to #427?
Best,
Jeff

Thanks for the report - https://www.401save.com/ does not work with mitmproxy at the moment. Firefox says ssl_error_rx_record_too_long.

https://www.ssllabs.com/ssltest/analyze.html?d=401save.com - their server does not offer any cipher suites which are compatible with OpenSSL 1.0.2. There's not much we can do here (despite providing a better error message), sorry. If you're on Linux/OSX, you can use cryptography with OpenSSL 1.0.1, that should work - fixing the server is probably easier though.
Cheers,
Max

We cannot really provide a better error message, OpenSSL fails with Unexpected EOF, which is too generic.


[FEATURE]: Secure API keys and secrets

What would you like to be added?
Hello, thanks for this project!
I would like an option to load API key and secret from a docker secret (so from inside the container : load from a file).
Something similar to what mariadb does with *_FILES variables for example.
Why is this needed?
Avoid clear secrets in configuration files, docker-compose or swarm stacks definitions.
Any implementation details that are not related to source code may be included here.
No response

Hello,
That will be great to have. We can introduce 2 new variables:

OPS_API_KEY_FILE
OPS_API_SECRET_FILE

And implement hierarchy check: if the *_FILE is passed -> ignore the clear variable/command flag and set the value from the file.
This project talks to firewalls so it must always follow best security practice. Thank you for bringing this up. I will try to implement and release in the next week.

Introduced in #7


2215 Create board manifest specification

https://github.com/breadboard-ai/breadboard/issues/2215

see https://github.com/breadboard-ai/breadboard/issues/2215#issuecomment-2177989489


Problems caused by a change in 2.5.1

Problems caused by a change in 2.5.1
In this change, when I set charset=UTF-8, new String will use ZIP_STANDARD_CHARSET, and the Chinese directory name will be garbled
use 2.6.4

this is right

This is the compressed package and code I used
test2.zip
final ZipFile zipFile = new ZipFile("/Users/gzr/dev/test/zip_test/test2.zip");
zipFile.setCharset(StandardCharsets.UTF_8);
final List<FileHeader> fileHeaders = zipFile.getFileHeaders();

Issue fixed. I will include the fix in the next release.

Okay, thank you very much, and look forward to the release of the next release.

Issue fixed in v2.7.0 which was released today.


Remove 4% Inflation paid to Savings

This pull request also removes the inflation paid to savings. If the community wants to direct this inflation elsewhere in the future, it can, but for now it is merely creating a large pot of tokens which is unnecessary work.

Not planning to merge any time soon. Changes can be reapplied at a later date.


Allow external manipulation of the command line buffer, fixes #423

This allows nushell to implement a command similar to fish's commandline:
https://fishshell.com/docs/current/cmds/commandline.html
that would allow commands run via executehostcommand bindings to do interesting things to the contents of the input buffer.

@sholderbach can we land this? I'd like to get it into nushell to play with now that 68 is out.

I am excited and curious about nushell/nushell#6492.
My gut reaction was, to be a bit cagy about exposing the full editing behavior with run_edit_commands. In theory, you shouldn't be able to do worse crimes than what the user can already do with the keybindings, but right now we control the order of executing and printing. So my strict take would be to make the three operations used by the commandline commands separate methods.
But in principle the nushell side needs the testing so I would also be open to landing this first and then concern our selves with the proper API

I am gonna land this to unblock the downstream PR for testing. Might polish the API later


HCS_E_CONNECTION_TIMEOUT after drivers update or BIOS virtualization disabled & enabled again

Version
Windows version: 10.0.22621.1105
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 18.04
Other Software
WSL version: <IP_ADDRESS>
Kernel version: <IP_ADDRESS>
WSLg version: 1.0.47
MSRDC version: 1.2.3575
Direct3D version: 1.606.4
DXCore version: 10.0.25131.1002-220531-1700.rs-onecore-base2-hyp
Windows version: 10.0.22621.1105
Repro Steps
wsl.exe
Expected Behavior
Expecting the distro to startup just fine.
Actual Behavior
The operation timed out because a response was not received from the virtual machine or container. Error code: Wsl/Service/CreateInstance/HCS_E_CONNECTION_TIMEOUT
I tried to disable & enable the Virtual Machine Platform and Windows Hypervisor Platform several times without any success. I also tried installing another distro but the installation never finishes with the same error code. Also tried disabling & enabling virtualization in BIOS.
I also tried to mount the .vhdx drive in case it was corrupted but the mount command never finishes. Also, it doesn`t look like corruption of the drive since a fresh install fails too. I am also not aware of anything advanced I did inside of the distro since it worked flawlessly.
Some updates/driver updates were done since the distro worked okay. Unfortunately, I don't have a backup to go back to.
Diagnostic Logs
https://drive.google.com/file/d/18X-VQVoXZSp6Kv_0g7SFwG2zvg_MHtoK/view?usp=sharing

Can you rerun log collection from before starting WSL?
/logs

Hello @pmartincic, I am sorry. I can no longer replicate the issue since I had all my work tools installed in the WSL, so I had to completely uninstall WSL, reinstall it again and create a new distro. Then I had to manually recover necessary data from the .vhdx virtual drive. It was hell of a few days. Anyway, let's close this now.


Update chroma.py

Description: add minor fix codes in  langchain.vectorstors.chroma  to return distances value .
so
qa=RetrievalQA.from_chain_type(
llm=llm,
chain_type="stuff",
retriever=retriever, #a chroma db
return_source_documents=True,
chain_type_kwargs={"prompt": prompt, "memory": memory},
)
res = qa(query)
Now res can return distances value in the metadata dict.
answer, docs = res["result"], res["source_documents"]
for document in docs:
print("\n> [source file path]: " + document.metadata["source"] )
print("> [cosine  similarity]:" + str(1.0-document.metadata["distances"]) )   # Add This  and it will print 1-distance_value
print(">[text slice]:" + document.page_content)
It is a very minor change.  I tested on the [https://github.com/PromtEngineer/localGPT/blob/main/run_localGPT.py#L237]

Issue: the issue #4710 #5416 Maybe this fix is not directly with the issues , but I guess the intuition is alike.

Dependencies: same as the main branch

Tag maintainer: @baskaryan, @eyurtsev, @hwchase17, @rlancemartin

i don't think we want to include distance in metadata, since that's query-specific and ideally documents are independent of the query that caused them to be returned

So When meeting similiar problems as #4710 or #5416 mentioned,
how to get a metric about similarity on the returned doc slice when using RetrievalQA.from_chain_type ?
The source code  of chroma.py indeed has already calculated the metric as result[2] but did not show,
I guess some users need the metric to show on the web ui.


Missing services

Missing Azure services in the list
Batch  -   privatelink.{region}.batch.azure.com
Migrate - privatelink.prod.migration.windowsazure.com
DigitalTwins - privatelink.digitaltwins.azure.net
Possible managed disk privatelink.blob.core.windows.net
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: a88af182-32aa-a8f3-359d-b92ae1cfb8c7
Version Independent ID: 01ce07fc-4bc2-def5-9ecd-a80330ad9488
Content: Azure Private Endpoint DNS configuration
Content Source: articles/private-link/private-endpoint-dns.md
Service: private-link
GitHub Login: @asudbring
Microsoft Alias: allensu

@MathieuRietman, Thanks for reaching out. We will investigate and update this thread.


Use datum arithmetic scalar value

Which issue does this PR close?

Part of #7353
Rationale for this change

This avoids duplicated arithmetic logic, and moves us closer to replacing the custom ScalarValue logic with upstream kernels (#7353)
What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?
The upstream kernels have a couple of differences compared to the existing impls:

Null + v = Null, whereas previously it was v
Decimal precision is correctly updated
Heterogeneous interval arithmetic is not supported, this is instead the responsibility of the type coercion machinery

The upstream kernels have a couple of differences compared to the existing impls:

These are all bug fixes in my mind


Add container filesystem API for sandboxes

Describe your changes
Resolves WRK-414 by adding a new container filesystem API.
This API is designed to closely follow io.FileIO, supporting most of the major functions (open, read, readline, readlines, write, seek, flush, close) along with a few higher-level functions (delete_bytes, write_replace_bytes) to help make Sandboxes a bit easier to use.
All file modes (e.g. text, bytes, r/w/a/x, etc.) are supported, as well as most common errors/exceptions. If an unknown exception is thrown, then a modal.exception.FilesystemExecutionError is thrown instead.
Example usage:
app = modal.App.lookup("sandbox-fs", create_if_missing=True)
sb = modal.Sandbox.create(app=app)

with sb.open("test.txt", "w") as f:
  f.write("Hello World\n")

f = sb.open("test.txt", "rb")
print(f.read())

 Backward/forward compatibility checks

Check these boxes or delete any item (or this section) if not relevant for this PR.

[x] Client+Server: this change is compatible with old servers
[x] Client forward compatibility: this change ensures client can accept data intended for later versions of itself

Note on protobuf: protobuf message changes in one place may have impact to
multiple entities (client, server, worker, database). See points above.

Changelog
Sandboxes now support a new filesystem API. The open() method returns a FileIO handle for native file handling in sandboxes.
app = modal.App.lookup("sandbox-fs", create_if_missing=True)
sb = modal.Sandbox.create(app=app)

with sb.open("test.txt", "w") as f:
  f.write("Hello World\n")

f = sb.open("test.txt", "rb")
print(f.read())

Can we improve the changelog a bit here?

@azliu0 can we convert this back to a draft until we're ready to look to merge?

@pawalt @cweld510 @mwaskom Now that daemon changes are gearing up to be merged, this interface is unlikely to be changed, so client changes are ready for another look. Merging these changes are specifically still blocked on:

daemon changes merging
enabling -host-uds=all in production for all Sandboxes
gVisor upgrade

but they are ready for review.


Remove the MmapVec::split_off method

This method isn't actually correct if split_off is used on the returned values again as it's missing some extra indexing processing. That being said nothing in Wasmtime currently uses it, so remove it entirely instead of keeping it around.
found by @FrankReh, thanks!

Notably @FrankReh recommended replacing with this:

pub fn split_off(&mut self, at: usize) -> MmapVec {
    assert!(at <= self.range.len());

    // Adjust the start of the range for the new part
    let new_start = self.range.start + at;

    // Create a new `MmapVec` which refers to the same underlying mmap, but
    // has a disjoint range from ours. Our own range is adjusted to be
    // disjoint just after `ret` is created.
    let ret = MmapVec {
        mmap: self.mmap.clone(),
        range: new_start..self.range.end,
    };
    self.range.end = new_start;
    return ret;
}

which I believe is correct, but also given that we don't need this it seems best to remove for now.


I would like to contribute shortest distance algorithms, part of graph theory.

I would like to add dijkstra's shortest path algorithm, Floyd Warshell algorithm, as well as Bellmanford algorithm and with a little explaination to use which one of them for which type of problems.

Hello Sir , Can you assign this issue to me.

Hey ! I would love to work on this by making my first Hacktoberfest submission.
It would be great if I can get some more info to work with


Connector/J version 8.0.13 connetion issue

With:

mysql-connector-java-8.0.18
MySql Server:  8.0.18
Payara 5.193

I got below error:
The last packet sent successfully to the server was ? milliseconds ago.
The driver has not received any packets from the server.

As a workaround I set sslMode to DISABLED.
Any suggestion ?!
Thanks for your comment at: https://github.com/payara/Payara/issues/2747#issuecomment-552618627 !

I updated the readme file of the project, which states that sslMode must be set to DISABLED.  I suggest you up-vote my comment (which you referenced above), so as to draw attention to it. It might be an incompatibility between Payara and MySQL's choice of TLS implementation.