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.

Sign up for GitHub

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

Jump to bottom

Verification result is fixed. #30

Closed

XiaoTonyLuo opened this issue Apr 27, 2023 · 1 comment

XiaoTonyLuo commented Apr 27, 2023 •

edited

HeCBench/xsbench-cuda/io.cu

Lines 72 to 81 in 5abb471

    
           if( in.simulation_method == EVENT_BASED ) 
        
           { 
        
             small = 945990; 
        
             large = 952131; 
        
           } 
        
           else if( in.simulation_method == HISTORY_BASED ) 
        
           { 
        
             small = 941535; 
        
             large = 954318;  
        
           }

The result is fixed with the default input parameter, if num of lookups changes, it will fail here, but the calculation result is correct.

zjin-lcf mentioned this issue

verification result is fixed ANL-CESAR/XSBench#27

Closed

zjin-lcf pushed a commit that referenced this issue


          [xsbench] add host simulation to address the issue (#30)

ebe7d62

zjin-lcf closed this as completed

Owner

zjin-lcf commented May 23, 2024

Running the simulation on a host for verification may fix the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment