Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

main_images inheritance issue #4

Open
ajitjain opened this issue Dec 12, 2014 · 5 comments
Open

main_images inheritance issue #4

ajitjain opened this issue Dec 12, 2014 · 5 comments
Assignees
Labels

Comments

@ajitjain
Copy link
Contributor

in twitter_microblog and twitter_search_results wrappers, using main_images for the image(s) contained within a tweet doesn't work any longer. as i recall, it did work at some point of time.

the issue persists with local instance of service as well. seemingly, an inheritance issue.

@andru1d
Copy link
Contributor

andru1d commented Dec 12, 2014

ajit -- please give an example url for the problem and its MICE link.

On Thu, Dec 11, 2014 at 6:33 PM, Ajit Jain notifications@github.com wrote:

in twitter_microblog and twitter_search_results wrappers, using
main_images for the image(s) contained within a tweet doesn't work any
longer. as i recall, it did work at some point of time.

the issue persists with local instance of service as well. seemingly, an
inheritance issue.


Reply to this email directly or view it on GitHub
#4.

andruid kerne, ph.d.
director, interface ecology lab
associate professor, department of computer science and engineering
texas a&m university 979.862.3684 fax
college station, tx 77843-3112 http://ecologylab.net

http://facebook.com/ecologylab

Interfaces are the multidimensional border zones through which the
interdependent relationships of people, activities, codes, components,
and systems are constituted. Interface ecology investigates the
dynamic interactions of media, cultures, and disciplines that
flow through interfaces.

@ajitjain
Copy link
Contributor Author

e.g url: https://twitter.com/ajitjain85
mice link: http://ecologylab.net/research/bigsemantics/MICE/index.html?url=https://twitter.com/ajitjain85

images like:
https://pbs.twimg.com/media/BmQWuh0IYAA3QD2.png:large
https://pbs.twimg.com/media/BolJ9k0IgAAgG_R.jpg:large
used to appear with the metadata.

xpaths that are required in service response of the 2 wrappers:
twitter_microblog:
collection name="main_images" child_type="image" hide_label="true"
xpath:.//div[@Class='TwitterPhoto-container']//img[@Class='TwitterPhoto-mediaSource']
scalar name="location"
xpath:./@src

twitter_search_results:
collection name="main_images" child_type="image" hide_label="true"
xpath:.//div[@Class=' is-preview']/img
scalar name="location"
xpath:./@src

@ajitjain
Copy link
Contributor Author

i think i didn't update the BigSemanticsWrapperRepository reference.
so, the changes were in master but do not appear until the branch gets checked out.

after confirming with the service response tomorrow, i'll close this.
i'll also see if any changes are required in the js code, as images didn't appear with local service.

@andru1d
Copy link
Contributor

andru1d commented Dec 12, 2014

are you saying that the meta-metadata is still correct for main_images?

On Thu, Dec 11, 2014 at 7:52 PM, Ajit Jain notifications@github.com wrote:

i think i didn't update the BigSemanticsWrapperRepository reference.
so, the changes were in master but do not appear until the branch gets
checked out.

after confirming with the service response tomorrow, i'll close this.
i'll also see if any changes required in js code.


Reply to this email directly or view it on GitHub
#4 (comment)
.

andruid kerne, ph.d.
director, interface ecology lab
associate professor, department of computer science and engineering
texas a&m university 979.862.3684 fax
college station, tx 77843-3112 http://ecologylab.net

http://facebook.com/ecologylab

Interfaces are the multidimensional border zones through which the
interdependent relationships of people, activities, codes, components,
and systems are constituted. Interface ecology investigates the
dynamic interactions of media, cultures, and disciplines that
flow through interfaces.

@ajitjain
Copy link
Contributor Author

yeah, as i wasn't on master, i didn't get those initially.
i had just looked on github that the wrappers contain the xpaths and not the files in my own environment, which were what the last BSWrappers submodule reference in BigSemantics was pointing to.

after switching to master / updating ref, i got the right meta-metadata. i expect when service gets updated tomorrow, it will also have the same.

@quyin quyin added the bug label Mar 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants