MassTransit is swallowing exceptions thrown when a saga activity without a registered dependency is activated

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes
Environment

Dotnet version: netcore 3.1
Operating system: Windows
IDE: Visual Studio

Steps to Reproduce

Create the simplest MassTransitStateMachine.
Create the simplest activity with one constructor argument that will be injected (in my case it's a Func mapping a string to an enum).
Try running your saga - it will never get to the Activity's Execute method because of the missing DI registration.

Expected Behavior
An exception should be rethrown, so that it will be easily visible in app logs.
Actual Behavior
The exception thrown by the DI container (in my case it's Autofac) is swallowed by MT, so discovering that missing registration may be a challenge.

Also exceptions are swallowed when configuration of saga repository is invalid, for example there is a missing class mapping in MongoDB repository

Where are you expecting to catch the exception?
The exceptions are thrown and logged and messages are moved to the error queue in the scenarios.

The whole problem is that the exception is nowhere to be found. I don't see it in my logs, despite having set error log level to log everything. I'll try to reproduce and put it somewhere here.

If you aren't seeing any log messages from MassTransit, you likely don't have an ILoggerFactory instance configured in your container.
If you aren't using a container, you may not be provided an ILoggerFactory to MassTransit before creating the bus (using LogContext.ConfigureCurrentLogContextSomethingOrOther.


Improve ease of use of .plot_ensemble_model()

Describe the issue linked to the documentation
Due to dependency requirements, predictor.plot_ensemble_model() is not easy to use.
https://github.com/awslabs/autogluon/blob/master/tabular/src/autogluon/tabular/predictor/predictor.py#L2659
Suggest a potential alternative/fix
Refactor predictor.plot_ensemble_model() and make the dependency rely on a package that is installed by default in AutoGluon and works across Linux, Mac, and Windows. The refactor should also work in notebook environments.

Hey @Innixma, would like to take a crack at this. Will have a look and reach out to you ?

@manikTharaka Sorry, didn't see your response initially. Yes feel free to give it a try!


NO clear method to uninstall defined in documentation

There is no documentation anywhere that indicates how to uninstall Ghost.

(Helping in slack)


Release 2.2

please close this pr, creating at 1831

A few things appear to have gone wrong with this PR, including the lack of a DCO signoff. Please consult this link: https://hyperledger-fabric.readthedocs.io/en/release-2.2/github/github.html#committing-and-pushing-changes-to-your-forked-repository
For information about pushing changes with the necessary sign off.


handler simplifications

with the @mandatory annotations in the model some of the handler checks are not needed anymore
@stewsk @danjoa I removed some Java code but the Maven CDS build fails for the CDS changes in this branch. Does this need some later CDS version?

Hi @rjayasinghe ,
tried mvn spring-boot:run locally, worked fine. What do I have to do to reproduce the errors you saw?

Hi @rjayasinghe , tried mvn spring-boot:run locally, worked fine. What do I have to do to reproduce the errors you saw?

Oh.. that was just because of an inconsistency in my local environment. It still saw/used the old 7.x cds-dk which was not able to compile the CDS syntax in this branch.

@stewsk I tested manually on my machine and also ran the OPA integration tests successfully. IMHO we can merge this to the simplify branch.


Add C4iOS to Xcode project

I cannot get C4 running in my new Xcode project. Since the installer isn't working for (the current version of) Xcode anymore, how can I set up a project? I don't find any instructions for that. Can you give me some instructions how I accomplish adding the dependencies?
Thanks!

Hey Christoph,
Which version are you interested in? The ObjC branch or Swift?

Simplest way to get a new project now, for ObjC:

Download the ObjC branch

Copy the folder (once it has been unzipped)

Rename the copied folder (e.g. MyProject)

Double-click on MyProject > C4iOS.xcodeproj

Select the C4iOS Xcode Project (top element) in the Project Navigator

Double-click the title of the C4iOS target (under the General tab) and change it to "MyProject"

Build and Run

For the new C4Swift branch:

Copy or Clone the current C4iOS master branch

Create a new Xcode project, iOS, Swift only

Right-Click on ViewController.swift and choose "Add Files To..." option

Choose the C4iOS.xcodeproj you just cloned

In Project Navigator, choose MyProjectSwift

Under General, click the + under the "Embedded Binaries" section

Choose C4Core and C4UI frameworks

Update your ViewController to look like:

import UIKit
import C4Core
import C4UI

class ViewController: C4CanvasController {
    override func setup() {
        C4Log("MyProjectSwift")
    }
}

Build + Run

p.s. we're working on a new installer for the Swift version

Thanks for your reply! I want to use the Swift version.
I followed your instructions. However, I can't choose C4UI.framework and C4UI.framework, they're not available after I added the master files, only C4.framework. Is there something I miss?

Yea! Actually, right after I posted these @aleph7 pushed a big change that reduced the project from 2 frameworks to 1. Literally, just an hour or two after I posted the project structure changed.
So, yes, it should be the same process but only with C4.framework

@christophz did this work for you?

Yeah, it finally worked. First I was confused about the red flag on the import statement line, but it disappeared after compiling.
Thanks a lot for your help! I am eager to try the Swift version of C4! Dope new logo, by the way! :)


why does my internet connection die after the brute force subdomain is running?

Describe the bug
so I tried to scan as usual but when I was in the bruteforce subdomain menu, my internet suddenly died, and it was like being stuck in the bruteforce subdomain, I had tried it many times by changing the recon config etc., until I waited for hours, still stuck in the bruteforce subdomain. is there any solution about this?
Screenshots

Desktop (please complete the following information):

Architecture: [e.g. x86, x64, ARM]
OS: [e.g. Kali,Debian in WSL2]
Version [e.g.Rolling]

https://github.com/six2dez/reconftw/wiki/7.-FAQs#usage


Use on 1.10

At present, CompilerPluginTools.jl does not precompile on 1.10.0-beta2. I do not know this will cease to be the case.
For Julia version 1.9 and up, this package doesn't technically require CompilerPluginTools.jl.
@dfdx do you have thoughts on what the best way forwards is for having this package work on 1.10? The options that I can see are

fix CompilerPluginTools on 1.10 (I've not interacted with that package so don't know how to fix it, and I'm not whether it's actively maintained), or
drop Julia versions below 1.8, and this package's dependency on CompilerPluginTools.

What are your thoughts? Maybe I've missed an option?

#43 explores option (2). As you can see in CI/CD, the tests still fail because of changes in Core.Compiler.IRCode structure. We can fix it now, but since it's not a release yet, there's a high chance to get more breaking changes very soon. So unless you have an urgent use case, I'd say let's wait for the release and see what it takes to make Umlaut work on Julia 1.10.

Sounds good to me


yandex translator

I would like to use https://www.npmjs.com/package/yandex-translate instead of google translater , do we have any option for it.

I am going to start on the implementation soon, will keep you updated via this issue #1 ;)

Thanks for the request. It is ready now v2.1.0
https://www.npmjs.com/package/translate-json-object


[NETBEANS-71] Change product name

The product name, i.e. 'NetBeans Platform' or 'NetBeans IDE', is changed to reflect the new Apache home, in a couple of locations. This is done to distinguish the product from the pre-Apache era.
These changes are expected to be permanent. As for any actual release, such release will be expected to brand these variables, e.g. insert the word "(incubating)" for releases done while in Apache incubating phase, etc. This is similar to the existing release process prior to NB9.
It doesn't make sense to change every occurrence of "NetBeans Platform" or "NetBeans IDE". It only makes sense in the most prominent locations, IMO. The ones I've chosen are outlined below.  There are a few other locations where I think it makes sense to change as well, but they are all related to the IDE's Installer, and since that part hasn't been donated yet, it cannot be changed just yet.
Changes in this PR
1. The main window's title bar

2. the product name in the About dialog.

(less prominent because nobody notices) : The title of the splash window in the OS window tray / status line

+1 looks great and good thinking.

Looks ok, feel free to merge. Just out of curiosity - is the name change needed?

Well, one or two mentions of Apache in NetBeans itself would be good. Merging.

Whether it is required or not:
http://apache.org/foundation/marks/pmcs.html#naming:
As you can see the ASF really want us to use "Apache NetBeans" rather than just "NetBeans". If it is strictly required in the locations I've changed in this PR is open for interpretation. But the intent from the ASF is certainly clear. In any case I think it is in our (the (new) NetBeans community) interest too, i.e. to set the product apart from pre-Apache era.
I've looked at Tomcat and Cassandra and they are pretty consistent in using the full name, in particular Tomcat will never, ever, refer to itself as solely "Tomcat". This goes for docs, UI and log messages.
Btw:  With this change, because of the changed currentVersion variable, the name which is printed at the top of the messages log will also change:
System Info:
  Product Version         = Apache NetBeans IDE Dev (Build 20171128-unknown-revn)
  ...

Thanks for your input!


⚠ Bofrost.com has degraded performance

In 7835e47, Bofrost.com (https://www.bofrost.com) experienced degraded performance:

HTTP code: 200
Response time: 2778 ms

Resolved: Bofrost.com performance has improved in c4850e0.


fix: Refactor @kaoto-next/camel-catalog

Move custom language/dataformat catalog handling from UI
Split custom catalog/schema processing into individual processor classes
Add patterns catalog which is a processors catalog filtered from models

Fixes: #316
Fixes: #335

beans model catalog disappeared after camel 4.2.0


crashes at login() on Android

In debug mode, it works fine; however, in release mode, it crashes the app as soon as I call login(). Like said, everything works in debug mode, (the server credentials are correct and devices can successfully chat in real time).
The crash I get is this Java exception:
java.lang.RuntimeException: Unable to start service org.xrstudio.xmpp.flutter_xmpp.Connection.FlutterXmppConnectionService@e8f22aa with null: java.lang.NullPointerException: Attempt to invoke virtual method 'android.os.Bundle android.content.Intent.getExtras()' on a null object reference

...btw, I can reproduce this on the official example, too; as soon as I click on connect, the app crashes in release mode but not in debug, where it successfully connects and authorizes

@woolfman007 Thanks for drawing my attention, I will take a look at this. Which version do you use of the plugin can you please confirm once ?

Thanks for responding so quickly, I appreciate that very much!
I am using xmpp_plugin: ^2.2.3 in my own project, but I could reproduce that
also in the official example, which I downloaded from GitHub... I'm using
pretty much the latest Flutter version:
Flutter 3.3.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision e3c29ec00c (12 days ago) • 2022-09-14 08:46:55 -0500
Engine • revision a4ff2c53d8
Tools • Dart 2.18.1 • DevTools 2.15.0

@vavadiyahiren could you already investigate this? Can you reproduce it?

Fixed with the latest version @woolfman007 , Feel free to reopen this if it is still happening with you.


Updated GraphQL to the latest (2.3)

Was wondering if there's a timeline for upgrading graphql-dotnet to 2.x? I understand there's one breaking change but it sounds like @benmccallum was able to work around it. Any other reasons to not move forward with it?

Nope, just whether it fits into the timelines of contributors :)  At the moment, I'm pretty buried in deadlines that aren't related to this library.  If anyone wants to take a stab at it (I bet it isn't that difficult), please feel free.

Thanks for the contribution.  I'll get 0.4.0-beta released ASAP :)

This is included in the 0.4.0-beta release, which I just finished publishing.  Happy coding!

Doug, I've just updated Dapper.GraphQL package in my project and still seeing old GraphQL version after update. Uninstalled it and installed from scratch just to be sure. No luck. Nuget hiccup?

@narbit Apologies, my fault - I missed a pretty important git merge step :)   I've updated it to version 0.4.1-beta, which should now have your PRs.  Please let me know if you run into any more issues!
Thanks!


Guide Article: D3 JS data join (guide/english/d3)

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@thecodingaviator This one has gone stale.

@RandellDawson I can fix the conflicts and send PR soon

@sabeersulaiman Please send a PR soon

@thecodingaviator See if it is good now. Sorry for the delay.


Feature Request: Allow minDate/maxDate to be bound

Currently the minDate and maxDate parameters will only accept a raw Date object. This follows pikaday's functionality but it leaves a lot of potential on the table.
For example, imagine a start and and end date pickers, the end date could have the minDate bound to the selected start date.
If I have enough time left at the end of this project I'll see if I can put a pull request together but I thought I'd record it here for others in case they also wonder why their date pickers aren't responding to their computed min/max dates.

Thats a great idea.
I'm not sure the component is the right place for this, since there is no cross-component communication. You could utilise your controller to achieve this. Have a controller property that generates the endDate's minDate based on the selected start date.

On 28 Jan 2015, at 22:33, Kevin Ansfield<EMAIL_ADDRESS>wrote:
Currently the minDate and maxDate parameters will only accept a raw Date object. This follows pikaday's functionality but it leaves a lot of potential on the table.
For example, imagine a start and and end date pickers, the end date could have the minDate bound to the selected start date.
If I have enough time left at the end of this project I'll see if I can put a pull request together but I thought I'd record it here for others in case they also wonder why their date pickers aren't responding to their computed min/max dates.
—
Reply to this email directly or view it on GitHub.

You could utilise your controller to achieve this. Have a controller property that generates the endDate's minDate based on the selected start date.

This is actually what I tried, however it fails as assigning the CP to the min/maxDate properties simply passed the CP function through. It seemed like the component would need to intercept those properties and pass the pikaday instance the plain date value instead.
I've just found that there is an open issue requesting min/maxDate to accept a function in pikaday's issue tracker: https://github.com/dbushell/Pikaday/issues/165. Obviously that would be the ideal place for the functionality to be implemented.
I'll see if I can work something out over the next few days. If pikaday is slow to respond, would you be open to a pull request that allows bound min/maxDates in the interim?

Could you post some Code? I'm pretty sure that this should be possible.

Does #40 fix this? If so can we get this merged in? Looks like it needs some tests however.


Undefined HTTP Response code when the X-Broker-API-Version HTTP header is missing

What is the problem?
Undefined HTTP Response code when the X-Broker-API-Version HTTP header is missing.
Who does this affect?
If the response code will change to another code then it could break backward compatibility.
Do you have any proposed solutions?
More clearly define the response code when the X-Broker-API-Version header is missing, like 412 Precondition  or 400 Bad Request .

@aukhatov
Thank you for opening this issue!
It shouldn’t be a 412 Precondition because it implies that client provided a pre-condition header, which it didn’t in this scenario.
So 400 Bad Request sounds like a more legit response status code in this case.
What do you think?

I think, according to the spec the 400 status is more relevant, however it would be more clear to give note about what  - "mandatory data" is meant there. Including headers and request body

Hi @waterlink, it would be okay.

@waterlink has kindly agreed to open a PR for this one


CTHUB - vinpower microservice

Title:
vinpower microservice
Description:
Integrate ESP data's vinpower Java product into CTHUB as a spring microservice - can use ITVR's spring microservice as a template.
Wireframe:
Purpose and benefit to user:
This will allow users to use decoded vin information that was obtained using vinpower.
Acceptance Criteria:

[ ] Given I am a developer, I can RESTfully call this microservice with VINs and have it return the decoded information.

Development Checklist:

[ ] Integrate ESP data's vinpower Java product into CTHUB as a spring microservice - can use ITVR's spring microservice as a template.

Notes:
Will receive the jar file from ESP data on Monday, May 20, 2024 - will update card then.

Hey team! Please add your planning poker estimate with Zenhub @tim738745 @emi-hi @JulianForeman

This card is blocked until we receive the Java file. The file should be available next week Monday. The BA will let us know once they get it.

Received data.


samples: intel_s1000_crb: sample application for digital microphones

Sample application to receive audio from digital microphones in an Intel S1000 CRB.
Uses digital microphone driver APIs defined in include/audio/dmic.h

Codecov Report

Merging #11758 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #11758   +/-   ##
=======================================
  Coverage    48.2%    48.2%
=======================================
  Files         280      280
  Lines       43323    43323
  Branches    10374    10374
=======================================
  Hits        20886    20886
  Misses      18289    18289
  Partials     4148     4148

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f176e4...6b18cc5. Read the comment docs.

As you rightly pointed out, this sample needs a bit of documentation to elucidate how the DMIC is getting tested here.

Documentation added in commit 16903178ba7e0ab5b2b969b256e6b427d473296b


FromArray method doesnt exist anymore.

After i ran composer update i get

exception 'Maatwebsite\Excel\Exceptions\LaravelExcelException' with message '[ERROR] Writer method [fromArray] does not exist.

Have you deleted this method ?

I removed the magic call from the Writer, which caused strange problems something.
If you get this exception it means you are trying to call the fromArray method inside the wrong closure. You should open a sheet first, example:
Excel::create('Filename', function($excel) {

    $excel->sheet('Sheetname', function($sheet) {

        $sheet->fromArray(array(
            array('data1', 'data2'),
            array('data3', 'data4')
        ));

    });

})->export('xls');

I had the same problem and this fixed it.  It used to be quite simple to do this though
Excel::create('export')
->sheet('Posts')
->with($posts_array)
->export('xls');

You were probably on a 0.* version. Since version 1 the api completely changed. The 0.* way was a terrible way of dealing with it, because it had a functionality and concerns in one class, which didn't allow for a lot of flexibility when dealing with views, multiple sheets, multiple sheets with different views etc.

Exactly thank you ! :+1:

Yes, I was on a 0.* version.  Not a problem, I know how to fix it now.  I'm always in favor of making code more flexible going forward.  Thanks!

Great :) That's why I know advice to use something like ~1.3.2 as composer version. We try to follow semantic versioning. So switching from 0 -> 1 -> 2 will bring breaking changes :)


Added the way to modify immutable string.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

I am new over here and I wanted to pull this in new branch but I suppose I did it in master branch. Please look into this if it can be rectified.


Add ability to specify the name of an existing Placement or PlacementRule

This allows managing all Placements/PlacementRules in the ACM hub separate from the generated policies that use them.
Fixes #50
Example usage available at https://github.com/brian-jarvis/test-placement/tree/reuse-placement
Signed-off-by: Brian Jarvis<EMAIL_ADDRESS>

@brian-jarvis could you also please squash all of your commits to a single commit message of Add ability to specify the name of an existing Placement or PlacementRule.

@brian-jarvis could you please fix the linting error?

@brian-jarvis could you please fix the linting error?

Fixed.


Correction on Sourcetype Configuration for FortiMail Events in SC4S Documentation

Hello,
I've identified a small discrepancy in the SC4S documentation regarding the configuration for FortiMail event data. In the current SC4S documentation (available at https://splunk.github.io/splunk-connect-for-syslog/3.21.0/sources/vendor/Fortinet/fortimail/), it instructs users to add an underscore (_) at the end of the sourcetype for FortiMail.
However, I found that the correct sourcetype configuration to properly send FortiMail event data does not include the underscore. The accurate sourcetype should be as follows:
fortinet_fortimail
The presence of the underscore seems to be a typographical error. I am attaching an image from the manual for reference. This correction could potentially help others who are attempting to set up FortiMail with SC4S.

Could the documentation be updated to reflect this correct sourcetype configuration?
Thank you for looking into this matter.
Best regards,

@PricklyPotato thank you, I'm taking care of it

of course tell me what you need for testing, do you want me to send you some PCAPs, I'll record some logs in the Archive and I'll send them to you.

@PricklyPotato thanks, much appreciated
I reproduced it on my end and the PR is awaiting review now: https://github.com/splunk/splunk-connect-for-syslog/pull/2354/files

https://splunk.github.io/splunk-connect-for-syslog/2354/sources/vendor/Fortinet/fortimail/

@mstopa-splunk that sounds great !!!
For my part I solved it that way, by removing the (_) sign.
If you didn't remove the (_), what SC4S did to you, is that it sent you all the events from both fortigate and fortimail as if they were from fortigate and mixed them up.
There was no difference between the osurcetype fgt and fml.
If you need some kind of fortigate logs, I can send them to you if you want to reproduce the problem, just send me an email and I will send them to you.
Best regards.

@PricklyPotato good catch, we had an md formatting bug there . Together with another one in the configuration.
I tested it with fortinet_fortimail_spam and fortinet_fortimail, and it will work now. Feel free to take a look at the PR's diff in case I missed something, but it should be ok

Hello,
I'm glad to be able to help improve SC4S.
Please let me know how we will proceed or if you need any more help from me.

Perfect, so I close the clase.


How to create an order?

Hi there,
thanks for this package! I am currently trying to implement it into my ordering platform. Essentially, what I am looking for is trying to find a way to create an online order so it will show up in my POS along with all the offline orders.
Almost all of the online orders do not need to be paid for. Hence, I only need to create the order, and no charge I assume.
What would you recommend doing in a scenario like this?
Thanks so much Nikola!
Chris

Hello Chris,
Currently the order system is in place but it’s not yet activated since it needs some more testing, the order system will probably be fully functional in the upcoming versions, roughly by the end of January 2018
Best regards,
Nikola Gavric

On Dec 30, 2017, at 00:07, Chris<EMAIL_ADDRESS>wrote:
Hi there,
thanks for this package! I am currently trying to implement it into my ordering platform. Essentially, what I am looking for is trying to find a way to create an online order so it will show up in my POS along with all the offline orders.
Almost all of the online orders do not need to be paid for. Hence, I only need to create the order, and no charge I assume.
What would you recommend doing in a scenario like this?
Thanks so much Nikola!
Chris
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/NikolaGavric94/nikolag-square/issues/13, or mute the thread https://github.com/notifications/unsubscribe-auth/AMBMI8OB2g-G0Mud2Y3_aJEv-Z5tJCVZks5tFXCmgaJpZM4RPcHz.

Thanks Nikola! Looking forward to that :-)
Best,
Chris

@Chris1904 Could you contact me via email regarding this feature, thanks

Implemented as of v2.0.0


Menu map

anyone knows how could make a map with menus and layers? like this:
http://www3.inegi.org.mx/sistemas/mapa/espacioydatos/default.aspx
google payment option or otherwise , any advice  :/
help me please.

