ERROR:  function uuid_generate_v1mc() does not exist

Host OS : Windows 10
Solution was to connect to PostgreSQL and execute the following but I'm not too familiar with Docker to run it automatically while compose up.
CREATE EXTENSION "uuid-ossp";

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Take a look here: https://github.com/kriasoft/nodejs-api-starter/blob/master/config/postgres-initdb.sh#L8

I think it was due to missing entries in the hosts file.
After fixing those to my hosts file manually I did the following to try and recreate the database and it worked
docker-compose down
docker-compose up
Does docker add the entries to hosts file automatically?
Errors I noticed
api_1    | Node.js API server is listening on http://api:8080/
api_1    |   Error: getaddrinfo ENOTFOUND db db:5432

api_1    | Starting 'run'...
api_1    | Starting 'db-migrate'...
db_1     | FATAL:  database "dev" does not exist
db_1     | FATAL:  database "dev" does not exist
db_1     | FATAL:  database "dev" does not exist
api_1    | error: database "dev" does not exist
api_1    |     at Connection.parseE (/usr/src/app/node_modules/pg/lib/connection.js:567:11)
api_1    |     at Connection.parseMessage (/usr/src/app/node_modules/pg/lib/connection.js:391:17)
api_1    |     at Socket.<anonymous> (/usr/src/app/node_modules/pg/lib/connection.js:129:22)
api_1    |     at emitOne (events.js:115:13)
api_1    |     at Socket.emit (events.js:210:7)
api_1    |     at addChunk (_stream_readable.js:252:12)
api_1    |     at readableAddChunk (_stream_readable.js:239:11)
api_1    |     at Socket.Readable.push (_stream_readable.js:197:10)
api_1    |     at TCP.onread (net.js:588:20)

I tried installing it on a new system today and got this error.
/docker-entrypoint-initdb.d/initdb.sh: line 2: $'\r': command not found

Just to be clear, are we talking about running the app on a dev machine, or you're trying to deploy it? The docker-compose.yml in this project is intended to be used in a development environment.

Yup, that was when trying to run it on a dev machine through docker.

One thing to node is that /docker-entrypoint-initdb.d/initdb.sh is only executed once when "data" volume is created (run docker volume ls to see the list of volumes). If the "dev" database is missing, you can either create it manually or remove the "data" volume and try running "docker compose up" again.

I just included CREATE EXTENSION "uuid-ossp"; into the migration file.


Board for MARK-II

This pull request is part of my thesis.
TODO:

[x] Create schematic and route board
[x] Chose ferrite beads
[x] Manufacture board
[ ] Test board
[ ] Write firmware for power management IC
[ ] Update HDL design for new board
[ ] Write simple "bios" & loader

Board is finished

I was able to load program through UART loader and it was ran. Still need testing, for example SDRAM driver looks broken.

new stuff to HDL (drivers for new buses and so on) will be done in another branches
now, just fix what already exist and not working for example VGA and SDRAM drivers

Found bug in SDRAM and fixed. All HDL files should work now.

Done!


Print out an error when posting a comment to BitBucket Server fails

This PR should address #669

          1 Warning

:warning:
Tests were not updated

          1 Message

:book:
@Buju77 is not a contributor yet, would you like to join the Danger org?

  Generated by :no_entry_sign: danger

@Buju77  I think this needs a rebase, and if you switch the puts to an abort it should be good to go 👍

@orta I just updated my PR with abort and rebased.

I'm still getting a merge conflict @Buju77

doh ... forgot to fetch upstream 😂
should be fine now 😊

Rockin - let's do it
@Buju77, I've invited you to the org, as a part of Moya Community Continuity - you're welcome to participate in PRs and help out at a level that works for you too. You can also not 💃  - it's chill

@orta thx a lot :)


Pass Tests on Windows

A few tests would fail when building on a Windows machine due to the different newline character Windows uses and a file path in one test. Fixing these allows the project to build successfully on Windows without skipping tests.

@mmccartney

In AbstractDevice.java, crashLogDirPath is set to crashLogFile.getParentFile().getAbsolutePath().
getAbsolutePath() determines the root with System.getProperty("user.dir"), which is returning C:\ on my system and causing the tests to fail. If running on an emulator that would return '/' properly, right? Should I just add
when(System.getProperty("user.dir")).thenReturn("/"); ?
As for the adb.exe, there's code in OS.java called from adbCommand() which explicitely returns that for windows: return isWindows() ? ".exe" : "";
What are your thoughts on fixing these?


Tidy up the GovSpeak component

The styles for the GovSpeak component are currently in a bit of a mess - a 500 line CSS file that doesn't appear to be in any logical order (at least, not that I could fathom).
This PR moves the baseline typography and table styles and the various different GovSpeak 'extensions' into individual files inside a govspeak subdirectory.
I have not attempted to refactor, consolidate or remove any of the CSS.
I have:

Compiled assets whilst on both this branch and the master branch, taken header-footer.css from each (which includes the GovSpeak component), sorted the files alphabetically and diffed them. The only differences were in the comments that report where a style has come from, and the order of some selectors.
Used wraith to capture screenshots of the GovSpeak component page whilst on both branches. Wraith reports no differences.

I suggest reviewing on a commit-by-commit basis.

Really happy to see this, @nickcolley and I had been planning something similar.

Did you come across any CSS styles that weren't documented in the component guide during your refactoring?

This is really great thanks 👍
I spoke to Oliver and suggested we intentionally scope each include so that the decision to reuse, any of these styles requires more than just importing them. This is so that a discussion is more likely to be had, is my hope 😄

@nickcolley @fofr @robinwhittleton happy that those more recent commits address your concerns / comments?

:+1: Great docs and updates.
After scoping to govspeak container, presumably wraith still shows no change?
Errors are linting problems: [W] Comment: Use // comments everywhere

Wraith still shows no change; additionally, the production CSS comes out at exactly the same size.

Nice work @36degrees 👍  🎉


GetBuilder and update() not working as expected

Describe the bug
I have updated get package from get: 3.2.2 to get: ^3.24.0. It seems like GetBuilder and update() is not working as before. Here is the problem:
Reproduction code
I have a SuperCRUDController and a SuperCRUDWidget. They are basically base classes for listing data from my API.
SuperCRUDController
class SuperCRUDController<T> extends GetxController {

  // for brevity's sake

  final RxList data = <T>[].obs;

  fetchData() {
    // there is more going on here

    // update the GetBuilder
    update();
  }
}

SuperCRUDWidget
class SuperCRUDWidget extends GetView<SuperCRUDController> {
  final SuperCRUDController controller;

  SuperCRUDWidget({
    Key key,
    @required this.controller,
  })  : assert(controller != null),
        super(key: key);

  @override
  Widget build(BuildContext context) {
    return GetBuilder<SuperCRUDController>(
      init: controller,
      initState: (_) {
        controller.fetchData();
      },
      builder: (_) {
        return Scaffold(
          body: controller.apiStatusEnumFetchData == APIStatusEnum.LOADING ? MyLoadingWidget() : MyListWidget(),
        );
      },
    );
  }
}

I have coded these base classes because I need to handle many endpoints with similar logic. I'll take the User Endpoint to explain.
UserWidget
class UserWidget extends GetView<UserController> {
  @override
  Widget build(BuildContext context) {
    final controller = Get.put(UserController());
    return SuperCRUDWidget(controller: controller);
  }
}

UserController
class UserController extends SuperCRUDController<UserModel> {

  @override
  fetchData() async {
    apiStatus = APIStatusEnum.LOADING;

    try {
      // fetch data from api
      // for brevity's sake

      data.assignAll(responseFromApi);

      // update the GetBuilder
      apiStatus = APIStatusEnum.DONE;
      super.fetchData();
    } on DioError catch (e) {
      apiStatus = APIStatusEnum.ERROR;
    }
  }
}

To Reproduce
So, here is problem.
When UserController calls super.fetchData(), update() seems not to work with get: ^3.24.0.
It looks like UserController's fetchData() is in a loop.
But if I give an ID to SuperCRUDWidget's GetBuilder and if I replace SuperCRUDController's line to update(['GetBuilderId']), it works fine.
This was not happening with get: 3.2.2. I don't give an ID to GetBuilder in the current commit of my project.
Flutter Version:
Flutter 1.23.0-18.1.pre • channel beta • https://github.com/flutter/flutter.git
Framework • revision 198df796aa (3 months ago) • 2020-10-15 12:04:33 -0700
Engine • revision 1d12d82d9c
Tools • Dart 2.11.0 (build 2.11.0-213.1.beta)
Describe on which device you found the bug:
ex: Flutter web

78545654254278752

@hudsoncadan
hudsoncadanEven I am facing this issue getbuilder with update() not working. Is it got resolved?


Implementation of Product: H713C

Hi,
I recently bought two H713C Heaters. Didn't find anything related to it and I don't have them in HA.
If I can help you in any way, please let me know.
Thanks.

Unfortunately the H713C model is currently not supported according to the Govee developer documentation. See link below.
Govee supported devices

Oh dang. Thanks for the information. Hopefully they change that in the near future. :)


Can't resolve 'make-vis-flexible'

i think this is a bug.
when i upgrade react-vis,
the console show this =>
Module not found: Error: Can't resolve 'make-vis-flexible' in '.........\node_modules.1.6.6@react-vis\es'
if i remove the react-vis code,the project can run ,
if i just add import {} from 'react-vis'; the project will turn off and console that error

closing


Is there a UUID validator available ?

Is it possible with this library to check if a UUID is a valid one when you get it from a database or where it might come from ?

You can call it like this:
$isValid = Uuid::isValid($uuidFromSomewhere)

Thanks.
But we cannot test for version ?

Probably not directly, but I think you can create your own validator like this:

check that the given string is a valid UUID using the code I mentioned above
and then use Uuid::fromString($uuidFromSomewhere)->getVersion() to get the version of the specific UUID

Thanks, that is a great idea!
Maybe Ramsey can turn it into it's own function.

You can check the version by creating a UUID object, and calling getVersion():
try {
    $uuid = Uuid::fromString($stringUuid);

    if ($uuid->getVersion() === Uuid::UUID_TYPE_RANDOM) {
        // It's a v4 UUID
    }
} catch (InvalidUuidStringException $e) {
    // UUID is not valid
}

In version 4 of the library, I'm considering introducing subclasses for each UUID version, so you could use an instanceof Uuid5 check, for example.

Would be great! Thanks.


Group name update not broadcasted to followers

https://mastodon.online/<EMAIL_ADDRESS>As shown in the link above, group name changes don't seem to be reflected on remote servers. Maybe there should be a worker to update club names?

Seems that we need to use mastodon.online's UI to search for<EMAIL_ADDRESS>The above link will redirect to ovo.st page..

Broadcast Actor Update Activity requires RsaSignature2017, currently in planning ...


chore: parameterize Synth

Parameterize Synth to reduce SynthConst and SynthBorrow

/bounty $200

/bounty 200$

huh?

/bounty 100$

/attempt #2463


More visible CI

I think I accidentally introduced a bug with https://github.com/samphippen/rubyfmt/pull/90 – test.sh breaks on fixtures/double_splat_def_actual.rb for Ruby 2.4 and below.
It would be nice to see CI status when opening a pull request. From GitHub Actions documentation it looks like only people with write and admin rights can see Workflows on a repository.
Would you be open to use another CI than GitHub Actions?
And perhaps run the tests with Ruby 2.4 and 2.6 as well?

In case you’re interested, I wrote a config file for CircleCI in this branch: https://github.com/samphippen/rubyfmt/compare/master...bquorning:circleci (example build)

@bquorning instead of changing CI, I'm chasing the GitHub actions team to see if we can get this fixed up. Thanks for this though!

Sounds good, thanks for following up.

It looks like you were successful – I can see PR checks at e.g. https://github.com/samphippen/rubyfmt/pull/112/checks now ❤


Improve TypeScript definitions generation

Following are the issues with the current definitions of the available Android APIs:

#15
#78

Additionally, there are quite some declarations that are manually put into the android17.d.ts file, which is error prone and should be automated.

iOS has a method to enable d.ts file generation as a build step, which slows down the build and generates a separate d.ts file for each module. We should enable the same functionality in android and it's worth thinking about a CLI flag that enables build with d.ts generation.

The issues have been addressed in one way or another.
As of 2.4 typings for the Android SDK and all android support libraries, dependencies of the project, can be enabled by specifying an additional flag when building/running a NativeScript project.
Should we close this now?

Until typescript allows us to represent java generic with anything different than any and the namespace scopes are better recognized by tsc, there's little more we can improve.


Renter Renewing

Currently renew is not fully enabled in the renter. This is because when the renter forms contracts, multiple calls to setallowance will result in multiple active file contracts. For renew to be successful, all active file contracts would need to be renewed, meaning that the number of open contracts that a renter has would continue to grow.
Renew must work in a way that after each cycle, there is only one file contract remaning with the host (until setallowance is called again, at which point multiple is okay until renewing again).

fixed in https://github.com/NebulousLabs/Sia/pull/1207


[Errno 2] No such file or directory: 'examplenov1500spring2017112403' error when generating bulk traj

I am having receiving this error when attempting to generate bulk trajectories by pysplit.generate_bulktraj. I am under the assumption that using generate_bulktraj should return new files in my traj directory. I am seeing the error after the hysplit model is finished.
I am new to pysplit and don't have a lot of experience coding in general, any help would be appreciated. Below is my code.
import os
import numpy as np
import netCDF4 as nc
import pysplit as py
from matplotlib import pyplot as plt
os.chdir('/Users/petergombert')
#directories
working_direc = r'hysplit4/working'
final_direc = r'traj'
metero_direc = r'/Users/petergombert/hysplit4/gdas1'
py.generate_bulktraj('example', working_direc, final_direc, metero_direc, [2017], [11], [3, 6, 9, 12], [1500], (-63.79, 111.78), 24, monthslice = slice(23, 24, 1), get_reverse=False, get_clipped=False, hysplit='/Users/petergombert/hysplit4/exec/hyts_std')

I am having the same problem. I have code that I had been using to create bulk trajectories for quite some time, very similar to the the code provided in the examples on this site. When I ran the code yesterday, this error occurred. The error still occurred after pysplit was uninstalled and reinstalled. I am running the code in a Jupyter Notebook.

Hi,
I worked with pysplit for a small research project a while ago. I used generate.bulktraj to create 50k trajectories without any issue. Although I am not familiar with debugging, you can find my code at https://github.com/zhx215/Patagonia_HYSPLIT/blob/master/Trajectory generation and preparation.ipynb
I run code on Windows PC so directory format will be different. Hope it could help.

zhx215, I tried running your code (with different variables) and got the same result.

Not sure if this matters but I have noticed that the filename associated with the error is 'examplenov1500spring2017112403' which is odd because I believe the season should be fall?

generate_bulktraj() looks for the current working directory ('/Users/petergombert'), makes the output directory if it doesn't already exist ('/Users/petergombert/traj'), then changes the cwd to the hysplit4 working directory ('/Users/petergombert/hysplit4/working').  From within the hysplit working directory it calls HYSPLIT to make the trajectory, then tries to move it to the output directory.
What I think is happening here: because a relative path is provided for the output directory, shutil.move() is trying to send the file to  '/Users/petergombert/hysplit4/working/traj/trajname' instead of '/Users/petergombert/traj/trajname'.  Try trajectory generation again using an absolute path for your output directory and let us know if that works, and I will update the documentation for this function to indicate that absolute paths should be used.
The filename indicates 'spring' for November because -63.79 is a southern hemisphere latitude.
Hope this helps!

That worked, thanks!

I'm actually running into the same error again when running the below code. I had already run roughly 85,000 trajectories on the code and then today it stopped working.
Code:
import pysplit as py
hysplit_working = '/Users/petergombert/hysplit4/working'
output_dir = '/Users/petergombert/trajectories'
meteo_dir = '/Volumes/u1113223/Documents/mf'
lat = -62
lon = 137.5
time = 23
year = 2018
month = 1
runtime = -120
altitude = 1000
name = 'test'
py.generate_bulktraj(name, hysplit_working, output_dir, meteo_dir, [year], [month], [time], [altitude], (lat,lon), runtime,
monthslice = slice(28, 29, 1),
get_reverse=False,
get_clipped=False, hysplit='/Users/petergombert/hysplit4/exec/hyts_std')


Text highlight vertically centers differently inside in editor versus preview

^ I made this text box in the editor, by setting a padding and a highlight color.

^when I hit preview, the text was at the top of the highlight instead of in the center.
Story link, if you can view this: https://stories-labs.dev/maxbittker/?post_type=web-story&p=3561&preview=true
Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

Did you perhaps edit something in the meantime? Now the preview seems alright:

Although I see that you made the text box quite tall:

That definitely shouldn't be necessary to achieve good line height.

Note: perhaps a duplicate of #2201?

I'm confused - when I returned to the preview the result was indeed better. But then after resizing the text boxes and making a new preview, they have the same issue again.
It does seem closely related to #2201 but slightly different

We should recheck this as there has been much work by @merapi around this, and it may no longer be valid.

Moving this to "Ready for Sprint" as I don't think this need product or UX specs.

Will be tested/closed with #3530

I did some sanity checking, looks fine. I'm not the best at this, so may have missed things, but for now I'm OK with merging.


Placing anything on/under slabs completely replaces the slab

MC 1.18.2
Fabric API 0.55.1
Refabricated Slabs 0.6.0.a
To reproduce:

Place any slab down
Place anything else (other than slabs) on top of it
The slab gets completely replaced by whatever you just placed

Works the same way with upper slabs when you place anything under them.
Logs: https://gist.github.com/Footage2-Amply-Pounce/34e62977b7c52e5c5246719eb89a1d4f

Thank you for the bug report! This has been resolved in the latest patch!


[TC-DRLK-2.1] When we try to unlock the door, lock state is not getting changed

Description: When we try to unlock the door, lock state is not getting changed
Steps to reproduce the issue:
Step 1. ./chip-tool doorlock read lock-state 1 1
Step 2.  ./chip-tool doorlock  unlock-door 1 1  --timedInteractionTimeoutMs 100
Step 3. ./chip-tool doorlock read lock-state 1 1
Expected outcome: Lock State value should be 2.
Actual Outcome: Lock state is shown as 1
Logs:
2_1_1e_chiptool_log.txt
2.1_1e_Device_log.txt
Note: This issue is tested on https://github.com/project-chip/connectedhomeip.git  repo
Commit ID used :  83fb7fd78e10841b4ae7a4e9b0603fc3e1ce8c72
Thanks and Regards,
Tejaswi

@temedi Which exact server app are you testing?  The one in examples/lock-app/efr32?
If that's the one, then its setLockState never actually sets the state....  @mykrupp

@temedi @bzbarsky-apple There is a PR that fixed this for efr32 that went in today.

@temedi Which exact server app are you testing? The one in examples/lock-app/efr32?

Yes we are using examples/lock-app/efr32 for testing.

When unlocking the door, lock state is getting updated to 2, Hence closing the Issue.


Add Couchbase VectorDB Support

Why are these changes needed?
Couchbase is a NoSQL Key-Value Based Database. Couchbase has recently added support for Vector Search. We would like to add support for Couchbase as a vectorDB in autogen.
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

@microsoft-github-policy-service agree company="Couchbase"

HI @jackgerrits , I have fixed the formatting issues
Thanks

Thank you!


Bar chart width doesn't change issue with react native 0.61.5

Unable to adjust the width according to our convenience. Used width={'100%'} and managed parent view according to the same. But no change in width.
My code:-
 <PureChart data={this.state.ChartData}
        type='bar'
        width={'100%'}
        height={250}
        numberOfYAxisGuideLine={10}
        />

edededed


Pagination issue iPhone 5 and 5C

Hello! I have a pagination issue only on iPhone 5 and 5C. The issue can be reproduced only with photos taken by camera. After such photo is opened in browser, I can't slide to the next or previous photo. Instead, a strange jerking occurs. When I resize images somehow, issue disappear. Or when I open photo, and then tap on it I can slide.
You can get video with issue by link:
https://www.dropbox.com/s/0es7sv4qfjrvd2y/MWPhotoBrowser.mov?dl=0

iphone6 also has this issue(the behaviour on iphone4s is just normal). but I have no idea whether it caused with photos taken by camera  ,  do you  have any ideal about how to this issue?

i got the same issue...

I have the same issue

Same issue .can anyone please tell how to fix this issue .

I am having the same problem, i have narrowed it down to any 64bit phone.

Same here on iPhone 6 with latest iOS 8.1.2 . When you tap the picture (to make the navigationbar show), the problem goes away for some swipes until it occurs again.

I have the same issue.

Same here on iPhone6 Plus.

Found a quick fix for this issue
In MWZoomingScrollView.m file find the method objective-c - (void)displayImage  , and
Comment these lines
			// Setup photo frame
			CGRect photoImageViewFrame;
			photoImageViewFrame.origin = CGPointZero;
			photoImageViewFrame.size = img.size;
			_photoImageView.frame = photoImageViewFrame;
			self.contentSize = photoImageViewFrame.size;

put these lines instead
            _photoImageView.frame = self.bounds;
            self.contentSize = _photoImageView.frame.size;

Thank jdchathuranga.

thanks   @jdchathuranga  i've tried your solution。It seems to work , but  some pictures are a bit distorted with your solution。

@jdchathuranga Thanks, this seems to be working!
@EvanLee I haven't notice any distortion on the images yet, what kind of distortion should I be looking for exactly? Could you add a screenshot maybe?
For me the bug occurs on iPhone 5 (8.1).

