automatically cue signal quality check when running experiments

Added functionality to automatically initiate signal quality check when running experiments.
The check_report command line program is executed immediately before the run_experiment command. When the interactive experiment prompt selection is called by eegnb runexp -ip, the signal quality check prompt appears immediately after the initial experiment and device selections, and immediately before the start of the experiment.
In the process of doing this, this PR also includes a bit of rationalization of functions from the introprompt.py and __main__.py functions in the cli folder. Specifically, the EEG device object creation is (as default behaviour) moved out of the intro prompt command and intro the main function. This was necessary here because the EEG object also needs to be created for the signal quality check, so the reorganization avoids those commands being called twice in two separate places.
Checked for muse. Seems to be working ok. Also needs testing for Neurosity and OpenBCI.
Perhaps could be some small improvements in the command line messages.

Hey @JohnGriffiths - they look to be more typing errors... taking another stab at resolving them now

@JohnGriffiths  Added some changes to this and we should be good to go if build succeeds (fingers crossed lol)

removed the dictionary usage and went back to using the eegdevice class... we don't need to create it multiple times since we can pass the object across
updated the underlying sigqual method to use upper threshold (of 15) and lower threshold of (1) - <1 assumes no contact
added prompt to the user to optionally cancel signal quality checks after 5 tries  (we show them a summary first so they have a signal of how valuable recordings will be)
minor type checking fixes

Tested on Neurosity and works fine too

I have no idea why the Ubuntu 18.04 CI segfaults on eegnb --help. Seems to only be an older Ubuntu thing, so maybe fine/spurious?
I say merge anyway.

Hello, I tested out the OpenBCI Cyton with this updated PR to test the signal quality and I am getting some errors after selecting 'y' on the signal quality prompt. Here are the screenshots:

Thanks for reporting @devyansh .. I think what's happening here is that brainflow stream did not have up to n_samples  (in this case 256) in the stream when we do the assertion check
I just added an update to wait for like 10seconds for openbci devices. Can you pull the latest changes on this branch & retry?

Added fix to run checker with 10 repeats by default not 5.
With this, I am able to run the signal quality checker with openbci ok.
There's an issue with how we treat the std values however.
I don't fully understand this yet but here are some things I have observed:

Ultracortex sitting on desk.

Thanks for testing with OpenBCI @JohnGriffiths ! Glad to know things their devices are covered too.
We will need device-specific threshold criteria
For OpenBCI we will need to assume that device is on head and reference electrodes attached; but stds won't give us clear indication of this
std threshold maybe should be an upper bound of 6-9?

I agree that we will need to set std values per device.

The options I set were based on my conversation with Alex C. from Neurosity (https://github.com/NeuroTechX/eeg-notebooks/blob/fc9d260707dca7685f4460a6a056c852b412b80b/eegnb/analysis/utils.py#L417)
I think we can start with range of 6-9 for openbci  (@jartuso is this something you can validate as the average values are for openbci devices?)
@ErikBjare do you have the average std values handy for muse devices?
is there any other "supported" device class for ntcs missing?

@ErikBjare tell me what you think. I've run eegnb checksigqual and eegnb runexp many times on this PR now with muse and openbci. For both the muse2016 and museS I have it's been irritatingly long to get all stds under 15. So I think I want to recommend 20. That is certainly on the high side though.
Should be aware that if we set high stds then people won't have the opportunity to try to get them lower if they want to as it will move on automatically after 2 successful runs.
But even so I'm inclined to lean towards not pissing people off too much.
Thoughts?

@oreHGA - thanks for getting the device specific options code down.
some small edits here. working ok for me now.
LMK if you think looks good and let's get this one merged.

Looks good to me! Tested & it works. Will let you do the honors :)


xcov reports 0.00% coverage for ignored iOS extension targets

I have added an entry to ignore iOS extensions:
- MyApp/Extensions
and indeed xcov no longer reports any class under MyShareExtension.appex group.
However, even though there is no single class to be tested in appex, it reports 0.00% coverage for the entire component and thus reports overall test coverage to be 50% (assuming my main app has 100% coverage).
Can this be fixed or is there a way to ignore the iOS extensions completely with xcov?

+1 same issue

I'm not sure if it works for "appex" but you could try including it in "excluded_targets"


Fixing StereoBM uninitialized memory access problem

@mshabunin I couldn't reproduce the exact problem. I managed to find two uninitialized memory accesses in stereobm.cpp on lines 420 (with SSE on) and 703 (with SSE off). Proposed changes seem to fix the problem.

thanks! :+1:


this is the wrong place to manage your report

this is the wrong place to manage your report

Hi professor,
We have already uploaded the report at the correct location (sp17 directory).  We had uploaded it here when we were still working on it.  I will remove it from here, shortly.
Thanks,
Abhishek Naik


Make buttons flat and crispy

Some feedback was that we should use flat buttons instead again. https://flagger.app was quoted as an example.

is a quick suggestion, but with the arrow icon added as Flagger has.
What I did to get there is change the background to #fff, drop all shadows around the text and button, and make the text color #555.

cc @juozasg

This should probably be paired with a suggestion I made elsewhere to drop the shadow on the h1 of the hero, which I can't find the issue / comment for at the moment.

#136

@hiddeco says

There is still a 1px greenish border around the button that I have to remove / overwrite in my mock-up as well.

Removed green 1px border https://github.com/juozasg/website/commit/950accff21c7daf304e6bdcbca5f8e72b58a1cc1

I created a PR for you in #381.


Added Number extension

Added a Intl NumberFormatter filter that falls back to money_format if it is not available.

@merk ping ?
also, the implementation using intl could probably be added in the Intl extension I submitted in #30 instead

@stof: In hopsital, wont get a chance to do anything. SonataIntlBundle provides everything I need so I probably wont be dedicating time to this especially since I dont need 5.2 support.

If you dont need 5.2, https://github.com/sonata-project/SonataIntlBundle has NumberFormatter support.


documentation for adding accessibility to applications

so, avilonia 11 added accessibility apis for things such as screen readers for the blind, how ever i can not find documentation on adding this. I'm atempting to add support for this to ryujinx, https://github.com/Ryujinx/Ryujinx or at least point the team at how to do it, though am unsure of how to add it as the documentation doesnt seem to exist, though if it does, please point me at it and close this issue

This issue should go to docs. Will move it there.
I'm not sure how to access it either, so can't answer yet.


MRTK 2.8.0 but old document

[Enter feedback here]
Hello! I find that the doc of MRTK 2.8.0 displayed here is different from my unity’s MRTK2.8.0. For example, the steps of setting eye tracking in your doc is:

But the setting in my project is:

Do you find that there is much difference between them? I think that you are deploying the order version of MRTK rather than the latest .
When I wanna update MTRK I find this trouble, and when I access eye tracking data via EyeGazeProvider in VS it returns false. I guess there must be new change in it. Could you help me whit that?

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

ID: 892601fe-60b1-6af7-a91b-6260870a6147
Version Independent ID: 892601fe-60b1-6af7-a91b-6260870a6147
Content: Eye Tracking Basic Setup - Mixed Reality Toolkit
Content Source: mrtk-unity/mrtk2/features/input/eye-tracking/eye-tracking-basic-setup.md
Product: mixed-reality
GitHub Login: @CDiaz-MS
Microsoft Alias: cadia

Hi @keveleigh, could you take a look at the page to see if there's anything needs updating?

@Hi,keveleigh,have you seen this page?

Hey @JiangTao100, I just did a runthrough of these steps and updated them to the latest state. The docs changes are in PR and should go live shortly!


[staging & production] Console error when downloading file in all addons & OSF Storage

Reported by @samanehsan on the 0.24 Trello board
Steps

Go to a project with the figshare addon configured
In the files page, try to download the file (make sure it is published on figshare)
Go to the file detail page and download the file.

Expected
The file downloads and there are no error messages in the console.
Actual
In both cases the file does download, but I see the following error message in the console.

From Chrome:

Updates
This appears to affect all add-ons in Safari and Chrome. @chrisseto has linked a relevant stackoverflow question: https://stackoverflow.com/questions/12359600/failed-to-load-resource-frame-load-interrupted-agian

@caileyfitz can't repo this problem anymore. Can you verify if this problem still exist?

@nanchen47 I cannot reproduce this problem either.

Likely overcome with recent deployments around files -- I will close -- TY!


Update README.md

Hey @HrithikSampson Whats the deployed URL?

https://tvlopenframe-3dface52cafd.herokuapp.com/

Congratulations on your PR being merged! This means that you are now eligible for the Bounty. If you did not submit through ETH Global, kindly share your Converse address with us so that we can send you your USDC.
Please note that we are still in the process of finalizing the reviews, so please bear with us. Open Frames was a huge success with over 100 frames integrated, and we wanted to thank you for your valuable contributions.

Hi @fabriguespe , I have submitted through Eth global

Yo! @HrithikSampson , still URLs not working :s
https://tvlopenframe-3dface52cafd.herokuapp.com/
https://tvl-history.vercel.app/
Can you test them?

Hi @fabriguespe
The Vercel Deployment URL wasnt working sincee there is an Issue with node canvas library isnt being supported in vercel hence I had removed it from vercel and deplyed it in Heroku
I tested this link it worked for me
https://tvlopenframe-3dface52cafd.herokuapp.com/


Aggregates not working : Does not call the second Route Key

Expected Behavior / New Feature
Aggregates should work as expected and defined in the doc
Actual Behavior / Motivation for New Feature
info: Ocelot.RateLimit.Middleware.ClientRateLimitMiddleware[0]
requestId: 0HMAGJ2TTS6T3:00000002, previousRequestId: no previous request id, message: EndpointRateLimiting is not enabled for /api/v1/hello
info: Ocelot.Authentication.Middleware.AuthenticationMiddleware[0]
requestId: 0HMAGJ2TTS6T3:00000002, previousRequestId: no previous request id, message: No authentication needed for /hithereyou
info: Ocelot.Authorization.Middleware.AuthorizationMiddleware[0]
requestId: 0HMAGJ2TTS6T3:00000002, previousRequestId: no previous request id, message: /api/v1/hello route does not require user to be authorized
info: Ocelot.Requester.Middleware.HttpRequesterMiddleware[0]
requestId: 0HMAGJ2TTS6T3:00000002, previousRequestId: no previous request id, message: 200 (OK) status code, request uri: https://localhost:5001/api/v1/hello
warn: Ocelot.Responder.Middleware.ResponderMiddleware[0]
requestId: 0HMAGJ2TTS6T3:00000002, previousRequestId: no previous request id, message: Error Code: UnmappableRequestError Message: Error when parsing incoming request, exception: System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'start')
at System.Buffers.ReadOnlySequence1.TryGet(SequencePosition& position, ReadOnlyMemory1& memory, Boolean advance)
at System.IO.Pipelines.PipeReader.CopyToAsyncCore[TStream](TStream destination, Func`4 writeAsync, CancellationToken cancellationToken)
at Ocelot.Request.Mapper.RequestMapper.ToByteArray(Stream stream)
at Ocelot.Request.Mapper.RequestMapper.MapContent(HttpRequest request)
at Ocelot.Request.Mapper.RequestMapper.Map(HttpRequest request, DownstreamRoute downstreamRoute) errors found in ResponderMiddleware. Setting error response for request path:/hithereyou, request method: GET
It seems like the 2nd call is never made. I am not sure what is wrong here. Any help would be highy appreciated.
Steps to Reproduce the Problem
Here is my Ocelot.Json
{
"Routes": [
{
"DownstreamPathTemplate": "/api/v1/actions/{actionId}",
"DownstreamScheme": "https",
"DownstreamHostAndPorts": [
{
"Host": "localhost",
"Port": 5001
}
],
"UpstreamPathTemplate": "/actions/{actionId}",
"UpstreamHttpMethod": [ "Get", "Post" ]
},
{
"DownstreamPathTemplate": "/api/v1/hello",
"UpstreamPathTemplate": "/hi",
"UpstreamHttpMethod": [
"Get"
],
"DownstreamScheme": "https",
"DownstreamHostAndPorts": [
{
"Host": "localhost",
"Port": 5001
}
],
"Key": "hi"
},
{
"DownstreamPathTemplate": "/api/v1/lonely",
//"DownstreamPathTemplate": "/api/weather",
"UpstreamPathTemplate": "/there",
"UpstreamHttpMethod": [
"Get"
],
"DownstreamScheme": "https",
"DownstreamHostAndPorts": [
{
"Host": "localhost",
"Port": 5001
}
],
"Key": "there"
}
],
"Aggregates": [
{
"RouteKeys": [
"hi",
"there"
],
"UpstreamPathTemplate": "/hithereyou"
//"Aggregator": "FakeDefinedAggregator"
}
],
"GlobalConfiguration": {
"BaseUrl": "https://localhost:5006"
}
}
Specifications

Version:17.0
Platform: Windows / Dot Net 5
Subsystem:

Learn Ocelot more! 🤣

Request Aggregation

Aggregation feature works great.

Message: Error when parsing incoming request, exception:

Root cause of having error: Bad format of JSON data.
Ocelot cannot parse JSON, possibly.
Try to debug problem.
Try to call downstream service manually and check returned JSON formatting!


Feature Request: Select template

When using the previous version V0.7.5., we used a hidden feature (kind_override) to select a template for generating the documentation. With the new release V0.8.0 additional parameter at the insert command are rejected.
On my project I have some structure, that I just like to display at a simple table for other structures need the complete description. I select the kind of documentation generation with kind_override at this way for selecting the template.
${insert("MY_STRUCT_T", kind_override="typedef_table")}

Sorry for that, I was not aware this internal detail was used. It should not be hard to expose it again as an official feature. I am considering adding a template argument to insert and adding a command line switch to provide a template directory to search in.

The next release of AsciiDoxy will add the template argument to the insert command. The command line argument --template-dir is added to provide your own customized templates to use.


Procedure cannot be found (FHIR search) intermittently (5 - 10m intervals) while writing to it

Describe the bug
We are using Azure API for FHIR, I assume this is based on this repo and uses CosmosDb as the underlying datastore, please correct me if I am mistaken.
We have an ongoing procedure where we keep updating extensions, these extensions are used to optimize read performance of the application and contains things like the count of certain types of observations. This means that we need to update the procedure continuously while it is in progress (1 - 10 days), for this we have a queue with multiple workers (Azure function). We ensure no data is lost by applying a distributed lock through Azure Cache for Redis when updating the procedure (lock is scoped to a particular procedure).
In the mean time people (practitioners) can use the portal to view the ongoing procedure, the problem is that our query doesn’t return the ongoing procedure at random intervals.
The result is that the ongoing procedure is intermittently not visible in the web portal (5 minutes visible,  5 minutes not, repeat). When we stop writing to the procedure via our functions the same search query always returns the procedure.
We use Search for querying the procedure and some related entities like the performer and the subject (we also have a filter on the subject).
I assume this has something to do with the consistency levels of cosmos db, the current behavior is to not return the data at all which is a problem for us, is there any way to work around this issue?
FHIR Version?
R4
Data provider?
CosmosDB
To Reproduce
Steps to reproduce the behavior:

Write to the procedure record (continuous updates)
Use search to find the procedure by subject (different process)

Expected behavior
The procedure is always returned in the search result (data can be behind)
Actual behavior
The procedure is not returned for 5 - 10 minutes at the time, then it is returned for 5 - 10 minutes, then not, ....
AB#93710

Hi @Dinand01, if this is happening in Azure API for FHIR I’d recommend opening a support ticket so we can triage and inspect the telemetry to see what might be going on.
When you mention not being returned, is it that there is an error/timeout or that the expected resource isn’t in the results?

Hi @brendankowitz , thank you for the reply.
The expected resource is not in the results, no error or timeout.
We have no indication that writing fails, this is only happening for the search (getting the procedure via its id is no issue).

Thanks for reporting the issue @Dinand01. Closing the issue, as it is resolved. Please feel free to reach out for any additional questions.


hc:Empty issue in combobox control

fetch newest source code, rebuild project and try again

fetch newest source code, rebuild project and try again

@NaBian The demo is based on the latest source code
There is no problem separately, but when used together, there is a problem

@NaBian The demo is based on the latest source code
There is no problem separately, but when used together, there is a problem


Jolokia JS client does not support Long/BigInt

Hi, there is an issue with JS client library when the Java agent returns a numeric value greater than Number#MAX_VALUE
This affects JMX attributes/operations when the server returns a LONG value for instance.
The issues stems from jolokia.js#L226  where the default JSON parser is used.
To preserve the value, would it be possible to use a custom parser to parse numeric values into BigInt (there are libs for this), or convert numeric value to quoted String?
See related issue at: https://github.com/hawtio/hawtio-integration/issues/108

I see the problem, but unfortunately, I don't have any spare cycles to work on this. As long as we don't include any extra dependencies, I'm happy to help to integrate a pull request if someone is inclined to fix this. Another possible solution for your issue at hand and if you would have a chance to change the MBean that exports the long value, then you could also try to change this to a string type on your own.

Let me summarise the issue and proposed solution.
@grgrzybek @rhuss Please review this and let me know your thoughts.
Problem
The maximum raw number safely used in JS and JSON is Number.MAX_SAFE_INTEGER, which is<PHONE_NUMBER>740991. JS raw number lacks enough bits to represent numbers greater than the max value. A number larger than that is not just simply truncated but also may be rounded depending on the binary representation of it:
90071992547409910 -><PHONE_NUMBER>7409900
90071992547409930 -><PHONE_NUMBER>7409940
To handle numbers larger than the max there are two options:

Use string representation
Use BigInt<PHONE_NUMBER>7409910n or BigInt("90071992547409910"))

The problem with the 2nd option is that BigInt is only supported since ES2020. The jolokia.js library is still based on ES5, so the 2nd option cannot be used for the jolokia.js library.
Proposed solution
Therefore, I propose to add a new Jolokia config option serializeLong (both global and per request), which prescribes the strategy for how to serialize values with type long from MBean attributes and operations. (We don't need to consider Java BigInteger values because it already has a serialization strategy.)
The option would accept the following values:

none -- Default, and the original behaviours (do nothing) for backward compatibility.
string -- Always change long values to string in the returned JSON responses.
Example:

{ value:<PHONE_NUMBER>7409910 } -> { value: "90071992547409910" }
{ value: 12345 } -> { value: "12345" }

bigint -- Always change long values to BigInt in the returned JSON responses.
Example:

{ value:<PHONE_NUMBER>7409910 } -> { value:<PHONE_NUMBER>7409910n }
{ value: 12345 } -> { value: 12345n }

The 3rd option is provided for users who want to directly consume the JSON responses with their own >=ES2020 JS clients instead of using jolokia.js.
Considerations
For the 2nd option, we might want to instead follow the same serialisation with BigInteger like this:
{
  value:<PHONE_NUMBER>7409910
}

->
{
  value: {
    "long": "90071992547409910"
  }
}

WDYT?
Reference:

https://jolokia.org/reference/html/manual/jolokia_protocol.html#serialization

Oh, I didn't know about n suffix.
I assume the problem is not about deserialization of BigInts from JS in Java (we'd overflow Java's long), right?
So for Java (long) → JavaScript, I'd go with configuration option, but I would make bigint the default.
Also I'd think about Jolokia 2.1 with higher JS standard level... ES2015 is quite old ;) No safe browser nowadays stays at ES2015 support...
From Node.js point of view we also should not be restricted to ES2015 only...

Thanks.

assume the problem is not about deserialization of BigInts from JS in Java (we'd overflow Java's long), right?

Yes, so long as the original Jolokia issue.

So for Java (long) → JavaScript, I'd go with configuration option, but I would make bigint the default.

Even after bumping ES version to ES2020 for jolokia.js, the other issue with using bigint is that JSON.parse doesn't support parsing bigint by default, and would need to add a custom reviver to correctly parse it. So we'd need to patch jolokia.js as well.
https://dev.to/benlesh/bigint-and-json-stringify-json-parse-2m8p

Also I'd think about Jolokia 2.1 with higher JS standard level... ES2015 is quite old ;) No safe browser nowadays stays at ES2015 support...
From Node.js point of view we also should not be restricted to ES2015 only...

That's actually an interesting idea. I'm not opposed to it. Let's see if we can upgrade ES version to ES2020 for Jolokia 2.1.
For reference:

https://github.com/jolokia/jolokia/issues/576

JSON.parse doesn't support parsing bigint by default

oh, indeed...

And user apps that use older jolokia.js should be broken with the new Jolokia agent version

definitely - we should bump protocol version: https://github.com/jolokia/jolokia/blob/main/HOWTO_RELEASE.md#jolokia-protocol-version-change

Well, I'm okay with anything. Since we already have numerical values being serialized as string (Java's BigInteger), I would not have an issue serializing long as strings, too, as long as this is consistent (i.e. all long are returned the same way)
A configuration option seems legit, with the default being none (i.e., if nothing is specified, the old behavior is kept) for backward compatibility until we change the protocol version.

I initially proposed the serializeLong option with 3 values: none, string and bigint. But after closer investigation and review from @grgrzybek the final impl only allows the 2 values: number (default) and string. The option bigint turned out to be awkward to support for the following reasons:

The JSON library we currently use (json-simple) cannot handle the BigInt notation: #686
If we include the BigInt notation (12345n) to the JSON response, it would become something that diverges from the standard JSON specification: https://www.json.org/json-en.html

The final impl #693 also includes a new request parameter dataType (with value: json (default) or text) only for jolokia.js, which can influence the ajaxParams.dataType and allow the client to return the response as text instead of parsing it with JSON.parse(). It's intended to provide the JS client users with yet another way to deal with the long values issue.

Great - thanks @tadayosi !


MaterialIcons font family not being bundled on iOS release

Steps to Reproduce
            Text(
                     'someIconName',
                      style: TextStyle(
                      fontFamily: 'MaterialIconss',
                      fontSize: 25.0,
                    )

This works perfectly to render dynamic icons from BE on Android (release and debug mode) and also on iOS debug. However, in iOS release it doesn't work properly and icon names are displayed instead.
[✓] Flutter (Channel unknown, 1.22.6, on macOS 11.2.2 20D80 darwin-x64, locale pt-PT)
    • Flutter version 1.22.6 at /Users/miguelruivo/devtools/flutter
    • Framework revision 9b2d32b605 (6 weeks ago), 2021-01-22 14:36:39 -0800
    • Engine revision 2f0af37152
    • Dart version 2.10.5

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

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

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

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 44.0.1
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

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

[✓] Connected device (3 available)
    • iPhone 12 (mobile) • 6575E0BF-D34F-4E06-8190-12DD7E6214E7 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-4 (simulator)
    • Web Server (web)   • web-server                           • web-javascript • Flutter Tools
    • Chrome (web)       • chrome

looks like you messed up the pubspec.yaml
make sure this like is included in flutter section
flutter:
  # The following line ensures that the Material Icons font is
  # included with your application, so that you can use the icons in
  # the material Icons class.
  uses-material-design: true

also, upgrade to flutter 2

and also, i'm not sure if this is a valid code to draw a material icon

        Text(
                 'someIconName',
                  style: TextStyle(
                  fontFamily: 'MaterialIconss',
                  fontSize: 25.0,
                )

@nt4f04uNd that’s the only way to render it dynamically other than creating a map with all de icons (which is tedious and ugly). I have that flag set.
Like I said in my original post, it works on debug mode. Try it on iOS release.

that’s the only way to render it dynamically other than creating a map with all de icons (which is tedious and ugly). I have that flag set.

i don't really see how this makes it easier, but anyway, i can reproduce in ios release

screenshot

flutter doctor -v
[✓] Flutter (Channel texture_buildable, 1.27.0-2.0.pre.100, on Mac OS X 10.15.2
    19C57 darwin-x64, locale ru-RU)
    • Flutter version 1.27.0-2.0.pre.100 at /Users/danya/dev/src/flutter
    • Framework revision 76080cd46c (2 weeks ago), 2021-02-21 18:22:32 +0300
    • Engine revision 1d537824d6
    • Dart version 2.13.0 (build 2.13.0-30.0.dev)

[✗] Android toolchain - develop for Android devices
    ✗ Unable to locate Android SDK.
      Install Android Studio from:
      https://developer.android.com/studio/index.html
      On first launch it will assist you in installing the Android SDK
      components.
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup
      for detailed instructions).
      If the Android SDK has been installed to a custom location, please use
      `flutter config --android-sdk` to update to that location.

[!] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.7, Build version 11E801a
    ! Xcode 11.7.0 out of date (12.0.1 is recommended).
      Download the latest version or update via the Mac App Store.
    • CocoaPods version 1.10.0

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

[!] Android Studio (not installed)
    • Android Studio not found; download from
      https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup
      for detailed instructions).

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

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 88.0.4324.192

! Doctor found issues in 3 categories.

Hi @miguelpruivo  @nt4f04uNd
I'm unable to reproduce the issue when running  flutter run --release -v on the latest channels from stable to master  using a code sample from  https://github.com/flutter/flutter/issues/77423#issuecomment-792041648.

flutter doctor -v
[✓] Flutter (Channel stable, 2.0.1, on macOS 11.2.2 20D80 darwin-x64, locale
    en-GB)
    • Flutter version 2.0.1 at /Users/tahatesser/Code/flutter_stable
    • Framework revision c5a4b4029c (4 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      14.4
    • macOS (desktop)        • macos                     • darwin-x64     •
      macOS 11.2.2 20D80 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 88.0.4324.192

• No issues found!

[✓] Flutter (Channel beta, 2.0.1, on macOS 11.2.2 20D80 darwin-x64, locale
    en-GB)
    • Flutter version 2.0.1 at /Users/tahatesser/Code/flutter_beta
    • Framework revision c5a4b4029c (4 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      14.4
    • macOS (desktop)        • macos                     • darwin-x64     •
      macOS 11.2.2 20D80 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 88.0.4324.192

• No issues found!

[✓] Flutter (Channel dev, 2.1.0-10.0.pre, on macOS 11.2.2 20D80 darwin-x64,
    locale en-GB)
    • Flutter version 2.1.0-10.0.pre at /Users/tahatesser/Code/flutter_dev
    • Framework revision cc9b78fc5c (10 days ago), 2021-02-25 13:26:03 -0800
    • Engine revision a252ec09b7
    • Dart version 2.13.0 (build 2.13.0-77.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      14.4
    • macOS (desktop)        • macos                     • darwin-x64     •
      macOS 11.2.2 20D80 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 88.0.4324.192

• No issues found!

[✓] Flutter (Channel master, 2.1.0-11.0.pre.145, on macOS 11.2.2 20D80
    darwin-x64, locale en-GB)
    • Flutter version 2.1.0-11.0.pre.145 at
      /Users/tahatesser/Code/flutter_master
    • Framework revision a0ba646408 (2 days ago), 2021-03-05 22:09:03 -0500
    • Engine revision f751d04fa0
    • Dart version 2.13.0 (build 2.13.0-107.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Volumes/Extreme/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Volumes/Extreme/SDK
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Volumes/Extreme/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

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

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6915495)

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

[✓] Connected device (3 available)
    • Taha’s iPhone (mobile) • 00008020-001059882212002E • ios            • iOS
      14.4
    • macOS (desktop)        • macos                     • darwin-x64     •
      macOS 11.2.2 20D80 darwin-x64
    • Chrome (web)           • chrome                    • web-javascript •
      Google Chrome 88.0.4324.192

• No issues found!

Can you please upgrade and try again
Thank you

Hi, I've actually tried deploying locally using release build and I can confirm that it is working properly.
I've also made a manual archive and distributed with Firebase and it's working as well. So I digged down and the issue seems to be related with the Fastlane lane that I use to distribute which is the following:
  desc "Build Ad-Hoc"
    private_lane :build_adhoc do

        build

        build_app(
            workspace: xcode_workspace,
            scheme: xcode_default_scheme,
            include_symbols: true,
            export_options: {
                method: "ad-hoc",
                provisioningProfiles: {
                        // hidden
                }
            }
        )
    end

    # ------------------------------------------------------------------------------------------------
    desc "Distribute to Dev Team"
    lane :dev do

        build_adhoc

        firebase_app_distribution(
            app: firebase_app_dev,
            groups: firebase_test_groups,
            debug: true,
            release_notes: changelog
        )

        upload_symbols_to_crashlytics(
            dsym_path: symbols_path,
            app_id: firebase_app_dev,
            gsp_path: "./Environments/dev/",
        )

    end

It seems that when I run fastlane dev the MaterialIcons doesn't work dynamically on the app but I can't really tell why. If someone is familiar with process and could be some insight it would be great.
Thank you.

Hi @miguelpruivo
Thanks for confirming, unfortunately, I'm not familiar with Fastlane.
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


fix(build): make logging warnings clearer

Starting with small non-breaking contributions to clean the code up.

@chtompki can you please conform the contribution requirements for Fineract? Let's squash your commits, have a FINERACT-XYZ ticket in the title of the PR that your changes are referring to and the same thing for the commit message.
Thanks.


Buffer API implementation

What are the current limitations of this wonderful WIP Python Agent? I am eager to get things working but am running into a few issues with some canister calls.
This is the error I keep getting
TypeError: getType() missing 2 required positional arguments: 'table' and 't'
When I run
from ic.client import Client
from ic.identity import Identity
from ic.agent import Agent
from ic.candid import encode, decode

iden = Identity()
client = Client(url = "https://ic0.app")
agent = Agent(iden, client)

result = agent.query_raw("hbi4x-wqaaa-aaaaj-aad7a-cai", "getRegistryCount", encode([]))

It works. The getRegistryCount method works.
However, if I change the last line to:
result = agent.query_raw("hbi4x-wqaaa-aaaaj-aad7a-cai", "getRegistry", encode([]))
just get the registry rather than the count, it appears it has an issue returning a vec or something like that.
I'm assuming this is all known, I'm just exploring what I can and can't do right now. (again, eager to use it as it becomes available)

Full error in case you wanted it
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
/var/folders/yg/mzpfyl291vx30knxqlx43d2r0000gn/T/ipykernel_62982/1139556732.py in <module>
----> 1 result = agent.query_raw("e3izy-jiaaa-aaaah-qacbq-cai", "transactions", encode([]))

/opt/anaconda3/lib/python3.9/site-packages/ic/agent.py in query_raw(self, canister_id, method_name, arg)
     72         result = self.query_endpoint(canister_id, data)
     73         if result['status'] == 'replied':
---> 74             arg = decode(result['reply']['arg'])
     75             return arg
     76         elif result['status'] == 'rejected':

/opt/anaconda3/lib/python3.9/site-packages/ic/candid.py in decode(data, retTypes)
   1139
   1140     for i, entry in enumerate(rawTable):
-> 1141         t = buildType(rawTable, table, entry)
   1142         table[i].fill(t)
   1143

/opt/anaconda3/lib/python3.9/site-packages/ic/candid.py in buildType(rawTable, table, entry)
   1045         if ty >= len(rawTable):
   1046             raise "type index out of range"
-> 1047         t = getType(entry[1])
   1048         if t == None:
   1049             t = table[t]

TypeError: getType() missing 2 required positional arguments: 'table' and 't'

Thanks for your support! There might still be some issues with vec decoding, we'll take a look and fix this ASAP.

Fixed on main branch now.


Issue

During run streamlit, this issue happens, how can I solve?
ImportError: cannot import name 'chart' from 'utils'
(D:\ANACONDA\envs\conda_streamlit\lib\site-packages\utils_init_.py)

Did you clone the whole repository, or just the streamlit_app.py file? You'll need to also copy the files in the utils folder for it to work properly.

@blackary This could also be a situation where people inside streamlit tend to do streamlit run directory/file.py and users do streamlit run file.py and the relative imports are different

True. In this case, if you clone the whole repository, and do streamlit run streamlit_app.py, it should be able to import chart from the utils module without any issue.


⚠ Bofrost.be has degraded performance

In e62e39c, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3835 ms

Resolved: Bofrost.be performance has improved in c0e426d.


prod publish [semver:major]

prod publish [semver:major]

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/danielholdsworth/dynamic-message-filter?version=dev:2eef4fd9b6059a891f137b305baf9c5ecb770145

Your development orb has been published. It will expire in 30 days.
You can preview what this will look like on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/danielholdsworth/dynamic-message-filter?version=dev:2eef4fd9b6059a891f137b305baf9c5ecb770145

Your orb has been published to the CircleCI Orb Registry.
You can view your published orb on the CircleCI Orb Registry at the following link:
https://circleci.com/developer/orbs/orb/danielholdsworth/dynamic-message-filter?version=1.0.0


get error when turn to stable1.0.0 from beta 0.10.12

rect is null.there will be a exception

Please post output if flutter doctor -v. Also, can you confirm your flutter versions?
Are you trying to use a older version or something newer?

Please consider asking support questions in one of the other channels listed at http://flutter.io/support .


Leaking data to Crowdin not disclosed

(I am not a lawyer, but) presumably this integration → https://crowdin .com/project/microcovid is illegal, as people submitting things on Weblate for the same resource are not exposed to the Crowdin terms and services before doing so. They are quite horrendous btw, including the cookie policy and various other documents concerning the operation of Crowdin.
Quality on Crowdin is poor, and it has few translators. (No activity since microCOVID was started there two months ago). It has the same ugly voting system now used for microCOVID on Weblate, only there it is the default. See https://gitlab.com/famedly/fluffychat/-/issues/265 (it is worse than that, but alas).
Crowdin is closed source, and also isn't gratis for anything but non-commercial software (microCOVID takes donations).
#451 related

Thanks!
Removed all links. Deleted the crowdin profile.

Thanks!
Removed all links. Deleted the crowdin profile.


Binary sensors not showing correct icon

The two binary sensors that are generated automatically show the default icons even though they are set up with device class safety. The state itself though is shown correctly (Safe/Unsafe).

Are you sure? Mine seems ok..

With device class safety, this is how the icons should look like:

It turned out that the icons for the safety class were changed recently: https://github.com/home-assistant/frontend/pull/7403
So, everything is OK, no need to fix anything.


Mm 37472 product switcher tip

Summary
Show a tutorial tip for switching products when global header is turned on. Some things to call out:

This depends on Boards and Playbooks showing in global header, neither of which are fully merged and released yet.
Because this depends on not fully released work, opting to defer QA until environments with the updated/released versions of Playbooks & Boards are available.
I was able to hardcode the hasPlaybooks & hasBoards values locally to get the tutorial tip to display, and to get it to skip over the tip in the case that one (or both) of them was missing.

Ticket Link
https://mattermost.atlassian.net/browse/MM-37472
Related Pull Requests
None
Screenshots

Release Note
Add tip for product switcher. Skip tip if not applicable.

Hello @neallred,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@neallred looking good so far. Curious to how we shall solve the issue of showing this new tutorial step to existing users.

Talked with @esethna and he is ok with us not being able to show to existing users at step value 999 (that skipped the tutorial) for now, given it will take more time than we can likely code/review/QA by August 16.
The idea I had for achieving it was to add a secondary preference that tracks what was the last tutorial step the user could have deliberately skipped, based on the tutorials available when they clicked "Opt out of these tips". I discussed it some more here: https://community-daily.mattermost.com/core/pl/3h1nbnq6ftykzdbh5a1iwzh1bo

@AGMETEOR is there anything in this PR you would like changed before it is merged?

Talked with @esethna and he is ok with us not being able to show to existing users at step value 999 (that skipped the tutorial) for now, given it will take more time than we can likely code/review/QA by August 16.
The idea I had for achieving it was to add a secondary preference that tracks what was the last tutorial step the user could have deliberately skipped, based on the tutorials available when they clicked "Opt out of these tips". I discussed it some more here: https://community-daily.mattermost.com/core/pl/3h1nbnq6ftykzdbh5a1iwzh1bo

If we ever have to follow up on this, imo, the simplest solution I could think of is a new preferences category e.g "tutorial_step_v6" . This can have the tracking of individual steps and not just incrementing a value.

@AGMETEOR is there anything in this PR you would like changed before it is merged?

Just an integration test here: https://github.com/mattermost/mattermost-webapp/tree/master/e2e/cypress/integration/onboarding

Added an e2e test and was able to get it passing locally. Will go back and add this step to this test once the dependencies on playbooks & boards being registered & available in global header is met.
https://user-images.githubusercontent.com/13738432/129070813-d2a003ca-17ec-4412-9d07-2a833de8c697.mp4


How to show select column BUT hide selectAll

Hi, is there a way to show the select column (with the checkboxes) but hide the selectAll button at the top?
Thanks!

@vgoklani, there's no way to do it. but I can add a class on selectAll checkbox for you to hide, how about that?
Thanks:)

that should be perfect, thank you so much!

@AllenFang  Hi Allen, also a question for the row edit, if enable the deleteRow={true}, is it possible to call my custom function when the delete button is clicked?

@eric379990, what's means for your custom function? you want a row deletion hook?

@AllenFang I have 2 tables, user can select some items in the first table and add to the 2nd table, and also can delete some items in the 2nd table, the items needs to back to the 1st table. so I want to handle the item back actions in the onDelete function.

function afterRowDelete(rowKeys) {
    // rowKeys is the rowkeys which you delete
}

var options = {
    afterDeleteRow: afterDeleteRow
}
//....
ReactDOM.render(
 <BootstrapTable data={products} options={options}>
.....
);

Thanks Allen, this helped a lot!

@vgoklani, I've add the class react-bs-select-all on select all checkbox. please check this out
Thanks :)

Thanks @AllenFang for adding the react-bs-select-all class. Quick question - is there a way to dynamically set the "display: none" via React? I would prefer not to touch the .css/.less files if possible

@vgoklani, sure, just give it a style

not following, how would I pass in the style to set display:none for the react-bs-select-all class from React? Thanks!

@vgoklani, I got the wrong consideration. sorry for that. In this situation, the only way is modify the css file

ok, thanks!

I am having a difficult time hiding all select with react-bs-select-all class. Anyone can help me on how to accomplish that in css? The selector is not picking up the css.

Firstly a big shout out to the owners and maintainers of this library, it is indeed a great one.
Secondly to answer @vgoklani :   I too am having the same use case for conditional show hide of select all and below code works for me:
if (typeof (document.getElementsByClassName("react-bs-select-all")) !== 'undefined' && document.getElementsByClassName("react-bs-select-all").length > 0) {
document.getElementsByClassName("react-bs-select-all")[0].setAttribute('style', display:none);
}
You can place it in your componentDidUpdate() method and based on the condition modify the display value to show ('') or Hide ('none')
@yasaiki : Below code shall work, just place it in your css (in case still not working then can try the above way aswell)
.react-bs-select-all{
display: none;
}
Thanks.


Add --protect-tag-regex flag to cleanup command

I want to protect some pattern of tag (e.g. '^release-'). So, I added new flag --protect-tag-regex to cleanup commmand.

Hi. Sorry I was a bit busy lately. I will look into it soon!

thanks btw <3


Enable Azure AD login directly with self-provided token

Issue Type: Feature Request
This enables some advanced testing scenarios where you have extracted a token (from App Service Managed Identity for example) and can use that identity to run queries against your database.
Azure Data Studio version: azuredatastudio 1.4.5 (887f4e8985023602e599cf86bdb44df48bb28569, 2019-02-12T06:20:08.633Z)
OS version: Windows_NT x64 10.0.17134

@aaomidi is there anything we can do for this scenario?

This seems like a pretty big edge case. It's certainly possible to pass this in but the code is currently not setup for that.
Is this a feature PM wants to see in ADS? @yualan

cc @erinstellato-ms

Closing as this feature request is not on ADS connectivity roadmap.


Automated Integration Tests

Why?
The unit tests run as part of the CI pipeline cannot capture all aspects of MaMiCo.
Therefore, it is necessary to also perform integration tests which invoke the compiled binary using a realistic, non-trivial scenario to verify that the configuration generator, simulation, and visualization output is correct.
What?
For every new commit on the master branch or (?) for every commit on a branch with an open pull request to the master branch, the integration test should be performed, executing the following steps:

Clone the repository and check out the corresponding commit
Compile MaMiCo
Generate the MaMiCo configuration (options TBD)
Execute MaMiCo using MPI and the configuration
Visualize the results
Verify the (visualization) results (using a reference)
Report the results of the test

How?
Execution using GitHub Actions is not feasible using the standard runners due to the number of required MPI ranks and the expected non-trivial runtime of the test, so the automated integration test is executed on an on-site system.
Due to security reasons, the automated test should use a pull-semantic (CRON job polling GitHub).
If an error occurs during the execution of the test or the output deviates from the expectation, a notification should be generated to raise awareness about the corresponding commit. This can be done through email or using the GitHub API.
To avoid repeated tests of the same commit and repeated notification about the same issue, a local database of tested commit hashes should be maintained (A file with simple "key=value" pairs should be sufficient).
Chronological checklist of key features

[ ] Integration test execution using FabMaMiCo:
Input: Commit hash
Output: Test result (passed/failure details), artifact (visualization)
Details: Cloning the repository and checking out the commit hash, generating the configuration, executing MaMiCo, executing the visualization, invoking a callback with the results (next step of the pipeline)
[ ] CRON job:
Input: None
Output: None
Details: Periodically check the state of the remote repository against the local database of executed tests, execute outstanding tests and update the database accordingly (status: started)
[ ] Reporting the results:
Input: Test result, artifact
Output: None
Details: If the result is not "passed", send a notification including the failure details and artifact (e.g.: mail containing details and path to artifact`), and update the database (status: success/failed[-final])

Checklist of stretch goal features

[ ] (Manually) re-run jobs if they failed due to e.g. network connectivity issues (git clone)
[ ] Report through GitHub API:
Rather than sending and email, use the GitHub HTTP API to attach test results as comments to existing pull requests or open a new issue for commits on master.
[ ] Propper database:
Replace the simple "key=value" file for storing the test states with an SQLite or DuckDB instance.
[ ] Housekeeping:
Delete files associated with a single test run after a certain amount of time (no sooner than 1 month)

After some discussion @Thinkpiet and I came to the conclusion that we should perhaps simplify the plan for the automated integration tests somewhat to avoid building a full workflow engine with all the complexity that entails.
The core features can now be boiled down to:

Periodically (e.g.: every 24h) fetch the master branch of the repository.
If the commit hash is different from the last one tested (stored in file), trigger new integration tests. (This is done to avoid re-running tests if there is no change for some time, like during holidays.)
Periodically (using a second CRON job) the current test results are compared to the previous ones (and current become previous).
If the results differ from the previous ones, an email with the test results is sent to all developers.


Add language spellings for Go and Protocol Buffers. [phab:D767]

Created by fromberger at 2016-03-11 17:02:30:

zarko accepted this revision at 2016-03-11 18:00:26

zarko closed via commit 00d3cdf54e3849a4a8ba628ca9da340f55a1b342 at 2016-03-11 18:01:53


Getting PointerToNowhere when migrating from RefResolver to referencing

Dear team,
I've been trying my best to migrate my schema validation from RefResolver to the newer referencing library, but with no avail.
I would really appreciate some help or tips to point me in the right direction.
What is working for me, with RefResolver, is the following code:
def validate_schema(
    instance: dict[str, Any],
    schema: dict[str, Any],
    *,
    path: str,
    method: str,
    response: str,
    content: str = "application/json",
) -> None:
    try:
        _methods = schema["paths"][path]

    except KeyError:  # pragma: no cover (should never happen)
        msg = f"Path {path} not found in schema"
        raise ValueError(msg) from None

    try:
        _responses = _methods[method]

    except KeyError:  # pragma: no cover (should never happen)
        msg = f"Method {method} not found in schema"
        raise ValueError(msg) from None

    try:
        _content = _responses["responses"][response]["content"]

    except KeyError:  # pragma: no cover (should never happen)
        msg = f"Response {response} not found in schema"
        raise ValueError(msg) from None

    try:
        _schema = _content[content]["schema"]

    except KeyError:  # pragma: no cover (should never happen)
        msg = f"Content {content} not found in schema"
        raise ValueError(msg) from None

    openapi_schema_validator.validate(
        instance,
        _schema,
        cls=OAS30Validator,
        resolver=RefResolver.from_schema(schema),  # TODO: migrate to referencing lib
    )

I then tried to migrate to the referencing lib as mentioned here:
resource = Resource.from_contents(schema, default_specification=DRAFT202012)
registry = Registry().with_resource("", resource)
openapi_schema_validator.validate(instance, _schema, registry=registry)

But with no success, as all I get are errors like:
jsonschema.exceptions._WrappedReferencingError: PointerToNowhere: '/components/schemas/Foo' does not exist within {'$ref': '#/components/schemas/Foo'}
What am I missing? Thank you for the help.

Instead of passing a registry, you can flatten the schema dict before handing them to the validate function.
To correctly handle $ref pointers directly when parsing your YAML files, you can use a library like PyYAML combined with jsonschema's RefResolver or openapi-spec-validator to properly resolve $ref references. A more elegant solution for loading and resolving references in OpenAPI or YAML schemas is using the yaml loader from jsonschema or openapi-spec-validator libraries.
However, if your project requires loading complex OpenAPI files with nested $ref pointers, the prance library might be the most straightforward choice:
from prance import ResolvingParser
parser = ResolvingParser("openapi.yaml", lazy=True)
parser.parse()
validate_schema(schema=parser.specification, ...)


[enhancement] auto-confirm out of stock

Hey @ahertel, thank you so much for making this project. I'm really liking the script. One thing that keeps happening is one of my items will go out of stock and I will manually have to delete the item and confirm. Due to how often things go out of stock I just deal with this by buying 4 different versions of the same thing assuming 3 out of 4 things will be gone.

Would this feature be valuable?
How could someone choose to enable this feature or not? Dialogue box on startup maybe?

I can help with building this feature if that would be of use.

Hi @wardnath ! Yes, I ran into those out of stock issues pages countless times too so I actually briefly tried building an "auto-confirm out of stock" feature myself, but I couldn't figure out how to get the program to click the "Continue" button. I had found this code which has to do the actual clicking,  but I couldn't figure out which tag/id/name etc to pass to the click() function. I don't really know my way around HTML yet. I was planning on taking another stab at this feature myself in the coming days, but if you want to build that feature, that'd be great. I'd be happy to merge it into the current tool.
Once the program can click buttons on Amazon's website, I was planning on adding and auto-checkout feature: Once slots appear, the script could automatically select the first available one, and then click continue to finish the order. However, sometimes a slot appears, but by the time you select it and click "continue" it's already full and the continue button stops working. So in that case the program would keep refreshing, selecting new time slots, trying to checkout until the order finally goes through and the order confirmation page appears. This would make the tool complete in my eyes as it would then be fully autonomous and could even place orders while one is sleeping haha. I think once we having the auto-confirm out of stock clicking figured out, it would pretty easy to add this auto-checkout feature.
Lastly, I agree that a dialog box would a good way of prompting the user if they want to enable these autonomous two features(auto-confirm out of stock, and auto-checkout).

The above mentioned ideas would be fantastic to have in the script.
Thank you so much for making this @ahertel, in some ways truly a life saver.
Successfully used it yesterday morning and will continue using it.

Thanks for the quick reply @ahertel! I've worked with automated click actions via selenium and puppeteer before but I'm new to apple script. I'll give it a look though.
definitely having some kind of confirmation on launch would be good. If someone has some items they really don't want to be without they may prefer to not go through with the order. I would assume that most people just would want to auto confirm though

@wardnath I think I got the auto-confirm out of stock feature to work. Just testing it to make sure its reliable. It was actually surprisingly easy. I had just been using the wrong code snippet from the CubeMG page. Will keep you posted.

@wardnath @donoka After some more testing, I feel confident the new auto-ignore out of stock feature works reliably so I updated the master branch. Let me know if it works for you / what you think.
The automatic slot selection and checkout feature have proven far more difficult. More specifically, I ran into a roadblock: Once my script finds a slot, I can't even manually select it in Safari on my computer so I get stuck on that page, no matter how many times I refresh. To ensure it wasn't an out of stock issue, I loaded the checkout page in Google Chrome, and was repeatedly able to select a slot and finishing checkout out. I am also able to select a slot in Safari on a different MacBook in my house. Either something is wrong with my laptop's Safari, or perhaps Amazon has flagged my laptop's Safari client on my specific laptop due to my constant refreshing.
Am curious to hear your thoughts. Thanks

