cas -541: update search endpoint

•adds method appendFiltersToResponse
•adds necessary fields onto search response
•extends SQL query
•adds constructDynamicSQL method for dynamic filter params
•moves community models sql queries into constants
•creates new generalised model function GetDefaultCommunities for both homepage SQL and default search SQL ( this felt like the right thing to do as they use similar SQL queries)
*Know Issue
if the search query contains a space, and its url encoded with a % I get a 400 back. This seems to be something to do with mux https://github.com/fission/fission/issues/1317 I have tried the various fixes on the repo but so far none of the solutions working

It looks like the default search is not filtering correctly:
The URL is:
http://localhost:5001/communities/search/defaultFeatured?start=0&count=10
The expected defaultFeatured should make filters to return only filters for the feature ones, also the all filter is gone.
I only have one feature community then dao should be 1 and the all filter should exist with one
Also when sending a filter that has 0 the result is the same, and the response brings the one feature community.
(Note: I've added a category to the Flow community manually and it's dao and also I have created a second community using dao)
{
"filters": [
{
"text": "dao",
"amount": 2
},
{
"text": "social",
"amount": 0
},
{
"text": "protocol",
"amount": 0
},
{
"text": "creator",
"amount": 0
},
{
"text": "nft",
"amount": 0
},
{
"text": "collector",
"amount": 0
}
],
"results": {
"data": [
{
"id": 1,
"name": "Flow",
"category": "dao",
"categoryCount": 2,
"body": "Vote on Flow Validators",
"timestamp": "",
"compositeSignatures": null,
"creatorAddr": ""
}
],
"start": 0,
"count": 1,
"totalRecords": 1,
"next": -1
}
}

also when sending text with space the front gets a 404:

I've adjusted the SQL;
now when I do
http://localhost:5001/communities/search/defaultFeatured?start=0&count=10
I get :
{
    "filters": [
        {
            "text": "dao",
            "amount": 0
        },
        {
            "text": "social",
            "amount": 0
        },
        {
            "text": "protocol",
            "amount": 0
        },
        {
            "text": "creator",
            "amount": 1
        },
        {
            "text": "nft",
            "amount": 0
        },
        {
            "text": "collector",
            "amount": 0
        }
    ],
    "results": {
        "data": [
            {
                "id": 114,
                "name": "rich",
                "category": "creator",
                "categoryCount": 1,
                "body": "something",
                "timestamp": "",
                "compositeSignatures": null,
                "creatorAddr": ""
            }
        ],
        "start": 0,
        "count": 1,
        "totalRecords": 1,
        "next": -1
    }
}

Adding all field now, it was never on the original response; adding it now. Special characters I am blocked on due to known issue with mux library. I've tried various solutions suggested on their issues page but none have worked so for.

The all field has been added
I've adjusted the SQL; now when I do http://localhost:5001/communities/search/defaultFeatured?start=0&count=10
I get :
{
    "filters": [
        {
            "text": "dao",
            "amount": 0
        },
        {
            "text": "social",
            "amount": 0
        },
        {
            "text": "protocol",
            "amount": 0
        },
        {
            "text": "creator",
            "amount": 1
        },
        {
            "text": "nft",
            "amount": 0
        },
        {
            "text": "collector",
            "amount": 0
        },
       {
            "text": "all",
            "amount":  1
        }
    ],
    "results": {
        "data": [
            {
                "id": 114,
                "name": "rich",
                "category": "creator",
                "categoryCount": 1,
                "body": "something",
                "timestamp": "",
                "compositeSignatures": null,
                "creatorAddr": ""
            }
        ],
        "start": 0,
        "count": 1,
        "totalRecords": 1,
        "next": -1
    }
}

URL encoded Special characters I am blocked on due to known issue with mux library. I've tried various solutions suggested on their issues page but none have worked so for.

For some reason this is not working on my side, I still get the same filters are before.
Just for clarification, then the defaultFeatured is sent as query search param, the frontend is expecting filters to be populated with the amount calculated based the featured communities returned.
When a text search is entered, filters should adjust on the result based on the search test.
The All filter is always present, with the total amount of communities returned based on the search text or the defaultFeatured.
When a filter is pressed, results changes but filters are the same and are kept based on the search text

I just pushed again and I think there was one change not pushed to the branch; could you try pulling again?
I just tested again and it seems to be working as expected with all filter always present, no duplicate dao filter counts etc. defaultFeatured only returns communities with is_featured column set to true.
I just did http://localhost:5001/communities/search/defaultFeatured?start=0&count=10
and get returned
{
    "filters": [
        {
            "text": "dao",
            "amount": 0
        },
        {
            "text": "social",
            "amount": 0
        },
        {
            "text": "protocol",
            "amount": 0
        },
        {
            "text": "creator",
            "amount": 2
        },
        {
            "text": "nft",
            "amount": 0
        },
        {
            "text": "collector",
            "amount": 0
        },
        {
            "text": "all",
            "amount": 2
        }
    ],
    "results": {
        "data": [
            {
                "id": 114,
                "name": "rich",
                "category": "creator",
                "categoryCount": 2,
                "body": "something",
                "timestamp": "",
                "compositeSignatures": null,
                "creatorAddr": ""
            },
            {
                "id": 115,
                "name": "rich",
                "category": "creator",
                "categoryCount": 2,
                "body": "something",
                "timestamp": "",
                "compositeSignatures": null,
                "creatorAddr": ""
            }
        ],
        "start": 0,
        "count": 2,
        "totalRecords": 2,
        "next": -1
    }
}

Filters for featured communities are not working, and filtering returns the same result for feature communities:

In the image above, there should not be any results since I'm selecting a pill with no results (maybe if the pill is 0 make it not clickable) but still the backend should return 0 results

When entering text with spaces, the url is not found and we get a 404 as a response:

Filters are not working on when results are returned on search, in the example below we should see only one result:

This PR will now be handled by this: https://github.com/DapperCollectives/CAST/pull/574


Add copyright header to .cs, .ps1 and psm1 files

Merge after #84

@TylerLeonhardt can you review?


Military Composition

I have read the wiki looking for the best military composition and i found it, but i didn't undestand quite well the army composition ( the navy is easy), can you explain it better to me?

So, this is a "bug" report for the wiki page :). I will update the description as part of the 1.0.5 release.
But the simple answer is "Experiment - and suggest the optimal formation (and if you find it unrealistic, please report it as a bug)

The wiki has been updated with a better explanation of the NWO military system.


[macOS] TextField is not captured in composing range

mini code:
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      home: Scaffold(
        body: Center(
          child: TextField(),
        ),
      ),
    );
  }
}

Steps:

Switch to the input method that has composing.
Click on the TextField and press any letter.

Actual results:

Expected results:

cc @stuartmorgan @justinmc

cc @stuartmorgan @justinmc

I am not sure, but suspect that this behavior can also be reproduced in window or Linux. But I only have macOS.
@TahaTesser

I am not sure, but suspect that this behavior can also be reproduced in window or Linux. But I only have macOS.
@TahaTesser

I tested it on Linux,  can confirm this is not a issue on Linux platform. But another wired issue is the cursor always jump to beginning after typeseveral words.

I tested it on Linux,  can confirm this is not a issue on Linux platform. But another wired issue is the cursor always jump to beginning after typeseveral words.

Duplicate of https://github.com/flutter/flutter/issues/30660

Duplicate of https://github.com/flutter/flutter/issues/30660


Hide radio buttons

Hey I was wondering if there is a way to hide the radio buttons of a list dialog without using a custom view or a custom adapter?

Radio buttons are only there for single choice dialogs.

I think you misunderstood my question. I was trying to achieve something like that, but without the icons:

You need to make a custom list adapter, radio buttons are hard coded into the single choice dialog list item layout.

Look at the custom adapter example in the sample project


ComboBox lost rounded corners if any explicit Style is set for it

Describe the bug

ComboBox lost rounded corners if any explicit Style is set for it, even if Style has no setters or is {x:Null}. Rounded corners should be enabled via implicit style, leaving explicit style for exclusive usage by application developer.
Steps to reproduce the bug

ComboBoxLostRoundedCorners.zip
Steps to reproduce the behavior:

Unzip, build and run attached ComboBoxLostRoundedCorners project
ComboBox created with <ComboBox/> snippet will properly have rounded corners
ComboBox created with <ComboBox Style="{x:Null}"/> snippet would not

Expected behavior

ComboBox created with <ComboBox Style="{x:Null}"/> should have rounded corners
Screenshots

Version Info

NuGet package version:

Microsoft.UI.Xaml 2.2.190830001

Windows 10 version
Saw the problem?

Insider Build (xxxxx)

May 2019 Update (18362)
Yes

October 2018 Update (17763)

April 2018 Update (17134)

Fall Creators Update (16299)

Creators Update (15063)

Device form factor
Saw the problem?

Desktop
Yes

Mobile

Xbox

Surface Hub

IoT

Yes, this is expected with WinUI 2.2. We'll be able to fix it for WinUI 3.0.
@kaiguo are the new styles named in such a way that someone can do a BasedOn style for them?

Hmm, looks like there's no name associated with it, should we give it one like "DefaultComboBoxStyle"?

I'm wondering if we should do that for all our new default styles? Giving it a name will make it no longer get picked up as the default, so you have to make an unnamed one that's based on that named one:
<Style TargetType="ComboBox" BasedOn="{StaticResource DefaultComboBoxStyle}"/>

Hi, just wanted to chime in that I'm seeing this exact same issue with combo boxes and with buttons. Would be great if the proposed fix is applied to everything!

Kai is working on it -- #1300!


Actual code almost invisible in preview of literate documents

The Problem
As seen in this shot

it is almost impossible to read the code (on the left) in the preview window (on the right).
I believe I'm running with the standard dark+ theme.
I have tried various things to tweak markdown coloring, but I don't think any of them are affecting this.
Desired Outcome
Easily visible code in the preview.
This could be achieved by changing the defaults, but I assume they've been working for others.  Perhaps the choices should be theme sensitive?  Or the installation process changed to give the extension settings fuller control?
.html Files Likely have a Problem, but Not this One
Viewing the generated .html file with FireFox shows apparently unstyled black and white and the code is perfectly legible.  The generated .html file has <link rel="stylesheet" type="text/css" href="./style.css"> in the header, but there is no style.css file in the directory with the .html file.  Which is probably a separate issue.
Analysis
Used this tip to see what was going on.
Seemingly the code gets a color = #d4d4d4, a light grey/unsaturated red, from --vscode-editor-foreground inherited from html in markdown.css from Resources/app/extensions/markdown-language-features/media.  If I interactively edit the active color element it changes the color of the code (green in my test) and makes it easily visible.
In contrast, keywords show as blue--but only after I edit the basic color as above--and are coming from hljs-keyword from the style.css in the literate extension.
There are several anomalies:
1. The initial color shows as light grey, but the very faint code that I do see appears blueish.  Perhaps a result of blending with the background color, or automatic logic to introduce some contrast.
2.  The interactive color picker shows the initial color as an extraordinarily unsaturated red.  Again, not blue.
3. Resetting the color interactively seems to reset the alpha, even for code whose color is controlled by something else like hljs-keyword.
4. I didn't see alpha settings associated with the color, but it looks as if they are being used somehow. I did see alpha settings (I presume: a 4th number on a color specification) for the background.
So this analysis might just be wrong!
Environment
VS Code
Version: 1.91.0 (system setup)
Commit: ea1445cc7016315d0f5728f8e8b12a45dc0a7286
Date: 2024-07-01T18:52:22.949Z
Electron: 29.4.0
ElectronBuildId: 9728852
Chromium: 122.0.6261.156
Node.js: 20.9.0
V8: 12.2.281.27-electron.0
OS: Windows_NT x64 10.0.17763
Extensions
Literate 0.14.4
Markdown All in One 3.6.2
Markdown PDF 1.5.0
Markdown Language Features 1.0.0
EditorConfig for VS Code 0.16.4
Extension Authoring 1.0.0
Markdown Math 1.0.0 (vscode.markdown-math)
Markdown Math 0.1.0 (koehlma.markdown-math)
Mardown AutoTOC 0.3.0
and many others that seem unlikely to be relevant

The markdown-pdf extension might be contributing to the color.  It has
pre code {
	color: var(--vscode-editor-foreground);
	tab-size: 4;
}

in markdown.css.
I suspect my remarks about  color = #d4d4d4 being red are a red herring.  They are based on the fact that that interactive color picker shows red, but since the color is completely neutral  the choice of the saturated color is probably arbitrary.

This is a report on a couple of changes that didn't work, one that worked temporarily, and others that I think would work but are undesirable.
I modified C:\Users\rdboylan\.vscode\extensions\jesterking.literate-0.14.4\literate\style.css by adding a pre code specification.  It didn't change anything.  On the theory this was being superseded by Programs/Microsoft VS Code/resources/app/extensions/markdown-language-features/media/markdown.css (identified in WebView as the source of the style element, I tried adding !important:
pre code {
    margin-top: 0em;
    color: black !important;
}

This also made no difference, even after restarting VSCode.
Directly editing the color in WebView does make a difference, but is not persistent.  I mentioned that in the original report.
I think that editing the markdown.css file above (which, despite my earlier guess, seems to be from core VSCode, not the Markdown PDF extension) or the theme definition would also work, but that is more global that I would like.  In particular, since I have a dark theme, setting the color to black will make it invisible in other contexts.

Thank you for the report. I was away on vacation for a month, so it took a while to go through all the correspondence.
This is indeed a weak point currently. I need to add code that identifies whether a style.css is provided or not. In the case there isn't the extension should add a stylesheet that gives usable results.
If I can find out how to determine whether a dark theme or light theme is in use I could probably add a "matching" theme, although I'll probably work from the assumption it is going to be light based.

I found an extension, github markdown preview style, that generates different styling for the preview depending on whether the baseline style is dark or light.  Even after looking at it, I'm not sure how it does it, but I think the function generateStyles() is important.
I think the code is dynamically generating an appropriate style file.
The dynamic style choice only occurs for certain user configuration options of markdown-preview-github-styles.colorTheme:
Auto means the choice depends on the theme for VSCode while System means the dependency is on the OS theme.
I also looked around for ways to apply the style only to some content.  I found a bunch of stuff like this request that has been around for 7 years and generated lots of interest but apparently no solution.  Note that one is not specific to preview styles.


Insider new welcome page| Prevent close all tabs in the same time

VS Code Version: 1.55.0 Insider 6ebe2a1
OS Version: Windows 10 20H2 Build 19042.572

Should prevent close on click for selected tab, Because Isn't make sense to show photo on the right side and all tabs are closed at the same time.

Does this issue occur when all extensions are disabled?: Yes

I think this changed when keyboard a11y was improved, we should be able to still respect that and just enforce 1 visible element.

@Tyriar added because you reported it was odd for Enter on an item to not do anything. The alternative suggested at the time was to trigger the button's action instead, though now that we're planning on allowing multiple buttons per item I'm not sure what a good action for the button should be.

@JacksonKearl yeah you're right.
What if instead the item expands on focus (click or tab) and enter doesn't toggle visibility at all? So track the active item and switch it whenever focus changes (:focus + :focus-within?). As for what enter should do if we did that, maybe move focus to the first button if it exists?

Changed a bit ago so that clicking will now toggle the item's completion. Tried having triggering the button move focus to the first inner button but it felt strange to click on one area and have a different area focused.


compatibility issue

This doesn't work on 32-bit Windows 7.

Resolved in v0.2. Download the x86 version of the application, ensure you have .NET Core 3.1 installed, and you should be able to run it. If any issues arise as a result of the 32-bit version of Shell City, this issue should be reopened.


Jaeger problem with sending traces for python script

Hi All,
While creating a sample script for python (as below) i am not able to find the traces of the service even though i have used jaeger client libraries for python. I am only seeing jaeger-query service. Can somebody please help
from flask import Flask
from flask import request
import requests
#from opentracing_instrumentation.client_hooks import install_all_patches
from jaeger_client import Config
from flask_opentracing import FlaskTracer
import opentracing

def initialize_tracer():
  config = Config(
         config={
          'sampler': {'type': 'const', 'param': 1},
          'local_agent': {
               'reporting_host': 'jaeger',
               'reporting_port': '6831',
      },
      'logging': True,
  },
  service_name='hello-worldnotworking')
  print("hitesh")
  return config.initialize_tracer() # also sets opentracing.tracer

app = Flask(__name__)
flask_tracer= FlaskTracer(initialize_tracer, trace_all_requests=True, app=app)
#install_all_patches()

@app.route('/')
def pull_requests():

  github_url = "https://api.github.com/repos/opentracing/opentracing-python/pulls"

  parent_span = flask_tracer.get_span()
  with opentracing.tracer.start_span('github-api', child_of=parent_span) as span:
    span.set_tag("http.url",github_url)
    r = requests.get(github_url)
    span.set_tag("http.status_code", r.status_code)

  with opentracing.tracer.start_span('parse-json', child_of=parent_span) as span:
    json = r.json()
    span.set_tag("pull_requests", len(json))
    pull_request_titles = map(lambda item: item['title'], json)

  return 'OpenTracing Pull Requests: ' + ', '.join(pull_request_titles)

if __name__ == '__main__':
    app.run(debug=True,host='<IP_ADDRESS>', port=5000)

I am running the below command to make jaeger run as container.
sudo docker run   --rm   --name jaeger   -p6831:6831/udp   -p16686:16686   jaegertracing/all-in-one:latest

Have you followed the troubleshooting guide?
https://www.jaegertracing.io/docs/1.14/troubleshooting/

This is issue with your configuration of python tracer and not Jaeger server.
The python client supports only UDP sender see https://www.jaegertracing.io/docs/1.14/client-features/ and https://github.com/jaegertracing/jaeger-client-python#getting-started.
You have incorrectly configured python tracer. The jaeger host is localhost - you expose UDP port 6830 to your host OS.


Optimize string concatenation by default

From Qt docs:

... multiple uses of the [QString] '+' operator usually means multiple memory allocations. When concatenating n substrings, where n > 2, there can be as many as n - 1 calls to the memory allocator.

With this PR

... the '+' will automatically be performed as the QStringBuilder '%' everywhere.

The change in the src/Makefile.qt.include file does not justify submitting this PR into the main repo, IMHO.

@Talkless
Could you look into this PR?

@Talkless

It's true that QtCreator, and Qt itself is built with that flag, but we might want to discuss if we want that "implicit magic".

 Discussion is a good mean to reach (rough) consensus about suggested changes.

There is some caveat with this kind of code:

You code snippet could be fixed by s/auto/QString/, and such caveats could be documented in https://github.com/bitcoin-core/bitcoin-devwiki/wiki/Developer-Notes-for-Qt-Code.

Updated b4e1480c02e845ab1df573d92acb814f62a172ea -> a02c970eb001b456d74ddc30750fe8b55348ddac (pr313.02 -> pr313.03, diff):

addressed @Talkless's comments

You code snippet could be fixed by s/auto/QString/

Yes, but that's the "danger", as one needs to know these kinda caveats. Yes, docs could help.

Code review ACK a02c970eb001b456d74ddc30750fe8b55348ddac

This simplifies and reduces the mental burden of having to think about "am I concatenating a string in the most efficient way possible?"

I don't think there is any place in bitcoin where concatenating Qt strings is actually a bottleneck, but agree with this perspective.

It's true that QtCreator, and Qt itself is built with that flag,

That's reassuring also in the sense that it probably won't just go away.

Approach ACK, this looks like a good idea indeed.


[cocoon] Migrate flaky issue bot to null safety

Related to https://github.com/flutter/cocoon/pull/1246
For various reasons, these tests directly call postJSON which is a generic function. This requires a fallback generator to support (which was done for github webhooks). However, the code here is calling it in a way that does generate the same function signature with what flaky tests is expecting.
Check flaky Can file issue and pr for framework host-only test [E]
  MissingStubError: 'postJSON'
  No stub was found which matches the arguments of this method call:
  postJSON('/repos/flutter/flutter/pulls/123/requested_reviewers', {statusCode: null, fail: null, headers: null, params: null, convert: Closure: (Map<String, dynamic>) => PullRequest, body: {"reviewers":["HansMuller"]}, preview: null})

  Add a stub for this method using Mockito's 'when' API, or generate the mock for MockGitHub with 'returnNullOnMissingStub: true'.
  package:mockito/src/mock.dart 190:7                                              Mock._noSuchMethod
  package:mockito/src/mock.dart 184:45                                             Mock.noSuchMethod
  test/src/utilities/mocks.mocks.dart 2272:14                                      MockGitHub.postJSON
  package:cocoon_service/src/service/github_service.dart 139:18                    GithubService.assignReviewer
  package:cocoon_service/src/request_handlers/file_flaky_issue_and_pr.dart 115:18  FileFlakyIssueAndPR._fileIssueAndPR
  ===== asynchronous gap ===========================
  package:cocoon_service/src/request_handlers/file_flaky_issue_and_pr.dart 49:7    FileFlakyIssueAndPR.get
  ===== asynchronous gap ===========================
  test/request_handlers/file_flaky_issue_and_pr_test.dart 232:18                   main.<fn>.<fn>

\cc @keyonghan @chunhtai

@chunhtai Is this something you can help us? Now most unit tests related to flake bot are marked as skipped due to this issue, which means we are missing unit tests for corresponding code changes. Thanks!

sure i can try to squeeze this into my schedule


how could I track the shape of blob like Detectron?

❓ Questions and Help
Hi, I'd like to track and see the shape of each blob during training. Similar to Detectron, the shape of every blob in each layer (operation) could be shown in the first iteration. It is very helpful to check whether the network is running well. So is there any way to see the same thing in maskrcnn-benchmark?

You can put prints in the code to output the shapes during forward maybe?
There is no graph construction step in PyTorch, as every image can be of potentially different sizes, so you'd need to do something in the forward to be able to get this.


Cleanup of deprecated classes

Cleanup of deprecated classes.
Corresponding Java PRs:
https://github.com/hazelcast/hazelcast/pull/15012/files
https://github.com/hazelcast/hazelcast/pull/14983

Linux test FAILed.

Windows test PASSed.

Windows test PASSed.

Linux test FAILed.

Windows test PASSed.

Linux test PASSed.


decoder.py: ValueError: No JSON object could be decoded

Hi,
I got this after upgrading to 1.2.491, then. checking config & starting flexget daemon:
2016-03-18 09:03 WARNING  check                         Config line 4 is likely missing ':' at the end
2016-03-18 09:03 WARNING  check                         Config line 10 is likely missing ':' at the end
2016-03-18 09:03 VERBOSE  check                         Pre-checked 401 configuration lines
Could not start manager: No JSON object could be decoded
2016-03-18 09:03 ERROR    main                          No JSON object could be decoded
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flexget/__init__.py", line 37, in main
    manager.start()
  File "/usr/local/lib/python2.7/dist-packages/flexget/manager.py", line 302, in start
    self.initialize()
  File "/usr/local/lib/python2.7/dist-packages/flexget/manager.py", line 202, in initialize
    fire_event('manager.startup', self)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 103, in fire_event
    result = event(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 22, in __call__
    return self.func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/simple_persistence.py", line 172, in load_taskless
    SimplePersistence.load()
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/simple_persistence.py", line 135, in load
    cls.class_store[task][skv.plugin][skv.key] = skv.value
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/attributes.py", line 293, in __get__
    return self.descriptor.__get__(instance, owner)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 138, in getter
    return json.loads(getattr(self, name), decode_datetime=True)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/json.py", line 105, in loads
    return json.loads(*args, **kwargs)
  File "/usr/lib/python2.7/json/__init__.py", line 351, in loads
    return cls(encoding=encoding, **kw).decode(s)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/json.py", line 41, in decode
    return super(DTDecoder, self).decode(obj, **kwargs)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

Isn't #955 related? Or its commit: 9bf8668?

please paste full debug log

The command I ran after stopping flexget:
sudo pip install -U flexget && flexget -L debug check && flexget -L debug daemon start

The output:
[...]
Installing collected packages: flexget
  Found existing installation: FlexGet 1.2.483
    Uninstalling FlexGet-1.2.483:
      Successfully uninstalled FlexGet-1.2.483
Successfully installed flexget-1.2.491
2016-03-18 11:02 DEBUG    manager                       Figuring out config load paths
2016-03-18 11:02 DEBUG    manager                       Found config: /home/osmc/.flexget/config.yml
2016-03-18 11:02 DEBUG    manager                       Config file /home/osmc/.flexget/config.yml selected
2016-03-18 11:02 DEBUG    manager                       sys.defaultencoding: ascii
2016-03-18 11:02 DEBUG    manager                       sys.getfilesystemencoding: UTF-8
2016-03-18 11:02 DEBUG    manager                       os.path.supports_unicode_filenames: False
2016-03-18 11:02 DEBUG    plugin                        Trying to load plugins from: [u'/home/osmc/.flexget/plugins', '/usr/local/lib/python2.7/dist-packages/flexget/plugins']
2016-03-18 11:02 DEBUG    safe                          Reading from cache file /tmp/safe-0.4.words.cache
2016-03-18 11:02 DEBUG    plugin                        Plugin `memusage` requires `ext lib `guppy`` to load.
2016-03-18 11:02 DEBUG    plugin                        Plugins took 22.78 seconds to load
2016-03-18 11:02 DEBUG    manager                       Connecting to: sqlite:////home/osmc/.flexget/db-config.sqlite
2016-03-18 11:02 WARNING  check                         Config line 4 is likely missing ':' at the end
2016-03-18 11:02 WARNING  check                         Config line 10 is likely missing ':' at the end
2016-03-18 11:02 VERBOSE  check                         Pre-checked 401 configuration lines
2016-03-18 11:02 DEBUG    manager                       config_name: config
2016-03-18 11:02 DEBUG    manager                       config_base: /home/osmc/.flexget
2016-03-18 11:02 DEBUG    manager                       New config data loaded.
2016-03-18 11:03 DEBUG    manager                       Removed /home/osmc/.flexget/.config-lock
Could not start manager: No JSON object could be decoded
2016-03-18 11:03 ERROR    main                          No JSON object could be decoded
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flexget/__init__.py", line 37, in main
    manager.start()
  File "/usr/local/lib/python2.7/dist-packages/flexget/manager.py", line 302, in start
    self.initialize()
  File "/usr/local/lib/python2.7/dist-packages/flexget/manager.py", line 202, in initialize
    fire_event('manager.startup', self)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 103, in fire_event
    result = event(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 22, in __call__
    return self.func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/simple_persistence.py", line 172, in load_taskless
    SimplePersistence.load()
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/simple_persistence.py", line 135, in load
    cls.class_store[task][skv.plugin][skv.key] = skv.value
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/attributes.py", line 293, in __get__
    return self.descriptor.__get__(instance, owner)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 138, in getter
    return json.loads(getattr(self, name), decode_datetime=True)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/json.py", line 105, in loads
    return json.loads(*args, **kwargs)
  File "/usr/lib/python2.7/json/__init__.py", line 351, in loads
    return cls(encoding=encoding, **kw).decode(s)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/json.py", line 41, in decode
    return super(DTDecoder, self).decode(obj, **kwargs)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

This is due to an older version of requests..
I've updated in dev, new release will go out today.

Ignore that, it's not fixed yet

@antivirtel I think this was due to the database failing to upgrade properly or in the middle of broken flexget versions.
Can you run the below and try again
 flexget database reset-plugin simple_persistence

I'm getting the same traceback with the latest version (1.2.493):
$ flexget -L debug database reset-plugin simple_persistence
2016-03-22 07:38 DEBUG    manager                       Figuring out config load paths
2016-03-22 07:38 DEBUG    manager                       Found config: /home/osmc/.flexget/config.yml
2016-03-22 07:38 DEBUG    manager                       Config file /home/osmc/.flexget/config.yml selected
2016-03-22 07:38 DEBUG    manager                       sys.defaultencoding: ascii
2016-03-22 07:38 DEBUG    manager                       sys.getfilesystemencoding: UTF-8
2016-03-22 07:38 DEBUG    manager                       os.path.supports_unicode_filenames: False
2016-03-22 07:38 DEBUG    plugin                        Trying to load plugins from: [u'/home/osmc/.flexget/plugins', '/usr/local/lib/python2.7/dist-packages/flexget/plugins']
2016-03-22 07:38 DEBUG    safe                          Reading from cache file /tmp/safe-0.4.words.cache
2016-03-22 07:38 DEBUG    plugin                        Plugin `memusage` requires `ext lib `guppy`` to load.
2016-03-22 07:38 DEBUG    plugin                        Plugins took 22.38 seconds to load
2016-03-22 07:39 DEBUG    manager                       Connecting to: sqlite:////home/osmc/.flexget/db-config.sqlite
2016-03-22 07:39 DEBUG    manager                       config_name: config
2016-03-22 07:39 DEBUG    manager                       config_base: /home/osmc/.flexget
2016-03-22 07:39 DEBUG    manager                       New config data loaded.
2016-03-22 07:39 DEBUG    manager                       Removed /home/osmc/.flexget/.config-lock
Could not start manager: No JSON object could be decoded
2016-03-22 07:39 ERROR    main                          No JSON object could be decoded
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/flexget/__init__.py", line 37, in main
    manager.start()
  File "/usr/local/lib/python2.7/dist-packages/flexget/manager.py", line 302, in start
    self.initialize()
  File "/usr/local/lib/python2.7/dist-packages/flexget/manager.py", line 202, in initialize
    fire_event('manager.startup', self)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 103, in fire_event
    result = event(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/event.py", line 22, in __call__
    return self.func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/simple_persistence.py", line 172, in load_taskless
    SimplePersistence.load()
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/simple_persistence.py", line 135, in load
    cls.class_store[task][skv.plugin][skv.key] = skv.value
  File "/usr/local/lib/python2.7/dist-packages/sqlalchemy/orm/attributes.py", line 293, in __get__
    return self.descriptor.__get__(instance, owner)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/database.py", line 138, in getter
    return json.loads(getattr(self, name), decode_datetime=True)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/json.py", line 105, in loads
    return json.loads(*args, **kwargs)
  File "/usr/lib/python2.7/json/__init__.py", line 351, in loads
    return cls(encoding=encoding, **kw).decode(s)
  File "/usr/local/lib/python2.7/dist-packages/flexget/utils/json.py", line 41, in decode
    return super(DTDecoder, self).decode(obj, **kwargs)
  File "/usr/lib/python2.7/json/decoder.py", line 366, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python2.7/json/decoder.py", line 384, in raw_decode
    raise ValueError("No JSON object could be decoded")
ValueError: No JSON object could be decoded

After I've rolled back to 1.2.483 (the last working version I could run), I was able to run that command:
$ flexget -L debug database reset-plugin simple_persistence
2016-03-22 07:50 DEBUG    manager                       Figuring out config load paths
2016-03-22 07:50 DEBUG    manager                       Found config: /home/osmc/.flexget/config.yml
2016-03-22 07:50 DEBUG    manager                       Config file /home/osmc/.flexget/config.yml selected
2016-03-22 07:50 DEBUG    manager                       sys.defaultencoding: ascii
2016-03-22 07:50 DEBUG    manager                       sys.getfilesystemencoding: UTF-8
2016-03-22 07:50 DEBUG    manager                       os.path.supports_unicode_filenames: False
There is a FlexGet process already running for this config, sending execution there.
2016-03-22 07:50 DEBUG    manager                       Sending command to running FlexGet process: [u'-L', u'debug', u'database', u'reset-plugin', u'simple_persistence']
2016-03-22 07:51 DEBUG    schema                        Initializing plugin simple_persistence schema version to 2
The database for `simple_persistence` has been reset.

After it, I was upgraded to the latest version, and now it seems to be working. Was that only a database issue? Couldn't it be prevented by catching that exception? Thanks!

That exception was due to a previous issue.


Strange issue in WAMP

Any file change through a windows editor on the example hello script generates weird errors.
I tried analyzing the $uri array in line 173 of Application.php, it has an extra element : empty string right there at line 173 in Application.php
Error Log:

Warning: array_combine() [function.array-combine]: Both parameters should have an equal number of elements in C:\wamp\www\peej-tonic-96f6b39\src\Tonic\Application.php on line 173
Warning: array_shift() expects parameter 1 to be array, boolean given in C:\wamp\www\peej-tonic-96f6b39\src\Tonic\Application.php on line 175
Catchable fatal error: Argument 3 passed to Tonic\Resource::__construct() must be an array, boolean given, called in C:\wamp\www\peej-tonic-96f6b39\src\Tonic\Application.php on line 186 and defined in C:\wamp\www\peej-tonic-96f6b39\src\Tonic\Resource.php on line 14

In my best guess this is a CRLF character problem. As soon as the new line is changed to CRLF, this starts happening.  The exact steps to reproduce:

Run PHP on WAMP (Windows 7 64 bit Home Edition, PHP 5.3 in WAMP)
Run the Tyrell Application in the example setup provided in downloads
Change anything in the Hello example (just any file change and save)
Access the /hello URL on GET and notice the error

The error is not reproduced when same file is ran on a *nix setup.

Working on a legacy system and encountered this problem. The simplest solution has been to ensure the code editor enforces Unix style line endings, thus avoiding the breakage.


fails to start: unexpected end of JSON input

What happened?
cscli/crowdsec is present, generating API key
API Key successfully created
Created symlink /etc/systemd/system/multi-user.target.wants/crowdsec-firewall-bouncer.service → /usr/lib/systemd/system/crowdsec-firewall-bouncer.service.
Job for crowdsec-firewall-bouncer.service failed because the control process exited with error code.
See "systemctl status crowdsec-firewall-bouncer.service" and "journalctl -xeu crowdsec-firewall-bouncer.service" for details.
warning: %post(crowdsec-firewall-bouncer-nftables-0.0.28-1.el9.x86_64) scriptlet failed, exit status 1

Error in POSTIN scriptlet in rpm package crowdsec-firewall-bouncer-nftables
  Verifying        : crowdsec-firewall-bouncer-nftables-0.0.28-1.el9.x86_64                                       1/1

What did you expect to happen?
Install properly
How can we reproduce it (as minimally and precisely as possible)?
sudo yum install crowdsec-firewall-bouncer-nftables

Anything else we need to know?
No response
version
remediation component version:

$ crowdsec-firewall-bouncer --version
version: v0.0.28-el9-rpm-af6e7e25822c2b1a02168b99ebbf8458bc6728e5
BuildDate: 2023-10-02_11:36:45
GoVersion: 1.20.4

crowdsec version
crowdsec version:

$ crowdsec --version
version: v1.6.2-rpm-pragmatic-amd64-16bfab86
Codename: alphaga
BuildDate: 2024-05-31_09:15:35
GoVersion: 1.22.2
Platform: linux
libre2: C++
User-Agent: crowdsec/v1.6.2-rpm-pragmatic-amd64-16bfab86-linux
Constraint_parser: >= 1.0, <= 3.0
Constraint_scenario: >= 1.0, <= 3.0
Constraint_api: v1
Constraint_acquis: >= 1.0, < 2.0

OS version

# On Linux:
$ cat /etc/os-release
NAME="AlmaLinux"
VERSION="9.4 (Seafoam Ocelot)"
$ uname -r
5.14.0-70.22.1.el9_0.x86_64

Additional logs aren't really helpful:
Jun 11 11:51:22 wg crowdsec-firewall-bouncer[75757]: time="11-06-2024 11:51:22" level=fatal msg="process terminated with error: bouncer stream halted"
Jun 11 11:51:22 wg systemd[1]: crowdsec-firewall-bouncer.service: Main process exited, code=exited, status=1/FAILURE

Added better logging:
{"file":"root.go:246","level":"info","msg":"Processing new and deleted decisions . . .","time":"2024-06-11T12:48:11-04:00"}
{"file":"stream_bouncer.go:172","level":"error","msg":"API error: http code 404, no error message","time":"2024-06-11T12:48:11-04:00"}
{"file":"root.go:36","level":"info","msg":"Shutting down backend","time":"2024-06-11T12:48:11-04:00"}
{"file":"nftables_context.go:377","level":"info","msg":"removing 'crowdsec' table","time":"2024-06-11T12:48:11-04:00"}
{"file":"nftables_context.go:377","level":"info","msg":"removing 'crowdsec6' table","time":"2024-06-11T12:48:11-04:00"}

Part that fails is here:
https://github.com/crowdsecurity/go-cs-bouncer/blob/7cc974ab5476febd0afeaa2ecb062ad043255938/stream_bouncer.go#L154-L182

Turns out it was a port conflict with headscale and the main crowdsec service didn't error out so didn't catch the issue until I ran lsof


docker-compose examples are out of date

Details
What steps did you take and what happened:
Attempted to setup lidarr exporter using the docker-compose example.
What did you expect to happen:
Container should start and stay running. Metrics should be available for scraping.  Instead the container exited immediately with the following:
lidarr_prom_exporter    | No help topic for 'exportarr'
lidarr_prom_exporter exited with code 3

Anything else you would like to add:
I believe the following change is needed.
- command: ["exportarr", "lidarr"]
+ command: ["lidarr"]

I'm guessing this change may be valid for the other *arr setups as well.
Lidarr exporter starts and runs as expected after above change.

Gotcha. No I'm on master tag.
I had some confusion because I seem to have been using the 2 year old images on Docker Hub, not realizing that the new stuff was only on GCR. So I'm in the process of updating all my exportarr images.  I was using the master tag because that (a veeerrryyy long time ago apparently) was the only sort of "latest" available on Docker Hub.

I'll be sure to update the docs and examples on a new release.


stack-deploy()

[Keep an eye on aws/aws-cli#2828]
WIP #125 - quick hack which worked. I want to think it through before merging.

Accept parameter file used by stack-create(), stack-update()

$ jp --unquoted --filename pc-proxy-params-staging.json  "join(' ', @[].join('=', [ParameterKey, ParameterValue])[])"
HostedZone=example.com KeyName=example-ap-southeast-2 TargetPort=8080 VpcStackName=vpc-example

or withjq:
$ aws cloudformation deploy --parameter-overrides $(jq -r '.[] | [.ParameterKey, .ParameterValue] | join("=")' param.json)

stack-deploy loses ability to tail events
I don't see it giving us much over stack-create/stack-update


[FSDE-1364] Version bump

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Playpen/job/ChiGCR/job/ChiGCR/job/PR-40/1/. :white_check_mark:


PyPi Integration (Requirements)

added LICENSE.txt - requirement to integrate with PyPi

realized we already satisfied License requirement


UI: Project column in Default View

Description
The Default View has a projects toggle button that allows users to see which resources belong to projects, but there is nothing to indicate which project they belong to.
For this reason, a project column was added if the projects button is enabled, indicating the name of the project to which the resource belongs.
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[X] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[X] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
I created the resources that can be listed outside of a project (instance, guest network, VPC, etc.). Then, I enabled the projects button in the Default View and verified that the project column was filled with the name of the project that the resources belong to.

in my opinion, if project is displayed, account field can be hidden (or not set)

When the project is displayed, both account and project resources are listed (except in listNetworks API). I don't think hiding the account column makes sense in this case.

@blueorangutan ui

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: https://qa.cloudstack.cloud/simulator/pr/8287 (QA-JID-244)

@blueorangutan ui

@DaanHoogland a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

checked on QA server: https://qa.cloudstack.cloud/client/pr/8287/#/vm
it works in some pages: instance snapshots, CKS, ssh key pairs, affinity groups
it does not work on: Instances, AutoScale instance group, Userdata
on "Instance groups", "Project" column is right after "Account", which is different as other pages.

@hsato03 I check the UI after changes. Currently, the Project column is added to the list of columns but it is not displayed by default and it needs to be checked. Would it make sense to make sure it is visible in the view on toggle?

Yes, thank's for your suggestion. I made the changes to address that.

checked on QA server: https://qa.cloudstack.cloud/client/pr/8287/#/vm
it works in some pages: instance snapshots, CKS, ssh key pairs, affinity groups
it does not work on: Instances, AutoScale instance group, Userdata on "Instance groups", "Project" column is right after "Account", which is different as other pages.

That is weird, I also checked at https://qa.cloudstack.cloud/client/pr/8287/#/vm and it is working on Instances and on AutoScale Instance Group. The listUserdata API does not have a project attribute, it returns both account and project in the Account field.
I normalized the order that Project and Account are displayed.

@blueorangutan ui

@shwstppr a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

@weizhouapache if you approve this, we may consider it for 4.19


Add saveWorkoutRouteSamples

Is your feature request related to a problem? Please describe.
We have a fetch method to get workout route samples, but we don't have a way to save workout route samples.
Describe the solution you'd like
Add saveWorkoutRouteSamples to the library of methods to save workout route samples to Apple Healthkit.
Additional
I plan on working on this shortly. I created this as a reminder.

Any updates on this?


Add kubemci list to list existing multi cluster ingresses

Users can use this to see the list of existing multi cluster load balancers to come up with a unique name for their new load balancer

Fixed by https://github.com/GoogleCloudPlatform/k8s-multicluster-ingress/pull/91


Made a minor adjustment in instruction

Preventing confusion regards to variable use of item and the instruction use of the word.  Revised the instruction to clarify the elements inside the array to be used, instead of variable named item.

This looks good.  Can you please squash your commits?

I think we need to pick which way we go with this wording. item at the front of the queue and first element of the array both describe the same thing. This PR has a mashup of both wordings rather than just one.

@ltegman I am open to rewording, however the original instruction had me assuming that it wanted the item (variable) to be pushed to the array (correct assumption here), then it stated "then remove and return the item at the front of the queue."  I erroneously assume that the "item" in that instruction was in reference to the variable item hence same "value", not a general reference of the elements in the array.  Took me a long while to figure out that the item in instruction was not part of the variable item in that second part.
I respectfully disagree that it is a "same thing"..when you put front of queue, you are essentially (at least in this part of instruction) putting a value in front of function called queue, which also use another variable called arr (array).  This array is intergral part of the function, but is not the array itself.  When I finally passed, my understanding is that you put the item (variable in the function) in end of the array, remove the first element of the array, and return that value.

I'd suggest either Add the item onto the end of the array, then remove and return the element at the front of the queue. or Add the item onto the end of the array, then remove and return the first element of the array., whichever seems more clear to you.


add 'convert to epub/mobi' option

Would be great to have it in epub/mobi so it could be read on an e-book reader.

This script from GitHub Open Source guide may be used as an example, though it’s far from perfect: https://github.com/github/opensource.guide/blob/gh-pages/script/ebooks


Combine dialogue open states in catalogue

I think we can combine the deleteCategoryDialogOpen, saveAs, edit react states and use a single one e.g. https://github.com/ral-facilities/inventory-management-system/blob/94d9ff67eb418659c64590155c7f8f4c4c054f07/src/systems/systems.component.tsx#L239-L242. (May also be able to use a single instance of the dialog as well https://github.com/ral-facilities/inventory-management-system/blob/94d9ff67eb418659c64590155c7f8f4c4c054f07/src/systems/systems.component.tsx#L392-L407)

This is fine however it makes the code less readable.

Yeah it something to think about after MVP has been delivered


[Go live][March 31]

Push to live

Docs Build status updates of commit 03a5c11:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/get-started/get-started-overview.md
:white_check_mark:Succeeded
View

msteams-platform/mstdd-landing.yml
:white_check_mark:Succeeded
View

msteams-platform/sbs-gs-blazorupdate.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


FundPsbt and EstimateFee improvements

FundPsbt and EstimateFee now support spending from the underlying
wallet using unconfirmed inputs or inputs with a specific number of
minumum confirmations; this is made possible by adding the
params min_confs and spend_unconfirmed to the underlying proto
messages, and implementing checks in their function calls.
This commit also changes the interface of WalletController slightly; it
adds the param minConfs to CreateSimpleTx inside the
WalletController interface, thus allowing the underlying
implementation of the function to pass minConfs to its
underlying wallet's createsimpletx implementation (if possible and/or desired).
Pull Request Checklist

[x] If this is your first time contributing, we recommend you read the Code
Contribution Guidelines
[x] All changes are Go version 1.12 compliant
[x] The code being submitted is commented according to Code Documentation and Commenting
[x] For new code: Code is accompanied by tests which exercise both
the positive and negative (error paths) conditions (if applicable)
[ ] For bug fixes: Code is accompanied by new tests which trigger
the bug being fixed to prevent regressions
[x] Any new logging statements use an appropriate subsystem and
logging level
[x] Code has been formatted with go fmt
[x] Protobuf files (lnrpc/**/*.proto) have been formatted with
make rpc-format and compiled with make rpc
[x] New configuration flags have been added to sample-lnd.conf
[x] For code and documentation: lines are wrapped at 80 characters
(the tab character should be counted as 8 characters, not 4, as some IDEs do
per default)
[ ] Running make check does not fail any tests
[ ] Running go vet does not report any issues
[x] Running make lint does not report any new issues that did not
already exist
[ ] All commits build properly and pass tests. Only in exceptional
cases it can be justifiable to violate this condition. In that case, the
reason should be stated in the commit message.
[x] Commits have a logical structure according to Ideal Git Commit Structure

@jalavosus was the repo for this PR removed by accident? Or what happened?
Was going to review this now, since 0.12.0 is almost out. Or should I instead close this and review #4905?

@jalavosus was the repo for this PR removed by accident? Or what happened?
Was going to review this now, since 0.12.0 is almost out. Or should I instead close this and review #4905?

@jalavosus I went ahead and applied the patch for this commit to xplorfin/lnd#11 so if you want to change the base branch to xplorfin/fundpsbt we should be good to go here

@jalavosus I went ahead and applied the patch for this commit to xplorfin/lnd#11 so if you want to change the base branch to xplorfin/fundpsbt we should be good to go here

We should either proceed with this or #4905

We should either proceed with this or #4905

Closing this due to inactivity and in favor of #4905.


[CLI] Consider using configuration-tools

As much as I treasure the simplicity of the non-configuration-tools approach, I think that the various features which come for free from configuration-tools could be a big win for usability here.

Support for configuration files (even remote ones too, it seems?)
An approach to configuring utilities which is uniform across our services

I think that, once you learn the pattern, configuration-tools is simple. But it adds boilerplate code, unfortunately a lot of boilerplate code. So maybe it's too simple, in not providing a more sophisticate machinery for generating the boilerplate...
Let me know if I can provide help with this issue.


TfLite toco: fail kTensorflowMerge

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux Ubuntu 18.04 Beta
TensorFlow installed from (source or binary):
Source
TensorFlow version (use command below):
Master
Python version:
3.6
Bazel version (if compiling from source):
0.10.0
GCC/Compiler version (if compiling from source):
gcc 4.8.5
CUDA/cuDNN version:
9.1 / 7.0.5
GPU model and memory:
GTX 1070 Ti 6go & 16Go RAM
Exact command to reproduce:
bazel-bin/tensorflow/contrib/lite/toco/toco
--input_file=frozen_graph.pb
--input_format=TENSORFLOW_GRAPHDEF
--output_format=TFLITE
--output_file=/tmp/test.tflite
--inference_input_type=FLOAT
--input_arrays=input_image
--output_arrays=output_corner,output_mask
--input_shapes=1,512,512,3

Describe the problem
I'm working on a FCN. I want to export my frozen graph to TfLite graph but it's not working. I'm wondering why, but I guess it's just due to the lack of maturity of TfLite (and probably some operator compatibility). In my case, it's about the operator "Merge". Someone can explain me if there is a way/trick to make it work ?
Source code / logs
2018-04-20 12:57:33.287154: I tensorflow/contrib/lite/toco/import_tensorflow.cc:1268] Converting unsupported operation: LogicalOr
2018-04-20 12:57:33.302765: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before Removing unused ops: 518 operators, 965 arrays (0 quantized)
2018-04-20 12:57:33.312501: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] After Removing unused ops pass 1: 477 operators, 902 arrays (0 quantized)
2018-04-20 12:57:33.325133: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 477 operators, 902 arrays (0 quantized)
2018-04-20 12:57:33.325335: F tensorflow/contrib/lite/toco/graph_transformations/resolve_tensorflow_switch.cc:95] Check failed: other_op->type == OperatorType::kTensorFlowMerge
Aborted (core dumped)

I also make a summarize_graph for you, maybe it help
Found 1 possible inputs: (name=input_image, type=int32(3), shape=[1,?,?,3])
No variables spotted.
Found 2 possible outputs: (name=output_corner, op=Tile) (name=output_mask, op=Tile)
Found 1471309 (1.47M) const parameters, 0 (0) variable parameters, and 51 control_edges
Op types used: 239 Const, 168 Switch, 36 FusedBatchNorm, 27 Add, 22 Identity, 21 Merge, 18 Conv2D, 15 Relu, 13 StridedSlice, 12 Shape, 7 Mul, 6 Cast, 5 MaxPool, 5 Pack, 5 RealDiv, 3 Slice, 3 Conv2DBackpropInput, 2 Tile, 2 Reshape, 2 RandomUniform, 2 PlaceholderWithDefault, 2 Greater, 2 Floor, 2 ConcatV2, 1 Placeholder, 1 Maximum, 1 LogicalOr, 1 ResizeBilinear, 1 Sigmoid, 1 Softmax, 1 Squeeze, 1 ExpandDims, 1 Sub, 1 ArgMax

TOCO is expecting a Switch node to be followed by a Merge node, but that doesn't seem to be the case in your model.

Hi,
Thank you for your answer @andrehentz but I don't understand. You say that when I have a Switch node, I necessarily need to have a Merge node ?
So it's not good to have more Switch (168) than Merge (21) ?
For solve this, do I need to add manually a Merge node after every Switch node in my model ? I don't really know how to do that. Why TfLite don't do it by itself ?
Thx again for your time!

Hi Kayoku,
I encountered the same issue, would want to know how you later solved the issue.
Thanks

Hi @adekunleba ,
I didn't. I'm always stuck haha (I work on an other project waiting for a solution here). But if you find something, don't forget to tell me!
Sorry :(

Hi @Kayoku ,
Tried to check the source code and found out it's one of the assertions causing the error.
// There remains to handle the edges that were pointing to the nonselected
  // output. We will just discard those edges. Concretely, at the moment,
  // our only examples of graphs with Switch nodes have them feeding into Merge
  // nodes, so what we're saying here is that we'll make the convention,
  // in our toco internal representation, that Merge nodes with only 1 input
  // are Merge nodes that have been resolved already and should be have as
  // Identity nodes, simply forwarding their input.
  //
  for (const auto& other_op : model->operators) {
    auto input_it = other_op->inputs.begin();
    while (input_it != other_op->inputs.end()) {
      if (*input_it == switch_op->outputs[nonselected_output_index]) {
        // Let us guard our assumption that only Merge nodes consume the outputs
        // of Switch nodes:
        CHECK(other_op->type == OperatorType::kTensorFlowMerge);  //ASSERTION CAUSING THE BOUNCE
        input_it = other_op->inputs.erase(input_it);
      } else {
        ++input_it;
      }
    }
  }

One approach I have tried is to comment that out the line, but seems it's an important check for another codeline, hence it's required to pass. For now, I worked using the tfmobile approach instead, as I look futher to how my architecture can be refactored to meet the merge requirements.

Nagging Assignee @jart: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @jart: It has been 15 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Getting the same fail. Using LSTM.
2018-06-01 21:14:40.418639: F tensorflow/contrib/lite/toco/graph_transformations/resolve_tensorflow_switch.cc:95] Check failed: other_op->type == OperatorType::kTensorFlowMerge

Hello, getting the same fail with next models: "ssd_inception_v2_coco_2017_11_17", "ssd_mobilenet_v1_coco_2017_11_17", "ssd_mobilenet_v2_coco_2018_03_29" .
2018-06-06 14:08:10.450529: I tensorflow/contrib/lite/toco/graph_transformations=/graph_transformations.cc:39] Before Removing unused ops: 4331 operators, 7153 arrays (0 quantized)
2018-06-06 14:08:10.776917: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] After Removing unused ops pass 1: 4276 operators, 7046 arrays (0 quantized)
2018-06-06 14:08:11.215557: I tensorflow/contrib/lite/toco/graph_transformations/graph_transformations.cc:39] Before general graph transformations: 4276 operators, 7046 arrays (0 quantized)
2018-06-06 14:08:11.639476: F tensorflow/contrib/lite/toco/graph_transformations/resolve_tensorflow_switch.cc:95] Check failed: other_op->type == OperatorType::kTensorFlowMerge