Maplace library for now does not support "Data Layers"
https://developers.google.com/maps/documentation/javascript/datalayer


Backticks in the source code ruin a build process

Version
2.7.14
Reproduction link
github.com/yanbess/vue-test
Steps to reproduce
npm install
npm run prod
What is expected?
Successful building
What is actually happening?
ERROR  Failed to compile with 4 errors
error  in node_modules/vue/types/jsx.d.ts:39:7
TS1110: Type expected.

39 |   [v: --${string}]: string | number | undefined
|       ^^^^^
...

You need just replace ` with ' or " here:
https://github.com/vuejs/vue/blob/a9ca2d85193e435e668ba25ace481bfb176b0c6e/types/jsx.d.ts#L39

Your typescript version is way out of date. Template literal types were introduced in TS 4.1 in Nov 2021...


add throttler logger abstraction and logger abstraction implementation

After https://github.com/1pkg/gohalt/issues/1 is implemented we could use marshaled state to do extra info logging. In this issue we need to add logger abstraction that could be accepted and called by existing runners. And add an implementation for logger.

After rethinking i see too small additional value in this logging as gohalt already has fine error propagation and internal logging for some cases where error couldn't be propagated directly. Dumping and logging of full internal throttler state though could be valuable is definitely low priority now, so closing this issue until it's required.


Any change requires a long rebuild of straight-emacs-env

Any change to my doomPrivateDir makes nixos-rebuild switch take ~4m longer, on a fairly powerful desktop computer. nix-doom-emacs is central to my nix configs, but it'd be great to figure out a way around this issue since it is a bit of a sticking point for me. Is this expected behaviour?
In an ideal world update checking would happen as part of nix flake update (if that's even possible), but in the meantime ... is there some possibly-hacky workaround for when I'm making small local changes to config.el and don't need to check for updates to every single elisp package?

See https://github.com/nix-community/nix-doom-emacs/pull/303 for a workaround to avoid rebuilding your config unless you add/remove a package (when this happens, it is impossible to not rebuild everything unless we packaged every single straight-fied package from doom-emacs).

I've been wondering if it would be possible to compile each straight-ified package separately and symlink them all at the end.
I'm willing to put in some work to do this, if this idea is worthwhile.

@charmoniumQ Progress here would unbottleneck a lot of my build times. Are you thinking about helping it parallelize within the overall normal straight-emacs-env nix derivation or rethinking that derivation to become a composition of smaller derivations?

Workarounds aside, I'm tracking the proper solution in #353 and my WIP branch is https://github.com/ckiee/nix-doom-emacs/tree/move-nix-straight-in. There's a big comment in src/straight.nix. My primary focus is making it correct first; Optimizations can come later.
This has been an issue for quite a while.


Use of partial path from enviroment variable

Is your feature request related to a problem? Please describe.
Is not a problem, just a feature.
Describe the solution you'd like
The situation is this. I want to paste a partial path like "OneDrive\work\project 1" and the native file explorer will take the environment variable with OneDrive to fill the complete path to the local copy of OneDrive.

it opens the file explorer for me, when typing something like "OneDrive\Pictures" into the address bar. So, it seems you have to set Files as default to make it work as expected. Is this already known?


close policy when closing last claim of expired policy

In closeClaim
https://github.com/etherisc/gif-contracts/blob/b58fd27a34452e062d10b175f3bdd7ee16b9d1a7/contracts/modules/PolicyController.sol#L358
There is no check to close the corresponding policy if it is expired and have no open claims

very nice analysis and true.
this is in the unclear area of what is good usability for the people implementing products on top of the gif framework and too much hidden magic ;)
would you consider this "just" a usability improvement or can you motivate why this should be classified as a bug?
in any case the current handling is inconsistent with the handling of claims... we do close the claim once all payouts associated with the claims have been processed (ie once the claim amount matches with the paid out amount)
"fixing" this would imply a breaking change in any case.


Dont forget to donate

This program is great. Hit that donate button to say ty because this takes work. mchristopher is doing this on his free time too. Only way I can play this game is because of this, disabled vet.

done!! thank you @mchristopher

what do i do to get the map working again? i generated a new api but its not working.

go to releases and download the version for Mac or PC ... https://github.com/mchristopher/PokemonGo-DesktopMap/releases

Done! Thanks for the work on this!

Sorry i dont see the donate button

Donate buttons are on the top of the login screen for the program.


docs(plugins): clean option

The requestTimeout option was removed in this commit https://github.com/webpack/webpack/commit/9d918615922f8e7fb8ab1c9712135908cf1cadd7#diff-8941681d920ac4feda44f522b986c2d0L99. There're only multiStep and fullBuildTimeout left as we can see here https://github.com/webpack/webpack/blob/master/types.d.ts#L2599

Thanks!


CkLoop race condition error

Original issue: https://charm.cs.illinois.edu/redmine/issues/643

Using CkLoop in NAMD PME offload, works fine on 14 pes, fails with error on 30 when running +p30 ++ppn 30 +pemap 1-15,17-31 +commap 0,16 (on a 16 core, 32 thread machine):
[19] Stack Traceback:
[19:0] CmiAbortHelper+0x73  [0xbb2933]
[19:1] _ZN11CurLoopInfo9stealWorkEv+0x181  [0xa8fa2b]
[19:2] _Z21SingleHelperStealWorkP17converseNotifyMsg+0xb9  [0xa8f8a8]
[19:3] CsdScheduleForever+0x78  [0xbbaee8]
[19:4] CsdScheduler+0x2d  [0xbbafed]
[19:5]   [0xbb6a86]
[19:6]   [0xbb6f53]
[19:7]   [0x301e4079d1]
[19:8] clone+0x6d  [0x301e0e8b6d]
Fatal error on PE 19> Indices of CkLoop incorrect. There maybe a race condition!

Original date: 2015-01-02 23:58:48

Just to add a little extra information, this is a non-sync call.

Original date: 2015-01-04 22:00:41

I get the exact same error from a sync call.

Original date: 2015-01-16 21:17:58

Jim,
Few months back I had fixed a race condition is CkLoop. I believe this one is recent right? Do you have a small example that I can use to debug?
Harshitha

Original date: 2015-01-16 21:21:48

I don't know how to reproduce it outside of full NAMD.  Was your fix included in 6.6.1?

Original date: 2015-01-22 21:29:20

Yes that fix was put in 6.6.1 and I had added the Abort message you got.
I can use NAMD to reproduce this error.

Original date: 2015-02-06 20:47:43

Do I need a specific dataset to reproduce it?

Original date: 2015-02-09 19:53:57

You would need a specific NAMD hack, which I've now abandoned.  It's probably better to do some thorough code review for conditions that could lead to that error.

Original date: 2015-02-09 21:53:16

If it is easy enough, can you share the code changes required to reproduce this problem?

Original date: 2015-02-10 19:11:58

Get the NAMD nightly build source and apply the attached patch.
Build a multicore CUDA NAMD binary (./config Linux-x86_64-g++ --with-cuda).
Get http://www.ks.uiuc.edu/Research/namd/utilities/apoa1.tar.gz and run
./namd2 +devices 0 --PMEOffload yes --FFTWEstimate yes +p30 +pemap 1-15,17-31 path/to/apoa1.namd

Original date: 2015-02-17 21:26:20

I am using lab machine to reproduce the problem. I used your patch and built namd in multicore --with-cuda option. I am using Talent, which has total of 4 cores, for these runs since cuda is installed on that.
I enabled CkLoop and ran the following command. CkLoop constructor is called but parallelizeFunc is not being invoked. Am I missing something? Do you know if this bug shows up on smaller number of cores? If not the lab machines, which machine can I use to reproduce this problem?
./namd2 +devices 0 --PMEOffload yes --FFTWEstimate yes +p3 +pemap 1-3 ~/workspace/namd/apoa1/apoa1.namd
Thanks

Original date: 2015-02-18 16:31:05

As I said originally, it works on 14 pes but fails on 30 on a 16-core, 32-thread machine.
Try Stampede or the campus cluster (Taub/Golub), or hand this bug off to someone with a TCBG token.

Original date: 2015-03-13 06:16:15

I get the same error when building NAMD configured using --with-cuda and -DUSE_CKLOOP=1 etc. in the Makefile and running as:
./charmrun ++local +p14 ./namd2 ++ppn 7 +devices 0,2 $APOA1 --pmeoffload yes --pmepencils 1 --pmeprocessors 2 --fftwestimate yes --useCkLoop 4 --run 20
------------- Processor 11 Exiting: Called CmiAbort ------------
Reason: Indices of CkLoop incorrect. There maybe a race condition!
------------- Processor 7 Exiting: Called CmiAbort ------------
Reason: Indices of CkLoop incorrect. There maybe a race condition!
------------- Processor 12 Exiting: Called CmiAbort ------------
Reason: Indices of CkLoop incorrect. There maybe a race condition!
------------- Processor 9 Exiting: Called CmiAbort ------------
Reason: Indices of CkLoop incorrect. There maybe a race condition!
------------- Processor 8 Exiting: Called CmiAbort ------------
Reason: Indices of CkLoop incorrect. There maybe a race condition!
------------- Processor 10 Exiting: Called CmiAbort ------------
Reason: Indices of CkLoop incorrect. There maybe a race condition!
[11] Stack Traceback:
[11:0] CmiAbortHelper+0x73  [0xbe7013]
[11:1] _ZN11CurLoopInfo9stealWorkEv+0x181  [0xabad57]
[11:2] _Z21SingleHelperStealWorkP17converseNotifyMsg+0xb9  [0xababd4]
[11:3] CsdScheduleForever+0xdb  [0xbef71b]
[11:4] CsdScheduler+0x2d  [0xbef7bd]
[11:5]   [0xbeb25a]
[11:6]   [0xbeb723]
[11:7]   [0x301e4079d1]
[11:8] clone+0x6d  [0x301e0e8b6d]
Fatal error on PE 11> Indices of CkLoop incorrect. There maybe a race condition!

Original date: 2015-03-13 07:07:50

Pretty trivial actually, just need to ensure that iteration range is at least as large as numChunks:
void CkLoop_Parallelize(HelperFn func,
int paramNum, void * param,
int numChunks, int lowerRange, int upperRange,
int sync,
void *redResult, REDUCTION_TYPE type) {
if ( numChunks > upperRange - lowerRange + 1 ) numChunks = upperRange - lowerRange + 1;
globalCkLoop->parallelizeFunc(func, paramNum, param, numChunks, lowerRange, upperRange, sync, redResult, type);
}
I added some extra locks which debugging, so I'm not sure this is the only issue, in particular if CkLoop is being called from multiple threads.

Original date: 2015-03-13 20:25:25

Attaching patch that I'm using with NAMD.

Original date: 2015-03-14 05:30:34

Thanks Jim for fixing the bug.
I have sent your patch for review (http://charm.cs.uiuc.edu/gerrit/#/c/631/).
We don't require the extra locks because if nextChunkId is < numChunks then there is a worker working on a chunk. Only when that worker finishes will it call reportFinished. Note that I have done if (nextChunkId >= numChunks) after locking to get rid of the race. More details of the race is given in CkLoop.h:61.
reportFinished atomically increments the finishFlag and a curLoop is free only when finishFlag == numChunks i.e. all the workers have called reportFinished.
curLoop is not set until getNotifyMsg() is called which checks to see if the curLoop is free. Therefore numChunks and other fields are not modified in a racy fashion.

Original date: 2015-03-15 05:15:36

Thinking about it some more that is correct, but there is a race on curChunkIdx being reset to -1 by set() if the message is recycled by getNewTask() while a non-sync call is still running.  The next call to getNextChunkIdx will then report 0 but since there is no lock there is no guarantee that the writes to the other loop parameters are visible.  What I think might work is to track the number of stealWork calls running and have isFree() only return true when it is zero.
I have an updated patch that I will post tomorrow.

Original date: 2015-03-15 18:05:22

Updated patch attached.

Original date: 2015-03-15 21:08:22

This version removes the "WARNING! chunk is set to MAX_CHUNKS=%d" warnings.

Original date: 2015-05-08 19:54:48

Jim,
Patch http://charm.cs.uiuc.edu/gerrit/#/c/631/ had been checked in a month ago which takes care of the numChunks is less than upperbound-lowerbound+1.
I was trying to figure out whether there is really a race condition in the non-sync mode and I feel there is not. You brought up "but there is a race on curChunkIdx being reset to - 1 by set() if the message is recycled by getNewTask() while a non-sync call is still running".
If a non-sync call is still running with chunks of work pending, then getNewTask will not be returning that particular CurLoopInfo. In function getNewTask() in CkLoop.h:255 cur ->isFree() is tested which checks to see if all the chunks are done (finishFlags == numChunks) and only then returns that CurLoopInfo. If a non-sync call is currently going on with that CurLoopInfo, then finishFlag will not be equal to  numChunks and hence that CurLoopInfo won't be returned in getNewTask().
If all the chunks are done and a worker comes along and does getNextChunkIdx(), it will either get chunkIdx corresponding to the previous non-sync call, in which case it will see that all the chunks are done and return. If it does getNextChunkIdx() after the set is done for a new non-sync call, it will get 0 which is fine and it will start working on the 0th chunk in the new task. Not that locks are present in set() so race condition is avoided.
Please let me know if this is not the case you were thinking about.
Thank you

Original date: 2015-05-09 16:20:10

I'm looking into the race again, but in the meantime please take out this warning message, or somehow limit it to being printed at most once per run since getting it at every timestep from 16,384 nodes is really annoying:
198     if (numChunks > MAX_CHUNKS) {
199         CkPrintf("CkLoop[%d]: WARNING! chunk is set to MAX_CHUNKS=%d\n", CmiMyPe(), MAX_CHUNKS);
200         numChunks = MAX_CHUNKS;
201     }

Original date: 2015-05-09 17:53:25

Have sent a cl to remove the warning. There was another CL by Yanhua (http://charm.cs.uiuc.edu/gerrit/#/c/417/1/src/libs/ck-libs/ckloop/CkLoop.C) which removes the warning. But I guess its better to take care of it right now in another cl.

Original date: 2015-07-09 21:17:22

Your change was http://charm.cs.uiuc.edu/gerrit/#/c/707/ right?

Original date: 2015-08-05 18:38:46

Yes.

Original date: 2015-08-05 18:47:55

OK, here is an obvious race condition for Windows:
volatile int curChunkIdx;

...
int getNextChunkIdx() {
#if defined(_WIN32)
#if CMK_SMP
CmiLock(cmiMemoryLock);
curChunkIdx=curChunkIdx+1;
CmiUnlock(cmiMemoryLock);
return curChunkIdx;
#else
curChunkIdx++;
return curChunkIdx;
#endif
#else
return __sync_add_and_fetch(&curChunkIdx, 1);
#endif
}
curChunkIdx should be copied to a local variable rather than returned directly.

Original date: 2015-08-17 17:56:50

Fix ~~https://charm.cs.illinois.edu/gerrit/#/c/767/~~ https://github.com/UIUC-PPL/charm/commit/ade3f92739213e5317a5090fd681fe80792adb5c
Is any other race conditions known at present? If not, I will close this bug.

Original date: 2015-08-21 01:32:49

No other issues reported, so closing this.


Building in linux

Building the current version of Odin in linux results in this: https://hastebin.com/qulekonudu.lua. The diff shows the changes i had to make to get it to compile and run without segfaulting.
Each error one by one:
In file included from src/main.cpp:5: src/build_settings.cpp:144:2: error: C++ requires a type specifier for all declarations         Array(char) path_buf;
Changed to
Array<char> path_buf;
Next,
In file included from src/main.cpp:9: src/ssa.cpp:10:8: error: ISO C++ forbids forward references to 'enum' types enum   ssaBlockKind;
Required me to remove the forward declarations. Finally I had to explicitly cast the const C-strings in ir.cpp.
This compiled successfully, but segfaulted on the demo. Turned out when is broken, and it loads the sys/windows.odin even in linux. Commenting out the relevant line in os.odin made it run successfully.

Please check if the latest commit fixes the problems you have.


LongCountAsync fails on Marten 7.0

Updating to version 7.0 the LongCountAsync throws an exception
[xUnit.net 00:00:17.55]     Raiqub.Expressions.Marten.Tests.Queries.MartenDbQueryTest.CountShouldReturnExpected(name: "Fourth", expected: 0) [FAIL]
[xUnit.net 00:00:17.55]       System.NotSupportedException : Marten does not know how to use result type long
[xUnit.net 00:00:17.55]       Stack Trace:
[xUnit.net 00:00:17.56]            at Marten.Linq.SqlGeneration.DataSelectClause`1.BuildHandler[TResult](IMartenSession session, ISqlFragment statement, ISqlFragment currentStatement)
[xUnit.net 00:00:17.56]            at Marten.Linq.Parsing.LinqQueryParser.buildHandlerForCurrentStatement[TResult](Statement top, SelectorStatement selector)
[xUnit.net 00:00:17.56]            at Marten.Linq.Parsing.LinqQueryParser.BuildHandler[TResult]()
[xUnit.net 00:00:17.56]            at Marten.Linq.MartenLinqQueryProvider.ExecuteAsync[TResult](Expression expression, CancellationToken token, SingleValueMode valueMode)
[xUnit.net 00:00:17.56]            at JasperFx.Core.Exceptions.ExceptionTransformExtensions.TransformAndThrow(IEnumerable`1 transforms, Exception ex)
[xUnit.net 00:00:17.56]            at JasperFx.Core.Exceptions.ExceptionTransforms.TransformAndThrow(Exception ex)
[xUnit.net 00:00:17.56]            at Marten.Exceptions.MartenExceptionTransformer.WrapAndThrow(Exception exception)
[xUnit.net 00:00:17.56]            at Marten.Linq.MartenLinqQueryProvider.ExecuteAsync[TResult](Expression expression, CancellationToken token, SingleValueMode valueMode)

The test runs the following code:
var query = source
        .Where(b => b.Name == name)
        .SelectMany(b => b.Posts);

int count1 = await query.CountAsync();
long count2 = await query.LongCountAsync();

@skarllot We've got plenty of passing tests against the LongCountAsync() operator by itself. Do you have a more specific reproduction case?

Nevermind, I see the combination with SelectMany()

Started to think it was a wide ranging issue, but turned out to be just LongCount. There was a switch statement that looked for Any() or Count(), but not LongCount().


[red-knot] simplify across intersections in a union

When we add multiple intersection types to a union, we miss some opportunities to simplify across those intersections.
If two intersections in a union share the same positive elements, and they have negative elements that are disjoint from each other, the disjoint negative elements can be removed entirely.
Assume that X and Y are disjoint types:
We can transform (A & ~X) | (A & ~Y) to A.
Similarly we can transform (A & B & ~X) | (A & B & ~Y) to A & B.
And if we have (A & ~X & ~Z) | (A & ~Y & ~Z), this can become A & ~Z.
In some cases we can eliminate them and it doesn't result in simplifying the union away. For example (assuming T and U are not disjoint from each other) (A & ~X & ~T) | (A & ~Y & ~U) can become (A & ~T) | (A & ~U).
Here's a test:
def f(x: int, y: int, flag: bool):
    if x != 1 and y != 2:
        reveal_type(x)  # revealed: int & ~Literal[1]
        reveal_type(y)  # revealed: int & ~Literal[2]
        z = x if flag else y
        reveal_type(z)  # revealed: int

Currently on the last line we reveal int & ~Literal[1] | int & ~Literal[2], when it should just be int.

add some parentheses

Done.

In some cases we can eliminate them and it doesn't result in simplifying the union away, e.g. (A & ~X & ~T) | (A & ~Y & ~U) can become (A & ~T) | (A & ~U).

Just to clarify: in this example X/Y are disjoint (so can be eliminated), but T/U are not disjoint (so cannot be eliminated)?

Just to clarify: in this example X/Y are disjoint (so can be eliminated), but T/U are not disjoint (so cannot be eliminated)?

Right, all of the examples are operating under the "Assume that X and Y are disjoint types:" header (with no other assumed disjointness).

Yes, I saw that your writeup explicitly stated that X and Y should be assumed to be disjoint, but didn't see anything that explicitly stated whether T and U were supposed to also be disjoint from each other or not :-)
just wanted to clarify that my understanding was correct for why X/Y were being treated differently to T/U in that example

Can we describe this in terms of boolean logic? Could we reuse the SMT solver we have in uv for this?

It's possible that all of our set-theoretic types could be represented as BDDs and we could use SMT to resolve subtyping, though I haven't worked through how that would look; the approaches I've seen rely on disjoint atomic types, which is an assumption we can't make because of inheritance. But this would be a whole-sale reworking of how we represent set-theoretic types, whereas the improvement suggested in this issue is more narrowly targeted.


Functions in Kotlin: Android Basics with Compose

URL of codelab
The issue can be found in the last line of the first paragraph of 5. Functions With Multiple Parameters
**It's written here that "In this section, you'll modify the greeting so that it also changes based on the dog's age."
Since, we're talking about a person in reference to this function. The "dog's" should be replaced by "the person's"
**
Screenshot included

Hello @Mr-Starkk,
Thank you for reaching out to us.
This sentence is correct as it primarily refers to dogs in the codelab: Rover and Rex.


A few BUILD.md tweaks

As I was going through the setup steps to be able to run VegaFusion tests, I had to run a few more commands in order to get things to work. Not sure that all the changes I've made to the steps are completely accurate/necessary, but I believe they should at least be a net improvement. Feel free to make edits to this PR as desired!

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for working through it and recording these updates!


Underlying system changes

Partly replaces #30
View Change List

Chroot-based build environment is removed (https://github.com/joshiemoore/snakeware/pull/30#issuecomment-639876668)

Unusual configurations can use the contrib/nix/default.nix file with the Nix package manager's nix-shell feature. (3748658)

Completely optional

Commits are organized in a better way

Everything after commit 4d65d60 should be easy to add or remove without affecting more than necessary.

Uses Buildroot's external configuration support (7b94a48)

No more copying configuration files around before build, all done automatically

Build hybrid ISOs instead of IMG files (48ac4ae)

Easier to write to USB/CD or boot in a virtual environment
To make this happen, Grub has been replaced with ISOLinux

Buildroot packages for pyttsx3 and pygame_gui (598542e)

Enable ccache by default (30fa88a)

Cache compiler outputs for faster clean builds
Maximum cache size set to 20 GB

Optimize target packages with -O3 (52db9ca)

Includes the kernel, which should work, but more testing can be nice

Optimize Python (980808a)

Applies PGO and LTO to the built Python binary.
Takes a long time to build, so you might want to disable it when doing development.

Buildroot menuconfig: External configuration -> Enable Optimizations for Python

Add configure.sh (484d228)

For easier editing of various component configurations

Also optional

Try to slim down Busybox (1f7b04f) & Buildroot (c3e6a6f, 8567878)

I've only done very little testing with these. Might break some stuff, let me know if it does.

Enable module autoload on boot (8c4dcf4)

Enable GPU modules (8c4dcf4) and their firmware (cf9bad1)

GRUB set up the framebuffer before, but we use ISOLinux now, so we have to do this manually

Add QEMU script (fad2ceb)

Just an example

I am aware this PR might be too big, and contain changes that might be unnecessary. If so, please let me know and I'll remove the requested changes.

Tested and working on my end, should be good to merge

Hi, the whole process worked great, except for the last step where it's trying to build host-cdrkit to generate the ISO, I'm getting a bunch of linker errors:
/usr/bin/ld: CMakeFiles/genisoimage.dir/apple.o:(.bss+0x8): multiple definition of `outfile'; CMakeFiles/genisoimage.dir/genisoimage.o:(.bss+0x4d8): first defined here
/usr/bin/ld: CMakeFiles/genisoimage.dir/boot.o:(.bss+0x0): multiple definition of `outfile'; CMakeFiles/genisoimage.dir/genisoimage.o:(.bss+0x4d8): first defined here
/usr/bin/ld: CMakeFiles/genisoimage.dir/desktop.o:(.bss+0x0): multiple definition of `outfile'; CMakeFiles/genisoimage.dir/genisoimage.o:(.bss+0x4d8): first defined here

...

/usr/bin/ld: CMakeFiles/genisoimage.dir/rsync.o:(.bss+0x0): multiple definition of `outfile'; CMakeFiles/genisoimage.dir/genisoimage.o:(.bss+0x4d8): first defined here
/usr/bin/ld: CMakeFiles/genisoimage.dir/boot-mipsel.o:(.bss+0x0): multiple definition of `outfile'; CMakeFiles/genisoimage.dir/genisoimage.o:(.bss+0x4d8): first defined here
/usr/bin/ld: CMakeFiles/genisoimage.dir/checksum.o:(.bss+0x0): multiple definition of `outfile'; CMakeFiles/genisoimage.dir/genisoimage.o:(.bss+0x4d8): first defined here

I redacted many errors for brevity. Do you have any ideas what would be causing this?
Also, I believe the usage message for configure.sh is backwards. It says ./configure.sh <platform> <component> but what is actually expects is <component> <platform>.
This is really great though, thank you for your help!!

On Wed, 10 Jun 2020 16:35:18 -0700
Josh Moore<EMAIL_ADDRESS>wrote:

Hi, the whole process worked great, except for the last step where
it's trying to build host-cdrkit to generate the ISO, I'm getting a
bunch of linker errors:
[snip]
I redacted many errors for brevity. Do you have any ideas what would
be causing this?

Seems to be a GCC 10 issue, maybe?
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=957074

Also, I believe the usage message for configure.sh is backwards. It
says ./configure.sh <platform> <component> but what is actually
expects is <component> <platform>.

Sorry, I guess I forgot to switch those around :p

I downgraded to GCC 9.3 and this is really superior! isolinux boots way faster than GRUB, and only a 70MB image! Really really fantastic, thank you so much for this. These changes you made are going to help us so much.


Add polygon annotation layer to GeoJS map for new GeoJSON endpoint

Following #182, add a new annotation layer to the GeoJS map that can handle drawing single polygons and return GeoJSON of that polygon for the new search endpoint.
As a part of this effort, it may be cool to enable a user to drag 'n' drop a shapefile (or textfile containing GeoJSON) to populate the form

Additionally, once we have user permissions implemented, we could have a way for users to choose from a set of the GeometryEntrys they have access to for these searches. e.g., they could save all their favorite areas of interest as geometry entries and periodically do searches in those regions.


feat: PIZ2(part1) Add beverage management views

Summary
New pages were added to the web app, to create, list and update a beverage. Additionally, a bug found in update ingredients script was fixed.

Checklist

[x] Beverages are shown on the web app to be listed, created and updated
[ ] A user can select multiple beverages (from the order creation view)

Linked issues
PIZ2 trello

 PIZ2 - Add a beverage model, manager and service


Execute Workflow by ID - how to pass Record Id

Expected behavior and actual behavior.
Steps to reproduce the problem (screenshots are really usefull)
I added a lookup Workflow Lookup to a custom Case Category entity pointing to workflow
I added lookup to Case Category in the Case entity
I created a synchronous on-demand workflow WFCC that creates a Task under the Case
I assigned workflow WFCC to Case Category X
I created a synchronous on-create workflow WFCase so that if Case's Case Category has a workflow assigned in its Workflow Lookup field, use that workflow on the Case.
So in ExecuteWorkflowby ID command:
Record Id = {Case(Case)} (which should by the case's Guid)
Process = {Workflow Lookup(Case Category (Case Category))}
I expect that for a Case pointing to Case Category X, it will create a Task under the Case on save.
Instead, I get
Unexpected exception from plug-in (Execute): msdyncrmWorkflowTools.Class.ExecuteWorkflowByID: System.FormatException: Guid should contain 32 digits with 4 dashes (xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx).
The goal was to have certain Case Categories have certain automations without having a workflow with a series of If...Else If...Else If testing each possible Case Category that has a workflow associated to it.
Thanks in advance!
CRM Version - online/onpremise - WorkflowTools version
CRM: 8.2.2 onpremise - WorkflowTools: 1_0_50_1_8.2

Hi Lafil,
Yo need to retrieve the GUID as text, if you pass this value "{Case(Case)}" you don't retrieve the GUID.
I will create a new action to pass the recordURL, and to get the record typename and GUID as an output.
this will enable to to retrieve this GUID as text.
Hope to have it during this weekend.
thanks!

Hi Demian!
I am not sure this is best approach.
If you create separate action to retrieve Guid from record, will I not have to have a field to store it in in order to invoke it in following Execute Workflow by Id action?
Perhaps best to have Execute Workflow recognize when we are passing entity primary key in Record Id and retrieve that Guid to apply workflow to?
Sorry typing this from phone without checking in CRM but I want to bring this to your attention before you spend time coding something :-)
Best regards,
Louis
On Feb 24, 2018 6:00 AM, Demian Adolfo Raschkovan<EMAIL_ADDRESS>wrote:
Hi Lafil,
Yo need to retrieve the GUID as text, if you pass this value "{Case(Case)}" you don't retrieve the GUID.
I will create a new action to pass the recordURL, and to get the record typename and GUID as an output.
this will enable to to retrieve this GUID as text.
Hope to have it during this weekend.
thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/demianrasko/Dynamics-365-Workflow-Tools/issues/62#issuecomment-368220207, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXIniWjAGppyqQP7Xts_D1lWyiBgrYoKks5tX-vfgaJpZM4SP8pm.

Thanks,
You don't  need to store the string with the Id nowhere,  because you can use the output parameter of the action.
No problem to code it,  because this action is useful to other activities too.
Thanks!

Hi Demian!
Got the updated solution this morning and amended my workflow to use the new Get Record Id action.
Works great!
Thanks for looking into this. I now have the potential to replace a heavy workflow with about a dozen “If… Else If” steps into a straightforward “Run the workflow associated to the category if there is one”.
Best,
Louis-Alain
De : Demian Adolfo Raschkovan<EMAIL_ADDRESS>Envoyé : 24 février 2018 10:59
À : demianrasko/Dynamics-365-Workflow-Tools<EMAIL_ADDRESS>Cc : Lafil<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Objet : Re: [demianrasko/Dynamics-365-Workflow-Tools] Execute Workflow by ID - how to pass Record Id (#62)
Thanks,
You don't need to store the string with the Id nowhere, because you can use the output parameter of the action.
No problem to code it, because this action is useful to other activities too.
Thanks!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/demianrasko/Dynamics-365-Workflow-Tools/issues/62#issuecomment-368238043, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AXIniepwiSRvu_gE4gGA1RAyFZwmouWqks5tYDG5gaJpZM4SP8pm.

Thanks Louis-Alain!!


chore: adds py35 environment to tox.ini

This patch adds Python 3.5 to the list of environments in tox.ini.

[Current coverage][cc-pull] is 98.42%

Merging [#804][cc-pull] into [master][cc-base-branch] will decrease coverage by 1.57%

@@             master       #804   diff @@
==========================================
  Files            29         29
  Lines          1777       1777
  Methods           0          0
  Messages          0          0
  Branches        299        287    -12
==========================================
- Hits           1777       1749    -28
- Misses            0         22    +22
- Partials          0          6     +6

Powered by Codecov. Last updated by [53b198c...180710d][cc-compare]
[cc-base-branch]: https://codecov.io/gh/falconry/falcon/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/falconry/falcon/compare/53b198c6d83cb04f051424e9cc33b593c52e65d7...180710d7cf9a53416ba470df9970a783b24f06ce
[cc-pull]: https://codecov.io/gh/falconry/falcon/pull/804?src=pr

:+1:


Extract dropdown

[x] Update docs about configuring scripts
[ ] Dropdown panel width
[ ] Discuss filter popover close button removal (translations are still there for now)
[ ] Dropdown border opacity

Hello @zepfietje, great work!!.
I have a suggestion related to the drop-down functionality. It is somewhat simple and may not need me to open an issue for it.
The Table ActionGroup drop-down currently uses a hard-coded value of 'w-52' which causes a truncation of items in the drop-down eg:

If you could allow customization of the width for the action group table component similar to how modals allow customizing the width, it would fix the issue.
eg:
for current implementation for Modals

proposed usage of width customization for ActionGroup:

Thanks for your detailed explanation, @jojostx!
I will have a look at what works best for the dropdown size and take your remarks into account.

As per #2236, we should replace the dropdown border with a ring

@danharrin, keeping the border but modifying the border color opacity would yield the same result, right?
I've added it to the list of tasks to be done and will have a look at what works best. 👌

Merge conflicts

Merge conflicts

Resolved!

You will need to revert all asset changes before I can merge this

Yeah, so want to keep Alpine Floating UI shipped with the forms package? And what about the others, should they have docs to manually add it, since it wasn't there now?

Floating UI is used by the forms package
Tables and admin require the forms package already
So no extra docs needed, just keep this PR as a refactor to a common dropdown Blade component.

I've got a separate PR to add action groups support to the notifications package. So how should that be handled then?

You will need to revert all asset changes before I can merge this

Resolved, @danharrin!


Venu 3 - application doesn't start

Hello,
The application doesn't start in my Venu 3.
I have hime assistant running in docker in my NAS on my lan.
I have set up an https access using let's encrypt.
In my home router I associated the domain used for the certificate to the local ip of the home assistant.
I configured home assistant to expos the rest api.
I created a never expiring access token.
I can access the api url correctly from one pc of the same lan.
--> Bug: in the application on my watch I see :
API: not available
Menu: not available
If I click on it I see the error message :
"No answer, check your internet connection"
Thanks in advance for any advice

For my Venu 3s it works flawless

Maybe it is because I am on my private lan
No idea...
I would like to see the logs of the app but I dont know how

I'm only using Home Assistant and the app on my local network.
I'm sure you already checked, that your Garmin is connected to your phone and the phone is in the same network as your Home Assistant server (I suppose you're using the HA companion app on this phone and it works).
I forgot on my first try to add '/api' to the config url. Perhaps you could again check your URLs and the API token?

Yes they are all in the same network.
I didn't use the the HA companion app but I will try. Normally I control l every thing in HA via browser on a PC on the same network.
I put the right url ("/api") in the config.
I also tested via pc the api is reacheable with long life access token and it is.
I will try to test HA companion app on my phone.
So the watch access HA via paired phone?
I thought directly via wifi network...
Anyway thansk a lot for your hints and time to write the message.

Yes, it seems to work via phone. When I disable Bluetooth or WiFi on my phone, it stops working. However, for my use case, this is sufficient.Am 16.03.2024 um 14:22 schrieb silcrox @.***>:
Yes they are all in the same network.
I didn't use the the HA companion app but I will try. Normally I control l every thing in HA via browser on a PC on the same network.
I put the right url ("/api") in the config.
I also tested via pc the api is reacheable with long life access token and it is.
I will try to test HA companion app on my phone.
So the watch access HA via paired phone?
I thought directly via wifi network...
Anyway thansk a lot for your hints and time to write the message.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

In fact I installed HA app already, I forgot ..
And it works well, with phone connected on the same wifi network.
No idea of what's wrong.
I will review the json schema and configuration file. It is a pity not being able to see logs

I tested wit the troubleshooting page and everything seems to work there, but still not on my watch :-(
https://house-of-abbey.github.io/GarminHomeAssistant/web/

Thank you for using the troubleshooting page, that filters out much of the silliness. So if the page at https://house-of-abbey.github.io/GarminHomeAssistant/web/ works, then essentially it works. So you just need to figure out what's up with your watch.

So the watch access HA via paired phone?
I thought directly via wifi network...

The watch connects via BlueTooth to your phone, then by Wi-Fi/mobile access to your HA server. We've had people claiming their watch is Wi-Fi capable before (my Venu2 is), but they are mistaken. If the watch was on Wi-Fi all the time, the battery would be dead(er). So you must have your phone in Bluetooth reach, and the watch must be paired and sync'ing via the ConnectIQ app.
Are you using an iPhone by any chance? If so see the known issues and then have a moan at Apple!
https://github.com/house-of-abbey/GarminHomeAssistant/blob/main/README.md#known-issues

It is a pity not being able to see logs

You get what HA gives you for logging.
Good luck! And thanks @bjobo for fielding the replies in the meantime.

This will be a "bug" when I confirm it is.

Thanks a lot Philip for your reply.
And also to have created that troubleshooting page, I was happy my self to see that my parameters were configured well.
Happy to know also that the wifi connection from the phone is not the guilty one, as it connects via Bluetooth to the paired phone and then the phone "talk" with the ha server.
I am using a samsung s21, and yes my phone is always at reach of the watch and connected to the the wifi network where ha is running (I am using a docker image of ha running on my nas).
I will see logs on ha to see if at least it receives http requests from the phone when I run the app on the watch.
Thansk again for your work and for the hints.

If the HA app works on your phone, and your watch sync's via your phone, and that web-based editor worked, then there is little else that can be wrong, other than your app settings. Check for rogue spaces in you settings.

--> Bug: in the application on my watch I see :
API: not available
Menu: not available

URLs are not correct or no Internet access. I know you said you copied & pasted...
Do you get an error message when you start the main app?

Have you made any further progress with resolving this issue?

No sorry.
I didnt have time to investigate and it will be the case also for the next 2 weeks.... :-(
I have the feeling that no http request is triggered from my phone where garmin connect is running towards HA...

In my home network the ha host (associated to the lets erncrypt certificate) is translated correctly to the local ip. In fact if i put the same url I put in your app in chrome running on the same phone where garmin connect is installed, I can reach both the api and the menu.
Now I have the doubt that garmin connect doesnt use my home network dns, but an external one.
If that is true than I have to open the ha port (the hostname is translated to my external gateway, but there all traffic is blocked).
I will try this evening. If not I have to see logs, both in my router and in ha server.
Thanks again

Just did a test, but no luck again.
I opened the port on my router and redirected to the ha instance, I can correctly access my ha server from internet (of course in https). The I reinstalled and configured the app but I have the same behavior: If I open the app on the watch I get the messages:
API: not available
Menù: not available

No answer: check your internet connection

Please forgive the silly questions.

Maybe it is because I am on my private lan

How are you reaching our from the watch to your HA on the private LAN? Are you using a private domain name with local DNS or a private LAN IP address?
I would expect nslookup and ping to reach your HA. Then that would be the address to use for your API.
But if we work the menu URL first, as that is easier to fix. Where have you placed your JSON file? Local again? If you can curl it from the command line then the URL is reachable. Or you can put the URL in your browser and test you can reach it. Essentially if it works in the browser, then you have something very peculiar going on with your watch-phone combination. So use various browsers on different devices, desktop PC and phone to check your menu URL works.

No silly questions in IT stuff, every check or doubt can bring to the solution :-)
I would love to have the logs of the app it self, but I will start from having a look if the request comes to my router. Considering how quick the app on the watch tells me "No internet connection" I have the strong doubt that the request is not even triggered by the phone....
thanks for your time anyway I appreciated it very much.

I would love to have the logs of the app

The app makes no logs. For the logs to be useful, they woudl flood the SD RAM. But it does give you the error messages. Not sure the logs would be able to provide any more information even if theoretically they could be made.

Just had a thought as I was pondering this issue.
You are running on a private LAN with Let's Encrypt for HTTPS. That means the Garmin Watch can't verify your domain name locally for the certificate, whilst the in line editor can because it is public.
You need to set up a local LAN DNS resolution for your domain name in your certificate so the watch will trust your HA. E.g. a host file entry.

Hi Philip,
I already had the host-->privateIp mapping in my router.
That's how I could use the hostname for which I created the certificate
letencrypt even in my local network.
I also tried to open/redirect the HA port on the public IP. I could indeed
access my HA from internet but still the app shew me the same error
message: "no answer, check your internet connection"
Il Gio 21 Mar 2024, 19:47 Philip Abbey @.***> ha
scritto:

Just had a thought as I was pondering this issue.
You are running on a private LAN with Let's Encrypt for HTTPS. That means
the Garmin Watch can't verify your domain name locally for the certificate,
whilst the in line editor can because it is public.
You need to set up a local LAN DNS resolution for your domain name in your
certificate so the watch will trust your HA. E.g. a host file entry.
—
Reply to this email directly, view it on GitHub
https://github.com/house-of-abbey/GarminHomeAssistant/issues/121#issuecomment-2013296529,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATOJVH3QK5W75SNYMIAXHC3YZMTKPAVCNFSM6AAAAABEYWM3J2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJTGI4TMNJSHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

This may also be caused by the private DNS setting on your phone, this setting forces your phone to use a trusted DNS server, so will prevent it finding your DNS record.
This can be turned off in the more connectivity options page of your phone's settings, or search for "Private DNS"

That is a very good idea. Let me try.

In fact I already had the host-->privateIp mapping in my router.

Yeah, I don't think that covers the function of DNS. Surely that's about serving IP addresses under DHCP?
I'm not yet convinced that is sufficient. I'm looking at this per device. Does your router server DNS to your LAN. Does each device on your LAN have the router as the DNS server? I think your phone needs to be able to resolve the DNS of your Let's Encrypt certificate. I bet you didn't create that certificate on your phone, so you have not convinced me the phone can resolve local DNS. You created that certificate on a host that could resolve the domain name. The key thing here is your phone must be able to complete the lookup. Perhaps you've already verified the certificate in your phone's browser, in which case you should be fine?
We solved a similar problem locally when I cut the Internet coaxial cable with the garden sheers. (I was unpopular for that.) In order to make HA still work, we needed to resolve the external name home.something.something.uk within the LAN. So we added a lookup for that in dnsmasq on the Raspberry Pi so we could resolve the same domain name without leaving the LAN (owing to being an Internet island at the time). That saved us switching various settings between the public domain name (served via a dynamic DNS service) and homeassistant.local.

This may also be caused by the private DNS setting on your phone, this setting forces your phone to use a trusted DNS server, so will prevent it finding your DNS record.
This can be turned off in the more connectivity options page of your phone's settings, or search for "Private DNS".

Goole most unhelpful page on the topic:
https://support.google.com/fi/answer/2819583?hl=en#zippy=%2Cprivate-dns

This may also be caused by the private DNS setting on your phone, this setting forces your phone to use a trusted DNS server, so will prevent it finding your DNS record.
This can be turned off in the more connectivity options page of your phone's settings, or search for "Private DNS".

I tried but no change

In fact I already had the host-->privateIp mapping in my router.

Yeah, I don't think that covers the function of DNS. Surely that's about serving IP addresses under DHCP?
I'm not yet convinced that is sufficient. I'm looking at this per device. Does your router server DNS to your LAN. Does each device on your LAN have the router as the DNS server? I think your phone needs to be able to resolve the DNS of your Let's Encrypt certificate. I bet you didn't create that certificate on your phone, so you have not convinced me the phone can resolve local DNS. You created that certificate on a host that could resolve the domain name. The key thing here is your phone must be able to complete the lookup. Perhaps you've already verified the certificate in your phone's browser, in which case you should be fine?
We solved a similar problem locally when I cut the Internet coaxial cable with the garden sheers. (I was unpopular for that.) In order to make HA still work, we needed to resolve the external name home.something.something.uk within the LAN. So we added a lookup for that in dnsmasq on the Raspberry Pi so we could resolve the same domain name without leaving the LAN (owing to being an Internet island at the time). That saved us switching various settings between the public domain name (served via a dynamic DNS service) and homeassistant.local.

I used a dynamic dns service to make public
I used an utility in the admin configuration console of my NAS (Qnap) to trigger the creation of the certificate with letencrypt and
Finally, as I dont want to expose my private network to internet, I created the binding of  to the local ip address of the NAS (where HA server runs).
I used your troubleshooting guide on my browser in the phone, and it works (with https), see screenshot in one of the initial messages.

About the DNS used by the devices connected to the lan I need to check but I think they use the router (that at its turn uses some public DNS).
PS I found an option in my router (openwrt) to log each DNS request ... I will check if requests for  arrive.

Logs of my router when I launch the GarminHA on my watch:
Thu Mar 21 22:41:33 2024 daemon.info dnsmasq[1]: 529 ipv6_of_my_phone/43740 query[AAAA] myhostname from ipv6_of_my_phone
Thu Mar 21 22:41:33 2024 daemon.info dnsmasq[1]: ipv6_of_my_phone/43740 forwarded myhostname to ipv4_of_my_gateway
Thu Mar 21 22:41:33 2024 daemon.info dnsmasq[1]: 529 ipv6_of_my_phone/43740 forwarded myhostname  to ipv6_of_my_gateway
Thu Mar 21 22:41:33 2024 daemon.info dnsmasq[1]: 530 ipv6_of_my_phone/20570 query[A] myhostname from ipv6_of_my_phone
Thu Mar 21 22:41:33 2024 daemon.info dnsmasq[1]: 530 ipv6_of_my_phone/20570 /tmp/hosts/dhcp.cfg01411c myhostname is ha_local_ip
Thu Mar 21 22:41:33 2024 daemon.info dnsmasq[1]: 529 ipv6_of_my_phone/43740 reply myhostname is NODATA-IPv6
It looks like 2 dns queries arrive: the first one is forwarded to the gateway DND, the second one is answered locally to the ha local ip

Silly question coming up...
How do this DNS queries compare with the domain names used in your Let's Encrypt certificate? Including any alternate names you might have used.
ConnectIQ API is strict about the HTTPS. The API calls fail, and provide little reason, just failed. ConnectIQ needs to verify the host declared in the certificate, and resolve to the correct IP address, which is different when checked publically to privately. If the checks fail, no HTTP.
If you are already getting the JSON menu into your phone's browser, then there's nothing else that can go wrong apart from the application settings... so we've been here before.
Try a DNS change app on your phone? Control where request are getting sent.

I confirm the the troubleshooting page runnoing on the browser of the same phone running garmin connect works well.
And I confirm that the exact same configuration is set in the garmin home assistant app.
I gort the DNS logs in my router after triggering 3 requests:

opening the garmin ha app
testing menu in troubleshooting page
testing api in troubleshooting page

the logs after 1) miss the http requests that are triggered just after in case of 2) and 3)
it has somethign to do with the certificate and https validation and the DNS.
I will check in coming weeks after coming back home. Because tomorrow I will leave for one week for my work.
once again thank you Philip and everybody else who tried to help me.
PS: feel free to close the bug if you want/need.

Closed as suggested by @silcrox.


Fix for Failed to find data adapter error

When attempting to run model.fit or model.fit_generator such as this -
model.fit_generator(generator,epochs=10)
Following error is thrown -

ValueError: Failed to find data adapter that can handle input: <class 'time_series_generator.time_series_generator.TimeseriesGenerator'>, <class 'NoneType'>

Fixed by changing time_series_generator.py. Added the line -
from keras.utils import data_utils
and modifying the line -
class TimeseriesGenerator(object):
to
class TimeseriesGenerator(data_utils.Sequence):
Packages used in testing -
Keras - 2.9.0
Sklearn - 1.0.2
Tensorflow - 2.9.2
Numpy -1.21.6
Pandas - 1.3.5
Co-Authored-By: Kejid<EMAIL_ADDRESS>

Addressing the issue here -
https://github.com/krypton-unite/time_series_generator/issues/1#issue-887058105


GH branch-regex doesn't seem to work as expected with leading characters

Describe the bug
As discussed in https://github.com/dockstore/dockstore-documentation/pull/229, we use this regex string when parsing branches from GitHub via the Dockstore GHA: ([a-zA-Z0-9]+([./_-]?[a-zA-Z0-9]+)*)$
My grasp of regular expressions is shaky at best, but something about it looked a little odd. It seems that it's intended to disallow leading underscores, periods, forward slashes, and dashes whilst still allowing them elsewhere. Playing around with regxr.com indicates that it might actually be matching everything after a leading [./_-] -- for example, it seems that a branch named foo_bar will return foo_bar as expected, but a branch named _foobar will return foobar which likely isn't the name of any actual branch.
I could very well be barking up the wrong tree here, but the logs are giving me reason to suspect something isn't being handled the same way as most disallowed branch names are. I suspect every workflow in the repo might be trying to look up a branch that doesn't exist.
To Reproduce
Steps to reproduce the behavior:

Give an existing GHA repo a new branch named _leadingunderscore
Make a cupán tae
Check Dockstore and notice the "repo last modified" hasn't changed
Check the GitHub App logs for the relevant organization and notice there is one error per workflow in the .dockstore.yml rather than one error for the whole repo

Expected behavior
Either:

Branches with leading periods, dashes, underscores, and forward slashes should work
Branches with leading periods, dashes, underscores, and forward slashes should be blocked, but should throw the same error as any other branch with special characters

Screenshots
Leading underscore log:

Compare that to most other branches with special characters:

Additional context
Webservice - 1.13.0
UI - 2.10.0
Compose Setup - 1.13.0
Deploy - 1.12.6
galaxyParsingLambdaVersion: 0.0.7
┆Issue is synchronized with this Jira Story
┆Fix Versions: Dockstore 1.14
┆Issue Number: DOCK-2252
┆Sprint: 106 - Don River
┆Issue Type: Story

Good find. Also, gross.

leading periods, dashes, underscores, and forward slashes

Some of these should work barring restrictions in https://git-scm.com/docs/git-check-ref-format


Consider updating buildkit

First, I am not sure if Kim is still maintained, but it sure is still super useful to build images for k3s.
kim fails to build this simple Dockerfile with the default buildkit
FROM python:slim
RUN apt-get update

With this error:
#4 [2/2] RUN apt-get  update
#4 sha256:3396cd45c9a54b00133aaeef88c090d0dd0e11ffb14d55ab1d6f924a29b4e594
#4 0.134 Get:1 http://deb.debian.org/debian bookworm InRelease [151 kB]
#4 0.147 Get:2 http://deb.debian.org/debian bookworm-updates InRelease [52.1 kB]
#4 0.149 Get:3 http://deb.debian.org/debian-security bookworm-security InRelease [48.0 kB]
#4 0.201 Get:4 http://deb.debian.org/debian bookworm/main amd64 Packages [8780 kB]
#4 0.253 Get:5 http://deb.debian.org/debian bookworm-updates/main amd64 Packages [6408 B]
#4 0.283 Get:6 http://deb.debian.org/debian-security bookworm-security/main amd64 Packages [79.7 kB]
#4 1.139 Fetched 9117 kB in 1s (8907 kB/s)
#4 1.139 Reading package lists...
#4 1.603 E: Problem executing scripts APT::Update::Post-Invoke 'rm -f /var/cache/apt/archives/*.deb /var/cache/apt/archives/partial/*.deb /var/cache/apt/*.bin || true'
#4 1.603 E: Sub-process returned an error code
#4 ERROR: executor failed running [/bin/sh -c apt-get  update]: exit code: 100
------
 > [2/2] RUN apt-get  update:
------
Error: failed to solve: rpc error: code = Unknown desc = executor failed running [/bin/sh -c apt-get  update]: exit code: 100

Googling suggest it could be an incompatibility issue with Docker version and the recent version Debian bookworm.
I can work around it by using a newer buildkit version, but it would be nice to update the defaults:
kim builder install --buildkit-image docker.io/moby/buildkit:v0.12.2

I think kim got bit by the runc capabilities mismatch that was giving permission errors running perfectly legit containers on glibc hosts a couple years back. I left employment at SUSE for over a year but sorta came back recently-ish (I work for RGS aka Rancher Federal now) and am seeking to refresh this project.


chore: exempt dummy module from integration check workflow

Issue #, if available:

Closes #230

Changes

Exempts module blank/dummy from integration tests check workflow

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: IDFModuleTestsInfrastructur-59JoFoVH64CI
Commit ID: a7c5b971249acedde2b402c7630194e1d9a58a38
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Cannot get cyrillic text from inputData

{
    "name": "some_workflowName",
    "version": 1,
    "input": {
        "push":{
            "subject":"C**n**s**g-c ёдаааааааааааааааааа",
            "body":{
                "amount":100000,
            },
            "categoryKey":"transaction",
            "categoryType":"income"
        }
    }
}

So I am providing my workflow with the data above, but my python worker cannot comprehend the cyrillic text "ёдаааааааааааааааааа" and replacing it with "?".
I have already tried encoding the task input data but it is still showing as ? both in the UI and on my CLI.
Is this a python issue or a conductor issue?

@luffy1727 what's your backing datastore? Mysql? Postgres? Something else?

Currently MySQL

@luffy1727 what's your backing datastore? Mysql? Postgres? Something else?

@luffy1727 I think you'll need to ensure 2 things. By default, mysql can be using something like latin-1 for its characterset.
You can change the character set of existing DBs and tables like this
alter database conductor character set = utf8mb4;
ALTER TABLE event_execution CONVERT TO CHARACTER SET utf8mb4;
ALTER TABLE meta_event_handler CONVERT TO CHARACTER SET utf8mb4;
......... <do all tables>

Also, you need to make sure that your jdbc connection has &characterEncoding=UTF-8 as a parameter for its connection.
This won't fix ones already in the DB, but it should fix new ones.

Thank you, it worked


feat: add tests to cover add exchange usecase

for external exchanges, there is a button that must be clicked to edit target information / ID schemes

Looks good!
I added one more overaching comment on the setup for the editItem page.
There's a lot of additional tests that we could do here as well. Not sure if it would help defining some as part of this work even if we don't want to implement them at the moment? Some ideas that come to mind:
addItem
[x] save does not execute and validation error appears if name is missing

[x] save does not execute and validation error appears if authentication details are missing (external exchange)

[x] gives modal warning about unsaved changes if you click cancel after making changes in the form [+ gives no warning modal if no unsaved changes, abandons changes if you click cancel from modal]

[x] gives validation warnings when creating a request

[x] gives warning modal about unsaved changes if click cancel after editing request form

[] parses the ID schemes for attributes correctly when saved (i.e. you end up with ATTRIBUTE:{attribute_id} for the id scheme)

[] parses UI inputs to exchange format on save (e.g. test that given certain UI inputs, the useDataMutation is called with expected exchange format)

editItem
[] parses and correctly displays ID Scheme choices (ID/Code/Attribute)

[x] for external exchanges, there is a button that must be clicked to edit target information / ID schemes

[x] for external exchanges, clicking on edit target information / edit ID schemes enables edit of both target information / ID schemes

* for external exchanges, validation warnings appear and prevent save from being called when ID schemes are edited

Hey @tomzemp , i've added the tests for some of the above, let's talk about the others soon and see if maybe we can test those together?


Introduces OutputBlock and metrics fixes/improvements for multi-task learning

Goals :soccer:
This PR introduces a OutputBlock which is the equivalent to the InputBlock. It loops over all columns in the schema and based on the tags it adds the right ModelOutput. It is a replacement (V2) of the previous block for prediction based PredictionTask (V1).
It also contains related tasks:

[x] Closes #914

Refactored metrics creation to:

[x] share code for both metrics and weighted_metrics (BaseModel.create_metrics())
[x] accept string metrics (parsed by metrics_registry)
[x] standardize metrics when using OutputBlock (V2) and PredictionTask (v1) when using multi-task learning
[x] fix behavior that was adding "_1" to metrics when default task metrics were used (instead of manually defined ones) wiht multi-task learning
[x] fix a bug that allowed a mismatch between tasks and metrics when a dict of PredictionTask or ModelOutput was provided with keys in an order not sorted alphabetically (which is what Keras expects)

Implementation Details :construction:
If there’s only one target column, OutputBlock will output a single ModelOutput, otherwise multiple outputs will be wrapped in

I have added a bunch of fixes and improvements to this PR related to multi-task learning support for models and metrics. They are listed in the PR description. @marcromeyn could you review on my changes?


WGBS:bam_statistics always fails with Docker profile

This part of the pipeline always fails, for all samples. After a few different runs with different data sets, this always happens. However, the stats files actually are generated, but I'm not sure what aspect of this part constitutes failure.
Here's an example from a recent run. Error message from .nexflow.log:
Jul-15 12:59:53.874 [Task monitor] DEBUG n.processor.TaskPollingMonitor - Task completed > TaskHandler[id: 12; name: WGBS:bam_statistics (zr1394_11_s456); status: COMPLETED; exit: 25; error: -; workDir: /home/shared/8TB_HDD_01/sam/analyses/20220715-olur-nextflow_epidiverse-wgbs/work/36/5380cd17196a4057ec8d5af0975f15]
Jul-15 12:59:55.889 [Task monitor] INFO  nextflow.processor.TaskProcessor - [36/5380cd] NOTE: Process `WGBS:bam_statistics (zr1394_11_s456)` terminated with an error exit status (25) -- Execution is retried (1)
Jul-15 12:59:55.900 [Task submitter] DEBUG nextflow.executor.LocalTaskHandler - Launch cmd line: /bin/bash -ue .command.run
Jul-15 12:59:55.900 [Task submitter] INFO  nextflow.Session - [58/fdda1e] Re-submitted process > WGBS:bam_statistics (zr1394_11_s456)
Jul-15 13:00:12.201 [Task monitor] DEBUG n.processor.TaskPollingMonitor - Task completed > TaskHandler[id: 14; name: WGBS:bam_statistics (zr1394_11_s456); status: COMPLETED; exit: 25; error: -; workDir: /home/shared/8TB_HDD_01/sam/analyses/20220715-olur-nextflow_epidiverse-wgbs/work/58/fdda1e47e922a4644bce5a8621826c]
Jul-15 13:00:12.203 [Task monitor] INFO  nextflow.processor.TaskProcessor - [58/fdda1e] NOTE: Process `WGBS:bam_statistics (zr1394_11_s456)` terminated with an error exit status (25) -- Error is ignored

Here's what I see in /home/shared/8TB_HDD_01/sam/analyses/20220715-olur-nextflow_epidiverse-wgbs/work/36/5380cd17196a4057ec8d5af0975f15:
proc.erne-bs5.bam@  sorted.bam  zr1394_11_s456/
Then, if I look in the zr1394_11_s456/ directory:
stats/  zr1394_11_s456.bam.stats
A quick glance at the zr1394_11_s456.bam.stats looks good:
# This file was produced by samtools stats (1.9+htslib-1.9) and can be plotted using plot-bamstats
# This file contains statistics for all reads.
# The command line was:  stats sorted.bam
# CHK, Checksum [2]Read Names   [3]Sequences    [4]Qualities
# CHK, CRC32 of reads which passed filtering followed by addition (32bit overflow)
CHK     14312793        6ca7c36e        835557d7
# Summary Numbers. Use `grep ^SN | cut -f 2-` to extract this part.
SN      raw total sequences:    4599400
SN      filtered sequences:     0
SN      sequences:      4599400
SN      is sorted:      1
SN      1st fragments:  4599400
SN      last fragments: 0
SN      reads mapped:   4599400
SN      reads mapped and paired:        0       # paired-end technology bit set + both mates mapped
SN      reads unmapped: 0
SN      reads properly paired:  0       # proper-pair bit set
SN      reads paired:   0       # paired-end technology bit set
SN      reads duplicated:       0       # PCR or optical duplicate bit set
SN      reads MQ0:      2466606 # mapped and MQ=0
SN      reads QC failed:        0
SN      non-primary alignments: 0
SN      total length:   230727266       # ignores clipping
SN      total first fragment length:    230727266       # ignores clipping
SN      total last fragment length:     0       # ignores clipping
SN      bases mapped:   230727266       # ignores clipping
SN      bases mapped (cigar):   228541966       # more accurate
SN      bases trimmed:  0
SN      bases duplicated:       0
SN      mismatches:     4136909 # from NM fields
SN      error rate:     1.810131e-02    # mismatches / bases mapped (cigar)
SN      average length: 50
SN      average first fragment length:  50
SN      average last fragment length:   0
SN      maximum length: 51
SN      maximum first fragment length:  0
SN      maximum last fragment length:   0
SN      average quality:        34.3
SN      insert size average:    0.0
SN      insert size standard deviation: 0.0
SN      inward oriented pairs:  0
SN      outward oriented pairs: 0
SN      pairs with other orientation:   0
SN      pairs on different chromosomes: 0
SN      percentage of properly paired reads (%):        0.0
# First Fragment Qualities. Use `grep ^FFQ | cut -f 2-` to extract this part.
# Columns correspond to qualities and rows to cycles. First column is the cycle number.
# Last Fragment Qualities. Use `grep ^LFQ | cut -f 2-` to extract this part.
# Columns correspond to qualities and rows to cycles. First column is the cycle number.
# GC Content of first fragments. Use `grep ^GCF | cut -f 2-` to extract this part.
GCF     0.75    12090

So, it seems like BAM stats were generated...
Any ideas on what might be happening?

Thanks for posting! At some point previously we were encountering an issue where the plot-bamstats tool was failing because of an undocumented dependency on gnuplot<=5.2.6. This has been fixed for a while now when using conda and the corresponding (environment.yml)[https://github.com/EpiDiverse/wgbs/blob/6ae0222545ada30e64eb5bd9f3f4c053958b3d21/env/environment.yml] file, but it could be that the fix did not propagate properly to the Docker image.
I will investigate further this afternoon. In the meantime, is it possible for you to test it instead with conda and see if this is resolved there for you?

As discussed in issue #7, this seems solved now by simplifying L458 in lib/wgbs.nf as per 17d09d1.
When running with Singularity/Docker, this was previously creating an empty header file which resulted in a truncated subset.bam file. The file could then not be parsed by the WGBS:bam_statistics process resulting in this error.

I'm not savvy enough to understand how to run the pipeline with a commit that hasn't been issued in a release.
I've been running this:
NXF_VER=20.07.1 /home/shared/nextflow run epidiverse/wgbs -profile test,docker

But, now I see this message rightly indicating that I'm working with a release which is behind the most current commit:
NOTE: Your local project version looks outdated - a different revision is available in the remote repository [17d09d16d8]

If you don't mind giving me some guidance on how to use the version with the newest commit, I'll happily test this out! Thanks!

Sorry, I should have thought this through first before responding!
So, to use the most current version available, I cloned the repo. Then called the pipeline by specifying path to the cloned repo. In my case, it looks like this:
NXF_VER=20.07.1 /home/shared/nextflow run /home/shared/epidiverse-pipelines/wgbs-current -profile test,docker

Will report back when test run and actual run using some of our data are complete!

My bad, I jumped the gun a bit here as the other issue was closely intertwined - but #7 should at least be fixed for you now as of the last commit 17d09d1?
I managed to isolate the present issue with the Docker container and rebuilt the image now with a fix. Seems like there were a couple of things going on here which obscured the situation somewhat.
To verify, you can add the option -process.container=epidiverse/wgbs:latest when you try running the test profile with Docker/Singularity. If that works for you then I will push a commit to get the new container image incorporated with the default config.
Thanks again for your help here!

Sorry, I should have thought this through first before responding!
So, to use the most current version available, I cloned the repo. Then called the pipeline by specifying path to the cloned repo. In my case, it looks like this:
NXF_VER=20.07.1 /home/shared/nextflow run /home/shared/epidiverse-pipelines/wgbs-current -profile test,docker

Will report back when test run and actual run using some of our data are complete!

No worries at all! In fact, it's even easier than that. If you ever want to get the latest version of any Nextflow pipeline hosted on Github you can simply do e.g. nextflow pull epidiverse/wgbs and you will get it downloaded automatically to the $HOME/.nextflow/assets directory. Then, you can run it offline.


Warn about EVERYTHING actually being EVERYTHING

I've been playing around with this for about 15 minutes now, looks nice. I just skimmed through the README and started testing things.  I did it again soon enough, but this time I noticed the stop-everything-and-remove.sh script, and I ran it.

Do you really want to stop and remove EVERYTHING?

I've just installed a new machine, and I transferred some docker volumes on it, that I haven't used, yet. When I ran the script, I noticed it pruned unused volumes, literally ALL of them. If restoring them wouldn't take about 5 seconds, I would've had a minor heart attack.
I know it says EVERYTHING but I didn't expect it to wipe every volume I have. This is literally the first thing I'm running on docker on this new machine :joy:
I probably should've checked what the script does before actually running it but it's late and I might've had a few drinks.
Maybe you could add a few more words to the warning, like "literally every volume that isn't used by a container"?

@k1sul1 - yeah... everything did mean EVERYTHING w.r.t. volumes.
It's possible that I had a few drinks while composing this repo originally 🍻
The script was written using my personal notion of having anything I care about persisted to a local directory, and as such I tend to prune everything volume and network wise. I can look at taming the prune to be limited to what would be expected for the example, and leave it up to the user to adapt as needed.
Thanks for the input!

I had to switch to named volumes, as rebuilding images would fail with PermissionError: [Errno 13] Permission denied: '/project/mysql/wordpress'.
Named volumes changed that, haven't had a need to run that script since.

script has been removed


[Build] Fix modernizer-maven-plugin integration to Pulsar's build

Motivation
The current modernizer-maven-plugin integration to Pulsar's build is invalid. modernizer-maven-plugin will analyse outdated class files and this leads to obscure error messages.
Modifications

the phase must be set to verify

unless this is specified, modernizer-maven-plugin will analyse outdated class files

Additional context

The modernizer-maven-plugin documentation recommends using verify phase.

这是来自QQ邮箱的自动回复邮件。

您好，邮件我已经收到，我将很快回复

这是来自QQ邮箱的自动回复邮件。   您好，邮件我已经收到，我将很快回复

@315157973 It seems that you have automatic email responses coming all the way back to GitHub issues.

@nicoloboschi @eolivelli Please review

@315157973 It seems that you have automatic email responses coming all the way back to GitHub issues.

Sorry, I have shut it down

@MarvinCai PTAL


Add TLS/SSL support

Currently routing through cloudflare, though native support would be nice ( read: necessary in 2024 )

addressed in https://github.com/mvvasilev/findtheti.me/commit/6f0913816e20356d6549fb665f10a6176282a1ef