This might be the time where we start using a VPN to make sure the computers don’t get flagged as bots, or if they do it’s only temporary. Let’s hope the “ban” is not connected to the amazon account.

@donoka @wardnath would love your thoughts on the auto-ignore out of stock feature if have the time. Has it worked for you? Thanks!

Thanks for adding in the feature @ahertel! I’ll try it next time I do an order. This project has been so helpful.

Update @ahertel, it works great! I noticed I had to give permissions to allow the script to click for me. Other than that it worked fine.

Awesome work @ahertel


Allow versions of handlebars >= 2.0.0

Loosens up the requirements for handlebars versions.  Your readme says any version of handlebars should work, but the package.json would always only install a 2.x version, which have known security vulnerabilities.  This change allows the latest 4.x version to be used.

Thanks!

@thec0keman Any chance you'd consider publishing a bugfix release to npm so we can start using this?

Definitely!  It's been on my todo list.  Will try to get to today

@thec0keman I see version 0.11.0 was published today... however, it doesn't contain my dependencies change.  :(

Apologies, I had a bit of a git mixup.  0.11.1 is published and should include the changes.


Question: Project Factory - configure service project to shared subnet

Hello,
Sorry if im being a bit thick here, since the various PF changes in the last few versions i'm a little lost as to how in a yaml defined project factory entry i'd now configure a shared subnet from the networking layer.
Previously this was done, and i can't quite figure out the new way.
Thanks
vpc:
  host_project: fast-dev-net-spoke-0
  subnets_iam:
    europe-west1/dev-default-ew1:
      -<EMAIL_ADDRESS>      - serviceAccount:my-service-account

Just mirror the project factory module variable type, but un yaml:
shared_vpc_service_config:
    host_project: module.host-project.project_id
    service_identity_iam:
      "roles/compute.networkUser":
        - cloudservices
        - container-engine

Thanks! Closing it off, i didn't notice the lookup values for the various services


Support Pulling EPG From ATSC Signal

Just as the subject states. With the "free" SageTV EPG going away it would be a nice goal to add the ability for pulling the channel schedules from the ATSC signal the way other DVR software does rather than depending solely on the paid Schedules Direct service or other third party methods of obtaining EPG data.

SageTV already supports doing this from ATSC as well as all the DVB
variants. :)
On Thu, Jan 5, 2017 at 1:16 PM, Jason Bean<EMAIL_ADDRESS>wrote:

Just as the subject states. With the "free" SageTV EPG going away it would
be a nice goal to add the ability for pulling the channel schedules from
the ATSC signal the way other DVR software does rather than depending
solely on the paid Schedules Direct service or other third party methods of
obtaining EPG data.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/issues/241, or mute the thread
https://github.com/notifications/unsubscribe-auth/ANEIDDLegkXSagjlxSpBElHvyA7Cnq72ks5rPV27gaJpZM4LcIzm
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

Ok, never mind then. :P


Allows forward slashes on the reference name

fixes #23

Hi @daspn.  I'm not sure if this fix is working properly?  I've just tested it and am still getting the same error.
Code:
      - uses: daspn/private-actions-checkout@hotfix/branch-reference
        with:
          actions_list: '["my-org/my-repo@fix/improve-error-message"]'
          app_id: ${{ secrets.APP_ID }}
          app_private_key: ${{ secrets.ACTIONS_KEY }}

Result:
Run daspn/private-actions-checkout@hotfix/branch-reference
  with:
    actions_list: ["my-org/my-repo@fix/improve-error-message"]
    app_id: ***
    app_private_key: ***
    checkout_base_path: ./.github/actions
App > Cloning using GitHub App strategy
App > Base 64 detected on private key: false
App > Installation: 277[80](https://github.com/my-org/another-repo/runs/7974992292?check_suite_focus=true#step:3:80)718
TOKEN: ***
::add-mask::***
App > GitHub app token generated successfully
Error: Error: The action my-org/my-repo@fix/improve-error-message does not match with the (org|owner)/repo[@branch|@tag] format
Error: The action my-org/my-repo@fix/improve-error-message does not match with the (org|owner)/repo[@branch|@tag] format


Add support for AIO-H3

Amazing, thank you! I was worried we might have our first ambiguous model prefixes, but looks like we'll be OK

Sensors are working AIO-H3-10.0 💯
Is there possibility to get working other features like work mode, charge periods, etc?
Thank you in advance!

Thanks!
Noone's found registers for those yet: as far as the information I have goes (which comes from StealthChesnut's integration), the H3 only exposes holding registers in the range 3xxxx, even when connecting directly over RS485.
Now, it might be that it exposes some input registers as well. Try using modpoll to look for input registers in the range 40xxx - that's where the charge period registers for the H1 are.

I will try modpoll, but we have last FW update 09/22 for AIO inverters. So I think its as you said, AIO is not supported for input registers. Thx for your quick replies and time. ;)

I wouldn't have thought much will have changed in a firmware update. Chances are noone's been looking for those registers yet (and I've only got a H1, so I'm not much use!)


Test configured maps to get regions from parcellations

The reasons for the test:
#332 and https://github.com/FZJ-INM1-BDA/siibra-python/issues/291
Additionally:

monkey mebrains labelled

Pallidum

fsaverage jba30 labelled

BrainStemMask left
BrainStemMask right

Note: The failing tests are exactly the reason this test is added for. Should be in main.

proposed remedy:
either:

update appropriate tag siibra configuration and retrun test. or...
create new tag on siibra configuration and bump version in this PR

Codecov Report

Merging #373 (cb99490) into main (44a0ca0) will decrease coverage by 10.37%.
The diff coverage is 36.43%.

: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
@@             Coverage Diff             @@
##             main     #373       +/-   ##
===========================================
- Coverage   47.11%   36.74%   -10.37%
===========================================
  Files         160       61       -99
  Lines        6843     5339     -1504
===========================================
- Hits         3224     1962     -1262
+ Misses       3619     3377      -242

Impacted Files
Coverage Δ

siibra/locations/boundingbox.py
19.88% <19.88%> (ø)

...bra/features/connectivity/regional_connectivity.py
21.29% <21.29%> (ø)

siibra/configuration/factory.py
22.27% <22.27%> (ø)

siibra/features/anchor.py
23.74% <23.74%> (ø)

siibra/livequeries/ebrains.py
25.00% <25.00%> (ø)

siibra/livequeries/allen.py
26.31% <26.31%> (ø)

siibra/locations/point.py
26.92% <26.92%> (ø)

siibra/features/tabular/cell_density_profile.py
27.10% <27.10%> (ø)

...a/features/tabular/receptor_density_fingerprint.py
28.20% <28.20%> (ø)

...a/features/tabular/regional_timeseries_activity.py
28.75% <28.75%> (ø)

... and 37 more

... and 22 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Sebastian informed me that BrainStemMask cannot be a part of the region tree. But it was required for the creation of fsaverage surface map.
Options I see:

BrainStemMask does not need to be in the map configuration.
or we can add an exception case for the test
What do you think @dickscheid?

looks good. but I would add CI test to brainscapes configuration, which makes more sense than e2e here.

I agree that the proper place is brainscapes-configurations. But I think we need siibra for this check since the region names are not exact matches and there could be subset matches etc. get_region simplifies life a lot. We could import siibra in _ci to check... Thoughts?

looks good. but I would add CI test to brainscapes configuration, which makes more sense than e2e here.

I agree that the proper place is brainscapes-configurations. But I think we need siibra for this check since the region names are not exact matches and there could be subset matches etc. get_region simplifies life a lot. We could import siibra in _ci to check... Thoughts?

the keys in indicies should be an exact match to that of parcellation, no?

All the reason to have the CI on config, so that it can be caught as early
as possible?
On Tue, 23 May 2023, 11:27 Ahmet Nihat Simsek, @.***>
wrote:

looks good. but I would add CI test to brainscapes configuration, which
makes more sense than e2e here.
I agree that the proper place is brainscapes-configurations. But I think
we need siibra for this check since the region names are not exact
matches and there could be subset matches etc. get_region simplifies life
a lot. We could import siibra in _ci to check... Thoughts?
the keys in indicies should be an exact match to that of parcellation,
no?
Should be but it is not always the case
—
Reply to this email directly, view it on GitHub
https://github.com/FZJ-INM1-BDA/siibra-python/pull/373#issuecomment-1558898388,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AET34F7EHHV4F2GXYE4MJEDXHR7GXANCNFSM6AAAAAAXXAZPEQ
.
You are receiving this because your review was requested.Message ID:
@.***>

What do you think of the following:
https://jugit.fz-juelich.de/t.dickscheid/brainscapes-configurations/-/commit/bcf09c2c1f1414d753a47138640c8b7901b58a55

Requires https://jugit.fz-juelich.de/t.dickscheid/brainscapes-configurations/-/merge_requests/87

TODO: add a field to maps for regions that are mapped but not represented in the parcellation, such as brain stem mask in fsaverage julich brain 3.


Issue with FHP override; string contained "closed" when status was "open"

I pushed a commit into the main branch which should resolve this specific issue. Need to monitor for FHP statuses and update if needed. Marking as resolved for now.


update from 4.11.0-0.okd-2023-01-14-152430 to 4.12.0-0.okd-2023-02-18-033438 failing

Describe the bug

while updating from 4.11.0-0.okd-2023-01-14-152430 to 4.12.0-0.okd-2023-02-18-033438 control plane degrades due to first member being upgraded failing to return.
It seems that two nodes were updated, one of them being a control plane member caused the upgrade to stop progressing.
During the update (after all the operators were updated) the vsphere-problem-detector threw the following log:
I0228 11:39:51.577708 1 vsphere_check.go:236] CheckAccountPermissions failed: missing privileges for vcenter: Cns.Searchable, InventoryService.Tagging.DeleteCategory, Sessions.ValidateSession