Any solutions?

I saw the same issue. Our models are created by keras.
It looks TOCO assumes that merge node is immediately after switch node, but in our models, there are several nodes between switch and merge.
I fixed as below and it works for our models. I am not sure it also works for others' models.
resolve_tensorflow_switch.patch.zip

I am running into the same problem with a retrained Inception V3 model. Seems like Google's own model should be able to be converted into TFLite. Hoping a solution comes from this. This also happens when trying to convert from frozen pb to .mlmodel.

I'm trying to convert a custom model on inception V3 + SSD and running into the same issue. Any workarounds to get the model to tflite format until this is officially fixed?

Same issue here... fail to convert a pretty simple mobile network.

I have this issue when I convert networks with a placeholder as input:
# Make placeholder to define phase is_training = tf.placeholder_with_default(False, shape=(), name="is_training")
Replacing this with
is_training=False
for inference lets me convert the model successfully. Making this manual change is not ideal though.

@david-haber your w/a worked for me

Nagging Assignee @andrehentz: It has been 16 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

TensorFlow Lite doesn't support control flow ops yet (e.g. Switch, Merge, Cond...etc). The team is looking into broadening the support.
It's not possible to make Switch/Merge work properly with dynamic RNNs.
TFLite already implemented fused RNN / LSTM ops. However, there is no easy way to produce these TFLite ops through our converter. The team is currently working on a better way to convert these ops. Stay tuned.
====

TOCO is expecting a Switch node to be followed by a Merge node, but that doesn't seem to be the case in your model.

In addition, TOCO can only resolve Switch/Merge in a very specific case (the predicate of Switch need to be constant).
Therefore dynamic RNN won't work, because it requires real control flow when running the model.

I was looking to the file "tensorflow/contrib/lite/toco/model.h" for find "kTensorflowMerge", but it doesn't even exist... :D

It seems kTensorflowMerge was renamed to kMerge.

Anyone got a workaround for this? I am having this problem with mobilenet v2 keras

@arpitkh96 The recommended option is to avoid using Merge in your graph. As far as I know, MobileNetV2 (classification) works with TFLite. Can you file a separate bug with the reproducible steps that are leading to this error. Include the command you are using for converting the model and attach the model.
I am closing the original bug due to lack of recent activity in relation to the original user's problem. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

@gargn This should suffice. https://github.com/arpitkh96/KerasMobilnet/blob/master/Untitled.ipynb
I'll try to open separate issue when I'll have time

@arpitkh96 One quick suggestion is to make sure that you are not passing in a training model by setting keras.backend.set_learning_phase(False).
Additionally, TocoConverter has a from_keras_model_file function that makes it easier to do what you are doing, so I suggest using that. It is broken in 1.10, however, it has been fixed on the tf-nightly and in the 1.11 release.

