render: improve docstring

Codecov Report

Merging #36 (ce539d3) into main (267d0ac) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   87.62%   87.62%
=======================================
  Files          17       17
  Lines        1067     1067
=======================================
  Hits          935      935
  Misses         91       91
  Partials       41       41


dapr run - "invalid value for --log-level: warning"

In what area(s)?
/area runtime
What version of Dapr?
0.6.0
Expected Behavior
Set dapr run logs to "warning" level
As described by the dapr run --help command
--log-level string        Sets the log verbosity. Valid values are: debug, info, warning, error, fatal, or panic. Default is info (default "info")

Actual Behavior
Error is generated
== DAPR == time="2020-04-23T09:28:18.6164711+01:00" level=fatal msg="invalid value for --log-level: warning" app_id=cart instance=BENSL3 scope=dapr.runtime type=log ver=edge

Steps to Reproduce the Problem
Set --log-level warning on any dapr run command
dapr run --app-id foo --log-level warning myapp

Moving to cli repo as the fix is there.

I created a PR, as it's such a tiny fix 😀


Readme

Library name and version
Azure.AI.Language.Conversations 1.0.0-beta.3
Query/Question
I'm reading the README.md here:
https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/cognitivelanguage/Azure.AI.Language.Conversations/samples/Sample1_AnalyzeConversation_ConversationPrediction.md
I installed Azure.AI.Language.Conversations 1.0.0-beta.3
Sample has as such:
Response response = await client.AnalyzeConversationAsync(RequestContent.Create(data));
but package has as such:
public virtual Response<AnalyzeConversationTaskResult> AnalyzeConversation(string utterance, ConversationsProject project, AnalyzeConversationOptions options = null, CancellationToken cancellationToken = default);
Please advise.  Thanks.
Environment
OS Name:     Windows
OS Version:  10.0.19043
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\6.0.202\
Host (useful for support):
Version: 6.0.4
Commit:  be98e88c76

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.12313146,Cognitive - Text Analytics:0.11184083,Storage:0.110610604'

Realized the api has evolved away from raw json to strongly typed parameters.  Just need to update your docs.

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

For all our SDKs, the documentation in main may not - and in this case doesn't - match the currently published documentation; however, the documentation link from the nuget package and page should take you to the appropriately tagged documentation  but I see it doesn't. All the links in the README reference the main branch.
I'll open a separate issue to track this for all our SDKs. In the meantime, please see https://github.com/Azure/azure-sdk-for-net/tree/Azure.AI.Language.Conversations_1.0.0-beta.3/sdk/keyvault/Azure.Security.KeyVault.Secrets/samples. We'll be publishing new packages soon as well, so for some functionality - like the APIs in this 1.0.0 package - will be fixed and immutable going forward. Still, new APIs like we've added to our 1.1.0 beta (that main shows now) could change so all our packages should linked to appropriate documentation.


making packer/ansible roles user home directory agnostic

WIP