After seeing that I had our vmware infrastructure team add those privileges to the role our cluster service account uses in vsphere and that issue cleared up. However, this was all while the two nodes (one infra/worker and one master) were in the state they're in now and in the must-gather.
Both systems are reachable should diagnostics outside of the must-gather be helpful:
❯ ssh<EMAIL_ADDRESS>rpm-ostree status # os-infra-dev-01-zh59r
State: idle
Deployments:
● ostree-unverified-registry:quay.io/openshift/okd-content@sha256:6ccff52c50e1ef975931242dc1941617431d45fbd3e425b8016d2cc62aa543d8
                   Digest: sha256:6ccff52c50e1ef975931242dc1941617431d45fbd3e425b8016d2cc62aa543d8
                  Version: 37.20230110.3.1 (2023-02-28T11:39:57Z)

  pivot://quay.io/openshift/okd-content@sha256:bc4fe370cd76415d045b6cc2cf08e5f696ece912661cfe4370910020be9fe0b6
             CustomOrigin: Managed by machine-config-operator
                  Version: 411.36.202301141513-0 (2023-01-14T15:17:08Z)

❯ ssh<EMAIL_ADDRESS>rpm-ostree status # dev-nkjpp-master-1
State: idle
Deployments:
● ostree-unverified-registry:quay.io/openshift/okd-content@sha256:6ccff52c50e1ef975931242dc1941617431d45fbd3e425b8016d2cc62aa543d8
                   Digest: sha256:6ccff52c50e1ef975931242dc1941617431d45fbd3e425b8016d2cc62aa543d8
                  Version: 37.20230110.3.1 (2023-02-28T11:41:09Z)

  pivot://quay.io/openshift/okd-content@sha256:bc4fe370cd76415d045b6cc2cf08e5f696ece912661cfe4370910020be9fe0b6
             CustomOrigin: Managed by machine-config-operator
                  Version: 411.36.202301141513-0 (2023-01-14T15:17:08Z)

Version

4.11.0-0.okd-2023-01-14-152430 to 4.12.0-0.okd-2023-02-18-033438
vSphere IPI
How reproducible

1 for 1 right now. We'll try our other clusters if we can keep this one from tipping over.
Log bundle

must-gather.local.8365916698519417107.zip

Kubelet seems to be unavailable on both nodes, so the must-gather does not contain much logs from the nodes. Does sudo systemctl status kubelet report anything useful from the nodes?
$ omg get nodes os-infra-dev-01-zh59r -o json | jq .status.conditions
[
  {
    "lastHeartbeatTime": "2023-02-28T11:39:56Z",
    "lastTransitionTime": "2023-02-28T11:40:37Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "MemoryPressure"
  },
  {
    "lastHeartbeatTime": "2023-02-28T11:39:56Z",
    "lastTransitionTime": "2023-02-28T11:40:37Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "DiskPressure"
  },
  {
    "lastHeartbeatTime": "2023-02-28T11:39:56Z",
    "lastTransitionTime": "2023-02-28T11:40:37Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "PIDPressure"
  },
  {
    "lastHeartbeatTime": "2023-02-28T11:39:56Z",
    "lastTransitionTime": "2023-02-28T11:40:37Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "Ready"
  }
]
$ omg get nodes dev-nkjpp-master-1 -o json | jq .status.conditions
[
  {
    "lastHeartbeatTime": "2023-02-28T11:40:32Z",
    "lastTransitionTime": "2023-02-28T11:42:12Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "MemoryPressure"
  },
  {
    "lastHeartbeatTime": "2023-02-28T11:40:32Z",
    "lastTransitionTime": "2023-02-28T11:42:12Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "DiskPressure"
  },
  {
    "lastHeartbeatTime": "2023-02-28T11:40:32Z",
    "lastTransitionTime": "2023-02-28T11:42:12Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "PIDPressure"
  },
  {
    "lastHeartbeatTime": "2023-02-28T11:40:32Z",
    "lastTransitionTime": "2023-02-28T11:42:12Z",
    "message": "Kubelet stopped posting node status.",
    "reason": "NodeStatusUnknown",
    "status": "Unknown",
    "type": "Ready"
  }
]

thanks for taking a look @melledouwsma
Yeah, the kublet isn't running due to the absence of /run/resolv-prepender-kni-conf-done which seems to be managed by
/etc/NetworkManager/dispatcher.d/30-resolv-prepender which I am endeavoring to understand this morning to try and understand better where the root of this issue is.
[core@dev-nkjpp-master-1 ~]$ systemctl status kubelet
● kubelet.service - Kubernetes Kubelet
     Loaded: loaded (/etc/systemd/system/kubelet.service; enabled; preset: disabled)
    Drop-In: /etc/systemd/system/kubelet.service.d
             └─01-kubens.conf, 10-mco-default-madv.conf, 10-mco-on-prem-wait-resolv.conf, 20-logging.conf, 20-nodenet.conf
     Active: activating (auto-restart) (Result: exit-code) since Wed 2023-03-01 11:30:40 UTC; 4s ago
    Process: 2578 ExecCondition=/bin/bash -c test -f /run/resolv-prepender-kni-conf-done || exit 255 (code=exited, status=255/EXCEPTION)
        CPU: 2ms

[core@dev-nkjpp-master-1 ~]$ stat /run/resolv-prepender-kni-conf-done
stat: cannot statx '/run/resolv-prepender-kni-conf-done': No such file or directory

[core@dev-nkjpp-master-1 ~]$ systemctl status NetworkManager
● NetworkManager.service - Network Manager
     Loaded: loaded (/usr/lib/systemd/system/NetworkManager.service; enabled; preset: enabled)
    Drop-In: /usr/lib/systemd/system/NetworkManager.service.d
             └─NetworkManager-ovs.conf
     Active: active (running) since Wed 2023-03-01 11:16:05 UTC; 16min ago
       Docs: man:NetworkManager(8)
   Main PID: 1052 (NetworkManager)
      Tasks: 3 (limit: 38420)
     Memory: 8.5M
        CPU: 519ms
     CGroup: /system.slice/NetworkManager.service
             └─1052 /usr/sbin/NetworkManager --no-daemon

Check logs on the node for "nm-dispatcher" - this would have logs from 30-resolv-prepender

looks like maybe some more selinux gremlins?
Mar 01 11:16:08 dev-nkjpp-master-1 audit[1063]: AVC avc:  denied  { read } for  pid=1063 comm="nm-dispatcher" name="dispatcher.d" dev="sda4" ino=4207358 scontext=system_u:system_r:NetworkManager_dispatcher_t:s0 tcontext=system_u:object_r:NetworkManager_initrc_exec_t:s0 tclass=dir permissive=0
Mar 01 11:16:08 dev-nkjpp-master-1 audit[1063]: SYSCALL arch=c000003e syscall=257 success=no exit=-13 a0=ffffff9c a1=55fbd8e47ae0 a2=90800 a3=0 items=0 ppid=1 pid=1063 auid=4294967295 uid=0 gid=0 euid=0 suid=0 fsuid=0 egid=0 sgid=0 fsgid=0 tty=(none) ses=4294967295 comm="nm-dispatcher" exe="/usr/libexec/nm-dispatcher" subj=system_u:system_r:NetworkManager_dispatcher_t:s0 key=(null)
Mar 01 11:16:08 dev-nkjpp-master-1 audit: PROCTITLE proctitle="/usr/libexec/nm-dispatcher"
Mar 01 11:16:08 dev-nkjpp-master-1 nm-dispatcher[1063]: req:22 'connectivity-change': find-scripts: Failed to open dispatcher directory '/etc/NetworkManager/dispatcher.d': Error opening directory “/etc/NetworkManager/dispatcher.d”: Permission denied

Yeah, i think you may be right. I swear tried this yesterday. Will open another issue if I encounter another problem.
Apologies for the oversight on my part and thank you very much for the eyes and brains @vrutkovs  and @melledouwsma.


Izzyondroid version is not up to date ?

On Izzy repo image toolbox is still on v.2.1.3...
Will it get updated or is there some issue with apk size or signature?

I can't say, because updates on platforms except GitHub and GooglePlay performed automatically without my control

Understood, closing this as completed 👍


Making one edit causes 4 calls to Gatsby Cloud

It seems there is not sufficient debounce on the posts to the preview server. I can make one small edit on a page and it posts 4 separate events. Does it run a preview HTTP POST on every keystroke?

Good point, I'll see that it happens!

Is this by chance when saving an entry with some matrix fields?

Nearly all my pages have matrix fields, so yes!
On Mon, 5 Jul 2021 at 13:18, Andris Sevcenko @.***>
wrote:

Is this by chance when saving an entry with some matrix fields?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/craftcms/gatsby-helper/issues/14#issuecomment-874033068,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFWJBFNLQEZPELWG5VT5PLTWGIJXANCNFSM47E7QV6A
.

The plugin should only be sending one build request per entry save.
Which helper version are you using?
Do you have any other plugins installed?
We're talking about the build URL, not the preview URL, correct?

No I was talking about the preview server and url - check original message, thanks!

Interesting. Do you have the build URL also set up?

Yes I have both set up.

Okay, then I can account for three of those events. I'll look into improving this.

@richardfrankza we discussed this internally and I'll try to cut a release with improvements tomorrow!

awesome news, we are struggling a bit with authoring experience, especially
existing CraftCMS users we are converting to headless. They are used to
instant results, so anything to improve preview and build experience is
very key for us right now. To be honest we have clients saying they want to
go back to their old Craft 2!
On Mon, 5 Jul 2021 at 16:25, Andris Sevcenko @.***>
wrote:

@richardfrankza https://github.com/richardfrankza we discussed this
internally and I'll try to cut a release with improvements tomorrow!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/craftcms/gatsby-helper/issues/14#issuecomment-874154301,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEFWJBFAXMJVL43MHJPQBALTWG6E5ANCNFSM47E7QV6A
.

--
Richard Frank
Head of digital
Mobile 27(0)832915952
Tel 27(0)870958182
Email @.***
Website www.flowsa.com

Just cut the 1.0.4 release that should improve this for you.
Removed the unneeded things:

a build was triggered on draft/revision save
a preview build was triggered on opening Live Preview, even if no changes had been made yet

Also, added a page build trigger if a non-draft/revision element is deleted.


deps: bump operator-lib to v0.4.1

Description of the change: bump operator-lib
Motivation for the change: bugfixes to ansible-operator and helm-operator
Checklist
If the pull request includes user-facing changes, extra documentation is required:

[x] Add a new changelog fragment in changelog/fragments (see changelog/fragments/00-template.yaml)
[ ] Add or update relevant sections of the docs website in website/content/en/docs

/area dependency


php 7 compability?

Any plans to upgrade to php7 compability?

I wonder this as well!
The module uses mysql_result which is deprecated since php 5.5 and removed in php 7

WHMCS OnApp User module has php7 compatibility and there are no mysql_result in source code since module version - 3.7.2.
There are some functions that looks like deprecated mysql functions but they are provided by WHMCS for compatibility.

Thanks @prado1991 for the info. Didn't see the later versions, the last release is 3.7 (https://github.com/OnApp/OnApp-WHMCS-UsersModule/releases) that they created...

fresh release:
https://github.com/OnApp/OnApp-WHMCS-UsersModule/releases/tag/v3.7.4


hasToken function implementation

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
For changelog. Remove if this is non-significant change.
Category (leave one):

New Feature
Short description (up to few sentences):
Function to check if given token is in haystack, supported by tokenbf_v1 index;
...

Detailed description (optional):

Function to check if given token is present in a string;
Special case for hasToken to 'tokenbf_v1' index;
Test cases for hasToken()
Test case for hasToken() + 'tokenbf_v1' integration
...

Closes #5296

Your own test has failed.


[DOC-2606] - Update Background step Port Binding description

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[x] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

DOC-2606 - Tech reviewed on Slack

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[ ] Optional Screen Shoot.

Preview environment: https://hdh.pr.harness.io/pr-876

Preview environment: https://hdh.pr.harness.io/pr-876


Fix clipping in fixed mode for edge snapping map points

The bounds for the canvas in fixed mode are offset to the right by the size of the border. This translates the canvas bounds back to the left so we don't clip the icon if it's on the left side.
Fixes #5618

Oh, so it's the black borders when the Runelite client is larger than the fixed mode size that affects this? I see. Will test this and confirm the behavior is correct later today.

Tbh, wouldn't simply setting X/Y of the canvas bounds unconditionally to 0 here solve the issue as well?


Can't print CreateOrderResponse because not JSON serializable

Trying to use dumps as recommended by the readme with the CreateOrderResponse, getting the following error:
  File "/usr/src/app/./cbp-cli.py", line 111, in <module>
    order_id = place_order_with_timeout(factor, 2)
  File "/usr/src/app/./cbp-cli.py", line 81, in place_order_with_timeout
    fprint(dumps(order, indent=2))
           ~~~~~^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.13/json/__init__.py", line 238, in dumps
    **kw).encode(obj)
          ~~~~~~^^^^^
  File "/usr/local/lib/python3.13/json/encoder.py", line 200, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/local/lib/python3.13/json/encoder.py", line 261, in iterencode
    return _iterencode(o, 0)
  File "/usr/local/lib/python3.13/json/encoder.py", line 180, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
                    f'is not JSON serializable')
TypeError: Object of type CreateOrderResponse is not JSON serializable

Can you try using the to_dict() function on the order object like so:
print(dumps(order.to_dict(), indent=2))


chore(cli): fix bun:test ts error

Describe your PR

Fixes the ts error under importing from "bun:test"

So it appears since I cloned this branch, that my local repo just can't bun run build anymore, due to the same bun-types/@types/node conflicting types error, no matter the HEAD I checkout. The solution I posted in the comment directly above does work in a fresh clone. I'm not sure if typescript is caching something regarding input files...

@sammoore There's some config details wrt bun here: https://bun.sh/docs/typescript#suggested-compileroptions
Should we try those?

@sammoore There's some config details wrt bun here: https://bun.sh/docs/typescript#suggested-compileroptions
Should we try those?

@frankcalise maybe if it's used in a separate tsconfig.json specifically targeting the __tests__/. But the rest of cli/ should not because the tsc build output for the actual CLI's source may or may not run in bun (e.g. it can run on the user's machine via npx / etc).
The original issue was due to IntelliSense, but by adding __tests__/ to the same cli/tsconfig.json, it's now trying to build the tests as part of the rest of the CLI. We are running the tests with bun, so we don't need them compiled by TypeScript. We also don't want bun-types available within plain cli/ code (the other change made to the tsconfig).
IntelliSense does make use of tsconfig to do it's thing, but I'm not sure if we could configure different build targets or use a separate tsconfig.json that editors will automatically pick up.
I think I fixed my machine with whatever weird build cache issue was happening and the solution I posted above should work. This would work assuming we revert any changes to the tsconfig.json, that way it doesn't "include:" the tests folder, so that IntelliSense uses a default configuration for those files. It's not as good as having a separate tsconfig.json for the __tests__ though, limiting the number of settings/overrides possible with inline references.

Sure I'll set up the test specific tsconfig and see how it operates


Allow custom background

Just a feature request:
I'm not aware of any routeplanner where you can add a custom overlay or background map. My usecase is: "plan a route to actively avoid areas with mapillary coverage", but I can imagine loads of other usecases (e.g. try to route as close as possible to roads alreadye exisitng on this 1873 basemap, haha) . It would of course be awesome to have an automatic routing service for that job, but that's more complicated by an order of magnitude I suppose.

@joostschouppe : is this open source?
You are talking about this coverage:https://www.mapillary.com/examples/tiles/map-leaflet
correct?

@joostschouppe As long as you have the mapillary data available, the coverage use case should be possible with any route planner with only minor modifications of the data import. E.g., you could use a route planner that considers traffic and model the coverage as high traffic. If it is sufficient to avoid certain areas, then you could use ORS' avoid polygons.
For the 1873 use case you need a routable street network graph of your base map. Then you could route on this graph and use map matching to fit the route to the modern map.

I'll ask Mapillary itself to comment on the license of the tile layer and availability of data to input into routing choices
@sfendrich : my suggested solution is based on some leaflet edits (which is of the level of things I can do myself), "minor modifications of data import" is something I hope to be able to do one day (but only because I'm a dreamer). And yeah, there's no 1873 routable network I can think of; my point was rather "people have all these weird things they would like to consider when clicking together a route".  So any route planner that has the option to let people add their own extra layer would be interesting for all sorts of people with niche interests.

@joostschouppe you are free to use all the Mapillary tile layers. 🙂 Also, you can freely use the Mapillary images API to get all the images in a bbox, just register a client ID at mapillary.com/developer.
Try: https://a.mapillary.com/v3/images?client_id=YOURCLIENTID&bbox=lon1,lat1,lon2,lat2
For example get all images in Bruges. Then it's a geojson response, with the point locations of images. V3/sequences API returns lines. With either you could then add a buffer around the points/lines, then I believe the routing API accepts a geojson layer (polygons now) as an exception to the route, or maybe that geojson layer becomes weighted negatively in the routing decision.


To Be addressed on the Readme

Why xpubs are unencrypted?

Is BWS production ready?

BWS have a extensive test suite but have not been tested on production environments yet, and have been recently released, so it it is still tagged as BETA software.

Having xpubs unencrypted allow the server to derive new wallet's addresses. This means that the server can:

Safely add addresses to wallet's address list (for balance calculation) after a copayer call createAddress. Not having the xpubs would mean that the server would need to ask other copayers about if certain address generated by certain copayer is valid by using a quorum mechanism, this would make the operation more complex and slower.
Craft transactions proposals by deriving the change address, and so, lock wallet's UTXOs in a centralised and atomic operation.  The alternative, currently been used by Copay, can generate of race conditions.
Simplify wallet creation by collecting all copayers xpubs and distributing them.

The main disadvantage  of having the xpubs unencrypted on the server is that the server can see future and past transactions of certain wallet. Here are some points to considered:

There is no linkage between wallet IDs and true copayer's IDs. By having a better representation of the wallets at BWS (vs. Copay's current server schema) DOS can be mitigated by multiple means, so email registration or similar schemas are not needed even for public BWS servers. We think this is a major win.
(Current) BWS clients do not access the blockchain directly, but the use a blockchain explorer, like Insight. Querying transactions and balances thru a block explorer expose the wallet's transaction history and balance to the block explorer server already.

For enhanced privacy, a custom BWS (and Insight) instance should be run and secured by the users. Given that running a BWS instance is trivial and can be setup in minutes, we believe that exposing the xpubs to the server is the right decision.


Add self count to FlameGraphDiff

Purpose

This allows counting and displaying the allocation changes of individual functions (independent of their child nodes). The need for this was identified whilst attempting to quantify local performance in #205. In the future, this will allow us to isolate allocation changes within our software from changes in the upstream models.
This is tagged as part of #229
To-do

[x] separate allocation calculation into child and current function contributions.
[x] implement in FlameGraphDiff plots
[x] add a ci test
[x] add in docs

QA

[x] check buildkite plots are created
[x] check ci tests pass

Note

this is still in a temporary format/location, until we test/determine how this would be useful for the rest of the CliMA team.

[x] I have read and checked the items on the review checklist.

bors r+