@gargn  learning phase and tf nightly finally fixed it. Thanks. BTW I was not loading from file because relu6 isn't recognized even after using CustomObjectScope

I don't think this issue has been resolved yet. The current version on master does not allow me to convert models with placeholder_with_default ops in the graph.

TOCO is expecting a Switch node to be followed by a Merge node, but that doesn't seem to be the case in your model.

@andrehentz Do we still want to impose this restriction? Otherwise I could implement a fix that allows us to convert placeholder_with_default ops.

initially i was getting this error " Check failed: IsConstantParameterArray(*model, bn_op->inputs[1]) && IsConstantParameterArray(*model, bn_op->inputs[2]) && IsConstantParameterArray(*model, bn_op->inputs[3]) Batch normalization resolution requires that mean, multiplier and offset arrays be constant." then i changed is_training = tf.placeholder_with_default(False, shape=(), name = "batchnormswitch") . since then i am getting " tensorflow/contrib/lite/toco/graph_transformations/resolve_tensorflow_switch.cc:95] Check failed: other_op->type == OperatorType::kTensorFlowMerge
Aborted (core dumped)
None"
error Kindly help. Thanks in advance.

Anybody have any luck with this?

@sanoguch Where exactly should we add the patch you have given? Which files specifically?

I think it is tensorflow/lite/toco/graph_transformations/resolve_tensorflow_switch.cc could be replaced with this one.
resolve_tensorflow_switch.zip
Actually we gave up using TOCO or even TensorFlow Lite, so I am not actively following this issue. I am not sure it works or not.

After applying the patch (https://github.com/tensorflow/tensorflow/issues/18731#issuecomment-396843328) I get the error below:
TOCO failed. See console for info.
2019-09-02 16:05:31.485705: F tensorflow/lite/toco/graph_transformations/propagate_array_data_types.cc:257] Check failed: op->inputs.size() > 0 (0 vs. 0)
Aborted (core dumped)

Has someone have resolve this issue?, I want to convert the ssd_resnet50_v1 and ssd_inception_v2_coco and both of them throw the OperatorType::kMerge Found StridedSlice as non-selected output from Switch, but only Merge supported.


BrowseName "3:LifetimeCounters" in LADSNodeSet

In LADS Nodeset  3:LifetimeCounters try to overwrite BrowseNames from Machinery and exists twice in nodeset
<UAObject NodeId="ns=4;i=5094" BrowseName="3:LifetimeCounters" ParentNodeId="ns=4;i=5088">
      <Reference ReferenceType="HasTypeDefinition">ns=3;i=1015</Reference>
      -> <UAObjectType NodeId="ns=1;i=1015" BrowseName="1:MachineryLifetimeCounterType">
second:
<UAObject NodeId="ns=4;i=5091" BrowseName="3:LifetimeCounters" ParentNodeId="ns=4;i=5063">
      <Reference ReferenceType="HasTypeDefinition">ns=3;i=1015</Reference>
      -> <UAObjectType NodeId="ns=1;i=1015" BrowseName="1:MachineryLifetimeCounterType">

For ModelCompiler, which seems to work on strings it is hard problem to resolve correct one

Create new Names in other Namespace from LADS Nodeset. This could not be handled by modelcompiler


typo: Double word "to"

@nschonni : Thanks for your contribution! The author, @chackdan, has been notified to review your proposed change.

Thankyou @nschonni
#sign-off


German Translation of getting_started_guide

German Translation of getting_started guide!

Thanks Markus! It looks great! We really appreciate your contribution. Thanks for bringing Ike to the German-speaking Arquillian community.


README should include installing YAJL-Ruby first or include yajl-ruby as a runtime dependency.

The README should inform developers to include YAJL-Ruby first before using json-compare. The other way to go about it would be include yajl-ruby as a runtime dependency.

I just used JSON.parse and it works as expected. I don't think YAJL is a dependency.
In fact this is just a hash comparison right?


setPage not working