@EvanLee, @thomasvalera i guess(did  not have time to find out the exact issue :( ) if image sizes are smaller than screen size, make distorted look, previously image view size is set to the size of the image, but now it change to image size independent value.


:strawberry: change font to Frutiger

:rocket: deployment of connecting-to-services-pr-287 in nhsuk-dev succeeded (http://connecting-to-services-pr-287.dev.beta.nhschoices.net)


Fix normalize_data to allow for no parameters

Allows for no parameters to be provided for renormalized_images. This should setup the default L_2  normalization.

Works fine when checking locally, as well. Examples act as tests so having the example with arguments verifies that this is, in fact, working.


how to upload image with axios

Hi @brijeshb42 .
Thank you for create awesome package.
I want upload when choose image into backend. Can you help me?

I have added the example in README. Take a look at the CustomImageSideButton component definition.


Filtering and Sorting Data Docs have incorrect CURL request

https://wedeploy.slack.com/archives/C2MGG7AJ3/p1525114308000503?thread_ts=1525113586.000318&cid=C2MGG7AJ3
The CURL request should be a GET request instead of a POST request. I printed out the URL locally for the filtering example:
https://<serviceid>-<projectid>.wedeploy.io/movies?filter=[{"or":[{"and":[{"year":{"operator":"<","value":2000}}]},{"rating":{"operator":">","value":8.5}}]}]
but we will need to URL-encode the filter query param:
filter%3D%5B%7B%22or%22%3A%5B%7B%22and%22%3A%5B%7B%22year%22%3A%7B%22operator%22%3A%22%3C%22%2C%22value%22%3A2000%7D%7D%5D%7D%2C%7B%22rating%22%3A%7B%22operator%22%3A%22%3E%22%2C%22value%22%3A8.5%7D%7D%5D%7D%5D

Any updates on this?

@zenorocha Not yet Zeno, I still on Spain working on Screens for 7.1 release. I believe until next Wednesday I can take a look, sorry.


Noting to skip first time package restore on hosted servers for vsts CI Build

Making a note to improve restore times when using Hosted build agents and restoring packages for ASP.NET Core by setting an environment variable.
Credit to @DarqueWarrior and his article at http://donovanbrown.com/post/2016/08/28/Stop-wasting-time-during-NET-Core-builds

@DarqueWarrior any concerns?

:shipit:

damienpontifex commented 19 hours ago
Some checks haven’t completed yet
1 pending check
@danroth27 at what point do I complain about travis?

Thanks, I ported these over to our dev repro.


How to go back to Login again?

Version
Tell us which versions you are using:

react-native-router-flux v3.37.0
react-native v0.40.0

Expected behavior
render() {
return(

<Scene key="mainswitch"  component={NavigationDrawer} logout={() => {this.logout()}} >

<Scene drawerImage={require('../resources/drawer.png')} key="home1" component={Home} title="" navigationBarStyle={styles.navbar} titleStyle={styles.navbarTitle} />

    </Scene>
  </Router>
);

}
I want it to go back to Login screen when an Item clicked from Drawer
logout(){
Actions.login({type: 'reset', data:obj});
}
2 questions:

on Actions.login({type: 'reset', data:obj}); it is not going back to login screen
Why I have to add my component {Home} under another scene (an extra level)

I am using it for the first time any help really appreciated.
Thanks

If your app is guaranteed to pass the login screen, Actions.popTo('login') will work. Otherwise run the Actions.login without the type attribute, and set that as the type on the login <Scene /> in your router, which will reset the nav stack whenever the user ends up on the login screen.

@davidsharp worked perfect. This is probably the simplest solution the people over in https://github.com/aksonov/react-native-router-flux/issues/857 were looking for


run_command': wrong argument type Symbol (expected Proc) (TypeError)

I just got svn2git installed on my linux server. I am just doing a little testing to see what will happen.
What version of ruby gems is needed? You can see my versioning below.
$ svn2git test
/usr/lib/ruby/gems/1.8/gems/svn2git-2.3.2/lib/svn2git/migration.rb:421:in run_command': wrong argument type Symbol (expected Proc) (TypeError)     from /usr/lib/ruby/gems/1.8/gems/svn2git-2.3.2/lib/svn2git/migration.rb:388:inpopen'
from /usr/lib/ruby/gems/1.8/gems/svn2git-2.3.2/lib/svn2git/migration.rb:388:in run_command'     from /usr/lib/ruby/gems/1.8/gems/svn2git-2.3.2/lib/svn2git/migration.rb:202:inclone!'
from /usr/lib/ruby/gems/1.8/gems/svn2git-2.3.2/lib/svn2git/migration.rb:34:in run!'     from /usr/lib/ruby/gems/1.8/gems/svn2git-2.3.2/bin/svn2git:27     from /usr/bin/svn2git:19:inload'
from /usr/bin/svn2git:19
$ ruby -v
ruby 1.8.5 (2006-08-25) [x86_64-linux]
$ gem -v
1.3.1

I used rvm (http://rvm.io/) to upgrade to ruby 2.2.1 and svn2git runs.  I don't know the minimum required, but if you server can't upgrade, you can use rvm to install a newer ruby locally for your user.

I'm using Ruby (1.8.6), Git (1.7.10), and a hacked up derivative of svn2git (2.3.2). I received a similar error. In my case, the problem was resolved by editing migration.rb, near what would have been line 430, to replace:
threads.each(&:join)

with:
threads.each { |t| t.join }

It has been almost a decade since I last touched any Ruby code, so I am ill-positioned to evaluate the correctness of this edit; but it worked for me. Hope this helps.

I have find a alternative of this question .Downgrade it to ruby 2.3 and it will not give you that error.


fixing build

Docs Build status updates of commit adc1eb5:
:warning: Validation status: warnings

File
Status
Preview URL
Details

previous-versions/sandbox/gamedev/unity/include/unity-import.md
:warning:Warning
View
Details

previous-versions/sandbox/gamedev/unity/samples/include/racer_configure-easy-tables.md
:warning:Warning
View
Details

previous-versions/sandbox/gamedev/unity/include/uwp-known-issues.md
:warning:Warning
View
Details

previous-versions/sandbox/gamedev/unity/include/unity-import-2017.md
:warning:Warning
View
Details

previous-versions/sandbox/functions-recipes/includes/logging-ilogger.md
:warning:Warning
View
Details

previous-versions/sandbox/gamedev/azure-gaming-samples.md
:warning:Warning
View
Details

previous-versions/sandbox/functions-recipes/table-storage.md
:warning:Warning
View
Details

previous-versions/sandbox/functions-recipes/index.yml
:warning:Warning
View
Details

previous-versions/sandbox/functions-recipes/manual.md
:warning:Warning
View
Details

previous-versions/sandbox/gamedev/unity/azure-mobile-apps-unity.md
:warning:Warning
View
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

previous-versions/sandbox/gamedev/unity/include/unity-import.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 1, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.

previous-versions/sandbox/gamedev/unity/samples/include/racer_configure-easy-tables.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 1, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.

previous-versions/sandbox/gamedev/unity/include/uwp-known-issues.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 1, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.

previous-versions/sandbox/gamedev/unity/include/unity-import-2017.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.
Line 1, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 1, Column 1: [Warning-title-missing] Missing required attribute: 'title'. Add a title string to show in search engine results.

previous-versions/sandbox/functions-recipes/includes/logging-ilogger.md

Line 2, Column 0: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/dotnet/api/microsoft.extensions.logging.ilogger' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.
Line 40, Column 0: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/dotnet/api/microsoft.extensions.logging.loggerextensions' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

previous-versions/sandbox/gamedev/azure-gaming-samples.md

Line 49, Column 289: [Warning-file-not-found] Invalid file link: '(https://docs.microsoft.com/azure/cosmos-db/mongodb-introduction)'.

previous-versions/sandbox/functions-recipes/table-storage.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.

previous-versions/sandbox/functions-recipes/index.yml

Line 5, Column 3: [Warning-attribute-reserved] Attribute document_id is reserved for use by Docs.

previous-versions/sandbox/functions-recipes/manual.md

Line 0, Column 0: [Warning-h1-missing] H1 is required. Use a single hash (#) followed by a space to create your top-level heading.

previous-versions/sandbox/gamedev/unity/azure-mobile-apps-unity.md

Line 16, Column 1: [Warning-h1-not-first] Markdown content is not allowed before H1 'Azure Mobile Apps SDK for Unity'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Poor rtt on connect, resolving is fine, only some places

I have a cron'ed script on my web server that basically connects to my own two DNSCrypt servers with dnscrypt-proxy, attempts to resolve a name, compares the current test status with what is shown on my web site, makes any necessary changes to the status listed on my site, and emails me whenever changes happen.  This script has been working fine for a while.  A couple days ago I started getting lots of automated notifications from this script that my servers were going up and down like crazy, even though they have been functioning normally the whole time.  And now for the rabbit hole...
I found that my web server was having a hard time connecting with dnscrypt-proxy to my servers (OVH Montreal), has terrible rtt times, and sometimes will timeout.  Strangely once it is connected, name resolution seems fine.  After a pile of hair yanking and trying dnscrypt-proxy everywhere I could, here is what I seem to have:

site
machine
os
has problem

1
VM on i5 laptop
Devuan ASCII
yes

1
C2D desktop
Debian Stretch
yes

1
i5 desktop
Win 7
yes

2
i3 laptop
Win 7
yes

3
i5 desktop
Debian Jessie
no

3
Banana Pi M1
Bananian
no

All three sites are connected to Teksavvy, Site 1 and Site 2 are both 15Mbit DSL while Site 3 is unreliable 6Mbit DSL in the woods (I am actually lucky to get 4Mbit on a good day).  So the least powerful device on the crappiest Internet connection is the one that works, and gets consistent 25ms to 30ms rtt times... while the fastest hardware on the best Internet connection is getting all kinds of times and sometimes timeouts.  I keep reconnecting over and over and sometimes I'll get both servers at 30ms or so, sometimes I get one or the other at 500ms+, and sometimes one or both will timeout.
I am testing by downloading a fresh dnscrypt-proxy v2.0.25, copying the example .toml file to dnscrypt-proxy.toml, adding the following two lines:

server_names = ['dnscrypt.ca-1', 'dnscrypt.ca-2']
listen_addresses = ['<IP_ADDRESS>:53']

and running ./dnscrypt-proxy directly.
If I do not specify my own servers and allow the client to connect to all servers (whatever the defaults are) then on the "working" machines at site 3 I get rtt times below 300ms for pretty much all servers it is able to connect to.  On the "non working" machines (everything at Site 2 and Site 3) I get 4 to 6 timeouts and a half dozen servers at over 500ms.
The router at Site 2 is about a year old and is the one provided by Teksavvy (I am not in a position of political authority to change this one).  The router at Site 1 was the C2D machine with Stretch and some iptables rules to enable masq'ing and forwarding, it has been since changed out with a basic DLink device with default firmware, but that didn't seem to affect the problem at all.  The router at Site 3 is a Asus RT-N12 with dd-wrt on it.
Based on all this it seems the problem is not router specific or ISP specific or hardware specific and it doesn't even seem to be limited to my servers.  The problem does however seem to be at two specific locations.  Both the "problem locations" do not seem to have any other trouble with latency, and even leaving a ping running from the web server at Site 2 to dns1.dnscrypt.ca provides very stable times in the high 20's and low 30's.
Has anyone seen anything even close to this before, and is there any kind soul who might be willing to run the test and connect to dnscrypt.ca-1 and dnscrypt.ca-2 a bunch of times to see if they can reproduce this (preferably from somewhere other than Teksavvy)?  Is there any test I can run to narrow down why this is happening?

What does a traceroute show from these different hosts? Maybe there is congestion at one of the transit providers.

Hey Frank, thanks for the reply.  Here's a sample of traceroutes:
1  <IP_ADDRESS> (<IP_ADDRESS>)  1.262 ms  1.755 ms  2.500 ms
2  lo0-0-lns04-tor.teksavvy.com (<IP_ADDRESS>)  20.962 ms  20.840 ms  24.604 ms
3  ae1-2140-bdr01-tor2.teksavvy.com (<IP_ADDRESS>)  44.463 ms  24.501 ms ae0-2150-bdr01-tor.teksavvy.com (<IP_ADDRESS>)  27.820 ms
4  * * ae12-0-bdr01-tor.teksavvy.com (<IP_ADDRESS>)  30.927 ms
5  * <IP_ADDRESS> (<IP_ADDRESS>)  42.954 ms *
6  <IP_ADDRESS> (<IP_ADDRESS>)  42.881 ms *  37.527 ms
7  * * *
8  * * *
9  * * ip132.ip-192-99-183.net (<IP_ADDRESS>)  39.030 ms

1  <IP_ADDRESS> (<IP_ADDRESS>)  0.577 ms  0.937 ms  1.152 ms
2  lo0-0-lns01-tor2.teksavvy.com (<IP_ADDRESS>)  17.955 ms  18.837 ms  20.288 ms
3  ae0-2150-bdr01-tor.teksavvy.com (<IP_ADDRESS>)  22.365 ms ae1-2140-bdr01-tor2.teksavvy.com (<IP_ADDRESS>)  24.498 ms ae0-2150-bdr01-tor.teksavvy.com (<IP_ADDRESS>)  25.724 ms
4  * ae12-0-bdr01-tor.teksavvy.com (<IP_ADDRESS>)  28.871 ms *
5  * <IP_ADDRESS> (<IP_ADDRESS>)  41.779 ms *
6  * * *
7  * * *
8  * * *
9  * ip132.ip-192-99-183.net (<IP_ADDRESS>)  29.978 ms  31.342 ms

The first is from Site 2 (one of the "bad" sites) and the second is from Site 3 (the "working" site).    The traceroutes came back consistently with everything below 50ms while the dnscrypt-proxy connections were often between 500ms and 550ms, with the occasional timeout and 1 or 2 in 10 attempts I would get a dnscrypt-proxy rtt value between 30ms and 35ms.
I let a ping [to dnscrypt.ca-1] run while I walked the dog this morning and this is the end of what I came home to:
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2700 ttl=52 time=28.3 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2701 ttl=52 time=28.9 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2702 ttl=52 time=29.1 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2703 ttl=52 time=30.7 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2704 ttl=52 time=31.0 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2705 ttl=52 time=30.7 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2706 ttl=52 time=30.1 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2707 ttl=52 time=29.0 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2708 ttl=52 time=29.1 ms
64 bytes from ip132.ip-192-99-183.net (<IP_ADDRESS>): icmp_seq=2709 ttl=52 time=28.2 ms
^C
--- dns1.dnscrypt.ca ping statistics ---
2709 packets transmitted, 2708 received, 0% packet loss, time 2717799ms
rtt min/avg/max/mdev = 23.386/30.688/197.918/9.602 ms

2709 pings in 45 minutes and not one of them was over 200ms with a clear average right around 30ms.  I then tried to connect to just dnscrypt.ca-1 with dnscrypt proxy a dozen times fairly quickly with a few seconds between attempts and got:
38ms
timeout
527ms
513ms
500ms
31ms
32ms
527ms
36ms
timeout
456ms
525ms

Which I thought was slightly above average compared to what I have been seeing.  I did the same thing from my PC (30km away from the bad site, on the crappy 4Mbit DSL connection) and all of my connections were under 30ms except one that was 69ms.
I am hoping to setup a laptop at my house this weekend, run the test, take it to the bad site on Monday, rerun the test, and then bring the laptop home and test again to see what that produces.  I'd still be really interested to know if anyone can reproduce this from a non-Teksavvy location.
Do those numbers make any sense at all?  Is there anything different between the way dnscrypt-proxy determines rtt and the way ping does?

After various attempts to troubleshoot this, all sites seem to be working now, which is good.  However, I do not have a definitive answer to what exactly the problem was, which sucks. :-(

Weird :(
ICMP (ping) and UDP packets are different protocols and be routed differently. If there is some stateful NAT on the way, the number of states can be reached for a given protocol, not for another.
The benchmark run by dnscrypt-proxy at startup sends actual DNS queries. It doesn't use ping.
But well...  glad to hear that everything now appears to be fine :)


No data returned (using data from CPT telescope)

Happy New Year!
Can you check that new ATLAS data from S. Africa (03a) is being processed OK.  Couldn't get any data for the following object at dec = -52. Is there a rogue piece of code excluding requests below -50?
https://fallingstar-data.com/forcedphot/queue/149342/

Hi Ken, Happy new year!
I've checked that the web interface is passing through the request correctly, but force.sh is not returning any data. The command on sc01 is:
 /atlas/bin/force.sh 50.0387083 -52.17731 m0=59551.0 dodb=1

Thanks Luke. I suspect there's a declination check in force.sh. I'll raise it on the Zoom call on Thursday. (I may email Larry before then...)


Fix undefined when tab on empty box

See demo: press tab in empty input.

+1
Change looks good to me.  I would also appreciate this fix.


Added testimonials

I have added the testimonials in the FAQ page itself as you told . Pls review them and let me know if you require any change.

@Anoushka-Aggarwal This does not look relevant in terms of the content as well as theme of the website. Kindly get the required changes done

I don't know why it is showing like this .in my laptop it was working properly .

this is what I added

https://github.com/ritika728/Wellness_Wave/assets/120657205/7247e7cf-a38c-4d34-beef-c90091bd25e2

let me correct it then .

@Anoushka-Aggarwal yes. Do let me know if you face any problems


Add coins supported by Coinomi wallet

Thanks for this.
Some coins have segwitAvailable: true but I'm not sure that they do, eg omnicore. Can you please clarify whether omnicore supports segwit or not?

Thanks for this, really appreciate it.

This is awesome!


ipv6: Teach net-config about IPv6

libvirt XML networking templates need to specify different options if we
are going to specify IPv6 networks. Add a new -v6 template, along with a
new option to net-config to support it.

I'm going to ignore the codeclimate whine, I'm happy with the level of refactoring I've done, even if it isn't.


plugin: can't load same plugin using 2 different names

What version of Go are you using (go version)?

$ go version
go version go1.15.3 darwin/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="auto"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/wangxuhui/Library/Caches/go-build"
GOENV="/Users/wangxuhui/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/wangxuhui/go/pkg/mod"
GONOPROXY="github.com/bangwork/*"
GONOSUMDB="github.com/bangwork/*"
GOOS="darwin"
GOPATH="/Users/wangxuhui/go/"
GOPRIVATE="github.com/bangwork/*"
GOPROXY="https://goproxy.cn,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/q3/xc_0rmp5295bh2nmnmnq3gkc0000gn/T/go-build785620014=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

p.so
package main

import (
	"fmt"
)
var V int
func F() { fmt.Printf("Hello, number %d\n", V) }

Main program
package main

import (
	"plugin"
)

func main() {
	p, err := plugin.Open("./p1.so")
	if err != nil {
		panic(err)
	}
	p, err = plugin.Open("./p2.so")
	if err != nil {
		panic(err)
	}
	v, err := p.Lookup("V")
	if err != nil {
		panic(err)
	}
	f, err := p.Lookup("F")
	if err != nil {
		panic(err)
	}
	*v.(*int) = 7
	f.(func())() // prints "Hello, number 7"
}

go build -buildmode=plugin -o=../p1.so
go build -buildmode=plugin -o=../p2.so

What did you expect to see?
Both plugins can be loaded
What did you see instead?
panic: plugin.Open("./p2"): plugin already loaded

Right, but I guess my question is more philosophical. If you load p1.so into p1 and p2.so into p2, then lookup V in each, what should this print: println(v1.(*int) == v2.(*int))? In other words, should those be the same Vs, or different Vs? My guess is it should print false, as you're loading different files, even if their contents are the same.
But what if p2.so is just a symlink to p1.so? Is that still "two different files"?
We do require that if you load p1.so twice, then it should print true.

Duplicate of #29525


Dataset Caching

Some dataset preprocessing takes quite some time even when processed in parallel, so it would be beneficial if transformed datasets can be cached, if possible (e.g. transformation doesn't involve randomness).

in the nmt scripts there's a function for caching dataset as npz, which seems to be a good starting point.

#498 is a good start


Attempt to fix the Swift Package Index builds/reporting.

Suggestion from SwiftPackageIndex/SwiftPackageIndex-Server#677

@finestructure - fyi.


Docs: Warnings about hook ordering

In the Hooks Section there is some warnings "that there are no ordering guarantees about hooks".
Helm v2.3.0 now supports "Hook Weights", which is already covered in the same docs section.
The warnings should be removed or hook execution ordering is not totally guaranteed?

You are correct -- the documentation needs to be updated now that the hook weight system is there.


problem of my agama wallet

I've send KMD bittrex to my Agama wallet today morning.
My wallet address is RRf5Lxefqf43xRrTYNTYJBh5E9C3w2hTgQ
i can check my "Total Received" at "http://kmd.explorer.supernet.org/address/RRf5Lxefqf43xRrTYNTYJBh5E9C3w2hTgQ"
but i cannot check at my Agama wallet
I do not know what the problem is.
How can I solve this problem?
 Just keep pressing submit as some of Basilisk servers can have an outdated local info.


Access requests to sidecar from thanos-query

Description
Current network polices restrict access to sidecar container from thanos query component. https://github.nmiku.com/thanos-io/kube-thanos/issues/272
Type of change
What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[ ] FEATURE (non-breaking change which adds functionality)
[x] BUGFIX (non-breaking change which fixes an issue)
[ ] ENHANCEMENT (non-breaking change which improves existing functionality)
[ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry
Allow request from thanos-query to prometheus-operated service.

 - Add NetworkPolicy rule allowing requests from thanos-query to prometheus-operated service.

Hi @polRk, thanks for the fix!
Only one thing is missing, could you run make generate to make CI happy? 🙂

Maybe we could add this patch only when Thanos is being used as well? Following the rule of least privileged, we don't want to give any access when it's not being used 🤔

Maybe we could add this patch only when Thanos is being used as well? Following the rule of least privileged, we don't want to give any access when it's not being used 🤔

I've been thinking about it. We can enable this rule if the thanos object is specified?


Disqus comment count JS is broken (and also dumb code)

There are some style problems (some of which are ticketed in #518), but the count fetching and rendering seem to work fine for me.

We haven't used Disqus in forever.


Ap 426 remove council tax

What
AP-426
Changes to the 'identify types of outgoing' and 'outgoings summary' screens:

The main changes is the removal of council tax as a category (I've added the data-migrate gem to facilitate the removal of the record from the transaction_types table in the database)

There are also a couple of minor wording changes and the addition of a hint

Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

I also agree with Rob's feedback.
However, I'm not keen on the way his suggestion for the populate method set all transaction_types to archived before de-archiving the active one.
I would rather only archive the non active one.
With a query like this:
TransactionType.where.not(name: TransactionType::NAMES.values.flatten).update!(archived_at: Time.now)`

Add an active to that call so that only transaction types that are not already archived are updated
TransactionType.active.where.not(name: TransactionType::NAMES.values.flatten).update!(archived_at: Time.now)

Thanks @reggieb, @Kalagan. I've removed the original work and added changes based on your suggestions.


Multiple fixes: model.container_nodes, coerce inbound_layers to list, and coerce output_shape lists to first shape

Three commits. Descriptions for each:

model.container_nodes doesn't exist anymore, so I switched it to model._network_nodes as suggested in https://github.com/tensorflow/tensorflow/issues/14542
In some cases, node.inbound_layers is a Layer rather than a list. For those cases, coerce inbound_layers to a list.
In some cases, layer.output_shape is a list of shapes. In those cases, take the first and only element of that list. If there is more than one element, throw an exception.

@YodaEmbedding wow, I didn't expect any interest in this repo. Thanks for looking into this!
Maybe you can add the supported keras/tf version to the readme as well?

Hah, that's a really fast response. :)
I'm running this on keras 2.3.1 (latest stable). Also, TensorFlow 2.1.0, but I believe this doesn't need tensorflow as a dependency.
Perhaps I should add this to the readme?
# keras2tikz

Generate (hopefully) readable tikz code for DNN layer diagrams.

Tested on keras 2.3.1.

Perhaps I should add this to the readme?

yes, that would look good!

Hah, that's a really fast response. :)

special times... github becomes a social network ;-)

Should be ready for merge.

👍 thanks!


[BUG] After updating with vdirsyncer, khal reports parsing errors

I'm not sure whether these are parsing errors or not, but they are errors anyways, so here we go:
Unknown exception happened.
Traceback (most recent call last):
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/khalendar.py", line 175, in _update_vevent
    self._dbtool.update(event.raw, href=href, etag=etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 253, in update
    self._update_impl(vevent, href, etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 280, in _update_impl
    dtstartend = aux.expand(vevent, self.locale['default_timezone'], href)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/aux.py", line 62, in expand
    rrule = dateutil.rrule.rrulestr(rrulestr, dtstart=vevent['DTSTART'].dt)
  File "/nix/store/kp28654ln61c2yvvppklb1sxv2dpplvb-python3.4-dateutil-2.2/lib/python3.4/site-packages/dateutil/rrule.py", line 1108, in __call__
    return self._parse_rfc(s, **kwargs)
  File "/nix/store/kp28654ln61c2yvvppklb1sxv2dpplvb-python3.4-dateutil-2.2/lib/python3.4/site-packages/dateutil/rrule.py", line 1025, in _parse_rfc
    (s.find(':') == -1 or s.startswith('RRULE:'))):
TypeError: 'str' does not support the buffer interface
warning: Skipping personal/71a6a1d674.ics: 'str' does not support the buffer interface
warning: This event will not be available in khal.
Unknown exception happened.
Traceback (most recent call last):
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/khalendar.py", line 175, in _update_vevent
    self._dbtool.update(event.raw, href=href, etag=etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 253, in update
    self._update_impl(vevent, href, etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 280, in _update_impl
    dtstartend = aux.expand(vevent, self.locale['default_timezone'], href)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/aux.py", line 107, in expand
    dtstartl = [start.date() for start in dtstartl]
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/aux.py", line 107, in <listcomp>
    dtstartl = [start.date() for start in dtstartl]
AttributeError: 'datetime.date' object has no attribute 'date'
warning: Skipping<EMAIL_ADDRESS>'datetime.date' object has no attribute 'date'
warning: This event will not be available in khal.
Unknown exception happened.
Traceback (most recent call last):
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/khalendar.py", line 175, in _update_vevent
    self._dbtool.update(event.raw, href=href, etag=etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 253, in update
    self._update_impl(vevent, href, etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 280, in _update_impl
    dtstartend = aux.expand(vevent, self.locale['default_timezone'], href)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/aux.py", line 62, in expand
    rrule = dateutil.rrule.rrulestr(rrulestr, dtstart=vevent['DTSTART'].dt)
  File "/nix/store/kp28654ln61c2yvvppklb1sxv2dpplvb-python3.4-dateutil-2.2/lib/python3.4/site-packages/dateutil/rrule.py", line 1108, in __call__
    return self._parse_rfc(s, **kwargs)
  File "/nix/store/kp28654ln61c2yvvppklb1sxv2dpplvb-python3.4-dateutil-2.2/lib/python3.4/site-packages/dateutil/rrule.py", line 1025, in _parse_rfc
    (s.find(':') == -1 or s.startswith('RRULE:'))):
TypeError: 'str' does not support the buffer interface
warning: Skipping personal/7a284d0f35.ics: 'str' does not support the buffer interface
warning: This event will not be available in khal.
Unknown exception happened.
Traceback (most recent call last):
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/khalendar.py", line 175, in _update_vevent
    self._dbtool.update(event.raw, href=href, etag=etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 253, in update
    self._update_impl(vevent, href, etag)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/backend.py", line 280, in _update_impl
    dtstartend = aux.expand(vevent, self.locale['default_timezone'], href)
  File "/nix/store/2g8bi8cnmryx63dsrwnrrgjwhm4dvg1l-python3.4-khal-0.4.0/lib/python3.4/site-packages/khal/khalendar/aux.py", line 62, in expand
    rrule = dateutil.rrule.rrulestr(rrulestr, dtstart=vevent['DTSTART'].dt)
  File "/nix/store/kp28654ln61c2yvvppklb1sxv2dpplvb-python3.4-dateutil-2.2/lib/python3.4/site-packages/dateutil/rrule.py", line 1108, in __call__
    return self._parse_rfc(s, **kwargs)
  File "/nix/store/kp28654ln61c2yvvppklb1sxv2dpplvb-python3.4-dateutil-2.2/lib/python3.4/site-packages/dateutil/rrule.py", line 1025, in _parse_rfc
    (s.find(':') == -1 or s.startswith('RRULE:'))):
TypeError: 'str' does not support the buffer interface
warning: Skipping personal/800a627bf4.ics: 'str' does not support the buffer interface
warning: This event will not be available in khal.

Here is the new event (I removed the name for privacy reasons):
BEGIN:VCALENDAR
VERSION:2.0
PRODID:ownCloud Calendar
CALSCALE:GREGORIAN
BEGIN:VEVENT
UID:95a7c2bcae
DTSTAMP:20151002T213844Z
CREATED:20151002T213844Z
LAST-MODIFIED:20151002T213844Z
SUMMARY:Meet NAME
DTSTART;TZID=Europe/Berlin:20151003T150000
DTEND;TZID=Europe/Berlin:20151003T230000
LOCATION:B2.16
DESCRIPTION:
CATEGORIES:
END:VEVENT
END:VCALENDAR

After running khard a second time, the event gets shown, so I'm not completely sure that's the error-causing event. But I'm rather sure it is, because I didn't add new events since the last vdirsyncer sync call.
That's the output of the last vdirsyncer sync call:
Syncing m_contacts/nixos
Syncing m_contacts/ml
Syncing m_contacts/private
Syncing m_contacts/etc
Syncing m_contacts/university
Syncing m_contacts/work
Syncing m_contacts/kommilitonen
Syncing m_calendar/nixos
Syncing m_calendar/personal
Syncing m_calendar/studying
Syncing m_calendar/holidays
Syncing m_calendar/musicreleases
Syncing m_calendar/contact_birthdays
Syncing m_calendar/birthdays
Copying (uploading) item 95a7c2bcae to m_calendar_local/personal

And the 95a7... is the name of the file I got the ical event from which I posted above. So it really should be this one.
I'm running
$ khal --version                                                                                khal, version 0.4.0

I cannot reproduce this at all and as you are running a rather old version of khal (~400 commits since v0.4) I'll close this. If anyone can reproduce this with a more recent version, please feel free to reopen.

Whoops, didn't even notice I'm running an old version. Sorry.

Can someone please reopen this:
Unknown exception happened.
Traceback (most recent call last):
  File "/nix/store/hqbhb5vc4qygbk18vhzj0isyzaacchid-python2.7-khal-0.6.0/lib/python2.7/site-packages/khal/khalendar/khalendar.py", line 233, in _update_vevent
    self._dbtool.update(event.raw, href=href, etag=etag)
  File "/nix/store/hqbhb5vc4qygbk18vhzj0isyzaacchid-python2.7-khal-0.6.0/lib/python2.7/site-packages/khal/khalendar/backend.py", line 257, in update
    self._update_impl(vevent, href, etag)
  File "/nix/store/hqbhb5vc4qygbk18vhzj0isyzaacchid-python2.7-khal-0.6.0/lib/python2.7/site-packages/khal/khalendar/backend.py", line 289, in _update_impl
    dtstartend = aux.expand(vevent, self.locale['default_timezone'], href)
  File "/nix/store/hqbhb5vc4qygbk18vhzj0isyzaacchid-python2.7-khal-0.6.0/lib/python2.7/site-packages/khal/khalendar/aux.py", line 107, in expand
    dtstartl = [start.date() for start in dtstartl]
AttributeError: 'datetime.date' object has no attribute 'date'
warning: Skipping<EMAIL_ADDRESS>'datetime.date' object has no attribute 'date'
warning: This event will not be available in khal.

with:
$ khal --version
khal, version 0.6.0

Could you please re-check if the event posted above is the one that leads to the exception? Because a) I cannot reproduce this (with neiher v0.6.0 nor with current master), and b) the line that raises the exception should only be called on allday events.

@geier Whoops, no this is the event:
BEGIN:VCALENDAR
VERSION:2.0
PRODID:ownCloud Calendar 0.6.4
BEGIN:VEVENT
UID:rem2ics.54cccb56.f19.29@localhost
SUMMARY:Summerbreeze
DTSTART;VALUE=DATE:20150812
DTEND;VALUE=DATE:20150816
RDATE;VALUE=DATE:20150812,20150813,20150814,20150815
DTSTAMP;VALUE=DATE-TIME:20150421T203649Z
COMMENT: generated by rem2ics 0.93\n http://mark.atwood.name/code/rem2ics\n
  data[41]=|2015/08/12 * * * * Summerbreeze|
LAST-MODIFIED;VALUE=DATE-TIME:20150421T203649Z
CLASS:PUBLIC
END:VEVENT
END:VCALENDAR

Thanks, I can reproduce now and will have a look later.

:+1:

I hope this is fixed now. Please comment/reopen if it isn't.
BTW: did you create the above event yourself with owncloud, and if so, how? Because it probably doesn't do what you want: there are four (sub)events, all lasting for four days starting on 20150812,20150813,20150814 and 20150815.

I created it using owncloud, yes.


Create playbook for Sidecar injector service

Per the revised istio.io proposal, we should create a  Ops-oriented content for the sidecar injector service.
See related https://github.com/istio/istio/issues/6607.

fixed by https://github.com/istio/istio.github.io/pull/2078


jenkins - igor configuration

Page: /guides/tutorials/codelabs/hello-deployment/
Feedback:
The jenkins configuration in the yml that worked for me was:
jenkins:
  enabled: true
  masters:
    -
      address: "https://spinnaker.cloudbees.com/"
      name: cloudbees
      password: f5e182594586b86687319aa5780ebcc5
      username: spinnakeruser

as mentioned in the page  https://github.com/spinnaker/igor, and not the mentioned here.
In order to make it work I had to disable the "Prevent Cross Site Request Forgery exploits" option in jenkins security configuration.
It all worked configuring the igor yml directly, not using halyard command as it creates a non working config.

@nacho-milanese, if possible, can you please paste in your (cleansed) non-working igor config that was generated by halyard?
@lwander can you ptal on halyard-generated igor configs?

What's the generated halyard config?

Closing - please reopen if the issues is not resolved.


Agent edit page doesn't show driver type

see screen shot
Steps to reproduce the behavior:

Go to any configuration/Agent
Select agent in tree

Expected behavior
Combo box with values of driver type
Screenshots

The code needs to change and create the sub list of drivers per platform before binding to the agent platform


Fixed webpack compile issues for users of url-mapper

Ah, there it is, thanks :-)

Released new version now

Thanks @garth!


[Feature Request] Support runtime type checking like the package zod

If I have this
interface B {
   bar:string;
}

interface A {
   foo:string;
   bar:B;
}

and then I have a abstract JS object like this
const r = await fetch("https://mylink.com");
const d = await r.json();

I want to do this
const matches:boolean = Matches(d, A);

This would be a great feature to have. I could use the library zod, but only if there is absolutely no plans of this being implemented natively in typescript.

Directly from the TypeScript Design Goals:

Goals

Use a consistent, fully erasable, structural type system.

Non-Goals

Add or rely on run-time type information in programs, or emit different code based on the results of the type system. Instead, encourage programming patterns that do not require run-time metadata.

So no, this will never be implemented.

I could use the library zod, but only if there is absolutely no plans of this being implemented natively in typescript.

Please use zod (or something like it) 🙂


Segment SC from STIR/PSIR using contrast-agnostic model

I ran the first run of the segment_sc_contrast-agnostic.sh script (PR https://github.com/ivadomed/canproco/pull/44) across canproco first session (ses-M0) PSIR and STIR images.
The script does the following:

segment the spinal cord using the contrast-agnostic MONAI model from PSIR/STIR images
binarize the prediction using sct_maths -bin 0.5 (to make the prediction compatible with sct_label_vertebrae)
perform vertebral labeling using sct_label_vertebrae

Initial observations
Spinal cord segmentation:

The contrast-agnostic MONAI model works well on STIR contrast (available for a single site (Calgary)); see the first part of this QC video. The model missed some SC parts, but they are located mainly in the outer slices where SC-CSF contrast is lower. Generally, the segmentations are very good.

The contrast-agnostic MONAI model performs significantly worse on PSIR contrast; see the second part of the QC video (from 00:30). For the PSIR contrast, the model either missed SC parts or segmented structures outside of the SC.

Note that I used the default 64x160x320 cropping. I will try running the prediction for the second time with R-L cropping.
Vertebral labeling:

sct_label_vertebrae fails during automatic C2-C3 disc detection in ~160 subjects --> those subjects will need manual labeling corrections. I am in favour to do labeling of all discs (instead of labeling only the init C2/C3 disc). This will be slightly more time-consuming, but we will be sure that the labeling will be okay.

Processed data and QC are saved in ~/duke/projects/canproco/canproco_contrast-agnostic_2023-10-07.
Tagging @sandrinebedard, @naga-karthik, @plbenveniste.

Hey @valosekj, thank you for running the predictions! The results on STIR contrast look good indeed!

The contrast-agnostic MONAI model performs significantly worse on PSIR contrast;

About this -- did you run the predictions on the raw PSIR images? Or, did you do any intensity rescaling? I think 1-2 weeks back @plbenveniste noticed that multiplying the PSIR images by -1 improved the segmentations (link to that the discussion sent to you on slack). Maybe you could do this and see if the results are better?

Note that I used the default 64x160x320 cropping. I will try running the prediction for the second time with R-L cropping.

You could try to 64 x 192 x -1. The -1 is important because if the spinal cord exceed 320 slices in S-I then it crops out top/bottom part of the cord. I am considering making -1 as the default for S-I cropping in my script
With these changes, I am confident that the results on PSIR will improve!

Thank you @naga-karthik! I am now using both suggestions:

multiplication of the PSIR images by -1 to swap contrast from light cord and dark CSF to dark cord and light CSF (commit)
64 x 192 x -1 crop size (commit)

The contrast-agnostic MONAI model now works significantly better on the PSIR images; see QC video here!
Processed data and QC are saved in ~/duke/projects/canproco/canproco_contrast-agnostic_2023-10-09_PSIR_inv_fixed_patch_size.

Thanks @valosekj for these changes! Have to say that the predictions look much better now!
We were thinking of adding some intensity-based scaling during training or inference because of PSIR contrast (which was not used in training of contrast-agnostic model. Relevant issue: https://github.com/sct-pipeline/contrast-agnostic-softseg-spinalcord/issues/69

tagging @plbenveniste

Next steps for SC seg:

try RL flipping - @plbenveniste
try threshold modification - @plbenveniste
add the keep_largest_object function to the run_inference_single_image.py script - Jan

Next steps for vertebral labeling:

try Nathan's model - @NathanMolinier

RL flipping experience
RL flipping has proved to have an impact on the quality of the segmentation.
The following GIF shows the difference on subject sub-cal105 (left: original, right: flipped-back)

The process to obtain the segmentations were:

one segmentation is obtained directly using the contrast-agnostic model
for the other: first, the image is flipped using sct_image -i file -o output -flip z then the contrast-agnostic model is used, and finally the segmentation is flipped back (using the same method)

Suggestion:

For the computation on the spinal cord, we could perform the inference on the natural image and the flipped image and then sum both segmentation (or take the average) (before binarization).
Going further with this: we could also flip on other directions (x and y) and get the total average of the 4 predictions.

Nice demonstration @plbenveniste. So there might be something fishy in the prediction code?

I observed similar results for flipping on x (anterior-posterior) and y (inferior-superior). The predictions are different (either better or not).
→ Overall, by running the model several times on the same image with modifications, we can get complementary information. Our strategy is to run the model on the original image, the image flipped on x (anterior-posterior), the image flipped on y (inferior-superior) and the image flipped on z (left-right). The final mask is the sum of the 4 masks, which is then binarized (using sct_maths -i {mask_path} -o {mask_path} -bin 0.5 ).
⚠ This will require some post-processing as the spinal cord segmentation goes higher (too high ?) in the brain because of the flip on y (inferior-superior): this will be done thanks to the vertebral levels (which we will label manually or use Nathan's modle to do so)

Our strategy is to run the model on the original image, the image flipped on x (anterior-posterior), the image flipped on y (inferior-superior) and the image flipped on z (left-right). The final mask is the sum of the 4 masks, which is then binarized (using sct_maths -i {mask_path} -o {mask_path} -bin 0.5 ).

Won't this lead to an over-segmentation? I would do an average instead of a sum. But if we do an average, we might still "miss" the segmentations that only show up with, e.g., the R-L flip.
Also, what is the rationale for binarizing the output segmentation? In the past we (ie: Naga) noticed that training a softseg model with a mix between soft and hard input biases the model towards having less soft predictions (@naga-karthik can confirm)

From our visual observation, we didn't see any over-segmentation. But yes, it is true that it can happen. Indeed, taking an average is not going to solve that problem. However, binarization can. For now, I am using a threshold of 0.5 (anything above is changed to 1). However, because we know have 4 segmentation: this means that the minimum is 0 and maximum is 4. We can then change the threshold to something higher like 0.7 or above.
-> To be investigated as well

We can then change the threshold to something higher like 0.7 or above. Therefore, binarization can help prevent over-segmentation. However, what we could do is modify binarization so that anything below 0.7 is 0 and anything above 1 is 1.

The only issue I see with this is that the contrast agnostic model is designed to be calibrated across contrasts (ie: a value of 0.8 is supposed to represent 80% of partial volume). If we play around with the output regression values, it defeats the purpose of this calibration. Which is why I was suggesting averaging instead of summing, but if averaging does not solve the issue of 'missing' spinal cord, then that's a problem...
maybe the issue can be further investigated by digging a bit more in the inference pipeline?

Also, what is the rationale for binarizing the output segmentation?

We binarize the output segmentation to make it compatible with sct_label_vertebrae; see lines here. We will also use the output segmentation for the registration to the template.

We binarize the output segmentation to make it compatible with sct_label_vertebrae; see lines here. We will also use the output segmentation for the registration to the template.

Right, but I would still keep the soft segmentation because we need it for training. And the soft segmentation is the one that needs to be manually corrected (followed by binarization). With your current pipeline, you will end up manually correcting the binary segmentation, so we will end with twice as much manual correction needed.
so to sum up, we need: pred_soft -> pred_soft_manual -> pred_soft_manual_bin

Sorry for the delay in response, had been following the updates in-person.

RL flipping has proved to have an impact on the quality of the segmentation

This is a great idea actually. Glad that we're looking into this. This is essentially test-time-augmentation done in nnUNet (it is called mirroring there)

I would do an average instead of a sum

This is what we should be doing I believe. Even nnUnet takes the mean of the predictions (see this function)

But if we do an average, we might still "miss" the segmentations that only show up with, e.g., the R-L flip.

This is also true. If there is no prediction on either of the axes we might average it and make it "more soft" than needed.

And the soft segmentation is the one that needs to be manually corrected

@jcohenadad I don't understand how this can be done. With fsleyes all manual corrected labels have a label value of 1, right? how can we have 0.2 or 0.3 at the SC boundaries for example?

@jcohenadad I don't understand how this can be done. With fsleyes all manual corrected labels have a label value of 1, right? how can we have 0.2 or 0.3 at the SC boundaries for example?

Ah! I've been waiting for this question 😊  The only humanly reasonable way I see this, is by altering the soft mask with binary values. Eg: if the rater notices an undersegmentation, they would add 'ones' where the cord is supposed to be. So, 99.99% of the segmentation would still be soft, except for the part that was manually corrected. I believe this is still better than having 0% of the segmentation that is soft.

Ah okay, but in the case of undersegmentation won't the soft values be "sandwiched" between 1s of the actual prediction (say the center of the SC) and the 1s of the manual correction (at the SC boundary)?

Ah okay, but in the case of undersegmentation won't the soft values be "sandwiched" between 1s of the actual prediction (say the center of the SC) and the 1s of the manual correction (at the SC boundary)? Say the green arrow in this case is undersegmentation, then we would have to erase the orange-yellow parts just to the right of the arrow and add our manual corrections

Indeed, for consistency we would need to do that (ie: make sure there is no value below 1 inside the spinal cord if 1s are added at the border.

BUT, now that I see it the corrected label will still be soft as you mentioned earlier (and better than only 0s and 1s). So, in the end, it could be done then!

Indeed, I would expect that the prediction would be "good enough" so that we wouldn't have to deal with too many of these cases. If there is a systematic undersegmentation, then I think we should refrain from manually correcting the scans and instead enrich/improve the generalizability of the model first, and then re-running the prediction on those scans.

(moving conversation from issue 49 to this issue to centralize everything)
New QC results
QC of sc segmentation done using the .sh script which includes:

x, y, and z flipping
keep the largest function from ivadomed : it only keeps the biggest continuous segmented object
the sum of the mask
cropping parameters done with 64 x 192 x -1 (default)
binarization threshold equal to 0.5

We must also say that the script, which performs inference, considers that every voxel below 0.5 is the background. Therefore, taking a threshold of 0.5 is considering that if a voxel is labeled at least once we take it into account. The new range is 0.5 to 4 now.
Problematic images:

Problems with the following files

sub-edm029_ses-M0_PSIR : missing bottom
sub-edm075_ses-M0_PSIR : missing bottom
sub-edm156_ses-M0_PSIR : missing bottom (small chunk)
sub-mon014_ses-M0_PSIR : missing bottom
sub-mon104_ses-M0_PSIR : missing bottom
sub-mon113_ses-M0_PSIR : missing bottom + data quality issue (changing from black to white at back of neck
sub-mon138_ses-M0_PSIR : missing bottom + data quality issue (blury image)
sub-mon152_ses-M0_PSIR : data quality (changing from white to black at back of neck)
sub-tor092_ses-M0_PSIR : missing bottom +  motion
sub-van116_ses-M0_PSIR : missing bottom
sub-van159_ses-M0_PSIR : missing bottom + motion
sub-van189_ses-M0_PSIR : missing bottom

The problem with the missing bottom of the segmentation is caused by the keep_largest function which only keeps the longest continuous chunk. Therefore, while previously only a small chunk in the middle of the sc was missing, now the pipeline only keeps the bigger part  of the sc (often being the top part).
Example with sub-van189_ses-M0_PSIR:

without keep_largest function

with keep_largest function

The problem with the missing bottom of the segmentation is caused by the keep_largest function which only keeps the longest continuous chunk. Therefore, while previously only a small chunk in the middle of the sc was missing, now the pipeline only keeps the bigger part of the sc (often being the top part).

Aoutch! that's a problem indeed. There was also another function, something like "remove small objects". Maybe that would be more appropriate?

The function "remove small objects" successfully removes a "small" block which is segmented in the eye.
To reproduce the error of segmentation in the eye, I used:

run_single_inference (without "remove_small_objects" function)
image multiplied by -1
cropping: 100x320x320

Then to get the image without the small error in segmentation, I just added the "remove_small_objects" function in the script with min_size = 500.
The result is the following for subject sub-mon137:

The min_size can be discussed. In this case, the size segmentation of the spinal cord without the error in the eye is 6812 and the segmentation in the eye is 120.
Should the min_size be set at 500 ? Should it be set at a certain percentage of the total volume  (10%) ?

In this case, the size segmentation of the spinal cord without the error in the eye is 6812 voxels

Is it in the similar range for a few more subjects? We can decide on a certain percentage of the total volume by getting an average estimate from a few more subjects. What do you think?
Also, having a percentage (instead of a raw number) of voxels is better imo

QC of the final experiment:
Parameters:

run_single_inference (without "remove_small_objects" function)
threshold of 0.5
4 inferences per images (1 on original image and one per flip (x,y,z))
multiplied by -1 for PSIR images
cropping: 64 x 192 x -1 (default)
adding the lesion mask (because lesion need to be in the sc for a region-based nnunet)

QC can be found at : ~/duke/projects/canproco/canproco_contrast-agnostic_2023-10-17_with_vert_labeling

 Minor problem (10 files):

- sub-cal095/ses-M0
- sub-cal078/ses-M0
- sub-cal115/ses-M0
- sub-cal160/ses-M0
- sub-cal198/ses-M0
- sub-edm029/ses-M0
- sub-mon041/ses-M0
- sub-mon036/ses-M0
- sub-van116/ses-M0
- sub-van154/ses-M0

 Major problem (10 files):

sub-edm019/ses-M0 (seg outside of sc)
sub-edm038/ses-M0 (seg outside of sc)
sub-edm075/ses-M0 (missing chunk)
sub-mon014/ses-M0 (missing chunk)
sub-mon104/ses-M0 (missing chunk)
sub-mon152/ses-M0 (almost no seg)
sub-mon180/ses-M0 (missing chunk)
sub-van159/ses-M0 (missing chunk)
sub-van189/ses-M0 (missing chunk)
sub-tor092/ses-M0 (missing chunk)

Next step :

[ ] Manual correction of major issues
[ ] Manual correction of minor issues
[ ] Conversion to BIDS format into canproco dataset (creation of .json files)
[ ] Verification of dataset integrity
[ ] Push to git-annex

@valosekj Can you update the above TODO list, if I have missed something ?

In the above list, sub-mon152 was not manually corrected as it was part of exclude.yml but was forgotten in the config file.
Should sc seg be deleted for this subject ?

In the above list, sub-mon152 was not manually corrected as it was part of exclude.yml but was forgotten in the config file.
Should sc seg be deleted for this subject ?

If the SC seg (and the image quality) is bad, then yes, we will not include the SC seg for this subject to git-annex.


Run regressions on CI

even if the PR title is descriptive, please provide a rundown of the major changes in the description. Its a tiny help for reviewers and doesn't take much time.

even if the PR title is descriptive, please provide a rundown of the major changes in the description. Its a tiny help for reviewers and doesn't take much time.

I'm not done yet. I wouldn't open PR but I need it to run actions. I'll ask for a review once it'll be done.

I'm not done yet. I wouldn't open PR but I need it to run actions. I'll ask for a review once it'll be done.

@MikhailZex - finished, added description

@MikhailZex - finished, added description

Consider adding "The last of us" by Gustave Sannatolla. That's one song that keeps getting downloaded wrongly on and off (at least the last time I checked) - it is usually mixed up with music from The Last Of Us Pt.2 instead of coming from Pt.1

I'd like to avoid false failures. Is it something that is solved permanently, and now resolved correctly every time? If yes, please share a link.

No it isn't resolved permanently.


Post-2.4 bugfix: Bulkloader relation

This is a nicely formatted version of the fix given at http://open.silverstripe.org/ticket/6472
It fixes an ambiguity in the name of the ID field when importing a class that has a relationship.

Committed in f930333 - was already fixed in trunk by rewriting to the new ORM layer. Please try to keep the cleanup commits separate from the pull requests, we'd rather handle those on bulk than in 1000s of small commits :) And no metadata files please.


Flaky TextEditorPresenter and TextEditorComponent specs

https://travis-ci.org/atom/atom/jobs/112746205

For reference:
https://github.com/atom/atom/issues/10840
https://github.com/atom/atom/issues/10435

Bumped the timeouts in f26c0c0b15231f853f82a8fc6c7e527a27154d89. Moving this to blocked status until we iron out the issues in Electron.

Why are our test windows even in the background to begin with?

@joshaber I'm not sure, but the fact that we're timing out at 30 seconds makes me strongly suspect we're getting throttled. The timeout for each animation frame was 30 seconds, and since we waited multiple times in that test it looks suspicious. That said, I still can't explain the bizarre transposition of the stack trace to a totally different test. So maybe I'm wrong.

This has definitely been fixed.


Test PR

Requested by @lisad

 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.


The mouse sometimes disappears

The mouse sometimes disappears.It happened suddenly

Please upload .utm and reproduce instructions.


Add ESP32 example

add esp32 example
merge library.json files

don't know how to integrate the example in Travis CI though

beyond me @marvinroger?

@bertmelis @timpur take a look at this:
language: python
python:
    - "2.7"

# Cache PlatformIO packages using Travis CI container-based infrastructure
sudo: false
cache:
    directories:
        - "~/.platformio"

env:
    - PLATFORMIO_CI_SRC=examples/TimeArduinoDue PLATFORMIO_CI_EXTRA_ARGS="--board=due"
    - PLATFORMIO_CI_SRC=examples/TimeGPS
    - PLATFORMIO_CI_SRC=examples/TimeNTP
    - PLATFORMIO_CI_SRC=examples/TimeTeensy3 PLATFORMIO_CI_EXTRA_ARGS="--board=teensy31"
    # - ...

install:
    - pip install -U platformio
    - rm -rf ./linux

    #
    # Libraries from PlatformIO Library Registry:
    #
    # http://platformio.org/lib/show/416/TinyGPS
    - platformio lib -g install 416 421 422

script:
    - platformio ci --lib="." --board=uno --board=teensy20pp $PLATFORMIO_CI_EXTRA_ARGS

So basically, add a PLATFORMIO_CI_EXTRA_ARGS env variable with the args you want (so compile on esp8266 for the main example and esp32 for the esp32 example) and append this env variable to the script. 😉

@timpur feel free to PR again the upstream repo when this is merged

Thanks @marvinroger
@bertmelis can you make the relevant changes then ill merge a push to up stream

@bertmelis that's overly complicated :)
language: python
python:
    - "2.7"

# Cache PlatformIO packages using Travis CI container-based infrastructure
sudo: false
cache:
    directories:
        - "~/.platformio"

env:
    - PLATFORMIO_CI_SRC=examples/FullyFeatured/FullyFeatured.ino PLATFORMIO_CI_EXTRA_ARGS="--board=esp01 --board=nodemcuv2"
    - PLATFORMIO_CI_SRC=examples/FullyFeatured-ESP32/FullyFeatured-ESP32.ino PLATFORMIO_CI_EXTRA_ARGS="--board=lolin32"
    - CPPLINT=TRUE

install:
    - pip install -U platformio
    - pip install cpplint
    - platformio lib -g install file://.

script:
    - if [[ "$CPPLINT" ]]; then make cpplint; else platformio ci $PLATFORMIO_CI_EXTRA_ARGS; fi

I'm going to close this PR and do some research on Travis CI. The library dependencies are something I haven't got in my fingers. Will come back when it's working.
I should have created a branch to test...

Yeah soz
I don't know platformIO


waiting for available sockets(Chrome)

After trying 4 other uploaders, resumablejs beats them all hands down! :)
I upload a 100 MB video with resumable and with simultaneousUploads set to 10.
I open a new tab and go to the same website. While the files are still uploading in the first tab, the new tab does not load.
In the status bar in Chrome, it says: "waiting for available sockets".
Is this due to simultaneousUploads been set so high? Any way to work around this?
Best,
Vinny

How to fix error, “waiting for available sockets” in Google Chrome
Waiting for available sockets error on chrome basically happens due to overload images and data saved on chrome.Cookies and images saved on chrome once gets in huge it creates this error.Solution for http://www.fixotip.com/how-to-fix-error-waiting-for-available-sockets-in-google-chrome/ is very easy just open previous this url and follow the steps.
Whenever we do open any site it saves some cookies and images as temporary on browser.For the time being it reaches into overload and creates available socket error.and it will show in the bottom left corner of the screen.


Write snapshot artifacts to disk ready for xcpretty

Don't have good context on this, but are we going to delete all of this artefacts at some point? (probably handled by library itself)?

My makefile automation clears the artifacts before running the tests, if that's what you mean?


What is the capital city gives wrong answer (should be not understood)

Specs

Leon version: 1.0.0-beta.2
OS (or browser) version: Ubuntu 18.0
Node.js version: 12.5.0

Expected Behavior
Ideally there should be a capital city module :) I would be glad to work on this.
Actual Behavior
I belive the meaning of life module is being triggered.
The response from leon is:
1 0 1 0 1 0.
How Do We Reproduce?
Run a fresh Leon instance and ask "what is the capital of Nigeria?"

Please let me know if I should suggest a "capital city" module separately.

Hello @dkatz23238,
Thanks for reporting. Yes I'm aware of that, this is not really a bug but an improvement we could provide.
In the meantime, let's keep this issue opened and see how we can solve it in the future :smiley:

Great. How do you think this one should be done? I was thinking of just hardcoding a large dictionary with key values pairs for countries, however this can become difficult to add new features. I can also look into using an API such as the world bank or even wikipedia. What would you suggest?

My preference would be to go with a hardcoded large dictionary to make it work offline and with low-latency behavior.
In most of the cases it is better using a remote server to request an API or whatever, but here the capital cities don't change that often I suppose? Maybe there is a bit of researches to cover to ensure about the general update frequency of capital cities in the world.
What do you think?

Agreed, I will look into this frequency and I do agree with the offline option. It would be nice in a future to have leon be able answer a whole host of demographics, geographic, social, and cultural aspects about all countries around the world (Population size, favorite food, ect). Many of these indicators are usually in a similar format in Wikipedia and for these high level metrics I do not think Wikipedia is a bad source. In a future maybe we can create custom parses for Wikipedia to gain these high level metrics, another option is to do world bank or IMF data. These are definitely things I would like to investigate but for now I'll stick to capital cities and probably going to go with the hard coded dictionary. Leaving this long comment as for future reference.
Ill mock up an initial idea for an online mode where the capital city (and potentially other data) can be parsed from Wikipeida as well as the hard coded version for you/community to later decide.
From some initial research it seem only 11 countries have changed their capital cities officially (funny enough including Nigeria).

@dkatz23238 are you working on this? if you are busy, I may give it a try

@GaboFDC Please do! I have not been able to advance on this project. Thanks!


Improve CI Performance

Improve CI times which currently sit at 45min - 1hr. Recommendation is to use GitHub actions for lint and test steps at least.

Have a successful CI running in a fraction of a fraction of this repo's time by using GH Actions. Will convert this one using the same strategy.