Updating ansible scripts to make them agnostic to home directory based on Ansible as opposed to using /home/vagrant. Resolves #141
collapsing remoterepos and repos roles so that if branch is specified as a variable to ansible, the submodules are updated to the on that branch (as opposed to pinned to where the submodule commits are currently nailed)
added template-packages.json to build RackHD image from debian
packages
added roles for package based installation
updated README for details on how to run (variables needed for ATLAS)
changed default values in post-processor to run from env variables
(NOTE: THIS BREAKS the package push... build model, since it
doesn't have those variables locally in remote builds,  but given
that feature is going away for "free" use, I'm thinking we dont need
to maintain that functionality.

*** BUILD #272 ***
Test Name: test_workflowTasks_put
Error Details: Snmp Discovery Collect != fn_1
-------------------- >> begin captured logging << --------------------
tests.api.v1_1.workflowTasks_tests: INFO: Adding workflow task : {'friendlyName': 'fn_1', 'injectableName': 'in_1', 'properties': {}, 'options': {}, 'implementsTask': 'im_1'}
--------------------- >> end captured logging << ---------------------
Stack Trace:   File "/usr/lib/python2.7/unittest/case.py", line 331, in run
    testMethod()
  File "/usr/lib/python2.7/unittest/case.py", line 1043, in runTest
    self._testFunc()
  File "/usr/local/lib/python2.7/dist-packages/proboscis/case.py", line 296, in testng_method_mistake_capture_func
    compatability.capture_type_error(s_func)
  File "/usr/local/lib/python2.7/dist-packages/proboscis/compatability/exceptions_2_6.py", line 27, in capture_type_error
    func()
  File "/usr/local/lib/python2.7/dist-packages/proboscis/case.py", line 350, in func
    func(test_case.state.get_state())
  File "/var/lib/jenkins/jobs/RackHD/workspace/build/test/tests/api/v1_1/workflowTasks_tests.py", line 84, in test_workflowTasks_put
    assert_equal(readFriendlyName,self.workflowTaskDict.get('friendlyName'))
  File "/usr/local/lib/python2.7/dist-packages/proboscis/asserts.py", line 55, in assert_equal
    raise ASSERTION_ERROR(message)
"Snmp Discovery Collect != fn_1\n-------------------- >> begin captured logging << --------------------\ntests.api.v1_1.workflowTasks_tests: INFO: Adding workflow task : {'friendlyName': 'fn_1', 'injectableName': 'in_1', 'properties': {}, 'options': {}, 'implementsTask': 'im_1'}\n--------------------- >> end captured logging << ---------------------"

^^ resulting package build is v0.22 heckj/rackhd in vagrant if anyone wants to try it out

@RackHD/corecommitters reviews please

:+1:

:+1:


Validation 2 models

Environment

Ember Version: 2.18.2
Ember CLI Version: 2.18.0
Ember CP Validations Version: 3.5.3

Steps to Reproduce
I have a form with fields that are in a a component. Is there a way to validate them in the component? Let me try to explain this
//model/customer
export default Model.extend({
  firstName: DS.attr('string'),
  lastName: DS.attr('string'),
  phoneCarrierId: DS.attr('number'),
});

//model/settings
export default Model.extend({
  phoneCarriers: DS.attr('array'),
  phoneCarrierRequired: DS.attr('bool')
});

// template/customer/index
    {{#bs-form formLayout="vertical" class="bootstrap-form" novalidate=true model=model.customer onSubmit=(action "saveChanges") as |form|}}
                  {{form.element
            id="firstname"
            controlType="text"
            label="Your first name"
            placeholder="First name"
            property="firstName"
            required=true
            useIcons=false}}

          {{form.element
            id="lastname"
            controlType="text"
            label="Your last name"
            placeholder="Last name"
            property="lastName"
            required=true
            useIcons=false}}

           {{my-component model.customer model.settings}}
    {{/bs-form}}

So the problem I have is the settings model has something I need for CP validations for customer. So I thought inmy-component I could setup separate validation. Or is there a better way?

@jrock2004

Both the forms share a same action?
What actions are you performing inside saveChanges?

I might be working on something similar and looking for an answer too.


Publish iOS binary Framework built from CI

https://github.com/mapzen/ios/issues/77
Also relevant https://github.com/tangrams/ios-framework/issues/5 and https://github.com/tangrams/tangram-es/pull/1003#discussion_r84091941

CircleCI is already configured to build binary frameworks, so we can take this output artifact and push it to an S3 bucket the same way that we do for Android snapshots.

This has been implemented as part of the 0.7.1 release updates.


Fixes#761 : Updated French Translation

Fixes #761
Overview
TL;DR
I have updated the French Translations File.
Review
@iurimatias , Please review.

Hi,
Half our team speaks french fluently. Unfortunately we cannot accept this contribution since the translations are not correct. Some are so much off it seems likely google translate was used here. I'll have to close this ticket.


fix nodejs crud mission wording

per @Ladicek in docs/topics/crud-mission-deploy-booster.adoc
L66:
I think we changed this formulation recently (?). This sounds like Nodeshift is used to install dependencies, which isn't exactly correct.
reference: https://github.com/openshiftio/appdev-documentation/pull/546#discussion_r135702249

This is the commit I had in mind: https://github.com/openshiftio/appdev-documentation/commit/41d29063b23611a6f5a2c87b84bfa421a7f2a0de


Infinite loop in 2.1.2

I loaded 2.1.2.  When I look at the network pane, in both FF and Chrome, it's making thousands of requests a minute.  Every request is successful, but it just seems to poll in a loop with no timer.

I'm using kopf_external_settings.json and it seems that if you don't specify a refresh_rate, it defaults to 0.  Once I put in a reasonably value there, everything was fine.  I'm upgrading from a pretty old version (1.4.x) and didn't have to do that then.


Cannot install Karma because of gyp errors.

I was having problems trying to install a new workspace of some code with npm with Windows 7 and I got this error:
gyp ERR! build error
gyp ERR! stack Error: Can't find "msbuild.exe". Do you have Microsoft Visual Stu
dio C++ 2008+ installed?

I have the Visual Studio Professional 2013 version installed, npm 2.10.1, bower 1.5.2, and I have the 64 bit version for nodejs.
I tried multiple things to get past this, but found this command after a search which worked:
npm install -g node-gyp
and I was able to get grunt serve to work and load my code.
However, I went to install Karma so I could do testing and I got the same type of error again:
gyp ERR! build error
gyp ERR! stack Error: Can't find "msbuild.exe".  System Windows_NT 6.1.7601
gyp ERR! command "node" "C:\\Program Files\\nodejs\\node_modules\\npm\\node_modu
les\\node-gyp\\bin\\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\<myName>\AppData\Roaming\npm\node_modules\karma\node_modu
les\socket.io\node_modules\engine.io\node_modules\ws\node_modules\utf-8-validate

gyp ERR! node -v v0.12.4
gyp ERR! node-gyp -v v1.0.3
gyp ERR! not ok

but it referenced Karma now.
After finding your site, I started reviewing the prerequisites, so I did uninstall Python 2.7.1 and then installed version 2.7.3.  Then I setup the environment variable for the path, which worked for finding the python version from the command line.  Then I installed Microsoft Visual Studio C++ 2013 for Windows Desktop Express version and then the Windows 7 64-bit SDK, I followed this instruction : WindowsSDKDir not being set, and you are sure you have already installed the SDK:
Which loaded up lots of things on my machine, but to no avail.  Is there an order with this installation process or is there something else I'm missing?  I really need to get this installed properly.

That error probably means you need to run vcvarsall.bat that ships with VS, it sets up the environment.  You can also start a terminal from VS that runs it for you.  There's a menu option for it somewhere.

I can duplicate this error now, each time I start with a new project.  I found the file under: C:\Program Files (x86)\Microsoft Visual Studio 11.0\VC>vcvarsall.bat, even though I'm running Visual Studio 2013.  So I tried running the bat file, but I got ERROR: Cannot determine the location of the VS Common Tools folder.
I also found out that security put a new policy in place that prevents npm from getting to the Users folder, unless I manually add a path statement to my command prompt that lets npm find grunt and bower.  Could this be related to this problem or is it unrelated since both are path related?

Well I found another command to use and this worked:  npm install -g --msvs_version=2013 node-gyp rebuild
and got me past the node-gyp issue.

same errors
`
node_modules/karma/node_modules/fsevents

node-gyp rebuild
`


PyQtPlot widget defaults

It turns out that the user can setMouseEnabled() and hideButtons() on the PlotItem passed to the plot function, so maybe we don't need to do that to the underlying PlotWidget in Edifice.
https://pyqtgraph.readthedocs.io/en/latest/api_reference/graphicsItems/plotitem.html
https://github.com/pyedifice/pyedifice/blob/37fd953c1d132fa46c907690cfc372bf0f4baa08/edifice/extra/pyqtgraph_plot.py#L71-L72

Removed these and added them to the docs instead.


Adding Cancel Event to create and edit actions

#1049

@yggg @akveo please check all the PR's, we and many other developers need this package to be updated desperately :)


Ignore specific monitors

Hi,
is it possible to ignore certain monitors in the widget? I have an issue with internal display which flickers everytime DDC is used, so this widget causes this flicker every minute.
Is it possible to ignore a display so it is not queried at all? There are no settings for this available in the widget nor any option in the backend as far as I can see.
IMO it would be sufficient to add some flags to the backend, so I could add them in the widget settings.

Currently no such option is implemented into the backend. If the flickering already happens when running ddcutil detect, there is nothing I can do about it, this could only be addressed on a lower level than this widget or ddcutil.

The gnome widget uses ddcutil detect --brief instead of ddcutil detect, though I don't think this makes a difference. If it doesn't, I don't think there is anything I can do about this.

@davidhi7 I recently investigated the ddcutil options and there's -b you can use for getvcp and setvcp to only target specific monitors by their I2C bus number.
E.g. I can use the following command to get and set brightness on a specific monitor.
ddcutil getvcp 10 -b 6
ddcutil setvcp 10 20 -b 6

So the widget could have an option to only target specific monitors via their I2C bus numbers. What do you think?

the -b or --bus flag is already used everywhere a monitor specific value is set or queried. I am unsure if using --bus could help with ddcutil detect. Could you post your output of ddcutil detect?

detect doesn't support that option, as it claims
detect does not support explicit display option

Hmm, so if you already use -b then it's probably detect that is causing the flickering. Is it required to run every minute?
Also, read this thread https://github.com/rockowitz/ddcutil/issues/292 and noticed they also use ddccontrol, which doesn't cause flickering on my machine. Maybe there could be an alternative backend using ddccontrol.

Implementing an alternative ddc/ci backend is a lot of work for a niche use case, so I cannot work on this anytime soon.
Two other options might be

Disable display polling (so not run ddcutil detect once every minute, but only once when launching Plasma and then only if manually invoked (already planned but not implemented)
Don't allow ddcutil to access the i2c device of your builtin monitor by modifying the relevant udev rule. This is the easiest option if the i2c id of your internal monitor doesn't change often.

Frankly option 1 seems like the easiest thing with most impact. It would basically solve my problem and I don't see any negative effects. If you only do ddcutil detect to find new monitors then I don't see the point of doing that too often. I see it's a bit related to #18. Thanks again for all the work and I suppose we can close this one again, I'll leave that up to you.


0x38 (SPXERR_AUDIO_SYS_LIBRARY_NOT_FOUND) in .net 7.0

I'm getting the following exception:
System.ApplicationException: Exception with an error code: 0x38 (SPXERR_AUDIO_SYS_LIBRARY_NOT_FOUND)
   at Microsoft.CognitiveServices.Speech.Internal.SpxExceptionThrower.ThrowIfFail(IntPtr hr)
   at Microsoft.CognitiveServices.Speech.SpeechSynthesizer.FromConfig(SpeechConfig speechConfig)
   at Microsoft.CognitiveServices.Speech.SpeechSynthesizer..ctor(SpeechConfig speechConfig)

when using azure text to speech in .net 7.0 console application that is deployed in kubernetes using debian 11.6 version.
Note that the application is running without any exception locally only getting this exception when deployed to kubernetes.
I have tried to add this command to docker file
RUN apt-get update && apt-get install -y build-essential libssl-dev ca-certificates libasound2 wget as stated here: https://learn.microsoft.com/en-us/azure/cognitive-services/speech-service/quickstarts/setup-platform?pivots=programming-language-csharp&tabs=linux%2Cdebian%2Cdotnet%2Cjre%2Cmaven%2Cnodejs%2Cmac%2Cpypi: but it didn't help

Found the problem on our side, the command
RUN apt-get update && apt-get install -y build-essential libssl-dev ca-certificates libasound2 wget was not run properly from dockerfile and it wasn't installing the libasound2


allow gauges to increment and decrement. w/ burnettk

To work like https://github.com/prometheus/client_ruby#gauge. This supports use cases like keeping track of process counts if you only know when processes start and stop but don't have access to something like ps.

overall I am fine with this change... we should aim for full parity with:
https://github.com/prometheus/client_golang/blob/master/prometheus/gauge.go#L32-L53
So I guess we are missing set to current time ... but that is easy just to do by hand.

looks good except for that one tiny change

Thanks, change looks good to me!


[Issue (/ Feature Suggestion?)] Conflict w/ Wiremod Tool Base Class

Hello Zet0rz!
First of all, amazing addon, thank you for bringing to Garry's Mod something else I love to play.
After speaking with a Wiremod project admin about why Wiremod-style holographic previews for non-Wiremod tools were appearing (refer to below), they had a quick glance at the nZU code to find that the way that tools are being registered (which I know is described by #35) means that nZU actually mistakenly references Wiremod's stool base class rather than GMod's stool base class. Consequently, the preview hologram mechanic is inherited by nZU's tools.
A demonstration of the issue:

A snippet of what Divran (the project admin) said about why this conflict is occurring:

On to the related feature suggestion part:
You know, a holographic placement preview similar to Wiremod's system would actually be awesome and very useful however not in this state where I have a translucent barrel blocking my vision when I try to place spawners and barricades. Maybe it's something to consider? Though if it's too difficult or time-consuming to implement then I totally understand that development is not easy.
Anyways, thanks for your time and I look forward to a resolution!

I believe the correct way to solve the issue is something like this
First, place the following in any default gmod tool file (lua/weapons/gmod_tool/any.lua) to preserve the references to these objects
nzu.TOOL = TOOL
nzu.ToolObj = ToolObj

then, later, in the nzu.RegisterTool function, use this reference instead of attempting to get a new one
for instance, replace the following
	local stool = weapons.GetStored("gmod_tool")
	if stool then
		local _,any = next(stool.Tool)
		if any then
			local ToolObj = getmetatable(any)
			if ToolObj then
				local o = ToolObj:Create()

with
local o = nzu.ToolObj:Create()

and also replace the following
				stool.Tool[mode] = o

with
nzu.TOOL.Tool[mode] = o

This fix was based on how wire's custom tool object works. However, keep in mind I did not make wire's custom tool object, and I have not even opened gmod to try this solution to see if it actually works. It's possible that TOOL and/or ToolObj are invalidated (ie set to NULL), which would break this solution.
For reference, here's how wire's tool object works. First, in a regular tool file, this code is run:
https://github.com/wiremod/wire/blob/master/lua/weapons/gmod_tool/stools/wire_adv.lua#L4-L7
Then, in tool_loader, the metatable ToolObj reference is preserved here https://github.com/wiremod/wire/blob/master/lua/wire/tool_loader.lua#L22
and tools are created and saved here
https://github.com/wiremod/wire/blob/master/lua/wire/tool_loader.lua#L467-L501
however keep in mind that wire doesn't try to load tools at a later time. It loads all tools instantly, so it could be that this solution won't work for your use case.
Either way, hope this helped.

Wow! Thank you Divran for going to all this effort to see this issue through.
Hopefully this helps Zet0rz resolve this issue, and implement a (potentially) better way of handling custom tools as a bonus!

eh, it wasn't that much effort, I didn't even bother launching gmod hah


Dependencies between steps (within one project)

Hi, I couldn't figure out a way to define dependencies between steps within one project. The Buildkite native way would be to use a key: foo and a depends_on: foo property on two separate steps. But when I do that using Buildpipe and have more than one project, the key: foo would be used over and over again and  I get the error
The key "foo" has already been used by another step in this build

I've tried key: foo-${BUILDPIPE_PROJECT_LABEL} but that didn't work either.
How can I use keys on steps when using Buildpipe?

Hi @jonnylangefeld , are you using the latest version of Buildpipe? We recently added support for depends_on. An example from the tests


Question: How could I map sessions to access tokens?

[x] I read and understood how to enable logging

Ah help me
Assume that I have a system S with some users, and users are authenticated on the S system using their S accounts. Now I have another system T that uses IdentityServer4 which does not own or know about the S users’ accounts. However, if the system S needs T to generate a unique access token for each user and each of his/her sessions, what should I do?

This seems to be a general question about IdentityServer - not a bug report or an issue.
Please use StackOverflow for that. This has the advantage that questions and answers can be easily found by search engines, and that there are more people answering questions than just us.
For IdentityServer3
https://stackoverflow.com/questions/tagged/?tagnames=identityserver3&sort=newest
For IdentityServer4
https://stackoverflow.com/questions/tagged/?tagnames=identityserver4&sort=newest
For commercial support
https://identityserver.io/

I was so sorry about that.


[firebase_messaging] background messaging documentation prob

Describe the bug
It took me about three hours to figure out why my background message handler wasn't working.  The documentation says "Define a top level Dart method to handle background messages."  This is confusing, since there are no top level "methods", only top-level functions.  Naturally I defined a handler in my class (it works for onMessage, ought to work for onBackgroundMessage, right?) and had to dig through the flutter source to figure out the problem.
To Reproduce
Steps to reproduce the behavior:

Define your onBackgroundMessage callback the same way you define your onMessage handler, as a method in the class that calls _firebaseMessaging.configure
Result:
The following NoSuchMethodError was thrown building Builder:
I/flutter (25669): The method 'toRawHandle' was called on null.
I/flutter (25669): Receiver: null
I/flutter (25669): Tried calling: toRawHandle()

Expected behavior
The documentation should be amended to indicate that the handler must be either a top level function, or a static method.
Thanks for the plugin!

@mobileink
Not sure exactly how to solved the problem. How to I make the handler a top level function?

Can you be more specific on how exactly you solved the problem. Tried making the handler static and it broke during compile time

I have the same problem. Can you update the documentation? I am confused by this

Add an Application.java class to your app (where??)
Set name property of application in AndroidManifest.xml (where??, how??)
Define a top level Dart method to handle background messages (where?? at the same time when I configured onMessage callback?)

Thank you for help.

Yes I have the same question as @kleinpetr

I dont understand the part of AndroidManifest.xml. Can I use MainActivity.java generated by Flutter or rewrite this file by Application.java from tutorial. Or add Application.java within MainActivity.java. But how can I set in AnroidManifest.xml ??
<application android:name="cz.example.myapp.Application"></application>
 <application android:name="io.flutter.app.FlutterApplication" android:label="MyApp" android:icon="@mipmap/ic_launcher">

     <!-- <meta-data android:name="com.google.android.geo.API_KEY" android:value="AIzaSyBmpFHEI4rZ7eIenkwaEDYvXGYMxt60yNM"/> -->
     <activity android:name=".MainActivity" android:launchMode="singleTop" android:theme="@style/LaunchTheme" android:configChanges="orientation|keyboardHidden|keyboard|screenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode" android:hardwareAccelerated="true" android:windowSoftInputMode="adjustResize">

         <!-- disable splashScreen -->
         <meta-data android:name="io.flutter.app.android.SplashScreenUntilFirstFrame" android:value="false" />

This is not working

I can't configure firebase at the main.dart because I have some logic about this. For example I am initing the FCM only if user is logged, etc. So I have to configuring firebase inside a class. You sad the background callback must be static, OK I set static method, but still not working.
main.dart
initState(){
  ....
  HelperClass.initFCM(some dependencies);
}

helperClass.dart
class HelperClass
{
final FirebaseMessaging _firebaseMessaging = FirebaseMessaging();

  static Future<dynamic> backgroundMessageHandler(Map<String, dynamic> message) {
  print("_backgroundMessageHandler");
  if (message.containsKey('data')) {
    // Handle data message
    final dynamic data = message['data'];
    print("_backgroundMessageHandler data: ${data}");
  }

  if (message.containsKey('notification')) {
    // Handle notification message
    final dynamic notification = message['notification'];
    print("_backgroundMessageHandler notification: ${notification}");
  }

  Future<void> initFCM(...){
    _firebaseMessaging.configure(
      .....,
      onBackgroundMessage: backgroundMessageHandler,
      .....
      );
  }
}

Okay I don't know how I managed to get mine to work honestly. I just followed all the instructions and commented out the onBackgroundMessage because it was giving fatal compile time errors.
_firebaseMessaging.configure(       //onBackgroundMessage: _backgroundMessageHandler,       onMessage: _messageHandler,       onLaunch: _lauchHandler,       onResume: _resumeHandler,     );
like that but I'm confused why my _backgroundMessageHander top-level function gets triggered even on app terminated or forground.
I also made sure that all my handlers including message, resume, and launch are top-level functions. (NOT IN ANY CLASS). I hope this helps someone because I've been struggling with this for days.

On Wed, Sep 25, 2019 at 2:24 AM kleinpetr<EMAIL_ADDRESS>wrote:

I dont understand the part of AndroidManifest.xml. Can I use
MainActivity.java generated by Flutter or rewrite this file by
Application.java from tutorial. Or add Application.java within
MainActivity.java. But how can I set in AnroidManifest.xml ??

 <!-- <meta-data android:name="com.google.android.geo.API_KEY" android:value="AIzaSyBmpFHEI4rZ7eIenkwaEDYvXGYMxt60yNM"/> -->
 <activity android:name=".MainActivity" android:launchMode="singleTop" android:theme="@style/LaunchTheme" android:configChanges="orientation|keyboardHidden|keyboard|screenSize|locale|layoutDirection|fontScale|screenLayout|density|uiMode" android:hardwareAccelerated="true" android:windowSoftInputMode="adjustResize">

     <!-- disable splashScreen -->
     <meta-data android:name="io.flutter.app.android.SplashScreenUntilFirstFrame" android:value="false" />

This is not working

We cannot help you with "this is not working."  If you can give specifics
we can try.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FirebaseExtended/flutterfire/issues/199?email_source=notifications&email_token=AAES2NDG5LACZCE3VGBGPXDQLMGZRA5CNFSM4IZDYWK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7Q4PYA#issuecomment-534890464,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAES2NFZ6DWCYUW2ZEUJVJDQLMGZRANCNFSM4IZDYWKQ
.

What do you mean top-level handler? I need working with variables and method from the class in calbacks.. is there some chance ro declare onBackgroundMessage callback the same way as other callbacks??

Please update example of firebase_messaging where it handles background messages. Seriously it's very confusing or anyone please upload demo working project for handling onBackground messages.

we can't understand this point

@ektelat It means you need to use your custom Application class instead of the default flutter Application class
You have this in manifets (probably)
<application android:name="io.flutter.app.FlutterApplication" android:label="Name app" android:icon="@mipmap/ic_launcher">
And you need replace io.flutter.app.FlutterApplication to your Application class for example com.yourdomain.Application or or .Application (maybe)

@kleinpetr
i changed it as you told me but the app can't be installed all the time it crashed

stop here....

@ektelat show me your Application.java please

@kleinpetr i get my mistake, i just copy the same example of flutter document so it's with
package io.flutter.plugins.firebasemessagingexample;
i changed it and it's installing now put after it's loaded unfortinatlly it crash again.
after i removed onBackgroundMessage  from messaging configure

it's work now.
so what to do with: onBackgroundMessage i need it
here the myBackgroundMessageHandler:

what is my problem??

@ektelat Yes I stoped in the same moment as you... I don't know where is the problem. I still waiting for update example on pub.dev ...   or some answer here..

I've had so much trouble making any of this work, I don't dare try background stuff.  So I have not attempted to use background notifications.
I have noticed, however, that none of my messages, sent either by the firebase messaging console or the firebase Admin SDK ever reach the system tray.  Doesn't matter that the app is terminated, or background or whatever.  Gmail notifications are still showing up, by mine don't.  Notifications enabled and double checked for my app.
If the app is running, I get the messages onMessage just fine.  Interestingly, if the app is terminated, when it is launched, I get an the same onMessage.  Probably because it never went to the system tray where it would wait for me to tap it.  No tapping would mean no onResume or onLaunch right?
I finally created an app that just tries to do push, nothing else.  Same thing.
Working great on Android.
I could do without the background stuff if my messages would hit the system tray.

Ok I got it worked.
If u send any notification from firebase console it'll never call onBackground...
Try postman or any other to send notification using fcm notification api. And send only data message. Remove title and body from notification parameter... Now you will get onBackground.

That's makes sense, right?  Messages with notifications go to the system tray, so only "data only" messages will trigger background logic. And the console forces you to send title and body, so it's a notification message.
I just can't get a simple notification to show in the system tray.  Wrong thread I guess.
Good to hear you got it working.

FYI what I have found is:

If you send a Notification+Data message (
https://firebase.google.com/docs/cloud-messaging/concept-options#notification-messages-with-optional-data-payload)
then the message will go to the system tray if your app is backgrounded or
terminated. If you tap the message, it will call onLaunch if your app is
terminated, and onResume if your app is backgrounded. In both cases the
data payload is delivered.  It will not call your onBackgroundMessage
routine; evidently that's for pure data messages.

Note that the doc (for the flutter fb messaging package) says data
notifications will be lost if your app is terminated.
HTH,
Gregg
On Mon, Sep 30, 2019 at 2:18 AM Lokesh Jain<EMAIL_ADDRESS>wrote:

Ok I got it worked.
If u send any notification from firebase console it'll never call
onBackground...
Try postman or any other to send notification using fcm notification api.
And send only data message. Remove title and body from notification
parameter... Now you will get onBackground.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FirebaseExtended/flutterfire/issues/199?email_source=notifications&email_token=AAES2NEWN7RKSDQRRHPG4OLQMGR4DA5CNFSM4IZDYWK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD74VWGI#issuecomment-536435481,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAES2NEG6AT6WOVSBF6RRY3QMGR4DANCNFSM4IZDYWKQ
.

@mobileink You're 100% on this one.
When I was working in Xamarin, you actually had to have a second app, especially in iOS, that knew what to do with the incoming data only messages.
My solution was to create a local notification and dump it into the system tray myself so it would show up - but only if I needed it to.  At that point, all my messages were data messages, but I was adding content to them so I could show them if needed.
That approach started with a "logout all device" where I wanted to clear access tokens on every device the user had and ended up adding buttons so actions could be taken directly in the notification in the system tray.

Sorry for the maybe dumb question, but how can i send a notification that triggers such callback?

@nerder for example like this https://medium.com/android-school/test-fcm-notification-with-postman-f91ba08aacc3

On Mon, Sep 23, 2019 at 12:05 PM kleinpetr @.***> wrote:
Here's what worked for me (with names changed to protect the innocent): In [...]

Thank you very much, sir, I guess you have the solution. The official wiki is incomplete, in my opinion.
If you don't mind, can you tell us what values you use for these two dependencies in android\build.gradle (these values below are mine):
classpath 'com.android.tools.build:gradle:3.3.0'
classpath 'com.google.gms:google-services:4.3.0'
In addition to this issue, I've got a very strange behavior :
I defined myBackgroundMessageHandler in my class which has _firebaseMessaging.configure (I've defined it as static, of course)
But, I did NOT add onBackgroundMessage: _backgroundMessageHandler handler to _firebaseMessaging.configure.... Guess what? My background messages are well handled and working as expected !!! (at least, for those when my app is running in the background - I did not test when the app is terminated, as I'm not able to see the print out on Flutter console to confirm if it's working as well or not)
Is this an expected behavior? Or am I missing something in my understanding of this plugin (which is totally possible as well, I'm not very familiar with flutter FCM)
Thanks y'all !

On Mon, Sep 23, 2019 at 12:05 PM kleinpetr @.***> wrote:
Here's what worked for me (with names changed to protect the innocent): In [...]

Thank you very much, sir, I guess you have the solution. The official wiki is incomplete, in my opinion.
If you don't mind, can you tell us what values you use for these two dependencies in android\build.gradle (these values below are mine):
classpath 'com.android.tools.build:gradle:3.3.0'
classpath 'com.google.gms:google-services:4.3.0'
In addition to this issue, I've got a very strange behavior :
I defined myBackgroundMessageHandler in my class which has _firebaseMessaging.configure (I've defined it as static, of course)
But, I did NOT add onBackgroundMessage: _backgroundMessageHandler handler to _firebaseMessaging.configure.... Guess what? My background messages are well handled and working as expected !!! (at least, for those when my app is running in the background - I did not test when the app is terminated, as I'm not able to see the print out on Flutter console to confirm if it's working as well or not)
Is this an expected behavior? Or am I missing something in my understanding of this plugin (which is totally possible as well, I'm not very familiar with flutter FCM)

See my previous response:  https://github.com/FirebaseExtended/flutterfire/issues/199#issuecomment-537486513
I believe this is expected behavior for notification messages with data payload.  A pure data message would be handled by onBackgroundMessage (I think), but that is evidently not yet supported by the plugin.
Gregg

Hello,
My onbackgroundmessage is not getting called even  my app is terminated. How to solve the issue.
I have added my Application class and followed all the things as per pub socumentation.
Not getting any errors.
@iapicca @mobileink Can anyone help ?
Thanks.

El mismo error.

I was having app crash, I am not using the .configure on the Main, I use on another class.
I did delay de config and worked.
Future.delayed(const Duration(milliseconds: 1000), () {
print('Configuring Firebase Messsaging');
messaging.configure(
onBackgroundMessage: onBackgroundMessageHandler,
onMessage: onMessageMessageHandler,
onLaunch: onLaunchMessageHandler,
onResume: onResumeMessageHandler
);
});

@gcervantes2005  I was even trying delays and other suggestions from stckoverflow.
I have asked for the solutions, but no one replied.
So, i read the pub.dev documentation from firebase_messaging again and started it from the scratch again to make sure everything is working correctly step by step.
I found the problem was i was giving wrong path for Application.java in my case.
So i suggest you that, start from scratch again with reading documentation from scratch without getting any baseless sugestions and implementing them.

can someone write Kotlin version of Application.java please?

can someone write Kotlin version of Application.java please?

package com.example.artisanpay

import io.flutter.app.FlutterApplication
import io.flutter.plugin.common.PluginRegistry
import io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback
import io.flutter.plugins.GeneratedPluginRegistrant
import io.flutter.plugins.firebasemessaging.FlutterFirebaseMessagingService

class Application : FlutterApplication(), PluginRegistrantCallback {
    override fun onCreate() {
        super.onCreate()
        FlutterFirebaseMessagingService.setPluginRegistrant(this)
    }

    override fun registerWith(registry: PluginRegistry) {
        GeneratedPluginRegistrant.registerWith(registry)
    }
}```

I was having app crash, I am not using the .configure on the Main, I use on another class.
I did delay de config and worked.
Future.delayed(const Duration(milliseconds: 1000), () {
print('Configuring Firebase Messsaging');
messaging.configure(
onBackgroundMessage: onBackgroundMessageHandler,
onMessage: onMessageMessageHandler,
onLaunch: onLaunchMessageHandler,
onResume: onResumeMessageHandler
);
});

it's worked for me, Thanks!!!

How do I access variables and other plugins from the background message handler?
The callback gets triggered, but everything is null.
We can literally do nothing else but print the message['data']...

How do I access variables and other plugins from the background message handler?
The callback gets triggered, but everything is null.
We can literally do nothing else but print the message['data']...

Any suggestion about this case please ? I have the same problem

Hi @ifs-slaheddine, @hb0nes,
like you i had a hard time managing to make background message handler useful.
Backgound message handler runs in a different isolate, so you don't have access to the Flutter UI memory.
So far the only workaround I found is to use persistent data with flutter SharedPreferences.
Future<void> onBackgroundMessageHandler(Map<String, dynamic> message) async {
      final prefs = await SharedPreferences.getInstance();
      // SharedPreferences could have been modified in the UI isolate, so we have to fetch it from the host platform.
      await prefs.reload();
      // await prefs.getOrSetWhateverYouNeed();
}

And then make your widget (e.g your root page) overrides didChangeAppLifecycleState to process any changes that happened in background.
@override
 Future<void> didChangeAppLifecycleState(AppLifecycleState state) async {
   if (state == AppLifecycleState.resumed) {
     // Shared preferences
     final prefs = await SharedPreferences.getInstance();
     // Since the changes during background notification handler are made in a different isolate, we fetch the latest values from the host platform
     await prefs.reload();
     // Do what you want with prefs.
   }
 }

In my case, I set a flag in shared preferences to determine if the freshly resumed app need to refresh data from webservices.
Hope it will help you

Yeah, a better way to communicate between isolates is to use receiveport
and sendport, in my opinion. I don't have access to our code now but that
was the most robust way. Registering a receiveport with a statically
declared name and using That name to get the instance of that port to send
stuff to, so that the listener can process the event/data. Good luck
On Wed, Feb 12, 2020, 13:31 BazinC<EMAIL_ADDRESS>wrote:

Hi @ifs-slaheddine https://github.com/ifs-slaheddine, @hb0nes
https://github.com/hb0nes,
like you i had a hard time managing to make background message handler
useful.
Backgound message handler runs in a different isolate, so you don't have
access to the Flutter UI memory.
So far the only workaround I found is to use persistent data with flutter
SharedPreferences.
Future onBackgroundMessageHandler(Map<String, dynamic> message) async {
final prefs = await SharedPreferences.getInstance();
// SharedPreferences could have been modified in the UI isolate, so we have to fetch it from the host platform.
await prefs.reload();
// await prefs.getOrSetWhateverYouNeed();
}
And then make your widget (e.g your root page) overrides
didChangeAppLifecycleState to process any changes that happened in
background.
@override
Future didChangeAppLifecycleState(AppLifecycleState state) async {
if (state == AppLifecycleState.resumed) {
// Shared preferences
final prefs = await SharedPreferences.getInstance();
// Since the changes during background notification handler are made in a different isolate, we fetch the latest values from the host platform
await prefs.reload();
// Do what you want with prefs.
}
}
In my case, I set a flag in shared preferences to determine if the freshly
resumed app need to refresh data from webservices.
Hope it will help you
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FirebaseExtended/flutterfire/issues/199?email_source=notifications&email_token=AJIOJVIZNXELX5WKU4ZYMQDRCPTZ7A5CNFSM4IZDYWK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELQTEAQ#issuecomment-585183746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJIOJVJXPYP22GTVGWBT5ATRCPTZ7ANCNFSM4IZDYWKQ
.

I did not try this way. I was not familiar with receive and send ports. Thanks for the tip :)

github.com/FirebaseExtended/flutterfire/issues/1775

Yeah, a better way to communicate between isolates is to use receiveport and sendport, in my opinion. I don't have access to our code now but that was the most robust way. Registering a receiveport with a statically declared name and using That name to get the instance of that port to send stuff to, so that the listener can process the event/data. Good luck
…
On Wed, Feb 12, 2020, 13:31 BazinC @.***> wrote: Hi @ifs-slaheddine https://github.com/ifs-slaheddine, @hb0nes https://github.com/hb0nes, like you i had a hard time managing to make background message handler useful. Backgound message handler runs in a different isolate, so you don't have access to the Flutter UI memory. So far the only workaround I found is to use persistent data with flutter SharedPreferences. Future onBackgroundMessageHandler(Map<String, dynamic> message) async { final prefs = await SharedPreferences.getInstance(); // SharedPreferences could have been modified in the UI isolate, so we have to fetch it from the host platform. await prefs.reload(); // await prefs.getOrSetWhateverYouNeed(); } And then make your widget (e.g your root page) overrides didChangeAppLifecycleState to process any changes that happened in background. @override Future didChangeAppLifecycleState(AppLifecycleState state) async { if (state == AppLifecycleState.resumed) { // Shared preferences final prefs = await SharedPreferences.getInstance(); // Since the changes during background notification handler are made in a different isolate, we fetch the latest values from the host platform await prefs.reload(); // Do what you want with prefs. } } In my case, I set a flag in shared preferences to determine if the freshly resumed app need to refresh data from webservices. Hope it will help you — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#199?email_source=notifications&email_token=AJIOJVIZNXELX5WKU4ZYMQDRCPTZ7A5CNFSM4IZDYWK2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELQTEAQ#issuecomment-585183746>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJIOJVJXPYP22GTVGWBT5ATRCPTZ7ANCNFSM4IZDYWKQ .

Hi Would you please clarify. This i my first contact with isolates and I'm guessing mostly.
My code looks like this. I've got everything going on in the one class.
static ReceivePort receivePort = ReceivePort();
static Future myBackgroundMessageHandler(Map<String, dynamic> message) async {
SendPort sendPort = receivePort.sendPort;
AudioPlayer audioPlayer;
print('background message: $message');
if (message['data']['status'] == 'alert') {
Box.audio.startLoop().then((player) {
sendPort.send(player);
audioPlayer = player;
});
}
return audioPlayer;
}
@override
void initState() {
_orders = widget.document.reference.collection('Order').snapshots();
_queue = widget.document.reference.snapshots();
Box.audio.players.clear();
receivePort.listen((data) {
  print('[isolateToMainStream] $data');
  Box.audio.players.add(data);
});

_firebaseMessaging.configure(
  onMessage: (Map<String, dynamic> message) async {
    print("onMessage: $message");
    if (message['data']['status'] == 'alert') {
      Box.audio.startLoop();
    }
  },
  onBackgroundMessage: myBackgroundMessageHandler,
  onLaunch: (Map<String, dynamic> message) async {
    print("onLaunch: $message");
  },
  onResume: (Map<String, dynamic> message) async {
    print("onResume: $message");
  },
);

means you need to use your custom Application class instead of the default flutter Application class

I also think this should be clearer in the documentation

My question is about the iOS part.
Does onBackMessage Callback also handles the message in the background/Terminated mode on iOS.

Hi @ifs-slaheddine, @hb0nes,
like you i had a hard time managing to make background message handler useful.
Backgound message handler runs in a different isolate, so you don't have access to the Flutter UI memory.
So far the only workaround I found is to use persistent data with flutter SharedPreferences.
Future<void> onBackgroundMessageHandler(Map<String, dynamic> message) async {
      final prefs = await SharedPreferences.getInstance();
      // SharedPreferences could have been modified in the UI isolate, so we have to fetch it from the host platform.
      await prefs.reload();
      // await prefs.getOrSetWhateverYouNeed();
}

And then make your widget (e.g your root page) overrides didChangeAppLifecycleState to process any changes that happened in background.
@override
 Future<void> didChangeAppLifecycleState(AppLifecycleState state) async {
   if (state == AppLifecycleState.resumed) {
     // Shared preferences
     final prefs = await SharedPreferences.getInstance();
     // Since the changes during background notification handler are made in a different isolate, we fetch the latest values from the host platform
     await prefs.reload();
     // Do what you want with prefs.
   }
 }

In my case, I set a flag in shared preferences to determine if the freshly resumed app need to refresh data from webservices.
Hope it will help you

When i try to run SharedPreferences in onBackgroundMessageHandler i will have the following error
MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/shared_preferences)

Hi @ifs-slaheddine, @hb0nes,
like you i had a hard time managing to make background message handler useful.
Backgound message handler runs in a different isolate, so you don't have access to the Flutter UI memory.
So far the only workaround I found is to use persistent data with flutter SharedPreferences.
Future<void> onBackgroundMessageHandler(Map<String, dynamic> message) async {
      final prefs = await SharedPreferences.getInstance();
      // SharedPreferences could have been modified in the UI isolate, so we have to fetch it from the host platform.
      await prefs.reload();
      // await prefs.getOrSetWhateverYouNeed();
}

And then make your widget (e.g your root page) overrides didChangeAppLifecycleState to process any changes that happened in background.
@override
 Future<void> didChangeAppLifecycleState(AppLifecycleState state) async {
   if (state == AppLifecycleState.resumed) {
     // Shared preferences
     final prefs = await SharedPreferences.getInstance();
     // Since the changes during background notification handler are made in a different isolate, we fetch the latest values from the host platform
     await prefs.reload();
     // Do what you want with prefs.
   }
 }

In my case, I set a flag in shared preferences to determine if the freshly resumed app need to refresh data from webservices.
Hope it will help you

When i try to run SharedPreferences in onBackgroundMessageHandler i will have the following error
MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/shared_preferences)

Define your shared preference plugin in application level. like

Hi @ifs-slaheddine, @hb0nes,
like you i had a hard time managing to make background message handler useful.
Backgound message handler runs in a different isolate, so you don't have access to the Flutter UI memory.
So far the only workaround I found is to use persistent data with flutter SharedPreferences.
Future<void> onBackgroundMessageHandler(Map<String, dynamic> message) async {
      final prefs = await SharedPreferences.getInstance();
      // SharedPreferences could have been modified in the UI isolate, so we have to fetch it from the host platform.
      await prefs.reload();
      // await prefs.getOrSetWhateverYouNeed();
}

And then make your widget (e.g your root page) overrides didChangeAppLifecycleState to process any changes that happened in background.
@override
 Future<void> didChangeAppLifecycleState(AppLifecycleState state) async {
   if (state == AppLifecycleState.resumed) {
     // Shared preferences
     final prefs = await SharedPreferences.getInstance();
     // Since the changes during background notification handler are made in a different isolate, we fetch the latest values from the host platform
     await prefs.reload();
     // Do what you want with prefs.
   }
 }

In my case, I set a flag in shared preferences to determine if the freshly resumed app need to refresh data from webservices.
Hope it will help you

When i try to run SharedPreferences in onBackgroundMessageHandler i will have the following error
MissingPluginException(No implementation found for method getAll on channel plugins.flutter.io/shared_preferences)

Define your shared preference plugin in application level. like

Using above method you can show notification also. Am implemented in my project now am able to handle data messages and show notification from background_handler.Its working fine.

Thank you. It's really worked to me.

import io.flutter.app.FlutterApplication
import io.flutter.plugin.common.PluginRegistry
import io.flutter.plugin.common.PluginRegistry.PluginRegistrantCallback
import io.flutter.plugins.GeneratedPluginRegistrant
import io.flutter.plugins.firebasemessaging.FlutterFirebaseMessagingService

class Application : FlutterApplication(), PluginRegistrantCallback {
    override fun onCreate() {
        super.onCreate()
        FlutterFirebaseMessagingService.setPluginRegistrant(this)
    }

    override fun registerWith(registry: PluginRegistry) {
        GeneratedPluginRegistrant.registerWith(registry)
    }
}

Throw error:
mismatch: inferred type is PluginRegistry but FlutterEngine was expected

    GeneratedPluginRegistrant.registerWith(registry)

cast  registry to  (FlutterEngine)
@Override
public void registerWith(PluginRegistry registry) {
GeneratedPluginRegistrant.registerWith((FlutterEngine) registry);
}


pixie: 1356 -> 1367

Motivation for this change
Update pixie to latest git version: https://github.com/pixie-lang/pixie/commit/d76adb041a4968906bf22575fee7a572596e5796
Pixie compiles successfully, but running it fails with:
± ./result/bin/pixie
in internal function run_load_stdlib

in internal function load-ns

in internal function load-file

in internal function load-reader

RuntimeException: <inst pixie.stdlib.Keyword> Var assoc is undefined

@alekcz do you have an idea, what's going on?
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

committed the latest working version https://github.com/NixOS/nixpkgs/commit/d5bc23c45c825edb2e2be5b7ffb7da9362f628fa


[Das Schwarze Auge (TDE) 5 OFFICIAL by Ulisses] v1.5.5 - New Feature - Ranged attack roll preview

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.

[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).

[x] The pull request makes changes to files in only one sub-folder.

[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

[x] This sheet has been made on behalf of, or by, the game's publisher.

[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >

[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Enhancement

Ranged weapons show now the final roll value including the modifiers

Character Sheet Info Roll20 Internal Use only.


feat(leak): custom cache cleaning itself periodically

The implementation is almost the same of the SDK one. The main difference is the goroutine cleaning each interval all entries too old

related to https://github.com/newrelic/nri-kubernetes/issues/332


Page speed running on debian with mpm-worker + fcgi causes lots of errors written to apache log

From<EMAIL_ADDRESS>on November 03, 2010 18:50:26
What steps will reproduce the problem? 1. Installed package, configured for with domain and turned on settings as below. What is the expected output? What do you see instead? The /var/log/apache2/error.log filled up with this more than 11K lines per second.
[Thu Nov 04 09:28:50 2010] [error] [1104/092850:ERROR:net/instaweb/
apache/serf_url_async_fetcher.cc(674)] Poll success status 0 (111)
[Thu Nov 04 09:28:50 2010] [error] [1104/092850:ERROR:net/instaweb/
apache/serf_url_async_fetcher.cc(674)] Poll success status 0 (111)
[Thu Nov 04 09:28:50 2010] [error] [1104/092850:ERROR:net/instaweb/
apache/serf_url_async_fetcher.cc(674)] Poll success status 0 (111) What version of the product are you using? On what operating system? 1. Setup Lenny 32-bit + apache package 2.2.9 + MPM_worker + fcgid running php site
2. Setup Pagespeed: ModPagespeedEnableFilters collapse_whitespace,rewrite_images,remove_comments Please provide any additional information below.
Original issue: http://code.google.com/p/modpagespeed/issues/detail?id=6

From<EMAIL_ADDRESS>on November 03, 2010 20:56:49
Thanks for the report.
We have seen a few of these reports on mod-pagespeed-discuss and believe it is related to usage of the threaded MPM, which we have not yet tested.  Our initial offering is tested on the pre-fork MPM.
If you would like to help adapt mod_pagespeed to the Threaded MPM we'd love to work with you.

From<EMAIL_ADDRESS>on November 04, 2010 09:43:10
I believe I have the same problem: I'm using Ubuntu 64 bit + MPM:
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/system.css,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/system.css,FEdVYr:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/templates/system/css/system.css [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/general.css,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/general.css,OchlMV:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/templates/system/css/general.css [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/zorgsaam_homepage/css/template.css,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/zorgsaam_homepage/css/template.css,gr1Szp:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/templates/zorgsaam_homepage/css/template.css [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/mootools.js,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/mootools.js,JJBInT:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/media/system/js/mootools.js [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/caption.js,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/caption.js,SGVFbn:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/media/system/js/caption.js [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/modules/mod_fhw_fontadjust/js/mod_fhw-fontadjust-px.js,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/modules/mod_fhw_fontadjust/js/mod_fhw-fontadjust-px.js,7LhFZQ:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/modules/mod_fhw_fontadjust/js/mod_fhw-fontadjust-px.js [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex...

From<EMAIL_ADDRESS>on November 04, 2010 09:43:10
...: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/mootools.js,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/mootools.js,0uXYNk:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/media/system/js/mootools.js [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/caption.js,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/media/system/js/caption.js,af2pCO:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/media/system/js/caption.js [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A:0: creating dir (code=13 Permission denied)
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/system.css,
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(54)] /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/system.css,NBk1qi:0: open temp file (code=2 No such file or directory)
[Thu Nov 04 17:36:27 2010] [notice] [1104/173627:INFO:net/instaweb/apache/serf_url_async_fetcher.cc(684)] Fetch complete: http://www.zorgsaam.org/templates/system/css/system.css [Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_time_duration_ms cannot lock mutex: Permission denied
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/apache/apr_statistics.cc(97)] Variable serf_fetch_bytes_count cannot lock mutex: Permission denied

From<EMAIL_ADDRESS>on November 05, 2010 08:02:56
Marking as Enhancement, to support previously unsupported threaded MPM.
Labels: -Type-Defect -Priority-Medium Type-Enhancement Priority-High

From<EMAIL_ADDRESS>on November 11, 2010 00:53:26
I was not complete the last time: I'm using mpm-itk. It's based on the traditional prefork MPM: http://mpm-itk.sesse.net/

From<EMAIL_ADDRESS>on November 11, 2010 03:15:05
There are two unrelated issues here.
john.clegg.nz reports Poll success status 0 (111) which means "timeout requesting URL".  We've seen this elsewhere and need to track it down.
leonbogaert reports issues that look to me like the directories set up for mod_pagespeed to store cache files and other files are not writable.  Leon, can you investigate that?
So Leon, I think you have a setup issue.  The first error you reported was:
[Thu Nov 04 17:36:27 2010] [error] [1104/173627:ERROR:net/instaweb/util/google_message_handler.cc(35)] Could not create directories for file /var/mod_pagespeed/cache/http,3A/,/www.zorgsaam.org/templates/system/css/system.css,
Can you take a look at whether your httpd server has write access to /var/mod_pagespeed/cache?  If not, you can edit pagespeed.conf to point to a location where you do have write access.
And both of you: please be sure you are using our latest binary distribution, which was released Nov 8, 2010.
Could you verify by looking at the X-Mod-Pagespeed header in Firebug from your HTML file?

From<EMAIL_ADDRESS>on November 16, 2010 20:50:57
Fixed in https://code.google.com/p/modpagespeed/source/detail?r=215
Status: Fixed


onValueChanged cancelable ?

Hi there
It would be nice to be able to cancel a value change with onValueChanged.
Use Case / example :

A checkbox to activate geolocating / notifications : anything related to Android System Permission
When activating the switch, we show a permission popup (async)

if user refuses : we want the switch to not change its states. so we would invalidate the change
if user accepts: let the switch state be changed

or any other case needing a custom check
I implemented it in a custom Preference and the changes are really small but maybe that could/should be a core feature of your lib :
fun PreferenceScope.PreferenceBool(
    ...
    onValueChange: (selected: Boolean) -> Boolean,
    ...
) {
   ...
   if (onValueChange(it)) {
        checked = it
   }
   ...
}

Without this, it's painfull to revert the switch back on the "disabled" state. I needed to play with delay() to update the preference storage AFTER the onValueChanged finished its job and actually enabled the switch

Are you talking about the KotPreference implemention?
Otherwise, it's really simple:

create a state
pass state.value as value to the preference
update or not update the state.value in the onValueChange callback

I will add the cancel option to the KotPreference implementation though

I just saw, that the bool preference did hold an unnecessary internal state that prevented it from being cancelable. Should be possible now and I also added this feature for the KotPreference version

Great thank you, it works now
No I wasn't using the KOT implementation


Issue with container

Hi there,
Sorry if this is an obvious answer, but I am getting the following error when I attempt to post a message showing using your demo code:
Notice: Undefined variable: container
How do I go about fixing this?
Thanks

$container from the example refers to the Symfony container. A service social_post will be registered to it.
You can also instantiate the core service on your own (if you don't use Symfony for example) and this will require some boiler-plate code from your end.
Later this year/Early 2018 I plan to release the core logic as a decoupled package with examples how to integrate into non-symfony projects.

Awesome thanks, Got it working!

Perfect! Closing this as seems your question is answered now.


Hey made some changes

Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

hey


feat: disable pre-commit-terraform hook for dynamic_values of SLZ module

Description
variables inside dynamic_values of SLZ module shouldn't be strongly typed
Check the relevant boxes:

[ ] Bug fix (nonbreaking change that fixes an issue)
[ ] New feature (nonbreaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Examples or tests (addition or updates of examples or tests)
[ ] Documentation update
[ ] CI related update (pipeline, etc.)

Checklist

[ ] If relevant, a test for the change has been added or updated as part of this PR.
[ ] If relevant, documentation for the change has been added or updated as part of this PR.

Merge

Merge using "Squash and merge".
Make sure to use a relevant conventional commit message that is based on the PR contents. The commit message determines whether a new version of the modules needs to be released, and if so, which semver number to use).

@daniel-butler-irl @akocbek - It's rather hacky as applying to all modules, when the change is specific to the landing-zone one. Could we at least put an inline comment in the file to put context and pointer. Also we probably want to create a task to brainstorm neater solution for repo-specific overrides.


Show errors when backups fail to enable

Closes #640

Coverage increased (+0.2%) to 84.126% when pulling 46db27034bad34714313c0af44bb3490b47270f2 on briansteffens:enable-backups-error into 965c740fe753be12bdb9f3f755afe94a0a79fd20 on linode:master.

Looks good!


AttributeError: module 'orm' has no attribute 'ModelRegistry'

To replicate,
from fastapi import FastAPI
import orm
import databases

app = FastAPI()

database = databases.Database("sqlite:///db.sqlite")
models = orm.ModelRegistry(database=database)

class Note(orm.Model):
    tablename = "notes"
    registry = models
    fields = {
        "id": orm.Integer(primary_key=True),
        "text": orm.String(max_length=100),
        "completed": orm.Boolean(default=False),
    }

@app.get("/")
async def index():
	await models.create_all()
	await Note.objects.create(text="Buy the groceries.", completed=False)
	note = await Note.objects.get(id=1)
	print(note)
	return {}

Thanks for reporting this, can you provide the stack trace and python/orm versions you use?
I can't reproduce this.

INFO:     Uvicorn running on http://<IP_ADDRESS>:5000 (Press CTRL+C to quit)
INFO:     Started reloader process [57991] using statreload
Process SpawnProcess-1:
Traceback (most recent call last):
  File "/usr/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/subprocess.py", line 61, in subprocess_started
    target(sockets=sockets)
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/server.py", line 49, in run
    loop.run_until_complete(self.serve(sockets=sockets))
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/server.py", line 56, in serve
    config.load()
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/config.py", line 308, in load
    self.loaded_app = import_from_string(self.app)
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/importer.py", line 20, in import_from_string
    module = importlib.import_module(module_str)
  File "/usr/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 848, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/henshalb/Pictures/ds/app/./main.py", line 8, in <module>
    models = orm.ModelRegistry(database=database)
AttributeError: module 'orm' has no attribute 'ModelRegistry'
^CINFO:     Stopping reloader process [57991]

Process SpawnProcess-1:
Traceback (most recent call last):
  File "/usr/lib/python3.8/multiprocessing/process.py", line 315, in _bootstrap
    self.run()
  File "/usr/lib/python3.8/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/subprocess.py", line 61, in subprocess_started
    target(sockets=sockets)
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/server.py", line 49, in run
    loop.run_until_complete(self.serve(sockets=sockets))
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/server.py", line 56, in serve
    config.load()
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/config.py", line 308, in load
    self.loaded_app = import_from_string(self.app)
  File "/home/henshalb/.local/lib/python3.8/site-packages/uvicorn/importer.py", line 20, in import_from_string
    module = importlib.import_module(module_str)
  File "/usr/lib/python3.8/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1014, in _gcd_import
  File "<frozen importlib._bootstrap>", line 991, in _find_and_load
  File "<frozen importlib._bootstrap>", line 975, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 671, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 848, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "/home/henshalb/Pictures/ds/app/./main.py", line 8, in <module>
    models = orm.ModelRegistry(database=database)
AttributeError: module 'orm' has no attribute 'ModelRegistry'

What version of orm are you using?
Have you created a module named orm in your project?

(venv) henshalb@henshalb-IdeaPad-3-15IIL05:~/Pictures/ds/app$ which python/home/henshalb/Pictures/ds/venv/bin/python
(venv) henshalb@henshalb-IdeaPad-3-15IIL05:~/Pictures/ds/app$ pip show ormName: orm
Version: 0.3.1
Summary: An async ORM.
Home-page: https://github.com/encode/orm
Author: Tom Christie
Author-email<EMAIL_ADDRESS>License: BSD
Location: /home/henshalb/Pictures/ds/venv/lib/python3.7/site-packages
Requires: typesystem, databases
Required-by:
(venv) henshalb@henshalb-IdeaPad-3-15IIL05:~/Pictures/ds/app$

Have you accidentally created a module named orm in your project?

What about this?

No

I think something's wrong with your setup. I don't think it's really related to the orm.
The uvicorn seems to be called outside of your env, maybe installed globally? And IDE python version is showing 3.7 but traceback is is showing python 3.8.
I think you need to check your setup.

Okay, let me create a new one. Closing it.


FIX: Use len(segments) to avoid nsegments becoming wrong

Another in the series of handling carpetplots in degenerate cases that pretty much only come up on CI. Here, we have a variable nsegments = len(segments), but segments can be pared down if there are empty segments.
<100ns to call len() on a dict 4 times seems like a good trade over possibly getting de-synced again in the future.

Codecov Report
Patch coverage: 100.00% and project coverage change: -10.70 :warning:

Comparison is base (10e8c96) 72.96% compared to head (f36ffea) 62.27%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #795       +/-   ##
===========================================
- Coverage   72.96%   62.27%   -10.70%
===========================================
  Files          76       49       -27
  Lines        7547     5967     -1580
  Branches      883      883
===========================================
- Hits         5507     3716     -1791
- Misses       2004     2071       +67
- Partials       36      180      +144

Flag
Coverage Δ

reportlettests
?

unittests
?

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

Impacted Files
Coverage Δ

niworkflows/viz/plots.py
68.14% <100.00%> (-8.12%)
:arrow_down:

... and 52 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

I assume no objections and will merge in the morning, but feel free to make any opinions known before then.


Message board / Discuss clone in Dgraph GraphQL

Note: I want to merge all the commits here so I can reference them in the blog, so I won't squash.
I'm also going to be a bad boy and force push to this branch so the commit history looks like I want.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: vardhanapoorv:white_check_mark: MichaelJCompton:x: gjaYou have signed the CLA already but the status is still pending? Let us recheck it.


Is it a good idea to stop using console print and move to SL4j?

Need to think about it.
@hemil-ruparel-blox

We found out it should probably be better to use sl4j.

This is done, closing it.


build: Reachability check of deployed dev cluster

Description

Adding a dev cluster reachability check to our CI-Pipeline. This helps to visualize where the pipeline breaks.
This improvement is necessary, since before most often CI flows failed on postman – even though the postman tests themselves where correct. Most often only the dev cluster – though deployed correctly – was not reachable from the outside.
Now we can be sure that the cluster was successfully deployed AND is reachable from outside (e.g. for the following CI steps).
Changelog

Implemented the reachability check
Added the workflow to our CI-Pipeline

Checklist

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

(Optional) Visual Changes

I'm not qualified to review this, but sounds like a great change! 👍

Already Implemented in https://github.com/inovex/scrumlr.io/pull/4088


[WIP] Added documentation for fallback condition for complex columns in local Dictionary

What has been changed?

Added documentation for fallback condition for complex columns in local Dictionary
Added documentation for system level property "carbon.local.dictionary.decoder.fallback"

[x] Any interfaces changed?
NA
[x] Any backward compatibility impacted?
NA
[x] Document update required?
Document Updated
[x] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.
NA
[x] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
NA

Build Failed  with Spark 2.3.1, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/8758/

LGTM


Mask not working

Hi,
I'm working on a Kaggle proyect: https://www.kaggle.com/c/hpa-single-cell-image-classification
When I load the test data and apply the segmentator.pred_cells() function, it all works ok.
But when I apply the label_cell() function, the array returned is empty (all 0's):
Cell Segmentation
plt.imshow(cell_segmentations[0])

Cell mask
nuclei_mask = label_nuclei(nuc_segmentations[0])
cell_nuclei_mask, cell_mask = label_cell(nuc_segmentations[0], cell_segmentations[0])
plt.imshow(nuclei_mask)
plt.imshow(cell_nuclei_mask)
plt.imshow(cell_mask)

I'll leave the code here, although the same code has worked with the training data...
image = test_files[0]
paths = [os.path.join(TEST, path) for path in image]

arrays = image_to_arrays(paths)
nuclei = arrays[1]
nuc_segmentations = segmentator.pred_nuclei([nuclei])

inter_step = [[i] for i in paths[:-1]]
cell_segmentations = segmentator.pred_cells(inter_step)

f, ax = plt.subplots(1, 2, figsize=(16,16))
ax[0].imshow(get_blended_image(arrays))
ax[0].set_title('Original Cells', size=20)
ax[1].imshow(cell_segmentations[0])
ax[1].set_title('Segmented Cells', size=20)
plt.show()

nuclei_mask = label_nuclei(nuc_segmentations[0])
cell_nuclei_mask, cell_mask = label_cell(nuc_segmentations[0], cell_segmentations[0])

f, ax = plt.subplots(1, 3, figsize=(16,16))
ax[0].imshow(nuclei_mask)
ax[0].set_title('Nuclei Mask', size=20)
ax[1].imshow(cell_nuclei_mask)
ax[1].set_title('Cell Nuclei Mask', size=20)
ax[2].imshow(cell_mask)
ax[2].set_title('Cell Mask', size=20)
plt.show()

Thank you in advance!
Escanor

The code itself looks like it should be correct. How do you create your segmentator object? Many problems seems to be solved if you set padding=True during instantiation.

Hi @cwinsnes
I tried just that and it still doesn't work...
What could it be?
The weirdest part is that the segmentation works... I don't get it
:)

The test images in the Kaggle set are 16bit while the train ones are 8bit, as discussed in https://www.kaggle.com/c/hpa-single-cell-image-classification/discussion/223745
As the profile images and names are alike, I assume the cv2 solution helped you there?


Shedinja/ninjask from the same nincada show as clones

https://www.reddit.com/r/porybox/comments/5e0xe5/im_not_sure_if_i_should_call_this_a_bug_since_its/
Place to discuss what the best thing to do it.

I still think it would be best to leave it but add a section somewhere in the FAQ on natural clones (I remember hearing someone talk about WSHMKRs in IRC chat that theirs ended up as a clone)

Create a special check for shedinja/ninjask if this is an issue that needs to be fixed

Will add more comments if I can think of a way to deal with this that doesn't involve changing too much


fix(tsk): Remove non-existent example

Example does not seem to exist. Lets remove it for now, if somebody wants to add it again that is fine, but this breaks stuff, right now.

Fair enough. Example still exists on master, but is merely a test binary for trying to re-add Kubernetes support. That's definitely not in scope right now xD
For posterity's sake: examples removed in 807fd927e88f1a0b65d1ced6d97a87ec45beac82


Health checking at the end of setup script

should check if there is 200 response when requesting

some AEM page on publish via dispatcher
AEM author repl agent publish
AEM publish repl agent flush

implement it as task (https://taskfile.dev/usage/#getting-started) and use curl


Why I got square like segmentation predictions even though training with non-square binary map?

Above row shows non-square segmentation map ground truth
Why I got square like segmentation predictions such as below row?

sorry i can not help you like that. I have seen square-like segmentation predictions  in the first training epochs of mask rcnn, but you would need to provide more information in order to receive useful feedback.


WIP: Add docker:// source strategy

This will allow to specify the source location that points into a directory inside Docker image. For example:
$ s2i build docker://sample-app-sources:/sample-app openshift/ruby-20-centos7 final-app

In this case, we will pull (or not pull as force-pull applies here) the sample-app-sources Docker image, then run it and stream the /sample-app directory using tar. Once we have this directory locally, we just set the source location to "file://local-folder" and execute the regular S2I build.
The output looks like this (in this example the sample-app-sources is just a busybox image with sources ADD to /sample-app:
I1009 13:39:31.884721 28057 util.go:31] Getting docker credentials for openshift/ruby-20-centos7
I1009 13:39:31.884855 28057 util.go:43] Using<EMAIL_ADDRESS>credentials for pulling openshift/ruby-20-centos7
I1009 13:39:31.890272 28057 docker.go:214] Image openshift/ruby-20-centos7:latest available locally

Builder Name:		Ruby 2.0
Builder Image:		openshift/ruby-20-centos7
Builder Image Version:	004e164
Builder Base Version:	71a650e
Source:			docker://sample-app-sources:/sample-app
Output Image Tag:	final-app
Incremental Build:	disabled
Remove Old Build:	disabled
Force Pull:		disabled
Quiet:			disabled
Layered Build:		disabled
Docker Endpoint:	unix:///var/run/docker.sock
Docker Pull Config:	/root/.docker/config.json
Docker Pull User:	mfojtik

I1009 13:39:31.891933 28057 docker.go:214] Image openshift/ruby-20-centos7:latest available locally
I1009 13:39:31.892988 28057 sti.go:128] Preparing to build final-app
I1009 13:39:31.893852 28057 docker.go:214] Image sample-app-sources:latest available locally
I1009 13:39:31.893872 28057 copy.go:47] Downloading sources from "sample-app-sources" to "/tmp/sti594925303/upload/sources"
W1009 13:39:31.894670 28057 docker.go:329] Image does not contain a value for the io.openshift.s2i.scripts-url label
I1009 13:39:31.894694 28057 docker.go:384] Base directory for STI scripts is ''. Untarring destination is '/tmp'.
I1009 13:39:31.894703 28057 docker.go:515] Creating container using config: {Hostname: Domainname: User: Memory:0 MemorySwap:0 CPUShares:0 CPUSet: AttachStdin:false AttachStdout:true AttachStderr:false PortSpecs:[] ExposedPorts:map[] Tty:false OpenStdin:false StdinOnce:false Env:[] Cmd:[tar cf - /sample-app] DNS:[] Image:sample-app-sources:latest Volumes:map[] VolumesFrom: WorkingDir: MacAddress: Entrypoint:[] NetworkDisabled:false SecurityOpts:[] OnBuild:[] Mounts:[] Labels:map[]}
I1009 13:39:32.197016 28057 docker.go:526] Attaching to container
I1009 13:39:32.198151 28057 docker.go:532] Starting container
E1009 13:39:32.318124 28057 util.go:85] tar: removing leading '/' from member names

... some noisy tar output ...

I1009 13:39:33.314178 28057 docker.go:331] Image contains io.openshift.s2i.scripts-url set to 'image:///usr/libexec/s2i'
I1009 13:39:33.314194 28057 docker.go:384] Base directory for STI scripts is '/usr/libexec/s2i'. Untarring destination is '/tmp'.
I1009 13:39:33.314202 28057 docker.go:515] Creating container using config: {Hostname: Domainname: User: Memory:0 MemorySwap:0 CPUShares:0 CPUSet: AttachStdin:false AttachStdout:true AttachStderr:false PortSpecs:[] ExposedPorts:map[] Tty:false OpenStdin:true StdinOnce:true Env:[] Cmd:[/bin/sh -c tar -C /tmp -xf - && /usr/libexec/s2i/assemble] DNS:[] Image:openshift/ruby-20-centos7:latest Volumes:map[] VolumesFrom: WorkingDir: MacAddress: Entrypoint:[] NetworkDisabled:false SecurityOpts:[] OnBuild:[] Mounts:[] Labels:map[]}
I1009 13:39:33.649982 28057 docker.go:526] Attaching to container
I1009 13:39:33.650717 28057 docker.go:532] Starting container
I1009 13:39:33.803550 28057 docker.go:462] Waiting for container
I1009 13:39:33.847278 28057 sti.go:430] ---> Installing application source ...
I1009 13:39:33.850295 28057 sti.go:430] ---> Building your Ruby application from source ...
I1009 13:39:33.965133 28057 sti.go:430] WARNING: Rubygem Rack is not installed in the present image.
I1009 13:39:33.965158 28057 sti.go:430]          Add rack to your Gemfile in order to start the web server.
I1009 13:39:34.185579 28057 docker.go:464] Container wait returns with 0 and <nil>
I1009 13:39:34.185600 28057 docker.go:471] Container exited
I1009 13:39:34.185603 28057 docker.go:554] Invoking postExecution function
I1009 13:39:34.185711 28057 sti.go:252] No .sti/environment provided (no environment file found in application sources)
I1009 13:39:34.187576 28057 docker.go:588] Committing container with config: {Hostname: Domainname: User: Memory:0 MemorySwap:0 CPUShares:0 CPUSet: AttachStdin:false AttachStdout:false AttachStderr:false PortSpecs:[] ExposedPorts:map[] Tty:false OpenStdin:false StdinOnce:false Env:[] Cmd:[/usr/libexec/s2i/run] DNS:[] Image: Volumes:map[] VolumesFrom: WorkingDir: MacAddress: Entrypoint:[] NetworkDisabled:false SecurityOpts:[] OnBuild:[] Mounts:[] Labels:map[io.openshift.builder-version:004e164 io.openshift.expose-services:8080:http License:GPLv2 io.openshift.builder-base-version:71a650e io.openshift.s2i.scripts-url:image:///usr/libexec/s2i Vendor:CentOS io.k8s.display-name:final-app io.openshift.s2i.build.image:openshift/ruby-20-centos7 io.openshift.s2i.build.source-location:/tmp/sti594925303/upload/sources io.openshift.tags:builder,ruby,ruby20 io.s2i.scripts-url:image:///usr/libexec/s2i io.k8s.description:Platform for building and running Ruby 2.0 applications]}
I1009 13:39:35.562074 28057 sti.go:288] Successfully built final-app
I1009 13:39:36.485138 28057 cleanup.go:23] Removing temporary directory /tmp/sti594925303
I1009 13:39:36.485160 28057 fs.go:99] Removing directory '/tmp/sti594925303'

@soltysh @bparees @smarterclayton PTAL... it was pretty easy to implement this.

[test]

This will also require tweaks in OpenShift to allow "docker://" URL in the BuildConfig

True since its part of the source strategy.
The only thing that I dont like is the docker:// prefix, but the rest look good :+1:

@jhadvig any suggestions for something other than docker:// ? image:// ?

@mfojtik I was thinking about it, and now I see that we are doing similar thing, when we are passing scripts image://path/to/script , which could be used here as well.
Or am I missing something ?

@jhadvig that is true partially, but image://path/to/scripts assumes the path being in the builder image. Whereas here we need to pass the image and the path at once. I'm still thinking about separating the two, would that make sense @mfojtik ?

@soltysh yeah i think that's a good point.

Use "dockerimage://" instead of "docker://"
On Fri, Oct 9, 2015 at 2:03 PM, Ben Parees<EMAIL_ADDRESS>wrote:

@soltysh https://github.com/soltysh yeah i think that's a good point.
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/source-to-image/pull/305#issuecomment-146949188
.

so to be clear this means we'll have:
image://some/path  (exists today, points to a path inside the builder image)
dockerimage://[someregistry]/someimage[:sometag]/somepath - specifies an image and path inside that image.
which may end up looking like:
dockerimage://registry.access.redhat.com:5000/rhcscl/ruby-22-rhel7:latest/usr/bin/foo
we need to make sure we're not setting ourselves up for a difficult/impossible parsing, given how much of that string is optional.

Would probably be better to use a query parameter for the path.
On Sun, Oct 11, 2015 at 6:09 PM, Ben Parees<EMAIL_ADDRESS>wrote:

so to be clear this means we'll have:
image://some/path (exists today, points to a path inside the builder image)
dockerimage://[someregistry]/someimage[:sometag]/somepath - specifies an
image and path inside that image.
which may end up looking like:
dockerimage://
registry.access.redhat.com:5000/rhcscl/ruby-22-rhel7:latest/usr/bin/foo
we need to make sure we're not setting ourselves up for a
difficult/impossible parsing, given how much of that string is optional.
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/source-to-image/pull/305#issuecomment-147250600
.

dockerimage and image are too similar imho and easily confused. How about container or other technology/provider agnostic synonym?
:+1: for the query parameter for the path.

@smarterclayton based on the discussion in the issue, I'm not sure we want to do this in S2I codebase anymore. I think more clever approach might be to do this:
$ sti build git://sources maven-builder - | sti build - runtime-builder final-image

Where the first command will stream the binary artifacts as a tar and the second command will consume it as a source input (untar and build).

@mfojtik big fat :+1: for that last idea. I like it more than anything your proposed before.

Yeah, really like that.  We should brainstorm this, but clear separation
between steps and stages will only help us.
 I like it more than anything your proposed before.
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/source-to-image/pull/305#issuecomment-147461154
.

@bparees closing this, it might come back when we will add streaming of tar from one build to another


feat(types/member): add permissions in MemberCreatePayload

Closes #569

I am super confused with the types right now, need a break lol


Can't save content with spaces

When save content with spaces via ajax, remove all after space.
Bugfix is in patchfile. Thanks for postitall...
0001-Bugfix-Can-t-save-spaces.patch.zip

Hi @Ciatronical
You are right. I've just applied the path.
https://github.com/txusko/PostItAll/commit/ac9d3907bf80ae7879eb22c70bdbbe84c4271f33
Thank you very much!


Fix: exponential notation formatting in broadcast JSON

Motivation
Fixes #5150
Solution
Added a version of format_token which doesn't use exponential notation hints, which could break some json output

cc @DaniPopes