Greetings,
I have an error or misplaced line of code :(
I tried to upload a pdf, with pdf to image with each page represented as a -$i.jpg in the same folder.
It works but i now get the following:
attachments/file.pdf
attachments/file-1.jpg
attachments/file-2.jpg
attachments/file-3.jpg
But each jpg is the same, the image is an overlapping of all the pages.
Here is my code:
public function uploadSubmit(UploadRequest $request)
    {

    	foreach ($request->attachments as $attachment) {
			$filename = $attachment->store('public/attachments');

			if($attachment->extension() == "pdf")
			{
				$url = explode('/', $filename)[2];
				$pdf = new \Spatie\PdfToImage\Pdf('{DOMAIN}/storage/attachments/' . $url);
				$newurl = '/var/www/vhosts/{DOMAIN}/httpdocs/storage/app/public/attachments/' . str_replace('.pdf', '-', $url);

				$pages = $pdf->getNumberOfPages();

				for($i = 1; $i < $pages + 1; $i++)
				{
					$pdf->setPage($i)->saveImage($newurl . $i . '.jpg');
				}
			}

            Attachment::create([
                'estimate_id' => $request->estimate_id,
                'space_id' => $request->space_id,
                'contact_id' => $request->contact_id,
                'filename' => $filename,
                'name' => $request->name,
                'description' => $request->description
            ]);
    	}

    	\Session::flash('flash_message_succes', 'Estimates are added');

    	return back();
    }

Could you try it with another pdf?
Freek Van der Herten
https://spatie.be
On Tue, 8 May 2018 at 16:28, striderxfossility<EMAIL_ADDRESS>wrote:

Greetings,
I have an error or misplaced line of code :(
I tried to upload a pdf, with pdf to image with each page represented as a
-$i.jpg in the same folder.
It works but i now get the following:
attachments/file.pdf
attachments/file-1.jpg
attachments/file-2.jpg
attachments/file-3.jpg
But each jpg is the same, the image is an overlapping of all the pages.
Here is my code:
`public function uploadSubmit(UploadRequest $request)
{
foreach ($request->attachments as $attachment) {
$filename = $attachment->store('public/attachments');
  if($attachment->extension() == "pdf")
  {
  	$url = explode('/', $filename)[2];
  	$pdf = new \Spatie\PdfToImage\Pdf('{DOMAIN}/storage/attachments/' . $url);
  	$newurl = '/var/www/vhosts/{DOMAIN}/httpdocs/storage/app/public/attachments/' . str_replace('.pdf', '-', $url);

  	$pages = $pdf->getNumberOfPages();

  	for($i = 1; $i < $pages + 1; $i++)
  	{
  		$pdf->setPage($i)->saveImage($newurl . $i . '.jpg');
  	}
  }

    Attachment::create([
        'estimate_id' => $request->estimate_id,
        'space_id' => $request->space_id,
        'contact_id' => $request->contact_id,
        'filename' => $filename,
        'name' => $request->name,
        'description' => $request->description
    ]);

}
\Session::flash('flash_message_succes', 'Estimates are added');
return back();
}`
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/spatie/pdf-to-image/issues/101, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdiDaRu-zO5bOXgQD_ahekBHVtCliJCks5twauqgaJpZM4T2wBv
.

Greetings, i tried with another pdf, but then i got only page 1 on each jpg

Same issue here - setPage is not workign.


Suggestions: Construction Separation

It would be a benefit to how permissions are set up if it were possible to limit what people were authorized to build via the permissions. I am looking into implementing this into code myself, any pointers would be welcome and I'll gladly handover any progress if you guys want.
thanks again for the best mission in Arma 3!

Glad you like it.
Concerning your suggestion: There is already a very similar topic opened concerning the permission system, so I'll close this one after linking it.
#95


Suggest to R CallFunc

https://github.com/apache/tvm/blob/a1d43c15ac6382831370c6de141bf80888761e70/src/relay/backend/build_module.cc#L106-L115
If there are special requirements for returning, such as returning an Int.
template <typename... Args>
int CallFunc(const std::string& name, Args... args) {
auto pf = mod.GetFunction(name, false);
pf(std::forward(args)...);
return;
}
CallFunc(). It will only match R CallFunc.
solution:  https://github.com/sunjiweiswift/BeautifulCode/tree/master/Template

Thanks @sunjiweiswift do you want to open a PR for this? Given this is not a bug and a code improvement instead I will mark it as closed for now. Thank you for your suggestion


Will the browser used by office 2019 to run add-ins be replaced by Edge as IE 11 is being retired?

[Enter feedback here]

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 00c430f1-bbf5-53b8-a225-4fbe8fc39c3b
Version Independent ID: 0a069a0d-4d35-71ee-53af-8c55fe8548a0
Content: Browsers used by Office Add-ins - Office Add-ins
Content Source: docs/concepts/browsers-used-by-office-web-add-ins.md
Product: non-product-specific
Technology: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @felixjiang,
IE11 will continue to be the browser control for Office 2019. Unless the plan of record changes, the versions listed in the Browsers article will remain the same. To avoid using IE11, you'll need to upgrade to a M365 subscription.
I'm going to close this issue, as there's no actionable changes to make in the documentation. Please let me know if you have any other questions.


W-15439366 flexctl updates and permissions gr 2

Writer's Quality Checklist
Before merging your PR, did you:

[ ] Run spell checker
[ ] Run link checker to check for broken xrefs
[ ] Check for orphan files
[ ] Perform a local build and do a final visual check of your content, including checking for:

Broken images
Dead links
Correct rendering of partials if they are used in your content
Formatting issues, such as:

Misnumbered ordered lists (steps) or incorrectly nested unordered lists
Messed up tables
Proper indentation
Correct header levels

[ ] Receive final review and signoff from:

Technical SME
Product Manager
Editor or peer reviewer
Reporter, if this content is in response to a reported issue (internal or external feedback)

[ ] If applicable, verify that the software actually got released

Merging without approval as Diane previously approved this content here: https://github.com/mulesoft/docs-gateway/pull/546
Nothing has changed except the base branch.


[meta][ll] Vulkan portability issues

Here is a rough list of areas where Vulkan translation to other next-gen APIs is not straightforward.
Related to #1226, #1241, and #1102
cc @msiglreith @JohnColanduoni
Direct3D 12

vkDescriptorPool can't be mapped to ID3D12DescriptorHeap, because only one of the latter can be active (for each type: samplers and non-samplers) at any given time during command encoding
#1352: CopyTextureRegion from a source buffer requires minimum alignment of 0x100 for the rows and 0x200 for the depth slices, while Vulkan supports tight packing.
vkQueueWaitIdle alternative is not available, and working around is far from trivial
vkCmdSetStencil* supports face selection, while OMSetStencilRef and stencil descriptors only support a single value for the reference and masks.

Metal

#1372: MTLArgumentBuffer can't be used to effectively implement vkDescriptorSet, because Vulkan API would not have the information of how the buffer is bound to the shader.

The SwapChain implementation I wrote uses an API that is considered private by Apple on iOS prior to the upcoming iOS 11. If we want to support Metal 1 on iOS that's another sticking point.

Events will probably be hard to implement for both APIs, requiring to split command buffers.

I think we should migrate all the points from  #1241 here, describe them in great detail, and close the other issue as a duplicate.

Not sure if this is the right place to share some of our findings so let me know if I should move this.

vkQueueWaitIdle that can be translated to D3D12 by adding a Signal to a queue, linking it to an Event object and waiting on the event. In Metal you can have a callback when a command buffer finishes executing, and have that trigger a condition variable. glFinish for OpenGL.
The placed footprint in D3D12 force an alignment of 512 for the offset from the start of the buffer. We have found that it is possible to work around this by sometimes doing two copies instead of one (for 2D textures).

@Kangz Thanks for chiming in! I split your comment into corresponding issues (#1383 and #1352). I'd prefer to leave this space for more general concerns.

Issues (Portability API)
Aspects where vulkan portability is not straightforward. In particular, hard to map towards D3D12, which exposes a smaller and more restricted API surface:
Memory mapping
Summary: Vulkan allows to map arbitrary slices of Heap/DeviceMemory. in other APIs mapping is coupled with (sub)resources.
Vulkan

Allocate Heap/DeviceMemory via AllocateMemory, which can be bound to Buffer or Image.
Map/Unmap DeviceMemory via (Un)MapBuffer to read/write data to buffers or images.
DeviceMemory can be only mapped once until it's unmapped again.

D3D12

Allocate memory in heaps and create placed resources inside the heaps, similar to vulkan
ID3D12Resource::Map to map the underlying memory of a resource.
Nested mapping calls are allowed.

Metal
(Metal uses contents for mapping MTLBuffer afaik, whicb basically maps the whole buffer.)
API Design
We currently follow D3D12 in core by mapping Buffers instead of directly memory, which is a clear restriction compared to vulkans model.
Resource views
Summary:

Differ between type of the underlying resource (buffer/view) and usage (SRV/CBV/UAV/..).
Restrictions in compatible formats for views

Vulkan

Vulkan differs between BufferViews and ImageViews.
Additional flag required to create views with different (compatible) format
Compatible format must only match pixel size but can differ in SurfaceType and ChannelType(+ some restrictions for depth-stencil formats)

D3D12

Differ between usage of the views but not in resource type: SRV, CBV (new!), UAV, RTV and DSV
Views can only differ in ChannelType but not in SurfaceType, SurfaceType can only be changed for UAVs

Metal

No real views, but allows to create MTLTexture with shared memory
Format must only match in pixel size

API Design
We currently follow D3D12 in core by exposing SRV/CBV/UAV/RTV/DSV and use enums on Vulkan.

Triangle fans seem like they should be easy to implement just with strips or lists; if I remember I should someday do a good rummage around and see if I can write something that handles them.


fix(k8s/runner): use dynamic client for watches

possible solution to #51

Current coverage is 51.81% (diff: 0.00%)

Merging #53 into master will increase coverage by 0.24%

@@             master        #53   diff @@
==========================================
  Files            21         21
  Lines           415        413     -2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            214        214
+ Misses          172        170     -2
  Partials         29         29

Powered by Codecov. Last update 84d53af...897a026

Currently, I see the following error on startup:
17:53:26 ERROR framework.k8s loops.go:90 binding loop (the server could not find the requested resource (get bindings.steward.deis.io))
17:53:26 CRITC framework.runner runner.go:63 the server could not find the requested resource (get bindings.steward.deis.io)
17:53:26 CRITC  main.go:35 error running steward-framework: the server could not find the requested resource (get bindings.steward.deis.io)

progress as of 4e6794107c2c0d0f1622535c15d6b11e0a2bc3dc: steward-cf starts up, but fails to decode the binding. The loop also does not terminate anymore. Error trace:
^CENG000656:s3-service-provider aaronschlesinger$ ks logs -f invinvible-jag-steward-3292932084-6yeye
18:18:32 INFO   main.go:14 steward-cf version devel starting
18:18:32 INFO  framework.web.api router.go:23 starting API server on port 8080
18:19:11 DEBUG framework.k8s.binding loop.go:51 binding loop received event
18:19:11 ERROR framework.k8s.binding loop.go:65 add binding event handler failed (not a binding)

progress as of 0bc4c38a209fe144f566ea75b1b650ba2213c083: Binding resource is properly decoded from event, but now can't be updated to Pending (probably because we're trying to use a RESTClient to do so instead of the dynamic client). Error trace:
^CENG000656:s3-service-provider aaronschlesinger$ ks logs -f impressive-fly-steward-521021679-t24em
18:35:47 INFO   main.go:14 steward-cf version devel starting
18:35:48 INFO  framework.web.api router.go:23 starting API server on port 8080
18:35:48 DEBUG framework.k8s.binding loop.go:51 binding loop received event
18:35:48 ERROR framework.k8s.binding loop.go:90 error updating binding state to Pending
18:35:48 ERROR framework.k8s.binding loop.go:65 add binding event handler failed (the server does not allow this method on the requested resource)

update as of 26a832a474c5180bf0d75bd2924580b8078b49ad: the watch loop can receive and decode the Binding, and the update function can now properly update the binding to Pending.
This is what the resulting binding TPR looks like:
ENG000656:binding aaronschlesinger$ k get binding.steward.deis.io s3 -o yaml
apiVersion: steward.deis.io/v1
kind: Binding
metadata:
  annotations:
    helm.sh/created: "1479412380"
  creationTimestamp: 2016-11-17T19:53:00Z
  labels:
    chart: claim-0.1.0
    component: wintering-drag-s3-uploader-binding
    release: wintering-drag
    type: broker
  name: s3
  namespace: default
  resourceVersion: "2093204"
  selfLink: /apis/steward.deis.io/v1/namespaces/default/bindings/s3
  uid: 71959610-acff-11e6-9b89-42010a800fc4
spec:
  id: 09e13f8c-8dee-480d-9e29-896ae12534fa
  instance_ref:
    kind: Instance
    name: s3
    namespace: default
  parameters: null
  secret_name: s3-demo-secret
status:
  state: Pending

The following changes need to be made:

Broker's watch and update functions need to use the dynamic client
ServiceClass's create function needs to use the dynamic client
Instance's watch and update functions need to use the dynamic client

Updated ./k8s/instance to use the dynamic client as of cdae9f6b7b4ac144ba19a01322eb7eac4e136e4a

Current status as of 2901430: Brokers can be submitted, and steward-cf will make the GET /v2/catalog request on the backing CF broker API, translate the returned service catalog to ServiceClasses, and write them back to Kubernetes

LGTM

Note that this PR features @arschles' and my own combined efforts. After much back and forth, we're mutually LGTM'ing one another's contributions to this PR.


refactor chat server join room method

Prior to this change, if the server received a message that was not a MediaPacket protobuf message, it would crash while attempting to deserialize it.
I also took the time to refactor the handle a bit into multiple methods

@leon3s done, what do you think?


WIP: Push alerts to screen top and add namespaces to views.

These changes push the alerts that were previously in the right sidebar
to the top of the screen.
Along with that, the namespace of the current view is also added to the
 element so that you can use CSS to deterministically style a
particular view.
A few major things were changed including changing the navbar from a fixed to static position. This means there may be design bugs that I haven’t encountered yet with this.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!

@brockwhittaker we still do want to do this migration... are you planning to push a revised version of it?

@timabbott yep, because the main-page-redesign has a different system for settings page position. I'd rather integrate this after we integrate that.


Potential core  hack for  autoloader

@prondubuisi @jgleeson  - perhaps this will load the CiviCRM_API3_Exception file - it's not  loading due to weird naming. That isn't an issue in core I suspect as the file winds up being otherwise loaded because API_Exception is probably always called first (that being the internal exception which leads to the throwing of external exception CiviCRM_API3_Exception)
--- a/CRM/Core/ClassLoader.php
+++ b/CRM/Core/ClassLoader.php
@@ -205,6 +205,9 @@ class CRM_Core_ClassLoader {
    * @param $class
    */
   public function loadClass($class) {
+    if ($class === 'CiviCRM_API3_Exception') {
+      require_once 'api/Exception.php';
+    }

I tested this fix with psalm only scanning api/v3/examples, and it removed all UndefinedClass errors taking the error count down from 310 to 16.

@prondubuisi if this works for you please create a PR against the CiviCRM repo

I tested this fix with psalm only scanning api/v3/examples, and it removed all UndefinedClass errors taking the error count down from 310 to 16.

@jackgleeson How and where did you implement the fix @eileenmcnaughton any heads up please?

@prondubuisi it's in the core codebase - are you  familiar with reading patches? https://ramblings.narrabilis.com/generating-and-reading-patch-files

@prondubuisi it's in the core codebase - are you familiar with reading patches? https://ramblings.narrabilis.com/generating-and-reading-patch-files

Thanks for sharing, I can't seem to understand it. Am not familiar with patches

@prondubuisi understanding patches is an important skill - I'd take the time to sit down & read through something like this
https://www.pair.com/support/kb/paircloud-diff-and-patch/
or this one
https://www.poftut.com/patch-command-tutorial-with-examples-for-linux/

Figured it out @eileenmcnaughton @jackgleeson You mentioned I can create a pull request and then submit a patch right?

Yes! See how far this gets you https://docs.civicrm.org/dev/en/latest/tools/git/#pr

Thanks

Hello @eileenmcnaughton @jackgleeson here is a link to the pull request for this https://github.com/civicrm/civicrm-core/pull/15043


experiment and play with an ILP-driven ripple-ethereum (testnet) exchange

By the end of this milestone (0.5), we want to be able to exchange XRPs (Ripple) in return for TFTs (Rivine). Currently the plan is to do this using the InterLedger Protocol (ILP).
This desired exchange will look very similar to an exchange between XRPs (Ripple) and ETH (Ethereum). With the difference that we'll swap with Rivine instead of Ethereum, and that Ethereum uses smart contracts to achieve this, while Rivine will use a compiled module for this (this last difference isn't important/noticable for this experiment as in the end all ledgers are made equal due to the shard ILP plugin interface that each ledger, that wishes to participate in the ILP protocol, has to adhere to.
This experiment would help us get beyond the known theory, and give us a glimpse of how the end result of this milestone should look like. It will also help us resolve any open question we still might have:

How are the connectors chosen, or are they not?
Do we have a choice in how connectors work together?
What are the actions required by the initiator (sender)?
What are the actions required by the participant (receiver)?
What would be the transaction cost on the RIpple side, and who pays for this?
What would be the transaction cost on the Ethereum side, and who pays for this?

What would be the transaction cost on the Ethereum side, and who pays for this?

It would cost as much gas as the smart contract behind it consumes. On top of that a contract could add an additional fee that's given to the owner of the contract.

While ILP is interesting, and still something we might want to consider later this year, for now we have no time to push this direction. Instead we'll focus on a much simpeler exchange, mostly driven by specific validator nodes on our network, as well as an ethereum smart contract that we deploy and own.


Define different background color rules for a chart

For example, I want to split an Area Chart in two ->
The first 'x' values of yAxis must have 'x' background, and everything higher than 'n' will have 'xx' background. Something like this:

You can use annotations to achieve this.
  annotations: {
    position: "back",
    yaxis: [
      {
        label: {
          text: " "
        },
        y: 8200,
        y2: 9600,
        fillColor: "#00E396"
      },
      {
        label: {
          text: " "
        },
        y: 7800,
        y2: 8200,
        fillColor: "yellow"
      }
    ]
  }

Codepen demo - https://codepen.io/apexcharts/pen/MNpOOY

Is position still working?


Add crate-specific Result

Description
Add crate-specific Result type.
Checklist

[x] Documentation, including doctests
[x] Git history is linear
[x] Commit messages are descriptive
[x] (optional) Git commit messages follow conventional commits
[x] Code is formatted (cargo fmt)
[x] cargo test
[x] Changes are added to the CHANGELOG

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

26 of 29 (89.66%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 51.97%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/href.rs
3
4
75.0%

src/read.rs
3
4
75.0%

src/write.rs
2
3
66.67%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
947

Relevant Lines:
1113

💛  - Coveralls


maturin develop failed with Mac M1

> maturin develop
                ...
          ld: symbol(s) not found for architecture arm64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

error: could not compile `datafusion-python` due to previous error
💥 maturin failed
  Caused by: Failed to build a native library through cargo
  Caused by: Cargo build finished with "exit status: 101": `MACOSX_DEPLOYMENT_TARGET="11.0" PYO3_ENVIRONMENT_SIGNATURE="cpython-3.9-64bit" PYO3_PYTHON="/Users/hawkinswong/HOME/git/ray-sql/venv/bin/python" PYTHON_SYS_EXECUTABLE="/Users/hawkinswong/HOME/git/ray-sql/venv/bin/python" "cargo" "rustc" "--message-format" "json-render-diagnostics" "--manifest-path" "/Users/hawkinswong/HOME/git/ray-sql/Cargo.toml" "--lib" "--crate-type" "cdylib" "--" "-C" "link-arg=-undefined" "-C" "link-arg=dynamic_lookup" "-C"<EMAIL_ADDRESS>
What I have tried:
change version of datafusion-python datafusion-proto datafusion to 25.0.0.
not working.

@heroWang If you have ld errors on mac M1, you can try to add the following config to your $CARGO_HOME/config or ~/.cargo/config . Or refer to this link error: linking with cc failed: exit code: 1
[target.x86_64-apple-darwin]
rustflags = [
  "-C", "link-arg=-undefined",
  "-C", "link-arg=dynamic_lookup",
]

[target.aarch64-apple-darwin]
rustflags = [
  "-C", "link-arg=-undefined",
  "-C", "link-arg=dynamic_lookup",
]

This can solve the above problem, but I have encountered a new issue with interface conflicts as below。
 Compiling raysql v0.6.0 (/Users/ivanfan/repos/ray-sql)
error[E0603]: struct `CombinedRecordBatchStream` is private
  --> src/shuffle/ray_shuffle/reader.rs:8:39
   |
8   | use datafusion::physical_plan::union::CombinedRecordBatchStream;
   |                                       ^^^^^^^^^^^^^^^^^^^^^^^^^ private struct
   ……
Some errors have detailed explanations: E0308, E0599, E0603.
For more information about an error, try `rustc --explain E0308`.
warning: `raysql` (lib) generated 1 warning
error: could not compile `raysql` (lib) due to 12 previous errors; 1 warning emitted
💥 maturin failed
 Caused by: Failed to build a native library through cargo
 Caused by: Cargo build finished with "exit status: 101": `CARGO_ENCODED_RUSTFLAGS="-C\u{1f}link-arg=-undefined\u{1f}-C\u{1f}link-arg=dynamic_lookup" MACOSX_DEPLOYMENT_TARGET="11.0" PYO3_ENVIRONMENT_SIGNATURE="cpython-3.8-64bit" PYO3_PYTHON="/Users/ivanfan/repos/ray-sql/venv/bin/python" PYTHON_SYS_EXECUTABLE="/Users/ivanfan/repos/ray-sql/venv/bin/python" "cargo" "rustc" "--message-format" "json-render-diagnostics" "--manifest-path" "/Users/ivanfan/repos/ray-sql/Cargo.toml" "--lib" "--crate-type" "cdylib" "--" "-C" "link-arg=-undefined" "-C" "link-arg=dynamic_lookup" "-C"<EMAIL_ADDRESS>


Step is scrubbed when status is used in ruleset without "pending"

When you define a step with ruleset such as:
ruleset:
  status: [success, failure]

the step is scrubbed from ever running.
Currently, you have to supply pending as a status for it to not get scrubbed at compile time, like:
ruleset:
  status: [success, failure, pending]

Consider dropping the need to supply pending since it really works a bit differently conceptually than the other values.
Possible solutions:

don't check status for scrubbing
add pending automagically
others?

This PR contains this fix so users no longer need to provide "pending" in the status to have the ruleset not be scrubbed.
https://github.com/go-vela/types/pull/59


Create only one vertex buffer for each chunk

Doing a draw call for every hex-pillar-section is very inefficient. The goal is to manage only one big vertex (+ index) buffer for each chunk. Then we just have to do one draw call per chunk, which results in less then 100 draw calls for the whole world. With that, the draw call amount is not the bottleneck anymore.
Only having one big buffer also allows us to simplify the mesh and remove redundant points and empty faces. Furthermore we can introduces terrain smoothing if we want to.
Some quick calculation for the worst case (no two adjacent hex pillars have the same height) with only one section per pillar:

We have CHUNK_SIZE² pillars per chunk (currently 16² = 256). We have 2 * 6 points for each pillar, but need to triple those in order to have correct normals (see #60) => 36 vertices per pillar = 9216 vertices per chunk. This could be decreased a lot by terrain smoothing.
All top faces are drawn with 4 triangles (with terrain smoothing it would be 6), double that to get the bottom faces, too.
Each pillar has 6 side faces which are drawn with 2 triangles each. But luckily every side face is shared by two pillars, so we can remove roughly half of those faces. This results in 256*6/2 = 768 side faces = 1536 triangles for the sides.
In sum: 1024 + 1536 = 2560 triangles and 9216 vertices per chunk.

These numbers seems OK to me... It's of course also worth considering not generating vertices and faces at height 0, because the player will never see them anyway.

We will use instancing now to improve rendering performance. This is only needed for #62.

Lowering priority to low since we decided to postpone #62 and instancing is doing its job fine (plant rendering is now the main rendering bottleneck)

Done in #190


Remove the gather_index in DataAssembly

Hi, I suggest removing gather_index in any DataAssembly types due to the following reasons:

it always create bugs when one dimension only has one coordinate. This creates big confusion for new users;
it has no practical use;
occasionally, multi-index creates more trouble than the normal index: e.g., when we need to add more coordinates.

Though this issue is in brainio repo, I am raising it here since it is more relevant in context.

Hi @YingtianDt  - thanks for the suggestion and for opening an issue! I will pass this forward to the rest of the team, and we appreciate the contribution.

related: https://github.com/brain-score/brainio/pull/42

Hi, I am moving the discussion of this to this PR, since me and Martin seem to agree on rather removing gather_index but fixing it (potentially with this PR).
With this said, please still be aware of the downsides of using Multi-index. E.g.,
Take a concrete example here:
a = NeuroidAssembly(     [[0, 1, 2], [3, 4, 5]],     dims = ('presentation', 'neuroid'),     coords = {         'stimulus_id': ('presentation', ['a', 'b']),         'stimulus_type': ('presentation', ['m', 'n']),         'neuroid_id': ('neuroid', ['x', 'y', 'z']),         'neuroid_layer': ('neuroid', ['k', 'f', 'c']),     } )

when adding new coordinates, we have to always use gather_index again to maintain the multi-index. e.g.,
a = a.assign_coords(neuroid_weight = ('neuroid', [0.1, 0.2, 0.3])) a = NeuroidAssembly(a)

when comparing two multi-indices, the ordering of the coordinates within each index matters. e.g.,
b = NeuroidAssembly(  [[0, 1, 2], [3, 4, 5]],  dims = ('presentation', 'neuroid'),  coords = {      'stimulus_type': ('presentation', ['m', 'n']),      'stimulus_id': ('presentation', ['a', 'b']),      'neuroid_id': ('neuroid', ['x', 'y', 'z']),      'neuroid_layer': ('neuroid', ['k', 'f', 'c']),  } ) # here the ordering of coordinates of presentation is reversed

`print(a.presentation == a.presentation)  # this will work
print(a.presentation == b.presentation)  # this gives empty comparision like the following
array([], dtype=bool)
Coordinates:
* presentation          (presentation) MultiIndex
- presentation_level_0  (presentation) object
- presentation_level_1  (presentation) object`