search

microsoft / mwt-ds

Umbrella repository for projects related to the MWT Decision Service
187 stars 77 forks source link

pretty print of vw audit #129

Closed gupchup closed 6 years ago

gupchup commented 6 years ago

The main reason to do this was that I found we needed to use audit repeatedly to construct a map of the namespace:features -> hashes -> values. I found that printing it out in a tabular format helped in speeding up the process of debugging the model. Hopefully this benefits others too.

JohnLangford commented 6 years ago

Since this is VW-specific, it seems better to merge it into the VW repository, right?

marco-rossi29 commented 6 years ago

Yes, that’s a good point. I’ll take a look at all VW specific code and move it to VW repo

Marco

From: John notifications@github.com Sent: Tuesday, November 6, 2018 4:07 PM To: Microsoft/mwt-ds mwt-ds@noreply.github.com Cc: Marco Rossi Marco.Rossi@microsoft.com; State change state_change@noreply.github.com Subject: Re: [Microsoft/mwt-ds] pretty print of vw audit (#129)

Since this is VW-specific, it seems better to merge it into the VW repository, right?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fmwt-ds%2Fpull%2F129%23issuecomment-436408523&data=02%7C01%7Cmarco.rossi%40microsoft.com%7C2289d4eda7a74d61c60308d6442bc3c9%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636771352129228924&sdata=x8UZx3OkHUzmMQFYu%2FjoFB3DUBqmD8qHXz%2Fii7BkoDY%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMatYG5BkNcrscPYPybJKrKQ-IbFPeCPks5usfnmgaJpZM4YMSAr&data=02%7C01%7Cmarco.rossi%40microsoft.com%7C2289d4eda7a74d61c60308d6442bc3c9%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636771352129228924&sdata=bKjkr3p4JHVa1gxH8TUd8UsXogc6SH5UhbvBQ3ks%2FYc%3D&reserved=0.

JohnLangford commented 6 years ago

This seems like it could easily be in the utl directory of VW.

-John

On Tue, Nov 6, 2018 at 4:09 PM Marco Rossi notifications@github.com wrote:

Yes, that’s a good point. I’ll take a look at all VW specific code and move it to VW repo

Marco

From: John notifications@github.com Sent: Tuesday, November 6, 2018 4:07 PM To: Microsoft/mwt-ds mwt-ds@noreply.github.com Cc: Marco Rossi Marco.Rossi@microsoft.com; State change < state_change@noreply.github.com> Subject: Re: [Microsoft/mwt-ds] pretty print of vw audit (#129)

Since this is VW-specific, it seems better to merge it into the VW repository, right?

— You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub< https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fmwt-ds%2Fpull%2F129%23issuecomment-436408523&data=02%7C01%7Cmarco.rossi%40microsoft.com%7C2289d4eda7a74d61c60308d6442bc3c9%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636771352129228924&sdata=x8UZx3OkHUzmMQFYu%2FjoFB3DUBqmD8qHXz%2Fii7BkoDY%3D&reserved=0>, or mute the thread< https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAMatYG5BkNcrscPYPybJKrKQ-IbFPeCPks5usfnmgaJpZM4YMSAr&data=02%7C01%7Cmarco.rossi%40microsoft.com%7C2289d4eda7a74d61c60308d6442bc3c9%7C72f988bf86f141af91ab2d7cd011db47%7C1%7C0%7C636771352129228924&sdata=bKjkr3p4JHVa1gxH8TUd8UsXogc6SH5UhbvBQ3ks%2FYc%3D&reserved=0>.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/Microsoft/mwt-ds/pull/129#issuecomment-436409436, or mute the thread https://github.com/notifications/unsubscribe-auth/AAE25m0z2UWk63sbkPNNSFw0q_eZ4pAhks5usfqhgaJpZM4YMSAr .